[Windows] Middleman requires rubyinstaller devkit

This is a request to add the requirement for rubyinstaller-devkit to be added to the readme. The dependency JSON requires the devkit to be installed in order to run. Without it middleman init will fail. Could you add this to the readme? I would do it myself, but I'm quite new to Github and don't know exactly how I would go about doing it.

Click the "edit" button when you are looking at the README.md


New character sheet for Call Of Cthulhu 6th Edition

"Translatable" character sheet for Call Of Cthulhu 6th Edition

Just for future reference, you don't need to generate the "translations" folder and contents. (unless you are including an alternate translation to start with) Our translation system will automatically create the folder and all of the language files based off the "translation.json" file in the root of the sheet directory. If you are going to include an alternate translation, you would only need to include that one file in the translations folder. Thank you for using the translation tools!


Improve websocket rate limits - Closes #722

What was the problem?
This PR resolves #722
How was it solved?

[x] Implement websocket rate limits (rate-limiter-fiexible)
[x] Configure WS rate limit

How was it tested?
Local + Jenkins

@nagdahimanshu Please make sure to update the respective docker environment files as well.


Ensure all files have CODEOWNERS

Problem
Considering how the monorepo is a complex mix of projects, languages and stacks, we cannot pick from the pool of all maintainers to review PRs. Hence the CODEOWNERS file is a complex Venn diagram of teams related to each part of the codebase.
In this complexity, it is easy for files and directories to be left with no owner at all.
Description

https://github.com/beaugunderson/codeowners is a tool to interact with the CODEOWNERS file. It can be used to find files that have no owner. Running it as a script in pre-commit can fix this issue.
Integrating it with pre-commit can help prevent these situations from being committed at all, but that is blocked on upstream issue https://github.com/beaugunderson/codeowners/issues/36 to be resolved.

Closing as a duplicate of #3554 which has a better solution and a contributor working on it.


Add pricing selection in DeployRequest and MigrationRequest

[ ] add a new mandatory BillingMode field to deploy and migration request.
[ ] Validate if the billingMode provided is supported by the service template.
[ ] If not supported then return BillingModeNotSupported exception.

let me do it


Adding position absolute onto the map container gets overwritten

Whenever I try to pass any position attribute into the styling for the container element, it gets overwritten by position: relative

try adding !important after absolute

I'm having the same issue, the mapElement div has a position: relative and the map does not show up on my page
And override with !important is not possible in React

@ASpehler would it possible to put it in containerElement with v6.0.0 and still work? I'm asking because mapElement is the actual DOM that Google Maps JavaScript API will render its DOMs on, and containerElement is just a wrapper around it.

@tomchentw yes it works too
containerElement={

}
mapElement={

}

Thanks @ASpehler for the feedback!
@nodegin @deadlocked247 did this solve your issue?

Please refer to Getting Help section in the README (or #469).


Shared file system backend: don't localize input files

"localizing input files" means to copy/link task input files from their original location into a TES-managed job execution directory. This is similar to Cromwell behavior. Probably, this is done in order to simplify the process of mounting the input files into a Docker container – instead of mounting each file individually, you can just mount one directory containing all inputs.
Localizing files adds complexity to the code implementing TES, and possibly introduces some unwanted behavior, and we ( @adamstruck  and I) were debating whether it's necessary.
One example of unwanted behavior that I have run into in Cromwell is that if the execution directory and the original file live on different physical drives (e.g. the execution directory is an NFS mount, and the original file is on a local drive), then the input file cannot be hard linked and must be copied. This added some overhead to setting up jobs under Cromwell and is one of those non-obvious gotchas that could easily be missed and cause some degraded performance.
One important edge case to consider is: what if the input file is not on the shared file system? If the file isn't localized, then the file won't be correctly shared with the worker and the job would fail. Some configuration either in TES or in the client-side library could probably do some sanity checking to prevent this, by ensuring all input files are a child of the shared file system.
Originally:
https://github.com/bmeg/task-execution-server/issues/27

One important edge case to consider is: what if the input file is not on the shared file system? If the file isn't localized, then the file won't be correctly shared with the worker and the job would fail. Some configuration either in TES or in the client-side library could probably do some sanity checking to prevent this, by ensuring all input files are a child of the shared file system.

This doesn't apply anymore, since Funnel is able to download/upload files from non-local storage systems (e.g. S3)

Related to #36.   I think this issue has been addressed.  As of #82, we mount in each input in read-only mode into the container.   Currently, we attempt to hard link all inputs into the task work dir prior to mounting. If the hard link fails we fall back to copy.
The alternative would be to skip hardlinking/copying and mount the local files directly.


Order issue: module declared after it's needed

Example:
var useIt = index.foo || index; // both undefined

var index = createCommonjsModule(function (module, exports) {
  // ...
});

Im not sure if it's because of plugin or redis package. Can it be fixed w/o manual copy-pasta every time?
How to replicate:

install redis npm package
use the following Rollup plugins:

plugins: [
  json(),
  resolve({
    jsnext: true,
    module: true,
    main: true
  }),
  commonjs({
    sourceMap: false,
    include: 'node_modules/**',
    extensions: ['.js'],

    // Node.js core modules
    ignore: [
      'crypto', 'string_decoder', 'util', 'events', 'url',
      'tty', 'fs', 'net', 'tls', 'stream', 'assert'
    ]
  }),
],

entry file:

import {createClient} from 'redis';
// or
import redis from 'redis';

How to manually fix:
Just copy-paste whole var index = createCommonjsModule(/* ... */); before it's needed.

I'm having the same issue.
As part of a bigger project, I'm using this plugin to convert the xmlbuilder package to ES6 modules. Within the package, XMLElement requires XMLNode (see here), but in the output, XMLElement is declared first, causing the dependency on XMLNode to return undefined, which in turn throws an error, crashing the whole bundle.

Large old CommonJS projects are rife with gnarly circular dependencies and other things that just don't translate well to ESM. redis is probably one such example. The workaround is fairly straightforward — don't bundle it:
// rollup.config.js
export default {
  external: ['redis'],
  ...
};

As for the xmlbuilder case, you probably just need to change the order of some require/import statements somewhere. The hard part is knowing which ones! (It is often very counterintuitive — one of the unfortunate consequences of declarative import over imperative require.)

@Rich-Harris Sorry for posting a comment in this archived session.
However, I met the same issue when trying to migrate from webpack to rollup.
Concretely, we were using webpack dllPlugin to bundle some npm modules (e.g. cerebraljs) into a single broswer_dll.bundle.js. But now we met the module declared after it's needed issue when bundling cerebraljs. It seems that I need to mark cerebral as external as you suggested but I have no idea how should I tell rollup where to find cerebral when hitting import cerebral from 'cerebral'. Should I using a


Fixed typos in TFLite metadata_writer_tutorial.ipynb

Several typos are fixed in TFLite metadata_writer_tutorial.ipynb

Hi @talumbau Can you please review this PR ? Thank you!

Hi @talumbau Can you please review this PR ? Thank you!

Hi @talumbau Can you please review this PR ? Thank you!


Beam search k-steps limiting output.

I have a trained model and tested on 1000 lines en-fr corpus, the beam search sample just fed me back less than 20 translations. In the line 60 of sample.py: https://github.com/lisa-groundhog/GroundHog/blob/master/experiments/nmt/sample.py#L60
for k in range(3 * len(seq)):

Is there a specific reason we need to use this number of steps? Since I think it's not large enough for my model to generate results end with enc_dec.state['null_sym_target']. And in my understanding the early truncation of beam search process does not make sense? - When do we stop it? Or just because my trained model is bad (so can't get correct output in infinite steps).

The limitation that a translation should be no more than three times longer than the source sentence seems reasonable. It helps to cut to infinite branches of beam search in which it is not going to generate end-of-sentence token at all. If your model does not generate this token in 3 * len(seq) steps, it is a not really well-trained model. then.


Github codes mismatch with local codes

Hi I'm trying to implement a subclass of Model while after copying source from github, I'm keeping getting this issue:
TypeError: _standardize_user_data() got an unexpected keyword argument 'check_batch_axis'
After taking me quite a long time to find errors, I checked my local source codes and find the my local version doesn't support 'check_batch_axis':
 def _standardize_user_data(self, x, y,                                sample_weight=None, class_weight=None,                                check_batch_dim=True, batch_size=None):
And this is the github code:
 def _standardize_user_data(self, x, y,                                sample_weight=None, class_weight=None,                                check_batch_axis=True, batch_size=None):
So could someone tell me how to keep my Keras updated? I'm using ubuntu 16.

Pls delete this one... I think I'm drunk....


POSIX: Sim environment makes assumptions about startup order

@molecule @mcharleb Based on my testing the sim environment seems to make assumptions about startup order to make the network connections. I need to create a reproducible test case, but my current impression is that I need to fix jMAVSim to constantly send something to the UDP port of the simulator, or to fix the simulator to start up immediately. It takes 30 seconds until it boots for me, so I'm not 100% clear yet what the conditions are.

@LorenzMeier The simulator waits for a first message of jmavsim before it starts.


Bugfix: an endpoint without a parent contaminated with its predecessor's parent.

The /another-login/ endpoint only worked correctly, because it was preceded by a pattern with an empty parent. Had the previous parent been not empty (as in the amended test), the last pattern would've been rendered as /router/another-login/, because of the parent_pattern variable reuse in the fixed loop.

@rczajka you patch is too complex, introduces several features. I prefer to have one feature per PR. It seems to be part #125 , part #117 . Could you confirm?


Timer sometimes freezes

You click on the tomato in order to start timer but it stays at the same minutes remaining forever until prompted again

I have seen this problem a couple of times. I think it might be related to #163.
It would be good to get more information on this, if possible. If you hit this issue again, can you check the logs and send me anything you see there (errors, debug information, ...)? See this short video on how you can access the extension's logs.
I will look into fixing #163 as well since it might be related to the issue you're seeing.

i just found it out,  if you close the browser , the timer freezes until
restart
On Feb 19, 2018 8:51 PM, "Chris Schmich"<EMAIL_ADDRESS>wrote:

I have seen this problem a couple of times. I think it might be related to
#163 https://github.com/schmich/marinara/issues/163.
It would be good to get more information on this, if possible. If you hit
this issue again, can you check the logs and send me anything you see there
(errors, debug information, ...)? See this short video
https://i.imgur.com/2ivV7b4.gifv on how you can access the extension's
logs.
I will look into fixing #163
https://github.com/schmich/marinara/issues/163 as well since it might
be related to the issue you're seeing.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/schmich/marinara/issues/170#issuecomment-366788282,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Af2SgX4WYMtHoLnhNuE2nFqMpdW6kDZFks5tWdCsgaJpZM4SJ4PA
.

