Option to have glamor prepend it's style tag to the head (instead of append)

In our app we use glamor alongside webpack with CSS Modules. In dev the loading of the CSS modules is pretty random, based on the way you load components.
The head might look like this in dev:
<head>
<title></title>
<style></style>
<style data-glamor>.box { margin: 0; }</style>
<style></style>
<style>.box { margin: 10px; }</style>
<style></style>
</head>

In production all the styles are put into a rendered in a singular stylesheet and loaded before:
<head>
<title></title>
<link href='bundle.css' />
<style data-glamor>.box { margin: 0; }</style>
</head>

Now the specificity is messed up.
A potential solution to this is to have glamor prepend it's style tag to the head:
<head>
<style data-glamor></style>
<title></title>
</head>

I am still on v2, wouldn't mind making the change to v2. But not sure how soon v3 is going to be released.

Presumably, this is aggravated because of using style loader in dev vs ExtractTextPlugin in prod builds?
While I understand the potential issue, I'd like to see a concrete example if you don't mind supplying one. Is this related to combining glamor classes and cssmodules classes on the same element? I can't imagine a case in which the classnames collide (as in the example above), but perhaps you've implemented a custom CSSModules hashing algo?

This is quite the edge case, aggravated because you're using glamor to inject global styles. Not sure we can (or should) do much here.

I can explain a little bit more about our use case. We are building a ui library outside of out web apps, and importing that library in our web apps. Since these web apps have different environments and css resets, we add a reset class (instead of global styles) to all of our components in the ui library. This makes sure that a component looks identical in different environments. So far this turned out to be very powerful. This are not global styles.
This reset class contains your basic reset: margin: 0; padding: 0; border: 0; amongst other things.
In our apps we have situations in which we might want to add a className to our components and add some specific styling that is not defined in our ui library component. The issue here is strongly related to style loader in dev vs ExtractTextPlugin in prod — but potentially spans multiple advanced configurations. In our case we would like to see glamor added before everything else, so we can overwrite without adding specificity. Example:
import { Box } from 'ui-library'
import css from 'module.css'

<Box className={css.specialBox} />

In this case we might  have something like this in module.css:
.specialBox {
  margin: 20px;
  padding: 20px;
}

In dev the specialBox class might be defined after the glamor style tag. In production the ExtractTextPlugin consolidates all the css modules in one and creates a stylesheet and renders it before glamor. This means in production the reset styles might overwrite our defined styles.
in dev:

In production:

Potential ways to approach this:

Manually add a . Have glamor look for this before creating a new one.
An option for prepend instead of append. This would mean it is considered the base style in any scenario
Do something in style-loader instead

I am a big fan and advocate of glamor. I am not here to force this technology to work for our specific use case. However, I do think when people start implementing glamor in their existing apps, they have to combine it with other technologies. These are real use cases, and some of the combinations are going to be really common. I think this combination is unfortunate but might be able to be fixed with additional configuration inside of glamor. In the end I can only see this drive to wider adoption by the community if glamor understands and answers to some common use cases. I hope we can figure something out that makes glamor better for all.

Fair points. Will consider.

Just driving by and noticed this issue. Here's how I solved the problem with a plugin called increase-specificity.js :)
/*
 * This plugin for glamor will prepend #id to the beginning of each selector
 * which glamor generates. So a selector like `.css-123abc, [data-css-123abc]`
 * will be changed to: `#id .css-123abc, #id [data-css-123abc]`.
 * This gives the selector higher specificity and should make it impervious
 * to other stylesheets on the page (component styles should always win a
 * conflict war with stylesheets).
 */
export default getSpecificityPlugin

/**
 * Gets the plugin function that uses the given id
 * @param {String} id the ID to prepend selectors with
 * @return {Function} the glamor plugin function
 */
function getSpecificityPlugin(id) {
	return specificityPlugin

	function specificityPlugin({ selector, style }) {
		selector = selector
			.split(',')
			.map(sel => `#${id} ${sel.trim()}`)
			.join(', ')
		return { selector, style }
	}
}

and the usage:
import { plugins } from 'glamor'
import getSpecificityPlugin from './increase-specificity'

// if the document.body.id doesn't exist, we'll make a random one and assign it
// then use that to increase our specificity
// making it random so it can't be used for anything other than this plugin.
document.body.id = document.body.id || `random-glamor-id-${Math.random().toString().slice(2)}`
plugins.add(getSpecificityPlugin(document.body.id))

And the unit test :sunglasses:
import getSpecificityPlugin from '../increase-specificity'

test('updates selectors to have the given ID', () => {
	const myId = 'abc-easy-as-123'
	const plugin = getSpecificityPlugin(myId)
	const input = {
		selector: '.css-234kjjdf, [data-css-234kjjdf]',
		style: { margin: 20 },
	}
	const result = plugin(input)
	expect(result).toEqual({
		selector: `#${myId} .css-234kjjdf, #${myId} [data-css-234kjjdf]`,
		style: input.style,
	})
})

I hope that helps! Should probably just open source this.
I should note that this makes the specificity pretty dang serious. aphrodite solves this problem by putting !important on everything (even more heavy handed). If you want to override styles globally for one reason or another, those overrides will most likely have to include !important to work. Either that or you have a deterministic id on the body and make them use that ID as well (and fight the specificity wars).
Good luck friends!

The workaround is a good one. Another one is to manually move the style tag with a data-glamor attribute. Closing this.


Tokens not arriving in my wallet

Hi,
I sent the below transaction (see screenshots) last night as a test to the wallet address I set up in your DeFi app.
The tokens haven't arrived, which is surprising - and yes I have refreshed the balance in the wallet app.
I checked again this morning - nothing.
I then tried to reset the wallet and now the app won't even synchronise (it's been stuck on the below Block for well over an hour this morning).

I've included the logs below too, as advised on this page: https://github.com/DeFiCh/app/wiki/How-to-submit-issues-for-Defi-App . However the config log can't be sent, as the file extension is .config, which is not supported by this site.

app_log.log
binary_log.log
Please advise what to do next to resolve this issue.
Thanks!

Hi @Ausafari , thank you for your feedback, your balances should appear after synchronization is completed.
Have you tried Reindex option?
When syncronization is complete, you may then Import Wallet / Restore Wallet via mnemonic seed.
If you're not on the latest v2.1.3, you may download it here https://github.com/DeFiCh/app/releases/tag/v2.1.3.

Hi, thanks for your help.
No - I haven't tried the ReIndex option and the synchronisation has gone nowhere for the past 48hrs+. That is, it is sitting on block 474985 and has not moved. I am using the latest version of the wallet - and even deleted the app and re-installed it to see if that might help - to no avail.
So, should I try the ReIndex option before synchronisation happens - given that process has gone nowhere for the past 48hrs+?

I have gone ahead and used the ReIndex option. The good news is:
. Synchronisation completed successfully.
. I was able to restore my wallet using the mnemonic phrase.
The bad news:
. My DFI balance still shows zero, when it should show 700DFI.

@Ausafari glad to hear your synchronization is successful.
In the CLI, you can check your balance by typing:
getbalance — UTXO DFI, DFI that you receive / send to your Defi app
gettokenbalances — DFI
The sum of both are your DFI displayed in your wallet.
You may try clicking on another tab in the wallet, sometimes the app lags so the balances are not displayed for awhile.

Thanks - the balance of 700 DFI is now appearing after I clicked another tab in the wallet. Thanks for your help. Now all I need to do is work out how to undertake Liquidity Mining on the app, as I am going around and around in circles, with no luck whatsoever. :/

Hi @Ausafari , good to know! You may actually join the Telegram Channel and learn more the community, or ask questions that you might have. Here's a quick how-to on LM https://defichain.com/learn/liquidity-mining-how-to/.


Deleting a file locks up Atom (Windows 7)

I'm getting a weird locking issue when deleting a file, or sometimes folder, in windows. This only happens if I've opened the file. Files I haven't touched delete just fine.
Process:

Open Atom
Click on file in side pane to open
Click "x" to close file
Right click file
Select "Delete"
Select "Move to Trash"
Get "Editor is not responding" error

More details, these are .js and .coffee files from a node app I'm working on. So these files have also been running as a server or process at some point.
Versions:
Atom 0.120.0. Windows 7 SP1.

Just tried to repro this and didn't happen - I suspect it has been fixed at some point since this bug was filed but the bug was not closed. Please re-open if newer versions of Atom are still crashing for you.


Add gitpod support

Inspired by https://github.com/cosmos/cosmos-sdk/pull/8090
We can set this up, so after creating a new contract from template, it will have gitpod support to edit it. Ideally, the cargo generate command can be part of that, but that seems a bit more complex.
The final result would be a user could click a button or two to open up a web IDE with a contract skeleton for their newly created contract (with their custom name). This should somehow be connected to a new github repo under their username. When they make the changes on gitpod, they get saved to their github repo.

I am working on this. Makes setting up environment for Dev Academy much simpler

Projects based on cosmwasm-template will have gitpod by default when after this feature is enabled. No extra setup required to make it work with cargo generate

I just approved it. Happy for you to try it out.
Do you need to authorise GitHub after each cargo generate?
I just added cosmwasm-template to gitpod

Authorization is needs to be done once on newly generated repo by owner.

