sending reserved JS names through ports

A couple folks have opened issues about running into problems when using reserved JS names as fields names in Elm, and then sending them out through ports:

#1672 - sending static out to JS
#1548 - sending interface out to JS
#1510 - sending default out to JS
https://github.com/elm-lang/core/issues/611 - sending arguments out to JS

For now, I recommend doing renames in your Elm code to work around the issue.
Note: Please open a new issue if you run into a new version of this! Having more examples from more people is better than piling on to existing issues. Some issues are great, some are really hard to understand. Having each person do it separately makes it easier to sort through it all!

With the following SSCCE:
port module Main exposing (main)

import Browser
import Html

main : Program () () ()
main =
  Browser.embed
    { init = \_ -> ( (), echo (TrickyNames 0 1 2 3 4 5) )
    , update = \_ _ -> ( (), Cmd.none )
    , subscriptions = \_ -> Sub.none
    , view = \_ -> Html.text "Check console."
    }

port echo : TrickyNames -> Cmd msg

type alias TrickyNames =
  { static : Int
  , default : Int
  , interface : Int
  , arguments : Int
  , this : Int
  , prototype : Int
  }

<html>

<head>
    <script src="elm.js"></script>
</head>

<body>
    <main></main>
    <script>
        var app = Elm.Main.embed(document.querySelector('main'));

        app.ports.echo.subscribe(function (message) {
            console.log(message);
        });
    </script>
</body>

</html>

I am seeing {arguments: 3, default: 1, interface: 2, prototype: 5, static: 0, this: 4} on the console. So this is fixed in the development build of the compiler.


Add configuration section for average price

Hi,
Would it be possible to add a config section that you can write the average price that you bought your share holdings onto (so that you can see how much your P/L is overall)? Something along these lines in .stonky.cfg:
[avg_price]
AMZN=2367.92
VGRO.TO=24.81
BTC-USD=11.983.70

It won't take into account dividends etc. but it's still good as a guide and I'd personally find it very useful. If this is something that you'd like, I can add make a pull request for it myself?
This is a great and super useful project; thanks a lot!

I don't think being able to set the average buying price would have any effects on the output
Just to recap in case its not clear:

"Watchlist" just shows ticker tapes for the listed securities
"Positions" report changes for the day
"Profit and Loss" reports the sum of changes for the day
"Balances" shows a sum of all held securities plus reported cash

Sorry, I should have been more clear. The section would be added in addition to code for calculating an OVERALL section, which would print something like this in the output, detailing the overall P/L on your holdings:
OVERALL POSITIONS
AMZN 27.07% 3204.95
VGRO.TO 6.89% 1709.40
BTC-USD -20.00% -239.77

(the crunched up above is calcuated from the positions, average price data as in the original prices in my issue message, and current price)


Readme cleanup

Just a little cleanup for the Readme.

Fix indentions and linebreaks
Use travis svg badge

:+1:


Oracle Errors

I'm trying to get the actual Oracle error (code/text) back from a failed connection, but that doesn't seem to be happening. Can this be done? Here is my sample code:
db, err := sql.Open("oci8", connectStr)
if err != nil {
	return nil, err
}
err = db.Ping()
if err != nil {
	db.Close()
	return nil, err
}

In the case of a bad user/password, db.Ping() throws an error of "ping failed", but the actual reason is unknown unless I connect to the database with another client. Is this something that I'm doing incorrectly?

You are not doing anything wrong.
How the Go SQL driver works is a little weird. It should return driver: bad connection. Have seen other drivers not return that but that is not what the Go SQL driver notes say to do. Have seen the solution in other driver to return additional error information with log, which probably something that should be done but is not implemented right now.
Go SQL driver notes:
https://golang.org/pkg/database/sql/driver/#Pinger

If Conn.Ping returns ErrBadConn, DB.Ping and DB.PingContext will remove the Conn from pool.

Not that I am aware of at this time.
@mattn
Any thoughts on this? Should I add log? Or have another idea on what you would like to see?

AFAICS, return ociGetError(rv, c.err) will solve this?

If you want to do the log method would be some thing like the following (not real code):
(already checked for error and has error)
err = ociGetError
if isBadConnection(err) {
log.print(err)
return driver.ErrBadConn
} else {
return err
}

I think user can check the err via conn.Ping() and what happen.

Pretty sure ping needs to return driver.ErrBadConn instead of the ORA error.
https://golang.org/src/database/sql/sql.go?s=21213:21265#L726

If I adjust the ping code to use ociGetError, I do get this (which is slightly more descriptive):
Could not connect to database: driver: bad connection

That still isn't an idea error for this condition. With RazorSQL I see the real reason for the failure:
ORA-01017:
invalid username/password; logon denied

I think it's fine for the ping to give a generic failure, but is there a way to get this ORA error?

That is what we are working on @kog4096 . Asking @mattn if it is alright to log that information or if there is another way he wants to do it.

Thanks @MichaelS11

I am skeptical to output log with Ping. For example, someone use Ping to check database connection repeatedly, the output will be unusable. If we should add this, I want to add ConnectHook that implemented on go-sqlite3 to call native driver function.

Up to you.
Here is what I did:
https://github.com/MichaelS11/go-cql-driver/blob/master/connection.go#L18-L77

How about to add _driverlogs=true in connection string?

I prefer that disabled logger in default.

PR https://github.com/mattn/go-oci8/pull/261

This should now be fixed. Can enable logging with something like:
OCI8Driver.Logger = log.New(os.Stderr, "oci8 ", log.Ldate|log.Ltime|log.LUTC|log.Llongfile)

Thanks, I'll check it out.

All good? Close this?

Sorry for the late response. I was able to do some testing, but I still don't seem to get the level of detail I was hoping for. I now get "Could not connect to database: driver: bad connection", but when connecting with a client like RazorSQL I get: "ORA-28040: No matching authentication protocol" (or other detailed ORA message). Is there a way to access an error like this?

Did you turn on the logging?
Example:
OCI8Driver.Logger = log.New(os.Stderr, "oci8 ", log.Ldate|log.Ltime|log.LUTC|log.Llongfile)

Thanks @MichaelS11 - that gets me closer. I now get:
Ping error: ORA-01012: not logged on
from the ping attempt, but the initial connection does not error as I'd expect in this case. The test case has a locked account; with sqlplus I get:
ORA-28000: the account is locked
I'll be doing more testing on this.

Welcome. Will look into it more later, see if anything can be improved.

@kog4096 This should now be fixed for login. Will error on database open instead of ping.
Please retest and close if fixed.


Mediapie hands full model less accurate than light model

Have I written custom code (as opposed to using a stock example script provided in MediaPipe): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04, Android 11, iOS 14.4): Ubuntu
Programming Language and version ( e.g. C++, Python, Java): Python
MediaPipe version: <IP_ADDRESS>
Solution ( e.g. FaceMesh, Pose, Holistic ): Hands

Full model does not detect hand in an image where is present whereas the light model does.
Since full model is supposed to perform better than light model, with min_detection_confidence set to 0.5, it should detect hand in an image where light model detected hand with same min_detection_confidence (actual confidence even higher)
Update: Infact this happens for lots of images, approximately 7% from a dataset size of 50k that I have.

Hi @MJ1998 , did you get time to check on this 2173 issue , i hope it helps you .

Hi @sureshdagooglecom in comment  it says "model quality depend not only on model capacity, but also on training dataset and target domain" so its possible and understandable.
But ideally it should not be working this way is what I am conveying here.

@MJ1998 This is an interesting observation. However, we cannot investigate on this due to lack of dataset you are using.
Would it be possible for you to provide the dataset it it does not violate intellectual property rights? it can be the case that the full model is not able to detect hand in a certain orientation/color/exposure conditions of the image.


[BUG] Title I can't use it on Windows 11.

Before submitting a bug report...

[X] This bug wasn't already reported.
(I have checked every bug report on GitHub)

Title

[X] The title is no longer "[BUG] Title" and I edited it with the right error name.

Describe the bug
python3.10.8
Copy and paste your error
Traceback (most recent call last):
File "main.py", line 9, in
from src import Browser, DailySet, Login, MorePromotions, PunchCards, Searches
File "C:\rewards\src_init_.py", line 1, in
from .browser import Browser
File "C:\rewards\src\browser.py", line 12, in
from src.userAgentGenerator import GenerateUserAgent
File "C:\rewards\src\userAgentGenerator.py", line 8, in
class GenerateUserAgent:
File "C:\rewards\src\userAgentGenerator.py", line 34, in GenerateUserAgent
browserConfig: dict[str, Any],
TypeError: 'type' object is not subscriptable
I also try on venv，but not working
Screenshots

Value of dashboard variable
don't have

im on windows 11 ,fine here
did you download all the things in requirements.txt?
as explained in the readme

You're missing some packages, if you look at console it tells you issues are stemming from line 9 on main.py which is a reference to other class's in the src folder that require packages from requirements.txt dev needs to add an exception handler for expected errors

If your problem is resolved - please close this issue out.

Close issue if prob is solved


Allow configuration push via HTTP endpoint?

In certain modes of execution it would be convenient if Prometheus would expose a way to directly push configuration file updates via an HTTP endpoint rather then having to orchestrate filesystem / signal issuance access.
My specific use case is when Prometheus is being executed by Marathon under Mesos - although the configuration is persistent, arranging access to underlying filesystem and enabling users to issue signals to trigger a configuration reload is not trivial.
This problem could be solved by wrapping the Prometheus container in some custom proxying to provide such functionality, or with a configuration management engine of some sort, but both of these would be effectively doing the same thing - additionally, Prometheus is in a position to parse and validate it's config file and push an error back to the client at the point of upload.
Does this idea seem aligned with how Prometheus is used?

We do have POST /-/reload to reload the config but I understand the problem here. So how I did it is put the file somewhere (S3) and run a script to update the config (manually, but I think Ansible can automate).
One issue with using HTTP to issue config is that there is no source of truth, it makes it easier but it also might cause debugging issues later. And I donot see any other monitoring/database system that offers config management via HTTP.

Technically there's no source of truth with Ansible either - i.e. anyone with access can push any configuration they want - which is of course fine, if you're in the middle of "nothing's working and I need it to do this". But it does seem like a workaround for something Prometheus would more sensibly manage itself - i.e. I'd rather gate the configuration push endpoint with my http proxy then try and open up a totally separate path to the server.

We've explicitly decided not to do this. You're free to build something on top of Prometheus to provide this.


what value should pass to 'http_path' in project.yaml file?

Hi team,
What value I should pass to 'http_path' in this projects.yaml?