Thanks for the details. Yes, that is exactly behavior as #163.
It's an unfortunate problem. The timer runs inside of Chrome, so it relies on Chrome being open. There might be some way to keep the timer running in the background. I will have to investigate further.
For now, the workaround is to keep at least one Chrome window open.

what i can think of is caching the timer parameters in some Temp folder in
chrome , thus removing the freezing problem, but you still have to have
chrome open , i think it is possible to have push notifications from an app
in chrome even when closed , you should research that
On Feb 19, 2018 11:12 PM, "Chris Schmich"<EMAIL_ADDRESS>wrote:

Thanks for the details. Yes, that is exactly behavior as #163
https://github.com/schmich/marinara/issues/163.
It's an unfortunate problem. The timer runs inside of Chrome, so it relies
on Chrome being open. There might be some way to keep the timer running
in the background. I will have to investigate further.
For now, the workaround is to keep at least one Chrome window open.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/schmich/marinara/issues/170#issuecomment-366814856,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Af2SgW0ayVwQtY3z_fcBHyvOKMIYBgZ4ks5tWfHbgaJpZM4SJ4PA
.


Time for a new release?

Should have probably incremented as we ticked off items from #26. Opinions on skipping a few minor versions and going to 0.9?

Or, alternatively, can just merge specific commits from develop into master and tag those? Going by number of significant changes, we'll be up to 0.7

Yes we can pick some of develop into master instead, and tag. Not able to do much atm as work is eating me up.

No worries, I'm happy taking care of it :+1:


I thought I already fixed  this

This isn't the issue, the issue is the plugin is still displaying album artwork for Unknown songs, so we need something that returns the message without album artwork if the artist/song is Unknown.

@matthewhesketh Yeah, but it will always return anything, so you can't really fix it i think

How about changing it to something like:
if jdat.results[1].artworkUrl100 then
		if string.match(output, 'Unknown') then
				functions.send_reply(msg, output)
				return
		else
				local artworkUrl100 = jdat.results[1].artworkUrl100:gsub('/100x100bb.jpg', '/10000x10000bb.jpg')
				telegram_api.sendChatAction{ chat_id = msg.chat.id, action = 'upload_photo' }
				functions.send_photo(msg.chat.id, functions.download_to_file(artworkUrl100), output, msg.message_id)
				return
else

etc.

(Updated comment)

@matthewhesketh Looks fine.

I would do it myself but I'm away from home at this point in time so I'll refrain from doing it.

Oh hi Florian xD

@matthewhesketh Upon request I could do that for you.

Isn't it iTunes that is giving Unkown instead of last.fm?

iTunes is returning the Unknown album artwork, if the song is Unknown I just want a text reply.

The unknown reply is typically returned when a user sends /np to mattata and hasn't played anything.

Does last.fm give unified unkown strings back, or is it like
Unkown
Unkown Artist
unkown
etc

Unified

It's always Unknown - Unknown

I'll check it out when i'm done with my english stuff

Which is pretty useful lol

Alright cheers dude

@joostlek It returns Unknown, random messages wouldn't make sense.
www.last.fm/api/show/track.getInfo

There are more things that doesnt make sense

WELP. This is an old issue, which was resolved a while ago (iirc). I'm gonna go ahead and close it now. I'm pretty sure the only reason I didn't close it sooner was to try and sabotage @joostlek's chances of getting 4 PRs for Hacktoberfest lmaoooo


Delete a resource instance

$ curl http://malmo.lizenn.net:8080/collections/blog/ -i -v
...
HTTP/1.1 200 OK
...
x-occi-location: http://malmo.lizenn.net:8080/collections/blog/68943bbd-61f1-336a-9e10-c76a29d299f2
$ curl -i -v -X DELETE http://malmo.lizenn.net:8080/collections/blog/68943bbd-61f1-336a-9e10-c76a29d299f2
...
HTTP/1.1 404 Not Found
...
This curl command would delete the resource.

I confirm that this issue is resolved but a new issue was opened https://github.com/erocci/erocci/issues/58


add roachprod install x:y charybdefs

See https://github.com/cockroachdb/cockroach/issues/20337.
Required for https://github.com/cockroachdb/cockroach/pull/31187.

This change is

I forgot to land this before I landed the PR that needs it.


[BUG] update tutorials to properly wait ready

The getting started tutorials do not properly await provider readiness. All tutorials should be updated to either wait for a READY event, or use a blocking provider mutator (ie: OpenFeature.setProviderAndWait(provider).
Relates to: https://github.com/open-feature/go-sdk/issues/238

tackling this change.


How to refresh the AppSync API key automatically?

Assume you have the following schema
type Todos @model
  @auth(rules: [
    { allow: public, operations: [read], provider: apiKey }
    ])
{
  id: ID!
  text: String!
}

To query it, you'll need a AppSync API key.
But how to refresh an API key?
For a Cognito user pools type of auth, Auth.currentSession()  do the job for you.
But can I refresh a public AppSync API key when using allow: public?

@yaquawa If you'd want to update/refresh/rotate the API-Key, you'd need to go through these steps with the Amplify CLI - https://docs.amplify.aws/cli/graphql-transformer/config-params#createapikey

You mean I have to rotate them manually?!

Why is this closed?


⚠ Bofrost.fr has degraded performance

In 67a6d77, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2321 ms

Resolved: Bofrost.fr performance has improved in e68899e.


Support generating package-visible TableModels to encapsulate DB implementation from other java packages

Sorry, I don't think I quite understand what the feature request is here. Can you provide an example of what you want to generate and a brief explanation of why it would be useful?

If I understand this correctly, you'd like to be able to generate model classes that are declraed class MyModel and not public class MyModel?

@jdkoren exactly!

@gallal I threw together a branch that implements this enhancement, but I think we need a little more convincing that this use case is worthwhile enough to warrant the extra complexity :) A package-private model class would have at least the following limitations:

It would have to be declared in the same package as the database class it belongs to, in order to be able to provide the database with its Table object. (Or you could implement some hack by which another class in the package provides Tables to the DB class, but that sounds super messy).
Database writes using that model class could only occur within that package.
You could only construct queries/do DB reads on that table from within that package. Even if you passed a cursor to a class in some other package, you'd lose all the ability to read type-safely from that cursor, because the model's Property objects wouldn't be visible.

To me, these limitations sound like they would be a burden that limits good code organization and package structure, so I'd personally be inclined to keep all model objects public. If you are reading this though and thinking "yes, that's exactly what I want" then it would be great to hear more about the general use cases you have in mind where this kind of data hiding would be helpful -- I'm certainly open to merging the change if you can convince us it can be used for good!

This issue hasn't seen any activity in a while so I'm going to close it. I'm still of the opinion that package private models don't have very many legitimate use cases given the limitations I outlined above. If anyone wants to try to convince me otherwise though, feel free to reopen the issue.


Make TLS/https optional

As mentioned in the post-close comments of this TLS error issue, providing optional plain HTTP would allow pio deploy to be supported on proxied/routed app platforms like Heroku.
I've been working on a PIO buildpack for Heroku using the current PredictionIO-0.9.6 release. Once the engine deploys & starts successfully, the Heroku router cannot negotiate a HTTP connection with the PIO engine.
Plain HTTP would also allow generic authentication proxies for the wide-open service provided by pio deploy.