i noticed some exceptions ex:
invalid character { in package name: {{project-name}}, the first character must be a Unicode XID start character (most letters or _)
During initial stand up. Maybe we could change that in the future?
For the todo could we link them to the extension and a screen shot of the button to press?

Besides that I think we're good here.
This solves the issue of specific systems not supporting ARM and reduces friction of developing (less steps to set up dev environment).

@PotasticPotatoe gitpod runs cargo build on launch to cache libraries on the launch. Since cosmwasm-template is not a rust project, this is expected. You are supposed to use gitpod on projects generated using cosmwasm-template repo.

ack, closing this ticket if no other issues

Great!


README

README
User Story
AS A bootcamp student
I WANT a codebase that follows accessibility standards
SO THAT the marketing agency site is optimized for search engines
Acceptance Criteria
GIVEN the challenge repository
WHEN I view the README for the repository,
THEN Repository contains quality README file with description, screenshot, and link to deployed application.

Repository contains quality README file with description, screenshot, and link to deployed application.


🤖 - How to close a custom dialog by outside click?

Description
How to close a custom dialog by outside click?
How to redo this example (https://taiga-ui.dev/dialog/custom) so that the dialog close when you click on overlay layer?

Logic for outside click and Escape button is isolated in TuiDialogCloseService. You need to provide it in your custom dialog, subscribe to it and close dialog upon emission. I have updated the example on the page you linked.


Try not to save BlockContext

Current checkpoint/rollback saves and restores three pieces of info:

States of arithmetic encoder
CDFs
All of the block contexts in a frame! (Wow.. We didn't mean this...)

So, let's try whether our RDO mode decision (esp. with future
extensions) can survide w/o saving 3).

Right, during RDO search, each mode will overwrite to those left/above contexts for coefficients and partition. The encoder works but I think mode decision is wrong due to overwritten left/above contexts.

Also because the affected parts of BlockContext are partition and skip, and we don't search those yet, but it'll break really soon once we add those.


Packet header changes for -11

Getting the packet header decoding working was a little difficult.  In the end I opted to partially teach your codec about tls:"varint" so that I could encode the packet number (the alternative was to encode it as two octets always, which was tempting, but would have made decoding hard still).  I'm not happy with the way that I am decoding the short packet header, but it's the best I could come up with on short notice.  If you have a better idea, I'd really like to hear it.
I updated the per-version salt, bumped the packet number, changed the calculation of per-packet overheads (it makes more sense to do that with a variable-length connection ID).
It might make sense to have configuration for the length of the connection ID, but I hard-coded that to 8 for now.  You don't have any means of configuring a connection/server, so that seemed hard enough that I wanted to punt on it.
There are three connection ID fields in Connection now, the initial is saved separate to client/server CIDs.  Using those simplified things a bit, so I refactored some code around sending packets, especially version negotiation, which had a special function that I just inlined since VN was the only user.  Also, in using those new fields, I noticed that there was an entirely unused function, which is now gone.

This is https://phabricator.services.mozilla.com/D1274

You may want to fix this conflict.


Repo sync for protected CLA branch

The pull request is created from master637393315074540000 to master to fix git push error for protected CLA branch

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


Add explicit subnet_cidr variable to Terraform

The subnet_cidr is one of the most important variables and it should be explicitly configurable.

LGTM


fix: 🐛 Fetch missing block.hash from middleware queries

Added hash property within block to be fetched from middleware queries.
(While standardizing input and output data from middleware, block hash
was added to output params wherever block number was returned.
But appently, the middleware queries were not updated to fetch the same.)

:tada: This PR is included in version 14.0.0-alpha.11 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 14.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 14.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


jsdoc deprecated tag is too aggressive

TypeScript Version:  4.0.0-dev.20200701
Search Terms: jsdoc deprecated tag
Code
interface ESMap<K, V> { }

/** @deprecated */
interface Map<T> extends ESMap<string, T> {}

declare const Map: new <K, V>() => ESMap<K, V>;

new Map<string, number>(); // 'Map' is crossed out

Expected behavior:

Map the value is not deprecated and should not be crossed out.

Actual behavior:

Map the value is considered deprecated, even though Map the type is not part of the value declaration of Map:

In VSCode:

In Playground:

Playground Link: https://www.typescriptlang.org/play/index.html?ts=4.0.0-dev.20200707#code/KYDwDg9gTgLgBAbzgXwNwCh0EsB2NhQBmAhgMbBwCiAygLLFgA8A0gDRwBqAfIipgPQAqQXAACAE2BgowUsXzi4g-tjwES5OPSYAVHqHw5xAZyp0GjYzCi4A5uz2JkmSaQA2xGXFIQcVrQwAXHA4wADucCzs3AAUAJRwALw8NNpRnFwYmOihEWlWNjj2IQCuALYARgRc8ahw-PxwAOTaTXBYpj5+WJIyipLSsvLAijHGwBQAjiVYpADW7TiEEHFAA
Related Issues: #38523

Probably look into getSemanticMeaning, then add some special cases. See if that is useful.

Fixed by #39323


Create RFC status tracking document

This tracks status of all unimplemented RFCs in one central place. Hopefully we won't forget to update this when new RFCs are added!
Rendered link

Seems like a candidate for GitHub projects?

I'm not sure how well this fits - we do have a PR for most RFCs but they are all merged, I'm unsure if projects can track pending work. We can also create issues for unimplemented RFCs but that means we may have issues that are open for months...
We can explore projects separately for this though.

Projects can be connected to PR's, but they don't have to be.
It's worh looking into, as they can track pending work.

It looks like there's two projects feature in GitHub: a new beta and the old feature. I'm going to merge this for now, once the new beta becomes the only official project feature we'll explore this in more detail.


Migrate to Go Modules

Migrate dependencies from glide to Go Modules and adjust wercker.yml.

@mrkuc Looks good. Thanks :relaxed:


GET calls to /api/messages fail with HTTP error 405 (Method Not Allowed)

I just got this setup and have seen consistent failures so far.  I modified some of the branding (via params) but otherwise haven't customized the code.  I can hit the base URL directly and get the placeholder "Visit Icebreaker  to learn more and set one up for your own team!" so I know the site is up and reachable.  Manually triggering the AppLogic does successfully send out pairings.

@winterminute Please share error log details from application Insights.


Add "ToS" and "Privacy Policy" links to Settings Screen.

When creating a wallet use can read terms and privacy policy.
After wallet is created there is no way to open it again. I think there should be links somewhere in wallet allowing opening it.

Important task after the new fork since we will definitely have new URLs.


QOL Suggestion: Open prefix folder in right click menu

It would be nice to have an option on right click menu that opens file manager at prefix location.

Thanks for the suggestion.
I just added the option.
Please update Fagus Launcher.


[Feature-66], draft version of flow implementation

A draft version of creating a flow definition and executing the flow using a Directed Graph. It has to be strictly a DAG for this to work.

@benas Finally I was able to get some working code on the table, Let me know your thoughts.

Coverage decreased (-8.2%) to 73.522% when pulling 6505f3e7908e57dbd2108c711e063795ed55b4e0 on gs-spadmanabhan:feature-66 into e38b828a3b32e5b8d420845da166742df7cef199 on EasyBatch:feature-66.

Coverage decreased (-8.2%) to 73.429% when pulling 932c5d7b8cc3d4acc79395a608bf829b12beae8e on gs-spadmanabhan:feature-66 into e38b828a3b32e5b8d420845da166742df7cef199 on EasyBatch:feature-66.


Add progress and error reporting API

Summary

Closes https://github.com/openvinotoolkit/datumaro/issues/142

Added progress and error reporting in Dataset.import_from and Dataset.export (backward compatible)
Added an ability to skip or fail on import and export problems with specific images or annotations
Updated COCO, VOC and YOLO formats
Now, Extractor ctor has only kw args. Subclasses must define their own c-tor, which accepts 1 positional and any kwargs. Unused kwargs must be passed to the base class.

How to test

Checklist

[ ] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

@IRDonch, please check the PR.


Fix HTML block for version 9

This Pull Request addresses issue: #8833
Requires an update of the Bedrock repository (https://github.com/concrete5/bedrock/pull/94).
We have updated Ace library to the latest version and tested if there are some breaking changes.

Thanks, this looks great.


Features - Add global dark theme with switch.

I assigned the issue to you , you can work on it now

I want to work on this issue under JWOC


Add compressing and decompressing modes

https://github.com/ros2/domain_bridge/commit/5007b226b2b3c1cf80644d3d950f78e5dfa6ff6e fixes a bug.
https://github.com/ros2/domain_bridge/commit/90a4fc950050a0acd49dc5f0513476ff25a67bd1 implements the compressing and decompressing modes.
There are a lot of things that could be improved but this is an okay first prototype.
e.g.:
# decompressing_bridge.yaml
name: decompressing_bridge
from_domain: 3
to_domain: 2
mode: decompressing
topics:
  chatter:
    type: example_interfaces/msg/String
    qos:
      reliability: best_effort

# compressing_bridge.yaml
name: compressing_bridge
from_domain: 0
to_domain: 3
mode: compressing
topics:
  chatter:
    type: example_interfaces/msg/String
    qos:
      reliability: best_effort

In one terminal:
ros2 run domain_bridge domain_bridge decompressing_bridge.yaml &
ros2 run domain_bridge domain_bridge compressing_bridge.yaml &
ros2 topic pub /chatter example_interfaces/msg/String "{data: asd}"

In another terminal:
ROS_DOMAIN_ID=2 ros2 topic echo /chatter

FYI @jacobperron

Just as a thought, it is probably worthwhile to look at the pluggable compression that rosbag2 has.  It's probably overkill for right now, but it might a) allow you to have pluggable compression, and b) share some code with rosbag2.

Just as a thought, it is probably worthwhile to look at the pluggable compression that rosbag2 has. It's probably overkill for right now, but it might a) allow you to have pluggable compression, and b) share some code with rosbag2.

Yes, I actually copy-pasted part of the code from there.
I thought about using their compression plugins, but I found it easier to use zstd directly here for a first prototype.

I tried this with the pc_pipe demo, and the compression rate is pretty good.
Results using a 640x480 rgb image:

A 921600B image was compressed to 187077B (~1/5).
A 4915200B pointcloud to 297651B (~1/16).

That's a really promising bandwidth reduction.

I think that the bridge mode (compressing/decompressing/normal) could be specified per topic and not globally, once we merge this I'll work on improving that.

@jacobperron friendly ping, I think we should at least merge the first commit https://github.com/ros2/domain_bridge/commit/5007b226b2b3c1cf80644d3d950f78e5dfa6ff6e.

@ivanpauno Submitting the bug fix as a separate PR would be nice, then we can merge that independently.

Done, see https://github.com/ros2/domain_bridge/pull/27.

There's linter errors on Rolling

Sorry, I forgot to check that. Fixed in https://github.com/ros2/domain_bridge/pull/24/commits/ee83cad0159d4dd9082c4d95d56753db3287c735, https://github.com/ros2/domain_bridge/pull/24/commits/940dca330a9ce87f1aa001929086d5515486cb8a.

It looks like there's an rclcpp API being used that's not available in Galactic. Preferable we could workaround this so we don't need to branch, but it's not a blocker.

Fixed in https://github.com/ros2/domain_bridge/pull/24/commits/ffef015e0c02105dea8395bd89b25d65278c8ada, https://github.com/ros2/domain_bridge/pull/24/commits/53b0628b7bfe82a81edca6efae5c1a283f253e1b.

Tests for the compression feature (to prevent regression)

I have refactored some of the existing tests in https://github.com/ros2/domain_bridge/pull/24/commits/7ce726eb37abdd7cb60eb156046fdfe799d703d1, to deduplicate code.
New tests were added in https://github.com/ros2/domain_bridge/pull/24/commits/6a438f37ff165eedf84dd9540f8bad17cf00fa52.

I think that we should make the compression mode per topic (as you suggest), since it's at the topic-level where this compression is happening.

I think that we can do that in a follow-up PR, IIUC the current plan was to compress of the traffic.

The Gpr branch is failing because main isn't compatible with galactic anymore (see https://github.com/ros2/domain_bridge/pull/27).

The Gpr branch is failing because main isn't compatible with galactic anymore (see #27).

I misunderstood the nature of #27. Is it possible we can keep the main branch compatible with Galactic if we factor out the generic pub/sub code (#28)? @rebecca-butler is taking a look at doing this right now.

I misunderstood the nature of #27. Is it possible we can keep the main branch compatible with Galactic if we factor out the generic pub/sub code (#28)?

I think so.
There will be probably some conflicts with this PR when we introduce the rclcpp generic pub/sub, as the API isn't exactly the same to the one here.
@jacobperron should we merge this as-is to avoid conflicts or do you prefer to first reintroduce compatibility with the galactic branch?

https://github.com/ros2/domain_bridge/pull/29 reintroduces compatibility with galactic, I think we can merge that one first and then go ahead with this one.

Now that #30 is merged. I'm hopeful that rebasing on main will resolve Galactic compatibility issues.

Now that #30 is merged. I'm hopeful that rebasing on main will resolve Galactic compatibility issues.

Rebased.
The rebase wasn't as easy as I would've wanted, because either:

The API create a GenericPublisher isn't flexible enough.
PublisherBase doesn't have a method to publish serialized messages.

Either thing would have made this patch easier.


Supports multiple chart updates without time delays

In the version that used the old Access web browser control, it was possible to refresh a chart multiple times without experiencing any time delays.
Since switching to Access's new Edge browser control, an empty HTML file must be loaded before each update, resulting in a delay of approximately 3 seconds.
Supports multiple chart updates without time delays

This line of code in the html file seems to be the culprit:
const autocolors = window['chartjs-plugin-autocolors'];

Released with version 3.27.02


Improve GAN

Here is a new thread for discussing how to proceed with the GAN experiments. For the first results see #29
Current status:

Working on precip-to-precip TIGGE to MRMS
Reproduced GAN fairly closely from Leinonen paper. For differences, see below.
Output is GAN-like (i.e. generative) and discriminator seems to do what it should but doesn't look realistic yet

Goal:

Create precip-to-precip GAN that produces realistic-looking output

Comments/observations:

Open question as to whether our forecast-to-observation setup changes things a lot. Therefore, @annavaughan's suggestion: Work with pure super-resolution setup until we have something realistic looking.
Data preprocessing seems to matter A LOT. Specifically, the weighted sampling and the log-transform. First job for me is to check again exactly whether everything is correct and make sure we pick a setup that does what we think it should! #45
There are some differences to the original Leinonen paper and to the Oxford/ECMWF work from what I tested

I was using spectral norm in G and D. Leinonen only in D. Oxford not at all.
I pretrained G using MSE for one epoch. Leinonen does not do this. Not sure about Oxford.
I added a L1 loss for the G. BUT because the G loss was so crazy, it shouldn't have any impact.
Leinonen uses L2 regularization for G. I am not doing this, not sure about Oxford.
Architecture is slightly different. Compared to Leinonen, I am concatenating only a single noise channel. We and Oxford are not using the two-path structure of the discriminator.
We are upscaling the low-res input and concatenating to the high-res image at the start of D. Oxford and Leinonen use a two-path structure before concatenating.
We are not using any output activation in G, Oxford use ReLU, Leinonen uses sigmoid.

Next steps:

[ ] Test best setup so far with pure SR setup
[ ] Test sensitivity of GAN results (visually at first) to several of the options above

@annavaughan @HirtM I will focus on this over the next week. Feel free to also run experiments. If you do, could you maybe just quickly announce this in this thread, so that we make sure we are not doing duplicate work. I have been experimenting in notebooks in the Experiments folder. Feel free to copy this setup.

First hint: Not using the log transform makes the GAN not work. This suggests that data preprocessing is super important.

I am training our current setup with the pure super-resolution setup on my branch stephans_gan as a notebook 07. Will update later on how it went.

Really weird: Using the pure SR option with the same setup that produced somewhat reasonable results before does not work.

This emphasizes that I need to check the log-transform/preprocessing.

@raspstephan I'm going to look at this now. Is the current best version the one in notebook 07?

Hi Anna, here a quick update. 07 contains the close-to-Leinonen setup with the pure super-resolution approach. THIS DOES NOT WORK!? 08 contains the same setup with regular TIGGE to MRMS, which "works". I just tested this again in 09 and 10 with the new weighted sampling (not yet fully commited because still running). SAME outcome. TIGGE to MRMS "works", pure SR does not. I have no idea why. Maybe we could have a quick meeting tomorrow to look at this together?

That would be great to meet tomorrow, I'm free between 12pm-2:30pm Munich time if sometime then works? I'm very confused why the pure SR doesn't work - I'm looking at the code now and it seems fine

Great, here a summary of my very confusing findings. BTW @annavaughan, I am working in the Experiments subdirectory of notebooks.
TIGGE-->MRMS works sort of, also with the new sampling method. (09 notebook). The loss implodes.

Pure SR (so MRMS coare-->MRMS fine) with the same setup does NOT work (10 notebook)

Testing a different discriminator architecture with two heads more similar to Leinonen produces a really weird discriminatory loss of almost exactly 10(!?). This is cause by the gradient penalty. No idea why.

We need to figure out why the pure SR behaves so differently from the TIGGE--> MRMs setup. This makes no sense to me at the moment...

@annavaughan Update on my experiments with lower learning rate. Started out encouraging but then wasn't so great after all.
First up, I ran the TIGGE-MRMS setup that has previously "worked" with 1e-4, now with 1e-5. Notebook 09. In the end the results look pretty similar to the original learning rate. Maybe a little less filamenty but still not realistic enough.

Then I ran the pure super-resolution setup with 1e-5 (notebook 10). Unfortunately, I didn't save the training logs. But after like 8 epochs the losses again exploded and the images became increasingly unrealistic.

This makes me wonder whether the LR was really at fault after all. Maybe it just delayed the inevitable... So no solution yet. How did you get on with your Leinonen setup? Happy to talk again tomorrow to debug.

@annavaughan So my MNIST test was a failure. I guess this tells us at least that it's not the data (which we already suspected) But then WHAT THE HELL IS IT!?!?!? It must be something in the networks or training that we both are doing wrong?? Well that is, if you MNIST experiments also turn out to fail. Keep me posted :)


Withdraw form waits for x confirmations but it only needs 1

In cases were funds are being sent to Vega, there is a network param for that defines how many eth confirmations wait before these funds can be used.
there is a pattern to wait this long and keep the user in a pending state until we've seen the 50th confirmation.
This logic is being applied to withdrawals where it shouldn't.
it shouldn't be applied to disassociation either (but I have not tested to see if it is)

The bugfix has been tested for withdrawals and we no longer have any waiting confirmation counters for withdrawal transactions and user lands directly on Incomplete withdrawals page.

@campbellssource - The current implementation for withdrawals is -  There's an interaction with Metamask and post the confirmation we wait for one block of confirmation. Let's raise a new ticket if we want to change the current implementation


add example select game dialog with two games

loopy doesn't really work, but just using it for testing

as i added the Pager stuff, i realized it isn't as friendly. i added a "selected" signal to this pager subclass, but would like to make an easier Pager to use in general in rmkit.
on that note: i noticed there's a few TODOs and upgrades to rmkit that you mention in the code: is there stuff i can upstream / add that you want? i like the vertical centering on buttons, for example.

as i added the Pager stuff, i realized it isn't as friendly. i added a "selected" signal to this pager subclass, but would like to make an easier Pager to use in general in rmkit.
on that note: i noticed there's a few TODOs and upgrades to rmkit that you mention in the code: is there stuff i can upstream / add that you want? i like the vertical centering on buttons, for example.

misc problems:

if dismissing pager, game doesn't refresh - the canvas (or whatever is container for the game) should re-render the game when render() is called. maybe it needs to be set to dirty?
when starting new game of loops then going back to lightup, there is weird redrawing issue - need to fully clear screen before swapping games

misc problems:

if dismissing pager, game doesn't refresh - the canvas (or whatever is container for the game) should re-render the game when render() is called. maybe it needs to be set to dirty?
when starting new game of loops then going back to lightup, there is weird redrawing issue - need to fully clear screen before swapping games

Awesome! I'll take a look at how this works later today. I was initially thinking about have an intro scene that would double as the picker (there's an android version that works like that), but I haven't thought it all through yet. I noticed that Pager is a subclass of Dialog (which I assume is meant to be used as an overlay?) I could see Pager being useful as a standalone Widget as well, but I haven't even gotten that far so I might not understand how it works yet :)

