Support for Multi entity devices

Hi
We have a bunch of Gang devices like sOnOff 4CH etc, where a single device has multiple entities. Here, what matters is the standby power is something valid for the whole device but power usage by individual entities can vary. The standby is not something that happens in multiple entities but is once per device (instead of once per entity). Grouping doesn't help as
1 it creates standby power for each of the entities
2 the standby power should be added to the individual entity usage
3 the Device power/energy entity should be attached to the device. (doesn't work with the group again).

Rishab

I understand your issue. It's a hard one. I don't want to make the component more complex by adding all kind of advanced configuration for this.
I would suggest to keep things simple and just add the standby_power to one of the lights.
Also add 0.6 to the power so the 0.6 will also be added to the group power when the light.tuya3gang_03 is off.
Alternatively you can also spread the standby_power accross all the connected lights.
light.tuya3gang_03: 0.3 / 7.3
light.tuya3gang_02: 0.3 / 9.3
entities:
  - entity_id: light.tuya3gang_03
    standby_power: 0.6
    fixed:
      power: 7.6
  - entity_id: light.tuya3gang_02
    fixed:
      power: 9

Is there a possibility to separate the sensors for Entity and Devices? I think that would completely solve this. As a side bonus will also allow for devices to have static power draw - little different to your implementation already.
On the same line and completely off-topic, how do you tell the sensor that it is related to a certain entity?
I mean if I create a sensor I can not get that sensor to be visible as a linked sensor to the respective entity.
Also even more off-topic - I am interested in creating a bunch of integrations for HA for a set of local services (Like my local ISP, IPL - sports event in India, etc). Would you be willing to get on a zoom to help me get started? The dev documentation took me nowhere...
Just trying to contribute more to HA, as there aren't a lot of contributors from India and wrt Indian services.
Rishab

Is there a possibility to separate the sensors for Entity and Devices? I think that would completely solve this. As a side bonus will also allow for devices to have static power draw - little different to your implementation already.

I do not grasp what you mean with this "separate the sensors for Entity and Devices". Could you elaborate.

On the same line and completely off-topic, how do you tell the sensor that it is related to a certain entity? I mean if I create a sensor I can not get that sensor to be visible as a linked sensor to the respective entity.

Also not sure about this question. There is not something like linking an entity to an entity in HA, Only entities can be part of a device.

Also even more off-topic - I am interested in creating a bunch of integrations for HA for a set of local services (Like my local ISP, IPL - sports event in India, etc). Would you be willing to get on a zoom to help me get started? The dev documentation took me nowhere... Just trying to contribute more to HA, as there aren't a lot of contributors from India and wrt Indian services.

Sorry, I don't have time to do individual development support. The maintenance of this component is taking all my spare time, and also have a busy development job.
I was also struggling with the development docs / environment when I started about a year ago. Did not have any python experience before. But persistence is key. I would suggest you to start with the most simple component. Also there are dev channels in discord where you can ask about any help.

Rishab

@rishabmehta7 Not sure what to do with this issue. Going to close this. Let me know if you have answers on questions above then it can be reopened.


Docker container- Oracle Full Client Install - Alternatives

Hello Oracle Team,
I have a need to build an image with Oracle Client including tools such has SQLplus, LoadJava, SQLLoader, tnsping etc..
The Oracle 19c Instant client has everything that I need but the loadjava, Not sure about tnsping yet.  Here are my questions

Is it possible to install/add loadJava alone to my Instant Client install?
If I do need a full client - is there a docker image I can download?
If there is no image for a full client , to keep the size in mind, I see there are two options
a. Download Oracle DB docker image -SLIM  (~ 2GB)
b. Do an Image based Oracle Full Client Install (~ 2GB)
Is there a difference between a and b in terms of Client tools and libraries?

Looking forward to knowing your recommendation.
Thank you
Chad

@nsundara and @kuassim should be able to help you with the Java topic.
In general, the client binaries and libraries are the same in full client, the DB, and in Instant Client. There maybe some relinking differences, and you may have applied your own patches etc.  The Licenses differ.
There is no Docker image for the full client.
Tnsping isn't in Instant Client.  Tnsping definitely has its place, but if you want to verify a DB is usable, you should make a full connection to it.  Otherwise you won't know if, for example, the archiver is stuck.  And then, if you are checking a lot, you need to avoid reconnection overheads, so use a small daemon with a connection pool (size 1), release the connection to the pool after checking the DB is OK.  And/or use DRCP pooling.

Thanks @cjbj for your reply. Adding again- @nsundara and @Kuassim.
I am not sure I agree when you say "In general, the client binaries and libraries are the same in full client, the DB, and in Instant Client. There maybe some relinking differences, and you may have applied your own patches etc. The Licenses differ."
When i see the bin directory in my 19c instant client it contains like 10 tools...genezi, adrci, sqlldr, sqlplus impdpm impm expdp and exp) where as the 19c database's bin file contains like 100. So clearly there is a difference.
In 19c instant client as an example where loadJava - a Utility traditionally supplied in Oracle Full Client  or Full DB (versions 12-18), is missing, Or other tools in the future I shall see are needed as I progress within my project, what is the method of adding them to my install?
I downloaded the 19c instant client along with with Tools, dev, runtime packages from
https://www.oracle.com/database/technologies/instant-client/linux-x86-64-downloads.html.
Please let me know. Thanks!

To clarify, each of the binaries & libraries in Instant Client is the same (modulo the earlier proviso) since the Instant Client packages are built from a full ORACLE_HOME install.  Yes, not everything is included.

Loadjava is part of the Oracle database client; you need to accept the OTN distribution license agreement and make sure it allows you to bundle and distribute.

I am trying to use the image-based full client in order to use the orapki utility alone. My build is not completing due to excessive logs and size of full-client image.
If the binaries and libraries are the same in instant client, then why can't oracle provide the orapki command-line utility in instant client?


Feat/53 make all cli flag inputs case insensitive

📦 Pull Request

Added github issues link for devs to submit feedback
Made all cli flags case insensitive

@jamesrp13 Please review the new changes.


can we make it extensible like you did in FSH.WebApi.Infrastructure.Common

not to harm repository purpose and concept
can we add module initializer (for next release)
There is more samples like
https://github.com/devmentors/modular-framework/blob/beca9186e08a627176191160c964964d020da529/src/Modular.Infrastructure/Modules/ModuleLoader.cs
What is it doing in this sample
discover asseblies
searching for Specific interface
then
Confgiure Services and Configure them... also controllers etc. similar to balzor way.
which lives in their own code..

I checked it... going with this way may couse extra issues ...
So i ll find another way to  contribute meanwhile develop solution


caching issue?

I'm trying to use this plugin to develop some static files, but I'm encountering some issue where the HTML output files are not updated unless I restarted gulp (gulp watch is run by default in my code)
https://github.com/zebapy/react-gulp-emails
TBH I don't know if this is just an issue with my code or this plugin, but I'm hoping to get some insight.
Reproduction:

Follow getting started steps in readme
Open browser sync localhost:3000
Navigate to example email (/communications/comms-newsletter.html)
Edit the example email from test to something else

Expected:

Browser should reflect the changed text
dist folder should have updated html file with new text

Result:

No change

If you stop gulp and restart it, the html file will be updated correctly.

I will try to take a look this weekend

Hey @zebapy, I opened a PR on your repo w/ a fix to the gulpfile which should resolve your issues.
I do not like that you have to pass the scripts through babel before gulp-render-react can process them. I will look into the possibility of ingesting babel files as well.

@abramz, Thanks so much for taking the time to look at this. Sadly, I'm not seeing this resolve the issue. I still have the same problem. Did you successfully have the browsersync update and the HTML files become updated with correct code?

Yep, and it logs to the console that the file was updated
On Sun, Dec 18, 2016, 6:32 PM Zeb Pykosz<EMAIL_ADDRESS>wrote:

@abramz https://github.com/abramz, Thanks so much for taking the time
to look at this. Sadly, I'm not seeing this resolve the issue. I still have
the same problem. Did you successfully have the browsersync update and the
HTML files become updated with correct code?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/abramz/gulp-render-react/issues/7#issuecomment-267868800,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD4v2YWvDEf15ekUqEnNizBTob53l4Ntks5rJezFgaJpZM4LPY1s
.

This is what I'm experiencing.

I'll check it out later tonight
On Sun, Dec 18, 2016, 7:01 PM Zeb Pykosz<EMAIL_ADDRESS>wrote:

This is what I'm experiencing. [image: gulpreact]
https://cloud.githubusercontent.com/assets/2199845/21299856/802c6686-c56d-11e6-889f-cc7d8020b333.gif
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/abramz/gulp-render-react/issues/7#issuecomment-267871726,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD4v2bWDZXOwhYHiYeqty4uiHcsKMuAPks5rJfOHgaJpZM4LPY1s
.

Thanks again. I'd be interested in seeing an example/ directory in this repo as the readme doesn't cover babel files, and what I have in my repo is my best guess to handling babel transpiling, but it obviously has an issue currently.

I made another branch to test the basics and it seems watching for changes the rebuilding just isn't working for me still. https://github.com/zebapy/react-gulp-emails/tree/gulp-react-example

I updated my PR on your repo, I have a new version of the module up that fixes the issue.
The way the module currently works, you would need to run gulp with babel in order to require es6 files. This is a limitation due to the module's use of require. You can see from the tests that it can ingest them just fine when doing this.
As per the guidelines of this module, it does 1 thing, render React components as strings. It does not deal with anything else because that is the philosophy of the gulp ecosystem.
This is an open source module. If you would like to change the way the the module ingests files or add documentation, feel free to open a PR with the changes. Otherwise I will add enhancements as I have time to do so.
resolved by #12

I hadn't considered needing to use babel CLI to run gulp first, that makes sense.
I may have misinterpreted what you previously said, I agree the module should serve a single purpose (only transforming react components into static html) instead of also transpiling babel (besides running babel to run gulp).
Thanks so much for this. Interesting to see it was an actual cache issue.


Inline score function

This is a hot path and it looks more like a left-over from a previous
design.

Nice! Looks good to merge!

Thanks!


Current Zally's API spec is incomplete

