chore(ngMock): add they helper for testing multiple specs

This doesn't work yet with inject when additional modules are used and new services are injected in.

Nevermind. This stuff works great. Please merge.

I believe that Jasmine sets this when calling the spec function so you should too to be consistent

I think this will make it harder to move to Jasmine 2.x, as in Jasmine 2.x, the function takes an optional done parameter

we can handle the done parameter in the wrapper --- but who cares about jasmine 2.x anyways?

also note: commit message is wrong, this is not an ngMock change

we can handle the done parameter in the wrapper --- but who cares about
jasmine 2.x anyways?

Angular/ngMock is also used by developers to test their modules, not only
to test angular itself

For Jasmine 2, the spec functions can take an optional done fn parameter, yes?
If we were to migrate to supporing Jasmine 2 (which is not trivial for ngMock) then we would define the spec function for they blocks would take an optional second parameter:
describe( ... ) {
  it('should do something async', function(done) {
    ...
    done();
  };

  var vals = [ ... ];
  they('should do something async', function(vals, function(val, done) {
    ...
    done();
  });

Is there anything else we are missing here?  If not then we should merge this once the other comments are addressed.

@petebacondarwin yes, and ok


Are system requirements up to date?

The requirements for Teams for Windows seem way off. It lists 3 GB of RAM as the required amount (i.e. you should never need more than 3 GB) yet regularly I see Teams use way more than that.

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

ID: 29a61ff5-b3a5-158d-149a-dec344060e05
Version Independent ID: 97fac351-a283-aaa8-cde1-fbd9cf4e8504
Content: Hardware requirements for Microsoft Teams - Microsoft Teams
Content Source: Teams/hardware-requirements-for-the-teams-app.md
Service: msteams
GitHub Login: @SerdarSoysal
Microsoft Alias: serdars

@asklar Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@asklar Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Sample of how to verify requirements

Includes

Demonstrates how to verify requirements with subsample on dataset
papermill script to verify all datasets
More version control instructions (still ongoing)

@alhandwerger - I made some of the improvements we talked about:

Saving more files locally for inspection in QGIS
Fixing some of the intermediate cell outputs (including the requirements)
Ignoring mac specific hidden files
General cleanup

Know you wanted to add some colorbars with labels too. Feel free to push them. We can review together.

@alhandwerger - I like the colorbars and other updates. Look great.
I changed the kernel in the notebook back to dswx_val. Hopefully that can be done when running it locally for you.
I also was using an older version of nbformat and there are lots of issues with regenerating id within each cells. This is a headache.
There are two options:

(Selected) nbdiff --ignore-id for local changes
(Possible) Ensure nbformat_minor is 4 as here and then resave.

2 does work. I just don't know how to ensure that except manually going into the json. I updated the readme for the option 1.


Docker image on docker hub ?

Hey @tuffnerdstuff ,
Now the project is pretty and use by several people, could you add a tiny github action maybe to build and push this cool project on docker hub ? 🙏🏻
I would be happy to simplify my stack by just using an external image 😃

Good idea! I'll look into it. It was not important for me, yet, as my test, dev and production server are the same 😁

I added an action for that. Images will now be pushed to https://hub.docker.com/r/tuffnerdstuff/hauk-snitch. Let me know if that works for you!

will try asap, thanks you !

It works well 😊


Missing status "incoming"

https://github.com/team-telnyx/call-control-ivr-example/blob/e22d84928d0a7349dd21b7632929a92979e0b87a/app/services/call_control/actions/gather_digits.rb#L12
Maybe "initiated" or "answered"?

Never mind. Resolved.


Missing swift- prefix from SDKs

I struggled a bit getting carton sdk install to work.
My first guess was to try carton sdk install 5.8, which didn't work. I then tried a tag name, for example: carton sdk install swift-wasm-5.9-SNAPSHOT-2023-08-01-a but that didn't work either.
I took me a while to figure out that I need to drop the swift- prefix from the tags. Is there a reason this is required, or is it a bug?

The version specifier used by carton sdk install follows the .swift-version file conversion used in swiftenv. This design was intentional, but I understand it's not the ideal situation. We can consider better diagnostics.


Rat In A Maze

Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
yes
Description
(Write your answer here.)
This problem is solved using backtracking.
Rat starts from 1st position of the grid and we have to find the path to reach at the end.
eg.
INPUT-
5
1 0 1 0 1
1 1 1 1 1
0 1 0 1 0
1 0 0 1 1
1 1 1 0 1
OUTPUT -
1 0 0 0 0
1  1 1  1 0
0 0 0 1 0
0 0 0 1 1
0 0 0 0 1
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)
#4369

@HarshCasper @siddharth25pandey @ankitaggarwal23
Pls review


Add DesktopNames to hyprland.desktop

Describe your PR, what does it fix/add?
Adds DesktopNames with canonical Hyprland name to hyprland.desktop. DMs or SMs would use this to set $XDG_CURRENT_DESKTOP.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
No.
Is it ready for merging, or does it need work?
Yes.

Related to this and #7083, do I understand correctly that currently Hyrpland unsets  XDG_CURRENT_DESKTOP and does not consider existing value? That might not be a good idea.

Related to this and https://github.com/hyprwm/Hyprland/issues/7083, do I understand correctly that currently Hyrpland unsets XDG_CURRENT_DESKTOP and does not consider existing value? That might not be a good idea.

No, that's on exit. On launch it overwrites it

sure, though Hyprland will set the env itself

Compositor can be a part of some desktop environment, which can be launched by a DM or SM, which can set XDG_CURRENT_DESKTOP according to DesktopNames from entry. This is why existing value should be left as is if no explicit env = XDG_CURRENT_DESKTOP,... statement exists in config.

sooo... we should only set it iff it's unset/empty?

Yes.

feel free to make a MR

#7347

#8110 resolved it


Extend Emacs client to be an entire alternative to cc-mode

cc-mode uses a lot of semantic analysis to do syntax highlighting and indentation etc. But using semantic highlighting, keyword highlighting, and a few simple pieces of regex, that could probably all be replaced. It would also require clang-format support, which @danielmartin is working on.
cc-mode is also very slow to update, for example it still doesnt indent correctly after noexcept in constructor definitions.
Semantic highlighting should be the hardest part, and that already works, so i think this is feasable.

clang-format and syntax highlighting exist now


wandb in DETR Pytorch StopIteration

I am using the code from detr github https://github.com/facebookresearch/detr
When I use wandb I get StopIteration error for no reason whatsoever (It accuses the backbone for not working).
I set up the whole wandb init, config, watch and log.
By only removing the: wandb.watch(model_without_ddp) it works normally.
So my conclusion is that wandb.watch might be changing something inside the model? This is so weird.
Could that be because it actually sends the model to another code for training? (from main.py to engine.py -> train_one_epoch() )
If you would like to replicate the error, it would be too complicated (clone github+download coco dataset).

number of params: 41302880
loading annotations into memory...
Done (t=2.72s)
creating index...
index created!
loading annotations into memory...
Done (t=0.34s)
creating index...
index created!
Start training
Traceback (most recent call last):
  File "main.py", line 280, in <module>
    main(args)
  File "main.py", line 228, in main
    args.clip_max_norm)
  File "/home/projet21/PAr104/depr/depr-alpha/engine.py", line 32, in train_one_epoch
    outputs = model(samples)
  File "/home/projet21/anaconda3/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/projet21/PAr104/depr/depr-alpha/models/detr.py", line 62, in forward
    features, pos = self.backbone(samples)
  File "/home/projet21/anaconda3/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/projet21/PAr104/depr/depr-alpha/models/backbone.py", line 101, in forward
    xs = self[0](tensor_list)
  File "/home/projet21/anaconda3/lib/python3.7/site-packages/torch/nn/modules/module.py", line 738, in _call_impl
    var = next((v for v in var.values() if isinstance(v, torch.Tensor)))
StopIteration

wandb: Waiting for W&B process to finish, PID 19485
wandb: Program failed with code 1.  Press ctrl-c to abort syncing.
wandb:
wandb: Find user logs for this run at: /home/projet21/PAr104/depr/depr-alpha/wandb/run-20210131_072740-16af2or9/logs/debug.log
wandb: Find internal logs for this run at: /home/projet21/PAr104/depr/depr-alpha/wandb/run-20210131_072740-16af2or9/logs/debug-internal.log
wandb: Synced 5 W&B file(s), 0 media file(s), 0 artifact file(s) and 0 other file(s)

Hey @lucastononrodrigues thanks for the ticket
We will be looking into this.

In the past year we've majorly reworked the CLI and UI for Weights & Biases. We're closing issues older than 6 months. Please comment to reopen.


build: bump version to v0.1.7

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.11%. Comparing base (4923d28) to head (0f923c2).

:exclamation:  There is a different number of reports uploaded between BASE (4923d28) and HEAD (0f923c2). Click for more details.
HEAD has 4 uploads less than BASE

Flag
BASE (4923d28)
HEAD (0f923c2)

5
1

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #149       +/-   ##
===========================================
- Coverage   99.80%   89.11%   -10.69%
===========================================
  Files          20        6       -14
  Lines        2101      818     -1283
===========================================
- Hits         2097      729     -1368
- Misses          4       63       +59
- Partials        0       26       +26

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


Cannot update BREAD

Laravel Version: 5.4.*
Voyager Version: 1.0.6
PHP Version: 7.0.22
Database Driver & Version: 10.0.31-MariaDB

Description:
I want to update some data and get
Argument 3 passed to TCG\Voyager\Events\BreadDataChanged::__construct() must be an instance of TCG\Voyager\Events\string, string given
Deleted tcg folder from vendor, reinstalled... nothing((
Screen of error: https://i.imgur.com/iIWaqNR.png

Same with PHP 5.6. Try PHP 7.1

And Media is blank page(((

i get same erro. can you fix this?  what can i do?

I got this error wfter update to the latest version from github.. solved my problem by restoring to the previous version :)))

Closing since it's solved.

Reverting to the previous version is not a solution. Reopening


Use a new query to get all available bases for current org

Follow-up for https://github.com/boxwise/boxtribute/pull/1685#pullrequestreview-2436147595

 2.0 FE: Intra-org shipments between bases of the same org

@fhenrich33 can you fix the build error, pls.


Python 2.7 backport

Per here, potentially useful for some projects still on the Python 2 tail.

I suggest that we build for py3.x only until we're happy with what will be in v2, and then try py27 support on temp branches and make a decision if it is worth it.

Py2 is firmly EOL. stdio-mgr will be Python 3 only.


Flask website has broken links

If you go to: http://flask.pocoo.org/
And you try to click on the microblog link or the twitter clone link, neither of them work because the URLs have changed.

Also, the PDF download link doesn't work.

I've already adressed these in the flask-docs repo, they will be removed in the next build.

flask-docs

Ironically, the flask-docs repo lacks docs on how to build PDF. Following the commands in ReadMe results in an error complaining there's no theme.


[FEAT] : Make footer text normal one line in small size

Feature description
I believe that rather than this footer we add one liner footer or else we can improve with wave footers
Screenshot

Hey @ayusshh19, we've updated the footer in Version 2 as:

if you still wish to show your creativity, locally setup the repo!
Join us on Discord for any help.

hi, assign me this i will fix the footer

@pranshugupta54 i would like to work on this issue , can you please assign me?

Hi @ayusshh19 ! Thanks for wanting to contribute. To begin, try setting it up on your computer. If you run into any issues or have questions, feel free to join us on Discord. We're here to assist!


Add Map-Reduce Example

I used the very popular "Word Count" example in order to explain Map-Reduce in detail. In another application of Map-Reduce I found the very popular term frequency–inverse document frequency (short TF-idf) very interesting.

I used the very popular "Word Count" example in order to explain Map-Reduce in detail. In another application of Map-Reduce I found the very popular term frequency–inverse document frequency (short TF-idf) very interesting.


Cannot figure out how to modify an existing tree

Hello,
thanks for retext — it is very elegant to use!
Although I have managed to create a plugin which filters a tree, it seems the next plugin in the chain does not inherit of the changes of the previous plugin.
Here is my high level code:
      retext()
        .use(retextStopwords, { stopwords: stopwordsFr })
        .use(retextKeywords)
        .process(text, (err, file) => {
            if (err) {
            return reject(err);
          }

          resolve({
            keywords: file.namespace('retext').keywords,
            keyphrases: file.namespace('retext').keyphrases
          });
        });

retextKeywords is the retext-keywords plugin but because it does not catch french stopwords, I ought to modify the tree to remove them. Here is the code of retextStopwords:
'use strict';

const filter = require('unist-util-filter');

module.exports = (retext, options) => {
  const stopwords = options.stopwords;

  return (node, file, next) => {
    const tree = filter(node, node => {
      return !(node.type === 'TextNode' && stopwords.indexOf(node.value) > -1);
    });

    file.namespace('retext').tree = tree;

    next(null, tree, file);
  };
};

When I check tree, it indeed does not contain the TextNodes I wanted to remove.
But these words are still taken in account by retextKeywords which is next in the use() chain.
Any tip or hint to perform this?
Thanks a lot :-)

Yes, the Root node cannot be changed: it must always be the same object. You cannot change it by overwriting file.namespace('retext').tree.
You can however change all children in the Root in any way you like!

Understood!
Problem is when I change the children, it seems to break the phrase detection as, I guess, the positions have changed?
'use strict';

const visit = require('unist-util-visit');

module.exports = (retext, options) => {
  const stopwords = options.stopwords;

  return (node, file) => {
    visit(node, 'WordNode', node => {
      node.children = node.children.filter(d => {
        return stopwords.indexOf(d.value) === -1;
      });
    });
  };
};

Does it mean I have to also iterate over the next sibling position and update the WordNode position start/end as well?

Pfew; quite the problem.
First off: no, start and end are not used in retext-keywords, changing those shouldn’t fix/break anything.
Now, retext-keywords depends on retext-pos. With your code, the transformers run in order as expected: retext-stopwords, retext-pos, and retext-keywords.
However, I just now noticed you were talking about French. I think there lies the problem, retext-pos caters especially to English, and only words with certain parts of speech classifications are eligible for inclusion in the results. retext-keywords does not use stopwords, just POS tags.
As a consequence, I cannot come up with a solution for this other than a) create a French JavaScript POS tagger (extremely hard), or b) fork retext-keywords to also support words without POS tags and not occurring in a configurable list of stop-words (and with forking I mean I’ll accept it back into upstream if you’d PR).
I’m currently not in a position to dig in myself, but if you’re interested in working on this I can definitely advise and help out: it’s been a while since I touched the code though!

