Fix wrong min-width percentage calculation for child

This PR fixes #872 by using the correct parent values instead of the grandparents.
Unfortunately that fix reveals a bug which causes two existing tests to fail, which are false negatives.

percentage_flex_basis_cross_min_height
percentage_flex_basis_main_min_width

The underlying bug is a misbehavior from the specs (https://www.w3.org/TR/css-flexbox-1/#algo-main-item).

When determining the flex base size, the item’s min and max main sizes are ignored (no clamping occurs).

Sadly the current algorithm takes the clamping into account. As the generated tests also have changed based on using the latest (83) chrome, it also looks that this has been a previous bug in chrome, too.
Despite the fact this PR is still valid, it just reveals another existing bug. But as Yoga does not apply to the web basis specs all together I suggest having this fix in place instead of using the wrong values for calculation. As the underlying bug (including min values for flex basis) is more predictable.
I also moved one of the tests to a different file to not get overridden by the generator.
@SidharthGuglani I highly appreciate your opinion on that one.
@emilsjolander I know it's a long time, maybe you have an idea here, too.  Perhaps I also have an oversight here.

Any chance of this being merged?

Ooh this explains the test failures I was seeing.
I do think we will want to fix this, but behind a flag. For React Native, we are planning on doing a push for layout conformance, where we know there will be enough changes to require to migrate themselves.


Telegram Support

Since it seems no progress is being made by Telegram themselves, how about someone looks into putting a backup option into SMS Backup Plus.?
The biggest problem is that it probably would need to be a small telegram client built into the app for it to access messages and media stored in the cloud.
https://github.com/DrKLO/Telegram/issues/384

sorry i won't add any more features to the app
https://github.com/jberkel/sms-backup-plus#can-you-add-feature-x-


remove unreachable warning in old_build_ext

this warning is impossible because Cython.Distutils imports
Cython.Distutils.build_ext which in turn imports
Cython.Distutils.old_build_ext and the warning does not fire when
Cython.Distutils.build_ext is in the parent stack frames.
I found this particular piece of code to be contributing almost 5%
of the startup cost of an application I contribute to.  The problem is
due to the use of inspect.getouterframes(...).  This eventually calls
inspect.getmodule which does a sys.modules.copy() which is very slow
for lots of modules.
Here's an example piece of code which demonstrates the performance issue:
set -euxo pipefail

rm -rf foo tmpvenv out.svg

virtualenv tempvenv
tempvenv/bin/pip install -e .

mkdir foo
cat > foo/__main__.py <<EOF
import pkgutil
import foo

for _, name, _ in pkgutil.iter_modules(foo.__path__, f'{foo.__name__}.'):
    __import__(name)

import Cython.Distutils
EOF

set +x
touch foo/__init__.py foo/a{0..10000}.py
set -x

time tempvenv/bin/python -m foo

while this seems a bit out there, there's far more than 10k modules in the
code base I was dealing with.
before my change this was taking ~5 seconds.  after my change it takes ~2.

if it was instead wanted to speed up this code, here's an alternative which avoids the expensive inspect calls:
diff --git a/Cython/Distutils/old_build_ext.py b/Cython/Distutils/old_build_ext.py
index 7c2774aa2..8881a0291 100644
--- a/Cython/Distutils/old_build_ext.py
+++ b/Cython/Distutils/old_build_ext.py
@@ -25,19 +25,23 @@ except ImportError:
     basestring = str

-def _check_stack(path):
-    try:
-        for frame in inspect.getouterframes(inspect.currentframe(), 0):
-            if path in frame[1].replace(os.sep, '/'):
-                return True
-    except Exception:
-        pass
+def _in_build():
+    paths = (
+        'setuptools/extensions.py',
+        'pyximport/pyxbuild.py',
+    )
+    frame = inspect.currentframe()
+    while frame is not None:
+        normslashes = frame.f_code.co_filename.replace(os.sep, '/')
+        if any(path in normslashes for path in paths):
+            print(normslashes)
+            return True
+        frame = frame.f_back
+
     return False

-if (not _check_stack('setuptools/extensions.py')
-        and not _check_stack('pyximport/pyxbuild.py')
-        and not _check_stack('Cython/Distutils/build_ext.py')):
+if not _in_build():
     warnings.warn(
         "Cython.Distutils.old_build_ext does not properly handle dependencies "
         "and is deprecated.")

I don't think this is desirable since it will cause a warning any time someone imports Cython.Distutils not from setuptools or pyximport so that's why I opted for the code deletion instead

Thanks for reporting this. I'd like to keep the warning. The check for build_ext.py in the frames was meant to avoid showing it when someone imports "the right thing", so it's unfortunate that it doesn't work as intended.
I've commented out the code for now and added a FIXME. We should eventually repair it as intended, but having it as it is now is useless. And harmful, as you pointed out.

seems wrong to take authorship of the patch, but to each their own

seems wrong to take authorship of the patch, but to each their own

Sorry for that. You're right, I could have mentioned your name in the commit comment.
Since this has a user visible effect, I'll make sure you're mentioned in the changelog (as we always do).

seems wrong to take authorship of the patch, but to each their own

Sorry for that. You're right, I could have mentioned your name in the commit comment.
Since this has a user visible effect, I'll make sure you're mentioned in the changelog (as we always do).

don't worry, I won't be contributing again

Thank you for your contribution.


More = white screen

In the "More..." section, selecting either "About this game" or "High scores" results in a blank white screen. Android Lollipop - CyanogenMod 5.1.1
PS: Great game!

Thanks for the interest ! :)
If I sent you a corrected apk, would you be able to try it and tell me if it's corrected ? I'm still running Kit Kat, and my poor computer has trouble running an Android emulator...
By the way, may I ask how you found that project ? I submitted it on f-droid, but it looks like it hasn't been added yet.

New APK here -> https://github.com/316k/android-tap-the-black-tiles/releases/tag/v1.0.2

Thanks for the quick reply - I'll test it shortly.
I did find it on F-Droid - yesterdays repo scan picked it up in the what's new section.
It hasn't made the fdroig.org homepage yet but it is there: https://f-droid.org/repository/browse/?fdid=org.legtux.m_316k.taptheblacktiles
The in-game scrolling can be a bit jittery, is that because it is using webview?

OK, the new version works - well done! I guess
I had to uninstall the fdroid version first because it was probably compiled with a different signature. That meant that all my high scores were wiped out, which made me find another bug: Before any scores are generated, going to the high scores page displays a blank black screen which looks like the program has stopped. Playing a game then going back shows the high score table correctly. Also, clearing high scores from the paranoid button(s) also leads to an empty page. It really needs some kind of text there.
BTW, idea for a new level: reverse scrolling, ie tiles move away from player instead of towards them.

Thanks for helping out, I'll correct that :)
I have splitted your idea and bug into two new issues.
The game is indeed all rendered in a webview, and playing with the DOM is a bit resource-hungry. The reverse scrolling mode existed in a previous version, but it was really hackish and it gave completely random results depending on the browser... I'll find a better way to do it and I'll re-enable it !


Performance for S3DIS dataset

Hi, thank you for the great work!
I have tried to train the model with S3DIS dataset, but the best mIoU I got is 0.607 at 82 epoch.
I wanna reproduce the model with 62.7 mIoU.
Can you give some more information to get performance in the paper?
It will be very helpful if I can know how you trained the best model.

Hi, how many GPUs and batchsize did you use to train the model?

I trained with 2 RTX Titan GPUs, batchsize of 8 totally, and 100 epochs as you did.

We trained the model for S3DIS with 4 GPUs (Titan Xp) and batchsize of 8 totally. Could you repeat this experiment again, with default settings. If you still cannot reproduce the results, I will check the code later.
Before this, how do you obtain 0.607 at eopch 82, by running ./test_s3dis.sh? or just in training process.
Much thanks.

Sorry for the confusion.
Actually, 0.607 was obtained in training process.
By running test_s3dis.sh, mIoU is 0.616 and mAcc is 0.681.
Can better mIoU be obtained by repeating the experiment again or using bigger max epoch?

I checked the log file, we obtained 0.605 mIoU in training process, and 0.627 in full testing (by running ./test_s3dis.sh) at epoch 60.
It is recommended that you use other checkpoints to evaluate the model (not the best, epoch 60 or 70 is fine). Due to randomness during training, it may work if you could repeat the experiment again. And in our experiments, 100 epochs are enough for S3DIS.

Thanks a lot for your prompt reply!!
Then I'll test with other checkpoints.


Changing the defaults

I've been using logxi for a while.  One of the things that continues to itch is that the defaults are very hard to control.
logxi establishes it's settings by reading environment variables in a package init() function.  All loggers created after that pick up those settings.  It's not possible to reliable modify those environment variables or change logxi's built-in defaults before logxi's init() function is executed.
What we're doing is running logxi's ProcessLogix*Env() functions, then recreating all our loggers to pick up the new settings, but that has a big drawback.  It means there needs to be a single block of code that knows about all the loggers that need to be recreated.  That discourages the use of package-private loggers.
I'm not sure what or if there's a good solution.  I see there's a logger map protected by a mutex with a comment about future support for runtime configuration changes...but since most code has direct references to logger instances...there would need to be an additional, concurrent-safe indirection between logger references and logger configurations that could change.
For now, I'm considering just forking logxi and added the bootstrap code to load environment defaults before logxi's init sequence.

I'm not sure if this helps. The way I use logxi for package-private logging is to create a logger with a namespace.
//// models/init.go
var logger  = log.New("my-models")

//// controllers/init.go
var logger = log.New("my-controllers")

You can then control how each package logs
LOGXI=my-models=DBG,my-controllers=ERR

That's what we do too, but we want different defaults for LOGXI and LOGXI_FORMAT when running tests.  Our makefile can set these, but the makefile is only useful when you want to run all the project's tests.  When we are focussing on individual packages or tests, we're using "go test ..." and its tougher to manage a different set of environment variables just for those test runs.  For our other configuration options (also configured via environment variables), we use a .env loader at runtime, but that happens too late for logxi...by the time we've run that, logxi has already configured itself.
Maybe I can just use the Suppress() function to turn off logs completely during tests.

I need to refactor how logxi reads environment vars. The unit tests are more complicated than they need to be because of it. Do you want to take a stab at refactoring it? I have limited time.

Thoughts on how to refactor?  I'm a little torn on the best approach.  There are good reasons to init logging in init() functions (mainly so logging can happen as early as possible in boot up).  If I add a more robust way to reconfigure existing loggers, it seems I'll have to add at least one extra mutex into the logger calls, which will impact performance.  Or move loggers to channels to avoid the lock, but again, not sure what the performance impact will be...
There's currently a "concurrent writer" which has a mutex...maybe I could move that mutex up the stack, to a "ConcurrentLogger"?  That would avoid adding another mutex.