Description of endpoint /api-violations doesn't contain any info on how to actually send swagger spec for linting.
paths:
  '/api-violations':
    post:
      summary:
        Retrieves list of API violations based on the uploaded swagger file
      description:
        Retrieves list of API violations based on the uploaded swagger file
      parameters:
        - $ref: '#/parameters/Authorization'
      consumes:
        - application/json
      produces:
        - application/json
      responses:
        200:
          description: API swagger is OK
          schema:
            $ref: '#/definitions/LintingResponse'
        400:
          description: Input file not parsable
          schema:
            $ref: 'https://zalando.github.io/problem/schema.yaml#/Problem'
        default:
          description: An array of error objects
          schema:
            $ref: 'https://zalando.github.io/problem/schema.yaml#/Problem'
      security:
        - oauth2:
          - uid

Also description "Retrieves list of API violations based on the uploaded swagger file" sounds very confusing to me - do I really need to upload my swagger file prior to calling this endpoint?
I propose to add actual spec for request body and change description.

The description is just wrong, you are right.
The other point I don't understand really as there is an entity definition for LintingRequest in the swagger file.

LintingRequest is not referenced in parameter list for this path.
Also there are several validation errors of our current API spec (just paste it to http://editor.swagger.io/#/ to see them)

Ok, I see.

Can be a good integration test: lint our own swagger specification.


Partial support for tinygo WASM & test pipelines

This PR contains fixes to allow compilation to WASM with tinygo. Technically it is possible to compile entire tzgo with these patches but any encounter with enconding/json panics so one should avoid such methods. All except rpc and Un/Marshal should be ok.
Changes & explanation:

block operations can not be [][]* - not supported by tinygo. But changing it to [][] should not cause any issues
(reflect/type).FieldByIndex - not supported by tinygo. Added wasm build tag and separate method to handle properly
tzgo param initialization used encoding/json through mixin - this caused any tzgo linked wasm to panic during initialization. Switched param initialization to embedded struct
github test pipelines - for tinygo we test targeting wasm and only tezos and base58 packages to test other we would have to move away from enconding/json in test files or add custom test files with build tags just for wasm target

This is no longer relevant. Tinygo got json serialization support and tzgo initialization changed in the meantime. I will create new PR if some changes are required.


NEXT-36 Introduce unit test for NEXT

Added tests for:

animated-value.js
map-size.js
misc.js

Tests are not added for:

ball-view.js - uses pixi.
connector.js - uses XMLHttpRequest. Can be mocked however.
constants.js - no need to test.
controller.js - uses pixi.
main.js - no need to test.
pointer.js - relies on Viewer object that passed into it; also mouse events.
viewer.js - uses pixi.

@xzfc please update your PR

Updated.


Updating to new version causes "shallow-compare" missing error

The Problem
I just updated react-redux-form and I get this error:
Error in ./~/react-redux-form/lib/utils/deep-compare-children.js
Module not found: [CaseSensitivePathsPlugin] `/Users/.../node_modules/shallow-compare/dist/shallow-compare.js` does not match the corresponding path on disk - File does not exist.

Steps to Reproduce
package.json:
{
  "name": "web_client",
  "version": "0.1.0",
  "private": true,
  "homepage": "http://foo.com",
  "devDependencies": {
    "react-scripts": "0.9.0",
    "redux-devtools-extension": "^2.13.0"
  },
  "dependencies": {
    "babel-polyfill": "^6.23.0",
    "crypto": "0.0.3",
    "dns": "^0.2.2",
    "isomorphic-fetch": "^2.2.1",
    "jsonwebtoken": "^7.3.0",
    "jws": "^3.1.4",
    "jwt-simple": "^0.5.1",
    "react": "^15.5.4",
    "react-dom": "^15.4.2",
    "react-redux": "^5.0.5",
    "react-redux-form": "^1.11.2",
    "react-router": "^3.0.2",
    "react-router-redux": "^4.0.8",
    "redux": "^3.6.0",
    "redux-form": "^6.6.1",
    "redux-logger": "^2.8.1",
    "redux-saga": "^0.14.4",
    "redux-thunk": "^2.2.0",
    "semantic-ui-css": "^2.2.9",
    "semantic-ui-react": "^0.67.0",
    "shallow-compare": "^1.2.1"
  },
  "scripts": {
    "start": "react-scripts start",
    "build": "react-scripts build",
    "test": "react-scripts test --env=jsdom",
    "eject": "react-scripts eject"
  }
}

Everything's up to date; not sure what this could be.

Shouldn't be pointing there - it should be pointing to node_modules/shallow-compare/lib/index.js. Try these two things:

Exact version: "shallow-compare": "1.2.1" (note missing caret)
rm -rf node_modules && npm install

I did step 1 and 2 as you suggested and it worked; it fixed another problem I was having about a missing saga module as well. Thanks.


Mailspring some mails unreadable?

Some mails in Mailspring are unreadable due to dark text on dark text, I see this issue is already reported for duckduckgo but I thought I'd report it for Mailspring as well.

@avelino can you take a look?

is the same problem mentioned in this issue #6


Delete old, stale frontend code

:)

!ci-build-me

!ci-build-me

!ci-build-me

!approved-for-mainnet


Add support for basic HTTP authentication

Electron supports it, as in https://github.com/atom/electron/pull/3250
I added this configuration to app/main.js and it worked, but I'd love having a default with a popup window. Keeping my password unencrypted on hdd is no fun.
Is that even possible with electron?

Maybe possible, the API has been just tested at dca9b62cda34025124195de6c31f0f961f19267a.
But Electron doesn't support input dialog such as window.prompt, so the app has to implement on its own.

Let me jump in here, I had solved the problem in a not-so-graceful but effective way once for a different app, perhaps this experience can come in handy (did spend some hours testing various libraries which refused to work before coming up with this):
var app = require('app');
var BrowserWindow = require('browser-window');

var prompt = require('sync-prompt').prompt;

var name = prompt('Username: ');
var password = prompt.hidden('Password: ');

app.on('login', function (event, webContents, request, authInfo, callback) {
  event.preventDefault();
  console.log("Replacing default auth behaviour.");
  callback(name, password);
});

Thank you for feedback. I think It will work, however users have to use command-line and keep it opened.
Note: callback() can be called asynchronously, so sync-prompt might not be necessary. Then, sync-prompt is a native module, so it's a little hard to compile release version. (In my environment, Windows 10 + Node 4.2.2 + VS2015 Update1, I could not compile sync-prompt.)

readline-sync might be nice for @mgielda's example. It looks JavaScript.

@yuya-oc indeed it works (sorry it took some time to answer) - so I agree, the updated snippet is:
var readlineSync = require('readline-sync'); //tested on readline-sync 1.4.1

var username = readlineSync.question('Username: ');
var password = readlineSync.question('Password: ', {
  hideEchoBack: true // The typed text on screen is hidden by `*` (default).
});

app.on('login', function (event, webContents, request, authInfo, callback) {
  event.preventDefault();
  console.log("Replacing default auth behaviour.");
  callback(username, password);
});

How should we proceed with that? There needs to be some config option for this I suppose. I agree that doing a graphical version later is a good idea but for now this would be good enough. Should we perhaps propose a pull request?

Closed by the commit message. This should not be closed yet.

Sorry! You are right of course. Habit from my own projects.


feat(Item): clamp composite bar item

Fixes #340

How will the element title be displayed if 2 lines are not enough? And please add to storybook item with long text and tag New


conn.on("refresh") Looping

var conn = new jsforce.Connection({
oauth2: {
clientId: process.env.SALESFORCE_CLIENT_ID,
clientSecret: process.env.SALESFORCE_CLIENT_SECRET
},
refreshToken: refreshToken,
instanceUrl: instanceUrl
});
conn.on('refresh', function(newAccessToken, res) {
console.log('Refresh Occurred :', newAccessToken);
});
This Loops Forever until you get a MaxListenersExceededWarning
every 200ms the "refresh" event fires.
Why is this happening.  I have never seen this before and I am stumped
Jeff

i am facing the same issue, did you find the solution to this?

this might be related to https://github.com/jsforce/jsforce/issues/1411
closing it because we can't tell if the org info was correct. Please open a new issue if someone can repro.


Refactor Reporter to not know about implementations

WHAT is this pull request doing?
The Reporter class is an odd bird and only used for debugging purposes. It knows too much about classes (even accessing instance variables) which prevents refactoring.
This PR refactors Reporter to not know anything about any implementation; instead "reportable" entities must include the Reportable module and specify their reportable objects with the reportables macro.
The macro may look scary at a first glance, but it will generate a __report method that will report all specified "reportables" to the Reporter instance. This makes the Reporter only responsible for the output, while each entity is responsible for what to report.
HOW can this pull request be tested?
Run lavinmq and send USR1 to observe output.

What's the benefit? "Prevents refactoring"? You have to change your reportables just the same way as you have to change the Reporter today. Now a lot of debugging code is scattered around in individual classes.

IMO changing the internals of a class shouldn't affect other classes. I encountered this when I was abstracting client/channel/consumer.

This makes sense. Could be used for general instrumentation and not only debugging?

The current approach doesn't work very well if you start to abstract things, like making Client an abstract class to introduce amqp clients and mqtt client, since these different client classes may have different internals.

Sure but question is if this is useful at all?
On Wed, 7 Aug 2024, 09:48 Jon Börjesson, @.***> wrote:

The current approach doesn't work very well if you start to abstract
things, like making Client an abstract class to introduce amqp clients
and mqtt client, since these different client classes may have different
internals.
—
Reply to this email directly, view it on GitHub
https://github.com/cloudamqp/lavinmq/pull/676#issuecomment-2272838906,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABL6TSH6MNMA4D44HJE4QDZQHGNLAVCNFSM6AAAAABHVRNE6WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZSHAZTQOJQGY
.
You are receiving this because your review was requested.Message ID:
@.***>


[Parsers] Fix *.xsaccess

As commented by @aihanrashidov:
Parsing issues with the .xsaccess artifact. The error is related to the parsing of the authentication key as specified in the issue when parsing a json like this:
{
  "exposed":true,
  "authentication":[{"method":"LogonTicket"},{"method":"Basic"}]
}

The value can be an array of multiple authentication methods but the parser is expecting only one.
Related to: #441