So if I understand well, best solution would be to implement the stopwords directly into retext-keywords, correct?
Or shall I mark the stopwords TextNodes as non-relevant for POS?
Although something I do not understand, is why retext-keywords (retext-pos) breaks because I altered the tree.


await app.render does not seem to be blocking

Koa handler:
await app.render(ctx.req, ctx.res, ctx.path, ctx.query);
ctx.respond = false
browser gets not found response, while next builds:
Building page: /app
DONE  Compiled successfully in 2513ms00:56:35
WAIT  Compiling...00:56:36
DONE  Compiled successfully in 214ms00:56:36
Any suggestions?

Interesting, can I have a sample app for this?

Ok I have found the problem after some debugging inside the next code. We need to set
ctx.respond = false  BEFORE the await next.render(...) so that next does not think that the response is already done and not render the component. The Koa sample application seems to be wrong in this regard and needs to be updated. I would prefer the 'owners' of the sample to comment.

Feel free to send us a PR.

The work around fix is to set ctx.respond = false before the next.render() call, but if one has passed the render responsibility to next, then is it necessary for it to test if the response == false and subsequently not render. Any opinion on the matter?

I assume await doesn't work well inside koa. May be you can convert the the promise return by app.render() into a thunk and yeild it.

await works perfectly, in fact that is way I went with Koa as opposed to express, because the whole system uses promises and generators. As I said above the problem is in the next code where it evaluates if res.respond == false and then returns prematurely, by resolving the promise without actually producing a response. I don't understand why this check is necessary give that I have handed off the rendering to next.


Add about menu item to show version info.

Add about menu item to show version info.

Coverage decreased (-0.01%) to 33.166% when pulling bd54cb528f74cff026cc00176649b5e2df38edd0 on wknet123:dev into bc52b212cf2f34d18549d875fbf05371dc1eff58 on vmware:dev.


Allow skipping tripleo-quickstart virtual infrastructure setup

In QE we would like to have the ability to create the virtual
test infrastructure outside the VM(acting as the provisioning host)
where dev-scripts are running in order to closely resemble a
baremetal environment. To achieve this:

we introduce an OOOQ_SETUP variable that controls if the
tripleo-quickstart-config playbooks run and create the virtual
infrastructure(vms, networks, etc)
introduce a BOOTSTRAP_SSH_READY var to control how long do we
wait for the bootstrap VM ssh service to come up. Since the
bootstrap VM runs as a L2 nested VM it needs more time to boot
create the baremetal and provisioning libvirt networks via virsh
and create sysconfig scripts for their associated bridges/interfaces
using the local dnsmasq is not needed as the DNS requirements are
set up externally
change the mechanism for determining the bootstrap VM IP address
by ping sweeping the baremetal network subnet and matching the mac
address. This is required since the DHCP server running on the
baremetal network is external to the provision host hence virsh
net-dhcp-leases will not return the IP address.