Another simpler approach would be passing defaults to log.New().


fix contradicting documentation and say a bit about the resident size…

… of a volume directory
fixes #9111

Cherry-picked to v9.2.x


Mouse location is wrong when device descriptors use scaling.

I've just spent the last 2 hours debugging why we were getting some odd issues in screenshots (particularly forms showing a list of items to select singly, and having multiple items show as highlighted).
Steps to reproduce
Tell us about your environment:

Puppeteer version: 1.11.0 & 1.16.0 (retested on newest after identifying what the problem was)
Platform / OS version: Debian Testing, Centos 7, Ubuntu 16.04
Node.js version: 10.15.3

What steps will reproduce the problem?

Use a device descriptor that requires scaling when taking the screenshot (both 'iPad' and 'iPhone6' show the issue).
Use either page.click() or page.hover() to move the mouse to a particular element in the page.
Take a screenshot.

What is the expected result?
The mouse location should be scaled in the same way that the page has been scaled when taking the screenshot so that the screenshot has the mouse located over the specified element.
What happens instead?
The mouse remains in the original unscaled (x,y) location prior to scaling, which means it is no longer located over the particular element that had been selected in the page.click() or page.hover() functions.
This means that for device descriptors that use scaling page.hover() cannot be used at all (the mouse is never located over the specified element when the screenshot is actually taken), and that if the page.click() item is in a long list of items it is essential to use page.move() to move the mouse into a location outside of where the scaled list will be located (otherwise it will hover over another element in the list, likely breaking the purpose of the screenshot and causing confusion when it appears that multiple items were 'selected').

If I understand correctly, you are doing this
await page.hover('my element');
await page.emulate(devices['Fancy Device']);
await page.screenshot();

And the page.emulate call is causing the page to move under the mouse, messing up your screenshot.
Assuming I didn't get it wrong, is there any reason you cant do this?
await page.emulate(devices['Fancy Device']);
await page.hover('my element');
await page.screenshot();

I don't think there is a right answer for what to do with the mouse after the viewport has been scaled. Your elements are probably in different positions. Some might be offscreen.

You have misunderstood. I am already setting up the device viewport at the beginning prior to moving/taking a screenshot (navigating around the site taking a sequence of screenshots - I've pulled out the specific bits of code that demonstrate where the issue occurs).
import * as path from 'path';
import * as puppeteer from 'puppeteer';
import * as devices from 'puppeteer/DeviceDescriptors';

const iPhone = devices['iPhone 6'];
const iPad = devices.iPad;
const outDir = path.resolve(__dirname, 'screenshots');
const pageLoadOptions: Partial<puppeteer.NavigationOptions> = { waitUntil: ['load', 'networkidle2'] };
const DEBUG_SCREENSHOTS = process.env.DEBUG_SCREENSHOTS;

const puppeteerLaunchOptions: puppeteer.LaunchOptions = {};
if (DEBUG_SCREENSHOTS) {
  puppeteerLaunchOptions.headless = false;
}

const takeScreenshots = async (viewport: puppeteer.Viewport, subdir: string) => {
  const browser = await puppeteer.launch(puppeteerLaunchOptions);
  const page = await browser.newPage();
  try {
    page.on('console', (msg) => console.log('PAGE LOG:', msg.text()));
    await page.setViewport(viewport);

    await page.goto('https://some-site.com', pageLoadOptions);

    await page.waitForSelector('#addBid');
    await page.click('#addBid');
    await page.waitForSelector('#reqFlight');
    await page.hover('#reqFlight');
    // This screenshot does NOT show the '#reqFlight' option as hovered - the mouse is over the list
    // header instead of over the list item requested.
    await page.screenshot({
      path: path.resolve(outDir, subDir, `FlightRequest.png`),
      type: 'png',
      fullPage: false
    });

    await page.click('#reqFlight');
    await page.waitForSelector('#CPT');
    await page.click('#CPT');
    // I have to add a page.mouse.move(0,0) here, or an option starting with "A" in the list is hovered
    // when the screenshot is taken, despite having explicitly requested that the mouse should be over
    // the '#CPT' option.
    await page.screenshot({
      path: path.resolve(outDir, subDir, `FlightDestinations.png`),
      type: 'png',
      fullPage: false
    });

  } catch (e) {
    console.log(e);
    throw e;
  }
  await page.close();
  await browser.close();
};

if (!DEBUG_SCREENSHOTS) {
  Promise.all([takeScreenshots(iPad.viewport, 'iPad'), takeScreenshots(iPhone.viewport, 'iPhone6')]).catch((err) =>
    process.exit(-1)
  );
}
else {
  takeScreenshots(iPad.viewport, 'iPad')
    .then(() => takeScreenshots(iPhone.viewport, 'iPhone6'))
    .catch((err) => {
      process.exit(-1);
    });
}

What I expected to see from the first screenshot is the option labelled "Flight Request" to be highlighted as the mouse hovering over it. What I actually see in the screenshot is some other element in the heading section of the page has the mouse hovering over it.
What I expected to see from the second screenshot is the option labelled "CPT" to be set to "active". What I actually see from the second screenshot is the option labelled "CPT" is set to active AND one of the other options starting with "A" highlighted as the mouse hovering over it.
The only "change" to the screen after hovering/clicking on an item are those done internally by puppeteer itself when I request the screenshot to be taken.

I see, thanks for the code. It certainly sounds like a bug. Does it happen the same with headless set to true/false?

Yes. It was only after setting headless to true, and adding a 30 second delay prior to the screenshot, that I finally saw that the screen itself was being set up correctly but the screenshot would leave the mouse where it had originally been set rather than scaling it with the scaling of the screenshot view resulting in the wrong elements being hovered.

We're seeing some strange behavior in my scripts. We want to simulate mouse movement and have that be shown in the script. We're using some math to create the cursor tracking between 2 sets of X/Y coordinates.
When emulating a mobile device, the coordinates become more and more "off", especially when scrolling is involved.


[WIP] upgrade fogg to 0.29.1 and terraform to 0.12.5

Upgrading to terraform > 12 (required by latest version of fogg) is a much bigger headache than I'm willing to take on right now


After changing DB name (connection) it takes default db name and throw error

I have two db for example
DB1  is default for all user
DB2 for admin so i made two different connection first one MySql and second one is MySql2
when i changed admin model connection i takes default connection and throw an error that table not exist,
here is my connection code in model i am using laravel 5.3
protected $connection = 'mysql2';
so i am wrong or there is bug

The package has nothing to do with db connection. You are missing something.

You made a great package it saved my lots of time for that Thanks a lot and i figured it out where is the issue it is in the
protected function validator(array $data)     {         return Validator::make($data, [             'name' => 'required|max:255',             'email' => 'required|email|max:255|unique:admins',             'password' => 'required|min:6|confirmed',         ]);     }
in the unique validation it is selecting default db but it changed it
into this
unique:mysql2.admins
and it works fine so again thanks a lot


Adding custom Three.js objects

Hi,
Is it possible to add custom objects to the stage's scene?
I had a go (though I'm still using an older NGL version where the THREE object is global) and managed to add an object, using the coordinates of an atom, but it didn't appear; I assume because of a lighting or coordinates issue?

Is it possible to add custom objects to the stage's scene?

There is no api for it and I am reluctant to generally support it.
What are you trying to do? Would something like a a stage.addMeshComponent method help?

I'd like to add a custom object to highlight particular residues as being important, without interfering with representations of the structure, i.e., as an extra layer of annotation.
stage.addMeshComponent would be great :) but I wouldn't want to ask you too support something that would require a lot of maintenance. Would doing it directly via the stage's viewer's Three scene not be desirable? That's what I tried (and failed!) to do in the older NGL version.
Many thanks,

The highlighting sounds like something as #158. Feel free to add what you need from a highlight method.
I have been "creative" with my use of the THREE.Scene object...
Supporting THREE.Object3D would be much more than allowing adding a mesh, so I would like limit it to meshes and hide the use of THREE behind an API.

To highlight residues or regions of interest at the moment I'm adding separate representations, prefixing them with hl_ so I can turn them on and off. This would be to go beyond that and be able to have those highlights on while showing further annotations on top of what the highlight is showing.
Here's what I tried so far:

// INSTANTIATE A LOADER
var loader = new THREE.JSONLoader();

// LOAD A RESOURCE
loader.load(
	// RESOURCE URL
	'assets/3d/star.js',
	// FUNCTION WHEN RESOURCE IS LOADED
	function ( geometry, materials ) {
		var material = new THREE.MultiMaterial( materials );
		var star = new THREE.Mesh( geometry, material );

		// NO IDEA IF THIS HELPS ^_^
		star.scale.set(1.5, 1.5, 1.5);

		// KNOWN ATOM POSITION
		star.position.set(36.9, -43.9, 60.6);

		stage.viewer.scene.add( star );
		stage.viewer.requestRender();
	}
);

I have been "creative" with my use of the THREE.Scene object...

😄

So, you would like to have a star geometry?

As an example, though it would be nice to customise. A star was requested by my supervisor!

So, a custom mesh for highlighting of atoms (residues, chains, models).

That would be perfect, if possible.

That would be perfect, if possible.

Will try

You can now add meshes as components using the new Shape class
var shape = new NGL.Shape( "shape" );
shape.addMesh(
    [ 0, 0, 0, 0, 1, 0, 0, 0, 1, 0, 1, 1, 0, 1, 0, 0, 0, 1 ],  // a plane
    [ 0, 1, 0, 0, 1, 0, 0, 1, 0, 0, 1, 0, 0, 1, 0, 0, 1, 0 ]  // all green
);
var shapeComp = stage.addComponentFromObject( shape );
shapeComp.addRepresentation( "buffer" );

or add it to StructureComponent
var shape = new NGL.Shape( "shape" );
shape.addMesh(
    [ 0, 0, 0, 0, 1, 0, 0, 0, 1, 0, 1, 1, 0, 1, 0, 0, 0, 1 ],  // a plane
    [ 0, 1, 0, 0, 1, 0, 0, 1, 0, 0, 1, 0, 0, 1, 0, 0, 1, 0 ]  // all green
);
structureComp.addBufferRepresentation( shape.getBufferList() );

Amazing! I will be away next week, but will test this as soon as possible. Really cool progress, as usual.

Given that you can add arbitrary meshes with shape.addMesh I close this.


Support pointer feature

[x] parse pointer
[x] code gen pointer
[ ] code gen pointer arguments
[ ] support pointer to pointer

Trying to work on arguments... 🤓


dynatrace webhook pod crashes due to failure of liveness and readiness probe

Describe the bug
I am trying to install dynatrace webhook with dynatrace operator helm chart - https://github.com/Dynatrace/dynatrace-operator/tree/main/config/helm/chart/default. After successful deployement of the chart, i can see that operator pod is running but dynatrace-webhook pod is not running and crashes after sometime due to failure of readiness and liveness probe. With below error
Readiness probe failed: Get "http://<IP_ADDRESS>:10080/readyz": dial tcp <IP_ADDRESS>:10080: connect: connection refused
  Warning  Unhealthy  96s (x3 over 116s)  kubelet
      Liveness probe failed: Get "http://<IP_ADDRESS>:10080/livez": dial tcp <IP_ADDRESS>:10080: connect: connection refused

From the error looks like , the port liveness and readiness probe that webhook is trying to run is not open or available. hence i tried to change to couple of ports as well. But still doesn work
To Reproduce
Steps to reproduce the behavior:

Go to 'https://github.com/Dynatrace/dynatrace-operator/tree/main/config/helm/chart/default"
Click on 'Deploy the helm chart"
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hi, are you using the official instructions to install the helm chart? https://www.dynatrace.com/support/help/setup-and-configuration/setup-on-container-platforms/kubernetes/get-started-with-kubernetes-monitoring/set-up-k8s-monitoring-helm#install-helm
It looks like you are using the helm chart from main, but with the operator v0.12.0 image, which is not supported at all.

Seems like you have used the wrong helm chart. I will proceed to close this issue.
If you need more assistance, please reach out via Dynatrace ONE support rather than Github. This helps our team respond as quickly as possible using the support team's tools and procedures.
Thanks!

Hi
No i am using the right helm chart from dynatrace which is mentioned in the document.
Sorry for the delay in reply,
Regards,
krimesh

Is this is the chart from main  https://www.dynatrace.com/support/help/setup-and-configuration/setup-on-container-platforms/kubernetes/get-started-with-kubernetes-monitoring/set-up-k8s-monitoring-helm#install-helm which is mentioned in the official document?


fix: Fix to delete ~/.finch when uninstalling finch

In the current implementation, uninstalling finch does not delete the ~/.finch directory.
This directory may hold several GB of container data disks, which can consume a lot of space and be left unremoved.
This issue has also been reported in issue/#457 and needs to be fixed.
Therefore, this fix will ensure that when finch is uninstalled, ~/.finch is also deleted to free up disk space for finch.
Issue #, if available: issue/#457
Description of changes: Details are described in commit messages.
Testing done: Yes

[x] I've reviewed the guidance in CONTRIBUTING.md

License Acceptance
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thanks for the contribution! I think since this deals with users' persistent data, it would be better to have it opt in instead when uninstalling, or at least give a prompt for if the user would like to keep their data.


Documentation: add link to V&V book

It'd be nice to add the link to impressive FESTIM V&V book on the main page.
The link: https://github.com/festim-dev/V-V-report

I'm keen on doing this


Remove all deprecated code

Overview
It will be one of the latest things before v1 release.

MERGED to #86


Python 3.13t support

Now that PyO3 0.23 is released, I think users will increasingly be trying to build for freethreaded Python.
What's the correct approach to do this? I have been working on this in https://github.com/pydantic/jiter/pull/165, I can get it to work but only by:

sending the path output setup-python instead of an interpreter version, because 3.13t is not recognised by maturin (I think)
on manylinux images, manually working out where to look inside the image

It would be nice if setting --interpreter 3.13t as a maturin flag just did the right thing, or maybe there's another similar easy way.
It's more complicated in jiter and pydantic-core because we use PGO optimization so have quite a fiddly build 🙈

See https://github.com/PyO3/maturin/pull/2310 which add -i python3.13t support, I haven't invest much time in it due to lacking 3.13t support in setup-python makes it hard to test it.

Thanks, commented with a suggestion on that issue 👍


Fix issues with authentication for hooks.

Change should address #199 and #198.
Events will be parsed once to get the repo, then again before passing down to any relevant jobs.  This allows for authenticating as needed for relevant jobs.

@arcivanov Will you please review and see if this is an acceptable compromise?

Let me test this to make sure. Rollback didn't seem to have fixed the issue for me and I was trying to figure out why.

Nope, something is still broken:
2015-09-26 07:09:15.160: Chuck Ocheret<EMAIL_ADDRESS> has commits in PR[45999990] that is to be merged
2015-09-26 07:09:15.160: Merging the pull request
2015-09-26 07:09:15.176: ERROR: Publisher 'Github Pull Request Merger' aborted due to exception:
2015-09-26 07:09:15.176: java.io.FileNotFoundException: {"message":"Not Found","documentation_url":"https://developer.github.com/v3/pulls/#merge-a-pull-request-merge-button"}
2015-09-26 07:09:15.177: 	at org.kohsuke.github.Requester.handleApiError(Requester.java:501)
2015-09-26 07:09:15.177: 	at org.kohsuke.github.Requester._to(Requester.java:248)
2015-09-26 07:09:15.177: 	at org.kohsuke.github.Requester.to(Requester.java:194)
2015-09-26 07:09:15.177: 	at org.kohsuke.github.Requester.to(Requester.java:182)
2015-09-26 07:09:15.177: 	at org.kohsuke.github.GHPullRequest.merge(GHPullRequest.java:283)
2015-09-26 07:09:15.177: 	at org.kohsuke.github.GHPullRequest.merge(GHPullRequest.java:269)
2015-09-26 07:09:15.177: 	at org.jenkinsci.plugins.ghprb.GhprbPullRequestMerge.perform(GhprbPullRequestMerge.java:186)
2015-09-26 07:09:15.177: 	at hudson.tasks.BuildStepMonitor$3.perform(BuildStepMonitor.java:45)
2015-09-26 07:09:15.177: 	at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:776)
2015-09-26 07:09:15.177: 	at hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:723)
2015-09-26 07:09:15.177: 	at hudson.model.Build$BuildExecution.post2(Build.java:183)
2015-09-26 07:09:15.177: 	at hudson.model.AbstractBuild$AbstractBuildExecution.post(AbstractBuild.java:670)
2015-09-26 07:09:15.177: 	at hudson.model.Run.execute(Run.java:1763)
2015-09-26 07:09:15.177: 	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
2015-09-26 07:09:15.177: 	at hudson.model.ResourceController.execute(ResourceController.java:98)
2015-09-26 07:09:15.177: 	at hudson.model.Executor.run(Executor.java:381)
2015-09-26 07:09:15.177: Caused by: java.io.FileNotFoundException: https://api.github.com/repos/cschcs/pyenv/pulls/24/merge
2015-09-26 07:09:15.177: 	at sun.reflect.GeneratedConstructorAccessor61.newInstance(Unknown Source)
2015-09-26 07:09:15.177: 	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
2015-09-26 07:09:15.177: 	at java.lang.reflect.Constructor.newInstance(Constructor.java:422)
2015-09-26 07:09:15.177: 	at sun.net.www.protocol.http.HttpURLConnection$10.run(HttpURLConnection.java:1889)
2015-09-26 07:09:15.177: 	at sun.net.www.protocol.http.HttpURLConnection$10.run(HttpURLConnection.java:1884)
2015-09-26 07:09:15.177: 	at java.security.AccessController.doPrivileged(Native Method)
2015-09-26 07:09:15.177: 	at sun.net.www.protocol.http.HttpURLConnection.getChainedException(HttpURLConnection.java:1883)
2015-09-26 07:09:15.177: 	at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1456)
2015-09-26 07:09:15.177: 	at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1440)
2015-09-26 07:09:15.177: 	at sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:254)
2015-09-26 07:09:15.177: 	at org.kohsuke.github.Requester.parse(Requester.java:458)
2015-09-26 07:09:15.177: 	at org.kohsuke.github.Requester._to(Requester.java:227)
2015-09-26 07:09:15.177: 	... 14 more
2015-09-26 07:09:15.177: Caused by: java.io.FileNotFoundException: https://api.github.com/repos/cschcs/pyenv/pulls/24/merge
2015-09-26 07:09:15.177: 	at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1835)
2015-09-26 07:09:15.177: 	at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1440)
2015-09-26 07:09:15.177: 	at java.net.HttpURLConnection.getResponseCode(HttpURLConnection.java:480)
2015-09-26 07:09:15.177: 	at sun.net.www.protocol.https.HttpsURLConnectionImpl.getResponseCode(HttpsURLConnectionImpl.java:338)
2015-09-26 07:09:15.177: 	at org.kohsuke.github.Requester.parse(Requester.java:454)
2015-09-26 07:09:15.177: 	... 15 more
2015-09-26 07:09:15.180: Started calculate disk usage of build