Exception:
o.e.d.c.s.api.AbstractSynchronizer - Resource [see below...] skipped due to an error: java.lang.IllegalStateException: Expected BEGIN_OBJECT but was BEGIN_ARRAY at line 6 column 21
com.google.gson.JsonSyntaxException: java.lang.IllegalStateException: Expected BEGIN_OBJECT but was BEGIN_ARRAY at line 6 column 21
	at com.google.gson.internal.bind.ReflectiveTypeAdapterFactory$Adapter.read(ReflectiveTypeAdapterFactory.java:176) ~[gson-2.2.4.jar:na]
	at com.google.gson.internal.bind.ReflectiveTypeAdapterFactory$1.read(ReflectiveTypeAdapterFactory.java:93) ~[gson-2.2.4.jar:na]
	at com.google.gson.internal.bind.ReflectiveTypeAdapterFactory$Adapter.read(ReflectiveTypeAdapterFactory.java:172) ~[gson-2.2.4.jar:na]
	at com.google.gson.Gson.fromJson(Gson.java:803) ~[gson-2.2.4.jar:na]
	at com.google.gson.Gson.fromJson(Gson.java:741) ~[gson-2.2.4.jar:na]
	at com.sap.xsk.xsaccess.ds.model.access.XSKAccessArtifact.parse(XSKAccessArtifact.java:27)

test.xsaccess:
{
	"cache_control" : "must-revalidate",
	"enable_etags" : true,
	"exposed": true,
	"prevent_xsrf" : true,
	"authentication": [{
		"method": "Form"
	}],
	"headers": {
		"enabled": true,
		"customHeaders": [
			{ "name":"X-Frame-Options","value":"SAMEORIGIN"},
			{ "name":"X-Content-Type-Options","value":"nosniff"},
			{ "name":"X-XSS-Protection","value":"1; mode=block"}
		]
	},
	"default_connection" : "a.b.c.setupDU.restricted.lib::User"
}

Related to #31


MHD_065: mapping between XDS RegistryError and FHIR OperationOutcome

This is not formally MHD_065, but that is the next number.. and it was mentioned in the readme, so it is somewhat an open issue (sorry).
Is it needed to have a mapping between XDS RegistryError and FHIR OperationOutcome?  add to mappings page a mapping between XDS RegistryError and FHIR OperationOutcome. At the element level, and also addressing OperationOutcome.issue.code vocabulary could be mapped to the XDS error vocabulary. between XDS https://profiles.ihe.net/ITI/TF/Volume3/ch-4.2.html#<IP_ADDRESS> and FHIR OperationOutcome.

Put as an open issue
Joe might provide a mapping

open issue is now indicated.

finished with revisions. this mapping may happen in the future.


Metrics & Pageviews take very long

I'm analyzing a relatively large website since Aug 21 and it already takes ages until the stats are loaded.

The metrics took 15.91s and the pageviews 19.68s to load. So after a week, it will take more than 30 seconds, which is way too long.

That sounds really long. Those queries should be using indexed columns so it shouldn't take that long. What kind of server is it running on?

Umami is hosted on a small Hetzner Cloud machine (CX 21). I'm using MySQL as database.

I'm running on a much smaller server (although fewer users) and I haven't had queries that long. Check your database and make sure the indexes were created.

Indexes are there.

You can try doing an EXPLAIN on the query to see what's going on. This is the query for metrics for MySQL:
select sum(t.c) as "pageviews",
  count(distinct t.session_id) as "uniques",
  sum(case when t.c = 1 then 1 else 0 end) as "bounces",
  sum(t.time) as "totaltime"
from (
  select session_id,
  date_trunc('hour', created_at),
  count(*) c,
  floor(unix_timestamp(max(created_at)) - unix_timestamp(min(created_at))) as "time"
from pageview
where website_id=?
  and created_at between ? and ?
  group by 1, 2
) t

Replace the ? with parameters.

Running the query itself, it takes 7,125s:
select sum(t.c) as "pageviews",
  count(distinct t.session_id) as "uniques",
  sum(case when t.c = 1 then 1 else 0 end) as "bounces",
  sum(t.time) as "totaltime"
from (
  select session_id,
  date_trunc('hour', created_at),
  count(*) c,
  floor(unix_timestamp(max(created_at)) - unix_timestamp(min(created_at))) as "time"
from pageview
where website_id=2
  and created_at between from_unixtime(1597788000) and from_unixtime(1598392799)
  group by 1, 2
) t

Explained:

id
select_type
table
type
possible_keys
key
key_len
ref
rows
Extra

1
PRIMARY

ALL

242784

2
DERIVED
pageview
ref
pageview_created_at_idx,pageview_website_id_idx
pageview_website_id_idx
4
const
242784
Using where; Using temporary; Using filesort

Also, what is the date range on that query?

7 days is set in the UI. However, the first entry was created on 2020-08-20 03:14:25.

Try creating a new composite index and see if that helps:
create index pageview_website_id_created_at_idx on pageview(website_id, created_at);

Still no change.
metrics: 10.96s
pageviews: 19.03s

@SoftCreatR do you think you can send me a mysql dump of your data? I only have postgresql data. I ran a test with 500K records and it returned in 30ms. I want to see if there is something mysql specific.

Ok, I mocked up a database in MySQL with a million records. Even this simple query it takes 5 seconds:
select session_id,
count(*) c
from pageview
where website_id=1
group by 1

The same query in Postgres takes 13ms. It seems MySQL has issues with group by and count in the query. May need someone with more MySQL experience to chime in here.

I guess, the problem here is the numeric grouping. I'm not sure, but it could be, that indexes don't have that much effect in this case.

@SoftCreatR try adding this index:
create index pageview_website_id_session_id_created_at_idx on pageview(website_id, session_id, created_at);

No change. Ill send you a dump of my db. Maybe that helps.

@SoftCreatR Getting closer to a solution. I've got the main queries down to less than 2 seconds.

@SoftCreatR try pulling the lastest build. I made some improvements. And make sure you create this index:
create index pageview_website_id_session_id_created_at_idx on pageview(website_id, session_id, created_at);

Created the index, pulled the latest commits, rebuilt it. Result:

It's even worse: It consumes all available RAM and CPU so it's completely useless for me atm.

So, after some analysis, 3 queries are fired:
select sum(t.c) as "pageviews",
    count(distinct t.session_id) as "uniques",
    sum(case when t.c = 1 then 1 else 0 end) as "bounces",
    sum(t.time) as "totaltime"
  from (
     select session_id,
       DATE_FORMAT(created_at, '%Y-%m-%d %H:00:00'),
       count(*) c,
       floor(unix_timestamp(max(created_at)) - unix_timestamp(min(created_at))) as "time"
     from pageview
     where website_id=2
     and created_at between TIMESTAMP'2020-08-21 22:00:00' and TIMESTAMP'2020-08-28 21:59:59.999000'
     group by 1, 2
 ) t

Execution time (Avg): 10.375s
select DATE_FORMAT(convert_tz(created_at,'+00:00','+02:00'), '%Y-%m-%d') t,
    count(distinct session_id) y
  from pageview
  where website_id=2
  and created_at between TIMESTAMP'2020-08-21 22:00:00' and TIMESTAMP'2020-08-28 21:59:59.999000'
  group by 1
  order by 1

Execution time (Avg): 2.750s
select distinct url x, count(*) y
  from pageview
  where website_id=2
  and created_at between TIMESTAMP'2020-08-21 22:00:00' and TIMESTAMP'2020-08-28 21:59:59.999000'
  group by 1
  order by 2 desc

Execution time (Avg): 18.004s

@SoftCreatR The data dump was very useful. I was able to fix a couple bugs from it. For example the domain field for the website should just be the domain, not including http. So thanks for that.
Using the new dump and  I made a few small improvements. Here are the numbers from the details page:

A little higher than I'd like but not terrible. But I think I understand what is going on with your server.  With the amount of hits you are getting, it's probably taking up all your resources for these queries. You might need a much stronger server or setup a read replica.
Originally I had a cookie using localStorage that would cache some data save a few queries, but I removed it due to GDPR concerns. It would probably help a lot in your case. I can add it back and make it configurable. What do you think? Since you're EU based it's probably more of a concern for you.


Support static prefixes & suffixes in dynamic path segments

This address #707 by adding support for routes of the form prefix$var$suffix. This is technically a breaking change in that you can no longer have $ in the name of a dynamic path segment param. I'm unsure if that is something people do. If it is concerning I can adjust this to only support prefixes which would not be a breaking change.
I'm not familiar with the code base so I apologize if this is incomplete or broken. It seemed like there wouldn't be any changes needed to react-router as all the logic was in router-core. I wasn't sure if the cli needed changes.

I fixed up some more of the conflicts. It looks like a few examples broke with this (infinite loops?). So we'll need to hold off on this.

This would need to be updated with all of the new stuff, including file based route generation.


Add route to manage admin event

DB
Create those in the db :

[ ] admin
[ ] lite_admin

routes

[ ] /POST to add an admin to an event
[ ] /DELETE to remove an admin ( only for admin)
[ ] Create role admin / sub-admin so that only the admin can delete the event or delete another admin
[ ] Check the return of the /GET routes (2 routes) to return an array of admin

Change

[ ] Modify the patch and delete to check the admin status

I'm gonna keep it simple and only creates admins, no lite_admins


[Snyk] Security upgrade python from 3.9-slim to 3.13.0b1-slim

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

cmd/suggestion/optuna/v1beta1/Dockerfile

We recommend upgrading to python:3.13.0b1-slim, as this image has only 50 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

436
CVE-2024-33601 SNYK-DEBIAN12-GLIBC-6673964
No Known Exploit

436
CVE-2024-33600 SNYK-DEBIAN12-GLIBC-6673969
No Known Exploit

436
CVE-2024-33602 SNYK-DEBIAN12-GLIBC-6673972
No Known Exploit

436
CVE-2024-33599 SNYK-DEBIAN12-GLIBC-6673974
No Known Exploit

500
Integer Overflow or Wraparound SNYK-DEBIAN12-ZLIB-6008963
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 72.998%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
3117

Relevant Lines:
4270

💛  - Coveralls


It will not work well when a empty struct convert to another.

1. What version of Go and system type/arch are you using?
1.20
2. What version of GoFrame are you using?
2.5.1
3. Can this issue be re-produced with the latest release?
Yes.
4. What did you do?
type StructA struct {
}
type StructB struct {
}
func Test_EmptyStruct(t *testing.T) {
gtest.C(t, func(t *gtest.T) {
	var s3 StructA
	var s4 *StructB

	err := gconv.Scan(s3, &s4)
	t.AssertNil(err)
})

}
5. What did you expect to see?
I want to get the s4 without error.
6. What did you see instead?
I got an error notice me StructA CAN'T be converted to StructB.

work well now.


discussion: reduce number of packages in the sdk

I think this sdk should have a single public package, and that package should be named wavefront.
I've noticed that telegraf imports only the senders package from this repo, and aliases it to wavefront
import (
    ...
	wavefront "github.com/wavefronthq/wavefront-sdk-go/senders"
)

