verifyToken and others are empty

I followed this guide to set things up https://hackernoon.com/setting-up-email-verification-in-feathersjs-ce764907e4f2.   email works, so, that's good.  Now I'm working toward generating the verifyToken url.  The part that's  not working is setting the token in the URL.  The tokens are not on the user by the time the url is generated.
Admittedly, I'm tired.  Hopefully I'm just missing something simple.
note: I'm hitting users.create from another service called signup sign up has no auth on it.   signup.create keeps a record of the sign up and any other details that may not be involved in the actual user model.  Organization name, for example.
I can see that the tokens are getting created and added to the object in verifyHooks.addVerification().  Here's a screenshot of the state of hook just before it's returned. https://www.evernote.com/l/ADodJylat31J77RtYMYJpF4i6K8rwet-DtY
As you can see, the verifyToken is there.  Here is the list of user before create hooks:
create: [hashPassword('password'), authenticate('jwt'), userCreateValidator(), verifyHooks.addVerification()],
I have stripped this down to hashPassword('password'), verifyHooks.addVerification() and tested.  The behavior is the same.
Here is the state of context.data just before triggering the resendVerifySignup case in notifier.js https://www.evernote.com/l/ADoginovCKxK7KnbCiYHrYXP6y_m1UPVNQI
As you can see, they're on the context.data object.
Then, in the notifier.js, the values are gone.  Here's another screenshot:  https://www.evernote.com/l/ADpZVElyGWJNb5kjyo511XLsYUJHJwAtqnI
Here are the user after create hooks: create: [       addUserToOrg(),       addUserToGroup(),       context => {         accountService(context.app).notifier('resendVerifySignup', context.result);       },       verifyHooks.removeVerification()],
I have stripped this down to only context => {         accountService(context.app).notifier('resendVerifySignup', context.result);       },       verifyHooks.removeVerification() and tested.  The behavior is the same.
other details...
`
node v12.16.3
"dependencies": {
"@feathersjs/authentication": "^4.5.3",
"@feathersjs/authentication-local": "^4.5.3",
"@feathersjs/authentication-oauth": "^4.5.3",
"@feathersjs/configuration": "^4.5.3",
"@feathersjs/errors": "^4.5.3",
"@feathersjs/express": "^4.5.3",
"@feathersjs/feathers": "^4.5.3",
"@feathersjs/socketio": "^4.5.3",
"@feathersjs/transport-commons": "^4.5.3",
"aws-sdk": "^2.673.0",
"compression": "^1.7.4",
"cors": "^2.8.5",
"feathers-authentication-management": "^2.0.1",
"feathers-hooks-common": "^5.0.3",
"feathers-mailer": "^3.0.1",
"feathers-sequelize": "^6.1.0",
"helmet": "^3.22.0",
"jwt-decode": "^2.2.0",
"node-redshift": "^0.1.5",
"nodemailer-smtp-transport": "^2.7.4",
"pg": "^8.0.3",
"sequelize": "^5.21.7",
"serve-favicon": "^2.5.0",
"winston": "^3.2.1"
},`

I wonder if the problem is not in the way you wrote the hook with the notifier, if you use brackets then you should use a return statement on the promise from the notifier, otherwise don't use brackets: context => accountService(context.app).notifier('resendVerifySignup', context.result). Indeed I guess the hook chain does not await the async notifier call so that the removeVerification hook is run before the actual processing of the notifier.
By the way in case you did not notice the removeVerification should also be used as an after get/find hook, the goal is to avoid getting the tokens on the client side.

Hi @claustres, I think I see the issue.  While addVerification adds the tokens to hook.data, the notifier is passed context.result which has no tokens.   I suppose what is happening it that the user is getting created prior to the tokens existing in the data object - perhaps.   This might be related to how I'm calling users.create from the signup service.
I removed the removed the removeVerification hook completely and retested.  The same behavior persists.
re the removeVerification hook - thanks for the heads up.  I had not finished the article yet.  I'll add them there too.

This is a strange behavior, hook.result is what is returned by the DB adapter after the create so that if hook.data contains the tokens then it should as well, unless it is removed by another process.

sequelize is building the insert like this INSERT INTO "users" ("id","email","password","createdAt","updatedAt") VALUES (DEFAULT,$1,$2,$3,$4)
after addVerification has returned.
My users model looks like this and the columns are in the table
`// See http://docs.sequelizejs.com/en/latest/docs/models-definition/
// for more of what you can do here.
const Sequelize = require('sequelize');
const DataTypes = Sequelize.DataTypes;
module.exports = function (app) {
const sequelizeClient = app.get('sequelizeClient');
const users = sequelizeClient.define('users', {
email: {
  type: DataTypes.STRING,
  allowNull: false,
  unique: true
},
password: {
  type: DataTypes.STRING,
  allowNull: false
},
googleId: { type: DataTypes.STRING },
githubId: { type: DataTypes.STRING },

isVerified: { type: DataTypes.BOOLEAN },
verifyToken: { type: DataTypes.STRING },
verifyExpires: { type: DataTypes.DATE },
verifyChanges: { type: DataTypes.STRING },
resetToken: { type: DataTypes.STRING },
resetExpires: { type: DataTypes.DATE },

}, {
hooks: {
beforeCount(options) {
options.raw = true;
}
}
});
// eslint-disable-next-line no-unused-vars
users.associate = function (models) {
// Define associations here
// See http://docs.sequelizejs.com/en/latest/docs/associations/
users.belongsToMany(models.organizations,{through: 'users_to_organizations'});
users.belongsToMany(models.groups, {through: 'users_to_groups'});
};
return users;
};
`
This is the signup after hook that creates the user
`module.exports = (options = {}) => {
return async context => {
const users = context.app.service('users');
const password = 'supersecret';
const user = {
  'email': context.data.email,
  'password': password,
};
//
users.create(user)
  .then()
  .catch((error) => {
    console.log(error);
  });

return context;

};
};
`

I bypassed signup and posted a new user with
{
	"email"<EMAIL_ADDRESS>	"password": "supersecret"
}

The same behavior persisted

solved...   user error
I had some junk in the users.class.js that was changing the data object.


İcons(images) instead of ring inner texts

Thx for the great library first of all. Do you think is there a function like we can use icons(symbols ) instead of the text inner rings. Thank you

I haven't added support for images, that'd be a good idea to implement.
Will leave this issue open.


three '-' won't work to make a separation line

I'm using
atom 1.18.0 x64 at Arch linux
markdown-preview-plus 2.4.10
I found that when I make a separation line
'>---'           without any  'space' , it doesn't work
but ,if I insert a 'space' between two charactor. it works
for example
'> ---'
'>- --'
'>-- -'
I'm not sure if it a bug, if so, I hope you can fix it in your next version

It sounds like you're having an issue with the markdown-preview-plus package, you can open an issue in the package repository https://github.com/atom-community/markdown-preview-plus.


Switch from Travis CI to Cirrus CI

Hello.
The same story as in #90: I can't connect GitHub applications for foreign repositories.
Reasons for switch:

Travis CI has phantom fails (another CI can has them too, but I want to try).
Travis CI has regular weird problems with Ruby versions or system gems.
Cirrus CI has Linux + macOS (like Travis CI) + Windows (wow), what's important for this project, I think.
Travis CI has low performance as I saw — it doesn't mean that another free CI will not have low performance too, but for now Cirrus CI looks good in this.

Another alternative: Circle CI (many Ruby projects switched to this, as RuboCop and something else about which I forgot).
But Circle CI:

has more complicated configuration;
doesn't have Windows support.