Last it worked for me was 2015-09-23 19:38:07.942: Pull request successfully merged UTC.

Sep 26, 2015 9:01:50 AM WARNING org.jenkinsci.plugins.ghprb.GhprbRootAction doIndex
Anonymous GitHub: org.kohsuke.github.GitHub@323d2c4a
Sep 26, 2015 9:01:50 AM WARNING org.jenkinsci.plugins.ghprb.GhprbRootAction doIndex
Anonymous issueComment: org.kohsuke.github.GHEventPayload$IssueComment@2b04eb93
Sep 26, 2015 9:01:50 AM INFO org.jenkinsci.plugins.ghprb.GhprbRootAction doIndex
Checking issue comment 'org.kohsuke.github.GHIssueComment@300cf729' for repo cschcs/pyenv
Sep 26, 2015 9:01:50 AM WARNING org.jenkinsci.plugins.ghprb.GhprbRootAction doIndex
Webhook GH Issue: org.kohsuke.github.GitHub@ad838cc
Sep 26, 2015 9:01:50 AM WARNING org.jenkinsci.plugins.ghprb.GhprbRootAction doIndex
Authed issueComment: org.kohsuke.github.GHEventPayload$IssueComment@5fb428ac
Sep 26, 2015 9:01:50 AM INFO org.jenkinsci.plugins.ghprb.GhprbWebHook handleComment
Checking comment on PR #24 for job HCS_-_PyEnv_-_Test_-_(PR)_-_(Autogen)
Sep 26, 2015 9:01:50 AM INFO org.jenkinsci.plugins.ghprb.GhprbPullRequest updatePR
Pull request #24 was updated on cschcs/pyenv at 9/26/15 9:01 AM by User:ocheret
Sep 26, 2015 9:04:53 AM INFO hudson.model.Run execute
HCS_-_PyEnv_-_Test_-_(PR)_-_(Autogen) #129 main build action completed: SUCCESS
Sep 26, 2015 9:04:55 AM WARNING hudson.model.AbstractBuild$AbstractBuildExecution reportError
Publisher 'Github Pull Request Merger' aborted due to exception:

And this
2015-09-26 09:04:55.762: Merging the pull request
2015-09-26 09:04:55.763: Merge PR GitHub is org.kohsuke.github.GitHub@5d909def
2015-09-26 09:04:55.763: Merging PR [org.kohsuke.github.GHPullRequest@61de8476] is anonymous: true