that means they create a new sender by invoking wavefront.NewSender, which I think looks better than senders.NewSender
I think we could consider moving the exported members from senders, histogram, and event into a wavefront directory/package, and move most of their unexported members into internal.
This would be a big change, but I do think it would make the API of the SDK easier to learn and so I'd like to consider changing this before v1

Looking at #143 , I have a couple of new thoughts on this:

wavefrontSender has a large implementation. I think we could move that type into an internal subpackage
the histogram, event, etc. packages are each very small, and having these types in separate packages means that you have to add multiple imports just to send a histogram or a span. I think that it'd be more convenient if those types moved into the senders package.


Support deletion of SageMaker Notebook Instances

This PR adds the SageMaker Notebook Instances as a supported resource of cloud-nuke. #281

[x]  Add Documentation for SageMaker Notebook Instances

[x] Add Tests for SageMaker Notebook Instances

[x] Add Code for SageMaker Notebook Instances

Output of test:
=== RUN   TestNukeNotebookInstance
=== PAUSE TestNukeNotebookInstance
=== CONT  TestNukeNotebookInstance
[cloud-nuke] INFO[2022-02-11T12:54:10+01:00] Random region chosen: ap-northeast-1
[cloud-nuke] INFO[2022-02-11T12:59:23+01:00] Deleting all Sagemaker Notebook Instances in region ap-northeast-1
[cloud-nuke] INFO[2022-02-11T13:01:29+01:00] Deleted Sagemaker Notebook Instance: cloud-nuke-test-OvfCJZ
[cloud-nuke] INFO[2022-02-11T13:02:31+01:00] [OK] 1 Sagemaker Notebook Instance(s) deleted in ap-northeast-1
--- PASS: TestNukeNotebookInstance (503.52s)
PASS
ok      github.com/gruntwork-io/cloud-nuke/aws  503.536s

I also had to update run the code-generation for the EC2API mock to run the tests.
In addition, I slightly update the AWS session creation in aws.go so that it supports sso profiles. Feel free to remove if this is outside of the scope of this PR.

Heads up that I went ahead and resolved the merge conflict for this in https://github.com/gruntwork-io/cloud-nuke/pull/332 so that it can be merged.


Add Getrange command

https://redis.io/commands/getrange/
@roseduan
If there is no problem, I will work on it.
This could be solved by simply adding a command using Get on the server side, but should it be added in package rosedb as well?

I think it should be added to the roasedb package as well, so I''ll proceed with that.


Receive message using "qname" rather than ":qname"

Hi,
I've follow to example, let message = rsmq.receive_message::<String>("myqueue", None).await?; but I always get queue not found error. So, I try using let message = rsmq.receive_message::<String>(":myqueue", None).await?;. Can I confirm that this is a mistake?

It seems that the JS library add the : to the queue name and this one doesn't. Let me check the JS source code and see. I will update the library if there is any mismatch.

I cannot see where are the : comming from. What library created the queue initially in the Redis instance?
As you can see in the tests, https://github.com/Couragium/rsmq-async-rs/blob/master/tests/test.rs#L18 it should work without prepending anything.

Wasn't check on my notification recently. There is : in my redis queue created by other node.js service by accident. I will proceed to close this issue. Thank you for verifying it.


Update feature-reliable.md

Hello
I've made some changes to make the text look more natural.
Please let me know if there are any additional repository management guidelines I should follow.
Thank you.

리뷰 감사합니다.
피드백 주신사항 수정하여 반영하였습니다 - 895f932


Expose web app kind as input to task

@sachinma - Can you review the changes to this task from Ibiza CD point of view.


jsonhelpers warning

not an issue at the moment, but could be an issue for other consumers
DragonFruit.Common.Data/Extensions/JsonExtensions.cs#L56
Converting null literal or possible null value to non-nullable type.

closed in b3ee0d


training.py _slice_arrays() fix crash when arrays are None

atomic component of #6928 and #7046

LGTM (albeit a better solution may have been to avoid calling slice when we have a None instead of an array, which would have made the logic of slice lighter (one less if per array), which is somewhat important given that it is called repeatedly inside of the training loop -- but that's not a blocker because the performance difference will not be noticeable).

If you time it, there is consistent 3% time increase when using the new function. However that's 3% of something small, surrounded by many things that are much more computationally intensive. Overall I don't think it's a big deal.

Sorry, more like 5%.
import numpy as np
import time

arrs = [np.random.random((100000, 10)),
        np.random.random((100000, 10)),
        np.random.random((100000, 10))]

def _slice_arrays_1(arrays, start=None, stop=None):
    """Slice an array or list of arrays.
    This takes an array-like, or a list of
    array-likes, and outputs:
        - arrays[start:stop] if `arrays` is an array-like
        - [x[start:stop] for x in arrays] if `arrays` is a list
    Can also work on list/array of indices: `_slice_arrays(x, indices)`
    # Arguments
        arrays: Single array or list of arrays.
        start: can be an integer index (start index)
            or a list/array of indices
        stop: integer (stop index); should be None if
            `start` was a list.
    # Returns
        A slice of the array(s).
    """
    if isinstance(arrays, list):
        if hasattr(start, '__len__'):
            # hdf5 datasets only support list objects as indices
            if hasattr(start, 'shape'):
                start = start.tolist()
            return [x[start] for x in arrays]
        else:
            return [x[start:stop] for x in arrays]
    else:
        if hasattr(start, '__len__'):
            if hasattr(start, 'shape'):
                start = start.tolist()
            return arrays[start]
        else:
            return arrays[start:stop]

def _slice_arrays_2(arrays, start=None, stop=None):
    """Slice an array or list of arrays.
    This takes an array-like, or a list of
    array-likes, and outputs:
        - arrays[start:stop] if `arrays` is an array-like
        - [x[start:stop] for x in arrays] if `arrays` is a list
    Can also work on list/array of indices: `_slice_arrays(x, indices)`
    # Arguments
        arrays: Single array or list of arrays.
        start: can be an integer index (start index)
            or a list/array of indices
        stop: integer (stop index); should be None if
            `start` was a list.
    # Returns
        A slice of the array(s).
    """
    if arrays is None:
        return [None]
    elif isinstance(arrays, list):
        if hasattr(start, '__len__'):
            # hdf5 datasets only support list objects as indices
            if hasattr(start, 'shape'):
                start = start.tolist()
            return [None if x is None else x[start] for x in arrays]
        else:
            return [None if x is None else x[start:stop] for x in arrays]
    else:
        if hasattr(start, '__len__'):
            if hasattr(start, 'shape'):
                start = start.tolist()
            return arrays[start]
        elif hasattr(start, '__getitem__'):
            return arrays[start:stop]
        else:
            return [None]

f1_times = []
f2_times = []
for _ in range(20):
    t0 = time.time()
    for _ in range(100000):
        _slice_arrays_1(arrs, start=3, stop=13)
    t1 = time.time()
    f1_times.append(t0 - t1)

    t0 = time.time()
    for _ in range(100000):
        _slice_arrays_2(arrs, start=3, stop=13)
    t1 = time.time()
    f2_times.append(t0 - t1)

f1 = np.mean(f1_times)
f2 = np.mean(f2_times)
print(f1)
print(f2)
print(100. * (f2 - f1) / f1)

Oh for that one function call! Yeah that makes sense, I agree. I was thinking about the typical overall main program loop like you described later on.
I do think it is a good idea to consider adding "training performance improvements with validating unit tests and unchanged program behavior" to the help wanted board.
I tried out Numba and it looks very nice, but I think it won't be easy to incorporate until the next release which claims to support list comprehensions. If that could provide a sufficiently substantial boost to the overall performance it might be worth considering. Installation was fine with pip.


BUG:The cards are improper

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
the heights of the cards are improper

please assign me issue and ill work on it
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Work on it !!

@huamanraj
I can align the cards properly, I have worked on the same issue with the benifits card
it has been 2 weeks since you assigned the contributor
please reassign me the issue


Fix enhanced line pragma indexing

Fixes https://github.com/dotnet/razor/issues/9051.

@chsienki @dotnet/razor-compiler for a second review, thanks


I can't sign in as I forgot my password, is there any way to retrieve my account?

I can't sign in as I forgot my password, it's a randomly generated password by google, is there any way to retrieve my account? I have a lot of progress that at it's current  state, is permanently lost, so could you please take action, @WillWam and help retrieve my account? label:bug

Sorry to hear you got locked out of your account! Currently I don't have any good account recovery methods, I'm still learning how to implement that. Hopefully sometime in the future I'll make it link up with people's emails so they can reset their passwords.
For now I can't do much, but I could delete your account and then you could recreate it. This would set your scores down to 0 but at least you would have the username you want again. Do you want me to do that?

I already have a new account with the same name, but I changed I for an L , so could you combine the statistics from my old account to my new one? If it's out of your limits, please keep the account just in case you discover an account recovery method in the future


Reference material spec for text scaling when available in AppBar

Follow up from https://github.com/flutter/flutter/issues/58093
When the material spec for accessibility text scaling is available add reference to it in docs, to educate and explain the text scale factor cap of 1.34 for titles.

Can someone explain to me where the cap of 1.34 comes from and how it is in line with the WCAG guidelines?
Our app fails criterium 1.4.4 (200 percent text scaling), so we disabled the cap of 1.34. Are titles excluded from this criterium?
In general, the non-lineair font scaling confuses our team how it is in line with this criterium.


Update realm-core to v13.1.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

12 of 12 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 89.021%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
2773

Relevant Lines:
3115

💛  - Coveralls

I will apply them separately after we merge this PR.
@desistefanova Should we create an issue for that?


->column() property should also work in form

Hello, really useful field type!
->columns() property works only in detailed view, would be very nice if columns rule also would work in form view.
I am not a JS guy, or else I could create a pull request :(

@fourstacks why this issue is closed without a replay of fix?


client side timeouts for pubsusb think about proper settings

Right now there are no client side timeout settings in pubsub mvp SDK we need to think about what these values should be set at for DataPlane operations.

Can we reference data client operation timeout defined here?
I know these are for the cache data operations, so do we want more relaxed timeout for the pubsub data operations (publish and subscribe)?


Set prerelease=true in release Workflow

Current dependencies on/for this PR:

main

PR #1286   👈

This comment was auto-generated by Graphite.


Added DB volume shown on device to attributes

Description:
Added DB volume shown on device to attributes. Much easier to know which volume level it is set at.
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hello @phithor,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

c4656eb3396cb4bfdcde7b2010035fe44a838226 - This commit has something that looks like an email address (j_matho@jthost.net). Maybe try linking that to GitHub?.
fee7bdd8a05fdb32a7fdbe1016448faac9334392 - This commit has something that looks like an email address (mathias@phithor.no). Maybe try linking that to GitHub?.
c42d8b6bbdfb1f1321f7a650eee599d67858719e - This commit has something that looks like an email address (mathias@phithor.no). Maybe try linking that to GitHub?.
9a9ab251009074bd33c57d3c720522d655a065da - This commit has something that looks like an email address (mathias@phithor.no). Maybe try linking that to GitHub?.
4997da748af3b0c0889d2d22b58b2bf4fda73297 - This commit has something that looks like an email address (mathias@phithor.no). Maybe try linking that to GitHub?.
5b6f1cc831e9f05b8ef3504c69ee50c4f7fc590d - This commit has something that looks like an email address (mathias@phithor.no). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:


more boilerplate: fix DISCLAIMER; add CONTRIBUTING

can you fix the ibm mention in the README?

that's the name of the old NPM module.  The thought was to leave the npm module intact for a short time until we are get things configured to publish and @openwhisk/composer or @openwhisk-composer/composer (so the README doesn't point to a non-existent npm module).   I agree we need to do this quickly...

I propose openwhisk-composer for the npm package name.

I changed the top-level header to Apache OpenWhisk composer and added a note that we will be releasing under a different npm module name soon.  @rabbah Good enough for now?


Origin/terminal support

In addition to the previous pull request, I propose this terminal support.
In Emacs alt+shift+1 (M-!) will run the command shell-command in the minibuffer. There is no minibuffer in vscode but I tried to emulate this behavior with a terminal I call "minibuffer" and attach to the Terminal pane. If you like the addition, you can ignore my previous PR and merge this one.

Hello, thanks for the submission.
There is a "toggle terminal" in vscode, accessible in the menu through View-Integrated Terminal. I use that through a customized shortcut for this usecase. Would that not be sufficient for you?

It was take it or leave it


Add documentation to the Sudo module

Initial draft of the Sudo documentation. Expecting feedback and edits before it gets merged in.

Still several of instances of missing .s at the end of sentences.

Thanks @joepetrowski


unicode font

there's a way to use Unicode-fonts on jimp loadFont ?

No the way that jimp works it has to be a bitmap font. I tried adding more fonts once and it was pretty hard. PRs welcome though!


”Tansa”, ”Suhuleț” are listed in the drop-down menu

Description:
When user writes ”Tansa”, ”Suhuleț” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Tansa"

Expected results
The location appears in the drop-down menu as "Sat Tansa, IAȘI (Tansa)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Suhuleț"

Expected results
The location appears in the drop-down menu as "Sat Suhuleț, IAȘI (Tansa)".


Remove third-party

https://github.com/PaddlePaddle/paddle-mobile/tree/develop/third-party depends on the solving of this issue, otherwise, cpplint will check third-party source code.

fixed


Fix Fix NPE when query empty hive transactional table.

Proposed changes
Backport from #27563
Fix NPE when query empty hive transactional table.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


New Stream reporting payload (Mercury)

Is your feature request related to a problem? Please describe.
It's not really a problem, its more like a enhancement.
Describe the solution you'd like
Update librespot code with new Stream reporting payload in Mercury.
E.g: New "unknown" id for TrackTransitionEvent (latest was 44). Weird / long number at the end of payload, example:<PHONE_NUMBER>000191.

Someone should port it from librespot-Java. I had not gotten around to it. It should not be super hard to do.

E.g: New "unknown" id for TrackTransitionEvent (latest was 44). Weird / long number at the end of payload, example:<PHONE_NUMBER>000191.

Btw, this is the core_version field, and it is a constant, it depends on the version of the application.

nice to know! from where do you have the identifier of that value?

nice to know! from where do you have the identifier of that value?

oh, just know things )