If you connect this application — I'll write config file and everything else.
(There is example of switching: https://github.com/AlexWayfer/flame/pull/60)

Basically, it resolved by #104.


Make fade out multiply the velocity of a block by the fade instead of it just setting the values.

I always hate that the fade out macro never takes into account the current velocity of the notes that are being faded.
I want the notes to instead of being a set value when the macro is used, to be a percentage of the value that is being used.

Fixed in beacf6a0b61e97073bfc7abac4220e33b575138c


Infinite loop on EOF check

Hello again! Sorry to bother you - after updating to recent master to have the template parsing, I encountered a problem when tokenizer gets into infinite loop. I think it may be caused by this change:
https://github.com/baszalmstra/header-parser/commit/c18dd5202b2b6b71f3fd413859b1a256f44a59c6
as reverting to commit before this does not have this problem.

Oh thats not good! Ill look into it. Would you be able to post a header
that causes the issue?
On Sat, 2 Nov 2019 at 02:17, noizex<EMAIL_ADDRESS>wrote:

Hello again! Sorry to bother you - I have no idea what happened, I
compiled this and ran hundreds of times in last few days and suddenly it
starts hanging on same headers I parsed successfully before. I tried
lastest master (with template changes), I tried reverting up to 3 commits
back and no way I could get this to run properly. Going slightly insane now
as I don't know why it would suddenly start doing this. I tracked it down
to the code that exists tokenizer on EOF:
char c = GetLeadingChar();
char p = peek();
std::char_traits::int_type intc = std::char_traits::to_int_type(c);
std::char_traits::int_type intp = std::char_traits::to_int_type(p);
if(!std::char_traits::not_eof(intc)
Both c and p are equal to -1 ('ÿ'), after to_int_type() casts intc & intp
== 255 while eof() == -1, so this never enters this check if it's
not_eof(). But I honestly have no idea how this suddenly became an issue
when it's been working for days? Maybe you can spot some problem, I tried
my headers, I tried example headers included in the project and each one of
them gets into infinite loop.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/baszalmstra/header-parser/issues/6?email_source=notifications&email_token=ABGDW74IF7F6YNB66WX6LETQRTIK5A5CNFSM4JIBYCZKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HWIBQNA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGDW73LWL6M6COL3F4BIMDQRTIK5ANCNFSM4JIBYCZA
.

Ah thanks! Ill look into it!

I fixed the bug in https://github.com/baszalmstra/header-parser/commit/b78dff9c0accd3350c920708ce7045c37ab7dad8 . Sorry for you troubles!

No problem at all, happy to help testing this! Thanks for helping so fast :) I tried to find the cause but I'd have to understand the whole tokenzier routine so I failed completely. I found one or two more issues which I'll report in another ticket.


Report is not generated in Storage accout

II followed the exact steps mentioned but am still encountering warnings.
Warning: Failed adding PivotTable 'P0': Cannot index into a null array.
Warning: Failed adding chart for pivotable 'P0': Cannot bind argument to parameter 'PivotTable' because it is null.
Warning: Failed adding PivotTable 'P2': Cannot index into a null array.
Warning: Failed adding chart for pivotable 'P2': Cannot bind argument to parameter 'PivotTable' because it is null.
Warning: Failed adding PivotTable 'P3': Cannot index into a null array.
Warning: Failed adding chart for pivotable 'P3': Cannot bind argument to parameter 'PivotTable' because it is null.
Warning: Failed adding PivotTable 'P4': Cannot index into a null array.
Warning: Failed adding chart for pivotable 'P4': Cannot bind argument to parameter 'PivotTable' because it is null.
No output is being sent to the storage account. Could someone assist me with this issue?

Hello @Valli495, the Excel file is being created?
All the required permissions were added?
All the required modules were installed?
Is this a large environment?
If you run the runbook in test mode, do you see any error?


openssl s_server -cert server.crt -key server.key -CAfile rootCA.pem -accept 2502 -www Enter pass phrase for server.key: Error setting EC curve

We are receiving above error to bye pass we used -no_ecdh then came to accept mode So kindly advice how to handle in c program to skip/off the ECDH

As noted in #3303 this is a bug in 1.0.2 that is fixed in recent versions. Since 1.0.2 is in security-fix only mode this won't be fixed.


Gross value in 'Overview' dashboard shows wrong value.

Description
The gross consumption value as reported in the 'overview' dashboard is way off. Like ten times too much.
Expected behavior
A proper value for gross consumption.
Screenshots

Environment

TeslaMate version: 1.21.1
Type of installation: Docker
OS TeslaMate is installed on: CentOS
User OS & Browser: macOS 11.1 + Chrome 90.0.4398.0

What time period did you set in Grafana and how much did you actually drive during that time versus how long was the car standing around (and online)?


source_id registration of CMCC-ESM2-SR5

activity_participation = [CMIP,C4MIP, LS3MIP, LUMIP]
aerosol = MAM3
atmosphere = CAM5.3 (1deg 288x192 longitude/latitude, 30 levels, top at ~2hPa)
atmospheric_chemistry = None
institution_id = CMCC
label = CMCC-ESM2-SR5
label_extended = CMCC-ESM2-SR5
land_ice = None
land_surface = CLM4.5 (BGC mode)
nominal_resolution_atmos = 100 km
nominal_resolution_landIce = None
nominal_resolution_ocean = 100 km
nominal_resolution_aerosol = 100 km
nominal_resolution_atmosChem = None
nominal_resolution_land = 100 km
nominal_resolution_ocnBgchem = 100 km
nominal_resolution_seaIce = 100 km
ocean = NEMO3.6 (ORCA1 tripolar primarly 1 deg lat/lon with meridional refinement down to 1/3 degree in the tropics; 362 x 292 longitude/latitude; 50 vertical levels; top grid cell 0.5 m)
ocean_biogeochemistry = BFM5.1
release_year = 2017
sea_ice = CICE4.0
source_id = CMCC-ESM2-SR5

@escoccimarro thanks for this, we need a couple of extra pieces of information and a couple of clarifications which I have guessed at (between the ### entries) below:
atmosphere = CAM5.3 (1deg; ###288x192 longitude/latitude###; 30 levels; top at ~2 hPa)
ocean = NEMO3.6 (ORCA1 tripolar primarly 1 deg lat/lon with meridional refinement down to 1/3 degree in the tropics; ###360 x 180 longitude/latitude;### 50 vertical levels; top grid cell 0.5 m)
nominal_resolution_aerosol = 100 km
nominal_resolution_atmosChem = None
nominal_resolution_land = 100 km
nominal_resolution_ocnBgchem = 100 km
nominal_resolution_seaIce = 100 km

we updated the information as required.
thanks

@escoccimarro thanks for this, peek at CMIP6_source_id.html and let me know if any further tweaks are required

@escoccimarro @tomaslovato it appears that we have datasets published for the CMCC-ESM2-SR5 model for the LS3MIP and LUMIP activities, however, these are not available through ESGF, were they retracted? The model appears to have been unregistered, somewhat erroneously as it doesn't appear in the CMIP6_source_id.html page.
Can you please confirm the details above are correct (in particular, whether this model will contribute to CMIP and C4MIP), and once I have these confirmed I will re-register, or please clarify if the datasets that we have should be purged, as it is an invalid contribution

@durack1 the datasets related to CMCC-ESM2-SR5 were unfortunately published before the update of the CVs table with the reordering of CMCC model participating to CMIP6 (as discussed in #846). In more detail, changes were made in https://github.com/WCRP-CMIP/CMIP6_CVs/commit/4705e28124103773c0ca23f68cfc56663020f888 and https://github.com/WCRP-CMIP/CMIP6_CVs/commit/d6660d7c814c5d0ace2c883bf32c7123bb0e3dc3.

@escoccimarro @tomaslovato it appears that we have datasets published for the CMCC-ESM2-SR5 model for the LS3MIP and LUMIP activities, however, these are not available through ESGF, were they retracted?

At that time, based on the indication from @MartinaSt we had to retract the data published under CMCC-ESM2-SR5  and republish them under CMCC-ESM2 (she also reported this known issue in #900), as the renaming was not allowed under ESGF due to the version tracking of data.
I confirm that CMCC-ESM2-SR5 was correctly unregistered and you can purge the related datasets from your server.
I hope this will help!

@tomaslovato thanks for clarifying, we'll clean up our local (LLNL/PCMDI) holdings so that these files are deleted. I note that CMCC-ESM2 also includes data for LS3MIP and LUMIP, so I assume the valid (correctly named) datasets are currently available, and deleting the erroneous data is the correct path.
@sashakames @mauzey1 @taylor13 ping! (Will close this issue when we have resolved the local erroneous data issue)

@sashakames, @mauzey1 can we schedule a time to chat about the discussion above https://github.com/WCRP-CMIP/CMIP6_CVs/issues/296#issuecomment-1130289604

@durack1 Open to chatting about this.  Looks like all data is retracted from the index already as I don't see the source_id in the search results.

@sashakames @mauzey1 @taylor13 action today has resolved the ESGF-LLNL inconsistency, so will close


Use consistent key in split

Reliance on the key is useful in some conditions, and the non-monotonicity of the key caused a very nasty bug in our code.

You don't actually need $k, the main thing you need is replacing yield from by foreach. (which will then just use the auto-incrementing default).

True, but it's more explicit this way.


Remove kategorie from seed entry and convert variety to a simple text field.

This pull requests implements the following changes requested by @markus2330 in #35:

completely remove "Kategorie" (seeds and plants database). So the very first entry field should be Art, then Sorte, then Year.

Variety, i.e., Sorte later will be select-able from database but for the first milestone it is simply text to be entered (As I assume we will not be able to implement the "Edit Plants" use case next week, which would be needed to add the varieties.)

Thank you for the review @markus2330.

please update the README.md from backend (there is no varieties anymore)
Yes I missed that. Its now fixed.

http://<IP_ADDRESS>:5173/seeds nor curl <IP_ADDRESS>:8080/api/seeds shows the seeds I just entered? Is it really inserted in the DB? This worked before.

I could not replicate this on my machine. Requests from the frontend, as well as plain http requests sent using curl resulted in new database entries.
Perhaps there was a bug somewhere that was now fixed by accident?
Please verify that this issue is still present with the latest changes and send the relevant curl command if the issue persists.

Documentation/frontend-tests could be improved

I agree, but I would prefer to open another pull request for this issue as it is unrelated to #35 in my opinion.

I could not replicate this on my machine.

My suspicion is that my backend was wrongly configured. Afaik the frontend does not give an error if the connection fails, so I did not notice. I added this to #35.
We now merged the PR with the new plant table (replacing varieties). This created quite a few conflicts here. Let us discuss today in which order we will finish the first milestone.

I was just resolving the merge conflicts and encountered a problem.
As it is, both the selction field for plants and for variety is called "Sorte".
How should these textfields be renamend?

I have noticed that the new seed property plant_id from the master branch is also called "Sorte" now.
How should we rename  the variety field to distinguish these two properties?

Ok, thank you for the answer.
I merged the changes from master into this branch by using two text fields for now.
After the build issues are fixed, I will look into merging both text fields as requested.

There is a double use for Sorte (German)/variety (English) field:

if entered in the Seed entry manually, the Seed database gets an entry variety and this is preferred
the "Plant" is a variety, then the data is used from the Plant database (variety in seed database is NULL in this case)

Users now get a single selectbox/typeahead that allows them to create items that are not in the list. If a new item is created, it will be stored in the variety field of the seed table, otherwise the seed_id will be set as before.

@lukashartl do you know when we will be able to deploy PRs? It would be handy here if I don't need to test locally.


Potential Nape Memory Leak

After running the garbage collector post test, 2-6 bodies left in memory, around 20-30 Vec2 instances
and several other loitering objects all created by Nape (especially ZPP_ objects)
Sample code to test
https://gist.github.com/devon-o/e76c292b78fef036845d
If previously remove bodies shapes it looks better, but still some objects exists
http://forum.starling-framework.org/topic/starling-nape-clicking-and-destroying-body

Two things:

Nape uses objects pools heavily (especcialy for zpp_ object types which you don't interact with in user-side code). There is a function on nape.util.Debug clearObjectPools() which will empty out this pools and leave any remaining pooled objects available for GC to dispose of.

It's very rare for a GC to 'actually' dispose of everything it possibly can, especcialy flash's one. In general the GC will not even attempt to dispose of data that is not trivially found to be collectable unless it needs to to allocate more objects, and won't try and do any deep cleans until it gets desperate, though in my experience it's not as bad as say, the garbage collection in V8 can be.

I'm not saying it's impossible for there to actually be a memory leak, but I would be incredibly suprised as memory usage was very, very heavily tested (all the samples on napephys.com for instance have perfectly flat memory graphs (beyond objects flash allocates itself because they don't care) once they've warmed up meaning nape isn't even allocating new memory at all and just re-using objects from the various pools at that point in time. That is true even if the samples are changed to periodically start again from scratch in the same running instance (effectively doing your startTest/endTest again and again)

fyi, although I doubt I have the code lying around anymore, the way memory was actually tested for nape (not to rely on fragile GC behaviours) was to actually compute the graph of 'all' reachable objects myself from any objects in scope in the test, and from every Nape class type (for static members) so that I could be 100% sure that at any point in time, no object was still reachable from some live reference following a test meaning that if there was ever any memory increase over time, it was due only to GC behaviour and not that there were actually any real memory leaks.

Thanks for reply, Luca.
As for 2nd point: in the gist above I changed number of items to 1000 and run test few times (runTest/endTest) and memory wastes heavily.
I understand that this is pretty large quantity and real apps uses pool, but early I don't encounter problems with memory in nape, and was surprised and decide to check.
I try clearObjectPools, and it did a trick, thanks a lot!
Is it available only for debug version and it's recommended to use in releas versions?
And what about recommendation to remove bodie shapes before destroy body?
Images without and with clearObjectPools:

clearObjectPools is available in release too.
One way that the app-code can introduce a memory leak when working with nape is however, to do things like 'new Vec2' all the time, since this will 'always' allocate a new Vec2 that nape may release depending on how you use it into the object pool.. but since you keep allocating things with 'new' the objects don't get used and the pool just gets bigger and bigger instead of being recycled. That didn't seem to be the case from the code you showed though.
Regarding removing shapes from body first, you could argue that for the GC to work efficiently it's always a good idea to smash objects apart (null'ing references etc) so that it's easier for the GC to know an object is not being used anymore (and in some cases with Flash GC alteast in the past, it would simply give up if an object graph was far too big, though a Body + Shapes is not near big enough for that).

Thanks!
So best way is to use Vec2.get, null objects references and if needed to clean all memory call clearObjectPools?

In some cases you can be even more careful and use Vec2.weak.
The difference between Vec2.weak and Vec2.get is that Nape automatically calls vec2.dispose() when you pass a 'weak' Vec2 to a nape function. eg:
// with get
var v = Vec2.get(x, y);
someNapeFunction(v);
v.dispose();

// with weak
someNapeFunction(Vec2.weak(x, y));

Of course, it's easier to introduce bugs with Vec2.weak if it get's released to object pool earlier than you realised. But yes, if there is a 'get' function you should always use it instead of 'new' (since get will in the worst-case fall back onto new itself if the object pool is empty).

If possible I usually use Vec2.get(x, y, true).
Thanks a lot for explanations, Luca!


Lazy creation of Neutron networks

When a slice is created, the Observer currently creates a Neutron network on every cluster.  Each network imposes some overhead on the Neutron gateway VM, as a number of processes need to be run for each network instance.  We've experienced the Neutron VM running out of memory as a result of too many networks.
We should consider lazy creation of networks on VM instantiation.

This one is not going into Axtell, to use it, please pull from master.


Thanks!

Thanks for building this plugin - exactly what I was looking for (I found others, but yours is the only one I found that also support Live Edit). Would be great to also have this as a community plugin!

Thanks for the feedback! I've opened a pull request to add this plugin as a community plugin.

This plugin has been submitted as a community plugin list.


master broken: unable to validate: open /config-schema.json: no such file or directory

This is probably due to the floating dependencies without #148, but recent builds in a clean Go directory break with:
$ make test
go test -race ./...
?       github.com/opencontainers/image-spec/cmd/oci-image-tool [no test files]
?       github.com/opencontainers/image-spec/image      [no test files]
--- FAIL: TestConfig (0.00s)
        config_test.go:162: test 6: expected validation failure false but got true, err schema application/vnd.oci.image.serialization.config.v1+json: unable to validate: open /config-schema.json: no such file or directory
…

Spun off from earlier discussion in #161.

And the problem is (not surprisingly, given the error message) with http/FileSystem (see discussion in #150).  The tests pass with the following patch:
diff --git a/schema/validator.go b/schema/validator.go
index abc8d6d..537314a 100644
--- a/schema/validator.go
+++ b/schema/validator.go
@@ -19,6 +19,7 @@ import (
 	"fmt"
 	"io"
 	"io/ioutil"
+	"path/filepath"

 	"github.com/pkg/errors"
 	"github.com/xeipuuv/gojsonschema"
@@ -44,7 +45,12 @@ func (v Validator) Validate(src io.Reader) error {
 		return errors.Wrap(err, "unable to read manifest")
 	}

-	sl := gojsonschema.NewReferenceLoaderFileSystem("file:///"+specs[v], fs)
+	//sl := gojsonschema.NewReferenceLoaderFileSystem("file:///"+specs[v], fs)
+	refPath, err := filepath.Abs(specs[v])
+	if err != nil {
+		return err
+	}
+	sl := gojsonschema.NewReferenceLoader("file://" + refPath)
 	ml := gojsonschema.NewStringLoader(string(buf))

 	result, err := gojsonschema.Validate(sl, ml)

Rolling back assorted dependencies, I the issue was introduced by xeipuuv/gojsonschema#103, and there are already comments there about broken loading as a result.  So folks who want a working test suite should pin thier gojsonschema before that PR landed or wait until they roll out a fix.

correct. rolling back https://github.com/xeipuuv/gojsonschema/commit/ac948b64b57aa1beeb91c87069730e6f24bc91d9 lets the tests pass again. :-\

I think https://github.com/xeipuuv/gojsonschema/pull/105 fixed a bug introduced in https://github.com/xeipuuv/gojsonschema/pull/103 - so you might not need to pin anymore...

nope. 105 did not fix the issue.
On Tue, Jun 21, 2016 at 1:43 PM, Pete Moore<EMAIL_ADDRESS>wrote:

I think xeipuuv/gojsonschema#105
https://github.com/xeipuuv/gojsonschema/pull/105 fixed a bug introduced
in xeipuuv/gojsonschema#103
https://github.com/xeipuuv/gojsonschema/pull/103 - so you might not
need to pin anymore...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/opencontainers/image-spec/issues/162#issuecomment-227516349,
or mute the thread
https://github.com/notifications/unsubscribe/AAEF6eMLXQhr52m7X1lzPZ_pGd7uyoU9ks5qOCLFgaJpZM4I6Uyb
.

This issue was moved to opencontainers/image-tools#23


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-27-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4164

Questions ?
Please refer to the Backport tool documentation


feat(build): init docker file to build and publish SDK to AWS S3, and…

… build and create Nginx standalone SDK server

I also added docker file to build + create bucket + publish sdk "more" fluent  (everything inside docker)


can't load SourceHanSansSC

Always loading.
font file:
SourceHanSansSC-Regular.otf.zip

The extension isn't the best at loading font files that large unfortunately but I've been working on trying to find a way around this though so stay tuned!

Closing this since it's finally fixed in v2.0.0 (49705d7080f0325dc3ae2666554e7bf8337b5125). You should be able to load the font in your editor now 👍


nixos/init-script: fix missing nixos config

The d3528cdc3dbe82a0551707fe869aec02bba72956 introduced error:
error: attribute 'nixos' missing
The correct config is clearly system.nixos thus just a simple swap and just missed in testing.
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1740


CAP_PERFMON works in runc but not docker

I've been trying to get CAP_PERFMON to work without success.
I've tried all the channel settings ("stable", "nightly", etc.) for a clean install from the latest get-docker.com script and consistently get:
# docker run --cap-add CAP_PERFMON hello-world
docker: Error response from daemon: invalid CapAdd: unknown capability: "CAP_PERFMON".

Poking around at runc suggests that my runc support CAP_PERFMON.  Here's the relevant section of my config.json,
                        "bounding": [
                                "CAP_AUDIT_WRITE",
                                "CAP_KILL",
                                "CAP_NET_BIND_SERVICE",
"CAP_PERFMON"
                        ],
                        "effective": [
                                "CAP_AUDIT_WRITE",
                                "CAP_KILL",
                                "CAP_NET_BIND_SERVICE",
"CAP_PERFMON"
                        ],
                        "inheritable": [
                                "CAP_AUDIT_WRITE",
                                "CAP_KILL",
                                "CAP_NET_BIND_SERVICE",
"CAP_PERFMON"
                        ],
                        "permitted": [
                                "CAP_AUDIT_WRITE",
                                "CAP_KILL",
                                "CAP_NET_BIND_SERVICE",
"CAP_PERFMON"

                        ],
                        "ambient": [
                                "CAP_AUDIT_WRITE",
                                "CAP_KILL",
                                "CAP_NET_BIND_SERVICE",
"CAP_PERFMON"
                        ]
                },

and runc run container1 completes successfully.
Here's what seems like the relevant version strings (I'm in Ubuntu 20.04 HWE):
# uname -a
Linux da-c3-small-x86-01 5.11.0-44-generic #48~20.04.2-Ubuntu SMP Tue Dec 14 15:36:44 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

# docker --version
Docker version 20.10.12, build e91ed57

# runc --version
runc version 1.0.2
commit: v1.0.2-0-g52b36a2
spec: 1.0.2-dev
go: go1.16.10
libseccomp: 2.5.1

I have the same on Debian stable, running the Debian docker package (20.10.5+dfsg1).
$ docker run -ti --cap-add=CAP_PERFMON test
docker: Error response from daemon: invalid CapAdd: unknown capability: "CAP_PERFMON".

FWIW, Intel PT does work with CAP_SYS_ADMIN, but that's a lot of permissions to allow a container!

This issue seems to suggest that CAP_PERFMON is currently broken:
https://github.com/docker/docker.github.io/issues/13731
Maybe this issue should be a duplicate?

The wording of #13731 is very confusing.  At the end, it says "Please mention that using these capabilities should work in versions after this merge.", which seems to suggest it's not broken.
As for whether this issue is a duplicate of #13731, #13731 is about a documentation issue rather than the fact that CAP_PERFMON works in runc but not docker.

Funnily enough, docker/cli has bash completions that specifically include the capability CAP_PERFMON.
https://github.com/docker/cli/blob/8d91c1442c8bef621ad4e7c2ccf129b9a8a11b1d/contrib/completion/bash/docker#L843
Really hoping this is fixed eventually :)

@thaJeztah Sorry to ping, but wanted to see if you knew about this. As far as we can tell, the only issue is docker does not allow CAP_PERFMON even when runc supports it.

https://github.com/docker/docker.github.io/issues/13731#issuecomment-1173776968

moby/moby#42011 has been reverted in the master branch, but is not included in the 20.10 release branch (yet), as 20.10 releases originally shipped with a version of containerd and runc that didn't support it (moby/moby#42011)).
Let me open a PR to amend the release notes for 20.10.0


Tuya PIR Motion Sensors

These motion sensors dont work with home assistant. I can't get them to run automations.

https://ae01.alicdn.com/kf/Hdfdca73288a444ae851047d999fc75ebI/Smart-WiFi-PIR-Motion-Sensor-Human-Body-Sensor-Detector-Home-Alarm-System-Smart-PIR-Motion-Sensor.jpg_Q90.jpg_.webp
A search on AliExpress will show you them
Smart WiFi PIR Motion Sensor Human Body Sensor Detector Home Alarm System Smart PIR Motion Sensor Tuya Smart Life with IFTTT

MS17110722

That one works for me with v1.5.0. Make sure you have Device Notification Service subscribed and authorised in your Tuya Cloud Service.

That one works for me with v1.5.0. Make sure you have Device Notification Service subscribed and authorised in your Tuya Cloud Service. You also need to restart HA Core after enabling Device Notification Service. The delay is quite bad for me though.

What is that under? I cant find that anywhere

What is that under? I cant find that anywhere

On the Tuya IOT page, go to Cloud, then My Services, and you should find Device Status Notification. Mine was already activated, so this did not help me with my Deltaco PIR sensor, which I reported about in Issue #465. Not sure what to do now, how to get HA to find my device.

No PIR functions correctly. Most do not display correctly. I've test 3 from different retailer that are tuya based. Work flawlessly within tuya smart app. Work correctly on initial import within HA. After reboot of HA will not update sensor period only has detection on startup. 12 hours later still not displaying motion detected. Same occurs for temperature/humidity with the added inconvenience the temperature display reading is incorrectly. Seems to convert Celsius to Fahrenheit and display this reading as Celsius. Strange that PIR and temperature/humidity work correctly initially but fault after reboot. Removing and re-installing device from Tuya smart app does not correct issue, will not function correctly right from the start within HA

I have the same problem, the only Zigbee PIR I still have in SmartLife app (that I didn't managed to pair with my USB Stick because of the zigbee channel it uses) does in fact appear in HA, but does not update afterwards. In SmartLife app and in Tuya Cloud testing, everything works fine, so I think this is a problem of communication/sync from the integration and/or Tuya IoT Cloud Platform.

That one works for me with v1.5.0. Make sure you have Device Notification Service subscribed and authorised in your Tuya Cloud Service. You also need to restart HA Core after enabling Device Notification Service. The delay is quite bad for me though.

Works for me as well, thanks! But yes, a lot of delay. I now use a binary sensor entity to ping the motion sensor every second. If it connects to the router, that means it registered movement. This delay is only a few seconds.
For this you have to add a binary sensor entity in your configuration.yaml:
binary_sensor:

platform: ping
host: ip address of your sensor on your router (this has to be the same address always, so create a reservation)
count: number of ping packets before the sensor is activated in HA
scan_interval: number of seconds between pings
name: sensor name in HA


MaximumNumberOfActiveVersions debug feedback

Goals

I want to keep the realm file on disk small, and to stop all bug related to it (multiple versions, unable to compress on launch)
Expected Results

I would like to query the Realm instance for number of active versions, to keep track of possible problem area's
Actual Results

You can set maximumNumberOfActiveVersions to a number, run your app and see if it triggers or not
I would like to request an option for me to live debug (or observe) how many active versions I have at a current point in time, making it easier to locate and repair relevant problems.
Currently I have resorted to checking the realm size on disk after every write, resulting in random increases in size that are nerver reclaimed until the app compresses on launch. So, if the app runs for too long, the size becomes too big, cant load anymore and bricks the app.
There have been multiple issues that I have been tracking about these problems, and while offering Ideas on what could cause the problem, they dont provide an active means of problem tracking, other then debugging by triggering MaximumNumberOfActiveVersions.
The cost of the call would not be relevant, as this would be a debug only check, letting you track the points where you are "version leaking"

We've considered this request and added it to our backlog to be implemented.


Resolve Logout Bug

Branch : Development1.0
ReGenerate Bug : Login/Register once and click on logout. It logs you out first and clears the token from localstorage but again calls the loaduser action.
Reason: loadUser action is being called in useEffect hook so that if user reloads the page or comes back again to the website and remain login.
But due to same reason on clicking logout its again calling the loadUser.
Solution Desired: on click of logout it should call the logout action and the token and user state should be set to null.

Can I try this?

sure

Thanks! I will attempt it on weekend.

This issue has been resolved with https://github.com/technojam/technojam-frontend/commit/ad2b9ed28a5ce8ce3283cc5bfcb9e20d0419d96c , just pull the latest code from development branch.


Update FDKConeBeamReconstructionFilter.json

Added Ram lak, Cosine, Hamming, and Shepp logan filters.

Thanks!


[Fixed in GH-6052] [after #5148] Check focus state in Safari

Tested indirectly though other tickets.


precompile codehash could be unsound

In new test cases for begin-tx with precompile #306, we found that call to precompile with value will cause CodeDB initialise the code hash of precompile to EMPTY_HASH (prev value: Word::zero() since non-exist).
This could be unsound since the code hash of precompile address would be inconsistent (Word::zero() for non-exist account, EMPTY_HASH for empty account).

fixed


Added dict to maintain duplicate Handle duplicate #defines during extraction #28

#28

@palanik1 does the duplicate defines detection change anything for the user ?

@sdsen No changes to the user.

lgtm.


Fix indentation of sparkExtraConfigs property for Spark HS and Spark TS

@idzikovskyi have we propagate same changes to ezaf repo as well?

@mayankgarg-23 I've found this issue while I was testing this charts for ERE. Spark TS is not available on EZUA, and as for Spark HS I'm going to backport these changes later.
Furthermore, that problem with indentation affects mostly Spark TS configuration, so I've updated Spark HS charts only to keep this section consistent across our charts.


how to determine pwm out pin for mks boards?

Hello, thanks for useful project. I wonder which pin to use pwm out for MKS-Monster8. Thanks..

Not familiar with a board. Usually the neopixel/led pin works best because of the 5v pullup. If hw pwm doesn't work, try software pwm, this is a known problem for many boards.

Not familiar with a board. Usually the neopixel/led pin works best because of the 5v pullup. If hw pwm doesn't work, try software pwm, this is a known problem for many boards.


List of stop values

Create a scraper for generating a list of stop values, this is the only source I can find (under "Find by route").
Endpoint is partially implemented, just needs the scraper to complete.

[x] Scraper
[x] Endpoint

In progress, https://github.com/kshvmdn/next-ride-api/commit/9a8afac5361b25b6e242cd2e9c9b5b8aa0b1f0f2.

https://github.com/kshvmdn/next-ride-api/commit/6884970ef1ecce0ed5105204c93a478ca33112f2 😄

Endpoint ✔
https://github.com/kshvmdn/next-ride-api/commit/ff06248e1f1c3634b9698416f1e208f464cbfa72


Fix consume() truncating incomplete lines from buffer

Prevent consume() from attempting to overwrite the string buffer with the empty string when parse() returns NULL.

This change makes sense and the test suite confirms no regressions, though is it feasible to have a test case that would fail with the old behavior and succeed with the new?

Thanks! :)


Add support for args/params for extraCredentials to be passed via headers to trino

Add support for args/params for extraCredentials to be passed via headers to trino
Thinking this link may be of use:
https://github.com/dropbox/PyHive/issues/311#issuecomment-586828649

+1 on this. It would be even better if we can pass in arbitrary headers. For our use case, the Host header is needed.

Hello @matt12eagles, @mapshen
Did you try:
engine = create_engine(...,
   connect_args=dict(
    http_headers={
      'X-Presto-Extra-Credential': f"access-token={ACCESS_TOKEN}"
    },
  )
)
``

@dungdm93 I can confirm this setting does take effect - is there a doc on the args users can pass into connect_args?


Add Style/OpenStructUse Cop

This flags any use of the OpenStruct type, whose use is now officially discouraged.
Originally written as part of https://gitlab.com/gitlab-org/gitlab-styles/-/merge_requests/92.
This PR addresses existing OpenStruct offenses in RuboCop itself as well.
Fixes https://github.com/rubocop/rubocop/issues/10206

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks for all the help @dvandersluis -- I believe this is ready for a final review. I added a changelog entry and squashed commits.
I noticed that the spellcheck jobs require additional approval?

This workflow is awaiting approval from a maintainer in #10217

Thank you again for your work on this! It's looking ready to go for me now.

I noticed that the spellcheck jobs require additional approval?

Github actions runs have to be approved for first time committers, it's running now.


Reflection-only context can execute code

This doc I believe incorrectly states that the reflection-only context cannot execute code. In fact, reflection itself can execute code, including when you query a member's custom attributes. Instances of attributes are returned. When those attributes are defined within that assembly, that requires that that attribute's constructor is executed, so in fact the reflection-only context is not a guarantee that code in the assembly will not execute. This is an important point so folks do not rely on it as a security boundary.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c8732511-e24f-2c9c-57c8-ce469c351e94
Version Independent ID: 8806bdc9-6cf1-cbcb-0e47-f6edbdcfd59c
Content: Best Practices for Assembly Loading
Content Source: docs/framework/deployment/best-practices-for-assembly-loading.md
Service: unspecified
Product: dotnet-framework
GitHub Login: @mairaw
Microsoft Alias: mairaw

I can't reproduce what you're describing.
Given this code for a library:
using System;

[A]
class A : Attribute
{
    public A() => Console.WriteLine("Boom");
}

And this code for an application:
using System.Linq;
using System.Reflection;

class Program
{
    static void Main()
    {
        string path = …;
        var a = Assembly.ReflectionOnlyLoadFrom(path);
        var t = a.GetTypes().Single();
        t.GetCustomAttributes();
    }
}

I get:
Unhandled Exception: System.InvalidOperationException: It is illegal to reflect on the custom
attributes of a Type loaded via ReflectionOnlyGetType (see Assembly.ReflectionOnly) -- use
CustomAttributeData instead.
   at System.Reflection.CustomAttribute.GetCustomAttributes(RuntimeModule decoratedModule, Int32 decoratedMetadataToken, Int32 pcaCount, RuntimeType attributeFilterType, Boolean mustBeInheritable, IList derivedAttributes, Boolean isDecoratedTargetSecurityTransparent)
   at System.Reflection.CustomAttribute.GetCustomAttributes(RuntimeType type, RuntimeType caType, Boolean inherit)
   at System.Attribute.GetCustomAttributes(MemberInfo element, Boolean inherit)
   at System.Reflection.CustomAttributeExtensions.GetCustomAttributes(MemberInfo element)
   at Program.Main() in C:\code\tmp\coreapp\Program.cs:line 11

Hmmm... it's been a very long time since I actually interacted with this API. Maybe I remember wrong or maybe that was a behavior in .NET 2.0.

Sorry for the trouble.

Should we close this then @AArnott?

Yes.


frankenphp_worker_php_ready_worker is skewing to negative values with worker restarts

What happened?
Hi.
We have excitingly been trying out the new 1.3.0 release. We immediately setup an alert on the frankenphp_worker_php_ready_worker metric on our environments.
We however observed that this metric has a decreasing pattern to it, that is not equal to the relationship of total_workers - busy_workers.
Look at this plot of the metric of a somewhat busy container (20RPS ish) (each line is a seperate container)

I was very worried of this metric, however it has now become a negative value, indicating that is keep decreasing even below 0. In our case we may see the ready_worker metric report -2, while the busy worker metric is 4 and total workers is 60. So in this case I assume the correct value is 56? Or close to 56.
It looks to be somewhat correlated to worker restarts, looking at this graph:

The service is responding as normal even with negative ready workers
Build Type
Docker (Debian Bookworm)
Worker Mode
Yes
Operating System
GNU/Linux
CPU Architecture
x86_64
PHP configuration
Not relevant

Relevant log output
No response

Yeah it looks like ReadyWorker is only called once, while StopWorker is called every time a worker restarts, skewing the metric.
@withinboredom after #1137 this could maybe also be refactored to pull metrics instead of pushing them, since we can just loop over all workers and the threads assigned to them.

This is possibly a merge-conflict resolution error on my part, but at a glance, it appears to be working as designed.
It is stopping workers, then it calls initWorkers, which should eventually call startNewWorkerThread, which should call metrics.StartWorker(worker.fileName), which should increment the worker count back up.
I think something is missing here.

this could maybe also be refactored to pull metrics instead of pushing them

I don't know how I would feel about a God System reaching into things just to extract metrics.

I think something is missing here.

Aha!
frankenphp_worker_php_ready_worker, per the documentation:

The number of workers that have called frankenphp_handle_request at least once.

This should be a static number once started. It doesn't (usually) increase. It is indicative of a successful startup, not part of request handling itself.

ah, yeah. The metric changed meaning during the merge conflict. I'll fix it.


[#4728] version message for pending publications

Resolves #4728

Pull Request Checklist:

[x] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

Create a new version of an existing resource
Open the new version, add required metadata, and publish the new version
Return to the old version of the resource.
You will no longer see a message about "DOI pending activation" like in the screenshot below

METRIC
VALUE

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-4730

@pkdash I suppose that is a good point. I think we want the pending DOI linked in the XML (instead of the HS URI)


Link to Micro-net pretrained weights

The Next Cloud link to the pre-trained micro-net weights does not work. It says it cannot find this page.

New link: https://drive.google.com/file/d/1Xg6j7aFkCoojMTX9KCFD75kMtDyYIV_U/view?usp=sharing


ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused --->not a single .py file is operating

HELLO , I am trying to find answer but i am not getting it ,is there any solution ??
please help!
shujat@shujat-Inspiron-7570:~/carla/CarlaSimulator/PythonClient$ python client_example.py
INFO: listening to server localhost:2000
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused
ERROR: (localhost:2000) failed to connect: [Errno 111] Connection refused

can you please tell me how you managed to solve this problem?

the same!
why close???

Having the same problem here


Switch JSON schema library

jsonschema works well, but has very bad performance, which is noticeable when working on this (slow tests).

I have found that ajv is no faster than jsonschema within tests; wherever the bottleneck is, it is not in the comprehension of the schema itself as previously thought (17s vs. 13s).
Additionally, ajv's handling of oneOf appears to be quite poor; it seems to zero-in on a particular option, then report why that option isn't compatible, when it might be that no option really fits what was given.


add nrexcl to mapping file.

Adding an nrexcl like option to the mapping file would be nice to avoid the
need to create phantom bonds to model the same exclusions as gromacs.

Also see issue #161 for details.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 11 Oct 2014 at 10:09

Related to #344


Markdown live preview doesn't update with missing file extension

VSCode Version: 1.9.1 Win
OS Version: Windows 10 & OSX

Steps to Reproduce:

Create and open markdown file without .md extension
Open preview editor to the side
Make changes to the file
Live update doesn't work, neither does reopening the preview
Rename the file to have the .md extension
Live preview still doesn't work
Close and reopen file and preview
Live preview now works

Is the language mode for the file set to markdown? Once you remove the extension, the file defaults to plain text mode which does not update the preview.
We could change the markdown preview to update for all files types. I believe this is being tracked by #17592

No after renaming any file which is recognized as having "Plain Text" content to have a .md extension, the content is recognized as "Markdown" but the preview does not update. You need to close the existing preview window and re-preview to get live Markdown updating.

Renames currently don't work after a preview has been opened. This is tracked by #12033
My question was: what happens if you switch the language mode of the plaint text document to markdown without renaming it? In my testing, this makes the preview start updating again. #17592 tracks supporting live preview update for other language modes besides markdown

Closing as designed. #17592 Tracks supporting previews for other content types besides markdown


Introduce global suggestions for supporting {% # theme-check-disable <type> %}

Is your feature request related to a problem? Please describe.
Currently, our language server doesn't offer suggestions to ignore theme-check offenses. I believe it would be a great to have a single implementation to ignore across all checks (something like: {% # theme-check-disable <check_name> %}).
I don't foresee this feature promoting excessive use of the ignore function. Given that comments in the code already create a certain level of noise, so this feature seems to be more beneficial than not providing those suggestions.

https://github.com/Shopify/theme-tools/blob/main/packages/theme-check-common/src/disabled-checks/index.ts#L15

Ahhhhh my bad I totally missed that. à la ESLint, I love this. Great suggestion! :D


Right-Margin Class for grid columns

Wouldn't it be helpful if we get some classes in the grids that could allow us to give right-margins of varying grid length preserving the responsiveness.
Check the following pen for the expected output with the current work-around :- https://codepen.io/rohitrajwani/pen/owKbPp

If you want it to fill the whole row, why don't you just use a new row container?

Creating a row for every element (which requires a right margin), won't it make the code a bit cluttered. There are work-arounds though, but having predefined helper classes would ease the task.

@acburst I've had times where this would be really useful. Depending on your screen resolution, you may want the following content on its own row or you may want it inline (if the screen is big enough).
Considering Materialize already makes a rule of "offset-s4":
.row .col.offset-s4 {
  margin-left: 33.3333333333%;
}

Seems like Materialize could also make a rule of "pad-s4" (I don't love the class name, but you get the idea):
.row .col.pad-s4 {
  margin-right: 33.3333333333%;
}

Adding a set of breakpointed classes like that could give a lot more flexibility to your designs, as opposed to having to force things into predefined rows, which may or may not be applicable at all breakpoints.

I currently have a customised version of _grid.scss to add padding and margin in columns.
The change is in the grid-classes mixin, and will therefor work on all breakpoints and column sizes.
@mixin grid-classes($size, $i, $perc) {
  &.offset-#{$size}#{$i} {
    margin-left: $perc;
  }
  &.pull-#{$size}#{$i} {
    right: $perc;
  }
  &.push-#{$size}#{$i} {
    left: $perc;
  }
  // add padding to columns
  &.padding-right-#{$size}#{$i} {
    padding-right: $perc;
  }
  &.padding-left-#{$size}#{$i} {
    padding-left: $perc;
  }
  // add margin to columns
  &.margin-right-#{$size}#{$i} {
    margin-right: $perc;
  }
  &.margin-left-#{$size}#{$i} {  // technically the same as &.offset-#{$size}#{$i}
    margin-left: $perc;
  }
}

Example:

         This column will be 12 cols wide.

         But the last 3 cols will be filled with padding and therefor empty.

         This column will be 9 cols wide.

         On the right side it will have 3 cols of margin and therefor empty.

Thanks.... It would the serve the purpose!! :+1:


Supporting chaining animations

Supporting chaining animations by switching the execution order of the completion block

@dataxpress rebase done

TY!


[object Object] being generated when trying to use @defer on upcoming RC #9355 which merges #9196

Which packages are impacted by your issue?
@graphql-codegen/visitor-plugin-common
Describe the bug
I am testing the upcoming release candidate (#9355) which merges #9196 as part of its new features.
When using the @defer directive, codegen generates a [object Object] instead of correct types.
Your Example Website or App
I can't disclose it
Steps to Reproduce the Bug or Issue

Given the following fragment

fragment TestFragment on InterviewType {
    id
    ... on InterviewType @defer {
      thumbnail
    }
  }

Generate types using the cli and the client-preset with the following settings:

const config: CodegenConfig = {
    ignoreNoDocuments: true, // for better experience with the watcher
    generates: {
        './src/generated/graphql/': {
            documents: ['src/shared/**/*.graphql', 'src/shared/**/*.ts*'],
            preset: 'client',
            presetConfig: {
                fragmentMasking: { unmaskFunctionName: 'getFragmentData'},
            },
            config: {
                scalars: {
                    JSON: 'Record<string, unknown>',
                    Upload: 'File',
                },
                useTypeImports: true,
                strictScalars: true,
            }
        },
    },
}

Check the generated type:

export type TestFragmentFragment = { __typename?: 'InterviewType', id?: string | null } & [object Object] & { ' $fragmentName'?: 'TestFragmentFragment' };

Expected behavior
As a user I expected correct generated types when using the @defer directive, but instead the following is being generated:
export type TestFragmentFragment = { __typename?: 'InterviewType', id?: string | null } & [object Object] & { ' $fragmentName'?: 'TestFragmentFragment' };

Screenshots or Videos
No response
Platform

OS: macOS
NodeJS: 18
graphql version: 16.6.0
@graphql-codegen/* version(s):
<EMAIL_ADDRESS><EMAIL_ADDRESS>
Codegen Config File
import { CodegenConfig } from '@graphql-codegen/cli'

const config: CodegenConfig = {
    schema: "", // whatever
    ignoreNoDocuments: true, // for better experience with the watcher
    generates: {
        './src/shared/generated/graphql/': {
            documents: ['src/shared/**/*.graphql', 'src/shared/**/*.ts*'],
            preset: 'client',
            presetConfig: {
                fragmentMasking: { unmaskFunctionName: 'getFragmentData'},
            },
            config: {
                scalars: {
                    JSON: 'Record<string, unknown>',
                    Upload: 'File',
                },
                useTypeImports: true,
                strictScalars: true,
            }
        },
    },
}

export default config;

Additional context
No response

I've been able to reproduce it.
It is working fine within a query, but as soon as the @defer directive is used within a fragment, it produces [object Object].

Fixed in https://github.com/dotansimha/graphql-code-generator/pull/9384


Boot function loading times

In recent tests boot functions take longer to load then any other type, while having less properties and default amounts in the code.
This needs a review and possible rewrite to make loading bootfunctions faster

Seems to be related to Nodejs's require function, which seems to be the cause of this issue.
Because this will probably be an issue in the future, i will rename this issue for the whole require issue and postpone this until a later release.

 Proposal to speed up nodejs's require methods


[Bug] Remove Extra White Space Between Main Content and Footer

Description
There is an extra white space between the main content and the footer of the webpage, which is visually unappealing and disrupts the overall layout. This space should be removed to improve the page's appearance.
Reproduction

Open the webpage.
Scroll down to the bottom of the page.
Observe the extra white space between the main content and the footer.

Expectation
The main content should flow smoothly into the footer without any unnecessary white space.
Screenshots

Resolution

[x] I would be interested in resolving this bug.

@nandu-99 I think the lighter grey is the background for the thumbnaillist and the main background color for the page is the white color. So the white space signifies the end of the thumbnaillist there. I would love to hear your and everybody else's thoughts as well.


getchar() documentation is incorrect

Acorrding to the documentation, getchar() returns number if input is “8-bit character”, otherwise it returns string. But actually for e.g. ≠ (U+2260, represented as "\xE2\x89\xA0") it returns number as well and U+2260 by no means is a “8-bit character”. It looks like fix would be just “remove ‘8-bit’ word from the documentation”, yielding text “If it is a character, the result is a number.”.

Yes, it is supposed to be "if it is a single character".
I'll change it.


PDF files all downloading blank when following examples in docs

First of all, thanks in advance for help. I am using jsPDF in my Angular app, and the pdf itself is being created, the pages are being added to the document, and it downloads, but the downloaded file is totally blank. I cannot for the life of me figure out why.
I am loading string data in from a JSON response from my server. I am stringifying it beforehand. Here is an example of the image file:
data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAKQAAACkCAYAAAAZtYVBAAAAAklEQVR4AewaftIAAAYcSURBVO3BQW4ERxLAQLKg/3+Z62OeGmjMSC4vMsL+wVqXOKx1kcNaFzmsdZHDWhc5rHWRw1oXOax1kcNaFzmsdZHDWhc5rHWRw1oXOax1kcNaFzmsdZEfPqTylyomlScVk8pU8URlqnii8qTiicpU8URlqphU/lLFJw5rXeSw1kUOa13khy+r+CaVT6g8UfmEyhsqU8VUMalMFZ+o+CaVbzqsdZHDWhc5rHWRH36ZyhsV31QxqUwVk8pU8aTiDZUnKlPFb1J5o+I3Hda6yGGtixzWusgP/3EVk8qk8pdUnlRMKk9UnlT8PzmsdZHDWhc5rHWRH/7jVJ5UPFGZKiaVqeKNikllqphUnlRMKlPFf9lhrYsc1rrIYa2L/PDLKn5TxaTyCZWp4onKGxVvVEwq31Rxk8NaFzmsdZHDWhf54ctU/pLKVDGpTBVPKiaVqeJJxaQyVUwqU8WkMlVMKm+o3Oyw1kUOa13ksNZFfvhQxb+p4ptU3qh4Q+UTKm9U/Jcc1rrIYa2LHNa6yA8fUpkqnqj8poqp4o2KJypTxaTypOKNijdUpoonKlPFpPJGxScOa13ksNZFDmtd5IdfpjJVfJPKE5WpYqr4hMpU8URlqniicrOKbzqsdZHDWhc5rHUR+wcfUHlSMalMFZPKk4onKlPFN6m8UfFE5UnFpPKkYlKZKiaVJxVPVKaKTxzWushhrYsc1rrID19WMalMFW9UvFExqbxR8Zcqnqi8ofJE5UnFpPKk4psOa13ksNZFDmtd5IfLVEwq31TxROUTFU9Upoo3Kp6ovFExqUwVT1Smik8c1rrIYa2LHNa6yA9/TOVJxVTxRGWq+ETFpDJVTCpPVKaKSWWqeKLypOKJypOKJypTxTcd1rrIYa2LHNa6yA9fpvKk4onKk4o3VKaKSWWqmCreUHmiMlW8UfGXVKaK33RY6yKHtS5yWOsiP3xZxROVqWKqeKNiUvmEyhsVb6h8QuVJxZOKSWWqeENlqvjEYa2LHNa6yGGti/zwZSpTxROVT1R8U8WkMlW8oTJVfFPFpPKk4o2Kv3RY6yKHtS5yWOsiP3yo4hMVb6g8qXijYlKZKiaVqWJSmSomlanijYpJZaqYVD6h8pcOa13ksNZFDmtd5IcPqUwVn1B5Q+WNikllqphU3qj4TSpTxaQyVUwqT1T+TYe1LnJY6yKHtS5i/+AXqTyp+CaVJxVvqEwVb6hMFZPKVPFE5UnFpDJVTCpTxaQyVfymw1oXOax1kcNaF7F/8ItUpoo3VJ5UfEJlqphUnlRMKlPFpDJVPFGZKiaVJxVvqEwVk8pU8U2HtS5yWOsih7Uu8sOHVKaKqWJSmSqeVEwqb6g8qXhS8URlqphUPlExqUwVk8obKlPFpPKXDmtd5LDWRQ5rXeSHP1bxhspU8UTlScWk8qRiUpkqJpUnFU9U3lCZKiaVqeITFZPKVPGJw1oXOax1kcNaF/nhl6lMFZPKVDFV/KaKJypTxaTypOKJylQxqTyp+EsqU8U3Hda6yGGtixzWusgPX6YyVUwqU8UTlScVU8Wk8k0qTyomld+k8obKE5WpYlL5TYe1LnJY6yKHtS5i/+A/TOU3VUwqTyo+ofKkYlKZKt5QeaPiNx3WushhrYsc1rrIDx9S+UsVTyqeqDypmFTeUJkqJpVPqLyhMlV8QmWq+KbDWhc5rHWRw1oX+eHLKr5J5Q2VqeINlScVk8pUMalMFZ9QeaPijYpJZaqYVKaKTxzWushhrYsc1rrID79M5Y2Kb1KZKt5QmVTeqHiiMlVMKm+ofFPFXzqsdZHDWhc5rHWRH/7PVXxTxaQyqfymikllqniiMlVMKv+mw1oXOax1kcNaF/nhP65iUnmi8qRiqphUpoo3VJ6oTBWTyjepTBWTyl86rHWRw1oXOax1kR9+WcVNKv5NFZPKVDGpPKl4ojJVTCo3Oax1kcNaFzmsdZEfvkzlL6k8UZkqJpUnFW+o/KaKSeVJxRsqU8VfOqx1kcNaFzmsdRH7B2td4rDWRQ5rXeSw1kUOa13ksNZFDmtd5LDWRQ5rXeSw1kUOa13ksNZFDmtd5LDWRQ5rXeSw1kX+B8uK725GhKqEAAAAAElFTkSuQmCC
Here is my code:
var doc = new jsPDF({
        orientation: 'p',
        unit: 'in',
        format: 'letter'
      })

for (i = 0; i < collectionSize-1; i++) {

        doc.text('This registration code is for access to files on behalf of '  + response[i].vId, 20, 30)
        doc.addImage(response[i].QR.bId , 'PNG', 15, 40, 180, 160)
        doc.addPage()

      }
      setTimeout(function(){
        const fileName = 'QR_Code_Export_'+ eId;
        doc.save(fileName)
      },5000);

    },
      err => console.error(err)
    );