So the anonymous GitHub object in the org.kohsuke.github.GHPullRequest@61de8476 being merged is not a result of anonymous GitHubs in the GhprbRootAction.

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

I think the issue is in how the lookup of credentials is done.  @arcivanov which version of the credentials plugin are you using?  And are the credentials that are supposed to be used for these jobs the first on the main configuration dropdown?

I think the context is what is messing things up.

So using the context for the jobs is limiting the credentials that are being returned to only global credentials.

@arcivanov please recheck, I believe the anonymous connections won't happen anymore.

Nope:
2015-09-26 15:49:21.736: Merge PR GitHub is org.kohsuke.github.GitHub@57687a53
2015-09-26 15:49:21.737: Merging PR [org.kohsuke.github.GHPullRequest@74497ecd] is anonymous: true
2015-09-26 15:49:21.752: ERROR: Publisher 'Github Pull Request Merger' aborted due to exception:

Is it possible we're actually doing something by https://github.com/jenkinsci/ghprb-plugin/commit/fdb2639f9985a335f6d06ea75dc33fc8ef63a781 ? Some transient we lose somewhere?

@DavidTanner I haven't touched credentials in months. I also haven't updated the credentials plugin recently and until Sept 23 it all worked, but I haven't attempted the merge until last night and that's where I discovered a failure. So it's something between the 1.28.2 and current.
I have an only credential for my bot and I reverified and resaved both globally and regenerated the jobs.

Is your credential set up with as global?

yes, has been like that for months.
I'm gonna start a fresh branch off of a release a few days back (Sept 22nd) and start working my way up the commit ladder.

@DavidTanner fyi: https://github.com/jenkinsci/ghprb-plugin/commits/merge_authentication_failure

@DavidTanner I'm getting this in the Jenkins main config page while testing:
Test basic connection to GitHub
Connect to API
Connected to https://api.github.com as null (null) login: smfybot
Notice null (null). This is with https://github.com/jenkinsci/ghprb-plugin/commits/merge_authentication_failure branch

@DavidTanner I think I know what the problem is - we're confusing credentials somewhere.
I'm looking at my Jenkins org.jenkinsci.plugins.ghprb.GhprbTrigger.xml and the <encodedAuthorization>token for account smfybot varies.
I think somewhere somehow we've stored the wrong value in and now it's being pulled from persistence and used.

@DavidTanner yes, I confirm that Pull Request stored in the "org.jenkinsci.plugins.ghprb.GhprbTrigger.xml" has no credentials, that why I can't do ANYTHING with it - authentication token always comes out to be null.
I think in general we need to merely store the REFERENCE to the last Pull Request state in the persistence map and look it up at runtime as opposed to deserialize to avoid these problems.

This is the difference between a successful merge build and an unsuccessful one:
           <pullRequestAuthor<EMAIL_ADDRESS>             <url>https://api.github.com/users/arcivanov</url>
             <id>5355666</id>
             <created__at>2013-08-31T23:46:27Z</created__at>
-            <updated__at>2015-09-17T18:01:57Z</updated__at>
+            <updated__at>2015-09-24T10:03:47Z</updated__at>
             <root>
-              <login>smfybot</login>
-              <encodedAuthorization>token hiddentoken</encodedAuthorization>
-              <users class="hashtable">
-                <entry>
-                  <string>smfybot</string>
-                  <org.kohsuke.github.GHMyself>
-                    <url>https://api.github.com/users/smfybot</url>
-                    <id>8435924</id>
-                    <created__at>2014-08-13T07:45:20Z</created__at>
-                    <updated__at>2015-06-29T08:15:44Z</updated__at>
-                    <root reference="../../../.."/>
-                    <login>smfybot</login>
-                    <avatar__url>https://avatars.githubusercontent.com/u/8435924?v=3</avatar__url>
-                    <gravatar__id></gravatar__id>
-                    <html__url>https://github.com/smfybot</html__url>
-                    <followers>1</followers>
-                    <following>0</following>
-                    <public__repos>0</public__repos>
-                    <public__gists>0</public__gists>
-                  </org.kohsuke.github.GHMyself>
-                </entry>
-              </users>
+              <users class="hashtable"/>
               <orgs class="hashtable"/>
               <apiUrl>https://api.github.com</apiUrl>
               <rateLimitHandler class="org.kohsuke.github.RateLimitHandler$1"/>
-              <connector class="org.jenkinsci.plugins.ghprb.HttpConnectorWithJenkinsProxy"<EMAIL_ADDRESS>+              <connector class="org.jenkinsci.plugins.ghprb.HttpConnectorWithJenkinsProxy"<EMAIL_ADDRESS>             </root>

@DavidTanner what is that that you merged just now?

Just to give you an update - the persisted data cleanup that you started incidentally solves the problem of keeping and permanently using anonymous GHPullRequest objects with their anonymous GitHub roots. I merged those and started fixing the unit tests. What is the c4e0d2 change? There seems to be a lot in it - what does it try to accomplish?

@DavidTanner This commit actually works and needs unit tests fixed. Please don't overwrite it.

This solves all the persistence problems and NPEs resulting from the change.

I had removed the init loop from the Repo, and was just putting it back.  The PullRequests only need to have init called once, not every time they are checking to see if something changed.

I also fixed the tests so they passed.

Did you test the code I added to branch before you reverted them?

The CI showed broken commit still, I did.

The tests all passed locally for me

Not on CI, sorry.

can you push again please?

https://jenkins.ci.cloudbees.com/job/plugins/job/ghprb-plugin/550/testReport/


Create foldable/large screen helper module

To simplify compose samples code by abstracting common code
(branch is based on test-compose-gallery branch)

The final change and test cases look good! I think once you remove the JWM dependency from the samples, you also need to remove FoldingFeature code. Let's ship it! :)

The final change and test cases look good! I think once you remove the JWM dependency from the samples, you also need to remove FoldingFeature code. Let's ship it! 😄

What does that mean?

The final change and test cases look good! I think once you remove the JWM dependency from the samples, you also need to remove FoldingFeature code. Let's ship it! 😄

What does that mean?

@hello-smile6 this comment was about removing unnecessary Jetpack Window Manager library dependencies and imports, and the issue was fixed in the last three commits of the PR

The final change and test cases look good! I think once you remove the JWM dependency from the samples, you also need to remove FoldingFeature code. Let's ship it! 😄

What does that mean?

@hello-smile6 this comment was about removing unnecessary Jetpack Window Manager library dependencies and imports, and the issue was fixed in the last three commits of the PR

Oh.


Add a free licensed book to r-resources.md

Add licensed free book to r-resources.md

due to restructuring, this conflicts; feel free to add a new PR if you like


Added Examples and Impoved starting chrome in headless mode.

Added an Example folder with a simple google search example/tutorial.
Improved starting chrome driver in headless mode by adding a headless parameter

LGTM!!


CB-21406 Extend Distrox Creation call on API with the azure database type field

CB-21406 Extend Distrox Creation call on API with the azure database type field
API related changes:

Extend DistroXDatabaseRequest with DistroXDatabaseAzureRequest
DistroXDatabaseAzureRequest contains the AzureDatabaseType: SINGLE_SERVER / FLEXIBLE_SERVER
if the DistroXDatabaseAzureRequest is missing, the default value is SINGLE_SERVER

Refactor Stack entity object:

Collect database properties in Database entity object
Add Database to Stack

For backward compatibilty the old database properties cannot be removed in one step from Stack, there is a followup jira for this: CB-22002

retest this please


Update manifest template

Future tarballs would now include the Sphinx documentation and makefiles and the project's LICENSE file. I have added the CHANGELOG.md and README.md, which are desirable for packaging in Linux distributions.
Closes #51

Thanks a lot for this @ghisvail!


ci: add xmlsec to the downstream tests

What problem is this PR intended to solve?
#2594 points out downstream breakage in the nokogiri-xmlsec-instructure gem.
Let's add it to the CI suite to prevent regression.

Blocking merger on https://github.com/instructure/nokogiri-xmlsec-instructure/pull/14

Now that downstream has patched, I've rebased to double-check that this is green now.

Downstream integration test is green: https://github.com/sparklemotion/nokogiri/actions/runs/3769661974/jobs/6409012842


Added is.leapYear(number:year) function

Hi Aras,
I added is.leapYear function + tests in this pull request.
Thanks
Hamid

Thanks for the PR @hamid6
Merged it.


install.sh failed on Windows10(msys64)

vim or neovim?

[x] gvim

Output of vim --version: 8.1.429
Operating system:

[x] Windows + msys64 + gcc 8.2.0 +  mingw-w64-x86_64-python3

Configurations related to LeaderF in vimrc:

Plug 'Yggdroot/LeaderF', { 'do': './install.sh' }

Describe your question, feature request, or bug.
Run install.sh failed.
lib can be compiled, but script failed at cp lib to autoload/leaderf/python .
Steps to reproduce
> cd ~/.vim/plugged/LeaderF
> ./install.sh
Begin to compile C extension of Python3 ...
...
cp: cannot stat 'build/lib*3.?/*.so': No such file or directory
> ll ./autoload/leaderf/fuzzyMatch_C/build/lib.mingw-3.7/
total 120K
-rwxr-xr-x 1 John None 61K Sep 27 17:32 fuzzyEngine-cpython-37m.dll*
-rwxr-xr-x 1 John None 54K Sep 27 17:32 fuzzyMatchC-cpython-37m.dll*