on that note: i noticed there's a few TODOs and upgrades to rmkit that you mention in the code: is there stuff i can upstream / add that you want? i like the vertical centering on buttons, for example.

Oh yeah, mostly just keeping notes as I run into things. I was planning on sending over some PRs, but I could start w/ issues if that would be faster/easier for you. So far the biggest thing is vertically centering text. I'm doing that with OutlineButton and the puzzles api needs to be able to draw centered text as well (e.g. lightup centers the text in squares). I'm not 100% sure my implementation of the latter (here) is correct -- in fact it looks like the text is a little lower than it should be.

Awesome! I'll take a look at how this works later today. I was initially thinking about have an intro scene that would double as the picker (there's an android version that works like that), but I haven't thought it all through yet. I noticed that Pager is a subclass of Dialog (which I assume is meant to be used as an overlay?) I could see Pager being useful as a standalone Widget as well, but I haven't even gotten that far so I might not understand how it works yet :)

on that note: i noticed there's a few TODOs and upgrades to rmkit that you mention in the code: is there stuff i can upstream / add that you want? i like the vertical centering on buttons, for example.

Oh yeah, mostly just keeping notes as I run into things. I was planning on sending over some PRs, but I could start w/ issues if that would be faster/easier for you. So far the biggest thing is vertically centering text. I'm doing that with OutlineButton and the puzzles api needs to be able to draw centered text as well (e.g. lightup centers the text in squares). I'm not 100% sure my implementation of the latter (here) is correct -- in fact it looks like the text is a little lower than it should be.