dbt_impala_demo:
  outputs:
    dev:
     type: impala
     host: <impala host>
     port: <impala port>
     dbname: <db name>
     schema: <db name>
     user: <user>
     password: <password>
     auth_type: ldap
     use_http_transport: true
     use_ssl: true
     http_path: <http path>
  target: dev

Hi @shwangdev,
'http_path' is an optional field, you can find that in JDBC_URL. More information is in this article.
Sample JDBC would look like this:
jdbc:impala://coordinator-cia-dbt-impala.dw-ciadev.a465-9q4k.cloudera.site:443/default;AuthMech=3;transportMode=http;httpPath=cliservice;ssl=1;UID=abrown;PWD=PASSWORD


feat: ini evmts

Current dependencies on/for this PR:

main

PR #146

PR #147   👈

This comment was auto-generated by Graphite.


Use NODE_BINARY from .xcode.env when running packager from Xcode

Summary
Before this change, during the "Start Packager" Xcode build step that runs the packager, packager.sh was using my system node version and not the one from nvm that is meant to be used with the project.
Changelog
[iOS] [Fixed] - Use NODE_BINARY from .xcode.env when running packager from Xcode
Test Plan
Perform a build using Xcode and confirm that the packager is using the correct version of node.

Hi @elsurudo, thanks for the PR. Can I ask you to rebase it on the latest commit on main?
There is a job that is red and it could be related to something that we did last week. Probably a rebase will fix it. If that the case, I'll proceed merging the PR!
Thank you so much for your patience!

@cipolleschi rebased – you're good to go :)


Make sure unit tests till work (build-test-web)

Been so focused on playwright, we didn't noticed the unit tests were no langer passing.


Analysis on how to utilize the first party cookies

Is your suggestion related to an experience ? Please describe.
Chromium has allowed grouping different domains to an entity called first party[2] thus allowing to share cookies among this set by marking cookies with the SameParty[1] attribute.
This feature might be utilized by the Identity Server to allow previously blocked scenarios due to 3rd party cookie restrictions such as iFrame embedded login.
We need to investigate and see what flows can be improved.
[1] https://groups.google.com/a/chromium.org/g/blink-dev/c/XkWbQKrBzMg/m/dIQckPbZAAAJ?pli=1
[2] https://www.chromium.org/updates/first-party-sets
Additional context
Previous Impact analysis with the 3rd Party Cookie restrictions: https://github.com/wso2/product-is/issues/10979

+1
We can enable it when all browsers supports this.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


GH2243: Improve DotNetCoreVSTest with ResultsDirectory

This PR contains some improvements to DotNetCoreVSTest and issue #2243

Improved documentation which reflex the capability of dotnet vstest as it should be assembly based and not project based.
Add support to specify the ResultsDirectory to specify where to store the test results.

@vlesierse your changes have been merged, thanks for your contribution 👍


aaa

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[ ] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

xxx

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


error while while executing burn

while executing burn, I get the below issue -
Something went wrong!
Cannot read property 'reverse' of undefined
and on running it again, I get -
status: 400
BAD REQUEST
: Invalid transaction (InvalidSignature): Transaction signature is invalid.

I have a similar issue, I am getting this once I am trying to burn:
Object { message: "HTTP Error: Requested page not reachable", status: "400 BAD REQUEST", requestURI: "https://test.bigchaindb.com/api/v1/transactions?mode=commit" }


PlotBitmap

What can we do to make ScottPlot better?
Perhaps a function to allow drawing a bitmap without swapping to a different control. This would be useful in the case of multiplots, as they can only be rendered as a static image: #429
Does your proposed feature allow users to do new things in source code? If so, show us how you'd like to be able to use the new feature:
var bmp = multiplot.GetBitmap();
plt.PlotBitmap(bmp)

Check the ScottPlot Roadmap to see if your feature is already planned for an upcoming release:
https://github.com/swharden/ScottPlot/blob/master/dev/roadmap.md

This is simple to implement, I'm just wondering if its outside the scope of ScottPlot. I could see it being inconvenient that most plottables can be added just by installing the nuget package, but multiplots (and perhaps some future plottables) need some trickery.
If this is added, I think PlottablePie, PlottableHeatmap and PlottableRadar would be best changed such that they return a bitmap, as there isn't much use in panning or zooming them.

plt.PlotBitmap(bmp)

This is interesting in itself. It may be useful somehow, but I can't think of an immediate use case, so I'm agnostic about its implementation.

var bmp = multiplot.GetBitmap();
plt.PlotBitmap(bmp)

I probably wouldn't go this direction though. The user wanting to display a Bitmap can just:
pictureBox1.Image = multiplot.GetBitmap();

I definitely like this idea, but considering all the other stuff that's going on (#412 💀) I don't plan to implement this soon. If you give it a go and issue PR I'd be happy to review it and merge it in!


feat: simplify interface for team secrets

As a result of last conversation during the demo, I simplified team secrets interface. Thanks for pointing this out 👍

@Morriz your change request is blocking now. Let me know if there is anything else that need a change.

Just that last thing


Use formatting standard for code contributions

I noticed the project has many different indenting styles, and manually indenting/formatting is no fun.
@linux-cultist, How do you feel about adding a .stylua.toml file to make it uniform and easier to work on? Do you have a personal preference that you could add that to the project?
I do not mind any style (or care about tabs vs. spaces), however, I think sticking to one is always nicer to work with. If you have no preference, I think copying a config from the official NeoVim project would probably be safest. The closest project is probably nvim-lspconfig.
What do you think about this?

Absolutely, let's do that. I also don't care which formatting as long as it's the same everywhere. Copying lspconfig sounds good to me. Go ahead. :)

Thanks for the plugin! It is a lifesaver to manage all my venvs.

Yeah I had the same use case when I stayed writing this plugin. I just couldn't understand why people wanted to open and close neovim every time they switched a venv, so I wrote this plugin. It didn't get much attention at all at first but slowly got more and more users, and I'm very happy to see it included in Lazyvim too. :)


Fix Issue #81

@jonyo
There are 2 issues:

I added \geo_templates\default\external\images\banners\sample_300x100.jpg

However, git is not picking this up and I do not know why.

I had to fix .git\hooks\pre-commit

Git was complaining about \r when I tried to commit changes.

I manually added sample_300x100.jpg to Github.

Just committed 3 files to remove the dev comments from the templates. I have no idea why it is showing the entire file as removed then everything added as new.

@jonyo See if this makes you happy.

@vicos59

I guess cs-fix "fixed" those.

Hmm, I bet this has to do with line ending settings, it might be fixed once I do #119 - at least I think that might be why.  When I run cs-fix myself it does not try to do that.
By the way, that thing you gave a screenshot for, it is saying "before" there was no line ending at the end of the file, and after there is.  So cs-fix added a newline at the end of all those files.  Which I thought was already done.
In either case, I'll look into it and if it turns out there are still files that it missed when I run cs-fix in my local, maybe do a dedicated commit that only makes those whitespace changes if needed.  I want to verify it is not related to windows line ending though so I'll do it in a different PR.

@vicos59 Also like I mentioned on the other PR, can you update the PR title?  Like I mentioned, the titles are used for the release changelog.
Its best if they have short description of the change, and if anyone wants more details they can then click on the PR to see it.  (instead of forcing you to click on every single one to see what it changed)


Helm chart 0.11.0 on the helm repo seems to be outdated

Despite having the same release version (0.11.0) the helm chart on the prometheus-community repository doesn't seem to include the actual release 0.11.0 of this repo.
I miss things like the security config section.

just realized the same, it does still deliver version 0.8 to me. You can however overwrite this with the help values image.tag: v0.11. Note the missing .0 compared to the chart version. Wished it was consistent with that :-/

Will raise the PR for same :D


"Building for Production" docs "Library Mode" section mildly broken

Describe the bug
Based on;

https://vitejs.dev/guide/#scaffolding-your-first-vite-project
https://github.com/vitejs/vite/blame/5ea70b3c3cd5f208471338c5832a3eba1aafb01d/docs/guide/build.md#L117-L185

npm create vite@latest myapp --template vanilla
cd myapp

echo "import { resolve } from 'path'
import { defineConfig } from 'vite'
export default defineConfig({
  build: {
    lib: {
      entry: resolve(__dirname, 'lib/main.js'),
      name: 'MyLib',
      // the proper extensions will be added
      fileName: 'my-lib'
    },
    rollupOptions: {
      // make sure to externalize deps that shouldn't be bundled
      // into your library
      external: ['vue'],
      output: {
        // Provide global variables to use in the UMD build
        // for externalized deps
        globals: {
          vue: 'Vue'
        }
      }
    }
  }
})" > vite.config.js

mkdir lib
echo "export default function square(n) {
  return n*n;
}" > lib/main.js

echo '{
  "name": "my-lib",
  "type": "module",
  "files": ["dist"],
  "main": "./dist/my-lib.umd.cjs",
  "module": "./dist/my-lib.js",
  "exports": {
    ".": {
      "import": "./dist/my-lib.js",
      "require": "./dist/my-lib.umd.cjs"
    }
  }
}' > package.json

npm install
npm exec vite build