In install.sh, should use cp build/lib*3.?/* ../python instead of cp build/lib*3.?/*.so ../python
In msys64, dynamic lib suffix is always '.dll' not '.so'

On windows, please use install.bat.

install.bat. can't run in cygwim or msys64.
On Windows, I always launch vim through shell like fish, it works well with tool chain like mingw64 gcc, clang, cmake and also with all .sh scripts and most linux env settings, I think lots of people handle things like this.
install.bat is not proper here, and it will never work.

What is in your build/lib*3.?/ ?

Fixed.


Per shellcheck, fgrep is deprecated

Use grep -F instead.
Small thing, but shellcheck is pretty good about these things.

That turned my screen gray and there’s an arrow up and down I can move around. That’s about it

@chendushek I have no idea how that would happen; grep shouldn’t change your screen in any way at all. This change simply drops a deprecated binary in favor of the still-maintained one, adding a flag so the behavior is the same.
What happens if you run the previous version of this script? The effect should be the same, good or bad.

@mathiasbynens this has been open for over a year, so I get the sense you’ve no intention of ever merging it 🙂 Totally cool.
I’m closing it to reduce a bit of clutter. Feel free to re-open & merge it if you want it!


Fixed issue that prevented the two arguments from being used together

I love this program and I use it daily to fix Windows 10+'s horrible issue where it just forgets to create previews for PNGs bigger than 1440p.
I wanted to set up a task that would periodically refresh the previews of a folder of mine, which gets new pictures I take in VRChat every few days or so.
I wasn't able to use the two arguments (-r and -s) together, because of how the program handled the arguments internally, so I fixed the code a bit.
I also took the liberty to update the project to .NET 4.8.

Hi! Thank you for your contribution. But you can already use both options:
WinThumbsPreloader.exe -rs "path"


make the copper app buildable directly from a CopperContext

Today we have a helper basic_copper_setup that sets up all the runtime for copper:
    let copper_ctx = basic_copper_setup(&PathBuf::from(logger_path), SLAB_SIZE, false, None)
        .expect("Failed to setup copper.");

But the generated application from the ron file gets separately the clock and the logger from that context.
    let mut application =
        PtCloudsApplication::new(copper_ctx.clock.clone(), copper_ctx.unified_logger.clone())
            .expect("Failed to create application.");

Maybe we could officialise this CopperContext and just have the generated application from cu29_derive get a context?

I'd like to try this one! Just to make sure I understand, after this change, the code that builds the application should be something like this?
    let mut application =
        PtCloudsApplication::new(&copper_ctx)
            .expect("Failed to create application.");

Also, I'm not sure if the idea is to replace the current "new" method, which would break code from this version that use it, or to keep the current "new" method and create another method that takes CopperContext.
I was going to suggest the From trait, so the application could be built with App::From(&context) or context.into(), but I'm not sure it's the optimal solution, because if sim_mode is set to true, the "new" method needs a callback parameter that is not available in the CopperContext.

maybe adding another new_from_context or something like that would be a little bit more flexible & idiomatic. That said we could always add parameters to the context.
Another proposal would be to use a builder pattern like : Application::new().with_context(...).sim_mode(...)... .build() // would be very nice no?
also I think we would need to move context in the cu29_runtime crate to make that cleaner.
Let's get a first draft going and we can jam on it.
Thanks for picking that up!

The builder pattern looks like a good solution!
I will experiment with the code and submit a draft when I have something.


Switch to crates.io version of avr-device

Hey,
I've published avr-device to crates.io and this removes all the build-dependency madness.  In this PR, I've updated your repo to reflect the changes.  You might want to update your blog-post as well :)
Disclaimer: I could only compile-test the change.  While this shouldn't break anything, I suggest, you quickly test for yourself if everything is still working ...

Currently it doesn't build on nightly, since the version is rustc 1.46.0-nightly (346aec9b0 2020-07-11), but the minimum version is rustc 1.47.0-nightly (0820e54a8 2020-07-23)

@audioXD now that rustfmt is available in the latest nightly, we can finally get rid of that!

@Rahix thanks, that's much better!
One thing to think about is using #[lang = "start"] instead of a macro. I used it in another project, and it allows you to remove #![no_main] and anything special around the main function. The reason the ARM stuff doesn't use it is because they're designed to compile on stable, which doesn't support defining that. If you think that we'll sometime compile on stable, sure, keep it, but if not, maybe consider switching to that.

The macro also allows much more customization, like the static mut transformations which is really helpful in embedded applications.  Also, when you start writing custom runtime code, this gives more flexibility.

@lights0123 I had that comment because when i build with the current nightly version that's not the actual latest I get get an error.
[audioxd@Tims-PC-001 async-avr]$ rustup override set nightly
info: using existing install for 'nightly-x86_64-unknown-linux-gnu'
info: override toolchain for '/home/audioxd/Documents/Projects/async-avr' set to 'nightly-x86_64-unknown-linux-gnu'

  nightly-x86_64-unknown-linux-gnu unchanged - rustc 1.46.0-nightly (346aec9b0 2020-07-11)

[audioxd@Tims-PC-001 async-avr]$ cargo build --target=avr-atmega328p.json -Z build-std="core" --examples
***SKIPED LINES***

error: could not compile `async-avr`.

To learn more, run the command again with --verbose.
warning: build failed, waiting for other jobs to finish...
error: linking with `avr-gcc` failed: exit code: 1
  |
  = note: "avr-gcc" "-Os" "-mmcu=atmega328p" "-Wl,--eh-frame-hdr" "-L" "/home/audioxd/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/avr-atmega328p/lib" "/home/audioxd/Documents/Projects/async-avr/target/avr-atmega328p/debug/examples/serial-12f51f827fcd8952.serial.dp74pwxw-cgu.2.rcgu.o" "-o" "/home/audioxd/Documents/Projects/async-avr/target/avr-atmega328p/debug/examples/serial-12f51f827fcd8952.elf" "-Wl,--gc-sections" "-L" "/home/audioxd/Documents/Projects/async-avr/target/avr-atmega328p/debug/deps" "-L" "/home/audioxd/Documents/Projects/async-avr/target/debug/deps" "-L" "/home/audioxd/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/avr-atmega328p/lib" "-Wl,--start-group" "-Wl,--end-group" "-Wl,-Bstatic" "/home/audioxd/Documents/Projects/async-avr/target/avr-atmega328p/debug/deps/libcompiler_builtins-49d9ec43f42b08f8.rlib" "-Wl,-Bdynamic" "-Wl,--gc-sections"
  = note: /usr/bin/avr-ld: unrecognized option '--eh-frame-hdr'
          /usr/bin/avr-ld: use the --help option for usage information
          collect2: error: ld returned 1 exit status

I feel like it ignores the target file, more specificaly the "eh-frame-header": false. So I suggest using rustup override set nightly-2020-07-24

@audioXD you haven't updated your nightly rust since the 11th, run rustup update nightly

NWM I just ran rustup update nightly --force and got the acctual latest version.

And yes i have tried running it without --force

@audioXD did you have miri installed? --force will force installation, even if certain components aren't available. Miri still isn't available for nightly since 2020-07-18, so that could be why

@Rahix Oh... Yes i did :p. I also notices that it has to be built with the --release flag


Added missing semicolons

Hi, we are moving to Standard Code Style, so I'm afraid this PR will not be accepted.

@dougwilson could you please elaborate what decision should we take on this kind of PRs? Should it be closed?

Your comment above is correct. Closing.


Error Adding To tenant all lookbooks

Describe the issue
After selecting any lookbook>add to tenant>Sorry, something went wrong
There was an unexpected issue.
An item with the same key has already been added.
Expected behavior
we expect the site details questionnaire.
Screenshots

Details (please complete the following information):

PnP Correlation ID: [Provide a correlation ID for the provisioning action]
Reference Template: [please provide the URL of the template in the Look book site. It should be something like:
[- https://lookbook.microsoft.com/details/{Template-ID}]
]
all of them fail

lookbook tempate

Primary language of the target tenant: 1033
Are you using a Tenant Global Admin account to provision the template: [Yes]

Additional context
Add any other context about the problem here.

Hi,
Thanks for using the Look Book site. Could you please try again? We don't see any actual issue, so it could be just a temporary hiccup. Please let us know. Thanks.

Hello, it has been happening for days every time we try…
The only thing to mention is that we have dedicated domains rather than .sharepoint.com  we have team.homedepot.com portal.homedepot.com onedrive.homedepot.com
Attached is fiddler trace. Pwd thd
From: Paolo Pialorsi @.>
Sent: Friday, June 17, 2022 4:37 AM
To: SharePoint/sp-provisioning-service @.>
Cc: Galindo, Kenneth @.>; Author @.>
Subject: [EXTERNAL] Re: [SharePoint/sp-provisioning-service] Error Adding To tenant all lookbooks (Issue #639)
Hi, Thanks for using the Look Book site. Could you please try again? We don't see any actual issue, so it could be just a temporary hiccup. Please let us know. Thanks. — Reply to this email directly, view it on GitHub [github.com]
Hi,
Thanks for using the Look Book site. Could you please try again? We don't see any actual issue, so it could be just a temporary hiccup. Please let us know. Thanks.
—
Reply to this email directly, view it on GitHub [github.com]https://urldefense.com/v3/__https:/github.com/SharePoint/sp-provisioning-service/issues/639*issuecomment-1158638304__;Iw!!M-nmYVHPHQ!Mj5qQlPbOQE73t9BiqnleFYdzwj4T1rNJhORVuOIl9iB1appVJeyUXKse67aqGPfTi0QPIN2OThUA-HFqgdPfowawkEediWuHA$, or unsubscribe [github.com]https://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AZVINMOVGH3WTY6HX7WSXD3VPQ2LNANCNFSM5Y7VMIVA__;!!M-nmYVHPHQ!Mj5qQlPbOQE73t9BiqnleFYdzwj4T1rNJhORVuOIl9iB1appVJeyUXKse67aqGPfTi0QPIN2OThUA-HFqgdPfowawkF4zJhImg$.
You are receiving this because you authored the thread.Message ID: @.***>

The information in this Internet Email is confidential and may be legally privileged. It is intended solely for the addressee. Access to this Email by anyone else is unauthorized. If you are not the intended recipient, any disclosure, copying, distribution or any action taken or omitted to be taken in reliance on it, is prohibited and may be unlawful. When addressed to our clients any opinions or advice contained in this Email are subject to the terms and conditions expressed in any applicable governing The Home Depot terms of business or client engagement letter. The Home Depot disclaims all responsibility and liability for the accuracy and content of this attachment and for any damages or losses arising from any inaccuracies, errors, viruses, e.g., worms, trojan horses, etc., or other items of a destructive nature, which may be contained in this attachment and shall not be liable for direct, indirect, consequential or special damages in connection with this e-mail message or its attachment.


Updated Electrum-Dash Wallet Package

Motivation for this change
The existing electrum-dash package is quite a few version numbers behind (2.4.1 vs <IP_ADDRESS>). The list of default servers it provides are no longer active, and there are, I believe, at least one or two security vulnerabilities in the old code.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Sure, no problem.


Uncaught TypeError: (0 , _debug2.default)(...).enabled is not a function

Code from example throws an error and doesn't work.
import debug from 'debug';
import createLogger from 'react-render-logger';

const prefix = 'Proj:Trace:performance';
const enableChecker = componentKey => debug(prefix + componentKey).enabled();
const renderLogger = createLogger(enableChecker);

export default renderLogger;

I was not found debug in package.json and installed it by npm i debug -S. Can you help to solve this issue?

Yep. enabled is not a function, it's a property in debug instance.
const enableChecker = componentKey => debug(prefix + componentKey).enabled

This will work normally.
Thanks for this issue, we'll update this example

Works perfect now, thanks

import debug from 'debug';
import createLogger from 'react-render-logger';

const prefix = 'Tinnkoff:trace:performance:';
const enableChecker =
  componentKey =>
    debug(prefix + componentKey);

const renderLogger = createLogger(enableChecker);

export default renderLogger;

Remove enabled, work to me


I need help

How to use command to get SP

https://github.com/elBukkit/MagicPlugin/wiki/SpellPoints#giving-spell-points

https://github.com/elBukkit/MagicPlugin/wiki/SpellPoints#giving-spell-points


added HTTPS option as new connection mode

Hi cr4igo,
thank you very much for your pull request. We will take a look at it in the next week.
Kind regards
Michaela

Hi,
thank you again for your contribution, we merged your code :)


[CWS] add support for fchmodat2

What does this PR do?
This PR adds support for the fchmodat2 syscall added in kernel 6.6.
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Show caption/description in lightbox

Hello,
This is a question but I can't myself label it as so.
I was wondering if the description is shown when the lightbox is open?
With the following

description

Would (Could) the description be shown in full screen view?
Cheers,
M

For me, the examples in demo don't work and captions only show up in the lightbox.
e.g.
![FastAPI readme](release-hatch-160/rich-readme.png){ data-title="foo" }

The caption feature only adds the caption in the lightbox, not affecting the image on the page.
I used the figure and figcaption and GLightbox caption feature at the same time in the demo page before, which seems may confuse users. I just removed the figure and figcaption of all demos and added another demo about the figure and figcaption on caption page. Hopes this could make the descriptions of caption feature clear.

Oh nice, thanks!!!


NullAway nullness analysis interaction

Information at:

https://eng.uber.com/nullaway/
https://github.com/uber/NullAway
https://arxiv.org/abs/1907.02127

Having those links in the archives might be helpful someday, but for now I don't think there's a purpose for this bug staying open.


adding administrative levels dataset

I am changing datamart-geo to accept gzipped files, which will need to be uncompressed after download. Will that work for openclean? My plan was for the hash to still be over the uncompressed data.

Yeah, the openclean reference data repository supports gzip decompression so we should be fine there.

I think the problem currently is that the hash in refdata is computed over the downloaded file, not the uncompressed file. We would need to add the option to automatically decompress the file after download. The question is whether we want to do this automatically for all compressed files or only for some (e.g., based on a flag in the file description metadata)?

@maqzi: This seems to only support text data? I see CSV and JSON, and gzip.open(..., 'rt')
I can change it to the hash of the compressed file in datamart-geo.

datamart-geo ~0.2.2~ 0.2.3 is released with support for gzipped data. The info.json will contain the sha1 of compressed data for compatibility with your code.
I probably won't actually put a gzipped file in there until I am done integrating OpenStreetMap data though.


Add operator quick install manifest

This adds create-manifest.sh to generate a single manifest file for
installing the operator. This will be used by the CI to generate a quick
install manifest file for every release.
With this, it'll be possible to install the operator by doing:
$ kubectl create -f https://github.com/storageos/cluster-operator/releases/download/<version>/storageos-operator.yaml
customresourcedefinition.apiextensions.k8s.io/storageosclusters.storageos.com created
customresourcedefinition.apiextensions.k8s.io/storageosupgrades.storageos.com created
customresourcedefinition.apiextensions.k8s.io/jobs.storageos.com created
namespace/storageos-operator created
clusterrole.rbac.authorization.k8s.io/storageos-operator created
serviceaccount/storageoscluster-operator-sa created
clusterrolebinding.rbac.authorization.k8s.io/storageoscluster-operator-rolebinding created
deployment.apps/storageos-cluster-operator created

This is a really good idea and worked well.  A few small things (none blockers):

check for yq and link to install page (https://mikefarah.github.io/yq/) if not found
create storageos-operator namespace if it doesn't exist
make build/cluster-operator supports setting image name with OPERATOR_IMAGE env var, but make generate-install-manifest takes an argument to set OPERATOR_TAG.  Can we use the OPERATOR_IMAGE method for both?  For testing it's useful not to use storageos/cluster-operator and use a private registry account instead.

Added a make target to install yq when generating the manifest and also moved the manifest to project root. Also, changed the env var to OPERATOR_IMAGE, like the other build target.


Fix German translations for UI

Some of the German translations for the UI are wrong, please include them. Thanks!

In addition, added German translations to local PHP bridge and fixe some Englisch translations.

Thanks @jrondorf !!


working in php7.4

I want to make a humble request. Please add support for php7.x, because I can't upgrade my libraries to 8.x yet.
thanks.

hi @apolinux you can use the branch https://github.com/php-toolkit/pflag/tree/1.x

NOTE :) It has not been maintained for a long time. Maybe there is a bug.


Typo

Typo in https://nextjs.org/commerce : Instead of File-Sytem Routing it should be File-System Routing.
(By the way, unfortunately the boxes on this page are not selectable to copy&paste text.)

Thank you! Fixed.


UinputError (NotFound)

Hi,
First of all. Thank you for the great tool.
I'm having trouble running kbct on my notebook without an external keyboard connected. This is what I run and the error I receive:
-> % sudo kbct remap --config /home/malte/dotfiles/kbct/config.yml
 INFO  kbct > Starting kbct event loop, pid=4038
thread 'main' panicked at 'Could not capture keyboard: UinputError(NotFound)', src/main.rs:212:58
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

This is my config:
- keyboards: ["Keychron Keychron K3", "AT Translated Set 2 keyboard"]
  keymap:
  layers:
    - modifiers: ["capslock"]
      keymap:
        h: left
        j: down
        k: up
        l: right
        u: pageup
        p: pagedown
        i: home
        o: end
        comma: delete

This is what evtest says, so the device name should be correct:
/dev/input/event3:      AT Translated Set 2 keyboard

I used kbct yesterday with my external keyboard connected and the same config and it worked flawlessly.
Do you have any idea what I am doing wrong? I already removed the K3 device from the config and double checked the device name for my internal keyboard.

Hi @maltesa and thanks for reporting thus bug.
I have two questions.

Are you able to use evtest to actually log the keystrokes ? What I think is that the name might be misleading and in reality another keyboard should be used.
Is it true that you will be able to use your keychron without AT Translated Set 2 keyboard and it breaks only when you add the latter to the list of keyboards in the yaml file ?

Hi @samvel1024,
thank you for your quick response.

Are you able to use evtest to actually log the keystrokes ? What I think is that the name might be misleading and in reality another keyboard should be used.
yes, I can log keystrokes for both keyboard

Is it true that you will be able to use your keychron without AT Translated Set 2 keyboard and it breaks only when you add the latter to the list of keyboards in the yaml file ?
No, it worked with both names in the config when the k3 was connected as well (office setup). But at home I was using only the notebook with its internal keyboard (AT Translated) and got the mentioned error. Currently I'm in my office and it works flawlessly even when I disconnect the K3. I enabled kbct again and will try it later on in my home setup. Maybe it didn't work due to hibernation or something like that.

I close this issue for now. And reopen it if the problem occurs again.

The same happened to me couple of times. I too have two keyboards (one is from my laptop, and another one is connected via USB) one of which I sometimes disconnect. Then it requires some magic to happen to make it work as even when I the change config, removing the disconnected keyboard, it still prints this error. Reboot doesn't help really.

Wow. I guess, i found the root cause. I just checked if uinput kernel module is loaded and found out that it isn't. When I loaded it, kbct started normally.

@yunmikun2 ❤ I will try that later 👍🏼

Loading modprobe uinput solved the problem for me. Thank you @yunmikun2 👍🏼


nocrypto.0.5.0 illegal instruction

on a Quad-Core AMD Opteron 2350 we got:
#0  _mm_aeskeygenassist_si128 (__C=<optimized out>, __X=...) at /usr/lib/gcc/x86_64-linux-gnu/4.8/include/wmmintrin.h:88
#1  _nc_aesni_derive_e_key (rounds=<optimized out>, rk0=<optimized out>, key=0x1fe0b30 "") at src/native/aes/aesni.c:106
#2  caml_nc_aes_derive_e_key (key=140673840720632, off1=<optimized out>, rk=140673840720576, rounds=<optimized out>) at src/native/aes/aesni.c:331
#3  0x00000000006c5005 in camlNocrypto__Cipher_block__fun_2213 () at src/cipher_block.ml:371
#4  0x00000000006c4f81 in camlNocrypto__Cipher_block__of_secret_with_1584 () at src/cipher_block.ml:368
#5  0x00000000006c5bf2 in camlNocrypto__Fortuna__create_1298 () at src/fortuna.ml:22
#6  0x00000000006c700c in camlNocrypto__Rng__create_1190 () at src/uncommon.ml:38
#7  0x00000000006c76ba in camlNocrypto__Rng__entry () at src/rng.ml:49
#8  0x0000000000504d19 in caml_program ()

so it's using instructions it shouldn't.
Also, it would be nice to have the detection at runtime, instead of at compile-time, as we often don't know where it will be deployed.

this seems to be a duplicate of #72, fixed in master (which now has configuration-time detection whether AES-NI is available or not). Could you please pin to master and report whether this works for you?

compile time detection does not solve the issue for us, as we will always be compiling on a different system than where we deploy...

@toolslive What is your scenario here? Where do you deploy to, and what capabilities do CPUs there have? How unacceptable is it for you to opam pin nocrypto to the release, then opam edit nocrypto to add --disable-modernity to the configure step? Is this a production scenario?
Asking, because this is currently a balancing act: while at some point dynamic detection will land, I want more exposure for this code before complicating it further. So we're stuck with a static decision. The current master unconditionally disables acceleration if you pass --disable-modernity, and if you --enable it, it checks whether the build machine supports the instructions and still disables them if the current host can't run them. configure defaults to --enable.
Ideally, we could export package-level configuration options to opam, and you could tweak your list of configuration variables and have the packages respect them, a scheme Gentoo linux has been using for years. But as it stands, there is no way to let a user directly influence compile-time options of packages through opam (other than pin-and-edit). So unless we start hacking around with virtual packages that denote your preference for CPU acceleration, nocrypto has to stick with a single choice.
This choice can then be either requiring people to manually opt in, or opt out of acceleration. I am trying to feel out the user base here and decide what makes more sense, and I guess it all boils down to how prevalent this scenario is, where the deployment machine lacks features the build machine has, and how inconvenient the current mechanisms of control we have are for the users.
Feedback more than appreciated. I didn't release an update precisely to get a little more info about where we run and how often we break with the current scheme.

typically what happens is that we build an executable that ends up in a debian or redhat package.
that package gets installed on core-i5, core-i7, amd opteron and other intelloids.
so we would prefer something like this:
// gcc extension, run when lib is loaded...
int have_sse_4_2 = 0;

void __cpu_detect(void) __attribute__((constructor));
void __cpu_detect(void) {
  unsigned int a = 0, b = 0, c = 0, d = 0, func = 1;
  int r = 0;
  r = __get_cpuid(func, &a, &b, &c, &d);
  if (r == 0) {
    /* Lookup failed */
    c = 0;
  }
  have_sse_4_2 = (c & bit_SSE4_2) != 0;
}