instead of using nmap to gather the ip, a better option would be to make sure bootstrap vm contains guest agent and uses that

Build SUCCESS, see build http://<IP_ADDRESS>:8080/job/dev-tools/386/

Few comments added - it'd be easier to review if you broke up the patch into a series of commits, or perhaps several PRs - not blocking on that but just noting it for future patches :)

Thanks much for the review Steve! I broke it into several different PRs for ease of review:
https://github.com/openshift-metalkube/dev-scripts/pull/229
https://github.com/openshift-metalkube/dev-scripts/pull/232
https://github.com/openshift-metalkube/dev-scripts/pull/233
https://github.com/openshift-metalkube/dev-scripts/pull/235
I'm going to leave this one for exploring how to use the NODES_PLATFORM option to achieve what I wish.

Build SUCCESS, see build http://<IP_ADDRESS>:8080/job/dev-tools/405/

Build SUCCESS, see build http://<IP_ADDRESS>:8080/job/dev-tools/406/

No longer needed. Changes were addressed in https://github.com/openshift-metalkube/dev-scripts/commit/1d74bd97550b3c84fe69e0effd130b2adad7485d


Combining multiple paths into a single spark_read_csv call

In Spark 2.0 I can combine several file paths into a single load().
How can I achieve this with sparklyr's spark-read-csv? Using a similar syntax on the path variable doesn't seem to work.

It should just work. Could you tell us a bit more about the directory structure and the Scala and R code you tried?

You are right. Looks like wildcards can be used similar to SparkR interface so it's actually easy to do. Thank you!

hi guys can you add this to the documentation?
this: does not indicate at all that I can use this feature to load multiple csv

path
The path to the file. Needs to be accessible from the cluster. Supports the "hdfs://", "s3a://" and "file://"protocols.


Please create new device profile - Zemismart TUYA WiFi Smart LED Ceiling Lamp 34W

HI,
I have this light:
Zemismart TUYA WiFi Smart LED Ceiling Lamp 34W
It shows firmware v1.0.0 -  BK7231N
Attached is the firmware for it.
firmware.zip
Many thanks.

This doesn't appear to be a standard Tuya firmware, the app partition is at a different offset, and it doesn't seem to be CRC16 compliant.  Not quite sure what all of that means yet.  The bootloader and storage areas appear to be valid.  I'll look into it more as time allows.

This is the teardown for it.
Currently running ESPHome on the device without any issues.

@openshwprojects This has now become a pattern of users who apparently tried dumping firmware with BK7231 Easy Flasher and would up with corruption from 0x11000 to 0x13000.  This one has a data chuck repeated, where the other just has all 0xFF throughout.
This was the first case, the other issue being #554 (where it was confirmed that the corruption was actually written to the flash).
Closing, as this is just a corrupt image we can't correct for.

@openshwprojects apparently this is the result of the advanced option Test read/write.  I would suggest giving a warning that it will corrupt the existing image at the very least.  Possibly change your test write address to OTA partition as well.

@Cossid  thanks for pointing that out, I would never expect that someone would wander to advanced options to test write before doing backup. I'll disabled it for now. Btw, if you are still looking for flash dumps, you can check here: https://github.com/openshwprojects/FlashDumps


Update Jedi dependency; closes #248