Issue: PDF is downloaded with the appropriate number of pages, but each page is blank
Expected behavior: PDF generated will add the one line of text and one image per page

Can you provide the generated pdf?

Yes as soon as I can get back to my computer after travel I will supply the
generated document.
On Mon, Mar 18, 2019 at 12:23 PM Aras Abbasi<EMAIL_ADDRESS>wrote:

Can you provide the generated pdf?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MrRio/jsPDF/issues/2317#issuecomment-473988071, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AgJQrL8EIVXIK938aqIkAe79Dmm5pshnks5vX72XgaJpZM4buiPH
.

QR_Code_Export_test.pdf

@arasabbasi here is the pdf attached

I will analyze them this weekend

Here is my package.json just in case it helps identify the issue:
"dependencies": {     "@angular/animations": "6.1.9",     "@angular/common": "6.1.9",     "@angular/compiler": "6.1.9",     "@angular/core": "6.1.9",     "@angular/forms": "6.1.9",     "@angular/http": "6.1.9",     "@angular/platform-browser": "6.1.9",     "@angular/platform-browser-dynamic": "6.1.9",     "@angular/platform-server": "6.1.9",     "@angular/router": "6.1.9",     "@nativescript/schematics": "^0.4.0",     "@ng-bootstrap/ng-bootstrap": "^3.2.2",     "@ng-idle/core": "6.0.0-beta.3",     "@types/jspdf": "1.5.3",     "angular2-qrcode": "^2.0.1",     "bootstrap": "^4.0.0",     "canvas": "^2.3.1",     "classlist.js": "^1.1.20150312",     "core-js": "^2.5.3",     "crypto-js": "^3.1.9-1",     "jquery": "^3.3.1",     "jspdf": "^1.5.3",     "lodash": "^4.17.5",     "moment": "^2.20.1",     "nativescript-angular": "^6.0.0",     "nativescript-checkbox": "^3.0.3",     "nativescript-drawingpad": "^3.0.4",     "nativescript-orientation": "^2.2.1",     "nativescript-screen-orientation": "^2.0.0",     "nativescript-theme-core": "~1.0.4",     "nativescript-ui-sidedrawer": "^4.3.0",     "ngx-toastr": "^9.1.0",     "open-iconic": "^1.1.1",     "pkg-config": "^1.1.1",     "popper.js": "^1.13.0",     "reflect-metadata": "~0.1.8",     "rxjs": "^6.3.3",     "rxjs-compat": "^6.3.3",     "rxjs-tslint": "^0.1.5",     "tns-core-modules": "^4.0.0",     "zone.js": "~0.8.26"   },   "devDependencies": {     "@angular-devkit/build-angular": "~0.8.0",     "@angular/cli": "^6.2.2",     "@angular/compiler-cli": "6.1.9",     "@angular/language-service": "6.1.9",     "@nativescript/schematics": "~0.4.0",     "@types/jasmine": "^2.8.6",     "@types/jasminewd2": "^2.0.3",     "@types/lodash": "^4.14.101",     "@types/node": "^6.0.116",     "codelyzer": "^4.1.0",     "jasmine-core": "~2.6.4",     "jasmine-spec-reporter": "~4.1.1",     "karma": "^1.7.1",     "karma-chrome-launcher": "~2.1.1",     "karma-cli": "~1.0.1",     "karma-coverage-istanbul-reporter": "^1.4.1",     "karma-jasmine": "^1.1.1",     "karma-jasmine-html-reporter": "^0.2.2",     "nativescript-dev-webpack": "^0.17.0",     "node-sass": "^4.7.2",     "protractor": "~5.1.2",     "ts-node": "^3.0.6",     "tslint": "^5.3.2",     "typescript": "2.9.2",     "webpack": "^4.20.2"

Ok, I checked the pdf and the offsets of the text and the images are wrong.

Thanks. Any advice on how I can fix them?
On Sat, Mar 23, 2019 at 4:05 AM Aras Abbasi<EMAIL_ADDRESS>wrote:

Ok, I checked the pdf and the offsets of the text and the images are wrong.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MrRio/jsPDF/issues/2317#issuecomment-475849610, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AgJQrFsA3es91xR0DEtCIcxd_LLq9bJcks5vZeA1gaJpZM4buiPH
.

Wait for bugfixes

Is there a timeline for that? Or can I revert to an older version? I have
to finish this up soon.
Messing with the offset numbers in the code won’t help?
On Sat, Mar 23, 2019 at 11:50 AM Aras Abbasi<EMAIL_ADDRESS>wrote:

Wait for bugfixes
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MrRio/jsPDF/issues/2317#issuecomment-475880459, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AgJQrLzypI-vWLnDD92V0d4Ud4Hfcsx9ks5vZk0sgaJpZM4buiPH
.

Wait a second.... I was building a testcase for your problem and realized, that you are setting inch as base unit. You have a letter-format, meaning you have 8,5 inch wide and 11 inch tall. If you do
doc.addImage(response[i].QR.bId , 'PNG', 15, 40, 180, 160)
You are telling the engine to position the image at 15 inch from left, 40 inch from top and make it 180 inch wide and 160 inch tall.
Change unit: 'in', to unit: 'pt' or unit: 'px',

Oh my goodness.  Thanks for catching that. That would
surely explain the issue. Let me give that a try real fast
On Sat, Mar 23, 2019 at 12:13 PM Aras Abbasi<EMAIL_ADDRESS>wrote:

Wait a second.... I was building a testcase for your problem and realized,
that you are setting inch as base unit. You have a letter-format, meaning
you have 8,5 inch wide and 11 inch tall. If you do
doc.addImage(response[i].QR.bId , 'PNG', 15, 40, 180, 160)
You are telling the engine to position the image at 15 inch from left, 40
inch from top and make it 180 inch wide and 160 inch tall.
Change unit: 'in', to unit: 'pt' or unit: 'px',
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MrRio/jsPDF/issues/2317#issuecomment-475882458, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AgJQrIDJj2zf9yNUw_PU2sbqu-EOboMCks5vZlKygaJpZM4buiPH
.

I consider this as solved. ;)