and then dispatch based on the detected capabilities.

 It's quite clear how to go about detecting features, the problem is that the dispatch phase complicates code paths and, as I said, sufficiently so that it will have to wait a little.
The question still stands: it sounds like whatever you are doing is specific enough -- and certainly non-casual enough -- that the solution of pinning and editing configure line for your particular scenario is workable. Am I mistaken here? Have you tried that?

--disable-modernity would work.

I'm still trying to gauge how to proceed here. While it is possible to influence the build with configure flags, I'm not sure that the current state is good. I'm thinking of maybe disabling modernity by default if this causes problems for too many users.
Leaving this open until hopefully some opinions pile up. :smile:

What exactly do you mean with "the dispatch phase complicates code paths" ?
As far as I can tell, once the feature detection is done all that is still needed is one if/else. You could even do the dispatch in ocaml if you would prefer that.
You could then have tests which encode with one variant en decode with the other (if both are available according to the feature detection).

Around five just for AES. I am working on an accelerated GCM which adds several more varying entry-points, and actually the OCaml code has to change depending on presence of PCLMUL. Then, there is using SSE to accelerate hashing (which is the current choke point if you are doing authenticated encryption), and this will add several more.
From a different POV, just the argument order of C primitives has a significant impact (~20% as of several months ago) on the overall performance. I need to benchmark how much is lost if there is a dispatch on entry; probably not too much but I expect it to factor.
Then obviously there is the fact that this is portable to ARM, which means that parts of dispatched-to code can be compiled in only conditionally.
Obviously AES itself can indeed be solved by a few switch statements, but the C code was written with being brain-dead-simple as first, second and third priority. So I want to get this out, have a bit of testing and feedback, settle the upcoming wave of changes, and then sit down and figure out how to do this without littering the code with both static and dynamic checks, while having feature-based decisions in one place, and working uniformly for other feature-dependant code. One thing I would currently like to avoid is having feature tests all over the C stubs.
So right now, I would rather disable acceleration in default builds than hack dynamic detection in just so it kinda works.