[REQUEST] Allow picking images from external app

I think allowing to pick images from external image apps would be great.
All image apps have their own way of sorting albums and images, so it's easier to find a picture with my favorite image app. Adding a button in the upper bar which opens the dialog to select an image would be great.
Maybe something like this here:
http://wintechtutorials.com/blog/android-capture-image-from-camera-and-select-image-from-gallery/

I'll have a look in to this, thanks for the suggestion 🙂


[WIP] Rounding time

See #36
To do:

[x] Settings page connected to the store
[x] Write tests for rounding settings
[x] Round durations on timesheet
[x] Round durations on reports

Rounded time on time sheets


Use template engine

Is your feature request related to a problem? Please describe.
By using a template engine like EJS, you can reference other inputs and perform simple calculations. For example, in a template like an invoice, you can calculate the subtotal by summing unitPrice and quantity, and you should be able to compute the total price based on the tax rate.
Additionally, making the schema containing variables read-only can reduce the logic during generation.
Describe the solution you'd like
Use EJS
Describe alternatives you've considered
Users of the library calculate templates and inputs.
Additional context
No response

It should be achievable by running EJS within replacePlaceholders. Be cautious about security, as XSS and other issues may arise.

This will handle at https://github.com/pdfme/pdfme/pull/620


classical shor: possible optimization

Hello,
I think there is a suspicious check in the attack in subject:
https://github.com/RsaCtfTool/RsaCtfTool/blob/c604a214e7b6612718dc4beade2d1a2361fdde72/attacks/single_key/classical_shor.py#L20C17-L20C17
        if gcd(n, a):

Assuming n is always greater than 0, this cannot be false. An option could be to remove the condition, but I think it have been put there for a reason. Is it possible the correct condition would have been:
        if gcd(n, a) == 1:

I may open a PR if one or other option is correct.
Many thanks
Best regards
Francesco

Hi
thanks for pointing it out, I wrote that piece of code by scratch and I forgot to check gcd(n, a)==1.
Please see my changes in the latest commit.

Looks good to me. Thanks for fixing it!
I'm rewriting the attacks in rust to benchmark them in order to compare the speed of the two languages. As soon as I have some number I may open a new issue. What do you think?

You are welcome.
Yes, you can let me know but the codebase of this project is in python3/sage, that won't change.
I'm closing this.

Dear @daedalus
this is my first rust project and I know there is large room for improvement:
https://github.com/fansanelli/rust_attack
The idea of transform your great suite into a library was turning in my mind from a while.
I hope it's fine for you the credits part (I'm a disaster with writing docs) or if you have any kind of comments about this or whatever please let me know.
I know what you said about the python+sage, but if I'll find a good trade-off, I'll tell you in order to re-evaluate an optional PyO3 dep for functions or attacks (or both).
A great idea came from https://github.com/alpertron/calculators that turned it's attacks in webassembly in order to run into a web browser. Rust may offer also this option. I didn't know how to reach you, if you want to continue this conversation maybe we can move it in "rust_attack" issues.

It look

It looks good to me.


[WIP] Several improvements related to CI and GitHub Actions

This pull request makes the following changes:

Run GitHub Actions CI on both Linux and Windows
Disable AppVeyor
README: add the GitHub Actions badge; delete the Travis badge; delete the AppVeyor badge
Update the TagBot configuration to the latest recommended configuration
When building the documentation, clone the General registry over Git - this ensures that we have access to the latest registered versions of all dependencies
Temporarily disable the doctests.

Awesome, this looks great!
Among the checks, I see:

continuous-integration/appveyor/pr — AppVeyor build failed

Why is this still there after removing Appveyor?
I updated a docstring for the new printing (Vector{Operation} instead of Array{Operation,1}) which caused a doctest failure. But do doctests happen on both versions? Would be good to have docs building again.

You labeled it a WIP; please let me know when it's good to merge.
Also, the nighly job completed quickly because REGISTER_COUNT=16, which means it unrolls much less.

Among the checks, I see:

continuous-integration/appveyor/pr — AppVeyor build failed

Why is this still there after removing Appveyor?