if dismissing pager, game doesn't refresh - the canvas (or whatever is container for the game) should re-render the game when render() is called. maybe it needs to be set to dirty?

That makes sense -- the puzzles api is supposed to keep track of the dirty region and call draw_update() when it has a region to refresh, according to the docs. I've assumed that drawing to the framebuffer is kind of expensive, but if it isn't Canvas::render() could just copy the full vfb each time.

if dismissing pager, game doesn't refresh - the canvas (or whatever is container for the game) should re-render the game when render() is called. maybe it needs to be set to dirty?

That makes sense -- the puzzles api is supposed to keep track of the dirty region and call draw_update() when it has a region to refresh, according to the docs. I've assumed that drawing to the framebuffer is kind of expensive, but if it isn't Canvas::render() could just copy the full vfb each time.

Awesome! I'll take a look at how this works later today. I was initially thinking about have an intro scene that would double as the picker (there's an android version that works like that), but I haven't thought it all through yet. I noticed that Pager is a subclass of Dialog (which I assume is meant to be used as an overlay?) I could see Pager being useful as a standalone Widget as well, but I haven't even gotten that far so I might not understand how it works yet :)

yeah, IPager is only used as a modal overlay at the moment. i think making a new scene with the game selection makes more sense, though. the pager could be used for setting game params (i was reading docs and i think games support preset parameters with names or something similar?), but its also possible to make a dropdown (or dropup).
i'll abandon this diff, since i like the intro scene better

Awesome! I'll take a look at how this works later today. I was initially thinking about have an intro scene that would double as the picker (there's an android version that works like that), but I haven't thought it all through yet. I noticed that Pager is a subclass of Dialog (which I assume is meant to be used as an overlay?) I could see Pager being useful as a standalone Widget as well, but I haven't even gotten that far so I might not understand how it works yet :)

yeah, IPager is only used as a modal overlay at the moment. i think making a new scene with the game selection makes more sense, though. the pager could be used for setting game params (i was reading docs and i think games support preset parameters with names or something similar?), but its also possible to make a dropdown (or dropup).
i'll abandon this diff, since i like the intro scene better

That makes sense -- the puzzles api is supposed to keep track of the dirty region and call draw_update() when it has a region to refresh, according to the docs. I've assumed that drawing to the framebuffer is kind of expensive, but if it isn't Canvas::render() could just copy the full vfb each time.

the render() is called when a widget is marked as dirty. a widget is usually marked as dirty if ui::MainLoop::refresh() is called (marks all widgets dirty) or the widget is manually marked dirty. the canvas will not have render() called on it often (only when changing scenes and hiding ' showing overlays, i think), but can add a logging line and check.

That makes sense -- the puzzles api is supposed to keep track of the dirty region and call draw_update() when it has a region to refresh, according to the docs. I've assumed that drawing to the framebuffer is kind of expensive, but if it isn't Canvas::render() could just copy the full vfb each time.

the render() is called when a widget is marked as dirty. a widget is usually marked as dirty if ui::MainLoop::refresh() is called (marks all widgets dirty) or the widget is manually marked dirty. the canvas will not have render() called on it often (only when changing scenes and hiding ' showing overlays, i think), but can add a logging line and check.


(wip) close #79

Description
All major changes were done in src/plonk/proof_system/commitment_scheme.rs.

Implement the tests outlined in #79, namely test_kate_open and kate_batch_open.
Fix the locks on line 370

No more progress can be made until transcripts are implemented.
Link to issue
#79
Type of change

[x] Bug fix (non-breaking change that fixes an issue)

Test plan (required)
Tests implemented:

[ ] test_kate_open
[ ] kate_batch_open

The latter of the two tests covers the change in batch_open pertaining to the locks.
Miscellaneous
Per the suggestion, the write locks in batch_open were bypassed by arcs and unsafe code. Arc cloning is cheap and enables concurrent writes to consecutive memory addresses:
(0..input_key.small_domain.size)
    .into_par_iter()
    .for_each(|i| unsafe {
        let mut opening_poly = (opening_poly.clone().coefficients.as_ptr() as *mut Fr).offset(i as isize);
        let mut shifted_opening_poly = (shifted_opening_poly.clone().coefficients.as_ptr() as *mut Fr).offset(i as isize);
        *opening_poly = input_key.quotient_polynomial_parts[0].read().unwrap()[i];
        for &(ref poly, challenge) in &opened_polynomials_at_zeta {
            *opening_poly += (*poly).read().unwrap()[i] * challenge;
        }
        *shifted_opening_poly = Fr::zero();
        for &(ref poly, challenge) in &opened_polynomials_at_zeta_omega {
            *shifted_opening_poly += (*poly).read().unwrap()[i] * challenge;
        }
    });

fantastic so far- is the test_kate_open test working yet?

Oops. I saw the 'not yet implemented' and assumed it was just the transcripts. I'll get that done in the meantime.

yep lmk as things continue :) pat is making good progress over on transcript stuff, i think you may well be able to get the batch test implemented (but obviously not passing) before he's done there.

@bmzig ready for review?

@bmzig ready for review?

Yes.


Fix Phosphor asset path resolution

This now uses Node's own module path resolution as a means to locate @phosphor-icons/core/assets both in development of this plugin, and on the end-user's machine.
It also updates the build scripts so that they will work on Windows :)

Tested this with a separate project using npm link, and it works well.


open() does not allow binary IO

Reading files opened by open("rb") returns a TextIOWrapper instead of something that allows binary IO. The emulator returns an instance of FileIO which seems to work just fine.
with open("somefile", "rb") as f:
    print(type(f)) # TextIOWrapper, expected be FileIO (or BufferedReader according to the Python documentation)

As a consequence, f.read() returns a str instead of bytes.

hmm.. it should return FileIO.. diving into it..

probably fixed in pull-request #166

@polyfloyd can you verify if current master solves this issue for you?

@annejan Yeah, it has.


update installation recommendations

closes #9623

[x] add instructions about updating envs
[x] add "pro tips" like using mamba, using different envs for each project, keeping a list of other packages you install (to make it easier to recreate envs), etc.

@drammock I have a few colleagues who are interested in getting started with MNE but havent installed it yet. They are scientists but not tech focused. If you wish, I could get them to follow the install instructions and give us user feedback, but it would happen sometime in the next week (not in the next day or two).

If you wish, I could get them to follow the install instructions and give us user feedback, but it would happen sometime in the next week (not in the next day or two).