maybe the dispatch should not be on function level, but on module level.

That was my idea, yeah. I wanted to have a few distinct bits of code that have different and feature-dependent code paths, then figure out how to neatly swap out modules for various alternatives.
Just had a chat with @hannesm; he thinks we should optionally take ./configure flags from an env var to make it easier to automate builds with non-default configurations.

Just had a chat with @hannesm; he thinks we should optionally take ./configure flags from an env var to make it easier to automate builds with non-default configurations.

I kind of dislike the implicitness of env vars. There may be a way to subvert opam's config files here. I have not tried this but if you have a look at the .opam/$SWITCH/config/global-config.config I think you can define variables in there and those will become available as opam variables. Maybe we could have some sort of opam config set $VAR $VALUE. I don't know what @AltGr thinks of that. This may not be the best solution in the long term and being able to specify variables at install time may may be a better option.

if you really want to pass a flag to opam via the env variabale, you can already do it. warning: it's ugly.
# an ugly opam file
pam-version: "1.2"
name: "toto"
version: "0.1"
build: ["echo" FOO]

$ opam pin add toto . -n
$ OPAMVAR_FOO=foo opam install foo -v
[...]
+ echo "foo" (CWD=/Users/thomas/.opam/profuse/build/toto.0.1)
- foo

If we could use .opam/$SWITCH/config/global-config.config to set variables, and have packages react to that, that would be absolutely smashing! Especially if we had a user interface for opam variables.
(It's also what Gentoo does, as mentioned, but at some point they went overboard.)
@samoht @AltGr -- thoughts?

You can put things in that file and that will work. You can also put things in <pkg>.config and call pkg:<var> in the opam file directly (doesn't work yet for self references, but will do in the next version).

See http://opam.ocaml.org/doc/manual/dev-manual.html#sec40 and opam config list to see how to use these .config files.

@samoht is this supposed to be generated by the build system so that packages can provide information about their configuration in the opam system for other packages to reuse ?

yup. but I guess you can use that to register some user preferences as well (eg. you can create a package crypto-conf with the configuration values and a hack to let the user change these values). But I agree that having opam config set is better than a hack.

Hm.
If I add a line foo: "bar" to ~/.opam/system/config/global-config.config, I can see it in opam config list via opam config var foo, and I can refer to it from a toto-like opam snippet.
If I create a file ~/.opam/system/config/nocrypto.config and add the same line, It doesn't register as nocrypto:foo. Same for any other package, installed or otherwise.
I've been re-reading the relevant portion of the dev manual, but I can't seem to figure out the central question: are package variables a closed set?
(opam 1.2.2)

Humph, the manual is wrong. The correct path is $prefix/lib/$pkg/opam.config which might not be the best name ever. /cc @AltGr

ah, the location for config files was changed some time ago from config/pkg.config to lib/pkg/opam.config, and we missed the update in the dev-manual. There was a good reason for the change, but I can't remember what it was -- I think that happened around the time we refactored pinned packages (i.e. in 1.2.0).
Implementing opam config set should be easy enough: the hard part is defining the interface and specifying what it does :)
I've already had requests for a way of specifying package configuration variables in advance.

ah, found it: https://github.com/ocaml/opam/issues/1385

Thanks, makes much more sense now. :smile: This almost works.
Problem 1:
opam seems to own $prefix/lib/$pkg. The good part is that I can create the directory and opam.config with the variables prior to installing a package, and installation picks up on that. The bad part is that the entire directory is removed on uninstall, which means that variables are gone on reinstall.
Wish: opam config set or such would set variables that would stick without being lost on reinstall.
Problem 2:
I can use the new var to optionally filter out an an argument, with ["--small-animals" {animals}]. But I would like to use a var to give value to an argument, like ["--%{animals}%-animals"] to either mean --small-animals or --medium-sized-animals. I can use the experimental boolean-to-string converter, and say ["--%{animals?medium-sized:small}%-animals"], but then I have to add a boolean to opam.config, like animals: true (-> --medium-sized-animals) and animals: false (-> --small-animals). If I want the user-facing variable to be a string, and get animals: "small" -> --small-animals and animals: medium-sized -> --medium-sized-animals, I need to be able to default it to something so that I don't get ---animals if it's not explicitly set (I think I need to default to --medium-sized-animals after all).
Wish: animals: "medium-sized" would parse in an opam file and the package would see the variable, but the variable would still be user-overridable. (Am I missing a way to default an arbitrary variable?)

There is the features field, but it is not supposed to be user-overridable (although it can refer to another variable). It was first intended for boolean values but the implementation handles strings too.

That looks about right, thanks!
But the fact that $prefix/lib/$pkg is deleted on reinstall, together with the custom package variables, means that this form of vars is not yet suited for user-facing configuration tweaks.
I added 444e627a29c0a70b3c6975e00185b9c1765e8027 and plan to release it tomorrow. With that, users can can echo "nocrypto-inhibit-modernity: true" >> ~/.opam/$(opam config var switch)/config/global-config.config to get rid of the machine-dependent code.
Going forward, I would like to avoid polluting global-config.config, though.

0.5.1 is tagged and out, pending merge to the opam repo.
Global opam variable nocrypto-inhibit-modernity can be used to force-disable acceleration without pinning or editing the package.
@toolslive @domsj
Clearly not the best solution, but will it do as an interim step?

But the fact that $prefix/lib/$pkg is deleted on reinstall, together with the custom package variables, means that this form of vars is not yet suited for user-facing configuration tweaks.

I don't think that .config file as it stands is the right mechanism. We want to distinguish a) .config file as they exist, that is a description of what was configured for other packages to use and b) being able to specify configuration parameters for a package.

Wish: animals: "medium-sized" would parse in an opam file and the package would see the variable, but the variable would still be user-overridable. (Am I missing a way to default an arbitrary variable?)

As I commented on the opam issue there should be a way to declare and document the variables, having a default value should be added to the list. But this discussion should be continued there.

for us, that interim option is ok.

@pqwy I have written a proposal there https://github.com/ocaml/opam/issues/2247 your feedback is welcome.

You can use -march=native (works with GCC and Clang) to get all the features of the build machine instead of specifying -m<feature> flags individually, however I don't recommend this: it may work fine for systems like gentoo, but can be quite a problem for any OS that distributes binary packages, and virtual machines are another problem.
There is also a GCC extension, GNU indirect functions (ifunc attribute) that can perform function dispatch at load time, but unfortunately only works on Linux (and perhaps FreeBSD) and not supported by Clang yet. Its probably possible to achieve something similar using function pointers and perform the dispatch once on startup (instead of load time like with ifunc) in a portable way.


node is not available win32 application!

And, the npm was not installed~
Thanks~

You have to install the 32 bit version explicitly on 64-bit operating systems.

If npm was not installed, there was probably a network problem. I suggest using a mirror.

@coreybutler
You said I had installed 32-bit version.
But why the bash give me this message?
Now using node v4.4.4 (64-bit)

the 64-bit?


Let's parse the zeroth-order claim formulas

Wait, I just realized that there could be a problem with text expansion. ZOC's can reference each other, i.e. P(1|2) = 0.6 and P(2|1) = 0.9. If we tried to expand that out, we would get "Conditional on this claim being true, P("Conditional on this claim being true, P("Conditional on ...")=0.9") = 0.6" nested forever.

I think I'm going to need to do something similar to definition attachments.

No, the only reason that definition attachments need to be below the claim is because we don't know where exactly to insert them inside the claim because language is messy. Placing the referenced text inside an attachment box wouldn't be any different from placing it inside the content box.

Okay, I think I'm making progress! There are some bugs to be worked out, though.

If an affine expression starts with a "-" then the program will parse it as subtraction and crash because there's nothing to the left of the "-" sign. Maybe check for affine expressions starting with a "-" and add a zero in front? But then that will appear in the displayed output.

Observations:

Use "implies" but only outside of probabilistic situations, allow conditionals but not in affine combinations.
Display a message with the accepted syntax beneath any invalid formula.
Shorten the claim ID and remove the author. The claim ID should be at most three characters long.

Remaining negative bug: -P(...) + ... as an affine expression refuses to parse.

Okay, the parser is working correctly now as far as I can tell! There are doubtless improvements to make, but at least it's working!

TODO:

Allow implies for logical formulas but only if outside a probabilistic setting (use a curried function).
Allow conditional probabilities but only in the form P( a | b ) = x.
I should display a message with the accepted syntax beneath any invalid formula.

For future branches:

I need to shorten the claim ID and remove the author. The claim ID should be at most three characters long.

TODO:

Allow implies for logical formulas but only if outside a probabilistic setting (use a curried function).
Allow conditional probabilities but only in the form P( a | b ) = x.
I should display a message with the accepted syntax beneath any invalid formula.
I need to create a list of forbidden Claim ID's (and, or, not, implies at least) and enforce the rules.

For future branches:

I need to shorten the claim ID and remove the author. The claim ID should be at most three characters long.

TODO:

Allow conditional probabilities but only in the form P( a | b ) = x.
I should display a message with the accepted syntax beneath any invalid formula.
I need to create a list of forbidden Claim ID's (and, or, not, implies at least) and enforce the rules.
Compute additional information, such as a parse tree or a dependencies set.

For future branches:

I need to shorten the claim ID and remove the author. The claim ID should be at most three letters or four numbers long. I could also follow a friend's suggestion and have the claim ID be editable, maybe using the same dropdown menu that I use for definition attachment and claim deletion.

Okay, I actually think I'm ready to merge this branch. The parsing seems to be working and the other tasks feel conceptually separate. The TODO's that I'm leaving to future branches are:

I should display a message with the accepted syntax beneath any invalid formula.
I need to create a list of forbidden Claim ID's (and, or, not, implies at least) and enforce the rules.
I should compute additional information as I parse, such as a parse tree (to be passed to the backend) or a dependencies set.
I need to shorten the claim ID and remove the author. The claim ID should be at most three letters or four numbers long. I could also follow a friend's suggestion and have the claim ID be editable, maybe using the same dropdown menu that I use for definition attachment and claim deletion.


failure is name is empty

happstack-authenticate fails correctly when provided with an empty name, but the message is strange:
Failed to decode JSON data.

The same happens with no password.

I would suggest that it fails incorrectly.  The code likely would allow for an empty username except that a different bug is preventing the JSON from being decoded.
I am working on a patch that will allow developers to provide site-specific username, password, and email policies. This JSON bug should be fixed independently.

Fixed in 86b11253551bbc904106c7e69925d5ac52056d4f
The angularjs client should now always submit valid JSON data.


Single person should revel the estimation

Hello Team,
Currently any team member can revel the estimation, it should be only to one person.
As now any one can revel complete estimation process need to start from first
Regards,

Agree, only allowing a designated person such as session creator to reveal, especially when using the offline query would be helpful.


Set process.title properly

To support pidof horizon, by default it shows in pidof node
This will save us from accidental kill when we are trying to kill another node app instead of horizon by killall node
PS. I've signed the CLA

This change is

looks good to me, thanks!


Special characters in M3U file can fail parsing

http://bit.ly/022qseed breaks the parser, It contains a special character at the beginning which makes the code in file "PlaylistParser" to fail with error "ParsingError.invalidSource".
Would be possible to treat this case in the parser? Something like
guard rawString.<remove-illegal-chars>.starts(with: filePrefix) else {
       throw ParsingError.invalidSource
 }

@cpvbruno Sorry for my late response here, could you please send a new link for the example above

Hey, sorry the URL was not mine and I don't have a new one, the case was that at the beginning of that file there were some hidden characters in the string, so the comparison of filePrefix was always returning false... I believe we can ignore this for now, these files come in some many wrongly formatted cases that its difficult to keep it up, I wish VLC provided their logic of parsing things hahaha VLC App parser any possible file you give them


Split every page in PDF?

I see that you have to pass the page number that you would like as an image, but it would be helpful if there is an option to split every page of a PDF to an image, rather than having to know the exact page number.

Hi!
Try something like this:
var pdfImage = new pdf("file.pdf");
pdfImage.getInfo().then(function (info) {
for(var i = 1; i <= info["Pages"]; i++){
pdfImage.convertPage(i).then(function (imagePath) {
fs.existsSync(i + ".png");
});
}
});

I, too, would like this cooked in, but would adding this functionality using the existing convertPage get too heavy with a Promise for each page?

Would like a convertAllPages method too!

I too would like a convertAllPages method!

I too would like a convertAllPages method!

+1

Please update to Version 2.0.0 to use this Feature:
var PDFImage = require("pdf-image").PDFImage;

var pdfImage = new PDFImage("/tmp/slide.pdf");
pdfImage.convertFile().then(function (imagePaths) {
  // [ /tmp/slide-0.png, /tmp/slide-1.png ]
});

Can anyone confirm this works? I can do my own gm convert commands in my container, but when I try to use this library, the call to convertFile() doesn't exit and I'm not seeing any extra CPU/memory activity for more than 1-2 seconds.
Maybe I'm still doing something wrong?
My working gm command
gm convert -verbose -density 150 ./test.pdf -quality 100 -type TrueColorMatte +adjoin test1/test1-%04d.png
My test code ( with 10,000 sec timeout )
lab.test('convert PDF to images', { timeout: 10000000 }, () => {
	return new Promise( (resolve, reject) => {
		const PDFImage = require('pdf-image').PDFImage;

		console.log('about to convert');
		let pdfImage = new PDFImage( pdfPath,
			{
		  		graphicsMagick: true,
				convertOptions: {
				'-density': '150',
				'-quality': '100',
			        '-depth': '8',
			        '-background': 'white',
			        '-flatten': null
				}
			}
		).convertFile()
		.then( imagePaths => {
			console.log('done converting'); // <--- THIS IS NEVER HIT
			console.dir( imagePaths )
			expect( imagePaths ).to.be.a.array();
			resolve();
		})
		.catch( err => {
			console.error( { errorConverting: err } );
			expect( err ).to.be.a.null();
		});
		console.log(' OUT! '); // This is logged immediately
	});
});


(PUP-4904) correct garbled ssh_authorized_key output

Something in the parser was choking on the long lines. Trimming them made it display properly.

CLA signed by all contributors.


wcpr crashes on Arch Linux

Traceback (most recent call last):
  File "./wpcr.py", line 94, in <module>
    symbols=wpcr(samples)
  File "./wpcr.py", line 43, in wpcr
    d = numpy.diff(b)**2
  File "/usr/lib/python2.7/site-packages/numpy/lib/function_base.py", line 1926, in diff
    return a[slice1]-a[slice2]
TypeError: numpy boolean subtract, the `-` operator, is deprecated, use the bitwise_xor, the `^` operator, or the logical_xor function instead.

Looks like there are issues within the following construct because d has not the right types for the subtraction in numpy.diff() (boolean instead of numerical).
b = a > midpoint(a)
d = numpy.diff(b)**2

Thanks! Fixed in 565dadf.


fix: landing page md properties

This fixes the landing page config properties. They were incorrect and thus not displaying properly.

should be merged

Thanks, but I have added the md file in 5cb148bab9a190431dab408d375b99843d59a7c5


Add checkpoint before additive model training so no rerun coalesce

@deerzq @saurfang

I am not sure if this will necessarily help you because transformed examples are one order of magnitude larger so they are slower to write and read in the event of failure, on the other hand checkpointing does potentially avoid shuffle the step in the event of failure. It is hard to say.
If we see SGD fails often especially due to OOM, we can look into mini-batch SGD instead where gradient is updated per epoch so it can be computed in parallel.

@saurfang Thanks for review, updated!

Please add detailed documentation, then LGTM


webfont classes implemented with 'class="cc-BTC"' (with a hyphen, rather than a space)

The latest version of webfonts defines the classes beginning with cc- (with a hyphen, rather than a space),  e.g. cc-BTC.
Icon colors seem not to work anymore.
As per the note here https://labs.allienworks.net/icons/cryptocoins/ icons should be implemented via class="cc BTC" (with a space).

Hi @dutu, thanks for info. The latest implementation of webfont generator has different config than the former one. Maintaining the webfont version is really a pain, there are many complications.. moreover the webfont generator broke again.
I'm trying to figure out a way how to solve all of those. So far, webfonts should be used with hyphen again. If you need, you can edit the CSS colors file manually (with hyphens), so it works again. Sorry for the trouble.

No problem. Thanks for your great work


Document Prospector processes

Printing Prospector Slips (Daily)
2 Step Process:
Step 1: (e-books for ILL)
FUNCTION: Notices:
20) Norlin Prospector: right click