Expected
Library builds by copying documentation verbatim.
Actual behavior
$ npm exec vite build
failed to load config from /home/user/workspace/myapp/vite.config.js
error during build:
ReferenceError: __dirname is not defined in ES module scope
This file is being treated as an ES module because it has a '.js' file extension and '/home/user/workspace/myapp/package.json' contains "type": "module". To treat it as a CommonJS script, rename it to use the '.cjs' file extension.
    at file:///home/user/workspace/myapp/vite.config.js?t=1657918746154:6:22
    at ModuleJob.run (node:internal/modules/esm/module_job:198:25)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:409:24)
    at async loadConfigFromFile (file:///home/user/workspace/myapp/node_modules/vite/dist/node/chunks/dep-561c5231.js:62653:31)
    at async resolveConfig (file:///home/user/workspace/myapp/node_modules/vite/dist/node/chunks/dep-561c5231.js:62281:28)
    at async doBuild (file:///home/user/workspace/myapp/node_modules/vite/dist/node/chunks/dep-561c5231.js:43272:20)
    at async build (file:///home/user/workspace/myapp/node_modules/vite/dist/node/chunks/dep-561c5231.js:43261:16)
    at async CAC.<anonymous> (file:///home/user/workspace/myapp/node_modules/vite/dist/node/cli.js:747:9)

Problem
Use of __dirname in the provided vite.config.js conflicts with the subsequent recommended package.json which recommends "type": "module".
Proposed solution
Each documentation section should be internally consistent such that a new user can follow the recommendations in that section without getting sidetracked by finding solutions to problems the docs have baked in.
Here the docs should probably do one of the following:

Refrain from using __dirname when recommending use of "type": "module" in package.json
Don't recommend use of "type": "module" in package.json when using __dirname in vite.config.js
Include a work-around such as those listed here in vite.config.js

Related (but not duplicate) issues

https://github.com/vitejs/vite/issues/6946
https://github.com/vitejs/vite/issues/9109
https://github.com/nodejs/help/issues/2907

Reproduction
Included in bug description
System Info
Not really relevant, but

$ node --version
v18.5.0

<EMAIL_ADDRESS>
Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Hi @sodatea, thanks for taking a look at this issue! I'd be happy to open a PR if we can agree on which of the strategies I've outlined above (copied below) is preferred - i.e. most in line with the direction the docs should be moving.  (I don't have a opinion which is best.)

Refrain from using __dirname when recommending use of "type": "module" in package.json
Don't recommend use of "type": "module" in package.json when using __dirname in vite.config.js
Include a work-around such as those listed https://github.com/nodejs/help/issues/2907#issuecomment-858874660 in vite.config.js

This should be fixed in https://github.com/vitejs/vite/pull/9121 which isn't released yet. It will use esbuild to polyfill __dirname so it works here too.

🤔Though the bugs are fixed, I think we should refrain from using __dirname with type: module.
Not all environments have __dirname polyfilled. It's not a good practice anyway.

I agree too. We could probably make the change altogether for ts too since cts and mts are now supported, in the next major.


Mixnet PoC: Split message into packets + Reconstruction

I implemented a missing part in the PR #288,

splitting a message into several Sphinx packets (with padding)
reconstructing the message by gathering all Sphinx packets related

This is necessary for measuring the realistic latency amplification of mixnet.
TODO:

Encrypting the actual payload for the destination mixnode in the route. (Currently, only Sphinx encapsulations are encrypted for each mixnodes).
Refactoring

I’ll open a new PR based on the design written in Notion: https://www.notion.so/Mixnet-Architecture-613f53cf11a245098c50af6b191d31d2


[bug] Break tags moved around

This happens when I place a
 tag between a closing </columns> and </row>.  It is moved above the opening
 in the  parent element.  Weirdly enough, this doesn't occur when placed between the closing </row> and </container>.

Can you post a full code sample to help us recreate it?

In /pages/index.html
<container>
  <row>
    <columns small="12" large="12">

Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod
      tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam,
      quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo
      consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse
      cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non
      proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

    </columns>

  </row>
</container>

This may be a browser specific bug and might not carry over to the emails.  In Firefox and Chrome they both move the
 tag but the compiled code has it in the correct spot.

You wouldn't do this though would you? It's invalid markup

        Some content

@JayBizzle Yeah, if it's invalid markup then some browsers might handle it differently, in which case there's not much we can do about it.
Maybe @tdhartwick can chime in on a better way to create vertical space between elements. I'm guessing that's what you're trying to do?

@JayBizzle very good point, if it's invalid markup then this is irrelevant.  Closing the issue!


worker-html.js couldn't be loaded [Angular 6]

Hi, I'm tying to use Ace in my Angular 6 application, and have an error in console:
GET http://localhost:4200/worker-html.js 404 (Not Found)
blob:http://localhost:4200/7b4eb2a2-2fe3-440c-aa09-b942beec5f0c:1 Uncaught
(anonymous) @ blob:http://localhost:4200/7b4eb2a2-2fe3-440c-aa09-b942beec5f0c:1

in  blob:http://localhost:4200/7b4eb2a2-2fe3-440c-aa09-b942beec5f0c:1, I've found:
importScripts('http://localhost:4200/worker-html.js');

It's interesting that the component looks like it's working.
Copying assets works somehow in angular-cli. I've tried to make a manual copy of this file (just for testing - angular-cli can't copy assets from node_modules directory), but it's not working properly. Maybe I'm doing something wrong.
My options in angular.json:
          "options": {
            "outputPath": "dist/app",
            "index": "src/index.html",
            "main": "src/main.ts",
            "polyfills": "src/polyfills.ts",
            "tsConfig": "src/tsconfig.app.json",
            "assets": [
              "src/favicon.ico",
              { "glob": "worker-html.js", "input": "src/assets/worker-html.js", "output": "./worker-html.js" }
            ],
            "styles": [
              "node_modules/bootstrap/dist/css/bootstrap.css",
              "src/styles.css",
            ],
            "scripts": [
              "node_modules/ace-builds/src-min-noconflict/ace.js",
              "node_modules/ace-builds/src-min-noconflict/theme-eclipse.js",
              "node_modules/ace-builds/src-min-noconflict/ext-language_tools.js",
              "node_modules/ace-builds/src-min-noconflict/mode-html.js",
              "node_modules/ace-builds/src-min-noconflict/worker-html.js"
            ]
          }

In package.json:
  "dependencies": {
    "@angular/animations": "^6.0.3",
    "@angular/common": "^6.0.3",
    "@angular/compiler": "^6.0.3",
    "@angular/core": "^6.0.3",
    "@angular/forms": "^6.0.3",
    "@angular/http": "^6.0.3",
    "@angular/platform-browser": "^6.0.3",
    "@angular/platform-browser-dynamic": "^6.0.3",
    "@angular/router": "^6.0.3",
    "brace": "^0.11.1",
    "core-js": "^2.5.4",
    "ng2-ace-editor": "^0.3.8",
    "rxjs": "^6.0.0",
    "zone.js": "^0.8.26"
  },

Component code is just the default in documentation:
import {Component, ViewChild} from '@angular/core';

//to use theme eclipse
//with angular-cli add "../node_modules/ace-builds/src-min/ace.js"
//and "../node_modules/ace-builds/src-min/theme-eclipse.js" to "scripts" var into the file angular-cli.json

@Component({
    template: `
  <ace-editor
       [(text)]="text" // possible two way binding (thx ChrisProlls)
        #editor style="height:150px;"></ace-editor>
  `
})
export class AceCmp {
    @ViewChild('editor') editor;
    text: string = "";

    ngAfterViewInit() {
        this.editor.setTheme("eclipse");

        this.editor.getEditor().setOptions({
            enableBasicAutocompletion: true
        });

        this.editor.getEditor().commands.addCommand({
            name: "showOtherCompletions",
            bindKey: "Ctrl-.",
            exec: function (editor) {

            }
        })
    }
}

How can I get rid of this error in console?
Thanks.

In example of Angular 6 application there are four warnings:
Could not load worker TypeError: e.split is not a function
    at a.t.moduleUrl (ace.js:1)
    at new f (ace.js:1)
    at Mode.createWorker (html.js:2405)
    at d.$startWorker (ace.js:1)
    at d.$onChangeMode (ace.js:1)
    at d.<anonymous> (ace.js:1)
    at a.t.loadModule (ace.js:1)
    at d.setMode (ace.js:1)
    at AceEditorComponent.push../node_modules/ng2-ace-editor/src/component.js.AceEditorComponent.setMode (component.js:123)
    at AceEditorComponent.set [as mode] (component.js:112)

In example of Angular 6 application there are four warnings:
Failed to load resource: the server responded with a status of 404 (Not Found)
blob:http://localhost:4200/b5ac2005-9cfb-4fb9-b277-e421b2ca0a5e:1 Uncaught
(anonymous) @ blob:http://localhost:4200/b5ac2005-9cfb-4fb9-b277-e421b2ca0a5e:1

did anyone find a solution to this errors , i'm trying to integrate ace editor in my angular 6 application and i keep getting this 4 errors @hamfatfat talked about

I think that @ori-sh  close this issue - maybe this should be added into Readme.md

{ "bundleName": "worker-json", "input": "node_modules/ace-builds/src-min/worker-json.js" } - This does't work in production build.

{ "bundleName": "worker-json", "input": "node_modules/ace-builds/src-min/worker-json.js" } - This does't work in production build.
try
"assets": [     "src/favicon.ico",     "src/assets",     "src/manifest.json",     { "glob": "worker-javascript.js", "input": "./node_modules/ace-builds/src-min/", "output": "/" },     { "glob": "worker-css.js", "input": "./node_modules/ace-builds/src-min/", "output": "/" },     { "glob": "worker-json.js", "input": "./node_modules/ace-builds/src-min/", "output": "/" } ]

Just :
"assets": [     "src/favicon.ico",     "src/assets",     "src/manifest.json",     { "glob": "worker-json.js", "input": "./node_modules/ace-builds/src-min/", "output": "/" } ]
is of for me!
Thanks a lot ;-)

I am facing issues around trying to make Ace work with Angular for few days now. More than the problem, I am struggling to understand what is causing the problem. Could someone please explain what is causing the issue (either with worker-.js or at time the mode-.js files.

I finally was able to make my code work.  My setup is different. I build my Angular application and the it is served from my Play server. The angular build is stored in Play's /public/ui folder. The requests should be in format /assets/ui/.. which gets mapped to /public/ui/... due to a rule in Play routes file
GET     /assets/*file               controllers.Assets.versioned(path="/public", file: Asset)
When I ran the code, I got error.
Uncaught DOMException: Failed to execute 'importScripts' on 'WorkerGlobalScope': The script at 'http://localhost:9000/worker-javascript.js' failed to load.
at blob:http://localhost:9000/3df21e42-fecb-4026-8bd6-f2b0d1d0540a:1:1
Earlier, I also got error Unable to infer path to ace from script src, use ace.config.set('basePath', 'path') to enable dynamic loading of modes and themes or with webpack use ace/webpack-resolver
It seems ng-ace-editor imports .js scripts (theme, mode, worker) based on the theme and mode of the editor. The theme and mode js files can be included in scripts.js but some worker-.js files can't be (I don't know why, maybe because worker ones are loaded dynamically using importScript.
The scripts section in Angular.json is (this will all get bundled in scripts.js in Angular's final bundle)
"scripts": [
"./node_modules/ace-builds/src/ace.js",
"./node_modules/ace-builds/src/theme-eclipse.js",
"./node_modules/ace-builds/src/theme-monokai.js",
"./node_modules/ace-builds/src/mode-html.js"
]]
To include worker-.js files, I added this rule because it angular-cli can't load from node_modules. So I had to copy the files from node modules to root of my ui build - https://stackoverflow.com/questions/41555624/how-to-include-assets-from-node-modules-in-angular-cli-project
"assets": [
"src/assets",
"src/favicon.ico",
{
"glob": "**/*",
"input": "./node_modules/ace-builds/src/",
"output": "/"
}
],
When I executed the code, I found error that http://localhost:9000/worker-javascript.js can't be loaded. I realised that my files are loaded in /assets/ui/ path and not in the server's root directory.  So I set the basepath to /assets/ui in the component's .ts file
import * as ace from 'ace-builds/src-noconflict/ace';
ace.config.set('basePath', '/assets/ui/');
ace.config.set('modePath', '');
ace.config.set('themePath', '');
In summary
basePath seem to be what is used to load scripts dynamically (eg worker scripts)
modePath and themePath are / as the mode and theme scripts are bundled in scripts.js and are available at root level
need to copy worker-.js files outside node_modules as angular_cli can't copy assets from node_modules


Feat: Filter genuine email

What feature?
Security Feature
In your application anyone can enter using temporary or disposable email.
I will add a feature so that no one could create an account using temporary or disposable email.
It will allow only genuine users in your database.
It will help to avoid any bot.
Assign me this work under GSSoC24, I will implement it.
Add screenshots
.
Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue

Hi @pankaj-bind
Can you tell me in brief how are you going to limit this?

Hi @pankaj-bind Can you tell me in brief how are you going to limit this?

It works in frontend, it does not affect backend, works as a filter which allows only genuine emails.
I will use an API which is free it will first checks is the email genuine or not, if yes then it proceeds otherwise, breaks operation and shows a popup "invalid email"
I have tried this, and this feature is very unique.

Assugned to you

Reopening the issue as it increases the quality of the application


gem install nokogiri fails on OS X El Capitan: no implicit conversion of nil into String

(Follow up from https://github.com/sparklemotion/nokogiri/issues/1504)
OS X v10.11.5 (15F34)
Xcode Version 7.3.1 (7D1014)
Ruby v2.3.1p112
Homebrew 0.9.9
RubyGems v2.5.1
$ sudo gem install nokogiri
Password:
Building native extensions.  This could take a while...
ERROR:  While executing gem ... (TypeError)
    no implicit conversion of nil into String

Couldn't find an mkmf.log file:
$ find ~/ -name mkmf.log
$

Can you please show the output from gem install -V nokogiri?

fixed via https://github.com/rubygems/rubygems/issues/1668


Custom type url

This set of changes is preparation for using type_url file option when packing into Any.

@AlexanderLitus, please address TODOs added in the changes of this PR.
Please also notice TODOs in the EntityPackages class and let's discuss.

[Current coverage][cc-pull] is 75.55%

Merging [#161][cc-pull] into [master][cc-base-branch] will increase coverage by 0.29%

@@             master       #161   diff @@
==========================================
  Files           128        129     +1
  Lines          4551       4606    +55
  Methods           0          0
  Messages          0          0
  Branches        497        497
==========================================
+ Hits           3425       3480    +55
- Misses         1022       1023     +1
+ Partials        104        103     -1

Powered by Codecov. Last updated by [6e9e172...9e7f922][cc-compare]
[cc-base-branch]: https://codecov.io/gh/SpineEventEngine/core-java/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/SpineEventEngine/core-java/compare/6e9e1725c46345ae5e3d2ed3b53b8608071b9d15...9e7f92247aba45faceea5698d30697400416ba6e
[cc-pull]: https://codecov.io/gh/SpineEventEngine/core-java/pull/161?src=pr

PTAL.

Renaming getter methods in KnownTypes was a right thing! Now the code reads better.
LGTM.


chore: merge latest dev branch

Description
What this PR does / why we need it:

Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Update latest changes in dev branch
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Helm Chart Change (any edit/addition/update that is necessary for changes merged to the main branch)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] Does the affected code have corresponding tests?
[ ] Are the changes documented, not just with inline documentation, but also with conceptual documentation such as an overview of a new feature, or task-based documentation like a tutorial? Consider if this change should be announced on your project blog.
[ ] Does this introduce breaking changes that would require an announcement or bumping the major version?
[ ] Do all new files have appropriate license header?

Post Merge Requirements

[ ] MAINTAINERS: manually trigger the "Publish Package" workflow after merging any PR that indicates Helm Chart Change

Will rebase instead


Use "odd numbers" to represent integer values

We have settled on representing all values using (32-bit) words; these may either represent literal values where possible, as is the case with integers, enums, and other small primitve types, or pointers to heap-allocated objects.
The memory manager needs to distinguish between these types of values because it must be able to recursively drop pointers embedded in heap-allocated structures. Whether a heap-allocated structure contains pointers can depend on the variant (e.g., Some bigObj : Maybe BigObj contains a pointer to BigObj while Nothing : Maybe BigObj does not, so this determination must be made at runtime.
Since pointers will always point to word-aligned values in the heap, we are guaranteed that the LSB is 0 for pointers. We can safely represent up to 31 bits of information by value, as long as we encode them such that the LSB for these values is always 1. For example. to represent an integer value x, we encode it as x << 1 | 1.
As a first step toward using words to represent a mix of values and pointers, we need to first adjust codegen to encode values using this scheme. It also needs to decode values back into integers to perform arithmetic before re-encoding them.

Update on progress!

I implemented marshaling/unmarshaling of SSLANG integers in Codegen.hs
and incorporated feedback from my draft PR.

I added a regression test called 31bitInts.ssl

Things I have tested and that work in 31bitInts.ssl:
integer literals
parameters of reference type
addition and subtraction of (1) and (2) together

Things I would like to test:
Non-reference parameters to functions
Local variables of both reference and non-reference type
Bit Not
Multiplication and division

Errors I'm getting trying to implement these tests:
main ( cout : & Int, x : Int) -> () = //pattern match error
/*sslc: pattern should be desguared into pattern match
CallStack (from HasCallStack):
  error, called at src/IR/LowerAst.hs:176:18 in sslang-<IP_ADDRESS>-KI8f4RQ6phR54Leprkbw8R:IR.LowerAst*/

let x = (5:Int) //should already be desugared
/*sslc: Should already be desugared
CallStack (from HasCallStack):
  error, called at src/IR/LowerAst.hs:133:32 in sslang-<IP_ADDRESS>-KI8f4RQ6phR54Leprkbw8R:IR.LowerAst*/

after 10 , (cout : & Int) <- (~39):Int //syntax error
/*sslc: 6:33    [241+1] :Syntax error
CallStack (from HasCallStack):
  error, called at .stack-work/dist/x86_64-linux-tinfo6/Cabal-<IP_ADDRESS>/build/Front/Parser.hs:1199:5 in sslang-<IP_ADDRESS>-KI8f4RQ6phR54Leprkbw8R:Front.Parser*/

after 10 , (cout : & Int) <- (108*1):Int
/*sslc: Prelude.head: empty list*/

Awesome!
For the errors you're getting:

for passing multiple parameters, just pass them separately for now, i.e., main (cout: &Int) (x: Int) -> (). We don't have structured binding just yet since we're blocked on pattern matching in the front end.
I'm not sure what the "should already be desugared" bug is. But you shouldn't need that type annotation.
we don't have any prefix operators right now other than & as a type operator. I'm not sure how best to fix that easily; you could add a bnot function and then fish for that in the lowering phase, but that might just add to bloat that shouldn't be there in the first place.
not quite sure what that's about either. Did you add in the typing rule for multiplication?

Fixed by #41


bloating file size

Just opening a .sav file and saving it immediately after bloats the file by roughly 2x. Why is that?
The game still opens it correctly.

@Swarkin could you please zip your sav file and attach it here so we can take a look? Thanks for the report!

This is the original file, produced by the game:
original.zip
This is the file after running the following code:
	let mut file = File::options()
		.read(true).write(true).open(PATH).unwrap();

	let gvas_file = GvasFile::read(&mut file, GameVersion::Default).unwrap();
	gvas_file.write(&mut file).unwrap();

processed.zip
More than doubling file size. The game surprisingly still reads the contents fine, my in-game inventory is populated correctly.

Hold on, it was an issue on my end. I was using the same file handle to read and write, expecting its contents to be overwritten instead of appended.
Sorry for that


can you provide some pretrained deblurring models?

Good work!
can you provide some pretrained deblurring models?
thanks !!!

Thanks for your attention and interest, we will release the trained checkpoints as quickly as possible.

I think this is what I'm looking for as well.  I'd like to try out all the video deblurring methods on a test clip of mine.  I'm a bit confused about how this would be done currently.   I've set up the conda env and installed and compiled all the dependencies but I can't seem to find the python script with arguments to simply try out a model of a video or image sequence.  Would you be able to provide a step by step on how this is done?

Hi, in fact, I provide a simple demo notebook on Colab, and you can run it to make a try quickly.


[Bug] Recursion & Stack Overflow in the VM

We need to review recursions in our VM code to ensure that they cannot cause stack overflow under any circumstances. This means either no stack overflow for any given inputs, or explicitly state that certain inputs that could cause stack overflow are prohibited by some internal invariants.
We also need to think about how gas comes to play with this. Also consider OOM.
Cases to Review:

[ ] During deserialization, a large number of SignatureToken::Reference(Reference(Reference(... can cause a stack overflow.

Would a sufficiently smart fuzzer be able to catch this?

Would a sufficiently smart fuzzer be able to catch this?
I think so

@sunshowers this issue was caught by your harness test btw (compiled_module)

With the help from @dtolnay, I explored this topic again today and came up with some code to drop a deep AST using heap space. Posting it here for future reference:
#[derive(Debug)]
enum Exp {
    Num(u64),
    Neg(Box<Exp>),
    Add(Box<Exp>, Box<Exp>),
}

impl Exp {
    fn is_trivial(&self) -> bool {
        match self {
            Exp::Num(_) => true,
            _ => false,
        }
    }
}

fn reduce(sink: &mut Vec<Exp>, e: &mut Exp) {
    match e {
        Exp::Num(_) => (),
        Exp::Neg(e) => {
            sink.push(std::mem::replace(&mut **e, Exp::Num(0)));
        }
        Exp::Add(e1, e2) => {
            sink.push(std::mem::replace(&mut **e1, Exp::Num(0)));
            sink.push(std::mem::replace(&mut **e2, Exp::Num(0)));
        }
    }
}

impl Drop for Exp {
    fn drop(&mut self) {
        let mut stack = vec![];
        reduce(&mut stack, self);

        while let Some(mut e) = stack.pop() {
            reduce(&mut stack, &mut e);
        }
    }
}

#[test]
fn test() {
    let mut e1 = Exp::Num(0);
    for _ in 0..200000 {
        e1 = Exp::Neg(Box::new(e1));
        e1 = Exp::Add(Box::new(Exp::Num(3)), Box::new(e1));
    }

    let mut e2 = Exp::Num(0);
    for _ in 0..200000 {
        e2 = Exp::Neg(Box::new(e2));
        e2 = Exp::Add(Box::new(e2), Box::new(Exp::Num(3)));
    }

    let e = Exp::Add(Box::new(e1), Box::new(e2));
}

The caveat is that you can no longer destruct an expression via pattern matching:
    match e {
        Exp::Add(e1, e2) => (),
        Exp::Num(_) | Exp::Neg(_) => unreachable!(),
    }

error[E0509]: cannot move out of type `Exp`, which implements the `Drop` trait
  --> reference/src/lib.rs:58:11
   |
58 |     match e {
   |           ^ cannot move out of here
59 |         Exp::Add(e1, e2) => (),
   |                  --  -- ...and here
   |                  |
   |                  data moved here
   |
   = note: move occurs because these variables have types that don't implement the `Copy` trait

However this is fine:
   match e {
        Exp::Add(_, _) => (),
        Exp::Num(_) | Exp::Neg(_) => unreachable!(),
    }

I'm actually a little surprised that _ has different semantics than a named variable here.

@dtolnay you mentioned to me the lang team was interested in an RFC improving this. Was it related to this conversation you were in: https://internals.rust-lang.org/t/pre-rfc-destructuring-values-that-impl-drop/10450 ? Any updates on that pre-RFC?


Unavailable position-based subscript setter crashes on runtime

After upgrading from TCA's IdentifiedArray to the version in this repo (0.1.1), my app is crashing in places where it used the position-based subscript setter.
There's a @available annotation in the code, but that doesn't show up (at least not in Xcode 12.5). It's the fatalError in the setter that hits.
The weird thing is that the error did show up in another place in my code.
I think it's unfortunate that has become a runtime issue, but I'm not sure how to fix that. I'm also not sure why the setter can't be implemented. I can live without it (I only use it in tests), but it was convenient to have.

Looks like this comes from a restriction of the underlying OrderedDictionary offset-based subscript. That one doesn't have a setter either. However, the docs seems to imply it should be used for in-place mutation...

I wrote a very naive implementation that works in my app: https://github.com/Thomvis/swift-identified-collections/commit/f6561f9ef2feb179b64c8aed7a6ee0b7570d8949.
Do you think something like this could be added to the library?

@Thomvis Dang, yeah I hit that exact issue. It appears to be a compiler bug 😞
We eliminated the index-based setter because we think it opens up the opportunity to create invalid collections, e.g. if you have an element at 0 identifying as "deadbeef" and then you try to insert another element at 1 that also identifies as "deadbeef". Swift Collections' OrderedSet type has the same design.
So, instead of adding _modify to the subscript, can you use the id-based subscript instead? For example:
mage.resources[id: mage.resources[0].id].used = 1

I think what we'll probably want to do in the library is remove the deprecation entirely, so that the compiler can catch the issue. It's a bit of a bummer that we can't offer a better migration path, but this compiler bug is out of our control!

@Thomvis we plan on removing this setter in #15, but wanted to make sure you could still mutate things via the other subscript.

@stephencelis yes, that works. I’ll probably end up creating a “idPosition” subscript helper in my test target to make it a bit easier. In all/most other cases, using the mutable position-based subscript was a mediocre choice anyway. Thanks!

Fixed in #15.


mpd service: remove user/group from conf

Motivation for this change
The options should not be set as we already change user with service
file, man mpd.conf says "Do not use this option if you start MPD as an
unprivileged user"
The group option actually is not documented at all anymore and probably
no longer exists.
These options get in the way of setting up confinement for the service,
as it would otherwise be pretty straightforward to setup, but even if
mpd is not root it would check the user exists within the chroot which
is more work (need to get nss working):
  systemd.services.mpd = {
    serviceConfig.BindPaths = [
      # mpd state dir
      "/var/lib/mpd"
      # notify systemd service started up
      "/run/systemd/notify"
    ];
    serviceConfig.BindReadOnlyPaths = [
      "/path/to/music:/var/lib/mpd/music"
    ];
    # ProtectSystem is not compatible with confinement
    serviceConfig.ProtectSystem = lib.mkForce false;
    confinement = {
      enable = true;
      binSh = null;
      mode = "chroot-only";
    };
  };

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

cc mpd maintainers @tobim @fpletz @ehmry @astsmtl

Nit: since this touches nixos the commit prefix should be "nixos/mpd: ...". But I'm gonna merge anyway.

Nit: since this touches nixos the commit prefix should be "nixos/mpd: ...". But I'm gonna merge anyway.

Ok for commit message, will adjust 2nd mpd pull request I have open.
Thanks!


Remove contains and iter from PyMappingProtocol.

The methods are not expected by CPython and are only explicitly
callable. To get iteration support, PyIterProtocol should be
implemented and to get support for x in mapping,
PySequenceProtocol's contains should be implemented.
https://github.com/PyO3/pyo3/issues/611
Thank you for contributing to pyo3!
Here are some things you should check for submitting your pull request:

[x] Run cargo fmt (This is checked by travis ci)
[x] Run cargo clippy and check there are no hard errors (There are a bunch of existing warnings; This is also checked by travis)
[x] If applicable, add an entry in the changelog.
[x] If applicable, add documentation to all new items and extend the guide.

Could be nice to eventually add a Rust implementation of a PyDict under examples.

[x] If applicable, add tests for all new or fixed functions

You might want to run tox (pip install tox) locally to check compatibility with all supported python versions. If you're using linux or mac you might find the Makefile helpful for testing.

Thanks!
Looks CI fails because of rustfmt check.
Could you please run cargo fmt on your local machine?

Sure, amended the commit!
Looks like I forgot to run that after removing the defs from the derive backend.

Thanks, again.


rdpwrap.ini for Win10 x64 Ver.10.0.19041.1202 Build 19042.1237

rdpwrap.ini.txt

升级后，我的也有相同的问题。希望能得到大神指引。谢谢！
After update, I have the same problem, below the rdpwrap.ini I am using. But seems not working. Hope can get help. Thanks!
rdpwrap.zip

升级后，我的也有相同的问题。希望能得到大神指引。谢谢！
After update, I have the same problem, below the rdpwrap.ini I am using. But seems not working. Hope can get help. Thanks!
rdpwrap.zip

你这个已经安装完成了，重启电脑或者在服务里面重启远程桌面服务就可以了

升级后，我的也有相同的问题。希望能得到大神指引。谢谢！
After update, I have the same problem, below the rdpwrap.ini I am using. But seems not working. Hope can get help. Thanks!
rdpwrap.zip

升级后，我的也有相同的问题。希望能得到大神指引。谢谢！
After update, I have the same problem, below the rdpwrap.ini I am using. But seems not working. Hope can get help. Thanks!
rdpwrap.zip

端口号：3389
不是6868
设置完，点击OK，系统重启应该就好使了。

Same problem here... 10.0.1904.1202... It says "[not supported"]


⚠ Bofrost.de has degraded performance

In 0797374, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1090 ms

Resolved: Bofrost.de performance has improved in 9702cf8 after 6 minutes.


ACL definition for screens

Each screen has to be properly cataloged. Not all the screens have to be seen by all roles, so it has to be defined and stated.

We need the flow of each role in order to decide which are the apropiate role to see a specific screen.
It has to be defined for @CloudOpting/sdggroup

We will imagine/propose the screens for each role.

Please find in the following link a first proposal for the screens of the UI to be seen by the different roles of CloudOpting (SS, SP & PP).
All comments are welcome.

WP5_Web screens of CloudOpting.docx

Seems that this already is in place. Closing it.


[BUG] "Could not determine kind of name for ...." when installing

Describe the bug
Error when installing the package via "go get"
To Reproduce
go get github.com/gotk3/gotk3/gtk
Expected behavior
Package installation success
Error messages
/root/go/src/github.com/gotk3/gotk3/glib/glib_since_2_42.go:30:44: could not determine kind of name for C.GNotificationPriority
/root/go/src/github.com/gotk3/gotk3/glib/glib_since_2_42.go:24:54: could not determine kind of name for C.G_NOTIFICATION_PRIORITY_HIGH
/root/go/src/github.com/gotk3/gotk3/glib/glib_since_2_42.go:23:54: could not determine kind of name for C.G_NOTIFICATION_PRIORITY_LOW
/root/go/src/github.com/gotk3/gotk3/glib/glib_since_2_42.go:22:54: could not determine kind of name for C.G_NOTIFICATION_PRIORITY_NORMAL
/root/go/src/github.com/gotk3/gotk3/glib/glib_since_2_42.go:25:54: could not determine kind of name for C.G_NOTIFICATION_PRIORITY_URGENT
/root/go/src/github.com/gotk3/gotk3/glib/glib_since_2_42.go:39:7: could not determine kind of name for C.g_application_get_resource_base_path
/root/go/src/github.com/gotk3/gotk3/glib/glib_since_2_42.go:49:2: could not determine kind of name for C.g_application_set_resource_base_path
/root/go/src/github.com/gotk3/gotk3/glib/glib_since_2_42.go:30:2: could not determine kind of name for C.g_notification_set_priority
Environment:
gtk3 version: 3.10.8-0ubuntu1
go version: go1.14.4 linux/amd64
os:Ubuntu 14.04 LTS

what is your glib version?
which gotk3 version do you use?

Ubuntu 14.04 LTS has glib version 2.40 -> did you use build tag glib_2_40? I think that's why it crashes

I just do this, nothing else : go get github.com/gotk3/gotk3/gtk
How to use that build tag?

when you do go install and go build, you need to use the -tags parameter with the input you need, e.g. -tags gtk_3_16 glib_2_40
look at the source files (e.g. https://github.com/gotk3/gotk3/blob/master/glib/glib_since_2_42.go). If they mention tags at the top and you want to use their functionalities, you need to add that tag version (or a higher one).

Please close the issue if everything is resolved, or provide more details on your build pipeline...


SDK update issue at Android Studio 1.0

I'm facing an issue while update Chat_SDk 4.+ to new Version 5.v.
My Android Studio is Latest 4.0.
Please check screenshot

@MOhsain can you show me what you added to your grade file?

i added these lines in project level build.gradle file ->
classpath "com.android.tools.build:gradle:4.0.0"
classpath "com.google.gms:google-services:4.3.3"
classpath 'com.novoda:bintray-release:0.9.2'
classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.71"
classpath 'com.jakewharton:butterknife-gradle-plugin:10.2.1'
classpath 'com.google.firebase:firebase-crashlytics-gradle:2.1.1'
classpath 'com.google.firebase:perf-plugin:1.3.1'
On Wed, Jun 17, 2020 at 7:44 PM Ben Smiley<EMAIL_ADDRESS>wrote:

@MOhsain https://github.com/MOhsain can you show me what you added to
your grade file?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chat-sdk/chat-sdk-android/issues/632#issuecomment-645418654,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFLC2XM6UYMH5XLKANWZUALRXDJDRANCNFSM4OATLRLA
.

@MOhsain could you plase show me both your project and app level build.gradle files?

How can I show all my projects to you?
On Thu, Jun 18, 2020 at 12:30 PM Ben Smiley<EMAIL_ADDRESS>wrote:

@MOhsain https://github.com/MOhsain could you plase show me both your
project and app level build.gradle files?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chat-sdk/chat-sdk-android/issues/632#issuecomment-645835499,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFLC2XIUJ63TZGCXZQI2DP3RXG675ANCNFSM4OATLRLA
.

You need to clean the project (delete the Build directories) and then zip it and email it to me<EMAIL_ADDRESS>Or invite me to your Git repo if you're using one. My username is ben_smiley.

Why are you closing this?
On Thu, Jun 25, 2020 at 5:28 PM Ben Smiley<EMAIL_ADDRESS>wrote:

Closed #632 https://github.com/chat-sdk/chat-sdk-android/issues/632.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chat-sdk/chat-sdk-android/issues/632#event-3482145554,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFLC2XLB7TPSVZO5F3ZRX6LRYM7GJANCNFSM4OATLRLA
.

Because I posted a reply a week ago and got no reply. We need to solve it or close the issue.

I would also recommend looking at this video:
https://www.youtube.com/watch?v=ZzfSd3hc3xw
I started with a clean Android Studio project and setup the Chat SDK.

But you are using Android Studio Version 3.6 and my Version is 4.0. I also
sent my code base to your given email address. Please check that
On Thu, Jun 25, 2020 at 6:02 PM Ben Smiley<EMAIL_ADDRESS>wrote:

I would also recommend looking at this video:
https://www.youtube.com/watch?v=ZzfSd3hc3xw
I started with a clean Android Studio project and setup the Chat SDK.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chat-sdk/chat-sdk-android/issues/632#issuecomment-649525970,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFLC2XIWEEZPCJCJ6D2TCJLRYNDEXANCNFSM4OATLRLA
.

Please reply to me?
On Mon, Jun 29, 2020 at 10:30 AM Mohsan Ikram<EMAIL_ADDRESS>wrote:

But you are using Android Studio Version 3.6 and my Version is 4.0. I also
sent my code base to your given email address. Please check that
On Thu, Jun 25, 2020 at 6:02 PM Ben Smiley<EMAIL_ADDRESS>wrote:

I would also recommend looking at this video:
https://www.youtube.com/watch?v=ZzfSd3hc3xw
I started with a clean Android Studio project and setup the Chat SDK.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chat-sdk/chat-sdk-android/issues/632#issuecomment-649525970,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFLC2XIWEEZPCJCJ6D2TCJLRYNDEXANCNFSM4OATLRLA
.

Hi @MOhsain this is an issue directly related to using Android Studio 4.0. It's related to this. I will update the project as soon as AS 4.0 comes out of beta. In the meantime, I would recommend that you use: AS 3.x, add the Chat SDK using Gradle and remove the bintray release dependency or follow the steps in the issue I posted.


config option optional is not validated

This option is effectively an enum but right now it's just string-typed, so if you do optional: pointerz it will effectively be equivalent to optional: value (the default). We should validate it!

I've fixed this in #252


Fix link to Vue framework

What kind of change does this PR introduce?
Minor documentation fix in README.md
Did you add tests for your changes?
No. I performed a manual  functional test. Link was 404 before, it  works now.
If relevant, link to documentation update:

Summary

The links to the Vue framework loader's (Loads and compiles Vue Components) Github page was incorrect and leading to a 404.  I felt motivated to try and help by fixing this issue quickly.
Does this PR introduce a breaking change?
Nope.

Other information
Hope this helps.

Thanks

You're welcome!


C# + PythonNet use case

Hi, awesome repo! :)
I was testing web-pdb using C# + https://github.com/pythonnet/pythonnet , specifically calling Python from a C# process. Example: PythonEngine.RunSimpleString("import web_pdb; web_pdb.set_trace()");
My issue/feature request/advice request, is that web-pdb is closing when coming back from the Python code to the C# side, getting the *** Thread finished *** message from https://github.com/romanvm/python-web-pdb/blob/master/web_pdb/init.py#L166 __init__.py:dispatch_return()
and I'd like it to continue to live until the process ends or maybe the WebPdb instance is disposed/shutdown.
Thoughts?
Thanks!

It makes absolutely no sense to run the webserver part of web-pdb after the interpreter exits the top-most execution frame of the code being taced. There is nothing to debug after that.

The C# + PythonNet use case I'm facing is about embedding python in C#, so it's going back and forth multiple times from C# to Python, so the interpreter exits the top-most execution frame multiple times too.

Long story short: you cannot share a debugger instance between different Python interpreter instances.

I understand, but in this case, it will just be 1 interpreter instance, 1 initial call to Py_Initialize().
I've tested just commenting out the if statement at https://github.com/romanvm/python-web-pdb/blob/master/web_pdb/init.py#L165 and it worked correctly, but it was just a hack so wanted to know your thoughts


Xcode not found for -lnative_static

Hello Iam using this template but I got error when trying to build on Mac/IOS env.
After i clone the repo, run the flutter_rust_bridge_codegen generate command then the flutter run for IOS 13;
As a result I got

`Failed to build iOS app
Error (Xcode): Library not found for -lnative_static
Could not build the application for the simulator.
Error launching application on iPhone 13.`

Could not fix.
I tought flutter 3.6 not supported is it possible ?
flutter doctor

[✓] Flutter (Channel master, 3.6.0-5.0.pre.12, on macOS 12.3 21E230 darwin-x64, locale en-TR)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.3)
[✓] Android Studio (version 2021.1)
[✓] IntelliJ IDEA Community Edition (version 2020.3.1)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.1)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.1)
[✓] IntelliJ IDEA Ultimate Edition (version 2020.3.2)
[✓] VS Code (version 1.73.1)
[✓] Connected device (3 available)
[✓] HTTP Host Availability
• No issues found!

This template is only tested against the stable branch of Flutter, though this might indicate a future breakage.


Make it possible to link a Jira issue macro

Did not find any reference for this :( Is this possible please? If not would it be hard to implement? Any pointers more then welcome!

@thrix as a quick workaround, you can do build your personal custom publisher and inlcude the following macro to put a references to single jira issue.
require 'asciidoctor'
require 'asciidoctor/extensions'

class JiraIssue < Asciidoctor::Extensions::InlineMacroProcessor
  use_dsl

  named :jira
  name_positional_attributes 'server'

  def process parent, target, attrs
    key = target
    jira_macro = %(
    <ac:structured-macro ac:name="jira" ac:schema-version="1">
      <ac:parameter ac:name="key">#{key}</ac:parameter>
    </ac:structured-macro>
    )
    create_inline parent, :quoted, jira_macro
  end
end

P.S. did this for me, will publish forked version soon with it

Hi @zeldigas, thank you for the suggestion. Would it be an option for you to create a pull request before publishing a forked version of the Confluence Publisher? This way, the whole community would profit for the feature. I am happy to review the pull request. Thank you!

@cstettler  Sure, I don't want to create and maintain a full fledged fork. I just played with publisher in my personal space to get a list of features I'm missing: https://github.com/zeldigas/confluence-publisher/commit/4648442f433a6e90797006232938b5277d34ad2c
I'll try to transform them in separate PRs soon.

Thanks for the pointers, looking forward for the release with this ;)

Hey @zeldigas / @cstettler  is there any new information in regard of the feature?

@yuecels I was under impression that project is a bit abandoned, but looks like there is a recent activity, that even includes updates to asciidoctor 2.x.
That's great news and I'll create PR this week.

@zeldigas fine thanks 👍🏿

Or use a passthrough like this:
++++
<ac:structured-macro ac:name="jira">
<ac:parameter ac:name="columns">key,summary,type,created,updated,assignee,reporter,priority,status,resolution</ac:parameter>
<ac:parameter ac:name="jqlQuery">
project in (Project, Other project)
AND issuetype in (Story)
ORDER BY issuetype DESC, key ASC
</ac:parameter>
</ac:structured-macro>
++++

Hi @thrix, @zeldigas and others, thank you for your input and contributions, and sorry for the delay in reviewing, busy times ...
I am indeed a bit reluctant to support this issue / merge the corresponding pull request. Let me explain why: the main purpose of the Confluence Publisher is to support writing technical documentations in a "docs-as-code" approach in AsciiDoc while still being able to publish the documentation to Confluence, which is often a requirement in enterprises.
This said, it is not the goal to support every AsciiDoc feature, nor every Confluence feature / macro. We are trying to find the sweet spot between features supported in both worlds and maintenance efforts. Every extension to either AsciiDoc or asciidoctorj pose the risk to be non-compatible with future versions (or in worst case end up in a conflict with future AsciiDoc features), as of course every usage of any Confluence macro does. That's why the use case should be valuable enough.
Regarding the list of JIRA issues, I think the proposal using passthroughs from @Alvis-pixel is very sensible, as anyway the configuration of the query and columns is rather lengthy, which in comparison might make the additional surrounding markup acceptable.
What is your use case for referencing individual JIRA tickets in your documentation?

Hi @thrix, @zeldigas and others, thank you for your input and contributions, and sorry for the delay in reviewing, busy times ...
I am indeed a bit reluctant to support this issue / merge the corresponding pull request. Let me explain why: the main purpose of the Confluence Publisher is to support writing technical documentations in a "docs-as-code" approach in AsciiDoc while still being able to publish the documentation to Confluence, which is often a requirement in enterprises.
This said, it is not the goal to support every AsciiDoc feature, nor every Confluence feature / macro. We are trying to find the sweet spot between features supported in both worlds and maintenance efforts. Every extension to either AsciiDoc or asciidoctorj pose the risk to be non-compatible with future versions (or in worst case end up in a conflict with future AsciiDoc features), as of course every usage of any Confluence macro does. That's why the use case should be valuable enough.
Regarding the list of JIRA issues, I think the proposal using passthroughs from @Alvis-pixel is very sensible, as anyway the configuration of the query and columns is rather lengthy, which in comparison might make the additional surrounding markup acceptable.
What is your use case for referencing individual JIRA tickets in your documentation?

@cstettler We document our whole confluence space in asciidoc in git, including our landing page which includes are milestones tracked in Jira.
I will look at the passthrough, not sure how that works.

@cstettler We document our whole confluence space in asciidoc in git, including our landing page which includes are milestones tracked in Jira.
I will look at the passthrough, not sure how that works.

I'd say that usage of passthrough looks very ugly compared to macro. After all that's the exact idea of macro - to provide an abstraction that is easy to use and hide details.
Speaking about overal point about merging - I think that you need to state it more explicitly in README file then. Looks like you have some list of things that you consider to be out of scope but they are not sounded that let users false idea of missing feature, while this will not be accepted/merged and so on.
Personally for me publishing tool should be more featurefull than for you - just as @thrix I see a value to use it for maintaining pages with more level of interactivity and confluence specific features. Looks like I can't get all features I want by waiting for PRs to be merged, so eventually I'm considering just to have my own fork that is more featurefull and open to extensions..

I'd say that usage of passthrough looks very ugly compared to macro. After all that's the exact idea of macro - to provide an abstraction that is easy to use and hide details.
Speaking about overal point about merging - I think that you need to state it more explicitly in README file then. Looks like you have some list of things that you consider to be out of scope but they are not sounded that let users false idea of missing feature, while this will not be accepted/merged and so on.
Personally for me publishing tool should be more featurefull than for you - just as @thrix I see a value to use it for maintaining pages with more level of interactivity and confluence specific features. Looks like I can't get all features I want by waiting for PRs to be merged, so eventually I'm considering just to have my own fork that is more featurefull and open to extensions..


Make easy custom name for password field to laravel auth.

Recently i have a necessity for change the name of field password.  Change this filds is very caothic. What the best form of replace for this field name.

Hi there,
Welcome to Laravel and we are glad to have you as part of the community.
Unfortunately this GitHub area is not a support area for general application issues. This is only for issues/bugs with the framework code itself.
I will be closing your ticket here. Instead please try asking your question on one of the many great community support areas that will likely give you a better answer more quickly:

Laravel Slack (https://larachat.co/)
Laravel.io Forum (https://laravel.io/forum)
Laracasts Forum (https://laracasts.com/discuss)
StackOverflow (http://stackoverflow.com/questions/tagged/laravel)

If you feel I've closed this issue in error, please provide more information about how this is a framework issue, and I'll reopen the ticket.
Thanks in advance.


typo in MPI docs

Describe the bug
docs/userguide/mpi_apps.rst contains this typo of the word "note":

However, it is important to not that Python Apps that directly use mpi4py is not supported.

It should be important to note
To Reproduce
Read the user guide linked from the parsl website
To build the docs locally, look in docs/README.rst
Expected behavior
No typos

@benclifford can i work on this.

@benclifford I have created a PR for this issue #3162. Can you review it?


Prepare for release v4.0.0-rc2

Updates VERSION
Updates CHANGELOG

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
531

Relevant Lines:
531

💛  - Coveralls


BinaryExpr removing name label is not well-documented

What did you do?
Ran the following query:
(sum by(__name__) ({__name__=~"metric_.*"})) / (count by(__name__) ({__name__=~"metric_.*"}))
What did you expect to see?
A result
What did you see instead? Under which circumstances?
error evaluating query "(sum by(__name__) ({__name__=~\"metric_.*\"})) / (count by(__name__) ({__name__=~\"metric_.*\"}))" (line 13): found duplicate series for the match group {__name__="metric_10"} on the right hand-side of the operation: [{__name__="metric_10"}, {__name__="metric_1"}];many-to-many matching not allowed: matching labels must be unique on one side

Environment

System information:
n/a

Prometheus version:
2.10

Alertmanager version:
n/a

Test file

I was able to reproduce this in a test case, below is the file. I expected both sum by (a) and sum by (__name__) to behave similarly (both work or both don't work).
load 5m
    metric_1{a="1"} 1
    metric_10{a="10"} 10
    metric_1000{a="1000"} 1000
    metric_10000{a="10000"} 10000

eval instant at 5m (sum by(a) ({__name__=~"metric_.*"})) / (count by(a) ({__name__=~"metric_.*"}))
    {a="1"} 1
    {a="10"} 10
    {a="1000"} 1000
    {a="10000"} 10000

eval instant at 5m (sum by(__name__) ({__name__=~"metric_.*"})) / (count by(__name__) ({__name__=~"metric_.*"}))
    metric_1{} 1
    metric_10{} 10
    metric_1000{} 1000
    metric_10000{} 10000

count removes the name (as a count of a metric is no longer that metric), so this is as-expected.

Brian is right but with a quick scan I don't see that documented properly

Count should just count the unique labelsets in the result based on the Grouping. We can see this with the following more simple examples:
count(prometheus_http_requests_total) by (__name__) -> prometheus_http_requests_total{} 7
count(prometheus_http_requests_total) -> {} 7
(this was done on a prometheus host itself) -- so count can and does return labels such as __name__ in the current implementation.

If you explicitely group by d name then it will keep it. Otherwise it will drop it, which is expected.

Sure, and in the initial issue report here I am explicitly grouping by __name__ so it should keep it (which it does). So we are back to it being an issue with BinaryExprs it seems :)

Here is a simpler way to reproduce the issue:
http://demo.robustperception.io:9090/graph?g0.range_input=1h&g0.expr=sum({__name__%3D~"(prometheus_build_info|prometheus_http_requests_total)"})%20by%20(name)%20%2F%20count(%7B__name__%3D~%22(prometheus_build_info%7Cprometheus_http_requests_total)%22%7D)%20by%20(name)&g0.tab=1&g1.range_input=1h&g1.expr=sum(%7B__name__%3D~%22(prometheus_build_info%7Cprometheus_http_requests_total)%22%7D)%20by%20(name)&g1.tab=1&g2.range_input=1h&g2.expr=count(%7B__name__%3D~%22(prometheus_build_info%7Cprometheus_http_requests_total)%22%7D)%20by%20(name)&g2.tab=1
If you click through the link you can see that the sum() and count() both work, but dividing the 2 doesn't work (with the error I pasted initially). This is a bug since I should definitely be able to divide those 2.

Here is a simpler way to reproduce the issue:
http://demo.robustperception.io:9090/graph?g0.range_input=1h&g0.expr=sum({__name__%3D~"(prometheus_build_info|prometheus_http_requests_total)"})%20by%20(name)%20%2F%20count(%7B__name__%3D~%22(prometheus_build_info%7Cprometheus_http_requests_total)%22%7D)%20by%20(name)&g0.tab=1&g1.range_input=1h&g1.expr=sum(%7B__name__%3D~%22(prometheus_build_info%7Cprometheus_http_requests_total)%22%7D)%20by%20(name)&g1.tab=1&g2.range_input=1h&g2.expr=count(%7B__name__%3D~%22(prometheus_build_info%7Cprometheus_http_requests_total)%22%7D)%20by%20(name)&g2.tab=1
If you click through the link you can see that the sum() and count() both work, but dividing the 2 doesn't work (with the error I pasted initially). This is a bug since I should definitely be able to divide those 2.

Binary expressions also drop label names.
I suggest you do the following: http://demo.robustperception.io:9090/graph?g0.range_input=1h&g0.expr=sum(label_replace({__name__%3D~"(prometheus_build_info|prometheus_http_requests_total)"}%2C"__name"%2C"%241"%2C"__name__"%2C"(.)%22))%20by%20(name)%20%20%2Fcount(label_replace(%7B__name%3D~%22(prometheus_build_info%7Cprometheus_http_requests_total)%22%7D%2C%22__name%22%2C%22%241%22%2C%22__name__%22%2C%22(.)%22))%20by%20(name)%20&g0.tab=1&g1.range_input=1h&g1.expr=sum(%7B__name%3D~%22(prometheus_build_info%7Cprometheus_http_requests_total)%22%7D)%20by%20(name)&g1.tab=1&g2.range_input=1h&g2.expr=count(%7B__name__%3D~%22(prometheus_build_info%7Cprometheus_http_requests_total)%22%7D)%20by%20(name)&g2.tab=1
sum(label_replace({name=~"(prometheus_build_info|prometheus_http_requests_total)"},"__name","$1","name","(.)")) by (__name)  /count(label_replace({name=~"(prometheus_build_info|prometheus_http_requests_total)"},"__name","$1","name","(.)")) by (__name)

The reason is that binary expressions also alter metrics values.

But binary expressions don't always drop their labels, e.g. http://demo.robustperception.io:9090/graph?g0.range_input=1h&g0.expr=prometheus_http_requests_total %2F prometheus_http_requests_total&g0.tab=1
So under what circumstances does it break those? In this case there is 100% overlap in metrics on the LHS and RHS so if the error I was getting before was valid I'd expect to hit the same case here. so it seems that maybe BinaryExpr only removes __name__?

But binary expressions don't always drop their labels, e.g. http://demo.robustperception.io:9090/graph?g0.range_input=1h&g0.expr=prometheus_http_requests_total %2F prometheus_http_requests_total&g0.tab=1
So under what circumstances does it break those? In this case there is 100% overlap in metrics on the LHS and RHS so if the error I was getting before was valid I'd expect to hit the same case here. so it seems that maybe BinaryExpr only removes __name__?

We only remove __name__. That is what makes sense in those queries.

So I think I found that logic -- https://github.com/prometheus/prometheus/blob/d1c00bb7c2d15d6780e9f77b119dcc33f6e479b1/promql/engine.go#L2089 it seems to be only for some operators. So it seems to get the behavior I want I have to do the label rewriting magic you had described earlier.

Related: #11397


certificates: add K8s 1.8 impl

Towards https://github.com/giantswarm/giantswarm/issues/1384
We need to get rid of kubernetes 1.7 dependencies completely in azure-operator. we also need to fetch cluster certs there. This is an attempt to achieve that.
This is heavily inspired from current certificatetpr package which this package deprecates.

I don't find the diff useful (below I made a PR with the diff), but I'm happy
to exaplain what's going on. We can also meet.
Here's the diff: https://github.com/giantswarm/certificatetpr/pull/41/files
If you ignore vendor files the change there isn't much code.

error.go

Rather standard file.

spec.go

Simple interface.

types.go

Structs and constants. No methods, no functions.

searcher.go

First 60 lines are boilerplate: https://github.com/giantswarm/certificates/blob/fa53ddad1cfcdc8d1e487700c42325e176bd1bc3/searcher.go#L1-L61
search body remained mostly unchanged. case watch.Added body was replaced with fillTLSFromSecret call: https://github.com/giantswarm/certificates/blob/fa53ddad1cfcdc8d1e487700c42325e176bd1bc3/searcher.go#L101-L140
This is new: https://github.com/giantswarm/certificates/blob/fa53ddad1cfcdc8d1e487700c42325e176bd1bc3/searcher.go#L63-L99
Reading the secred and filling TLS struct is new: https://github.com/giantswarm/certificates/blob/fa53ddad1cfcdc8d1e487700c42325e176bd1bc3/searcher.go#L142-L168

searcher_test.go

Tests for fillTLSFromSecret.

I just realized this is a new project. So we have cert-operator, and the CertConfig type for our generated clientset. Why is this called certificates? I found it always odd to call everything "cert" but then having a "certificatetpr" project.

@xh3b4sd would certs be a better name for this project?

Not sure if it would be "better", but it would align better with the naming of the other projects.

s/better/preffered/. I'll rename the repo. It's not imported anywhere yet.


GEOMESA-1126 Fixing typos in metrics docs

Signed-off-by: Emilio Lahr-Vivaz<EMAIL_ADDRESS>

merging

887064e805e25253495ba44926dc8619b7a28441


loadshed records duration in different units

https://github.com/komuw/ong/blob/2b9e70a09c9a9f117548a49fd7bedb54397f29cf/middleware/loadshed.go#L57
The duration there will be an int64 but it will have different units like
dur:  12.866179ms
dur:  100.152µs

Instead we should use; https://github.com/komuw/ong/blob/2b9e70a09c9a9f117548a49fd7bedb54397f29cf/middleware/log.go#LL37C19-L37C51

Also audit all other durations in the codebase.


Request: Be able to delete save files/data points

I think the x-axis gets filled with every save file possible, meaning it gets stretched the more saves you have in a time period.
(I'm not 100% sure that it's really the case, but I think that's the case)
I'd like to be able to remove save files/data points so I can fix the x-axis, when I have a period ingame where I save very often.
Another solution (more or less) would be to only count autosaves in the first place.
Or even better: To not scale the x-axis by number of save files/data points but by the date itself directly. (meaning more saves in a period would increase accuracy without bloating the x-axis)

The x-axis is scaled by date, not data point count. The more saves you create the more data points that will be in Stellaru. You can reduce the shown date range by adjusting the slider in the top right of Stellaru. Does that cover it or have I misunderstood your request?

Funnily enough, I wanted to make a screenshot of the issue... but for some reason Stellaru forgot all the data of that save file. No idea what happnened, first time ever, I did nothing different, I opened Stellaru, loaded save file, and for some reason no past data points were there. Really weird.
But besides that issue...
Instead of a screenshot you now get this artistic piece. When you save more often the x-axis gets stretched because it has more data points. That means the data/graph gets stretched too.

I see what you mean. That should not be happening. I convert the date to a number of days and use that value for the x-axis, so there should be no distortion. May I ask how often you save in order to get this to occur? I'd like to be able to reproduce it.


Vertical text alignment is slightly off

Hi again,
I found the vertical text alignment to be slightly off when rendering text using RenderText  with a maxLineWidth.
This is also a TODO in the code. I have created PR#93 for this, but still wanted to create an issue.
Before the PR:

After the PR:

I have not checked / used justify as of yet.

fixed in #93


White Space Between Header and Content

I have my app and it’s working fine on iOS 10 but as iOS 11 is out now so i am testing on iOS 11. I am getting a strange issue. For the first view when we don’t touch anything on screen, design is fine but after we change something or update something i am getting white screen between header and content. And i am getting this issue only in iOS 11. Check the below screenshot:

I have posted this issue on ionic forum too.
https://forum.ionicframework.com/t/white-space-between-header-and-content/106733

Hello! Thank you for opening an issue with us!
Have you applied all of the changes mentioned here? https://blog.ionic.io/ios-11-checklist/
One note on that is that version 3.7.0 of the Ionic Framework is scheduled to be released sometime today, which you would then use instead of the nightly build that is mentioned. Otherwise, all of those instructions should still be valid.
If you still get the white bar after that, please provide a sample application via GitHub that demonstrates your issue so we are able to duplicate it for our fixing and testing.
Thank you for using Ionic

@MrPeproni I will also add that with iOS 11, we're pushing folks to upgrade to the new wkwebview plugin.
https://github.com/ionic-team/cordova-plugin-ionic-webview
As UIWebview has many issues, WKWebview fixes all of them. Including this odd gap in apps

@kensodemann Please check my ionic info
    ionic (Ionic CLI) : 3.12.0

global packages:

    cordova (Cordova CLI) : 7.0.1

local packages:

    @ionic/app-scripts : 1.3.7
    Cordova Platforms  : android 6.2.3 ios 4.4.0
    Ionic Framework    : ionic-angular 3.2.1

System:

    Android SDK Tools : 26.1.1
    ios-deploy        : 1.9.1
    ios-sim           : 5.0.12
    Node              : v6.11.0
    npm               : 3.10.10
    OS                : macOS Sierra
    Xcode             : Xcode 9.0 Build version 9A235

Misc:

    backend : legacy

@MrPeproni i had the same problem.
I was able to fix it adding WKWebView.
Here a guide that explain how to implement the change.

@centos94 Trying to install this.

As @centos94 and @mhartington have pointed out (and as is suggested in the blog post I previously linked), you should try WKWebView.
General comments on your ionic info which probably have nothing to do with this particular issue, but could save you time in the future.
Your version of the Ionic Framework (3.2.1) is old. At this point though, 3.7.0 is so close that I would wait until it is announced and then upgrade to that.
There will also be an upgrade to @ionic/app-scripts coming out that you will want to grab as well. That won't solve this issue, but you will want it.
Your cordova-ios plugin is slightly outdated. The latest is 4.5.1, but if you install that you will need to remove the deprecated console plugin. More info: http://cordova.apache.org/news/2017/09/22/plugins-release.html
That is all that sticks out at me from your ionic info

@kensodemann @centos94 @mhartington Thanks to all for the help. Installed WKWebView and issue is solved now.
But getting Preflight response is not successful error. I think backend developer can solve this.

@MrPeproni if backend developer can't solve that, in the page that i linked you before, there is a workaround at that problem, specifically at this point.

Glad to hear that took care of your issue. I am going to close this now. Thank you again for using Ionic.

I don't think this issue is solved, I am still seeing this both in regular web view and wk web view on ios 11 - with wk web view it does not appear immediately, but after the page is re-rendered (if you press a button etc). This issue also only appears if the scroll content is bigger than the content itself (if the page is scrollable).

@gregor-srdic can you provide a sample project and push it to github for testing. Please open a new issue as well.

in iOS 11.1 this issue has been fixed for me


Horizontal FL with custom partition

Hi,
I am currently trying out fedtree for horizontal FL on the adult dataset using the config parameters that you give in the example. The only difference is I partitioned the dataset differently to have an uneven label distribution (i'll call the partitions data_p0 and data_p1, libsvm files). I have 2 questions concerning that:

Is there a way to give the custom partitions to the fit(X,y) method in the python interface?
Using the config files I got an error in the first training round in the TriggerCalcTree method

Weirdly if I used data_p0 for both parties, the training ran error-free, similarly using data_p1. Only when giving data_p0 to party 0 and data_p1 to party 1 the error occurred.

Thanks for your help!

Hi @alx399 ,

No. Currently the python interface does not support custom partitions.
Can you provide the datasets so that I can debug? Thanks!

Hi @QinbinLi ,
I attached the 2 partitions. Thanks for your support!
a9a_iid_p0.txt
a9a_iid_p1.txt
The python interface is very nice, maybe you could add it as a feature.


fix: Fixed price cache initialising with 0

On some screens the price cache was sending down 0 because the values in the useEffect call were not updating in between the screen rendering and attempting to utilise the price cache in some operation.

@UncleSamtoshi @samerbuna I'm going to merge this without approval.  It's a small change and it would be good to include it in the release to avoid any bugs from cropping up (we already found one which @UncleSamtoshi fixed but there might be others we haven't found yet).


[@nstudio/nativescript-filterable-listpicker] Customize image

Hey guys! I'm having trouble customizing how my list looks, specially when customizing how the image looks. I am unable to set a width via .flp-image

I solved it myself vie PR, the image's width was always set to 30 so it was ignoring the CSS.


[Idea] ArangoDB for Wikidata

Wikidata has some scalling issues with its triplestore. The database is enormous and this type of engine does not support scalling so well. Could changing to a mixed model like ArangoDB be a key to resolve this issue ?
I opened the discussion here since it is not really what GitHub is meant for...
Wouldn't a mix database system that handle both JSON documents and graph functions, like ArangoDB, provide a better scalability to enormous knowledge graphs like Wikidata than a classical quadstore?

Hi,
The relation of ArangoDB and Tripplestores have been discussed on stackoverflow with the conclusion that there can be no direct automated translation - A conversion of the datamodel is required.
@smyth64 has been discussing this in
https://github.com/arangodb/arangodb/issues/1913#issuecomment-447632284 resulting in https://github.com/smyth64/arangodb-wikidata-importer. This could already be a good starting point for an eventual data model conversion.
The question to ask would probably be which queries to use this in, and how to best work with it.
A good starting point would probably be the ArangoDB Graph course.
Another topic covered in the linked article is search. Since now ArangoSearch is available - a good question could be how it could be included here.

Hi @MorganeAlinai,
has dothebart's comment been helpful?
Do you have any further questions?

@MorganeAlinai
I am closing this issue as solved for now.
If you have any further questions, please comment and we will reopen.


⚠ SpeakUpConference.com has degraded performance

In 8a2d09c, SpeakUpConference.com (https://www.speakupconference.com) experienced degraded performance:

HTTP code: 200
Response time: 13074 ms

Resolved: SpeakUpConference.com performance has improved in b621a58 after 11 minutes.


[NodeRecorder] - Pause recording feature

Changes:

Add isPaused property to NodeRecorder
Don't write to buffer if isPaused
Add pause() function
Add resume() function
Add tests for this functionality in RecordingTests

Also reverted a previous PR because it didn't do much other than break the Cookbook. Sorry 😣. But it works now 🙃!

Thanks!


Create angular app

@rultor merge

@rultor merge

@abdealijk OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@abdealijk Done! FYI, the full log is here (took me 9min)


Add helper-element-padding option

This PR is adding a helper-element-padding option: https://github.com/usablica/intro.js/blob/master/intro.js#L95-L96
Thanks!

Thanks @lsg-braymon !

No worries. Thanks!
When will be next release?

@lsg-braymon just published v2.6.4.


Add an A-Text label

What problem does this solve or what need does it fill?
Text is a big and complicated topic by itself. Text problems are often unrelated to Bevy UI.

Done! Label away.


[DO NOT MERGE] VLM offline benchmark with MMMU-Pro vision

Opened this PR to share the mini offline batch inference benchmark with MMMU-Pro vision.
We may iterate on this and add support for other frameworks for comparison purposes.

Oops - realized I'm submitting this from the main of my fork. Going to close it and reopen another PR.