A bug in Jedi (davidhalter/jedi#1148) mutates sys.path, breaking module imports. The bug isn’t present in Jedi versions ≤ 0.11.1, so we cap the dependency at 0.11.1.
When Jedi is fixed, we can revisit, but as of 10 days ago (2018-08-09) the bug still seems to be present..

Fixed in new version; see this comment.


Cleanup sys.path after __pex__ is imported.

This fixes #1954 by ensuring all vendored code is uninstalled from the
path and bootstrap code is demoted to the end of sys.path after
__pex__ is imported.
Fixes #1954

@zmanji the merge of main into #1959 was pretty ugly; so I reversed things and just merged your commit here into an up2date upstream/main then applied my patch on top. Thanks for finding this and getting the fix rolling.


[REQUEST] Add Support for Running Games with Command-Line Arguments

Is your feature request related to a problem? Please describe.
Currently, there is no built-in functionality to run games with command-line arguments. This limitation restricts users from passing specific parameters to game executables, which is often necessary for configuring game settings, debugging, and enabling special modes or features.
Describe the solution you'd like
I would like to have the ability to execute game files with command-line arguments directly within the application. This feature should support the following:

[x] A method to specify the game executable file path.
[ ] An input field for entering command-line arguments.

The ability to handle different argument formats, including quoted strings and special characters.

Error handling for invalid paths or arguments.

Describe alternatives you've considered
No response
Additional context
Adding support for command-line arguments will enhance the application's flexibility and usability, especially for gamers who rely on custom configurations and advanced settings.

Duplicate of #302

duplicate, but much more detailed, I will keep this issue open.


Renew SSL certificate for website

Expires next month (Nov 14)

See #180 for details

Automatically renewed


MOB-11: Uploading a new art item to the profile

What's up?
Each user that have profile should add a new art item to their profile by using an upload button which is located on the main navigation bar.
To Do

[x] Activating the upload art item button in the main navigation bar.
[x] Creating a screen for uploading.
[x] Title, description and tags should be addable.

Deadline
29.11.2022 @14.00
Additional Information
The implemented features might be static at the deadline. However they have to be active before 06.12.2022 @14.00
Reviewers
@ooodogodogodogo

Reviewed and branch is merged.


Intercept Network Calls in Webview

Hi,
My issue is more of a question than a feature-request. I wanted to know if there is a way to intercept network calls(like any AJAX calls) made inside the WebView. Do we have anything similar to onNavigationStateChange as this only triggers when url changes occur. Please correct me if I am missing out on something.
TIA

maybe you could inject some javascript to wrap the built-in fetch and XMLHttpRequest.

Something like the following code should work to log calls:
XMLHttpRequest.prototype.openParent = XMLHttpRequest.prototype.open;
XMLHttpRequest.prototype.open = function (...params) {
console.log("########## NEW REQUEST");
console.log(params);
this.openParent(...params);
};

Thank you @evanc  and @DANIELVISPOBLOG  for the response. What I want to do by intercepting network calls and load the assets from the app cache if available. Is this possible?
The Webview default caching also works but it clears the cache when the webview component is unmounted.
Could you please help me with this.

I don't think you'll be able to do that.
If it's really critical that you maintain the cache maybe instead of unmounting the webview when you leave whatever page you could mount it at a high enough level in your tree that it's always mounted, and then show/hide it as needed, absolute positioned to fit on top of where you want it to go.
You'd said in the first post that you were looking at stopping AJAX requests, which generally are not cached.

Something like the following code should work to log calls:
XMLHttpRequest.prototype.openParent = XMLHttpRequest.prototype.open; XMLHttpRequest.prototype.open = function (...params) { console.log("########## NEW REQUEST"); console.log(params); this.openParent(...params); };

How we can inject it

@pravz71 , can you share your code if you did

   const INJECTED_JAVASCRIPT = `(function() {
      var open = XMLHttpRequest.prototype.open;
      XMLHttpRequest.prototype.open = function() {
          this.addEventListener("load", function() {
              var message = {"status" : this.status, "response" : this.response}
              window.ReactNativeWebView.postMessage(JSON.stringify(message));
          });
          open.apply(this, arguments);
      };})();`;

<WebView
          style={[{
            alignSelf: 'stretch',
            width: Dimensions.get('window').width,
            height: Dimensions.get('window').height,
            marginTop: Platform.OS === 'ios' ? 36 : 0
          }]}
          source={{ uri: azureLoginUrl }}
          javaScriptEnabled={true}
          domStorageEnabled={true}
          decelerationRate="normal"
          javaScriptEnabledAndroid={true}
          onNavigationStateChange={(navState) => { this._handleTokenRequest(navState) }}
          startInLoadingState={true}
          renderLoading={() => {
            return this.displaySpinner()
          }}
          injectedJavaScript={INJECTED_JAVASCRIPT}
          scalesPageToFit={true}
          onMessage={event => {
            console.log('>>>', event);
          }}
        />

But not working to get API response.

@DeepakSharma04 do u have the full code of this implementation ??

https://github.com/react-native-webview/react-native-webview/blob/master/docs/Guide.md#controlling-navigation-state-changes


Bridging fork up to date

Hello.
Please sync your fork before make any changes and pull requests.
Read more here.
Thanks.

Update


show_query() function method for objects

Unnecessary I think. It doesn't generate full sql statements.


Set category and query before performing search

Sets the query and category correctly when the "Show only results with digital matches" checkbox is clicked. I also realized there was a bug in the logic for the online flag, so I've fixed that.
My only reservation here is that the select doesn't behave in the same way as the checkbox. Maybe I'm overthinking it, but that seems somewhat problematic?
Fixes #386

My only reservation here is that the select doesn't behave in the same way as the checkbox. Maybe I'm overthinking it, but that seems somewhat problematic?

Hm, I overlooked that initially but I agree that the lack of a consistent pattern is problematic. I tested this pattern quickly with three folks, and I agree that the best approach is to make the select apply automatically/ initiate the search without having to additionally click on the search submit button so that it matches the function of the checkbox (and the facets).

@HaSistrunk LMK how that looks!


Removes all duplicate follow notifications.

this will check the guids of follow notifications, and not show any duplicates so users won't see follow spam.

Can we not just filter out the dupe in the collection in parse? It seems like it could be problematic to have the UI out of sync with the collection.

I'm going to remove the handling of duplicate follow messages in the notifications (the websocket code will remain, but will be updated).
After doing some more work on it, trying to fix the get_notifications API data in the client causes numerous issues.
The unread count becomes inaccurate, and since the unread count can include follow notifications that haven't been fetched yet, and the client can't fix that (ie: there are 11 unread duplicate follow messages, but the client only knows about the 6 duplicates it has fetched).
It also results in some very weird behavior where if you're being spammed, and the client fetched 6 duplicate notifications, only 1 is shown. We'd have to loop the fetch to keep fetching more notifications until we had at least 6 non-duplicates, which seems like a very bad idea.


string literal treated as any if nested in object hierarchy

Flow version: v0.114.0
Expected behavior
should error when type does not match
Actual behavior
problems are detected only if value is not nested

Try-Flow

// @flow-strict

type Route = {|
  +layout: 'cotne'
|}

type ModuleState = {|
  +route: Route
|}

declare var moduleState: ModuleState

switch (moduleState.route.layout) {
  case 'cotne':// ok
  case 'not so cotne':// should error
}

if (moduleState.route.layout === 'cotne') {}// ok
if (moduleState.route.layout === 'not so cotne') {}// should error
if (moduleState.route.layout === 42) {}// should error

declare var route: Route

switch (route.layout) {
  case 'cotne':// ok
  case 'not so cotne':// error is ok
}

if (route.layout === 'cotne') {}// ok
if (route.layout === 'not so cotne') {}// error is ok
if (route.layout === 42) {}// error is ok

workaround with typeof
// @flow-strict

type Route = {|
  +layout: 'cotne'
|}

type ModuleState = {|
  +route: Route
|}

declare var moduleState: ModuleState

const layout: typeof moduleState.route.layout = moduleState.route.layout

switch (layout) {
  case 'cotne':// ok
  case 'not so cotne':// should error
}

if (layout === 'cotne') {}// ok
if (layout === 'not so cotne') {}// should error
if (layout === 42) {}// should error

@dsainati1 this is not about refinements actually since i am not trying to refine anything in my examples.

It's now fixed


Shade all components

Addresses #2
Are there any other artifacts that should be excluded by default?

Fails on the sonar code coverage, but this pr does not add new code, so must be a current issue anyway.


Refresh ChildItems

Hello,
I have implemented the expandable Recycler view & its working fine. I am having a functionality that I need to refresh the child items with respect to parent item click. How to achieve it?

You can either override handleClick in your header view holder, override toggleExpandedItems in your adapter, and do whatever custom logic there that you need to.

Thanks for your kind reply. After I have override the toggleExpandedItems  method I am getting the parent item position. I have a toggle button in the header item & it contains mutiple child items.When the user changed the toggle state in the header item ,At the same time I need to change all the child items toggle state ex (true or false) with respect to that parent item toggle state.
With that clicked parent item position I need to notify to all my child items.How to achieve it? I am hard to find the solution...

I think since you have the parent item position in toggleExpandedItems you can iterate over the next items (which will be your child items) until you get to the next header item.

Thanks for your kind reply.Already I have iterate all my parents and children. When I was clicked the parent item I need to notify to all my children. In the toggleExpandedItems() method I have an position but how I will iterate the children items & also how will i get the reference of my child items in the toggleExpandedItems()  method.

I would do something like this
for (int i = position + 1; i < getItemCount() && getItemViewType(position) != TYPE_HEADER; i++) {
    // do something with child item at position i
}

Thanks for your kind reply. I have try with the above condition & am getting the below values.
position = 2  i.e parent item position
getItemCount() = 4 i.e total parent items
getItemViewType(position) = 1000
for( int i = position + 1 ; i < getItemCount() && getItemViewType(position) != TYPE_HEADER ; i++){
// am not getting the logs.
Log.i("toggle","dddddddddd" + i);
}

getItemCount() = 4 i.e total parent items

No, that's 4 total visible items including parents and children. Also, when you override toggleExpandedItems make sure you are calling super.toggleExpandedItems first, to make sure the new child items are added.

Sorry, I had a mistake in my sample code. getItemViewType(position) should not be called - it should be getItemViewType(i). Like this
for( int i = position + 1 ; i < getItemCount() && getItemViewType(i) != TYPE_HEADER ; i++) {
  }

Bro, Its working great When I was closed the expandable view it was triggered i.e I can see the logs and also printed the child position. When It was expanded will not be triggered i.e The logs are not printed. I need to get the child position when it was expanded.Please help me....

Sure, because after you called toggleExpandedItems the items have been collapsed! If the items are being collapsed, you should process them BEFORE you call super.toggleExpandedItems

sure, Many thanks...U have helped me a lot with your patience. Even though I have asked many doubts & questions U have quickly replied me and also given a quick solution. Thanks  a lot.....


PyTorch warnings about .grad access - safe to ignore?

Thanks for the amazing package! I just played around with it and found that PyTorch emits two warnings when using the package:
/path/to/python/lib/python3.10/site-packages/taichi/lang/kernel_impl.py:763: UserWarning: The .grad attribute of a Tensor that is not a leaf Tensor is being accessed. Its .grad attribute won't be populated during autograd.backward(). If you indeed want the .grad field to be populated for a non-leaf Tensor, use .retain_grad() on the non-leaf Tensor. If you access the non-leaf Tensor by mistake, make sure you access the leaf Tensor instead. See github.com/pytorch/pytorch/pull/30531 for more informations. (Triggered internally at aten/src/ATen/core/TensorBody.h:489.)
  if v.requires_grad and v.grad is None:
/path/to/python/lib/python3.10/site-packages/taichi_splatting/misc/autograd.py:8: UserWarning: The .grad attribute of a Tensor that is not a leaf Tensor is being accessed. Its .grad attribute won't be populated during autograd.backward(). If you indeed want the .grad field to be populated for a non-leaf Tensor, use .retain_grad() on the non-leaf Tensor. If you access the non-leaf Tensor by mistake, make sure you access the leaf Tensor instead. See github.com/pytorch/pytorch/pull/30531 for more informations. (Triggered internally at aten/src/ATen/core/TensorBody.h:489.)
  grads = [tensor.grad if tensor.grad is not None else None

It seems like gradients are still propagated, but those warning are a little concerning - are they safe to ignore? Can you provide more context on what's happening here? The first one is related to a Taichi kernel launch and looking at the source probably okay. The second one refers to taichi_splatting.misc.autograd and I'm not sure what the restore_grad function is trying to do (I can't find an active reference).

These are annoying warnings that arise from taichi tinkering with the
.grad value of a torch Tensor. There should be a better way for them to
handle the gradient integration. I think they shouldn't touch the .grad
value at all (for example, call it something else like .taichi_grad or
allow tuples of (value, grad) as input to their kernel.grad function)!
I have raised this with them here:
https://github.com/taichi-dev/taichi/issues/8339
On Mon, Mar 18, 2024 at 4:08 PM Christoph Lassner @.***>
wrote:

Thanks for the amazing package! I just played around with it and found
that PyTorch emits two warnings when using the package:
/path/to/python/lib/python3.10/site-packages/taichi/lang/kernel_impl.py:763: UserWarning: The .grad attribute of a Tensor that is not a leaf Tensor is being accessed. Its .grad attribute won't be populated during autograd.backward(). If you indeed want the .grad field to be populated for a non-leaf Tensor, use .retain_grad() on the non-leaf Tensor. If you access the non-leaf Tensor by mistake, make sure you access the leaf Tensor instead. See github.com/pytorch/pytorch/pull/30531 for more informations. (Triggered internally at aten/src/ATen/core/TensorBody.h:489.)
if v.requires_grad and v.grad is None:
/path/to/python/lib/python3.10/site-packages/taichi_splatting/misc/autograd.py:8: UserWarning: The .grad attribute of a Tensor that is not a leaf Tensor is being accessed. Its .grad attribute won't be populated during autograd.backward(). If you indeed want the .grad field to be populated for a non-leaf Tensor, use .retain_grad() on the non-leaf Tensor. If you access the non-leaf Tensor by mistake, make sure you access the leaf Tensor instead. See github.com/pytorch/pytorch/pull/30531 for more informations. (Triggered internally at aten/src/ATen/core/TensorBody.h:489.)
grads = [tensor.grad if tensor.grad is not None else None
It seems like gradients are still propagated, but those warning are a
little concerning - are they safe to ignore? Can you provide more context
on what's happening here? The first one is related to a Taichi kernel
launch and looking at the source probably okay. The second one refers
to taichi_splatting.misc.autograd and I'm not sure what the restore_grad
function is trying to do (I can't find an active reference).
—
Reply to this email directly, view it on GitHub
https://github.com/uc-vision/taichi-splatting/issues/8, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAITRZMLCAHFB5VQJ3BVXZLYYZLDRAVCNFSM6AAAAABE2ZAK5CVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE4TCMJTGM2DKMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks a lot for the quick response - the linked issue explains this in much more detail. Fingers crossed this will be fixed upstream in Taichi. :)


replace native splice

As per https://github.com/hughsk/display-tree/pull/2 - this is indeed faster,
like about 15% than native splice in this context. I'll take it :sparkles:

Oh, also worth noting that this does not account for GC since it's a rather tight loop. Relieving GC pressure is always +1


update descriptions and roles relative to hackweek service website

There are a number of updates to the descriptions of project and tutorial roles, in light of our new definitions on the main hackweek service website.
Originally posted by @aaarendt in https://github.com/uwhackweek/jupyterbook-template/issues/16#issuecomment-930439226

Currently underway with #27

@aaarendt what's the status of the education team roles?

@JessicaS11 let's close this issue and I'll open a new one that relates to updating the best practices for tutorial development section. I suggest we do this in parallel with the content that Naomi and Nicoleta are planning to deliver for ICESat-2.

After #75, the tutorials section is now just examples with the suggested layout for an event.
All previous content describing how to create a tutorial is in the hackweek guidebook: https://uwhackweek.github.io/hackweeks-as-a-service/resources/tutorials.html
I suggest going forward that any procedural content on how to contribute a tutorial is described in ./CONTRIBUTING.md


fix(typescript): use Proxy instead of modifying LanguageService instance

Referring to https://github.com/microsoft/typescript-template-language-service-decorator/pull/31, we found that some TS plugins use Proxy but do not implement the set method, which causes the method of decorateLanguageService to override LanguageService functions to be invalid.
This PR attempts to change decorateLanguageService to use Proxy to solve this problem without requiring modification of other TS plugins to improve Volar TS plugin compatibility.
cc @volarjs/core-team, @piotrtomiak (Please let me know if you think this is potentially breaking in WebStorm.)

This makes the following TS plugins now compatible. 🎉


Simulation of  Modelica.Utilities.Examples.readRealParameterModel usually fails

Reported by otter on 16 Oct 2010 22:44 UTC
The simulation of
Modelica.Utilities.Examples.readRealParameterModel

is only successful, if the current directory of the Modelica simulator is one directory above from the directory where MSL is stored. The reason is that in this model basically a call of the form:
file = "Modelica/Resources/Data/Utilities/Examples_readRealParameters.txt;
Modelica.Utilities.Streams.readLine(fileName, iline)

This issue should be fixed by introducing also URI-support for the file-reading functions (such as "readLine") of MSL.
This should be fixed in the next Modelica Specification 3.3.
In Dymola, this problem is currently handled by using the Dymola-specific built-in function classDirectory() to inquire the directory where the model is present where this function is called. The solution with URIs is, however, much better and consistent with the other usages of URIs.

Migrated-From: https://trac.modelica.org/Modelica/ticket/439

Comment by hansolsson on 1 Sep 2011 15:11 UTC
A new ModelicaServices allowing URIs for Modelica 3.3 will resolve that without any changes in the specification.

Modified by ahaumer on 2 Mar 2012 09:16 UTC

Comment by dietmarw on 7 Mar 2012 14:16 UTC
See also #443 for other parts of the library depending on this.

Modified by dietmarw on 7 Mar 2012 14:17 UTC

Comment by hansolsson on 11 Apr 2012 13:29 UTC
Waiting for the updated ModelicaServices that allows resolution of Modelica-URIs.
The MoelicaServices has been sent to Martin Otter - there are unrelated issues with updating it, as I understand.
The reason for resolving the Modelica-URIs with one function in ModelicaServices instead of having it in the file-reading functions are:

Only one place to update.
Resolving a Modelica-URI might require that the resources are extracted from a data-base and written to file; having such code inside the generated model becomes complicated.
Seems possible to combine with placing the resources in e.g. a source code FMI; without putting all library resources in the FMI.

Comment by dietmarw on 11 Apr 2012 14:06 UTC
The thing is we need to find something that works already according to Modelica Spec 3.2. If we can't then we should possible think about not doing any more releases MSL3.2.x but waiting for Spec 3.3 and making the MSL3.3 the next version.

Comment by hansolsson on 11 Apr 2012 15:17 UTC
Replying to [comment:6 dietmarw]:

The thing is we need to find something that works already according to Modelica Spec 3.2. If we can't then we should possible think about not doing any more releases MSL3.2.x but waiting for Spec 3.3 and making the MSL3.3 the next version.

I don't know if the comment above is aimed at this ticket or a general comment.
I agree that we in general have to consider that possibility.
However, for this ticket: ModelicaServices models implement functionality that is not standarized in the Modelica 3.2 spec, like 3d-animation. The point with ModelicaServices is exactly to be separate out the tool-dependencies in one package with tool-specific implementations.
Thus it is not necessary to update the specification to add this functionality for the interface of ModelicaServices (the interface just takes an URI-string and should return a string with a file-name).
How to implement that is up to the tool-vendors, similarly as how to implement 3d-animation; and a default implementation could just assume/document that you are in the right directory (as the current example).

Comment by dietmarw on 26 Jul 2012 06:13 UTC
This issue comes also up in combination with tables (#785). So how should we proceed to fix this?

Comment by otter on 4 Oct 2012 12:27 UTC
Fixed in 3c0057c98c16b7688cf857040b7f5b8cee670109 by introducing a tool specific function "loadResource" in ModelicaServices and call this function from the readRealParameter model (the file name is now given as an URI).

Modified by otter on 4 Oct 2012 12:28 UTC

Modified by hansolsson on 3 Apr 2013 14:41 UTC

Changelog modified by hansolsson on 3 Apr 2013 14:41 UTC
The example no longer requires a specific current directory; instead relies on new loadResource function in ModelicaServices.

Comment by dietmarw on 3 Apr 2013 14:44 UTC
Thanks Hans, we just need to keep in mind that in order to appear in the listings this ticket should either be unhidden or listings should later also include hidden tickets.

Modified by otter on 31 Jul 2013 16:00 UTC

Changelog modified by otter on 31 Jul 2013 16:00 UTC
Simulation of  Modelica.Utilities.Examples.readRealParameterModel no longer requires a specific current directory; instead relies on new loadResource function in ModelicaServices.


Secret masking improvements

Enforce masking of secrets in internal logger.  Also enforce masking  in colorized log if Printf used.
This PR implements the masking of secrets in logger output and revises how debugging is set up, with a focus on logs involving secrets. The secrets are made available only after the playbook is loaded, and they're now passed as parameters to setupLog function. Relevant tests were updated and an additional test case for Printf function with secrets was added.
see #181

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

9 of 9 (100.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.1%) to 84.601%

Files with Coverage Reduction
New Missed Lines
%

cmd/spot/main.go
2
81.79%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
2681

Relevant Lines:
3169

💛  - Coveralls


Upgrade deps

@boopathi did you see the lint issue https://travis-ci.org/babel/babili/jobs/186154496


Wait for admin password to get hashed before persist (new admin)

Similar to #487 await is also needed a few lines below where default admin password is hashed for a new admin.

Thanks @corneliusludmann ! 👍


OnTouchUpGestureDetector.onTouchUp does not take the MotionEvent as a parameter

not technically a bug, but doing so would be more consistent with other GestureDetector interfaces.

fixed in 2.0.2


Test failures with Intel14+MKL (Cholesky), Intel14 and g++ 4.4.7 (AgradMixMatrixMdivideLeftSPD.fv_exceptions)

Intel 14 with MKL fails over some Cholesky unit test.
Intel 14 and g++ 4.4.7 fail with AgradMixMatrixMdivideLeftSPD unit test

compressed log files sent by mail

Failures:

g++ 4.4.7

[ RUN      ] AgradMixMatrixMdivideLeftSPD.fv_exceptions
test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp:526: Failure
Expected: mdivide_left_spd(fd2, rvf2) throws an exception of type std::domain_error.
  Actual: it throws a different type.
test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp:527: Failure
Expected: mdivide_left_spd(fd2, rvf1) throws an exception of type std::domain_error.
  Actual: it throws a different type.
test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp:528: Failure
Expected: mdivide_left_spd(fd2, vf1) throws an exception of type std::domain_error.
  Actual: it throws a different type.
[  FAILED  ] AgradMixMatrixMdivideLeftSPD.fv_exceptions (1 ms)
...
[ RUN      ] AgradMixMatrixMdivideLeftSPD.ffv_exceptions
test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp:1493: Failure
Expected: mdivide_left_spd(fd2, rvf2) throws an exception of type std::domain_error.
  Actual: it throws a different type.
test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp:1494: Failure
Expected: mdivide_left_spd(fd2, rvf1) throws an exception of type std::domain_error.
  Actual: it throws a different type.
test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp:1495: Failure
Expected: mdivide_left_spd(fd2, vf1) throws an exception of type std::domain_error.
  Actual: it throws a different type.
[  FAILED  ] AgradMixMatrixMdivideLeftSPD.ffv_exceptions (1 ms)
...
[  FAILED  ] AgradMixMatrixMdivideLeftSPD.fv_exceptions
[  FAILED  ] AgradMixMatrixMdivideLeftSPD.ffv_exceptions

Intel 14

[ RUN      ] AgradMixMatrixMdivideLeftSPD.fv_exceptions
test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp:526: Failure
Expected: mdivide_left_spd(fd2, rvf2) throws an exception of type std::domain_error.
  Actual: it throws a different type.
test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp:527: Failure
Expected: mdivide_left_spd(fd2, rvf1) throws an exception of type std::domain_error.
  Actual: it throws a different type.
test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp:528: Failure
Expected: mdivide_left_spd(fd2, vf1) throws an exception of type std::domain_error.
  Actual: it throws a different type.
[  FAILED  ] AgradMixMatrixMdivideLeftSPD.fv_exceptions (1 ms)
...
[  FAILED  ] AgradMixMatrixMdivideLeftSPD.fv_exceptions

Intel 14 with MKL:

[ RUN      ] AgradMixMatrixCholeskyDecompose.mat_1st_deriv
unknown file: Failure
C++ exception with description "cholesky_decompose: Cholesky decomposition of m failed" thrown in the test body.
[  FAILED  ] AgradMixMatrixCholeskyDecompose.mat_1st_deriv (2 ms)
[ RUN      ] AgradMixMatrixCholeskyDecompose.mat_2nd_deriv
unknown file: Failure
C++ exception with description "cholesky_decompose: Cholesky decomposition of m failed" thrown in the test body.
[  FAILED  ] AgradMixMatrixCholeskyDecompose.mat_2nd_deriv (0 ms)
[ RUN      ] AgradMixMatrixCholeskyDecompose.mat_3rd_deriv
unknown file: Failure
C++ exception with description "cholesky_decompose: Cholesky decomposition of m failed" thrown in the test body.
[  FAILED  ] AgradMixMatrixCholeskyDecompose.mat_3rd_deriv (0 ms)
...
[  FAILED  ] AgradMixMatrixCholeskyDecompose.mat_1st_deriv
[  FAILED  ] AgradMixMatrixCholeskyDecompose.mat_2nd_deriv
[  FAILED  ] AgradMixMatrixCholeskyDecompose.mat_3rd_deriv

I'm guessing these are due to tolerances given where they're at and what else we've seen.  I updated the milestone to 2.8.0, but we can push this back to 2.8.0++ if necessary.  Motivation from Sebastian Weber on stan-dev:

The g++ 4.4.7 is part of RedHat Linux Enterprise 6.5 which is installed in many production systems.

Was that one of the tests Ben removed recently?
On Sep 2, 2015 2:02 PM, "Bob Carpenter"<EMAIL_ADDRESS>wrote:

I'm guessing these are due to tolerances given where they're at and what
else we've seen. I updated the milestone to 2.8.0, but we can push this
back to 2.8.0++ if necessary. Motivation from Sebastian Weber on stan-dev:
The g++ 4.4.7 is part of RedHat Linux Enterprise 6.5 which is installed in
many production systems.
—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/math/issues/133#issuecomment-137192620.

It's a few tests that are still failing.  We just need
to fix the tests and verify that they work.
I'm on the sparse multiply thing.

Bob

On Sep 2, 2015, at 2:04 PM, Daniel Lee<EMAIL_ADDRESS>wrote:
Was that one of the tests Ben removed recently?
On Sep 2, 2015 2:02 PM, "Bob Carpenter"<EMAIL_ADDRESS>wrote:

I'm guessing these are due to tolerances given where they're at and what
else we've seen. I updated the milestone to 2.8.0, but we can push this
back to 2.8.0++ if necessary. Motivation from Sebastian Weber on stan-dev:
The g++ 4.4.7 is part of RedHat Linux Enterprise 6.5 which is installed in
many production systems.
—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/math/issues/133#issuecomment-137192620.

—
Reply to this email directly or view it on GitHub.

Hi!
I analyzed the matter and it appears that the test checks possibly for the wrong expectation, i.e. the test expects a std::domain_error to be thrown when infact a std::invalid_argument is thrown from the check_size_match call. Here is a small patch fixing this:
diff --git a/test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp b/test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp
index bb2c4e9..a1ef28b 100644
--- a/test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp
+++ b/test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp
@@ -523,9 +523,9 @@ TEST(AgradMixMatrixMdivideLeftSPD,fv_exceptions) {
EXPECT_THROW(mdivide_left_spd(fd1, rvf2), std::domain_error);
EXPECT_THROW(mdivide_left_spd(fd1, rvf1), std::domain_error);
EXPECT_THROW(mdivide_left_spd(fd1, vf2), std::domain_error);

EXPECT_THROW(mdivide_left_spd(fd2, rvf2), std::domain_error);
EXPECT_THROW(mdivide_left_spd(fd2, rvf1), std::domain_error);
EXPECT_THROW(mdivide_left_spd(fd2, vf1), std::domain_error);

EXPECT_THROW(mdivide_left_spd(fd2, rvf2), std::invalid_argument);
EXPECT_THROW(mdivide_left_spd(fd2, rvf1), std::invalid_argument);
EXPECT_THROW(mdivide_left_spd(fd2, vf1), std::invalid_argument);
}

TEST(AgradMixMatrixMdivideLeftSPD,matrix_ffv_matrix_ffv1) {

uhh, posting patches here doesn't seem to work well (text gets processed in an unexpected manner).
I am happy to share the patch; albeit I guess the change is obvious.

What we need are GitHub pull requests.  But we can fix this
one easily enough.  No idea why it'd be passing on other machines
if the error's this basic.

Bob

On Sep 16, 2015, at 9:17 AM, wds15<EMAIL_ADDRESS>wrote:
Hi!
I analyzed the matter and it appears that the test checks possibly for the wrong expectation, i.e. the test expects a std::domain_error to be thrown when infact a std::invalid_argument is thrown from the check_size_match call. Here is a small patch fixing this:
diff --git a/test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp b/test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp
index bb2c4e9..a1ef28b 100644
--- a/test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp
+++ b/test/unit/math/mix/mat/fun/mdivide_left_spd_test.cpp
@@ -523,9 +523,9 @@ TEST(AgradMixMatrixMdivideLeftSPD,fv_exceptions) {
EXPECT_THROW(mdivide_left_spd(fd1, rvf2), std::domain_error);
EXPECT_THROW(mdivide_left_spd(fd1, rvf1), std::domain_error);
EXPECT_THROW(mdivide_left_spd(fd1, vf2), std::domain_error);
• EXPECT_THROW(mdivide_left_spd(fd2, rvf2), std::domain_error);
• EXPECT_THROW(mdivide_left_spd(fd2, rvf1), std::domain_error);
• EXPECT_THROW(mdivide_left_spd(fd2, vf1), std::domain_error);
• EXPECT_THROW(mdivide_left_spd(fd2, rvf2), std::invalid_argument);
• EXPECT_THROW(mdivide_left_spd(fd2, rvf1), std::invalid_argument);
• EXPECT_THROW(mdivide_left_spd(fd2, vf1), std::invalid_argument);
}
TEST(AgradMixMatrixMdivideLeftSPD,matrix_ffv_matrix_ffv1) {
—
Reply to this email directly or view it on GitHub.

ok, let me get familiar with your pull request process, etc. This one is very easy to fix indeed. However, once this one was out of the way, another test blocked me -
test/unit/math/prim/mat/err/check_pos_definite_test.cpp:66
a DEATH test does not trigger a  death when executed on our linux platform. Should I go for a new issue?
... I really think a testing system which tests all the way through and gives a summary page over what failed and what not would be very useful going forward. Would you guys agree on that or is this not needed as such a functionality already exists somewhere?

I thought there was a way to configure that behavior, but I don't
see it in the Python tests, which claim they only take one argument.
Daniel?
We don't usually bother because the tests cascade --- when one fails,
it causes other things to fail, so you get a huge overload of false positives.
I don't know what you mean by DEATH test.  You mean the EXPECT tests
in Google test?
Also, "make help" still does all this extra work --- it takes like 15s
to run on my machine.

Bob

On Sep 17, 2015, at 3:00 AM, wds15<EMAIL_ADDRESS>wrote:
ok, let me get familiar with your pull request process, etc. This one is very easy to fix indeed. However, once this one was out of the way, another test blocked me -
test/unit/math/prim/mat/err/check_pos_definite_test.cpp:66
a DEATH test does not trigger a death when executed on our linux platform. Should I go for a new issue?
... I really think a testing system which tests all the way through and gives a summary page over what failed and what not would be very useful going forward. Would you guys agree on that or is this not needed as such a functionality already exists somewhere?
—
Reply to this email directly or view it on GitHub.

Well, there is the -k switch which tells make to keep on going even if there was an error. However, this only works within a directory and still stops early. But I think I found a solution, i.e. I can simply find all those _test.cpp files with find and then call with its exec functionality a runTest.py on it. That should do it, right?
Concerning test/unit/math/prim/mat/err/check_pos_definite_test.cpp:66 - in that file a couple of EXPECT_DEATH tests do not cause a program termination on my machine. The absence of program termination does cause the test to fail (platform is Linux). I guess I open a new issue for that one.

Yes, please open a new issue for those.
Action is still on me to get this patch in.
On Tuesday, September 22, 2015, wds15<EMAIL_ADDRESS>wrote:

Well, there is the -k switch which tells make to keep on going even if
there was an error. However, this only works within a directory and still
stops early. But I think I found a solution, i.e. I can simply find all
those _test.cpp files with find and then call with its exec functionality a
runTest.py on it. That should do it, right?
Concerning test/unit/math/prim/mat/err/check_pos_definite_test.cpp:66 - in
that file a couple of EXPECT_DEATH tests do not cause a program termination
on my machine. The absence of program termination does cause the test to
fail (platform is Linux). I guess I open a new issue for that one.
—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/math/issues/133#issuecomment-142207756.

Closing because we fixed this ages ago by lowering the arithmetic tolerance to deal with MKL's inaccuracy.


use at-muladd for boundary coefficients in fast and threaded mode

This should give a noticeable speed-up - in particular in situations with relative many boundary nodes and few interior nodes, e.g., in https://github.com/trixi-framework/Trixi.jl/pull/1275.

Coverage decreased (-3.01%) to 88.038% when pulling 656b2c71c4a6efdc4a1028e2120c3fe6e3f7a0b1 on hr/MulAdd into ae5e60f3d80e06f60a2e0907be26547122bc93ac on main.


Doesn't return a generator function?

AssertionError: app.use() requires a generator function
Any idea why? Its already a generator function.

Hmm, could you share a minimal repo where you face this issue? I can check it out.

https://gist.github.com/moiz-frost/825024b4951fef71293022235e637bf7

Hmm, it doesn't look like you're using koa-combine-routers here, you're just using its implementation details in your combinedRoutes.js. There are a few things that could be causing this issue, I'm guessing it probably has to do with using a version of koa-compose that is meant for Koa v2 instead of Koa v1 which is what you're using.
If you try switching your code to use koa-combine-routers and you still face issues, let me know and feel free to reopen this.

Thanks, that seems to do the trick. I'm also wondering why do I have to not call the imported routes instead of just passing them in app.use()?
Line 24


datetime_range.module field type stubs

Part of https://github.com/mglaman/phpstan-drupal/issues/359
Create stub files for the following field types:

\Drupal\datetime_range\Plugin\Field\FieldType\DateRangeItem

i am intersted to do this work


Disable cake diagnostics endpoint for requests that don't specify a file

Cake's diagnostics engine calls the generic CodeCheckService, which generates all diagnostics for the solution if called without a specific file path. This results in duplicate errors in C# projects when VS Code requests project level diagnostics. The solution is to only compute diagnostics for specified files.
Fixes https://github.com/OmniSharp/omnisharp-vscode/issues/1830
Diagnostics window in VS Code now looks like:

cc @mholo65

@mholo65 It seems like Handle() does a bunch of pre- and post-processing on the request and result type that every overrider shouldn't have to be concerned with. I therefore added HandleCore where endpoints only have to be concerned with the actual call to the underlying service. Thoughts?

@rchande that’s a valid point. Instead of adding a new HandleCore method, what do you think of adding a bool IsValid(TRequest request) with the intention to validate the request?

@mholo65 Good idea, I'll do that.


How to debug syntax highlighting?

Hi,
first of all I'm really impressed with the project so far. The idea is awesome, the progress so far is impressive. I'm seriously rooting for it.
Problem
I have a syntax theme I built that is largely inspired by Vim's built-in Delek. You can find it here: https://open-vsx.org/extension/ambv/theme-chromodynamics-delek
It works fine in VScode with no warnings AFAICT. In Onivim2 the window is colored right but no syntax highlighting at all can be seen. This is true for all file types I tried (Python, Lua, JS, C++, HTML, etc.)
Compare VScode:

Onivim2:

How to debug?
Now, given the amount of work that's ahead of you I'd like to debug this myself and fix the theme (I guess there's something wrong about it, the other themes I tried worked.) However, I need some pointers as to how I can debug this. I tried running oni2 -f but it's pretty chatty. oni2 -f --quiet is less so but I didn't see anything syntax highlighting related apart from the proper registration of my theme as a plugin and some unhandled messages from the host around $registerTextEditorDecorationType.

Thanks for the kind words!
The easiest way to debug this issue would probably be with my PR (https://github.com/onivim/oni2/pull/2255) which adds a debug tool to check the tags (from textmate) and colours (textmate -> theme). It was originally added to track down a specific issue, but I should probably tidy/update it/merge it, since its useful generally for these sorts of issues. I also show in there the way to Compare against VSCode.
That would give us the source of the error at least, is it an error going from code to syntax tags (i.e. a textmate error, which is unlikely) or a textmate to colours error, which implies the theme is being loaded/setup wrong.
Its probably useful for us to generally hook up the same debug floats that Code has, to query the scopes and colours dynamically.
The second way is using the debug log, though you'll probably want to include --log-filter "Syntax", but I don't know how much info we currently log about this sort of thing.
I expect this is going to be some form of fallback issue, where we've not fully sorted out the "if no colour, fallback to C then B then A". If that is the case, it will cause issues with a decent number of themes.

The second way is using the debug log, though you'll probably want to include --log-filter "Syntax", but I don't know how much info we currently log about this sort of thing.

There's not much indeed.

log with `--trace --log-filter "Syntax"`

[INFO]  [ 0.141s] Oni2.Service_Syntax : Init called
[DEBUG] [ 0.142s] Oni2.Syntax.Client : Starting executable: /Applications/Onivim2.app/Contents/MacOS/Oni2_editor and parentPid: 5646 with args: --syntax-highlight-service=5646:/var/folders/17/pdb9_z8x1wx9xrpmb514tmx00000gp/T/exthost-8097bf-sock2
[INFO]  [ 0.153s] Oni2.Syntax.Client : Notifying theme changed.
[INFO]  [ 0.153s] Oni2.Syntax.Client : Notifying treeSitter changed: false
2020-11-01 17:07:07.251 Oni2_editor[5646:8933329] Setting Updater automatic checks: 1
2020-11-01 17:07:07.251 Oni2_editor[5646:8933329] Setting Updater feed URL: https://v2.onivim.io/appcast?channel=master&licenseKey=&platform=macos
2020-11-01 17:07:07.251 Oni2_editor[5646:8933329] Setting Updater feed URL: https://v2.onivim.io/appcast?channel=master&licenseKey=&platform=macos
[INFO]  [ 0.291s] Oni2.Syntax.Client : Connected to server
[INFO]  [ 0.291s] Oni2.Syntax.Client : Initialized
[INFO]  [ 0.291s] Oni2.Service_Syntax : onConnected
[INFO]  [ 0.291s] Oni2.Service_Syntax : Starting buffer subscription for: 2
[INFO]  [ 0.350s] Oni2.Syntax.Client : Notifying theme changed.
[INFO]  [ 6.367s] Oni2.Service_Syntax : Starting buffer subscription for: 3
[INFO]  [ 6.368s] Oni2.Service_Syntax : Stopping buffer subscription for: 2
[INFO]  [ 6.666s] Oni2.Syntax.Client : Received token update
[INFO]  [ 6.666s] Oni2.Syntax.Client : Received token update
...

Reading through the textmate debug helper issue it looks promising that this is just a difference in scope matching or scope priority, etc. I will have to figure out how to build my own Onivim 2 first to debug this.


[WIP] Move UTF-8 checking to stage 2 (up to +15% throughput)

This improves throughput by more than 11% overall on twitter.json on AVX2 Kady Lake R.
All files in jsonexamples got faster. Many were only 2-5% faster, but a number were in the 10-15% range, including twitter.json
The reasoning:

Do independent work in places with less CPU utilization: Stage 2 has a lower instruction-per-cycle density than stage 1. UTF-8 checking likely soaks up more CPU than it can in stage 1, just by virtue of there being more CPU to soak.
Do mispredicts in places with less CPU utilization: Correspondingly, stage 2's lack of congestion means it will generally find the mispredicting if statements quicker--there is much higher chance the if statement will be scheduled sooner just because the processor can't run quite as far ahead.
Clumping mispredicts together is good: Stage 2 already mispredicts more often in general, so we will get much-reduced mispredict impact when two mispredicts happen in short sequence. If you're already mispredicting, you may as well do another mispredict right away!
Don't check if UTF-8 is valid outside strings: UTF-8 outside of a string is invalid by virtue of not being ", [], {}, ,, :, true, false or a number. It is given a structural mark and will be rejected in stage 2 by the same logic that rejects ture and ~. By processing utf8 in stage 2 during string parsing, we don't run full utf-8 validity checks for empty blocks or blocks with just numbers.  I doubt this affected twitter.json much, but there are files that will get at least some improvement here. Indeed, random.json gets about a 15% improvement
Make stage 1 less sensitive to ordering: I'm sort of annoyed about this voodoo performance ordering dance I had to do, and this helps that--but I'm not yet sure if there's a broader principle here.
NOTE: as implemented, this will process utf-8 checks once per string rather than once per block. This means if a 32-byte block (16 on Westmere) contains more than one string, it will do redundant UTF-8 checks. On the other hand, we use an unaligned load at the start of the string rather than doing it block by block, so strings that stretch across block boundaries may get away with fewer utf-8 scans. It's possible to optimize this by remembering the end of the last scan, but given these results, I guess it's not even needed (at least not to start).

Given that it fails 2 of the UTF-8 checks, I have some investigation to do :)

OK, it turns out stage 2 was optimizing the utf8 check away and this is totally bogus :) I think there's something to it still--the fact that we're running the utf8 check over and over and over can't be good for business here :)

Oh! So basically after you eliminate escape characters. Yeah, that could rescue the performance of this.

While I was in there I noticed we're reloading the quote and backslash bits every time we hit a backslash. I'm being there is savings to be had there by clearing the bits as we use them, until they are empty. It still seems better to crunch the backslashes and write them out the first time we get them though, in stage 1.

While I was in there I noticed we're reloading the quote and backslash bits every time we hit a backslash. I'm being there is savings to be had there by clearing the bits as we use them, until they are empty. It still seems better to crunch the backslashes and write them out the first time we get them though, in stage 1.

You are correct that we are probably handling backslashes in a suboptimal manner.
It is not clear to me how to do better at this time, however. You refer to this notion of "crunching the backslashes". Ok. So in stage 1, we do identify the backslashes that need to be "crunched" (whatever "crunched" means) but how do you do it?
You sketched it here:
https://github.com/lemire/simdjson/issues/228
Ok. So you write "single-character escapes yield single-byte replacements". Right. A single-character escape uses two bytes, and you must replace it with one byte. This means that you can, indeed, always do the processing "in situ" because escaping is always less space efficient...  And we have the information in stage 1.
In fact, in stage 1, we know exactly where the strings are, we know where the escapes are. It is clear we could do more... But how to do more is the tricky thing.

Heh, I was about to go put my new insights on the stage 2 from here, and realized that I have nothing new to add. Past me apparently knew things!
The only new thought is we might be able to do stage 2's escape/quote processing more efficiently by processing backslashes one SIMD width at a time instead of restarting after each backslash. It's not certain, though. Maybe the overhead of clearing backslash bits after processing them will end up worse the SIMD load and eq check. I've got a patch almost compiling, but I'm trying to take it easy for a few days, so it'll be a bit :)

I'm eager to read code that does escaping more efficiently.


SPARKC-181:Implements leftJoinWithCassandraTable

Created CassandraLeftJoinRDD  (copy/paste from CasssandraJoinRDD) to implement RDDFunctions.leftJoinWithCassandraTable

I really appreciate the effort here. I know we've let this ticket sit for a while. I am a little worried about the implementation because of the big copy paste. I'm hoping that we can do this with a bit less code. Maybe if we extend the CassandraJoinRDD you will have less code duplication. This will also help with our coming refactor where we separate out the serializable from non-serializable code. I'll try to devote some time to a full code review after C* summit.

Unfortunately, we can not simply extends CassandraJoinRDD becase of RDD.compute method, for left join it must return Iterator[(L, Option[R])] not Iterator[(L, R)](like in CassandraJoinRDD). To completely eliminate copy/paste we need to refactor CassandraJoinRDD too.

Can't we extend CassandraJoinRDD[L,Option[R]]?

Here is the problem, CassandraJoinRDD must extend CassandraRDD[(L, R)] and CassandraLeftJoinRDD must extend CassandraRDD[(L, Option[R])] cause they are collections of different objects. I do not think that we can extend one from another. May be we need to rethink CassandraRDD class in terms of self types and type classes not inheritance?

I think we can avoid that problem but extending the JoinRDD but the
RowReader ends up breaking us because we need to present a rowReader of
type Option[R] if we directly extend JoinRDD.
Something like
https://gist.github.com/RussellSpitzer/40d6aa6c1f0ff29c033d
The issue there is we'd also have the magic for creating a RowReader
Option[R] which means we add a little more logic to the RowReaderClass but
that brings us back to our core problem which is the API for RDD's is too
tightly coupled with the Spark Internals.
We discussed this on the checkpointing serilization ticket. We need to
refactor all the CassandraRDD classes so that their internal computation is
not so tightly coupled with their api. If for instance we had a
CassandraJoinRDDFactory, it could generate the proper rowReader and
rowWriter, then it creates a CassandraInnerJoinRDD and CassandraLeftJoinRDD
and we wouldn't have so many issues. If you'd like to be involved in those
discussions I can be sure to let you know when we start getting into the
muck :)
On Tue, Sep 15, 2015 at 2:44 PM Alexey Ponkin<EMAIL_ADDRESS>wrote:

Here is the problem, CassandraJoinRDD must extend CassandraRDD[(L, R)]
and CassandraLeftJoinRDD must extend CassandraRDD[(L, Option[R])] cause
they are collections of different objects. I do not think that we can
extend one from another. May be we need to rethink CassandraRDD class in
terms of self types and type classes not inheritance?
—
Reply to this email directly or view it on GitHub
https://github.com/datastax/spark-cassandra-connector/pull/832#issuecomment-140555736
.

Yes, sure. Keep me in touch!

I had planned to implement a Left Join and I've found this issue. I'm interested by this feature, is there anything missing ? I'd like to help if needed.

Hi, eallain-poctu
Here is the ticket SPARKC-279 where you can propose solution or something.

+1 for this. We use a similar solution in production today and it would be great to have this in the general connector API.

+1 for this. We use a similar solution in production today and it would be great to have this in the general connector API.

+1

We have been taken a really long time on separating out our RowReader/Writer code, wondering if we should just rebase and merge this and clean it up when we do separate the other modules

Hi everybody, googd to know that @ostefano made this pull request. Let me suggest you a small fix: I have just moved all identical code in CassandraJoinRDD and CassandraLeftJoinRDD(copy-paste) to single trait AbstractCassandraJoin. Whatr do you think? I use my old fork. So if it is ok I will try to merge it with @ostefano pull request or something.

Merged from collaboration branch

https://github.com/datastax/spark-cassandra-connector/pull/1003


Reject unknown singularity fields

Includes #777
See changelog entry and tests for explanation :)

          1 Warning

:warning:
Big PR: 765 lines of code changed

  Generated by :no_entry_sign: Danger

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9f541d7). Click here to learn what that means.
The diff coverage is 66.19%.

@@            Coverage Diff            @@
##             master     #780   +/-   ##
=========================================
  Coverage          ?   55.25%
=========================================
  Files             ?      273
  Lines             ?    15822
  Branches          ?        0
=========================================
  Hits              ?     8743
  Misses            ?     6337
  Partials          ?      742

Impacted Files
Coverage Δ

ext/otpl/otpl.go
69.93% <66.19%> (ø)

Continue to review full report at Codecov.

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

Note this is failing checks due to a single unit test failure, nothing to do with the code in this PR. I thought I'd fixed that in #785 but somehow it's still happening, will take another look but don't think this PR should be blocked by it @JC1738