Prepare
email printer:
change email<EMAIL_ADDRESS>to<EMAIL_ADDRESS>print
check “Do Not Clear” box
prepare
send notices.
close
close

Step 2: Prospector Slips
FUNCTION: Notices:
20) Norlin Prospector: right click

prepare
local printer: print
choose: E157 Blue Prospector Phaser 5550
print
uncheck “Do Not Clear” box
send notices
finish printing
close
save changes? No
close

Checking Out Prospector Loans (Daily):
FUNCTION: INN-Reach Check-out to Remote Site:
•	scan barcode:
•	OK
Reports: (6 different reports):
FUNCTION: INN-Reach-Reports

Report Type: Paged Too Long (run monthly)

Minimum days: 4
Maximum days: 5
Sort by: Item Location
Click Prepare

Report Type: Requested Too Long

Minimum days: 80
Maximum days: 80
Sort by: Item Location
Click Prepare

Report Type: In Transit Too Long (run monthly)

Minimum days: 21
Maximum days: 21
Sort by; Item Location
Click ‘Prepare’

Report Type: Received Too Long (run 1-2 times a month)

Minimum days: 15
Maximum days: 15
Sort by; Item Location
Click ‘Prepare’

Report Type: Returned Too Long (run monthly)

Minimum days: 21
Maximum days: 21
Sort by; Item Location
Click ‘Prepare’

Report Type: Institutional Overdues: Reconciliation process (run once a year)

Minimum days: 182
Maximum days: 912
Sort by; Item Location
Click ‘Prepare’

Returning Unwanted Items:
FUNCTION: INN Reach Return Unwanted Item:

Scan barcode:
Press OK

The correct email to change to is<EMAIL_ADDRESS>(There's a dash between CU and ILL).


Cannot runas system user in windows 10 1709

This is caused by the Win10 exploit protection/mitigation policies enabled by Process Hacker during startup... Can you check the Event log > System for a warning event with id 7039?

there is an event ID 7009 when try to run as SYSTEM
A timeout was reached (30000 milliseconds) while waiting for the ProcessHackerYOIKYBAGSGVFGCR service to connect.

@univert
You will need to import this registry key as a work-around until we receive information from MS:
Windows Registry Editor Version 5.00

[HKEY_LOCAL_MACHINE\SOFTWARE\WOW6432Node\Microsoft\Windows NT\CurrentVersion\Image File Execution Options\ProcessHacker.exe]
"MitigationOptions"=hex(b):00,10,11,00,11,01,10,01

@dmex processhacker should stop self inflicting and checking these mitigations when running as a service in PhInitializeMitigationPolicy() . Because if it is a service it is not possible to exit and create a child service  process to enable these mitigations.


adds doc describing settings DSL

Issue number:
Contributes to #203
Description of changes:
This adds meta-documentation on a DSL to describe in semi-structured TOML
Terms of contribution:
By submitting this pull request, I confirm that my contribution is made under
the terms of the licenses outlined in the LICENSE-SUMMARY file.

@stmcginnis I added a high-level overview to address the mental model gap.

See ecs.toml in #206 for a more extensive example.