thank you! silly mistake but silver linings, it's working now

Hello, I am having the same issue using jsPDF in my React app, and the pdf itself is being created, and getting rendered but is totally blank.
Here is an example of the image file: data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAMgAAADICAYAAACtWK6eAAAB4ElEQVR4Ae3BQQGAQAACMORlaMPY1CsgDbZd9/t8AX41wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMB9jEA7Mzjk1LAAAAAElFTkSuQmCC
Here is my code-
if (fileExt == "tiff" ){
let urlCreator = window.URL || window.webkitURL;
let imgData = urlCreator.createObjectURL(file);
let base64Data = ''
  Jimp.read(imgData, function (err, image) {
    image.resize(200,200);
    image.getBase64(Jimp.MIME_PNG, function(err, data) {
    //  image.getBuffer(Jimp.MIME_PNG, function(err, data) {
      console.log(data);
      base64Data=data

      // BASE 64
      //callback(data)

    });
  });

  let doc = new jsPDF({
    orientation: 'p',
    unit: 'pt',
    format: 'letter'
  })

 onload = function(){
  doc.addImage(base64Data, 'PNG', 15, 40, 558, 600)
  };

 let pdfURL = doc.output("bloburl");
  console.log("PDFURL:",pdfURL)
  this.setDocumentUrl(pdfURL)

}