Ugh. AppVeyor. You have to log in to the AppVeyor website (https://ci.appveyor.com/project/chriselrod/LoopVectorization-jl) and disable or delete the chriselrod/LoopVectorization-jl project.
Once that's done, let me know - I'll push another commit to this branch to make sure AppVeyor is actually disabled.

I updated a docstring for the new printing (Vector{Operation} instead of Array{Operation,1}) which caused a doctest failure. But do doctests happen on both versions? Would be good to have docs building again.

Let's try running doctests on nightly instead of '1'.

I moved the doctests to run on Julia nightly, and they are passing now!

Codecov Report

Merging #159 (7d60138) into master (38b169c) will decrease coverage by 1.92%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #159      +/-   ##
==========================================
- Coverage   92.00%   90.07%   -1.93%
==========================================
  Files          33       33
  Lines        4529     4526       -3
==========================================
- Hits         4167     4077      -90
- Misses        362      449      +87

Impacted Files
Coverage Δ

src/precompile.jl
0.00% <0.00%> (-98.64%)
:arrow_down:

src/LoopVectorization.jl
66.66% <0.00%> (-8.34%)
:arrow_down:

src/costs.jl
81.96% <0.00%> (-3.52%)
:arrow_down:

src/lower_load.jl
76.50% <0.00%> (-2.19%)
:arrow_down:

src/lower_memory_common.jl
94.16% <0.00%> (-1.67%)
:arrow_down:

src/lower_compute.jl
77.27% <0.00%> (-1.52%)
:arrow_down:

src/lower_constant.jl
84.15% <0.00%> (-1.00%)
:arrow_down:

src/determinestrategy.jl
96.65% <0.00%> (-0.80%)
:arrow_down:

src/reconstruct_loopset.jl
90.00% <0.00%> (+0.32%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 38b169c...7d60138. Read the comment docs.

Good to go now.

Once that's done, let me know - I'll push another commit to this branch to make sure AppVeyor is actually disabled.

I clicked around on their website after making my comment and saw a "only run is there is an appveyor.yml file" and clicked that. Glad that it seems to have worked.
Thanks for the PR!


Push & Publish Images to Docker hub

Why not enable users to publish their container to docker hub, by doing that helping teams share containers super fast.

That's a great suggestion, feel free to create a pull request for it ;)

Yeah right! I'll see what I can start with :+1:

:+1:

+1
The ability to export containers to the user's repo and/or to a local package would very useful to me and many others, I'm sure. This way users could make their own base images and clone them at will in addition to sharing them with the community.

Thanks!

+1


Extend obsoletion action to allow replaced_by and support OBO merge semantics

From @cmungall on May 19, 2015 15:22
The obsolete action current adds the owl:deprecated axiom and removes logical axioms, as well as label prefixing to get around the protege rendering bug, all good.
We would also like a way to easily add consider or replaced_by annotations. This is not too painful in the annotation editor, but OE users are used to doing this with a drag. No idea how easy this...
Another useful feature would be support OBO merge semantics. This means adding an annotation
obo:IAO_0000231=obo:IAO_0000227

i.e. 'has obsolescence reason' = 'terms merged'
to the class. In combination with a replaced_by, this will generate an alt_id in OBOF, as specced here:
https://github.com/owlcs/owlapi/issues/317#issuecomment-67412550
This may be best served by a different action, "Obsolete term for merge" that adds the merge annotation, but leaves it to the user to specify replaced_by?
@balhoff sorry to invade your github repo, we're not presupposing you will do this, I just wanted a place to track this, and your plugin seems the best start
Copied from original issue: balhoff/obo-actions#1

From @balhoff on May 19, 2015 20:16
No problem, would be great to expand this tool.


[BEAM-9615] Use UUIDs in Go SDK Schemas

Straightforward, not much to add.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @lostluck


add empty swift_prefix option value to .proto files

Motivation.
proto-swift-gen script adds the package prefix on a snake case fashion to Swift Struct files.
This makes the code difficult to read and has no convenience to us.
Example:
CompactBlock -> Cash_Z_Wallet_SDK_RPC_CompactBlock
When adding option swift_prefix = ""; the resulting struct Name is CompactBlock

Looks good to me. Only question, do you think an empty string prefix would introduce any risk for collisions with wallet maker's existing name spaces? Or are swift dependencies combined in a way where that can't happen?

Looks good to me. Only question, do you think an empty string prefix would introduce any risk for collisions with wallet maker's existing name spaces? Or are swift dependencies compiled in a way where that can't happen?

If their name collide they would refer to the Module's name first themselves.
Let's say that we chose that our SDK is branded "SwiftZCash".
they would have to go with SwiftZCash.CompactBlock


Vstest.console.exe runs as if the test name is a partial pattern rather than a full name.

Description
Vstest.console.exe runs as if the test name is a partial pattern rather than a full name.
e.g. "vstest.console.exe UnitTests.dll /Tests:Beta /Platform:x64 /InIsolation" runs any test with 'Beta' in the name
Steps to reproduce
Make two tests with one being named as the substring of another. Use vstest.console.exe to run the test with the shorter name.
Both tests will run when only one should.
Expected behavior
Only the specifically named test should run. Specifying wildcards would work e.g. "/Tests:Beta*" would run every test starting with "Beta"
Diagnostic logs
D:\depot-UE4-Mainline>"C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe" D:\depot\Encoder\Binaries\x64\Release\UnitTests.dll /Tests:StabbingTSE_Encoder /Platform:x64 /InIsolation
Microsoft (R) Test Execution Command Line Tool Version 15.9.1
Copyright (c) Microsoft Corporation.  All rights reserved.
Starting test discovery, please wait...
Loading and instantiating ../TestData/StabbingTSE/
.. scene ../TestData/StabbingTSE/ loaded and instantiated in 259.254 millisecond(s)
Loading and instantiating ../TestData/PointedStabbingTSE/
.. scene ../TestData/PointedStabbingTSE/ loaded and instantiated in 286.104 millisecond(s)
Passed   StabbingTSE_Encoder
Passed   PointedStabbingTSE_Encoder
Total tests: 2. Passed: 2. Failed: 0. Skipped: 0.
Test Run Successful.
Test execution time: 1.6650 Seconds
Environment
Windows 10 Professional. Visual Studio Community 15.9.16.

@primalspacesystems This is by design, and has the behavior ever since this argument existed. Changing the behavior can possibly break a lot of scenarios for users.
Can you try using the testcasefilter argument instead ?

Using "... /TestCaseFilter:"StabbingTSE_Encoder" /Platform:x64 /InIsolation" generates the exact same results.
Cheers
John

Am experiencing this is there a workaround if you want to run only just one tests

I am in the same situation but TestCaseFilter is working for me. In powershell for me
vstest.console.exe /TestCaseFilter:"Name=$testName" ...


⚠ AnimeBytes has degraded performance

In e313a1c, AnimeBytes ($AB) experienced degraded performance:

HTTP code: 200
Response time: 1269 ms

Resolved: AnimeBytes performance has improved in 99c6d2f.


[4510] Add rspec-retry gem to Register

Context
Trello: https://trello.com/c/Bcoyw3fw/4510-s-capture-more-information-around-the-flakey-tests-so-we-can-try-and-fix-them
We have some issues with flakey specs on Register - we are going to add rspec-retry as a first step and then build some form of reporting for the flakey specs.
We are using a DfE Digital forked version of rspec-retry because this has some custom functionality. Apply are also using this.
Changes proposed in this pull request

Add DfE-Digital forked version of rspec-retry
Set to retry 3 times across failing tests

Guidance to review

Run the test suite locally and take a look at CI, make sure nothing untoward.
Check if you think retrying 3 times is the right number of tries

Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 [S] Capture more information around the flakey tests so we can try and fix them


Fix import OFX file with non-ascii characters content

It is not fixing the non-ascii characters problem. https://github.com/jseutter/ofxparse/pull/179 will handle the right scenario


Sometimes, clicking automatically starts a highlight

Prerequisites

[X] Can you reproduce the problem in safe mode?
[X] Are you running the latest version of Atom?
[X] Did you check the debugging guide?
[X] Did you check the FAQs on Discuss?
[X] Are you reporting to the correct repository?
[X] Did you perform a cursory search to see if your bug or enhancement is already reported?

For more information on how to write a good bug report or enhancement request, see the CONTRIBUTING guide.
Description
Sometimes, especially after long periods of use, clicking in the text editor area automatically starts selecting text.
Steps to Reproduce

Open a file
Left-click somewhere in text (don't drag)

Expected behavior: The text cursor moves to where you click.
Actual behavior: This happens, but sometimes, it starts a text selection. Moving the cursor will show that text is being selected
Versions
Atom    : 1.11.1
Electron: 0.37.8
Chrome  : 49.0.2623.75
Node    : 5.10.0

I am running Atom on Ubuntu 15.10 (Linux kernel version 4.2.0).

Closing as a duplicate of https://github.com/atom/atom/issues/11691. Feel free to subscribe there for updates.


Does the kernelized memory need training?

I use the kernelized memory when evaluate STCN while did not use it when training. But the result showed a slight decrease. The raw J&F-Mean of davis2016val is 0.916, davis2017val is 0.853and davis2017testdev is 0.755 . After I use the kernelized memory, J&F-Mean of davis2016val is 0.913, davis2017val is 0.852 and davis2017testdev is 0.750 . Does it because the kernelized memory need training? But why it need training since it has no trainable parameters?

It does not need training. It might be because KMN is implicitly solving the memory domination problem using spatial query constraint, leading to diminishing returns when used in conjunction with STCN.


Update process spawning in coverage.js for mocha 2.2.x

Bumps mocha to 2.2.1, and fixes the error highlighted in https://github.com/mochajs/mocha/issues/1585:
0.80s$ npm run-script coverage
><EMAIL_ADDRESS>coverage /home/travis/build/sass/node-sass
> node scripts/coverage.js
/home/travis/build/sass/node-sass/node_modules/.bin/_mocha: 1: /home/travis/build/sass/node-sass/node_modules/.bin/_mocha: /bin: Permission denied
npm ERR<EMAIL_ADDRESS>coverage: `node scripts/coverage.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>coverage script.
npm ERR! This is most likely a problem with the node-sass package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node scripts/coverage.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls node-sass
npm ERR! There is likely additional logging output above.
npm ERR! System Linux 2.6.32-042stab090.5
npm ERR! command "/home/travis/.nvm/v0.10.36/bin/node" "/home/travis/.nvm/v0.10.36/bin/npm" "run-script" "coverage"
npm ERR! cwd /home/travis/build/sass/node-sass
npm ERR! node -v v0.10.36
npm ERR! npm -v 1.4.28
npm ERR! code ELIFECYCLE
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /home/travis/build/sass/node-sass/npm-debug.log
npm ERR! not ok code 0

Thanks @danielstjules! :+1: :tada:

@danielstjules, @dougwilson,
9 of our CLI tests are failing on AppVeyor with timeout of 2000ms exceeded:
https://ci.appveyor.com/project/sass/node-sass/build/715/job/fdom3ikwcnejwm43#L952. No doubt v2.2.1 fixed a lot of other issues we were having on AppVeyor (as reported here: https://github.com/mochajs/mocha/issues/333#issuecomment-70220636). But now all 676+ tests pass locally and those nine fail on AppVeyor.
Any ideas how to investigate this issue further?

Have you tried bumping up the timeouts for those tests? Also, it might be worth setting up a branch that watches stdout and stderr for those child processes. See if the child process is doing anything when running on appveyor.

I'll take a closer look soon, but a timeout bump may help, since it sometimes seems like AppVeyor does execute a bit slow, so since the timeouts are based on wall clock time and not CPU time, it can be easy to trip them on a slow environment.
Another possible thing to try locally is just to run your tests piped to more and see if that causes them to fail. I do this a lot because it sometimes brings up issues I see on AppVeyor and not on my local. Ex npm test | more.

Thanks guys. I tested with this diff:
diff --git a/test/cli.js b/test/cli.js
index 78edace..f9c1d3c 100644
--- a/test/cli.js
+++ b/test/cli.js
@@ -7,6 +7,8 @@ var assert = require('assert'),
     fixture = path.join.bind(null, __dirname, 'fixtures');

 describe('cli', function() {
+  this.timeout(0);
+
   describe('node-sass < in.scss', function() {
     it('should read data from stdin', function(done) {
       var src = fs.createReadStream(fixture('simple/index.scss'));

(disabling suite-specific timeout)
and the result is a never-ending failure: https://ci.appveyor.com/project/am11/node-sass/build/job/36c5pdnjvexmjd4n.

I don't see you listening to the close or exit event from the spawn in this test: https://github.com/sass/node-sass/blob/c97411eefafa837130033fce7aa1112caa37270d/test/cli.js#L135-L152

Is it necessary? We are using .once('data' though. Could it be that it exits the child-process before executed?

Right, but that's the test it's stuck on. Clearly if the child process never sends data (perhaps because of a test failure), then hat test will hang forever, which is what's currently happening.

@dougwilson, tried both close and exit, it is still stuck at same place.
Commits: https://github.com/am11/node-sass/commit/e69d50dc7f79dcb34dcd1cbe56b1dd89bdc0b750 and https://github.com/am11/node-sass/commit/bdd98f8ec564d492409bd31f3682f6ee9a5c1252.

You need to listen on thie child process, not the stream. i.e. bin.once('exit' or bin.once('close'. More information can be found in the Node.js documentation: https://nodejs.org/api/child_process.html#child_process_class_childprocess

Sorry I missed that and thanks for the link! 8-)
Unfortunately, it seems like it does not fire those events either: https://ci.appveyor.com/project/am11/node-sass/build/job/as3t5so2i09tab1e, corresponding commit: https://github.com/am11/node-sass/commit/2069bcf9b0af68c08608cfeea3f75167cf335abe.

Ah, gotcha. I just realized you're calling ./bin/node-sass --watch. I suppose that process would never exit, now would it :/? I mean, if it's hanging there, it would seem that the node-sass bin is just never detecting the file change. What happens if you change https://github.com/am11/node-sass/blob/2069bcf9b0af68c08608cfeea3f75167cf335abe/test/cli.js#L162 to like 5000? Maybe the file is being touched before the child process is actually ready to start watching?

@dougwilson good catch, I have added necessary kills in #888.


RPC setup fails with higher concurrency

Issue
I'm close to using LavinMQ as my replacement for REST, but ran into a blocker. I'm getting this error in the worker.rb process (see below), when doing concurrent requests:
#<Thread:0x00007f393670c738 /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/channel.rb:327 run> terminated with exception (report_on_exception is true):
worker.rb:95:in `rescue in send': WorkerClient.send > Connection closed (504) CHANNEL_ERROR - second 'channel.open' seen (20/10) (RuntimeError)
        from worker.rb:59:in `send'
        from worker.rb:31:in `block in start'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/channel.rb:555:in `consume_loop'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/channel.rb:327:in `block (2 levels) in basic_consume'
/var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/connection.rb:180:in `rescue in write_bytes': Connection closed (504) CHANNEL_ERROR - second 'channel.open' seen (20/10) (AMQP::Client::Error::ConnectionClosed)
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/connection.rb:175:in `write_bytes'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/channel.rb:541:in `write_bytes'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/channel.rb:49:in `open'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/connection.rb:102:in `channel'
        from worker.rb:64:in `send'
        from worker.rb:31:in `block in start'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/channel.rb:555:in `consume_loop'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/channel.rb:327:in `block (2 levels) in basic_consume'
/var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/connection.rb:177:in `write': closed stream (IOError)
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/connection.rb:177:in `block in write_bytes'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/connection.rb:176:in `synchronize'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/connection.rb:176:in `write_bytes'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/channel.rb:541:in `write_bytes'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/channel.rb:49:in `open'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/connection.rb:102:in `channel'
        from worker.rb:64:in `send'
        from worker.rb:31:in `block in start'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/channel.rb:555:in `consume_loop'
        from /var/lib/gems/3.1.0/gems/amqp-client-1.1.7/lib/amqp/client/channel.rb:327:in `block (2 levels) in basic_consume'

Reproduction
client.rb
# frozen_string_literal: true

$stdout.sync = true

require 'bundler/setup'
require 'amqp-client'
require 'json'
require 'parallel'
require 'zlib'

Signal.trap('INT') do
  puts "[#{Time.now.strftime('%Y-%m-%d %H:%M:%S')}] - Received SIGINT, shutting down"
  exit
end

Signal.trap('TERM') do
  puts "[#{Time.now.strftime('%Y-%m-%d %H:%M:%S')}] - Received SIGTERM, shutting down"
  exit
end

module Client
  @conn = AMQP::Client.new('amqp://guest:guest@localhost').connect

  def send(data:, queue:)
    mutex = Mutex.new
    cv = ConditionVariable.new
    response = nil

    ch = @conn.channel
    ch.basic_consume('amq.direct.reply-to', no_ack: true, worker_threads: 1) do |msg|
      msg.body = Zlib.gunzip(msg.body) if msg.properties.content_encoding == 'gzip'
      msg.body = JSON.parse(msg.body) if msg.properties.content_type == 'application/json'

      mutex.synchronize do
        response = msg.body
        cv.signal
      end
    end

    ch.basic_publish_confirm(
      Zlib.gzip(data),
      '',
      queue,
      reply_to: 'amq.direct.reply-to',
      content_type: 'application/json',
      content_encoding: 'gzip',
      persistent: true,
      mandatory: true,
      timestamp: Time.now.to_i
    )

    mutex.synchronize do
      cv.wait(mutex)
    end

    ch.close

    response.transform_keys(&:to_sym)
  rescue StandardError => e
    raise("#{name}.#{__method__} > #{e.message}")
  end

  class << self
    include Client
  end
end

start = Time.now

array = 150.times.collect { '{"msg":"Ping!"}' }
responses = Parallel.map(array, in_threads: 100) do |data|
  Client.send(data:, queue: 'test.queue')
end

puts "Finished all requests at rate of #{(responses.length / (Time.now - start)).round} rps"
puts responses.last.to_json

worker.rb
# frozen_string_literal: true

$stdout.sync = true

require 'bundler/setup'
require 'amqp-client'
require 'json'
require 'zlib'

Signal.trap('INT') do
  puts "[#{Time.now.strftime('%Y-%m-%d %H:%M:%S')}] - Received SIGINT, shutting down"
  exit
end

Signal.trap('TERM') do
  puts "[#{Time.now.strftime('%Y-%m-%d %H:%M:%S')}] - Received SIGTERM, shutting down"
  exit
end

module Worker
  @conn = AMQP::Client.new(ENV.fetch('MQ_URL', nil), connection_name: 'Worker').connect

  def start(threads: 0)
    ch = @conn.channel
    ch.queue_declare(ENV.fetch('MQ_QUEUE', nil))

    ch.basic_consume(ENV.fetch('MQ_QUEUE', nil), no_ack: false, worker_threads: threads) do |msg|
      msg.body = Zlib.gunzip(msg.body) if msg.properties.content_encoding == 'gzip'
      msg.body = JSON.parse(msg.body) if msg.properties.content_type == 'application/json'

      data = WorkerClient.send(data: msg.body, queue: 'test.rpc.queue').to_json

      ch.basic_publish(
        Zlib.gzip(data),
        '',
        msg.properties.reply_to,
        content_type: 'application/json',
        content_encoding: 'gzip',
        persistent: true,
        timestamp: Time.now.to_i
      )

      msg.ack
    end

    sleep
  rescue StandardError => e
    raise("#{name}.#{__method__} > #{e.message}")
  end

  class << self
    include Worker
  end
end

module WorkerClient
  @conn = AMQP::Client.new(ENV.fetch('MQ_URL', nil), connection_name: 'WorkerClient').connect

  def send(data:, queue:)
    mutex = Mutex.new
    cv = ConditionVariable.new
    response = nil

    ch = @conn.channel
    ch.basic_consume('amq.direct.reply-to', no_ack: true, worker_threads: 1) do |msg|
      msg.body = Zlib.gunzip(msg.body) if msg.properties.content_encoding == 'gzip'
      msg.body = JSON.parse(msg.body) if msg.properties.content_type == 'application/json'

      mutex.synchronize do
        response = msg.body
        cv.signal
      end
    end

    ch.basic_publish_confirm(
      Zlib.gzip(data.to_json),
      '',
      queue,
      reply_to: 'amq.direct.reply-to',
      content_type: 'application/json',
      content_encoding: 'gzip',
      persistent: true,
      mandatory: true,
      timestamp: Time.now.to_i
    )

    mutex.synchronize do
      cv.wait(mutex)
    end

    ch.close

    response.transform_keys(&:to_sym)
  rescue StandardError => e
    raise("#{name}.#{__method__} > #{e.message}")
  end

  class << self
    include WorkerClient
  end
end

Worker.start(threads: 100)

worker_rpc.rb
# frozen_string_literal: true

$stdout.sync = true

require 'bundler/setup'
require 'amqp-client'
require 'json'
require 'zlib'

Signal.trap('INT') do
  puts "[#{Time.now.strftime('%Y-%m-%d %H:%M:%S')}] - Received SIGINT, shutting down"
  exit
end

Signal.trap('TERM') do
  puts "[#{Time.now.strftime('%Y-%m-%d %H:%M:%S')}] - Received SIGTERM, shutting down"
  exit
end

module Worker
  @conn = AMQP::Client.new(ENV.fetch('MQ_URL', nil), connection_name: 'Worker').connect

  def start(threads: 0)
    ch = @conn.channel
    ch.queue_declare(ENV.fetch('MQ_QUEUE', nil))

    ch.basic_consume(ENV.fetch('MQ_QUEUE', nil), no_ack: false, worker_threads: threads) do |msg|
      msg.body = Zlib.gunzip(msg.body) if msg.properties.content_encoding == 'gzip'
      msg.body = JSON.parse(msg.body) if msg.properties.content_type == 'application/json'

      data = { msg: 'Pong!' }.to_json

      ch.basic_publish(
        Zlib.gzip(data),
        '',
        msg.properties.reply_to,
        content_type: 'application/json',
        content_encoding: 'gzip',
        persistent: true,
        timestamp: Time.now.to_i
      )

      msg.ack
    end

    sleep
  rescue StandardError => e
    raise("#{name}.#{__method__} > #{e.message}")
  end

  class << self
    include Worker
  end
end

Worker.start(threads: 100)

Steps to reproduce (start everything in new tabs):

docker run --rm -p 5672:5672 -p 15672:15672 cloudamqp/lavinmq
MQ_URL='amqp://guest:guest@localhost' MQ_QUEUE='test.queue' ruby worker.rb
MQ_URL='amqp://guest:guest@localhost' MQ_QUEUE='test.rpc.queue' ruby worker_rpc.rb
ruby client.rb

If you change the 150.times.collect { '{"msg":"Ping!"}' } to 2.times.collect { '{"msg":"Ping!"}' } it works.
Interestingly this does not fail on my M1 MacBook, even with higher numbers. But on a server with weaker shared CPUs it does so, starting at only 50 threads.

FYI: There's also a related chat on the 84codes Slack: https://app.slack.com/client/T01K2BK0K6G/C01K2BK1PT6

The library isn't thread safe and the problem here is that two threads are trying to open a channel at the same time and the same channel id is used.

@spuun the README here says "fully thread-safe" so this is a big that we should address? Not "close as completed"

Please take a look at https://github.com/cloudamqp/amqp-client.rb/pull/30 which implements a RPC server and client directly into the lib.

I was too quick here.


refactor detected event processing code

Refactor event.rs by separating it into smaller files.

Codecov Report
Patch coverage: 4.50% and no project coverage change.

Comparison is base (e9538e9) 52.35% compared to head (a411f80) 52.35%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #65   +/-   ##
=======================================
  Coverage   52.35%   52.35%
=======================================
  Files          46       52    +6
  Lines        9245     9245
=======================================
  Hits         4840     4840
  Misses       4405     4405

Files Changed
Coverage Δ

src/graphql/event.rs
74.60% <ø> (+22.28%)
:arrow_up:

src/graphql/event/conn.rs
3.12% <3.12%> (ø)

src/graphql/event/ldap.rs
3.92% <3.92%> (ø)

src/graphql/event/ftp.rs
4.00% <4.00%> (ø)

src/graphql/event/http.rs
4.00% <4.00%> (ø)

src/graphql/event/rdp.rs
6.66% <6.66%> (ø)

src/graphql/event/dns.rs
8.00% <8.00%> (ø)

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


Remove-pattern-string-from-compute-forwarding-rules.patch

This patch removes a spurious string in some descriptions
The underlying issue was fixed in pulumi-java since, and we should be OK to remove this patch.

Closed via https://github.com/pulumi/pulumi-gcp/pull/2355.


[cosmos] allow custom HttpClient

Packages impacted by this PR
@azure/cosmos
Issues associated with this PR
Fixes #23026
Describe the problem that is addressed by this PR
Add httpClient option to CosmosClientOptions to allow for setting a custom transport.
Checklists

[x] Added impacted package name to the issue description
[x] Added a changelog (if necessary)

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/cosmos


The "for" attribute is stripped out by the sanitizer

Add it as a protected exception for setAttribute() use

Fixed in v0.2.0


System.ComponentModel.Win32Exception (13): Permission denied

Problem encountered on https://dotnet.microsoft.com/learn/aspnet/hello-world-tutorial/run
Operating System: linux
Hi. Im facing an issue in asp.net tutorial. After installing i just created an app using:
 $ dotnet new webApp -o myWebApp --no-https

Then i just ran:
myWebApp/$ dotnet run

which fails with this traceback:
System.ComponentModel.Win32Exception (13): Permission denied
   at System.Diagnostics.Process.ForkAndExecProcess(String filename, String[] argv, String[] envp, String cwd, Boolean redirectStdin, Boolean redirectStdout, Boolean redirectStderr, Boolean setCredentials, UInt32 userId, UInt32 groupId, UInt32[] groups, Int32& stdinFd, Int32& stdoutFd, Int32& stderrFd, Boolean usesTerminal, Boolean throwOnNoExec)
   at System.Diagnostics.Process.StartCore(ProcessStartInfo startInfo)
   at System.Diagnostics.Process.Start()
   at Microsoft.DotNet.Cli.Utils.Command.Execute()
   at Microsoft.DotNet.Tools.Run.RunCommand.Execute()
   at Microsoft.DotNet.Tools.Run.RunCommand.Run(String[] args)
   at Microsoft.DotNet.Cli.Program.ProcessArgs(String[] args, ITelemetry telemetryClient)
   at Microsoft.DotNet.Cli.Program.Main(String[] args)

Here is dotnet --info:
.NET Core SDK (reflecting any global.json):
 Version:   3.1.102
 Commit:    573d158fea

Runtime Environment:
 OS Name:     arch
 OS Version:
 OS Platform: Linux
 RID:         arch-x64
 Base Path:   /usr/share/dotnet/sdk/3.1.102/

Host (useful for support):
  Version: 3.1.2
  Commit:  e5d3faa5b3

.NET Core SDKs installed:
  3.1.102 [/usr/share/dotnet/sdk]

.NET Core runtimes installed:
  Microsoft.NETCore.App 3.1.2 [/usr/share/dotnet/shared/Microsoft.NETCore.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

Needless to say that i dont know asp.net.. so any help is wellcome.

It seems like you don't have execute permissions for whatever path it is trying to execute. @wli3 @dsplaisted any ideas what it could be?

How did you install the .NET Core SDK?  I think this might happen if you install it as root and then try to use it as a normal user, or vice versa.

Hi! Sorry for the delay i was on a trip.
I just solved it.
Altought at the time to post this question i double checked execution permissions in the project folder(traceback explicitly says it), it was a permission issue, as @scalablecory mentioned.
So... here's the situation:
I've got a "projects" folder where i put all my sources, which is on another partition mounted with fstab. It has execution permissions but among all options it seems like exec directive should be after user directive. So i changed:
#/etc/fstab
UUID....  <mount_point> <fs_type> <some_opts>,exec,user <dump> <pass>
to
UUID....  <mount_point> <fs_type> <some_opts>,user,exec <dump> <pass>

And it ran smoothly, so im closing this issue as it was an OS issue rather than dotnet.
Thanks for the answers anyway!!

Hi @chudix
Can you please tell me how you solved this issue?
I am facing exactly this permission problem. and unable to solve.

Hi @Arslan-Munir.
As i mentioned in closing comment it was a wrong fstab configuration, refer to that for a deeper explanation. There is a thing with user and exec options position so i just change my fstab file and reset.
Anyway... if you are seeing that error message you should check for execution permissions in the project's folder. Or maybe check @dsplaisted comment:

How did you install the .NET Core SDK? I think this might happen if you install it as root and then try to use it as a normal user, or vice versa.

Hope you find the solution. :)

@chudix
I follow this guideline to install dotnet core on Ubuntu
https://docs.microsoft.com/en-us/dotnet/core/install/linux-ubuntu
My project is in another mounted disk.
When i run dotnet run from that folder, i got permission error.
The permission are like this

Can you help me? I would really thankful. Its almost three days now and i am stuck.

I'll do my best.
I suggest:
Make a dummy project on your os disk/partition where you installed dotnet.
$ dotnet new dummy -o dummy --no-https

Then run it with dotnet run. If it runs then the problem is how you are mounting the disk for sure.
dummy/$ dotnet run

Btw... how are you mounting it? I remember that permissions seemed to be right but that option misconfig was messing all silently.

@chudix i did what you said. and i got the same error on both dotnet run and dotnet watch run.
I mount disk at the time of installation. One thing i notice that when i restart or power on the machine, the partitions are again mounted. but data inside them not lose.

So it seems like it's the install rather than a mounting issue.
I would uninstall dotnet via apt and give it a try to a non admin scripted install...
https://docs.microsoft.com/en-us/dotnet/core/install/linux-ubuntu#scripted-install
Anyway, since i'm not an ubuntu user i cant help you troubleshot the apt installation. I'll give a try on a VM later... and see what happens.
Maybe you could take that issue to the ubuntu forums?


Dartpad doesn't work at all in Firefox 74 (Nightly)

No code appears in the editor, and what look like they may be the edges of buttons appear there instead, but don't do anything.

FWIW, I've been using DartPad with Firefox 75.0 on Linux without any problems.

We haven't had any new reports of similar issues for a few years now, so I'm going to close this.
Thanks again for reporting this and providing all of those details!
If anyone reading this experiences a similar issue on an evergreen browser, please let us know by filing a new issue. Thanks :)


for loop edge case handling

this happens occasionally in the unmodified ss13 repos, where the statement expression has side effects as well

I'd rather not use any lookahead if possible (SavePosition()/RestorePosition()). Can this be done another way?

This fixes #527


Fix globs in runIf configurations.

Several tests have not been running on presubmit after the glob behavior was fixed in the cocoon backend. This change updates runIf configs to make consistent use of globs.
Bug: https://github.com/flutter/flutter/issues/122157
Pre-launch Checklist

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] I read the Tree Hygiene wiki page, which explains my responsibilities.
[X] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[X] I signed the CLA.
[X] I listed at least one issue that this PR fixes in the description above.
[X] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making, or this PR is test-exempt.
[X] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

cc @keyonghan, I thought https://github.com/flutter/flutter/issues/122157 was fixed by https://github.com/flutter/cocoon/pull/2516. Were ALL of these runIf conditions really wrong?

Cocoon backend glob fix was https://github.com/flutter/cocoon/pull/2506

Cocoon backend glob fix was flutter/cocoon#2506

So all of these runIf conditions were wrong for 2 weeks because they didn't actually glob and we added a $ to the end of the regex?

ll of these runIf conditions were wrong for 2 weeks because they didn't actually glob and we added a $ to the end of the regex

That's correct, we created that fix because the builds using the glob were not being run correctly but didn't check the configs in .ci.yaml apart from the ones referenced in the bug.

ll of these runIf conditions were wrong for 2 weeks because they didn't actually glob and we added a $ to the end of the regex

That's correct, we created that fix because the builds using the glob were not being run correctly but didn't check the configs in .ci.yaml apart from the ones referenced in the bug.

Oh interesting. I'm surprised this wasn't more disruptive.


chore: re-enable integration tests

Update test harness and re-enable integration test profile
closes: https://github.com/open-feature/java-sdk/issues/113

In most cases I would wait for approval from @justinabrahms but this is only re-enabling existing testing.


QR Code Reader

Description 📜
Fixes #1503

Type of change 📝
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] Add a new project
[x] Modified or enhanced the existing projects
[x] Improve the readme of any project

Domain of Contribution 📊
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] Project using HTML, CSS & JS

Checklist ✅
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] I have performed a self-review of my own code or work.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I'm GSSoC'22 contributor

Screenshots / Gif (Recommended) 📸

https://user-images.githubusercontent.com/82721772/167534679-13a41a95-ce18-41d1-8421-f602c2d74841.mp4
In last, it is showing "Can't Scan......." because it that time I uploaded a normal pic not QR. So, it shows that message to user.

@gurjeetsinghvirdee pls review my PR.

https://www.codingnepalweb.com/qr-code-reader-html-javascript/ copied from here


Split up media APIs

I suggest splitting the media APIs up into:

Blogs (e.g. Medium)
Books (e.g. Good Reads)
Images (e.g. Unsplash, Flickr, Imgur, dribbble)
Media (e.g. Chuck Norris Database, Rijksmuseum)
Music (e.g. Soundcloud, iTunes, TheMovieDB)
Social (e.g. Facebook, Twitter, G+)
Video (e.g. YouTube, Vimeo)

This issue should be fixed in PR #231

This issue can be closed. PR #231 has been merged into master.


Change the submodule URL to use the HTTPS scheme.

Fixes #19.

that looks good to me, @AlanQuatermain / @Lukasa any issues with this?

@swift-nio-bot test this please