Please bring this to<EMAIL_ADDRESS>(http://predictionio.incubator.apache.org/support/). GitHub issues is no longer supported after ASF migration and will eventually shutdown. Thanks!


Add _GNU_SOURCE to s2n_certificate.c

Need this for platforms with glibc < 2.10
Issue # (if available):
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

:exclamation: No coverage uploaded for pull request head (gnu_source@52dccf8). Click here to learn what that means.
The diff coverage is n/a.


Mask Inversion

It would be great if it were possible to quickly and easily invert a mask in img2img.
Either in the image editor to be able to right click and invert the mask, or on prompt level which causes an inversion node to be added.

Note OP was originally also asking about <segment: for the same request

For the first part, within the Image Editor, there's now an Invert button when you right-click a layer

You can now do negative thresholds to invert segment masks


OSError: results_gen_sym.dat not found

Hello! I recently watched your wonderful presentation of the code at Oxford university and I am very interested in using it in my own work.  I have successfully installed the code. However, when I run the test command
import aifeynman
aifeynman.get_demos("example_data")
aifeynman.run_aifeynman("./example_data/", "example1.txt", 60, "14ops.txt", polyfit_deg=3, NN_epochs=500)

to verify that the code is working I get the following error message:
Trying to solve results/gradients_gen_sym_example1.txt_train
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/zcapjru/feyn/lib/python3.8/site-packages/aifeynman/S_run_aifeynman.py", line 274, in run_aifeynman
    PA = run_AI_all(pathdir,filename+"_train",BF_try_time,BF_ops_file_type, polyfit_deg, NN_epochs, PA=PA)
  File "/home/zcapjru/feyn/lib/python3.8/site-packages/aifeynman/S_run_aifeynman.py", line 156, in run_AI_all
    bf_all_output = np.loadtxt("results_gen_sym.dat", dtype="str")
  File "/home/zcapjru/feyn/lib/python3.8/site-packages/numpy/lib/npyio.py", line 961, in loadtxt
    fh = np.lib._datasource.open(fname, 'rt', encoding=encoding)
  File "/home/zcapjru/feyn/lib/python3.8/site-packages/numpy/lib/_datasource.py", line 195, in open
    return ds.open(path, mode, encoding=encoding, newline=newline)
  File "/home/zcapjru/feyn/lib/python3.8/site-packages/numpy/lib/_datasource.py", line 535, in open
    raise IOError("%s not found." % path)
OSError: results_gen_sym.dat not found.

This seems to suggest that there’s a missing file. Has someone run this code since the last update of the repository?
Thank you so much in advance and looking forward to using the code,
Jaime

Hi, I have the same issue and no idea how to solve it...
Thanks and best wishes,
Andreas

Hi, count me in. I have the same error.
Some ideas?

Hi, count me in. I have the same error.
Some ideas?

Same error here...maybe a results file was supposed to be generated but wasn't,,,?

Same error here...maybe a results file was supposed to be generated but wasn't,,,?

I had the same error.
I reset the repo to commit  3629ffe337f28bfba7f0dead54be63809a212d17 and its working fine.
I suspect there is something going on in PR #22 because i saw the file being mentioned in the code but I don't know what exactly is causing this behaviour.

@florianthonig Thanks for the hint!
Did you do something like this:
cd AI-Feynman
git reset --hard 3629ffe337f28bfba7f0dead54be63809a212d17

which gives:
HEAD is now at 3629ffe Merge pull request #15 from andrewtsoc/feynman-package

Or am I missing something?

@florianthonig Thanks for the hint!
Did you do something like this:
cd AI-Feynman
git reset --hard 3629ffe337f28bfba7f0dead54be63809a212d17

which gives:
HEAD is now at 3629ffe Merge pull request #15 from andrewtsoc/feynman-package

Or am I missing something?

Yes, thats what I have done

Yes...thanks Florian!
I created a local repository and used git reset to get back to your suggested commit.
But after that, unfortunately the sample problems wouldn't run as it did not recognize the aifeynman commands.
Did anyone experience that subsequent issue?

Yes...thanks Florian!
I created a local repository and used git reset to get back to your suggested commit.
But after that, unfortunately the sample problems wouldn't run as it did not recognize the aifeynman commands.
Did anyone experience that subsequent issue?

Yes in the older version the module name is just feynman.
Take a look at the example in the project.

So I'm noticing that the AI Feynman repository has been updated on Dec. 6 2020. I keep getting the OSE error that is the topic of this string of comments. I'm wondering what the current fix is for this issue. I'd appreciate any help. Thanks for your time.

well, this shouldn't be considered as a proper solution, I'm still facing the same problem and the git stash hasn't solved it.
Can we ping the author?

Late follow up but I had this error when pip installing aifeynman. It resolved itself when I downloaded 2.0.7 from pypi and installed locally in editable mode. Don't know why but may be to do with compiler issues that aren't addressed when installing directly via pip.

A very late reply, but I was unable to recreate your issue (i.e. results_gen_sym.dat was created successfully).
The file should be created upon the call
subprocess.call(["feynman_sr_mdl5"], timeout=try_time)
on line 60 of S_brute_force_gen_sym.py since this runs the fortran routine symbolic_regress_mdl5.f90. The file you want is opened on line 133 of that routine and printing happens on line 185. Given that you've called this with a timeout (and try_time = BF_try_time), I wonder whether you were timed out before you got to line 133 or 185. Since it's within a try-except, the TimeoutExpired exception won't be noticed so you're code carries on and a file is never made.
There could be another reason, but I wonder if you increase BF_try_time (the example suggests 60 but that could be increased) it will work. Alternatively, If you remove the try-except it may be clearer why the fortran routine goes wrong.

In my case it was an OS error that I fixed following this issue.

In my case it was an OS error that I fixed following this issue.

How did you fix the problem？


#1285 - Added missing package export

@rkosegi could you please send this pull request to the develop_2.0 branch?

@fehguy done, I think issue can be closed.
Thanks.

thank you


Pulls/master/silverstripe early2020

DON'T MERGE
Will do an actual PR to upstream repo FriendsOfPHP/security-advisories
Update:
PR to FriendsOfPHP https://github.com/FriendsOfPHP/security-advisories/pull/444

Closing this issue as the actual PR is to FriendsOfPHP - link in description


Roles based access is broken

Hi m8,
"New" Jhipster generator uses authorities in stead of roles property key in data attribute of state config:
data: {
                **_authorities_**: ['ROLE_ADMIN'],
                pageTitle: 'Entity Audits'
            },

ill change this

if you like you can do a PR


subsys: testsuite: coverage: allow access to gcov internals

This allows external applications wishing to report coverage data over different interfaces the ability to do so by exposing the gcov list. Follow on commits will test it in the shell, when the PR lands and also add something to choose the output mechanism via Kconfig.

This allows external applications wishing to report coverage data over different interfaces the ability to do so by exposing the gcov list.

can you please elaborate what those external applications are and how this will be used?

This allows external applications wishing to report coverage data over different interfaces the ability to do so by exposing the gcov list.

can you please elaborate what those external applications are and how this will be used?

Right now the only way to get the coverage data out is via the printks in the coverage dump function. However, reading through the coverage file it appears as if at one time there was a desire to allow that data to be dumped to other interfaces. This is my intention here. By exposing the gcov list we can write surrounding code around getting the gods file data out and allow choices via Kconfig on how to report that data. I have something internally that I have put together that does this which I plan to start upstreaming in chunks once I can get this change which I need to get it fully working and cleaned up. My use case specifically involves structuring the data in length and values as opposed to printk to uart and then that data is dmaed to a host. So allowing that type of customization is my overarching end goal. This can also streamline the process a little since it won't dictate to users that to obtain coverage data it has to be written out as ascii hexadecimal values and then post processed via a script using magic character sequences.

Right now the only way to get the coverage data out is via the printks in the coverage dump function. However, reading through the coverage file it appears as if at one time there was a desire to allow that data to be dumped to other interfaces. This is my intention here. By exposing the gcov list we can write surrounding code around getting the gods file data out and allow choices via Kconfig on how to report that data. I have something internally that I have put together that does this which I plan to start upstreaming in chunks once I can get this change which I need to get it fully working and cleaned up. My use case specifically involves structuring the data in length and values as opposed to printk to uart and then that data is dmaed to a host. So allowing that type of customization is my overarching end goal. This can also streamline the process a little since it won't dictate to users that to obtain coverage data it has to be written out as ascii hexadecimal values and then post processed via a script using magic character sequences.

ok, sounds good, been looking for something like this and was considering using gdb to as a way to get coverage data (see for example https://www.thanassis.space/coverage.html). I agree the current method is suboptimal. Please upstream your implementation ASAP ;-)

Right now the only way to get the coverage data out is via the printks in the coverage dump function. However, reading through the coverage file it appears as if at one time there was a desire to allow that data to be dumped to other interfaces. This is my intention here. By exposing the gcov list we can write surrounding code around getting the gods file data out and allow choices via Kconfig on how to report that data. I have something internally that I have put together that does this which I plan to start upstreaming in chunks once I can get this change which I need to get it fully working and cleaned up. My use case specifically involves structuring the data in length and values as opposed to printk to uart and then that data is dmaed to a host. So allowing that type of customization is my overarching end goal. This can also streamline the process a little since it won't dictate to users that to obtain coverage data it has to be written out as ascii hexadecimal values and then post processed via a script using magic character sequences.

ok, sounds good, been looking for something like this and was considering using gdb to as a way to get coverage data (see for example https://www.thanassis.space/coverage.html). I agree the current method is suboptimal. Please upstream your implementation ASAP ;-)

The article you linked is an interesting read. I had not thought of doing it that way. Maybe I can add that as an additional choice in my implementation.


[Question] How are the plugins intended to be installed?

I use scoop to manage my packages and I was wondering how the plugins are supposed to be installed.
Am I to clone the source, compile and drag en drop the generated dlls I want to use?

@floppydisken I think you might have run into the same problem I ran to.
The documentation available at https://workspacer.org/config/ does not match with the stable version you may have gotten via scoop (v. 0.9.10). If you have that version installed, you won't, for example, not be able to configure the TitleBar plugin described here: https://workspacer.org/config/#title-bar.
If you want you can install an unstable release: https://github.com/workspacer/workspacer/releases (see Assets). Keep in mind that unstable releases may change quickly and may contain unknown bugs.
You can easily define your own plugins in your config if you wish. I did that in my config workspacer.config.csx if you need inspiration. Just keep in mind that no error handling is added and therefore if you copy those plugins they'll most likely break.

I installed workspacer with the [workspacer-stable-0.9.10.msi file.
My plugins directory contains the directories workspacer.ActionMenu, workspacer.Bar and workspacer.FocusIndicator.
How can I add for example the gap plugin?


Deep Learning Race Driver

Hi,
I was wondering if you will be uploading Unity environment and scene for the Deep Learning Race Driver from the following video? I was interested in see the car controlled by the ML-Agents.
https://www.youtube.com/watch?v=gEf9V03HWv0&t=3s

Hi @SachinAbhyankar - I decided against uploading the race driver project for now. For a couple of reasons: The track generation code is quite a mess and needs major refactoring. Also, I hacked my way through Sergey Makeev's Arcade Car Physics in order make it controllable through ml-agents. I love the physics behaviour he created, but Sergey himself claims it's just a preliminary take on the topic, and I'm afraid my hacks didn't make the code more comprehensible. Finally, my raycast detection logic doesn't generalize well to other kinds of tracks yet, see https://github.com/SergeyMakeev/ArcadeCarPhysics/issues/3#issuecomment-528780074

Ok. I can understand. I was interested in the terrain used in this demo. It will be useful to get the Unity environment for this. Also, it will be good to know how to ML-Agent Academy / Agent / Area is setup for this. I can refer to the other demos as well.

You can now find a simplified version of the project at https://github.com/mbaske/ml-simple-driver


Refactor usage of concurrency with reactor

Terasology does not use or promote a consistent feature set or technologies for handling concurrency.
In some situations the number of threads are over-provisioned, and in some cases its under-provisioned. Some workloads might benefit from more threads but selectively changing the number of threads by hand will lead to over-provisioning on some systems and under-provisioning on another system.
Poor utilization of threads will end up with situations where the kernel is switching between different tasks. If every thread is a CPU bound task then the processor will spend a lot of time context switching between the different tasks.
We propose dropping ThreadManagerSubsystem and use the functionality provided by Project Reactor.
Currently, TaskMaster is used to push tasks to an off thread where the use case of TaskMaster is probably not needed and the spawned thread is mostly unused.
Current situation

Chunk-Unloader (allocate 4 threads)

int list to queue event for BeforeDeactivateBlocks
very light load so threads are mostly idle
at any moment a single thread will become active

Chunk-Updater (allocates 8 threads)

generates chunk mesh and queue them back on the main thread

LOD Chunk Generation

only triggered for handling different levels of lod

ThreadManagerSubsystem (allocates 16 threads) only used for a couple situations and the fixed number of threads does not properly utilize the number of cores. and hardly used so all these threads are idle.

loadingGame
create new world
saving screenshot

TaskMaster

Behaviors (1 thread)
FlexiablePathFinding (4 threads)
Pathfinding (1 thread)

[x] drop usage of the ThreadManagerSubsystem easily can be replaced by usage of Observable
[x] chunk Mesh generator https://github.com/MovingBlocks/Terasology/pull/4786
[ ] remove usage of TaskMaster

[x] #4800
[x] #4786
[x] https://github.com/Terasology/Behaviors/pull/77
[ ] https://github.com/Terasology/FlexiblePathfinding/pull/24
[x] ~~https://github.com/Terasology/Pathfinding/pull/62~~ — module removed
[ ] chunk unloader probably does not benefit with parallelism

[ ] check whether Reactor threads need to be explicitly configured as daemons so they don't prevent the process from closing if they get stuck
[ ] LOD Chunk generation uses a custom set of threads easily replaced with logic in this PR (https://github.com/MovingBlocks/Terasology/pull/4786) . we need to do something slightly different because constantly replacing the static mesh is un-optimal and produces a lot of extra traffic between the GPU and CPU.

Do we have anyone familiar enough with the various implementations of the reactive pattern in Java to evaluate Project Reactor (with Flux and Mono APIs) in comparison to RxJava2?

I tried to sketch out what we're currently doing and using (no guarantee for completeness):

@skaldarnar that's right got me confused because ExecutorService just creates a pool of threads and each instance of TaskMaster will introduce its own set of threads. we might get into a situation where the processor is constantly context switching between a lot of computational threads.

Do we have anyone familiar enough with the various implementations of the reactive pattern in Java to evaluate Project Reactor (with Flux and Mono APIs) in comparison to RxJava2?

there are a lot of weird facilities to map one to the other and vice versa. probably want to provide one use case so we don't end up juggling a lot of these reactive libraries.

@pollend I've updated your initial post by adding links to ReactiveX concepts and tweaking the sentences a bit here and there. I hope I did not change the semantic anywhere.

got me confused because ExecutorService just creates a pool of threads and each instance of TaskMaster will introduce its own set of threads

You plan forward sounds good to me on first read. I just wanted to share that picture I made, where we can hopefully remove a few of the green boxes in the future, and replace others with RX concepts.

https://github.com/MovingBlocks/Terasology/pull/4799 @skaldarnar this should correctly expose reactivex to module space. you need to use the GameThread wrapper though :/

I'd like the roadmap to include a plan for replacing the debug thread view; currently ThreadMonitorPanel and the [RunningThreads metrics mode] use ThreadMonitor.
https://github.com/MovingBlocks/Terasology/issues/4637 links to a few options that work with ExecutorServices. At the time, I wasn't checking for integration with RxJava or Project Reactor. They might have something built-in, or a different integration option for these other libraries.

I'd like the roadmap to include a plan for replacing the debug thread view; currently ThreadMonitorPanel and the RunningThreads metrics mode use ThreadMonitor.
#4637 links to a few options that work with ExecutorServices. At the time, I wasn't checking for integration with RxJava or Project Reactor. They might have something built-in, or a different integration option for these other libraries.

hmm, would be a nice way to do some kind of server metrics. processing results from the running tasks?

some main problems with RXJava. There is a table of mappings that define a flowable, Observable, Single, Maybe, and Completable and mappings from one to the other. 0..N defined for Floablw and Observable but only flowable has backpressure. Single is one element only but Maybe is 0 or 1. Completable is 0 element or error. there are different methods to map one type to another. the inconsistent API is difficult to follow. The mappings make this pretty confusing where reactor has Flux and Mono. Flux is 0...N and mono is just a single object. I like this a lot better then the former.
While working with Flowable I managed to get into a situation where objects getting pushed to my Flowable were not getting handled by the subscriber where this pattern seems to be more reliable with reactor. I was looking at this benchmark comparing the two and found the reactor implementation is quite a bit faster: https://github.com/MovingBlocks/Terasology/pull/4786
https://github.com/akarnokd/akarnokd-misc/issues/7

Reactor explicitly has built-in integration with micrometer, which makes for a good default answer to which of the metrics implementations in #4637 we should use.

Reactor explicitly has built-in integration with micrometer, which makes for a good default answer to which of the metrics implementations in #4637 we should use.

me and @DarkWeird were discussing this.

We briefly discussed this in today's org meeting and I quickly updated the graphic to match the current state of things:

I  should refer my pr there  #4865

And i have a several thinks about networking via reactor.
Reactor project have reactor-netty lib:

network subsystem should looks smaller and easier with it.
We can use HttpClient from reactor-netty.
It should make module downloader and ModuleListDownload more faster(?) And non blocking (especially for ModuleListDownloader, which use separate thread via new Thread :( )
We can make refactor of DiscordRPC. (-1 thread)

So my first performance test plan was simply to spawn 42 deer in a CoreGameplay + WildAnimals module combo and look at the FPS...
Good news: With the current FlexiblePathfinding develop, I have 60 FPS
Bad news: With this branch checked out, I have 1.7 FPS - and no, the decimal point in there is no accident...


Update options documentation in readme

Update the documentation for polymorphic_embeds_many/2 to reflect option name change from :type_field to :type_field_name.

Sorry for the inconvenience :-) took a look at the wrong version while looking at the documentation.


README input secrets misleading

secrets in input is typed as a list(map(string))
when it really should be a
list(object({
name                  = string
secret_data           = string
}))

there's also next_rotation_time and rotation_period which you can only know about by looking at the examples and source code.  Would be nice to have this documented anywhere.


Update the test suite

ℹ This issue is part of the Epic #7
To test the latest toggle features in #22 it's required adjusted the current test suite

Closes #23
It should be adjusted for each issue.


Add eslint/prettier/sonarjs

Add eslint, prettier and sonarjs to the codebase.
Add .vscode folder with settings.json and extensions.json (recommended eslint vscode plugin and prettier vscode plugin).
Added some basic eslint rules for formatting (this can be changed and adjusted as needed).
This will result in a lot of errors and warning until the codebase is adjusted and/or some eslint rules are changed.

nothing wrong with a little order i guess


Update test-glossary.md

According to the source code of TestCaseResult, one outcome was missing. 'In progress' is listed there.
In addition the count of the outcomes was incorrect.

#sign-off

@ramiMSFT: I'm sorry - only the author of this article, @vinodjo, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

Is @vinodjo still active?

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


Submitting a build to Coverity Scan

After speaking to Coverity's senior staff engineer Mel, we got the advise to create a new project, as simply renaming a project thats ownership got transfered might not to the trick.
As this is already done, we still have to submit our first builds for analysis. I assume that this can only be done by either @OpusV or myself, as we may need to submit the first one using coverity build tools as told in the admin console, prior to submit builds via Travis CI again.

This is also somehow urgent and should not be pending until the next Sprint (two weeks), as I am not sure if the registration gets auto-deleted, if we wont submit the first job within X days.

I just submitted a build to Coverity Scan. Now we have to wait for the final application review, before we can start looking into the analysis.


Broken links

Hi, I was looking at the new graph feature and saw there are some broken links. Thought I would let you know.
I was on:
https://github.com/netlify/labs/blob/main/features/graph/documentation/README.md
And here is one of the broken links:
[Start a local CLI session with Netlify Dev](https://github.com/netlify/labs/blob/main/features/graph/documentation/authentication.md#start-a-local-cli-session-with-netlify-dev).
Thanks!

These links are currently broken as well -

The "gravity" repo links within the "Example repo and tutorial" section.
All of the links in the "Get started with Netlify Graph" list

Thanks for this feedback. We are archiving this repo now that these docs have moved over to docs.netlify.com. If you have further feedback on Netlify Graph, please fill out our survey. For docs-specific feedback, you can use the feedback form at the bottom of the docs page.


File Upload Vulnerability

I would like to contribute to this repo by explaining about attack mentioned above. Please assign me this issue under JWOC 2023.
Thank You!

@White-rose69 You may start working on this issue

@SANTHOSH17-DOT Is this a dead project that I've been working on??
Because I am getting very late replies now

I'm currently studying at a college. So I have academic and other works as well. I hope you understand it. LOL you've commented just before 6 hours.
I have the same concern that I've shared it for xss


Child fails to complete handshake

What Is the issue?
Getting this in the console, following your basic example:
Uncaught DOMException: Failed to execute 'postMessage' on 'Window': Invalid target origin 'null' in a call to 'postMessage'.

Provide issue context below using code examples, images, or links

Parent instantiation:
import Postmate from "postmate"
Postmate.debug = true

const handshake = new Postmate({
    container: document.getElementById('virtual-adviser-initial-container'),
    url: 'https://virtual-adviser.test/cross-origin-enquiry/',
    classListArray: ["mw-full"]
});

handshake.then(child => {
    child.get('height')
      .then(height => child.frame.style.height = `${height}px`);
    child.on('some-event', data => console.log(data)); // Logs "Hello, World!"
});

Tried using v1.4.2 and ^v1.4.8 with the same issue. Can you shed any light as to why this would be happening? Many thanks 👍🏼

+1


Implement optimized median filter

Easy optimization to improve normalize-by-median: it's simple to observe that checking if the median of a set is greater than some cutoff is equivalent to checking if more than half the elements of that set are greater than some cutoff. The latter avoids doing a lookup for every kmer every time, and avoids a costly sort. On a small dataset (1m ecoli reads), this was an 18% performance improvement. Not bad for 5 minutes!
TODO: Add a couple tests

Cool!

[ ] Is it mergeable?
[ ] Did it pass the tests?
[ ] If it introduces new functionality in scripts/ is it tested?
Check for code coverage with make clean diff-cover
[ ] Is it well formatted? Look at make pep8, make diff_pylint_report,
make cppcheck, and make doc output. Use make format and manual
fixing as needed.
[ ] Did it change the command-line interface? Only additions are allowed
without a major version increment. Changing file formats also requires a
major version number increment.
[ ] Is it documented in the ChangeLog?
http://en.wikipedia.org/wiki/Changelog#Format
[ ] Was a spellchecker run on the source code and documentation after
changes were made?

ready for review @ctb @mr-c

Does this return identical results to old approach?

Yup. All normalize-by-median tests pass with the new function patched in.

jenkins, test this please you useless drunk

jenkins test this please and give it a pass or the floggings will continue

@ctb @mr-c @luizirber okey now pls review

Please leave for me to merge - tnx.

@ctb I keep expecting that I've overlooked something really obvious and there's no way a 20% performance increase would be so easy, but shrug

A few comments --
the code didn't return identical results on data/100k-filtered.fa, due to a round-off error.  See rounding comment on hashtable.cc.
Please add tests for single k-mer, med < cutoff; single k-mer, med > cutoff; odd # k-mers, med < cutoff; odd # of k-mers; med > cutoff; and any other edge cases you can think of (that's all I got).  The homopolymer run tests are not sufficient ;).  You should enforce comparison with the old function, too, to make sure they both return the same results.
can we implement the old function without the sort? it seems fairly straightforward to do so although there would still be a performance hit.  This shouldn't be your job to do, but can you create a few issues around (a) improving old function, and (b) replacing old function with this function in places where it would work?
tnx!

@ctb can do on the extra tests and whatnot.
We definitely can't do the old function without the sort -- the key difference here is that the actual median isn't calculated, only that it's greater than some value. There are definitely more efficient median algorithms if we want to replace the naive sort-based one we're using now thoug

On Mon, Mar 09, 2015 at 01:15:38PM -0700, Camille Scott wrote:

@ctb can do on the extra tests and whatnot.

ok, let me know

We definitely can't do the old function without the sort -- the key difference here is that the actual median isn't calculated, only that it's greater than some value. There are definitely more efficient median algorithms if we want to replace the naive sort-based one we're using now thoug

sure, we should look at the current code and think about where we might
optimize.

Stop being useless Jenkins, and dammit Jenkins, test this please!

Stop being useless Jenkins, and dammit Jenkins, test this please!

Stop being useless Jenkins, and dammit Jenkins, test this please!

@ctb, ready for review

Two questions for consideration cc @mr-c --

should we store the results of normalize-by-median on (say) 100k-filtered.fa, as a test for consistency going forward?

filter_on_median is an unsatisfying function name, since it doesn't actually filter anything. Would 'median_greater_than' or something like that be clearer? Suggestions welcome.

+1 for storing known-good output. How big, when compressed? Worse case we keep it elsewhere and let Jenkins deal with it.

Very well -- I'll add a comparison test to known output of the mentioned file and change the function name.

@ctb @mr-c okay I added the new test -- it adds ~5mb the repo. Is that manageable?

Hrmm.. that would nearly double the distribution size..
On Thu, Apr 2, 2015 at 4:38 AM Camille Scott<EMAIL_ADDRESS>wrote:

@ctb https://github.com/ctb @mr-c https://github.com/mr-c okay I
added the new test -- it adds ~5mb the repo. Is that manageable?
—
Reply to this email directly or view it on GitHub
https://github.com/ged-lab/khmer/pull/862#issuecomment-88684884.

@ctb @mr-c further thoughts on this??

We need a separate repo to store larger pieces of test data and have tests that run only on Jenkins by default. @ctb What do you think?

Would something like git's LFS work?
https://github.com/blog/1986-announcing-git-large-file-storage-lfs

@bocajnotnef In this case the file is 5Megabytes so GitHub LFS wouldn't be needed. I don't want to make the git clone of the main repository larger than necessary. Another option is to add the 5M file but exclude it from packaging and adjust the test to skip if the file isn't present.

On Thu, Apr 09, 2015 at 08:55:01AM -0700, Michael R. Crusoe wrote:

@bocajnotnef In this case the file is 5Megabytes so GitHub LFS wouldn't be needed. I don't want to make the git clone of the main repository larger than necessary. Another option is to add the 5M file but exclude it from packaging and adjust the test to skip if the file isn't present.

What about a 'khmer-data' repository?  Although that seems kind of silly.

On Thu, Apr 09, 2015 at 08:19:19AM -0700, Michael R. Crusoe wrote:

We need a separate repo to store larger pieces of test data and have tests that run only on Jenkins by default. @ctb What do you think?

Since the data is mostly going to be read-only, it seems silly to have a full
repo.  Maybe their new data extension is the way to go for that?  And have a
khmer-data repo that stores the pointers?

How about we get the optimization merged for now, then open an issue for supporting github LFS or some other variety of data hosting to be addressed in the future?

+1
On Mon, Apr 13, 2015 at 6:33 PM Camille Scott<EMAIL_ADDRESS>wrote:

How about we get the optimization merged for now, then open an issue for
supporting github LFS or some other variety of data hosting to be addressed
in the future?
—
Reply to this email directly or view it on GitHub
https://github.com/ged-lab/khmer/pull/862#issuecomment-92518716.

[ ] create an issue
[ ] make mergable

@camillescott - @mr-c is going to address data size problem for test by putting that elsewhere, but other than that, all you need to do is make it mergable, I believe.

It can go into https://github.com/dib-lab/khmer-testdata ; just commit directly

ping @camillescott

@mr-c need access to khmer-testdata repo

jenkins, test this please

There is so much winning in this PR, @camillescott.

Suggest squashing as in #660.

Jenkins, test this please

@ctb ready for final review / merge

also pleeeeeeeeeease merge this before merging any other PR's in, it's all squashed and pretty and I've had to update and tweak little things on this branch way too many times :'(

LGTM; nice work.

Yay, thanks!


Task exec logs lost when indexing is not enabled

Describe the bug
The workers add logs to the TaskResult returned to conductor server which does NOT enable indexing, then task log api (tasks/{}/log)return nothing!
Details
Conductor version:
Persistence implementation: MySQ
Queue implementation:  MySQL
Lock: Zookeeper
Workflow definition:
Task definition:
Event handler definition:
To Reproduce
Steps to reproduce the behavior:

start conductor server with indexing enable=false
poll task and return TaskResult with logs added by calling addLogs
call logs api
not logs added return

Expected behavior
return the logs
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Just read the source code, no idea why discard logs but it just works as code saying.


open the same db in read only mode

Error: Database already open. Cannot acquire lock.
I have one process which does read & write, another process which does read

rocksdb allows to open in readonly mode

See #678

rocksdb read only mode cannot sync with the write, secondary db instance doesn't work well either, if we cannot find a very useful solution, then I think we can abandon this

I believe this is available now since the document clarifies:
https://docs.rs/redb/latest/redb/struct.Database.html
Opened redb database file:
Multiple reads may be performed concurrently, with each other, and with writes.
pub fn begin_read(&self):
Read transactions may exist concurrently with writes

@axww AFAICT what you're quoting applies only within one process. What the OP asked for is multiple processes opening the same database. AFAIU as per https://github.com/cberner/redb/issues/678 this should be possible in the future, but is currently not implemented.


Can't run the BASIC of Basics

I pulled the report, and ran examples/demo_basic.ipynb
I pulled the PDF, added my key and then ran the cell with this inside
from llama_parse import LlamaParse
documents = LlamaParse(result_type="text").load_data("./attention.pdf")
I got this error
Error while parsing the file './attention.pdf': Failed to parse the file: Internal Server Error

I'm seeing the same error for about 20 different pdfs.

Getting errors as well.

@BinaryBrain , things are all good on my end now.

Good to know! Feel free to ping us if you encounter more issues.


change ssid/password

change ssid/password  with random content

solved in https://github.com/valerio-vaccaro/SFYL-Wallet/commit/bcb26501c459b619e3d6cf7a9d2c285b7993f57d


Scroll to top causes everything to render big then shrink small

Summary of the bug
Using scroll to top causes a rendering issue when refreshing the website.
🪜 How to reproduce
On iPhone, scroll to top to trigger a refresh, notice how the arrow gets larger then resizes to its normal size.
🧐 Expected behaviour
Should not flash
📸 Screenshots or code
If you can, add screenshots to show the problem. If you need to show a code example, you can include a snippet or link to a repository with an example.
🖥 Desktop

OS: [e.g. macOS]
Browser: [e.g. chrome, safari]
Browser version: [e.g. 22]
Website version: [e.g. V1, V2]
(As displayed on the site top navigation)

📱 Mobile

Device: iPhone XR
OS: [e.g. iOS8.1]
Browser: Safari
Browser version: [e.g. 22]
Website version: [e.g. V1, V2]
(As displayed on the site top navigation)

Additional info
Tell us anything else useful to help us fix or understand the problem.

Fixed


Occupancy not shown?

The problem
In the screenshots of the add-on, we see occupancy, but in my installation (4 Nest Protects) it is not shown.
What version of this integration (ha-nest-protect) has the issue?
0.3.7
What version of Home Assistant Core has the issue?
core-2022.6.5
Device / Model
Nest Protect (battery)
Diagnostics information
No response
Home Assistant log
No response
Additional information
No response

This integration will add the most important sensors of your Nest Protect device (CO, heat and smoke) and the occupancy if your device is wired (to main power).

Device limitation, as discussed in the README.


4.1.6 Windows release is missing PHP DLL driver files

Release 4.1.5 has DLLs for PHP 7 and PHP 7.1 but the 4.1.6 Windows/Linux release is missing the PHP DLL files for windows
Please add the windows PHP 7 and PHP 7.1 DLLs for the pdo_sqlsrv and sqlsrv drivers to the release

/Ping @Hadis-Fard
When can we expect the 4.1.6 Windows drivers to be posted to the release page?

@photodude You should have it tomorrow :) 👍

@meet-bhagdev Thank you. Looking forward to the update.

@photodude DLLs are now up in our PECL repository, you can download them from:
https://pecl.php.net/package/sqlsrv/<IP_ADDRESS>/windows
https://pecl.php.net/package/pdo_sqlsrv/<IP_ADDRESS>/windows

Thanks, I was hoping for the files to also be included in the release page here, guess I'll change our appveyor code for pulling the dlls.

@Hadis-Fard the PECL files are missing the php_pdo_sqlsrv.dll module, they only have the PHP_sqlsrv.dll module
I checked the following files

http://windows.php.net/downloads/pecl/releases/sqlsrv/<IP_ADDRESS>/php_sqlsrv-<IP_ADDRESS>-7.1-nts-vc14-x64.zip
http://windows.php.net/downloads/pecl/releases/sqlsrv/<IP_ADDRESS>/php_sqlsrv-<IP_ADDRESS>-7.0-nts-vc14-x64.zip

Just noticed that the packages are separated unlike what we have for the 4.1.5 release files on the release page.

https://pecl.php.net/package/sqlsrv/<IP_ADDRESS>/windows
https://pecl.php.net/package/pdo_sqlsrv/<IP_ADDRESS>/windows


third party crate support with mirror file path

I'm using a rust crate mirror links: mirrors.ustc.edu.cn and racer with vim YCM
when I update dependents library, the third party crate source code is just download into the .cargo/registry/src/mirrors.ustc.edu.cn-XXXX/   path.
but I found that, racer does not catch the right define of the crate code. just show Errors:
"RuntimeError: Can't jump to definition."  how could I resolve this problem. any help will be appreciate, thanks.

You can add the support for it.
I don't know about racer path implementation.


Container Apps Integration Prep

Related to:  https://github.com/microsoft/vscode-azurecontainerapps/pull/170

There were changes made to main since the old approval, so I pushed up the new changes, will have an associated PR on container apps as well to get the new context values to match up properly

Update: Changes made


Error placing bid?

[22:27:51] Something happened when trying to bid on *******
[22:27:55] Error placing bid on *********
{
"futApiStatusCode": 500,
"reason": ""
}
what is this ? please help

Usually a temp ban. What RPM settings are you using, and do you have auto stop enabled? It should clear up within 24 hours, but I would decrease your RPM or shorten the amount of time you are running the AB.
How long did you have it running before the 500 errors started showing up? If you want to run 24/7 you should set RPM to 3-4. Previous testing shows 10 RPM to be safe for 6-8 hours before temp ban is issued. Others have reported that you can make 4000-5000 requests per day (hence the 3-4 RPM)
To be safe, I would enable auto stop and have it set to how ever many hours it would take your current RPM settings to hit 4000 requests. so 10 RPM would be max of 6.5 hours.

i have it on 12 but when i sign in on my ps4 i can make bids on players
i have the message 2 hours ago

You need to wait 24 hours for an to go away.

You can still bid on console. It's just web app that is banned.


Stuck "Turning on Setting Sync..."

Testing #141275

Setting is turned off
I have an missing comma in my keybindings.json
From the Account icon, turn on setting sync
Go through the sign in flow and turn on setting sync
An error message shows, so I looked at the logs and fixed the missing comma in keybindings.json
🐛 How can I complete turning on settings sync now? I don't see a command or other way to do so:

Did you reloading the window?

I also got into this state (testing on Windows).

Turn off Settings Sync on one machine
Change tasks.json on both machines to introduce a conflict
Turn Settings Sync back on, resolve the merge conflict manually by selecting to accept remote
Change doesn't take effect, check gear and notice I'm also stuck "Turning on Settings Sync...". I reloaded the window, but it's still in this state

I also tried reloading the window, but the problem persisted.

I can reproduce what @alexr00 has reported and fixed it.
@bamurtaugh I am not able to reproduce the issue you reported. Can you please file a separate issue and may be share the video with steps?

@sandy081 When I tested the TPI again today, I didn't end up stuck in this state, so perhaps it was a fleeting issue? If I encounter it again, I'll be sure to file an issue.


Setting to enable/disable injecting of LAGOON_GIT_SHA into DeploymentConfig

Currently we inject the GIT SHA of the current build into the deploymentconfig of many services:
https://github.com/amazeeio/lagoon/blob/master/images/oc-build-deploy-dind/openshift-templates/node/deployment.yml#L98-L99
https://github.com/amazeeio/lagoon/blob/master/images/oc-build-deploy-dind/openshift-templates/node-persistent/deployment.yml#L114-L115
https://github.com/amazeeio/lagoon/blob/master/images/oc-build-deploy-dind/openshift-templates/nginx/deployment.yml#L101-L102
This provides a nice feature that if the application code needs to access the Git Hash of the current build it has it accessible in an environment variable LAGOON_GIT_SHA.
Unfortunately though this also means that every deploymentconfig will redeploy it's containers even if the docker image itself has not changed. That can cause much longer deployments and even sometimes service interruptions where they would not be needed.
Therefore the idea is to somehow define within .lagoon.yml globally for all services or in docker-compose.yml per service if the application needs the LAGOON_GIT_SHA env variable and only if it is requested by the service actually inject it into the deploymentconfig.
It probably is possible to have the default to not inject it anymore as I don't think many applications today actually use LAGOON_GIT_SHA, definitely the lagoon internal tests are using it, but there we could just adapt the docker-compose.yml

Ok, after discussion with Michael, we will have the git_sha injection disabled by default. We will add a project-level entry to the .lagoon.yml, which will then allow the git_sha to be injected into the deployment configs of services which need it.


fix error dialog from appearing after loading material res

It was a wrong expression in a if statement that always triggered the error dialog and returned in the "_file_selected" method.
Everything else seemed to be correct I think.

btw see this pr #66718 to compile for visual studio

Thanks!


How can I use Autogluon in SageMaker?

I'm trying to evaluate Autogluon results with my on prem YOLO V3 results. Do not have a GPU to run Autogluon on-prem. So, wish to run the same on Sagemaker. But, saw some bug in one of its requirement libs in another issue here. Do I have to wait for the dependent library to be fixed before using it on SM or can I still use it?

Here are the package versions I used in SageMaker Docker container. Please expect an example later to run on SM.
numpy==1.16.0
scipy==1.3.0
spacy==2.1.8

gluoncv
gluonnlp
graphviz
scikit-optimize
ConfigSpace==0.4.11

boto3==1.9.224
awscli==1.16.253

requests
nose

matplotlib
seaborn==0.9.0
jupyter==1.0.0

### AutoGluon DistributedScheduler remote workers ###
dask[complete]==2.6.0
paramiko==2.4.2
distributed==2.6.0
tornado

## tabular prediction ##
tqdm
psutil
catboost
lightgbm==2.3.0
pandas==0.24.2
scikit-learn==0.21.2

FYI.
An example of using image classification docker is available:
https://github.com/zhanghang1989/AutoGluon-Docker

We are actually working on a SageMaker container for AutoGluon, hopefully we can give further updates in the next few weeks.

Please refer to the SageMaker tutorial on using AutoGluon: https://github.com/aws/amazon-sagemaker-examples/blob/master/advanced_functionality/autogluon-tabular/AutoGluon_Tabular_SageMaker.ipynb


Updating release notes for 2.7.0; updates logo

Summary:
Updates the release notes.
Updates the logo.
Intended Effect:
Includes release notes for v2.7.0.
How to Verify:
Build the manual. The release notes don't affect the build.
Side Effects:
None.
Documentation:
Manual logo has changed. No doc necessary.
Reviewer Suggestions:
None.

Jenkins, retest this please.


Seems broken ( cannot build at all )

Hi, I have just tried starting up with iOS simulator. It is showing these errors:

i18n library error (solved by deleting the missing lib)
RCTGeoLocation library error (solved by deleting the missing lib
react-native-keychain library missing (solved by installing back react-native-keychain)
node_modules/react-native/scripts/.packager.env : No such file or directory (don't know if it will make APP broken or not yet )
iOS simulator Red screen - Native module cannot be null. - NativeEventEmitter.js:36:16 (Don't know the cause, unable to fix)

Sorry to say i don't have access to mac. so i never debug it on ios. It is working fine on android.
Please go through RN0.60 docs  for setting up in ios
Please understand  if you make it work a PR is welcome
If you find a solution. A pr is welcome.

Can not build on ubuntu 18.04. Spent a whole day trying different fixes for the various problem I come across. Getting fed up. Please help

@drtobbyas  what is the problem you are facing..please add error logs etc

@HarishJangra This is the error I'm getting
This is my system spec
CPU~Dual core Intel Core i5 M 560 (-MT-MCP-) speed/max~2342/2667 MHz Kernel~4.15.0-54-generic x86_64 Up~1:40 Mem~6686.6/7902.7MB HDD~500.1GB(82.7% used) Procs~284 Client~Shell
Ubuntu 18.04
Lubuntu desktop installed
$ yarn android:dev
yarn run v1.17.3
$ react-native run-android --appIdSuffix debug
info JS server already running.
info Installing the app...

Configure project :app
WARNING: API 'variantOutput.getPackageApplication()' is obsolete and has been replaced with 'variant.getPackageApplicationProvider()'.
It will be removed at the end of 2019.
For more information, see https://d.android.com/r/tools/task-configuration-avoidance.
To determine what is calling variantOutput.getPackageApplication(), use -Pandroid.debug.obsoleteApi=true on the command line to display more information.

Task :app:compileDebugJavaWithJavac FAILED

Deprecated Gradle features were used in this build, making it incompatible with Gradle 6.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/5.4.1/userguide/command_line_interface.html#sec:command_line_warnings
29 actionable tasks: 1 executed, 28 up-to-date
/mnt/01D47BE1D8128F20/drtob/goodies/programming/olalomitv/react-native-easy-starter/android/app/src/main/java/com/olalomitv/MainApplication.java:4: error: cannot find symbol
import com.facebook.react.PackageList;
^
symbol:   class PackageList
location: package com.facebook.react
/mnt/01D47BE1D8128F20/drtob/goodies/programming/olalomitv/react-native-easy-starter/android/app/src/main/java/com/olalomitv/MainApplication.java:32: error: cannot find symbol
List packages = new PackageList(this).getPackages();
^
symbol: class PackageList
2 errors
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:compileDebugJavaWithJavac'.

Compilation failed; see the compiler error output for details.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 18s
error Failed to install the app. Make sure you have the Android development environment set up: https://facebook.github.io/react-native/docs/getting-started.html#android-development-environment. Run CLI with --verbose flag for more details.
Error: Command failed: ./gradlew app:installDebug -PreactNativeDevServerPort=8081
/mnt/01D47BE1D8128F20/drtob/goodies/programming/olalomitv/react-native-easy-starter/android/app/src/main/java/com/olalomitv/MainApplication.java:4: error: cannot find symbol
import com.facebook.react.PackageList;
^
symbol:   class PackageList
location: package com.facebook.react
/mnt/01D47BE1D8128F20/drtob/goodies/programming/olalomitv/react-native-easy-starter/android/app/src/main/java/com/olalomitv/MainApplication.java:32: error: cannot find symbol
List packages = new PackageList(this).getPackages();
^
symbol: class PackageList
2 errors
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:compileDebugJavaWithJavac'.

Compilation failed; see the compiler error output for details.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 18s
at checkExecSyncError (child_process.js:629:11)
at execFileSync (child_process.js:647:13)
at runOnAllDevices (/mnt/01D47BE1D8128F20/drtob/goodies/programming/olalomitv/react-native-easy-starter/node_modules/@react-native-community/cli-platform-android/build/commands/runAndroid/runOnAllDevices.js:75:39)
at buildAndRun (/mnt/01D47BE1D8128F20/drtob/goodies/programming/olalomitv/react-native-easy-starter/node_modules/@react-native-community/cli-platform-android/build/commands/runAndroid/index.js:137:41)
at then.result (/mnt/01D47BE1D8128F20/drtob/goodies/programming/olalomitv/react-native-easy-starter/node_modules/@react-native-community/cli-platform-android/build/commands/runAndroid/index.js:103:12)
at process._tickCallback (internal/process/next_tick.js:68:7)

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

@drtobbyas  - Please double check your package names in android folder , AndroidManifest.xml , Mainapplication.java etc

Run yarn
run yarn run clean:debug

fixed in new release


Use GetFeature + API lease approach for accessing SKCanvas

Introduces
[Unstable]
public interface ISkiaSharpApiLeaseFeature
{
    public ISkiaSharpApiLease Lease();
}

[Unstable]
public interface ISkiaSharpApiLease : IDisposable
{
    SKCanvas SkCanvas { get; }
    GRContext GrContext { get; }
    SKSurface SkSurface { get; }
    double CurrentOpacity { get; }
}

Adds object? GetFeature(Type t) to IDrawingContextImpl.
This approach allows:

to proxy calls to IDrawingContextImpl without implementing the skia interface
to potentially implement support for SkiaSharp even if drawing context is using a different graphics backend under the hood

You can test this PR using the following package version. 0.10.999-cibuild0022796-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0022914-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]


UI feedback when dropping a subtitle

Brilliant work gents. You just made my Chromecast worth its price.
A tiny improvement would be to give feedback in the UI when one drops a subtitle on the cat. Currently, you don't know if it registered or not.

Any ideas on what kind of feedback?

I'm looking into making a big thumbs up :+1:  appear and face out. Would that work?

I think a generic toast/message design would work. Let me do a mockup real quick. Could solve #18 too.

@bmathews nice, sounds great.

Something like this? I don't know how the subtitle stuff works, but this kind of toast could say anything and be positioned in the center or corners as needed.

Looks great for the buffering. As for subtitles, I think it depends on the fade in/out effect we add to it.
Which icon do you use for the buffering animation?

I'll get you a css class with an inline svg background, one sec.

This should do:
.loading {
    display: inline-block;
    height: 16px;
    width: 16px;
    background-image: url('data:image/svg+xml;utf8,<svg width="16" height="16" viewBox="0 0 300 300" xmlns="http://www.w3.org/2000/svg" version="1.1"> <path d="M 150,0 a 150,150 0 0,1 106.066,256.066 l -35.355,-35.355 a -100,-100 0 0,0 -70.711,-170.711 z" fill="#ffffff"> <animateTransform attributeName="transform" attributeType="XML" type="rotate" from="0 150 150" to="360 150 150" begin="0s" dur="1s" fill="freeze" repeatCount="indefinite" /></path></svg>');
}

A general toast makes a lot more sense :+1:


Tags overflow on project card

Acceptance Criteria

[ ] No overflow on both Grid + List cards

How do you want it to be handled? Should they be truncated or there should be scroll bar?

hey @itexpert120, I think we can wrap it, it good to display all the tags, since it vital information of the project, and we also use them in filters.

How do we wrap this in the list view?

How do we wrap this in the list view?

You can add a max-width within which it will wrap, or may be ask Cecilia for a design, she might have better ideas about how to handle it.

That sounds like a plan, I'll ask her


Effects

Idea
Functions indicate which effects they need access to in a bitmap.
This allows the VM to run them in the correct context and combine the multiple VMs into one.
If an op is hit that the effect isn't enabled for then the predicate will panic.
Effects
Some potential effects

State
Looping
Jump Forward
Jump Anywhere
Mutable memory
Read access to previous functions memory
Mutable access to previous functions memory
Panic
Expensive functions like crypto

Implementation
Use a byte at the start of any function and have bit flags for each effect. Max 8 effects

Just noting that effects analysis has since been implemented in #196, but we don't yet make use of this in essential-check.


jazzy release

Can we please get a bloom release to jazzy and rolling? Specifically, releasing to rolling will save you from me nagging about releases in the future as new releases will always branch of rolling and therefore automatically will have a bloom release for any future ROS distribution.

@fmessmer Any chance for a jazzy and/or rolling bloom release?

Pull requests for jazzy and rolling releases are open now:

https://github.com/ros/rosdistro/pull/43653
https://github.com/ros/rosdistro/pull/43654


cfr: 0.147 -> 0.148

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/cfr/versions.
meta.description for cfr is: '"Another java decompiler"'.
meta.homepage for cfr is: '"http://www.benf.org/other/cfr/"

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.148 with grep in /nix/store/hqmjbb86fhvai0p9qxqdff407xys44k9-cfr-0.148

directory tree listing: https://gist.github.com/80c5baafce8509966780fbf48b11b221

du listing: https://gist.github.com/e866d5ab67c61945b7be22a508b3ee46

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

4 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
cfr

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/hqmjbb86fhvai0p9qxqdff407xys44k9-cfr-0.148 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A cfr https://github.com/r-ryantm/nixpkgs/archive/8ec3a3596e12fb0a470a217062f3980a66d76ad3.tar.gz

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

nox-review happy for me macos 10.13, binary executes.


Update README

This adds a new part in the README containing a list of  'other codes using chemfiles'
@frodofine does the description of lemon sounds good, or do you prefer something else? Also, are you using chemfiles in another project?
@ezavod and @pgbarletta, are you using chemfiles in publicly available projects?

Looks good to me!
I am using it in other projects, but they are not public (yet). I'll add them here when they do become public.

I added mine.

I have nothing to add yet. I am using chemfiles mainly in quick and dirty analysis scripts.

Thanks everyone!


V5 version?

Submitter: Pottman
Email<EMAIL_ADDRESS>Will you do a V5 version of this?

Maybe eventually. That's as much as I can guess for something like that. I don't have a reason why I wouldn't, so I might as well include all editions, if I can. Time and energy though, they are hard to come by...

i have some files in my repo that might be useful to you:

enums
typings
translations

I might still continue working on that stuff myself but your project seems to have a wider scope so I'd gladly share.

I'd gladly accept any help with this project. So far I've been the sole developer on it. If you'd like to add some resources or even fork and make your own branch, any help is appreciated.

I've never used angular unfortunately, but might try to help a bit if I ever get some free time.
thank you for the great work! :)


telegram bot

create a telegram bot that allows marking in different rooms

Some early prototyping was here:
https://github.com/melvincarvalho/telemarking


Getting the docker-compose example working on Windows MinGW / MSys

These are some changes I made in order to get the docker-compose example working on windows in a MinGW / MSys bash terminal.
The change to accept Lf / Cr / CrLf line-endings is nice to have when shutting auto-crlf conversion off in windows - which is a general practice when having projects build docker containers (script creation needs to have Lf line-endings when adding to linux-based containers).
Use of the 'sh' command tells MSys explicitly this is a shell-script and it can use the MSys local binary for that.
The other two changes (in world.sql and start_metacat_test_cluster.sh) were required within the container environment to get the example running - not sure why this was any different within Windows than *nix machines.
There is also a few groovy tests that are failing for the Hive connector build (within PartitionUtilSpec.groovy) that I see are not failing on *nix machines - I haven't fixed that yet though (just skipping tests on the build at this point).

Build likely failed due to that cassandra init.cql reference - apologies, I didn't see the local volume reference in the docker-compose yaml - I will revert that part specifically and figure out why it wasn't mounting on my test.


Refs #24397 -- Sped up model reloading in ProjectState.

Created bulk_update() context manager on StateApps. Sped up unregistering
models in reload_models() by using this context mananger.
Follow-up patch for https://code.djangoproject.com/ticket/24397.

merged in 039d7881b441c6b358a963d652ffa2eada3a8892, thanks!


little confused about train_cf.py

Hi, sir,thanks for the implementation! I'd think it is a great work!
I am little confused with specific counterfactual training steps. It seems that the network in "train_cf.py" only take single counterfactual parent into account during training. But there are actually multi counterfactual parents when infering(Like the case shown in Hugging Face).
So should I run the "train_cf.py" for every counterfactual parent (thickness->intensity->digit) one by one during training ?

Hi,
Thanks for your message.
So the counterfactual training setup we used involves interventions on a single parent at once, that is, for each batch element we randomly select which parent we would like to randomly intervene on. This intervention is then used to compute all the counterfactual parents depending on the associated causal graph.
It is of course possible to intervene on more than one parent at once, although we didn't observe any advantage in doing so in our experiments.
Best,
Fabio


Load more stories

Would it be possible to have pyhn load more stories than just 60?
I would love to be able to go through the entire pagination available.

Pyhn need a little rework to take advantage of new HackerNews API. It will be difficult to do it now without migrating Pyhn to official API.
Contributions are welcome to start this migration work. :smile:


removed unwanted characters

sample_data/schools_grade12_2081.tsv contained unwanted characters (parentheses) at lines viz. 106 & 378 which have been removed.

thank you for contribution. however this is not a valid change, please redirect your efforts to open Issues


Fix a typo in logcapture.py

Coverage remained the same at 100.0% when pulling 87ad6b98c164d49b07072fd93bc434b3e557102a on felixonmars:patch-1 into 3b2cfff17e9bbd19a8cd0977a322776462e80005 on Simplistix:master.


Subscription config wouldn't overwrite the resources in the operator pod if the config is empty

Description of the change:
Fix #1953 #1940
Subscription config wouldn't overwrite the resources in the operator pod if the config is empty.
Motivation for the change:
By default the subscription config is empty, but it will force empty the existing resource configs in the CSV.
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /docs
[ ] Commit messages sensible and descriptive

/ok-to-test

/test unit

@horis233 Looks like this PR also needs a rebase if you have a second to update it.

@timflannagan
I will close this PR since it has already been fixed by https://github.com/operator-framework/operator-lifecycle-manager/pull/2010


Sending tab to a firefox instance

Is there a way to trigger a "Send link to Device" feature using this client? From what I see it can only list already opened tabs, but maybe I am missing something...

Hmm as far as I know the send-tab-to-device feature has nothing to do with firefox-sync.
(You can read about the available API here and here.
If you find an API you can probably spin up a small bash script, but I think it's out-of-scope for this project

for record on firefox side send tab is handled in https://github.com/mozilla/gecko-dev/blob/ed2b06f15a5081b2b5e4ab101184c1272d249235/services/fxaccounts/FxAccountsCommands.sys.mjs
and open/close remote tap API uses identity.mozilla.com/cmd/ as prefix, but firefox sends request with same oauth scope key as sync use, idendty.mozilla.com/apps/oldsync .


Replaced uses of SecurityRoles by Set mappedRoles where the SecurityRoles functionality is not needed

Description
This code change is just in preparation for the change in #4380 as requested in https://github.com/opensearch-project/security/pull/4380#discussion_r1621325450 .
In the course of #4380 , the SecurityRoles class will be replaced by a new concept.  This PR already replaces the usages of SecurityRoles where no methods are used on it except the getRoleNames() method. This method call can be easily replaced by the Set<String> mappedRoles, which is computed by PrivilegesEvaluator before the SecurityRoles instance is created. This change elliminated the coupling of the interfaces that consume that information to the SecurityRoles class, thereby enhancing code quality.

Category:  Refactoring
Why these changes are required? The changes in #4380 will abolish the SecurityRoles class in the end.
What is the old behavior before changes and new behavior after changes? No behavioral changes.

Issues Resolved

https://github.com/opensearch-project/security/pull/4380#discussion_r1621325450

Testing
[Please provide details of testing done: unit testing, integration testing and manual testing]
Check List

[ ] New functionality includes testing - no new functionality
[ ] New functionality has been documented - no new functionality
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@peternied @cwperks @reta  I am getting test failures on DelegatingRestHandlerTests - I guess this is a regression caused by the changes done in https://github.com/opensearch-project/OpenSearch/pull/13772 to RestHandler.  I guess this can be fixed by adding supportsStreaming() to DelegatingRestHandler.

@peternied @cwperks @reta I am getting test failures on DelegatingRestHandlerTests

Fixing it right away,  thank you @nibix

@reta Thanks for the quick fix!

@peternied @reta I am getting now another CI failure where I am a bit lost.
* Where:
Build file '/home/runner/work/security/security/build.gradle' line: 473

* What went wrong:
A problem occurred evaluating root project 'opensearch-security'.
> Cannot change resolution strategy of dependency configuration ':runtimeClasspath' after it has been resolved.

However, gradlew assemble on my own system runs fine.

I ran into more CI issues, a fix is here: #4446

@nibix I'm not sure why some of the tests have not run. Can you rebase and push to see if it kicks off the CI checks? I'm not able to force start them.

@cwperks Done, but we had again test failures in a windows job. Would you mind to restart these again?


Create a UI design

I wish to create a UI design using html and css.

for what purpose you want UI design? please specify

I wish to add product card UI Design.

can you please share your project link ? where i will add product card UI code and send you the pull request .


Cropper is not a function

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report
[ ] Feature request
[ ] Documentation issue or request

Current behavior

On my symfony project with webpack encore, I am using PrestaImageBundle https://github.com/prestaconcept/PrestaImageBundle , I have installed cropper, as stated in the documentation https://github.com/fengyuanchen/jquery-cropper#installation but when uploading the image I ended up with this error in my console :
cropper.js:158 Uncaught TypeError: this.$container.$preview.children(...).cropper is not a function
at Cropper../vendor/presta/image-bundle/Resources/public/js/cropper.js.Cropper.prepareCropping (cropper.js:158)
at FileReader.reader.onload (cropper.js:72)
There are no 404 error, only this one when uploading an image.
Expected behavior

It should give me the option to crop the image to the desired size
Minimal reproduction of the problem with instructions
But instead I end up with this error

What is the motivation / use case for changing the behavior?

Environment

jquery-cropper version: X.Y.Z

Browser:
- [x] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

Others:

here are links and config used :

in base template

in twig config
twig:
default_path: '%kernel.project_dir%/templates'
form_themes: ['bootstrap_4_layout.html.twig']
in js file
import 'jquery/dist/jquery.min';
import 'cropperjs/dist/cropper.min';
import 'jquery-cropper/dist/jquery-cropper.min.js';
import * as Cropper from '../../vendor/presta/image-bundle/Resources/public/js/cropper.js';
import '../css/form.scss';
in scss of my form
@import "/../../node_modules/cropperjs/dist/cropper.min";
@import '../../vendor/presta/image-bundle/Resources/public/css/cropper.css';
If any of you has an idea why I end up with this error, please let me know, We have read documentation several times and still don't understand why it doesn't work.

@Zortaes did you get to resolve this issue? I'm getting the same error.


[EasyCI] Do not prefix Symplify\EasyCI* namespace on scoper

@TomasVotruba this is to avoid error ref https://github.com/rectorphp/rector-src/runs/4829684998?check_suite_focus=true#step:5:7

Run vendor/bin/easy-ci validate-file-length packages rules src tests

Error: ] Class "Symplify\EasyCI\ValueObject\Option" not found

as it is not part of included vendor that need to be prefixed.
From inspiration https://github.com/rectorphp/rector-src/blob/6b965fe744f438b0614597bec485ad8ec23831d9/utils/compiler/src/PhpScoper/StaticEasyPrefixer.php#L30-L31

All checks have passed 🎉 @TomasVotruba it is ready for review.

Thank you


Default cmake build uses openblas instead of MKL

Description
Default CUDA-free build links to openblas AND mkl. openblas is used instead of mkl, which leads to pure performance.
Environment info (Required)
----------Python Info----------
Version      : 3.5.2
Compiler     : GCC 5.4.0 20160609
Build        : ('default', 'Nov 12 2018 13:43:14')
Arch         : ('64bit', 'ELF')
------------Pip Info-----------
Version      : 8.1.1
Directory    : /usr/lib/python3/dist-packages/pip
----------MXNet Info-----------
Version      : 1.5.0
Directory    : /home/stsukrov/workspace/mxnet/python/mxnet
Hashtag not found. Not installed from pre-built package.
----------System Info----------
Platform     : Linux-4.4.0-1074-aws-x86_64-with-Ubuntu-16.04-xenial
system       : Linux
node         : ip-172-31-1-212
release      : 4.4.0-1074-aws
version      : #84-Ubuntu SMP Thu Dec 6 08:57:58 UTC 2018
----------Hardware Info----------
machine      : x86_64
processor    : x86_64
Architecture:          x86_64
CPU op-mode(s):        32-bit, 64-bit
Byte Order:            Little Endian
CPU(s):                72
On-line CPU(s) list:   0-71
Thread(s) per core:    2
Core(s) per socket:    18
Socket(s):             2
NUMA node(s):          2
Vendor ID:             GenuineIntel
CPU family:            6
Model:                 85
Model name:            Intel(R) Xeon(R) Platinum 8124M CPU @ 3.00GHz
Stepping:              4
CPU MHz:               3000.000
BogoMIPS:              6000.00
Hypervisor vendor:     KVM
Virtualization type:   full
L1d cache:             32K
L1i cache:             32K
L2 cache:              1024K
L3 cache:              25344K
NUMA node0 CPU(s):     0-17,36-53
NUMA node1 CPU(s):     18-35,54-71
Flags:                 fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss ht syscall nx pdpe1gb rdtscp lm constant_tsc arch_perfmon rep_good nopl xtopology nonstop_tsc aperfmperf pni pclmulqdq monitor ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand hypervisor lahf_lm abm 3dnowprefetch invpcid_single kaiser fsgsbase tsc_adjust bmi1 hle avx2 smep bmi2 erms invpcid rtm mpx avx512f rdseed adx smap clflushopt clwb avx512cd xsaveopt xsavec xgetbv1 ida arat pku
----------Network Test----------
Setting timeout: 10
Timing for Conda: https://repo.continuum.io/pkgs/free/, DNS: 0.0155 sec, LOAD: 0.0708 sec.
Timing for FashionMNIST: https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, DNS: 0.0348 sec, LOAD: 0.2342 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.0010 sec, LOAD: 0.3944 sec.
Timing for Gluon Tutorial(en): http://gluon.mxnet.io, DNS: 0.0896 sec, LOAD: 0.7671 sec.
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0006 sec, LOAD: 0.9998 sec.
Timing for Gluon Tutorial(cn): https://zh.gluon.ai, DNS: 0.2689 sec, LOAD: 1.0834 sec.
Compiler (gcc/clang/mingw/visual studio):
gcc-5
MXNet commit hash: d684c59049970794e8f7365f58b03e13801b44a3
Build config:
cmake -DUSE_CUDA=OFF
tracing the line used to link the libmxnet.so produces
/home/stsukrov/cmaketmp/cmake-3.13.2-Linux-x86_64/bin/cmake -E cmake_link_script CMakeFiles/mxnet.dir/link.txt --verbose=1
/usr/bin/c++ -fPIC -Wall -Wno-unknown-pragmas -Wno-sign-compare -O3 -msse2 -std=c++11 -mf16c -fno-builtin-malloc -fno-builtin-calloc -fno-builtin-realloc -fno-builtin-free -fopenmp -std=c++0x  -shared -Wl,-soname,libmxnet.so -o libmxnet.so CMakeFiles/mxnet.dir/dummy.c.o  -L/home/stsukrov/workspace/incubator-mxnet/build/mklml/mklml_lnx_2<IP_ADDRESS>80928/lib -Wl,-rpath,/home/stsukrov/workspace/incubator-mxnet/build/3rdparty/mkldnn/src:/home/stsukrov/workspace/incubator-mxnet/build/3rdparty/openmp/runtime/src:/home/stsukrov/workspace/incubator-mxnet/build/mklml/mklml_lnx_2<IP_ADDRESS>80928/lib: -Wl,--whole-archive libmxnet.a -Wl,--no-whole-archive libmxnet.a 3rdparty/mkldnn/src/libmkldnn.so.<IP_ADDRESS> -lopenblas -lrt -ljemalloc /usr/lib/x86_64-linux-gnu/libopencv_highgui.so.2.4.9 /usr/lib/x86_64-linux-gnu/libopencv_imgproc.so.2.4.9 3rdparty/openmp/runtime/src/libomp.so -lpthread -llapack -ljemalloc 3rdparty/dmlc-core/libdmlc.a -lpthread -llapack -lmklml_intel mklml/mklml_lnx_2<IP_ADDRESS>80928/lib/libiomp5.so /usr/lib/x86_64-linux-gnu/libopencv_core.so.2.4.9 -lpthread -ldl
So blas is linked before mklml_intel
Minimum reproducible example
Get the mxnet, build it, run the benchmark
Steps to reproduce
(Paste the commands you ran that produced the error.)

git clone --recursive ...
mkdir build && cd build
cmake -DUSE_CUDA=OFF
make -j20
ldd libmxnet
Run https://github.com/apache/incubator-mxnet/blob/master/example/image-classification/benchmark_score.py

What have you tried to solve it?

Linked manually to mklml_intel with a brutal fix: https://github.com/stsukrov/incubator-mxnet/commit/eebc17b04602b448686231a9e1fd1722bf919da7
Rerun the benchmark
Got the declared performance

@mxnet-label-bot add [bug, build, cmake]

@mxnet-label-bot add [bug, performance]

@lebeg is working on the fixing for cmake and BLAS related issues.

Thanks, Patric!
I'm in contact with @lebeg and we will track the issue.

Yes, it is a significant problem. I did fix it in https://github.com/apache/incubator-mxnet/pull/11148, but fix was never merged. I think initially the thought behind it was that presumably MKLML wouldn't have all necessary math functions included and therefore openblas was linked to cover this cases. In reality MKLML functionality is fully sufficient, so openblas should not be linked at all when MKL or MKLML is used.

@stsukrov This issue has been fixed totally by #14877.
You could try in latest master branch.

Closing. Feel free to reopen if any issue is observed again.

Thanks!


Refactor to save off normalized attr/prop name

This is a part of #283 that we actually need to fix issues we see in Ember with nomalization. This change is to create an instance of the managers that know about the normalized attr/prop name.

I think we can revisit in the future. The specialized managers are still singletons.


Fix deprecated warning with analyzer 5.12.0

Sorry, analyzer ^5.12 support only Dart 2.19 and above

Should I raise analyzer and Dart for 2.19, or should I leave it at 2.18?

Problem is due to source_gen ^1.3.0

It's a BREAKING change.  Let increase the major version and mark it as a pre-relaase one.

Like retrofit_generator 7.0.0 and analyzer 5.12?

@trevorwang check now plz