Hello, I am having the same issue using jsPDF in my React app, and the pdf itself is being created, and getting rendered but is totally blank.
Here is an example of the image file: data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAMgAAADICAYAAACtWK6eAAAB4ElEQVR4Ae3BQQGAQAACMORlaMPY1CsgDbZd9/t8AX41wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMDTA1wNQAUwNMB9jEA7Mzjk1LAAAAAElFTkSuQmCC
Here is my code-
if (fileExt == "tiff" ){
let urlCreator = window.URL || window.webkitURL;
let imgData = urlCreator.createObjectURL(file);
let base64Data = ''
  Jimp.read(imgData, function (err, image) {
    image.resize(200,200);
    image.getBase64(Jimp.MIME_PNG, function(err, data) {
    //  image.getBuffer(Jimp.MIME_PNG, function(err, data) {
      console.log(data);
      base64Data=data

      // BASE 64
      //callback(data)

    });
  });

  let doc = new jsPDF({
    orientation: 'p',
    unit: 'pt',
    format: 'letter'
  })

 onload = function(){
  doc.addImage(base64Data, 'PNG', 15, 40, 558, 600)
  };

 let pdfURL = doc.output("bloburl");
  console.log("PDFURL:",pdfURL)
  this.setDocumentUrl(pdfURL)

}