Great! Let's get this merged in and backported after I address your other comment (tomorrow, it's late here). Then your colleagues' feedback can be addressed in a second PR

@rob-luke @cbrnr @hoechenberger this one is ready for final review/merge.

PR is awesome

Then your colleagues' feedback can be addressed in a second PR

Will do, ill try not to interfere while I watch them :)

Don't ever try Julia because that's how package management should be in Python 😉 .

or npm. Python packaging is really the worst, it was maybe okay-ish 10 years ago but these days, it's almost unacceptably bad.

Thanks @drammock !

backported

Thx @drammock
regarding comments on packaging. Npm is easier as it’s only JavaScript not python, c++, OpenGL / vtk etc. Regarding Julia yes maybe but I want to see something flying with the complexity of mne dependencies. You want to help packaging -> reduce dependencies and keep complexity low ! Packaging across platforms is just damned hard !

reduce dependencies and keep complexity low

Yes, that's true no matter what programming language.


[BUG/Help] There appear to be 1 leaked semaphore objects

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
mac上20g空闲内存， 用.bfloat16()加载模型，报如下错误，是内存爆了吗？
UserWarning: resource_tracker: There appear to be 1 leaked semaphore objects to clean up at shutdown
warnings.warn('resource_tracker: There appear to be %d '
Expected Behavior
No response
Steps To Reproduce
run 'python cli_demo.py'
Loading checkpoint shards:   0%|                                                                    | 0/8 [00:00<?, ?it/s][1]    11244 segmentation fault  python cli_demo.py
/Users/anaconda3/lib/python3.10/multiprocessing/resource_tracker.py:224: UserWarning: resource_tracker: There appear to be 1 leaked semaphore objects to clean up at shutdown
warnings.warn('resource_tracker: There appear to be %d '
Environment
- OS: MacOS
- Python: 3.10.9
- Transformers: 4.27.1
- PyTorch:
- CUDA Support (`python -c "import torch; print(torch.cuda.is_available())"`) : False

Anything else?
No response

Did you solve it?

Please use mps backend
https://github.com/THUDM/ChatGLM-6B#mac-上的-gpu-加速

遇到同样问题，请问解决了吗


Fix "failure rate" bugs

Fixes msan build break in Travis.
Fixes enable-crypto-mdebug build break.

@kaduk you pinged me off-line, please take a look.

I restarted the Appveyor build, it seems it failed because github was inaccessible

Can you explain what the bug was? Why would msan care?

The bug was that it was making malloc "fail" too often, and most places weren't checking for malloc returning NULL.

@ekasper In particular the "roll > md_percent" vs "roll < md_percent" comparison.

Where does this run on Travis and why would it affect msan? This stuff is behind enable-crypto-mdebug, isn't it?

Where does this run on Travis and why would it affect msan? This stuff is behind enable-crypto-mdebug, isn't it?

I was wondering about that...  I think the msan stuff is something else...

Ah, @ekasper you're right.  I forgot my build script has enable-crypto-mdebug always set....

thanks!


MSID: 89596 Version: 1 DOI: 2<IP_ADDRESS>6468

"msas": "Cell Biology"
"msid": "89596"
"preprintDoi": "10.1101/2<IP_ADDRESS>6468"
"Reviewed Preprint posted": "2023-08-08"
"Sent for peer review": "2023-06-03"
"Posted to bioRxiv": "2023-06-26" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>6468v1/")
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=89596)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/89596/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>6468)
[x] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/89596/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/89596)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/89596

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/89596

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/89596/v1/89596-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/89596)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Figure supplement titles displaying inconsistently (larger size compared to main figures); elocation IDs missing from references


Adding Impression Pixel

Peter and co want to be able to see distribution of viewership in the context of all this Track 2 work we're doing.
Adding an impression pixel is the only way currently that github can't curtail.
@Azure/azure-sdk-eng

Can one of the admins verify this patch?

Generally I'm fine with this - but a few observations:

I'm not a fan of having it add extra lines at the bottom of the page. When the readme redesign is published I'd be inclined to move this into a different location further up the page (i.e. at the end of the 'badge graphics' row for things like build status, etc).

You mentioned track two, but the files you've updated are either not part of our track one or track two, or not part of our track two. Are you also planning to track our track one directories?

Going forward, do I need to add this requirement to our checklist for importing other track one SDK components into our repo readme files? Or will you be intermittently applying this?

Generally I'm fine with this - but a few observations:

I'm not a fan of having it add extra lines at the bottom of the page. When the readme redesign is published I'd be inclined to move this into a different location further up the page (i.e. at the end of the 'badge graphics' row for things like build status, etc).

I'm ok with this. It is inserted by regex, so once I have a "hook" I can place it wherever we want.

You mentioned track two, but the files you've updated are either not part of our track one or track two, or not part of our track two. Are you also planning to track our track one directories?

I applied the pixel to any readme. I'm just attempting to get every shred of data I can. I only mentioned track 2 for context as to why I'm shotgunning a hundred readme changes :) .

Going forward, do I need to add this requirement to our checklist for importing other track one SDK components into our repo readme files? Or will you be intermittently applying this?

I will be A) intermittently applying this or B) automating the run of this tool. Depending on how valuable this data turns out, I will invest additional efforts to add commits to PRs that move or insert a readme, etc. I don't personally feel like this should be something that library writers need to worry about.


feat(api): Add .meta endpoints

Endpoints:

Located under /api/v1/.meta
Requires authenticated users
version returns the hubuum version number (text/plain(
runtimes returns versions of some core components (json)
debug returns versions of all modules and the settings used (minus the SECRET_KEY)

Changes:

This current patch forces version tags to be on the form vX.Y.Z (with the v being required).

TODO:

Timestamp for last change is not implemented.
Not tested with tags.

Fixes #148

coverage: 100.0%. remained the same when pulling d5ca6a75a4d3b401faa825e2a33cde068991bc90 on terjekv/Version-modelling-and-reporting into 3d19aa06a64b83b712b85c7ecfb1bc119a88bc6f on main.


Roll engine 5088735e5f54..e73dbf6a6f99 (8 commits)

https://github.com/flutter/engine/compare/5088735e5f54...e73dbf6a6f99
git log 5088735e5f54f7fb34215b6665cb4da1fd8f075d..e73dbf6a6f991332315e20cd712314750051d885 --no-merges --oneline
e73dbf6a6 Roll src/third_party/skia 389b4b267d15..7eba20aef187 (2 commits) (flutter/engine#8209)
bc8ebcb96 Export FlutterSemanticsUpdateNotification and improve docs (flutter/engine#8203)
b404c69fd [scenic][SCN-1054] Move back off of SetTranslationRH (flutter/engine#8174)
71e2d7d8c Roll src/third_party/skia 4a522b7e916c..389b4b267d15 (32 commits) (flutter/engine#8205)
ad5b722a7 Simplify the fallback waiter and add traces for vsync scheduling overhead. (flutter/engine#8185)
6980a8356 [platform_view] iOSP platformView composition optimize. (flutter/engine#8202)
568841923 Disable build_ios due to large queue times. (flutter/engine#8204)
629072277 Merge only gpu and platform threads for platform views, fix deadlock. (flutter/engine#8045)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jsimmons@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5698023542751232 , https://cirrus-ci.com/task/6120236007817216 , https://cirrus-ci.com/task/4994336100974592 , https://cirrus-ci.com/task/6683185961238528 , https://cirrus-ci.com/task/5557286054395904


flutter_acrylic.Window.hideWindowControls() not support on linux

ubuntu 22.04
when call this function
it show the error:
MissingPluginException (MissingPluginException(No implementation found for method HideWindowControls on channel com.alexmercerind/flutter_acrylic))

The same hideWindowControls() does not work on Windows 10. There are no rounded corners either.


Enter create unwanted space

Hello
after I press enter, it creates unwanted space - how to avoid that?
With other words, instead of
this is
new line
I need something like this:
this is
new line.
Thank you.

You need to press Shift+Enter to insert a <br> instead of creating a new paragraph.


Add function path_result which returns Result instead of Option.

This preserves the error stack.

@segfaultDelirium Apologies that I didn't see this when you originally  posted. Are you still interested in the PR? Please rebase if so, thanks.


use manually build flutter mapbox package

use new version of the mapbox flutter plugin to support ios

This is now not needed anymore since version 5 was released!


Editable install of package failed to update importlib.metadata.version

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
pyproject.toml: https://github.com/data-apis/array-api-extra/blob/main/pyproject.toml
Issue description
Sorry for the lack of reproducible example here, this is a bit convoluted. I asked this on Discord and @ruben-arts directed me to open an issue here.
I was (rebase) pulling the latest 14 commits from https://github.com/data-apis/array-api-extra/commits/main/ into another branch. In the process there was a merge conflict with the lockfile so I deleted it to complete the rebase. After rebasing, I used pixi update to regenerate the lockfile.
As part of the rebase, the version of my package updated from '0.1.dev0' to '0.1.2.dev0'. This version is updated dynamically.
After the rebase and update, pixi ls correctly showed the new version of the package:
Environment: dev
Package                        Version     Build                 Size       Kind   Source
...
array_api_extra                0.1.2.dev0                                   pypi   . (editable)

However, importlib.metadata.version failed to update:
# running ipython in the same env
In [1]: import importlib

In [2]: importlib.metadata.version("array_api_extra")
Out[2]: '0.1.dev0'

pixi update failed to resolve the issue. However:
pixi clean
pixi update
pixi shell -e dev

fixed the issue.
Expected behavior
It would be nice if pixi update alone fixed the issue and loaded the updated version in IPython.

I'm not sure this is a bug. importlib.metadata.version reads the .dist-info metadata of a package, which is only written when a build frontend (e.g., uv or pip) invokes a build backend to produce a wheel. Which is typically expensive and not what you want for an already-installed editable install.
It looks like what is happening here is:

For pixi update, dependencies are updated but uv isn't invoked to build an editable install from scratch.
For pixi clean && pixi update && pixi shell -e dev the clean step actually removes .dist-info (because it's part of the environment), so pixi update probably detects that the project is no longer installed as editable and then has to do a full reinstall.

I suspect (1) is by design, and everything is working as intended. Also note that for editable installs with say pip in virtual env, the same happens - if you update the version of your package, that won't be reflected in what importlib.metadata.version returns until you explicitly re-run pip install . -e.

Thanks Ralf. I was about to close this issue, although maybe some flag to pixi update to also update editable installs could be considered? If I am the only person who has hit this, though, probably fine to just close.

This in this conversation we're discussing the --force-reinstall (name tbd) which will happen. So I'll close this in favor of that. Thanks @rgommers!


Set up Mailgun on Jenkins

@AndreasAntener @dagar Can we set up Mailgun (https://www.mailgun.com) on our Jenkins instance? That should resolve the broken email sending and would be quite enabling.

Done.
I'm not sure how noisy jenkins is by default, but I guess we'll see.

Not quite, need to add a proper subdomain to use it this way. @LorenzMeier I'll email you the details.

Done!


Unable to build openssl-qat-engine container image: requires a shell more modern

I'm using this Docker file https://github.com/intel/intel-device-plugins-for-kubernetes/blob/main/demo/openssl-qat-engine/Dockerfile
docker build --rm -t openssl-qat-engine .
Step 9/22 : RUN sed -i -e 's/cmn_ko$//' -e 's/lac_kernel$//' quickassist/Makefile &&     KERNEL_SOURCE_ROOT=/tmp ./configure &&     make quickassist-all adf-ctl-all &&     install -m 755 build/libqat_s.so /usr/lib/ &&     install -m 755 build/libusdm_drv_s.so /usr/lib/ &&     install -m 755 build/adf_ctl /usr/bin/ &&     cd /ipp-crypto/sources/ippcp/crypto_mb &&     cmake . -B"../build"     -DOPENSSL_INCLUDE_DIR=/usr/include/openssl     -DOPENSSL_LIBRARIES=/usr/lib64     -DOPENSSL_ROOT_DIR=/usr/bin/openssl &&     cd ../build &&     make crypto_mb && make install &&     cd /intel-ipsec-mb &&     make && make install LIB_INSTALL_DIR=/usr/lib64
 ---> Running in 0ee11cbdbe59
./configure: This script requires a shell more modern than all
./configure: the shells that I found on your system.
./configure: Please tell<EMAIL_ADDRESS>and
./configure<EMAIL_ADDRESS>about your system, including any
./configure: error possibly output before this message. Then install
./configure: a modern shell, or manually run the script under such a
./configure: shell if you do have one.
The command '/bin/sh -c sed -i -e 's/cmn_ko$//' -e 's/lac_kernel$//' quickassist/Makefile &&     KERNEL_SOURCE_ROOT=/tmp ./configure &&     make quickassist-all adf-ctl-all &&     install -m 755 build/libqat_s.so /usr/lib/ &&     install -m 755 build/libusdm_drv_s.so /usr/lib/ &&     install -m 755 build/adf_ctl /usr/bin/ &&     cd /ipp-crypto/sources/ippcp/crypto_mb &&     cmake . -B"../build"     -DOPENSSL_INCLUDE_DIR=/usr/include/openssl     -DOPENSSL_LIBRARIES=/usr/lib64     -DOPENSSL_ROOT_DIR=/usr/bin/openssl &&     cd ../build &&     make crypto_mb && make install &&     cd /intel-ipsec-mb &&     make && make install LIB_INSTALL_DIR=/usr/lib64' returned a non-zero code: 1

cc @mythi @eadamsintel

@devimc hmm. I remember seeing something similar but cannot recall what is was. FWIW, we build this image part of our CI on Ubuntu 18. and 20.04 so it should be OK.

@devimc I just tried it on an Ubuntu 20.04 system and it built fine for me. I tried twice once with a 3 month old clearlinux:base image I had and once with the newer one 3 days ago and no issue.   I am using Docker 20.10.7 on an Ubuntu 20.04.2 system with a 5.4.0-81 kernel.
There might be something interesting about your setup that would be worth knowing.  If you have time can you try running a base clearlinux docker image and then going through each of the steps manually until you get to step 9 and see if you can reproduce the same issue?   An alternative and maybe faster would be to set an environment variable in the Dockerfile  ARG CONFIG_SHELL=/bin/bash and try again.

Fixed - I have updated my system (ubuntu 18.04) and rebooted it, thanks


[zh-cn] updated /overview/working-with-objects/kubernetes-objects.md

Updated some typos and formats:
content/zh-cn/docs/concepts/overview/working-with-objects/kubernetes-objects.md

Preview: https://deploy-preview-35469--kubernetes-io-main-staging.netlify.app/zh-cn/docs/concepts/overview/working-with-objects/kubernetes-objects/

/lgtm
/approve


multiple set fitting doesn't capture the fitting parameters correctly on Linux system

Dear Xraylarch community,
I'm using Larch to fit multiple datasets at the same time. It works well on my windows 11 computer. However, it doesn't work correctly on my linux computer(Ubuntu 22.04.3 LTS).
I implemented the fitting by using the Cu example in Larch example folder as an example. Instead of using the same parameters for all datasets, I assigned different variables for Einstein temperature, denoted theta1, theta2, and theta3, and set them to vary, as shown in the picture below.

However, I got the result only theta3 is in the vary mod, but the program automatically to turn theta1 and theta2 to the fixed mod. This doesn't happen on my Windows computer. Here is the report:

In addition, I attemped to add the fix_unused_variables=False option in feffit function.

Although a warning was prompted to say "Feffit Warning: unused variables: theta1, theta2", the fitting is running successfully without misunderstanding parameters:

I'm really curious what happens in the codes. Is any different setting for Linux and Windows? In the attachment, I send all the files I used for the test as a compressed file.
multi_fitting.zip
Best regards,
Kaifeng Zheng

@kaifengZheng Thanks for the report and sorry for the trouble.  I can confirm that I see that too.  I think I have a fix, but want to run some more tests.  I hope to get a fix post later today or tomorrow.

Thank you very much, Matt!


search function and stopwords

It seems that the dataset-viewer search function returns no hits if one searches for terms such as “what”, “can”, “which”, and so on. Has the indexing function removed stopwords like this? The rows are returned if one uses the SQL console, but the returned rows in the SQL console don’t give access to the column with audio, for a dataset that includes audio files. Is there a way to search for stop words like this in the default datasets Viewer? It would be really useful if all of the textual content in a column could be searchable.

Has the indexing function removed stopwords like this?

Yes, we use a default list of stopwords, which contains 571 words, including "what," "can," and "which." You can view the complete list here:
DuckDB English Stopwords List.
But, as @severo  mentioned in this discussion, we now support language-specific stemmers for monolingual datasets. Using a default English stopwords list for all languages no longer makes sense. However, DuckDB currently lacks a straightforward way to assign stopwords based on language as it does for stemmers (we would need to seed a stopwords table for non-English datasets). Therefore, for now, the best approach is to set the stopwords parameter to 'none'. cc. @lhoestq
If users want to remove stopwords for specific monolingual datasets (e.g., English), this could be a candidate for a custom configuration at the dataset card level. Keep in mind that removing stopwords like "what," "can," or "which" helps focus on more meaningful terms, improving search relevance. It also reduces the size of the search index and speeds up queries, which is crucial for performance in the Datasets Viewer.

@AndreaFrancis thanks for the reply. It is true that content words are typically more meaningful than are prepositions, articles, and so on, but it is very much the case that a researcher may be interested in filtering a dataset for function words or, especially, collocations containing function words. The way the plugin works at the moment, a search string like "never experience" is equivalent to "experience".
In addition, and as you note, the current configuration actually removes words in datasets in other languages if they are homographs with English stopwords. So, for example Swedish "by" (village) is unsearchable in a Swedish-language dataset with the current viewer parameters.
I understand that it affects speed, but I think it would be great if no stopwords are used and/or custom configurations could be made available.

The rows are returned if one uses the SQL console, but the returned rows in the SQL console don’t give access to the column with audio, for a dataset that includes audio files.

Hi @stcoats we just shipped Audio / Image support in SQL Console Results. I am not sure which dataset you are trying to view / query, but it should work now 🤗 I know it doesn't fix the stemming, but hopefully can still help! Let me know if you have any questions


Non application path and configurable/brandable UI

This draft PR (waiting on 1.11.0.Final) updates the paths to be under the new non-application path.
It also changed the UI to work like other UI (that can be themed) and the path is now configurable.
Fix #23
Fix #17

We can now merge this as 1.11.0.Final is out. Another PR to update the UI a bit is on the way (waiting on 1.11.1.Final) to align with the Dev Console and other UIs. See https://github.com/quarkusio/quarkus/pull/14300

We can now merge this as 1.11.0.Final is out. Another PR to update the UI a bit is on the way (waiting on 1.11.1.Final) to align with the Dev Console and other UIs. See https://github.com/quarkusio/quarkus/pull/14300


Get updated contacts from date

How to get updated contacts list  ? I know full sync but is there a way for getting only updated OR deleted contacts from a particular date ? please help.
One option is full sync and compare changekey but I have some 800 contacts and its going to be time taking oen if I do full sync every time when user logged in.

If you're using the SyncFolderItems operation, you can store your SyncState to keep track of where you are in the sync. If you get the SyncState from your operation, store it, the next time you use it you're only fetching changes since the last sync.

Oh ok I got it Thank you so much Sir. I am storing syncstate.

Hi,
I have one more query. I am using exchange server 2007 and if I want to
know deleted calendar events then in that case I see only one option that
is full sync every time and comparing missing event ids with my database
table. Is there any other way to find deleted items so that I can delete
them from my database.
Regards,
Nagesh
On Fri, Apr 22, 2016 at 3:48 AM, Garethp<EMAIL_ADDRESS>wrote:

If you're using the SyncFolderItems operation, you can store your
SyncState to keep track of where you are in the sync. If you get the
SyncState from your operation, store it, the next time you use it you're
only fetching changes since the last sync.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-213137362

Why do you think you have to do a full sync every time? According to the MSDN docs on SyncFolderItems, you can pass a sync state in Exchange 2007. When you do this, you don't receive all of the items as they exist at that point, you receive the events. So you should get DeleteItem events as well, and that'll tell you when a calendar item was deleted

yes I tried that as well but it doesn't show deleted item. that is the
reason I asked you. it only shows created items and updated items.
I deleted one event from calendar but it is not showing. yes I am using
sync state.
On Sun, May 8, 2016 at 1:10 AM, Garethp<EMAIL_ADDRESS>wrote:

Why do you think you have to do a full sync every time? According to the MSDN
docs on SyncFolderItems
https://msdn.microsoft.com/en-us/library/office/aa563967(v=exchg.150).aspx,
you can pass a sync state in Exchange 2007. When you do this, you don't
receive all of the items as they exist at that point, you receive the
events.
Reply to this email directly or view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-217664426

When you do your sync state where the delete item is meant to show up, can do you a dump on the response you receive and past it to a gist or something? If you could do a dump earlier in the NTLM to get it in the raw XML, before it becomes objects, that would be best.
As a side note, are you using this jamesaires library, or did you check out my one? If you swap to my one, I might be able to give more in depth help. And is there a specific reason you're using Exchange 2007?

Hi,
I tried a lot and after a few try I started getting deleted item as well. I
was using old sync state and because of that I wasn't getting deleted item.
Now I am getting it thanks a lot for your HELP and quick response. I will
send new queries to you if any.
Regards,
Nagesh
On Sun, May 8, 2016 at 1:59 PM, Garethp<EMAIL_ADDRESS>wrote:

When you do your sync state where the delete item is meant to show up, can
do you a dump on the response you receive and past it to a gist or
something? If you could do a dump earlier in the NTLM to get it in the raw
XML, before it becomes objects, that would be best.
As a side note, are you using this jamesaires library, or did you check
out my one? If you swap to my one, I might be able to give more in depth
help. And is there a specific reason you're using Exchange 2007?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-217703457

Hi,
Auto discover is not working for me. I don't know why but here is the error
message.
Warning: dns_get_record(): Dns Query failed in
C:\wamp\www\exchange\php-ews-master\EWSAutodiscover.php on line 499
Warning: Invalid argument supplied for foreach() in
C:\wamp\www\exchange\php-ews-master\EWSAutodiscover.php on line 361
Regards,
Nagesh
On Mon, May 9, 2016 at 10:53 AM, Nagesh Sawant<EMAIL_ADDRESS>wrote:

Hi,
I tried a lot and after a few try I started getting deleted item as well.
I was using old sync state and because of that I wasn't getting deleted
item. Now I am getting it thanks a lot for your HELP and quick response. I
will send new queries to you if any.
Regards,
Nagesh
On Sun, May 8, 2016 at 1:59 PM, Garethp<EMAIL_ADDRESS>wrote:

When you do your sync state where the delete item is meant to show up,
can do you a dump on the response you receive and past it to a gist or
something? If you could do a dump earlier in the NTLM to get it in the raw
XML, before it becomes objects, that would be best.
As a side note, are you using this jamesaires library, or did you check
out my one? If you swap to my one, I might be able to give more in depth
help. And is there a specific reason you're using Exchange 2007?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-217703457

I would suggest you not use Autodiscover. You should already know the host and version you're working with

yes but my client wants autodiscover so that user dont have to remember his
server name. Please suggest.
On Thu, May 12, 2016 at 2:02 AM, Garethp<EMAIL_ADDRESS>wrote:

I would suggest you not use Autodiscover. You should already know the host
and version you're working with
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-218581175

I tried first example from this url but error
https://github.com/jamesiarmes/php-ews/wiki/Autodiscovery
On Thu, May 12, 2016 at 2:35 AM, Nagesh Sawant<EMAIL_ADDRESS>wrote:

yes but my client wants autodiscover so that user dont have to remember
his server name. Please suggest.
On Thu, May 12, 2016 at 2:02 AM, Garethp<EMAIL_ADDRESS>wrote:

I would suggest you not use Autodiscover. You should already know the
host and version you're working with
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-218581175

Hi,
if I use $server = 'outlook.office365.com'; or $server = 'outlook.com'; it
works well for me but I have question
will it work for all exchange server versions? or should I use first one
for autodiscover.
Regards,
Nagesh
On Thu, May 12, 2016 at 2:39 AM, Nagesh Sawant<EMAIL_ADDRESS>wrote:

I tried first example from this url but error
https://github.com/jamesiarmes/php-ews/wiki/Autodiscovery
On Thu, May 12, 2016 at 2:35 AM, Nagesh Sawant<EMAIL_ADDRESS>wrote:

yes but my client wants autodiscover so that user dont have to remember
his server name. Please suggest.
On Thu, May 12, 2016 at 2:02 AM, Garethp<EMAIL_ADDRESS>wrote:

I would suggest you not use Autodiscover. You should already know the
host and version you're working with
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-218581175

I suggest not using Autodiscover, because it can take three or four requests before finding the server. Or it could make 5 requests and not find it at all. It can slow down the application a lot of you're doing Autodiscovers all the time.
Also, that won't work if your exchange server isn't hosted online. If it's a locally hosted server, by a different name, then that won't work

Autodiscover will get used only once after that based on response I will
store server name in database. But the problem is its not working for me.
On Thu, May 12, 2016 at 1:46 PM, Garethp<EMAIL_ADDRESS>wrote:

I suggest not using Autodiscover, because it can take three or four
requests before finding the server. Or it could make 5 requests and not
find it at all. It can slow down the application a lot of you're doing
Autodiscovers all the time.
Also, that won't work if your exchange server isn't hosted online. If it's
a locally hosted server, by a different name, then that won't work
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-218690082

What's the email address that you're trying to use to autodiscover?

Hi,
Its working on live server but not on wamp server.
Regards,
Nagesh
On Thu, May 12, 2016 at 3:18 PM, Garethp<EMAIL_ADDRESS>wrote:

What's the email address that you're trying to use to autodiscover?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-218710608

What's the email address that you're trying to use to autodiscover?

I asked client he said please do not share email address this is clients
email address.
On Thu, May 12, 2016 at 7:26 PM, Garethp<EMAIL_ADDRESS>wrote:

What's the email address that you're trying to use to autodiscover?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-218764776

Well, can you post it with just the domain? I can't do much to help you, but I can do even less if I don't know what domain you're trying to query

Hi,
How to split threaded email messages in PHP EWS. I am not getting any
solution online.
For example. I want to read only single message at a time and not entire
thread.
Regards,
Nagesh
On Fri, May 13, 2016 at 2:10 PM, Garethp<EMAIL_ADDRESS>wrote:

Well, can you post it with just the domain? I can't do much to help you,
but I can do even less if I don't know what domain you're trying to query
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-218984676

Hi,
Are you there please reply to my email. I am confused with exchange email
thread.
Regards,
Nagesh
On Wed, Jun 1, 2016 at 7:55 PM, Nagesh Sawant<EMAIL_ADDRESS>wrote:

Hi,
How to split threaded email messages in PHP EWS. I am not getting any
solution online.
For example. I want to read only single message at a time and not entire
thread.
Regards,
Nagesh
On Fri, May 13, 2016 at 2:10 PM, Garethp<EMAIL_ADDRESS>wrote:

Well, can you post it with just the domain? I can't do much to help you,
but I can do even less if I don't know what domain you're trying to query
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-218984676

The messages are already split. You should only be getting one at a time

oh is it. but when there is conversation between 2 users and when they
exchange emails with same subject line message id remains same. let me me
paste example here. Now in the following email message there are 2 text
body which I want separate for inserting it in mysql with sender email id
etc.
*1. good to see this message. *
2. this is the test by exchange mail id

good to see this message.
On Wed, Jun 1, 2016 at 6:57 PM, Nagesh Sawant<EMAIL_ADDRESS>wrote:
this is the test by exchange mail id
On Thu, Jun 2, 2016 at 9:32 PM, Garethp<EMAIL_ADDRESS>wrote:

The messages are already split. You should only be getting one at a time
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-223338315,
or mute the thread
https://github.com/notifications/unsubscribe/AFRKSWaueoCQfCfe1YwFge3BGE6fz2EJks5qHv57gaJpZM4ILJHJ
.

when I pass message id it is showing body text as
["Body"]=>
object(stdClass)#17 (2) {
["_"]=>
string(438) "good to see this message.Â On Wed, Jun 1, 2016 at 6:57 PM,
Nagesh Sawant<EMAIL_ADDRESS>wrote:this is the test by exchange mail id

"
["BodyType"]=>
string(4) "HTML"
}
On Fri, Jun 3, 2016 at 12:02 AM, Nagesh Sawant<EMAIL_ADDRESS>wrote:

oh is it. but when there is conversation between 2 users and when they
exchange emails with same subject line message id remains same. let me me
paste example here. Now in the following email message there are 2 text
body which I want separate for inserting it in mysql with sender email id
etc.
*1. good to see this message. *
2. this is the test by exchange mail id

good to see this message.
On Wed, Jun 1, 2016 at 6:57 PM, Nagesh Sawant<EMAIL_ADDRESS>wrote:
this is the test by exchange mail id

On Thu, Jun 2, 2016 at 9:32 PM, Garethp<EMAIL_ADDRESS>wrote:

The messages are already split. You should only be getting one at a time
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-223338315,
or mute the thread
https://github.com/notifications/unsubscribe/AFRKSWaueoCQfCfe1YwFge3BGE6fz2EJks5qHv57gaJpZM4ILJHJ
.

Yeah, you'll have to come up with a way to do that yourself

syncfolderitem will return only 512 results what if there are more than 512
results? will it next time return other 512 results or is there any other
work around paging type of.
$request = new EWSType_SyncFolderItemsType;
$request->SyncState = $sync_state;
$request->MaxChangesReturned = 512;
On Fri, Jun 3, 2016 at 12:50 AM, Garethp<EMAIL_ADDRESS>wrote:

Yeah, you'll have to come up with a way to do that yourself
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-223394427,
or mute the thread
https://github.com/notifications/unsubscribe/AFRKSe5mvbIWqEHxD1WWzaE9C0c9z8PQks5qHyzsgaJpZM4ILJHJ
.

It will return the next 512 items from the SyncState that you passed in. So it's automatically paged

ok Thank you so much.
On Sat, Jun 4, 2016 at 2:41 AM, Garethp<EMAIL_ADDRESS>wrote:

It will return the next 512 items from the SyncState that you passed
in. So it's automatically paged
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-223695698,
or mute the thread
https://github.com/notifications/unsubscribe/AFRKSbE2l2qaeWcbXy_Yx9TSqzXnpYdQks5qIJibgaJpZM4ILJHJ
.

Hi,
Autodiscover on live server works fine but there is is error for contact
get list or for sync folder request. What can be the issue.
Fatal error: Uncaught SoapFault exception: [soap:Client] The specified
server version is invalid. in
in /home/public_html/application/libraries/EWS/ExchangeWebServices.php on
line 391
Regards,
Nagesh
On Sat, Jun 4, 2016 at 6:19 PM, Nagesh Sawant<EMAIL_ADDRESS>wrote:

ok Thank you so much.
On Sat, Jun 4, 2016 at 2:41 AM, Garethp<EMAIL_ADDRESS>wrote:

It will return the next 512 items from the SyncState that you passed
in. So it's automatically paged
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-223695698,
or mute the thread
https://github.com/notifications/unsubscribe/AFRKSbE2l2qaeWcbXy_Yx9TSqzXnpYdQks5qIJibgaJpZM4ILJHJ
.

The issue could be that you specified the wrong server version for your server or feature?

Hi,
I am using email inbox sync but its getting timeout because there are too
many emails are there in the inbox and I am using while loop.
//IncludesLastItemInRange
while($lastItemInRange != 1) {
what to do in this case please suggest.
Regards,
Nagesh
On Sun, Jun 12, 2016 at 8:09 PM, Garethp<EMAIL_ADDRESS>wrote:

The issue could be that you specified the wrong server version for your
server or feature?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jamesiarmes/php-ews/issues/326#issuecomment-225438570,
or mute the thread
https://github.com/notifications/unsubscribe/AFRKSbWWpkauCYeGeAmlyQbqOM3gWhlKks5qLBo7gaJpZM4ILJHJ
.

Don't use a loop. If you're storing and re-using the sync state, then you can sync between 100-1000 items per script. Meaning instead of Running it in a loop, you can just schedule the script to run once a minute, or once every 5 minutes, and then it'll catch up on it's own

Hi,
Do you know which is the best solution for implementing video chat in
website.
Regards,
Nagesh

Hi,
I have one question. I am using EWS authentication but how to store user
password. Oauth2 is only available in exchange 365 server.
What is the option for storing passord in database.
Regards,
Nagesh
On Thu, Jul 7, 2016 at 3:56 PM, Nagesh Sawant<EMAIL_ADDRESS>wrote:

Hi,
Do you know which is the best solution for implementing video chat in
website.
Regards,
Nagesh

Hi,
When I use login details of exchange server 2010 it gives me error for eg.
my autodiscover doesnt work. contact list page gives error like this
Fatal error: Class 'EWS_Exception' not found in
C:\wamp\www\exchange\php-ews-master\ExchangeWebServices.php on line 1221
what can be the issue My curl version is also > 7.23
On Wed, Sep 14, 2016 at 8:03 PM, Nagesh Sawant<EMAIL_ADDRESS>wrote:

Hi,
I have one question. I am using EWS authentication but how to store user
password. Oauth2 is only available in exchange 365 server.
What is the option for storing passord in database.
Regards,
Nagesh
On Thu, Jul 7, 2016 at 3:56 PM, Nagesh Sawant<EMAIL_ADDRESS>wrote:

Hi,
Do you know which is the best solution for implementing video chat in
website.
Regards,
Nagesh

Hi,
When I use login details of exchange server 2010 it gives me error for eg.
my autodiscover doesnt work. contact list page gives error like this
Fatal error: Class 'EWS_Exception' not found in
C:\wamp\www\exchange\php-ews-master\ExchangeWebServices.php on line 1221
what can be the issue My curl version is also > 7.23

Fatal error: Uncaught exception 'EWS_Exception' with message 'SOAP client
returned status of 401' in
C:\wamp\www\exchange\php-ews-master\ExchangeWebServices.php:1221 Stack
trace: #0 C:\wamp\www\exchange\php-ews-master\ExchangeWebServices.php(547):
ExchangeWebServices->processResponse(NULL) #1
C:\wamp\www\exchange\contact_list.php(58):
ExchangeWebServices->FindItem(Object(EWSType_FindItemType)) #2 {main}
thrown in C:\wamp\www\exchange\php-ews-master\ExchangeWebServices.php on
line 1221
On Mon, Sep 26, 2016 at 11:19 PM, Nagesh Sawant<EMAIL_ADDRESS>wrote:

Hi,
When I use login details of exchange server 2010 it gives me error for eg.
my autodiscover doesnt work. contact list page gives error like this
Fatal error: Class 'EWS_Exception' not found in
C:\wamp\www\exchange\php-ews-master\ExchangeWebServices.php on line
1221
what can be the issue My curl version is also > 7.23

for exchange 2010 I still get error please provide some solution..
Fatal error: Uncaught exception 'EWS_Exception' with message 'SOAP client
returned status of 401' in
C:\wamp\www\exchange\php-ews-master\ExchangeWebServices.php:1221 Stack
trace: #0
C:\wamp\www\exchange\php-ews-master\ExchangeWebServices.php(1064):
ExchangeWebServices->processResponse(NULL) #1
C:\wamp\www\exchange\contact_sync.php(63):
ExchangeWebServices->SyncFolderItems(Object(EWSType_SyncFolderItemsType))
#2 {main} thrown in
C:\wamp\www\exchange\php-ews-master\ExchangeWebServices.php on line 1221
On Mon, Sep 26, 2016 at 11:19 PM, Nagesh Sawant<EMAIL_ADDRESS>wrote:

Hi,
When I use login details of exchange server 2010 it gives me error for eg.
my autodiscover doesnt work. contact list page gives error like this
Fatal error: Class 'EWS_Exception' not found in
C:\wamp\www\exchange\php-ews-master\ExchangeWebServices.php on line
1221
what can be the issue My curl version is also > 7.23

https://github.com/Garethp/php-ews/issues/65
On Mon, Sep 26, 2016 at 11:19 PM, Nagesh Sawant<EMAIL_ADDRESS>wrote:

Hi,
When I use login details of exchange server 2010 it gives me error for eg.
my autodiscover doesnt work. contact list page gives error like this
Fatal error: Class 'EWS_Exception' not found in
C:\wamp\www\exchange\php-ews-master\ExchangeWebServices.php on line
1221
what can be the issue My curl version is also > 7.23

@nagesh314 I'm having trouble following this thread as there seem to be multiple issues that you're looking for support with. As it seems your original issue has been resolved, I am closing this issue. If you have further issues that you need support with, please create a new issue for each individually.


Small perf improvement for visible cell related methods

Changes in this pull request
This is a minor optimization for a few list adapter methods to return early if we know there are no currently visible cells.
Checklist

[ ] All tests pass. Demo project builds and runs.
[ ] I added tests, an experiment, or detailed why my change isn't tested.
[ ] I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
[ ] I have reviewed the contributing guide

This is a pretty simple change, but am wondering if any concerns with this before I go through the checklist.

@manicakes what's the latest plan w/ this? Still working on it or should we close?

@rnystrom I'm going to do some research on this and if data supports that this improves perf, I'll put this back up. Will close it for now to reduce noise.