Excluded a property from the index

For doc purposes

CLA is not applicable (I'm a Googler).
Thanks,
Walter
On Mon, Jul 11, 2016 at 11:03 AM, googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/java-docs-samples/pull/270#issuecomment-231814994,
or mute the thread
https://github.com/notifications/unsubscribe/ABH-xfOom-zjCwW2RKwLH6lhPqC2VM3hks5qUoVbgaJpZM4JJn7P
.

The line is 106 characters long it must be 100 or less characters - please fix.

Done, made line < 100 chars.

Not sure that passes our style guide, but I'm going to accept it.
LGTM


Add support for CopyBits op

Fixes #61

The test breakage seems unrelated to this set of changes, @Asa-Kosto-QTM is looking into that.

I will wait to merge this until the tests are fixed in another PR.


Make resulting copyright holder more explicit in Manifestation registration

Some small API changes that will also affect github.com/bigchaindb/coalaip-http-api.
Also had a thought that, instead of always registering the Work and Manifestation for the given user, a user could be provided with the plugin to be used when registering these "ownership-less" assets. For example, DACS could provide their own account to hold these assets while only the Rights go to the actual end user.
Thoughts @TimDaub?

Current coverage is 93.18% (diff: 100%)

Merging #27 into master will not change coverage

@@             master        #27   diff @@
==========================================
  Files            11         11
  Lines           367        367
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            342        342
  Misses           25         25
  Partials          0          0

Powered by Codecov. Last update a9369d4...4ee9ecc


Fixing figure caption

Currently the figure captions are of the form "Figure 3.1.:", which seems weird to me: why the point followed by a column? This PR removes that in the sidefigure and figure* environments, it might need adjustments for other kinds of floats as well. Please add a commit to this PR if this is the case!

Thanks! Having a dot after numbers is a German convention which I always disable by passing the numbers=noenddot option to kaobook, and I personally like the semicolon, but on the other hand I also see how it clashes with the dot (at the very least, this shouldn't be the default behaviour... my bad, sorry about that). Maybe a good compromise could be to choose the labelsep with an option, so that you could choose either "enddot + space" or "noenddot + semicolon". What do you think? :)

I agree, I like the semi-colon too! Your solution seems better than mine, I think in this case it might be a lack of documentation about the option. Probably adding a note somewhere on that, and maybe putting that option in the examples document might be enough? Otherwise indeed implementing the option so that people who want to keep the German convention can is also a way to go.

I guess this PR won't be merged. :-)


[ko] Korean Translation for 3-transport lesson 2

Add README.ko.md

@jimbobbennett She finished the translation! Plz merge this!


Drawing/Writing?

Would it be possible to introduce drawing functionality? like that you can draw whatever you'd like on the block. Maybe the option to draw in 32x32 would also be nice to be able to write on blocks.

I have thought about it. While that would be cool, that definitely would not match that vanilla feel I am aiming for.

Well, it could make a good addon mod. If someone wants to contribute: sure.


Want to add a folder for dynamic programming problems in python

I have a folder which has python code samples for solving different dynamic programming problems. Would like to add it in as a separate folder.

Can you assign me this?


Rsbuild support

Are there any plans to add support for Rsbuild? Most projects will probably want to use a framework instead of configuring Rspack themselves(probably why CRA, Vite, vue-cli are well known)

hmm, can investigate it


create org.osbuild.ostree.aleph stage

Similar to the aleph file created for builds of FCOS based on ostree commit inputs, this adds an aleph file that contains information about the initial deployment of data when the disk image was built
A new stage is preferred here as both the org.osbuild.ostree.deploy and org.osbuild.ostree.deploy.container stages need an aleph file and use of the aleph file may depend on the project/product. For example, right now CoreOS is the only project that uses an aleph file, but others may want it in the future.

Creating as draft for now since this won't pass until a new version of rpm-ostree gets released and into Fedora and into the snapshot updates.

I'm OK with this but I think we have a tension here in osbuild around the bigger picture question of whether these stages should contain significant custom logic on their own, or just call out to external binaries.
The logic here is clearly one that probably could go in ostree upstream, for example - for the same reason that bootc has an aleph.

requested snapshot update in https://github.com/osbuild/osbuild/pull/1480#issuecomment-1843607117 to unblock this.

hey @mvo5 Looks like there are some test.run.test_stages test failures now that I updated things to F39. Is there a way to regenerate those diffs? I see you did a lot of them in https://github.com/osbuild/osbuild/pull/1441

========================================================================================================= short test summary info ==========================================================================================================
FAILED test/run/test_stages.py::TestStages::test_copy - AssertionError: before values are different: sha256:d38ce6163200bfdfa564bc5f9d4a59fb82ae83dc235de835fe97eaafb46de7a2, sha256:125ed26603dabd0f76eb0e8bc44e90fd60940640ec5dbf4ea100c443d6eb15cf
FAILED test/run/test_stages.py::TestStages::test_dracut - FileNotFoundError: [Errno 2] No such file or directory: '/var/tmp/tmp9b5nwvnb/tree/boot/initramfs-6.5.6-200.fc38.x86_64.img'
FAILED test/run/test_stages.py::TestStages::test_firewall - AssertionError: Lists differ: ['/et[128 chars]ones/trusted.xml', '/etc/firewalld/zones/trusted.xml.old'] != ['/et[128 chars]ones/public.xml', '/etc/firewalld/zones/truste[42 chars]old']
FAILED test/run/test_stages.py::TestStages::test_hostname - AssertionError: Lists differ: ['/etc/hostname'] != ['/etc/hostname', '/etc/kernel']
FAILED test/run/test_stages.py::TestStages::test_keymap - AssertionError: Lists differ: ['/et[34 chars].conf.d/00-keyboard.conf', '/etc/vconsole.conf'] != ['/et[34 chars].conf.d/00-keyboard.conf', '/etc/kernel', '/etc/vconsole.conf']
FAILED test/run/test_stages.py::TestStages::test_locale - AssertionError: Lists differ: ['/etc/locale.conf'] != ['/etc/kernel', '/etc/locale.conf']
FAILED test/run/test_stages.py::TestStages::test_oscap.remediation - AssertionError: Lists differ: ['/boot/grub2/grubenv', '/etc/authselect/cus[1037 chars]ash'] != ['/etc/authselect/custom/hardening', '/etc/a[850 chars]ash']
FAILED test/run/test_stages.py::TestStages::test_oscap.remediation_extra - AssertionError: Lists differ: ['/boot/grub2/grubenv', '/etc/authselect/cus[1353 chars]unt'] != ['/etc/authselect/custom/hardening', '/etc/a[1166 chars]unt']
FAILED test/run/test_stages.py::TestStages::test_rpm - AssertionError: Lists differ: ['/dev', '/etc', '/etc/machine-id', '/etc/pk[13559 chars]wal'] != ['/afs', '/bin', '/boot', '/dev', '/etc', '/[63132 chars]/yp']
FAILED test/run/test_stages.py::TestStages::test_sshd.config - AssertionError: before values are different: sha256:7e333391dcb59343f577eb700ca5b344d8e6da28bc736c305a42c5290e966859, sha256:d72b96c3e68685cdbacc572126204d098ce7feeb6f60cc17059976d55c53d3f2
FAILED test/run/test_stages.py::TestStages::test_timezone - AssertionError: Lists differ: ['/etc/localtime'] != ['/etc/kernel', '/etc/localtime']
FAILED test/run/test_stages.py::TestStages::test_update_crypto_policies - AssertionError: length of differences different
FAILED test/run/test_stages.py::TestStages::test_users - AssertionError: before values are different: sha256:d5c638b9377359ab9b8208688016b3e5f5d03fa6af7fe022edb9f7fc321d448c, sha256:da6b9ba10db7b25c23208d8e15339ef83d7b3e1aed787ac5c85c359d76fdf45d
========================================================================================== 13 failed, 68 passed, 1 skipped in 6739.54s (1:52:19) ===========================================================================================

ok I'm hoping the tests pass now!

Out of curiosity, how and where is this consumed? Is this informational only (if so, where is it typically viewed?) or does it enable some functionality?

Out of curiosity, how and where is this consumed? Is this informational only (if so, where is it typically viewed?) or does it enable some functionality?

There are at least two places where it's used. One is the MCO in OpenShift (see https://github.com/openshift/machine-config-operator/pull/4034) and the other is bootupd (see https://github.com/coreos/bootupd/pull/585). I think the data is mostly informational at this point, but I guess the consumers can use the data for whatever they please. Having the data (which is data about what software this disk image was deployed from) is really useful for debugging problems and trying to determine what in what cases the problem arises.

All checks have passed
71 successful checks

🎉


feat: support multi-exponentiations with varying window-widths (PROOF-893)

Rationale for this change
Remove the hard-coding around the window-widths used in the Pippenger partition version of the multi-exponentiation algorithm.
This will allow us to try different window withs values and potentially improve performance.
What changes are included in this PR?
Removes hard-coding of window with in various locations.
Are these changes tested?
Adds tests for different window widths.

:tada: This PR is included in version 1.72.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


TileSet Atlas

Godot version
3.5 rc1
System information
windows 10, GLS3
Issue description
using a TileSet Atlas, turns the tileSheet into a JigSaw puzzle... making it impossible to use.

#-----------//------------//----------------

Steps to reproduce
using a tileAtlas
Minimal reproduction project
No response

This is a limitation of the old tile editor. It keeps tiles as a list, without any structure.
It's fixed in the new tile editor in 4.0.

This is a limitation of the old tile editor. It keeps tiles as a list, without any structure. It's fixed in the new tile editor in 4.0.

I know, but godot 4 still has some unfinished features, making unusable for me at the moment,
Ive been experimenting with 	"Tiled Map Importer for Godot 3", the problem i think is that it creates the same tile.png image multiple times on each difereent tileMap node

-------------//--------------//-

Ive been experimenting with "Tiled Map Importer for Godot 3", the problem i think is that it creates the same tile.png image multiple times on each difereent tileMap node

Please report issues related to the Tiled importer on its dedicated repository, not here.

@jonSP12
As a simple workaround, I would recommend to prepare the texture how you like to use it. So if you want the atlas tile panel to be 10 tiles wide, make sure to edit your texture in any graphics software to be only 10 tiles wide.

This is difficult to resolve properly without breaking compatibility. It can however be mitigated by using some hacks, like a special property in TileSet that forces constant column count. The issue is unlikely to be resolved though.

This is difficult to resolve properly without breaking compatibility. It can however be mitigated by using some hacks, like a special property in TileSet that forces constant column count. The issue is unlikely to be resolved though.

By compatibility you mean projects using the tileMap node would stop working ?
Couldnt it be done by added a new node tool ? the v4 tileMap, would look on godot 3

By "compatibility" you mean projects using the tileMap node would stop working ?

Yes.

Couldnt it be done by adding a new node tool ? the v4 tileMap, would look on godot 3

The 4.0 tiles can't be backported, because they rely a lot on rendering changes. Also there is no need to make a separate class.
As I said, we could just add a property that forces constant number of columns in the editor. It's a hacky solution, but wouldn't break compatibility.

This is difficult to resolve properly without breaking compatibility. It can however be mitigated by using some hacks, like a special property in TileSet that forces constant column count. The issue is unlikely to be resolved though.

There's no need to break compatibility anyhow, subtile count (to get the proper column count) can be calculated with already available TileSet methods. I think the only problematic part is that ItemList (used to show the subtiles in the editor) doesn't support horizontal scrolling. Meaning it would probably need to be nested into a ScrollContainer and that ItemList's minimum size would need to be accordingly updated manually so it would have proper number of columns and so it won't show its own vertical scroll. I'll see if it's easily doable.

For the record, I closed this on behalf of the Code of Conduct team, as the OP was banned due to harassing contributors. They're restricted from further interacting so this issue cannot progress. If someone confirms that the issue is still reproducible, you can open a new issue.


Usepackages are not folded correctly when lines include comments

Result is this multi fold which hides only package names.

Current workaround: Don't use comments ffs :P


GaussianDerivativeImageSource::SetParameters Miscommented?

This seems to be miscommented.
Instead the parameter vector seems to contain --- order in each dimension, sigma in each dimension, mean. See here. Am i right or am i missing something?
Also, am i right in inferring that the sigmas and mean are specified in physical space. It would be nice to mention this in the doc so the API user isnt confused.

Feel free to fix.
Thanks
S
On Mar 26, 2015 10:42 AM, "Deepak Roy Chittajallu"<EMAIL_ADDRESS>wrote:

This
https://github.com/KitwareMedical/TubeTK/blob/master/Base/Filtering/itktubeGaussianDerivativeImageSource.h#L116
seems to be miscommented.
Instead the parameter vector seems to contain --- order in each dimension,
sigma in each dimension, mean. See here
https://github.com/KitwareMedical/TubeTK/blob/master/Base/Filtering/itktubeGaussianDerivativeImageSource.hxx#L68.
Am i right or am i missing something?
Also, am i right in inferring
https://github.com/KitwareMedical/TubeTK/blob/master/Base/Filtering/itktubeGaussianDerivativeImageSource.hxx#L171
that the sigmas and mean are specified in physical space. It would be nice
to mention this in the doc so the API user isnt confused.
—
Reply to this email directly or view it on GitHub
https://github.com/KitwareMedical/TubeTK/issues/403.


FIX Stop recursion on detection of external URL

If a potential external URL is detected, stop internal recursively checking
matches otherwise infinite loop conditions  may be triggered, leading to a 404
instead of the actual redirect

This is resolved in https://github.com/nglasl/silverstripe-misdirection/commit/ca1fcc15e9ad559a4f5949122acc66e886823ab4. Thanks for jumping onto the issue :)


[Feature]: Add fields content_color and background_color to template chip card

Is your feature request related to a problem?
I tend to use colors according to sensor's state values. For example: in a battery sensor, above 50% is green, 20-50% is orange and below 20% is red. Also, with a custom icon for each range too.
When trying to follow this using the template chip card, I face two problems: no support for content color and no support for background color (chips are small, it feels better to play with its background color too). This is related to #445 .
Describe the solution you'd like
Add two fields to template chip card: content_color and background_color (or just background) that support templates.
Describe alternatives you've considered
I tried using custom styling on the chips card but feels too hack-ish.
Additional context
No response

It is not a priority for now to update the chips design. For design consistency. I don't think it's a good idea to allow text, background and icon color. Only one color must be allowed to simplify the config.

I still think chips are a special case and would be great to be able to achieve something like this (from Chips - Material Design):

Would you consider merging a PR for supporting this or do you prefer to keep this as is? Not sure but maybe I could give a hand here.
Thanks!

Came looking for the same.
My use case if that I want to highlight where temperature or humidity are outside of my desired range.
I have the icons coloured conditionally and it works OK but it would be mush easier to identify issues if I was able to change the background colour to make the chips stand out.


Produce a syncmer if any minimum is at the required position

Because of @drtconway’s questions in #463, I also looked a bit closer at the SyncmerIterator implementation, and my impression is also that the handling of ties is possibly inconsistent. In Edgar’s syncmer paper, it says that "ties may be handled in different ways". Ties occur when there are multiple s-mers in the k-mer that have the same minimum hash value. One of the loops in the current code picks the rightmost minimum, another the leftmost one, so I’m not entirely sure what the intention is. I think the current implementation outputs a syncmer if the hash of the s-mer at position $t$ is the leftmost minimum (or maybe rightmost in some circumstances).
I tried to simply make the three places consistent where the minimum hash value and its position are updated, but this leads to slightly fewer syncmers being generated (I haven’t fully worked out why).
Instead, my suggestion is to change it such that a syncmer is generated if the hash of the s-mer at position $t$ is minimal. The advantage is that we no longer need to keep track of where the minimum occurred and can therefore remove qs_min_pos, slightly simplifying the code.
Slightly more syncmers are generated, but there’s essentially zero effect on accuracy and runtime.

I added a commit with a test that ensures syncmers are canonical. The test actually fails on main (for the case that the syncmers are generated over a long string consisting only of A).

Given discussion in https://github.com/ksahlin/strobealign/issues/463, I now fully accept this commit.

However, there’s a slight inconsistency once again due to the way how ties are handled: Because the syncmer was chosen if the leftmost minimum was at that middle position, this is different in the reverse complemented version of the k-mer because the leftmost minimum would then be the rightmost one. The changes I made in https://github.com/ksahlin/strobealign/pull/464 happen to fix that issue as well.

Great, seemed important to get this handled.

I measured whether avoiding repetitive syncmers would be beneficial, and it turns out it has very little effect in terms of runtime or accuracy. When in doubt, I think keeping it simple is the best option, so I have removed the respective commits from this PR.
The relevant commit in this PR is therefore 92c6105. It leads to the following changes in accuracy.
Comparing accuracy (paf)

5a7de44 Merge pull request #473 from ksahlin/skip-unaligned
92c6105 Produce a syncmer if any minimum is at the required position

library
5a7de44
92c6105
difference

sim5-drosophila-50-se
76.9207
76.9434
+0.0227

sim5-drosophila-75-se
83.4332
83.4574
+0.0242

sim5-drosophila-100-se
84.9742
84.9698
-0.0044

sim5-drosophila-150-se
87.5069
87.5199
+0.0130

sim5-drosophila-200-se
89.2447
89.2588
+0.0141

sim5-drosophila-300-se
91.2737
91.2808
+0.0071

sim5-maize-50-se
39.0947
39.1029
+0.0082

sim5-maize-75-se
51.1997
51.1972
-0.0025

sim5-maize-100-se
58.7611
58.7602
-0.0009

sim5-maize-150-se
70.7927
70.7971
+0.0044

sim5-maize-200-se
77.6704
77.6729
+0.0025

sim5-maize-300-se
84.6004
84.6080
+0.0076

sim5-CHM13-50-se
74.2728
74.2958
+0.0230

sim5-CHM13-75-se
82.7277
82.7271
-0.0006

sim5-CHM13-100-se
85.3533
85.3325
-0.0208

sim5-CHM13-150-se
88.6346
88.6368
+0.0022

sim5-CHM13-200-se
90.4095
90.4160
+0.0065

sim5-CHM13-300-se
92.1489
92.1565
+0.0076

sim5-rye-50-se
36.5498
36.5431
-0.0067

sim5-rye-75-se
49.1617
49.1549
-0.0068

sim5-rye-100-se
57.3582
57.3552
-0.0030

sim5-rye-150-se
70.0147
69.9844
-0.0303

sim5-rye-200-se
77.0137
76.9929
-0.0208

sim5-rye-300-se
83.8602
83.8594
-0.0008

sim5-ecoli50-50-se
10.7429
10.7365
-0.0064

sim5-ecoli50-75-se
13.6476
13.6556
+0.0080

sim5-ecoli50-100-se
15.8012
15.8174
+0.0162

sim5-ecoli50-150-se
20.1191
20.0893
-0.0298

sim5-ecoli50-200-se
23.3852
23.3933
+0.0081

sim5-ecoli50-300-se
28.3011
28.3073
+0.0062

Average difference se: +0.0016

ok, agreed. Approved.


Convert to the bench built-in once it is ready

Tracking issue: https://github.com/rust-lang/rust/issues/29553

Related issue https://github.com/rust-lang/rust/issues/50297

Heads up, the most likely case is that following https://github.com/rust-lang/rust/issues/50297 bench will be made an external crate that consumes those  APIs (shouldn't really change the heart of this issue though)

Hopefully the change would be still minimal :)

I opened https://github.com/xiph/rav1e/pull/507 as an alternative path. Criterion offers quite a number of good features out of box and seems a good fit since we plan to give a try at native SIMD now that at least some of it is available on stable rust.

Closing this issue since the alternative solution was merged.