no edge has been created because no source vertices

var g = orient.getGraph();
try{
  var user = request.getUser();
  var vertex = 'CREATE VERTEX EventNormal SET created=Date(), updated=Date(), title=?, caption=?';
  var v = g.command('sql', vertex, [title, caption]);
  var erid = v.getRecord().field('@rid');
  var created = v.getRecord().field('created');
  var edge = 'CREATE EDGE NewEventNormal FROM (SELECT FROM OUser WHERE name=?) TO (SELECT FROM EventNormal WHERE @rid=?) SET created=?';
  g.command('sql', edge, [user, erid, created]);

  g.commit();
  return v;
}
catch(err){
  g.rollback();
  response.send(500, "Error on creating new user", "text/plain", err.toString() );
}

sometimes i received this error
{
"error": "com.orientechnologies.orient.core.exception.OCommandExecutionException: No edge has been created because no source vertices\r\n\tDB name="MyApp""
}
now i solved the problem. is not in the transaction but in the login session. i can create a edge only if i logged with a VERTEX user. but sometimes when i open a new schede by the studio dashboard i see that the logged user is not the admin but the root (also if i logged with an admin account). finally the root user not is a VERTEX and is normal that i not can create a edge for this user.  the bug is in the login session

@DThink
yes if you login with root, that does not have a record in your database, that create edge will
fail.
Which bug in the login session?

when i log with admin or other user different by root, if i open a dashboard link in another browser schede i see that the logged user is the root user

@DThink
that i think it's probably caused by the Basic Authentication. Studio and ODB Http relies on that, and i think the browser inject the saved credentials.


Restore the constructor to prevent RecursionError at 1000+ tests

When the fixture aresponses is used in more than 1000 consecutive tests (empirically, 940+), it hits the RecursionError when calling its ClientRequest's constructor. All tests that follow, also fail regardless of what and how they test.
Caused by the constructor function being replaced, calling the original pre-replacement function, and never restoring the original function. As a result, every test adds +1 to the recursion, until it reaches some limits where Python raises a RecursionError for all frames (~995 frames from the entry point for a pure-Python script, and roughly ~940 level within pytest's context).
For the testing purposes, a limit is artificially lowered to 100 (of which 40-50 levels are taken by pytest & co, 10 for aresponses, and nearly 40 are consumed by recursion). The default 1000 is too slow (several seconds, not instant enough). Besides, it can vary in the future.

A sample stack trace without the fix (for search engine indexing):
Traceback (most recent call last):
  File "/Users/nolar/src/aresponses/tests/test_server.py", line 397, in test_not_exceeding_recursion_limit
    async with session.get("http://fake-host"):
  File "/Users/nolar/.pyenv/versions/3.9.0/envs/arsp39/lib/python3.9/site-packages/aiohttp/client.py", line 1117, in __aenter__
    self._resp = await self._coro
  File "/Users/nolar/.pyenv/versions/3.9.0/envs/arsp39/lib/python3.9/site-packages/aiohttp/client.py", line 492, in _request
    req = self._request_class(
  File "/Users/nolar/src/aresponses/aresponses/main.py", line 282, in new_init
    self._old_init(_self, *largs, **kwargs)
  File "/Users/nolar/src/aresponses/aresponses/main.py", line 282, in new_init
    self._old_init(_self, *largs, **kwargs)
  File "/Users/nolar/src/aresponses/aresponses/main.py", line 282, in new_init
    self._old_init(_self, *largs, **kwargs)

  [Previous line repeated 38 more times]

  File "/Users/nolar/.pyenv/versions/3.9.0/envs/arsp39/lib/python3.9/site-packages/aiohttp/client_reqrep.py", line 305, in __init__
    self.update_host(url)
  File "/Users/nolar/.pyenv/versions/3.9.0/envs/arsp39/lib/python3.9/site-packages/aiohttp/client_reqrep.py", line 369, in update_host
    username, password = url.user, url.password
  File "/Users/nolar/.pyenv/versions/3.9.0/envs/arsp39/lib/python3.9/site-packages/yarl/_url.py", line 50, in __get__
    val = self.wrapped(inst)
  File "/Users/nolar/.pyenv/versions/3.9.0/envs/arsp39/lib/python3.9/site-packages/yarl/_url.py", line 476, in password
    return self._UNQUOTER(self.raw_password)
  File "/Users/nolar/.pyenv/versions/3.9.0/envs/arsp39/lib/python3.9/site-packages/yarl/_url.py", line 467, in raw_password
    return self._val.password
  File "/Users/nolar/.pyenv/versions/3.9.0/lib/python3.9/urllib/parse.py", line 154, in password
    return self._userinfo[1]
  File "/Users/nolar/.pyenv/versions/3.9.0/lib/python3.9/urllib/parse.py", line 189, in _userinfo
    userinfo, have_info, hostinfo = netloc.rpartition('@')
RecursionError: maximum recursion depth exceeded while calling a Python object

@nolar Wow that couldn't have been the easiest to debug and fix. Thank you!

@nolar Wow that couldn't have been the easiest to debug and fix. Thank you!

It took some time indeed ;-)
You're welcome! And thank you for this nice tool — it is a pleasure to test with it, and to simulate K8s API locally.

It took some time indeed ;-)
You're welcome! And thank you for this nice tool — it is a pleasure to test with it, and to simulate K8s API locally.

Glad to hear it's been useful. thank you

Glad to hear it's been useful. thank you

I've pushed aresponses 2.1.1 to pypi with this fix.

I've pushed aresponses 2.1.1 to pypi with this fix.


Nikon/Canon raw images

Any chance we can get raw image support for Nikon and Canon formats?

Hello, support for image formats is via libvips, so you'll need to ask John over at https://github.com/jcupitt/libvips/issues/new.
I believe there are quite a few not-quite-standard standards for "raw" images, so an intermediate library such as libraw would help.
Even if libvips were to support import via libraw, I would need to clarify the licensing terms of libraw before including it in the pre-built binaries, but you would at least be able to use a global install of libvips.
libraw is currently dual licensed via either (a) LGPL2.1 but not later, which is incompatible with Apache 2.0 or (b) CDDL, which is incompatible with LGPL 3.

My understanding is that almost all RAW decoders (even the one in Photoshop) are based on dcraw.exe, a command-line program maintained by Dave Coffin. Quite reasonably, Dave is not interested in maintaining a library, it's simply a Windows command-line program, and that's it.
There have been at least four separate attempts to make a nice, reusable library out of dcraw, but (I think?) they've all floundered in one way or another. It's not an simple thing to do: it needs to look like a proper library (support for threads, exceptions, internationalization, resource management etc etc.), but it has to be built on a code base which has none of those things. And worst of all, it has to track future changes to dcraw.exe, so you can't just rewrite it.
libraw looks like the best one at the moment, but who knows. If the maintainers get bored it'll die very quickly. A friend of mine (one of the Dopus authors) made his own, that might be worth looking at too.

Dcraw is an excellent piece of software for processing camera RAW files. I once attempted to see if it can be converted to a shared library that can be invoked from NodeJS. The problem is, it is a single C source file with 10K plus lines and next to nothing in terms of documentation. I soon gave up and instead thinking about invoking it as a child process. Unless you have an army of engineers like Adobe it seems like a mammoth task.

Thank you all for the thought put in thus far. I've looked at dcraw for this in the past, I'll continue down that avenue.

Here's the wrapper my Dopus friend made:
https://www.pretentiousname.com/pnop_source/index.html#about_dcrawrap
He talks a little about the issues with the code and maintenance problems it generates for libraries which wish to use it.

Thanks @jcupitt  that's very useful. @PCaponetti  libraw also tried to remove the same issues with dcraw in terms of threadind. see the link here:
http://www.libraw.org/about#better
Also there is an NPM module with libraw support in case you are interested:
https://github.com/m0g/node-libraw

Wow thanks again to all.  I'm going to look deeper into node-libraw now :)
Seems like that will be a little easier

@PCaponetti - I looked at the source code of https://github.com/m0g/node-libraw and apparently this is a synchronous module and will block the event loop - meaning once libraw processes one RAW file all requests will wait including http. I am not sure if this will serve your need.

oddly enough, I think that works for the electron application I'm
creating.  Thank you for the update on this
On Mon, Nov 21, 2016 at 12:30 AM, Sachin Walia<EMAIL_ADDRESS>wrote:

@PCaponetti https://github.com/PCaponetti - I looked at the source code
of https://github.com/m0g/node-libraw and apparently this is a
synchronous module and will block the event loop - meaning once libraw
processes one RAW file all requests will wait including http. I am not sure
if this will serve your need.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lovell/sharp/issues/616#issuecomment-261847252, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA93OPf5LV7OBdpGEKT6Tiy_YPzVA23nks5rASx9gaJpZM4KkAQg
.

--
Sincerely
-Paul Caponetti

Closing as there's not a lot sharp can do here. For Node users, node-libraw appears to the best option.


Automatically set max_num_batched_tokens

This PR removes the default value (2560) of max_num_batched_tokens and sets it based on the model's maximum length.

@zhuohan123 Sorry for the commits after requesting your review. Now the PR is ready for review!

For user who do not want auto set, after this PR, are there any idea to explicitly set this param?


Use hardcoded library version on bStats

Is there an existing issue for this?

[X] I have searched the existing issues

🐛 Describe the bug
No response
✔ Expected behavior
No response
👣 Steps to Reproduce
No response
💻 Platform

[X] Bukkit
[ ] Sponge
[ ] Other

⭐ Versions
v2.5.4-beta
✍ Additional context
No response

Done #227


fix compatibility issue with other chunk loaders

https://github.com/Chicken-Bones/ChickenChunks/issues/28


[Doc] fix broken links to Node.js releases

A couple of README files have different link than most packages' in this repo, likely due to in-flight PRs when we update the link to point to github one. These are now broken. This PR replaces them with the one used in other packages.

API change check
API changes are not detected in this pull request.

js - maps - ci is failing in main and unrelated

/check-enforcer override


[mongo-cxx-driver] Fix compilation error under VS17.11.2

Fixes https://github.com/microsoft/vcpkg/issues/40807
F:\tv\buildtrees\mongo-cxx-driver\src\r3.10.2-4749d7b703.clean\src\mongocxx\include\mongocxx\v_noabi\mongocxx/events/heartbeat_failed_event.hpp(50): error C2039: 'string': is not a member of 'std'
C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Tools\MSVC\14.41.34120\include\__msvc_string_view.hpp(22): note: see declaration of 'std'
F:\tv\buildtrees\mongo-cxx-driver\src\r3.10.2-4749d7b703.clean\src\mongocxx\include\mongocxx\v_noabi\mongocxx/events/heartbeat_failed_event.hpp(50): error C3646: 'message': unknown override specifier

Download upstream PR 1193 to fix the issue.

[X] Changes comply with the maintainer guide.
[ ] SHA512s are updated for each updated download.
[ ] The "supports" clause reflects platforms that may be fixed by this new version.
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

Usage test pass with following triplet:
x64-windows

Thanks!


(bug) ngUpgrade requires click event to finish modal render

When opening a bootstrap modal within ngUpgrade, the modal fails to completely render. A successful renders after click event, or after a long time of waiting (a single long frame according to DevTools Timeline).
ngUpgrade sample code: https://github.com/sime/angular2-modal-ngUpgrade

Angular version: 2.0.0-rc.4

I'm not familiar with ngUpgrade, anyone want's to pitch?

I'm actually having the same issue, but not using ngUpgrade.
Background darkens, but no modal until a button press or resizing the window, or waiting about 105 seconds. Then the modal fully renders.
Looking at the DOM changes, the div with the "modal-content" class inside the modal-container component only contains a div with display: none. Once you wait long enough, resize the browser, press a key, etc., the modal-content component shows up.
@sime are you sure your issue is definitely ngUpgrade related?
@shlomiassaf any ideas on what's going on here?

@dennissheppard It was a little while ago, and I managed to reproduce with ngUpgrade. And IIRC I was not able to reproduce with an Angular 2 only site.

Looks like the button click is happening outside of angular2 zone.
I would try to do this in your component:
Inject NgZone dependency:
constructor(private zone:NgZone) {}

And on click callback wrap all your code in:
this.zone.run(() => {
  //your code here
});

I hope it helps.

Hi @animator013
What should NgZone wrap? I wrapped the contents of the open() method in AppComponent and the modal opened in the same state.
https://github.com/sime/angular2-modal-ngUpgrade/blob/af6f47a5f01526bf284861983608184a1491f651/src/app/app.component.ts#L15-L16
Also I don't believe click event is happening outside of the zone as the modal starts.

Hi again.
I had the same problem with InteractJs. The modal wasn't showing exactly the same way. Another action was required to open it. But interactjs is external library and events were outside of angular2.
I looked into your code and it seems it is not your case. Then it must be related with ngUpgrade. But at least you gave it a shot. Sorry for the inconvenience.

I am facing the same issue. Did anyone got the solution? Following is the config and sample code.
The model-content  is rendered without the component first. And then after a key press it triggers on change and then renders the component passed.
"@angular/common": "2.4.1",
"@angular/compiler": "2.4.1",
"@angular/core": "2.4.1",
"angular2-modal": "^2.0.1",
Code
let builder = new BSModalContextBuilder(
{ data: {} } as any,
undefined,
BSModalContext
);
let overlayConfig: OverlayConfig = {
context: builder.toJSON()
};
this.modal.open(ComponentName, overlayConfig);


library view command

I'm getting Cannot read property 'then' of undefined when I run particle library view neopixel I'm not sure why


tt_lib.tensor.glu, geglu, reglu, swiglu - inconsistent documentation

For ops:

tt_lib.tensor.glu
tt_lib.tensor.geglu
tt_lib.tensor.reglu
tt_lib.tensor.swiglu

The documentation does NOT state that operands need to be on device. But when operations are run with inputs in system memory we get error:
input_tensor_a.storage_type() == StorageType::DEVICE
info:
Operands to unpad need to be on device!

Merged to main, hence closing this issue.


[labs/testing] Call site location by stack trace might not work in webkit

Which package(s) are affected?
Other/unknown (please mention in description)
Description
Seeing failures in this workflow. https://github.com/lit/lit/actions/runs/5744321662/job/15570775282?pr=4062#step:8:506
This is due to webkit stack trace not including the ?wtr-session-id querystring in the source file url, which we rely on here https://github.com/lit/lit/blob/51c431ee2a4c28a6e1a2fe6b0069ad24c7df0df3/packages/labs/testing/src/lib/fixtures/ssr-fixture.ts#L60-L62
Cause seems to be playwright version greater than 1.32. Unsure if it's playwright + webkit issue, or webkit straight up not including the querystring in stack.
Solution might be to find a different way to pinpoint the source file, maybe finding the first file that isn't an internal @lit-labs/testing file.
Reproduction
TODO
Workaround
Pin version of playwright.
Is this a regression?
Yes. This used to work, but now it doesn't.
Affected versions
0.2.1
Browser/OS/Node environment
.

or webkit straight up not including the querystring in stack

I don't think it's that the querystring isn't in the stack, I think it's that the stack frame that includes it is pushed out of the 10 frame window that's output in error.stack. Somehow there are more lines for the test framework in Webkit not.

The behavior varied depending on the version of playwright. I've seen the mocha stacks, but also the correct lines without the querystrings that won't match the regex.


Udpate fedora 27 to fedora 28

Should be possible to update to fedora 28 in inventory?
https://github.com/jborean93/ansible-windows/blob/2225681acbb33a6a84e2b0af54126985856dcbda/vagrant-linux/inventory.yml#L14-L16

I don't see why not, feel free to send through a PR.

Thanks, updated with https://github.com/jborean93/ansible-windows/pull/10


JSEN requires unsafe-eval

Unsafe-eval is discouraged when using Content-Security-Policy (CSP) best practices. Unfortunately this breaks JSEN.

Yes, JSEN relies on dynamic function compilation through eval and new Function() which is exactly what CSP prevents. It's still safe when you're using your own predefined schemas and do not allow user-specified schema compilation. But it doesn't work in a CSP-enabled runtime environment.


feat(e2e): add artillery to perform stress tests

Context
We need to integrate artillery.io into our SDK to perform stress tests.
Proposed Solution
Added a e2e test mainly with the purpose of artillery.io POC.
Important Changes Introduced
From this line:

WORKERS=1 starts only one worker: details will follow;
node -r ts-node/register ../../node_modules/.bin/artillery running artillery this way node-ts does the magic: we can require a .ts file which imports the SDK without the need to rebuild nothing.

In artillery.ts are defined some types to make more comfortable the custom hooks development.
Probably this is not a best practice, but I think this is something important to know.
In the custom hooks file I used a global variable, since artillery.io runs more workers, the global variables are not shared between the workers, so neither between hooks executions. In order to make the global variable shared between all the hooks executions I had to specify to run only one worker.

Hi @rhyslbw , @mkazlauskas , @iadmytro and @IvayloAndonov ;
this should be ready to be reviewed.
Since the test is run through artillery rather than jest,  I wasn't able to load the environment from the .env file, so to run the tests currently following command is required:
STAKE_POOL_PROVIDER_URL="http://localhost:4000/stake-pool" yarn artillery:stake-pool-query

or change the target to perform the tests against a different system from our local network.

Please rebase and squash in preparation for final review @iccicci

Hi @rhyslbw and @iadmytro ;
I rebased this with artillery bump to v2


What is the reason for this error? It has been updated several versions and sometimes login is possible and sometimes login is not possible.

Describe the bug
What is the reason for this error? It has been updated several versions and sometimes login is possible and sometimes login is not possible.
The log is as follows：

Thank you for your answer。
Steps to reproduce
This error has always been present during each upgrade
The version of Memos you're using
v20
Screenshots or additional context

The logs you provided are correct.


Use Hamcrest assertions instead of JUnit in integrations/cdi/datasource-ucp (#1749)

Part of #1749
I've already signed OCA. It's an automation problem.
Backport 2.x

/oca-checked

/trigger


[BUG] Game "hangs" for a few seconds after playing a song

Is there an existing issue for this?

[X] I have searched the existing issues and none matched.

Operating System
Windows 10
CPU
i5-12600KF
GPU
Radeon RX 6700 XT
Storage
WD_BLACK SN770 NVMe
Game Version
4.16.0HF
Game Mode
dance
Theme
Simply Love v5.1.2 (latest)
Describe the problem
Oftentimes after clearing a song I get 3-4 seconds (or longer) pauses when leaving the gameplay screen. It doesn't happen 100% of the time but it is frequent. I haven't reproduced the issue with the default theme yet (but I have given it just a few tries, I never use default).
I think this is an old problem. It happened to me back in 2021-09-04 (using latest OutFox release at that time), but maybe less frequent at that time (smaller Stats.xml ?).
I have attached logs I copied just after the freeze happened.
I have checked the "theme errors" (that can be displayed using the debug menu, "Show recent errors") but nothing was shown there.
Logs.zip
Describe what should happen
The game shouldn't hang
Relevant Log output
No response

Please note that I am using Dropbox to keep my scores synced and backuped. I got to think about it some more, and I'm strongly suspecting OF is trying to acquire a lock after Dropbox has put a lock on the file (for uploading after last change). Since duplicates appeared in my Stats.xml at some point and it grew huge (10MB), it would explain why I am starting to notice these delays more and more. Sorry for the useless bug report if this theory is true. I will try to confirm it.

We cannot write to a file that is locked, so the main thread will hang until the lock is released.
You should turn off Dropbox when playing as it can cause dual entries and corruption.


Local build is not possible at the moment

Some packages are blocked by internal firewall of the gov. My current VPN tools have issues in term of SDK for routing all connections (or maybe I am dumb in doing that which is more likely).
For now, I should rely on poor (maybe rich) github servers to deal with this. Literally, I am doing direct development in production environment.

It had nothing to do with internet. Issue was the 'mini_racer' gem in Gemfile. Simply comment this out and install latest nodejs.


Plugin failed to show error when tab character available in yaml

Plugin failed to show error in YAML file when there is tab character at any place in file.
As per my knowledge TAB characters is not allowed in YAML. It would be better if plugin show error for tab character.

Thanks for the issue @richivarma! We'll look into this when we have some time.


Added the grid Layout

Hey @JesseRWeigel ,
I have implemented the grid layout it looks like this.

I have even made the default layout dynamic by storing the last layout in localstorage.


tsconfig paths not work if dir name start with dot.

TypeScript Version:  3.7.3

Search Terms:
Code
./.temp/test
export enum Test {
  test
}

The following code does not work
import { Test } from '@temp/test'

paths: {
"@temp/*": [
        "./.temp/*"
      ]
}

Expected behavior:
import not error tips.
Actual behavior:

Playground Link:
Related Issues:

This seems to work perfectly fine. Both when running tsc and ts-node (with tsconfig-paths registered).
However, there is something around how glob works with folders starting with a dot that makes it harder to work with them. The important point might be to have the ./.folder/**/* added to your include section in tsconfig (even though it seems to work without it for me too). Also, you might want to try adding "baseUrl": "." to your tsconfig.

I detected a problem in my project where Storybook configuration files were not being type-checked because they are located in ./.storybook. This solved it for me:
  "include": ["src", ".storybook/**/*"],


Display issue on LandVoc page

Hi all,
I'm noticing that with the new responsiveness design for the pages, many views are looking pretty awful. Found another one on the LandVoc page for the themes (see below). The images are tiny, basically can't be seen properly + the text in such a narrow column basically makes it unreadable.

Can we fix this?

2 issues here:
1/ description in taxonomies can't be trimmed (like with node content / teasers). - maybe improve later
2/ in those terms someone entered a narrative rather than a definition.
Corrective action: hide all description for now.
of course it can still be improved but first we need a plan and a design


Integrating my own controller?

Can you tell me how I can integrate own controller?
The problem I have right now is I am using a different component which scrolls to "anchors". However the problem with such configuration is that I can go to the anchor/page with my own controller but any mouse scroll references the page that react-full-page was before and not necessarily the page I am now. Is there a way to manually set the current page so my other controllers and this one have the same "currentPageState"?

Hello, I am trying to understand what is it you are struggling with. Can you explain or provide some code which will help me to grasp "mouse scroll references the page that react-full-page was before and not necessarily the page I am now".
And do you have one ore multiple instances of FullPage simultaneously?

I suppose you can use ref to FullPage instance and use scrollToSlide method


Update to newest Helm version for bundled helm.exe

What happened?
During devspace deploy, an error was thrown by Helm. Investigation showed this was caused by an oudated helm.exe binary that DevSpace automatically downloaded into ~/.devspace/bin/helm.exe. The helm.exe version downloaded by DevSpace is v3.9.4 while v3.11.1 is already available.
What did you expect to happen instead?
DevSpace should download the recent Helm version v3.11.1.
How can we reproduce the bug? (as minimally and precisely as possible)

Run devspace deploy in a project that uses Helm
Run %userprofile%/.devspace/bin/helm.exe version.

Local Environment:

DevSpace Version: v6.3.0
Operating System: windows
ARCH of the OS: AMD64
Kubernetes Cluster:
Cloud Provider: local
Kubernetes Version: v1.25.4

Anything else we need to know?

Found the culprit: https://github.com/loft-sh/utils/blob/aa8ef229d561edb3b8cdea22b4b1a61536ce9dbf/pkg/downloader/commands/helm_v3.go#L19


Adds annotations API and macros to axom

Summary

This PR is a feature
It adds an annotations API to support profiling in Axom
It relies on caliper for performance profiling and adiak for capturing metadata
It replaces the old performance profiling macros (AXOM_PERF_MARK_{SECTION,FUNCTION}) with new ones

AXOM_ANNOTATE_BEGIN(name) -- begins an annotation region
AXOM_ANNOTATE_END(name) -- ends an annotation region
AXOM_ANNOTATE_SCOPE(name) -- begins an annotation region that will automatically end at the end of the current scope
AXOM_ANNOTATE_METADATA(key,value,category) -- adds a key/value pair of metadata to the run
Note: Due to how AXOM_PERF_MARK_SECTION was implemented, this adds a bunch of whitespace changes. I'd suggest reviewing with GitHub's "Hide Whitespace"

It incorporates annotations into a few quest examples -- quest_containment, shaping and quest_marching_cubes

We can add these to more of axom in follow-up PRs.

Also adds simple RAII wrapper classes for intializing/finalizing MPI (axom::utilities::raii::MPIWrapper) and the annotations (axom::utilities::raii::AnnotationsWrapper) and incorporates them into a few examples.

We can add these to more of axom in follow-up PRs.

Resolves #142

TODO:

[ ] Check if core still needs an explicit dependency on nvToolsExt, of if this dependency now comes from caliper (when available)
[ ] Add examples of annotations output

[ ] ... with report service,
[ ] ... with the counts service
[ ] ... with the nvtx service

[ ] I got this running with the roctx service, but had trouble viewing the results (w/ chrome's profiler and with perfetto). We should probably resolve this in a follow up PR. I added a<EMAIL_ADDRESS>host-config on rzvernal to see if a newer version would help, but it did not.

I will add a follow-up documentation PR after these changes have been reviewed/accepted.

Thank you!
Documentation is absolutely needed, and I'm fine with it being in a follow-on PR.  Same with examples (second check box).  Same with roctx (third check box).  Please do check on core dependency on nvToolsExt (first check box).  Approved, pending that first check box and my review remarks.


Image resizeMode support

It doesn't seem resizeMode is supported, right?
Do you have plan for this? I know that CSS can do things background-size:contain, but that's only on a background. not sure how to address this while keeping the , but maybe with a container (that would use the Image style) and some CSS transforms on the image?

Image component in react-web support resizeMode property.  See: https://github.com/taobaofed/react-web/blob/master/Libraries/Image/Image.web.js#L34

mmh I've experience cases when it doesn't behave the same. I'll need to figure a simple code to reproduce.

I wonder if this is related to this commit
If so and the new behavior is decide I'll make a pull request.
I found that a default style of { opacity: 1 } was being applied to my image and so when I added renderMode through a style it thought <Image style={styles.imageStyle}/> was an array of styles.
But because Image (in some cases) returns a dom element  instead of a native component <View> the style property isn't processed and becomes 0:[object Object], 1: [object Object]
Anyway the fix was to remove the native check.

good catch. But I don't think we should remove the whitelisting of this..
I'm the author of this change and this was breaking for me: when you use your own react component that accept a style, it should not be overrided by react-web internal, it's way too intrusive. E.g. it breaks the use of libraries like react-canvas.

Still, I don't understand where the bug is, because it looks like Image will just forward the ...props to  so it should already have been style processed, why is this not the case ?

React-dom doesn't support Arrays of style objects. That is a feature of react-native only.

Is the problem with #37 the fact that it tries to process a functor? In which case === 'object' is your problem and not the Array.isArray().
As far as I see it, the purpose of react-web is to provide the exact same API as react-native but with a different target.
In this case a user would expect an Array of styles to work regardless of the underlying implementation of the component (they shouldn't have to care about the target, just their react-native code), especially in this case where the component only breaks under certain conditions (which is clearly unexpected).
I think a better fix is to always process styles, but return the functors results before doing so.

extendCreateElement seems to overrides React.createElement which is going to always be called when you write JSX (<Image/> JSX is transformed into a call to React.createElement(Image, props)), so AFAIK it will always happen when you write <Image/>, this is why I don't understand this.
As you said, react-dom does not support array of style, and react-web shouldn't alter this fact, otherwise it means breaking compatibility with any other react(-dom) library. This should only be fixed for react-web "compatible" components. Now how to detect this belonging to category is the whole question, but I disagree with the idea that this should happen for all elements that goes into createElement whatever if it belongs to react-web or not.
In react-native's context, style is only consumed for React Native UI components (that goes into RN rendering) but it does not happen for non RN UI Component like: <Connect/> of react-redux, <Route> of react-router, <Text> of react-canvas (just to take some simple examples).

I think its possible that we view the role of react-web differently.
My view is that it is a layer of abstraction that allows the development of a primarily react-native application with a html output.
That html output could be supported by react-dom or angular or anything else, it doesn't actually matter. It just so happens that it maps quite well to react-dom.
However I do agree that the underlying render library shouldn't be broken due to invasive changes, but I don't think having consistent react-native features (+ syntactic sugar) and not causing breaks to the react-dom library are mutually exclusive.
In the same way react-web translates <Image/> to a valid react-dom element I expect style arrays [{},{}] to translate to a valid react-dom style object, the whole point of react-web is to do this translation.
As for react-canvas it takes an object (as you would expect from sane developers, they have simply mimicked react-dom's api) so as long as we translate to something react-dom can handle there's no problem there. Is there a gist of breaking code that you could provide? I genuinely have no breaking issues when using react-canvas and processed styles together.
Anyway it feels like we're getting slightly off topic, I'll look to see if I can find a solution that can be agreed upon.
I found there are other breaks with <Image/> and arrays of styles that my fix doesn't fix anyway, e.g. when the style becomes a div it just adds the background image styles as if the styles are always an object, so you can end up with an array of styles with attributes on the array :.
Also in regards to "why doesn't the style processing apply to  itself?" what would that even look like? Custom components are not elements, they return elements.
To make it process the style property passed to a native component we'd have to do the processing in an extendCreateClass call not extendCreateElement.

as far as I remember, one of the problem I encounter with react-canvas was when using <Text style={{ lineHeight: 10 }} /> where 10 gets converted to a "10px" format.
I can try to put a full example to repro this if you want it.

No need, that makes perfect sense. I wouldn't want that to happen either, I am only talking about the flattening step.
However I have found that fixing these lines allows resize mode to work and arrays of styles get applied to the <Image/> correctly for the
.
Which makes it even more bizarre that it doesn't work in the case of

yeah i'm not getting why it doesn't in

Ok I've done some digging and found that its actually another react-native library which expects to manipulate the style of the children it wraps.
What essentially happens is the image.props.style gets processed and flattened and then its parent component decides it needs more styles adding to it, which won't get reprocessed (because the element is already created) and when  is created it will get these new not flattened styles.
Wrapping Component Render -> Create element (Image) -> Flattens image.styles --> wrapping component pushes updates each Child's styles --> Image Render --> create element (img)...
This is reasonable behavior for any react-native library, even react-web does it in  when it creates a div, only when it creates a div element it uses the view constructor so the styles get processed again (which is why the div part works).


No review date in the review section

No review date in the review section

https://github.com/facundoolano/app-store-scraper/issues/13

@fbamidele1 Had the same issue. Just released that the xml representation of the data has the updated attribute. See https://itunes.apple.com/us/rss/customerreviews/id=389801252/xml
So you can just parse this data instead and transform it to JSON afterwards. Hope this helps.


configuration_aliases support

New Terraform changed the way alternative providers should be declared in modules, but this plugin doesn't support it yet unfortunately.
Installation details

[x] IDE version (Help->About->Copy to Clipboard)
[x] intellij-hcl plugin version (Settings->Plugins)
[x] Terraform version (terraform -v)

IDE version:
PyCharm 2021.1.1 (Professional Edition)
Build #PY-211.7142.13, built on April 21, 2021
Licensed to …
You have a perpetual fallback license for this version.
Subscription is active until May 22, 2022.
Runtime version: 11.0.10+9-b1341.41 x86_64
VM: Dynamic Code Evolution 64-Bit Server VM by JetBrains s.r.o.
macOS 11.4
GC: G1 Young Generation, G1 Old Generation
Memory: 1126M
Cores: 16
Non-Bundled Plugins: org.intellij.plugins.hcl (0.7.10), mobi.hsz.idea.gitignore (4.1.0), name.kropp.intellij.makefile (211.6693.108), org.logtalk (1.0.15), org.toml.lang (<IP_ADDRESS>68-211), rocks.blackcat.vcl (0.7.2), idea.plugin.protoeditor (2.3.1)

Plugin version: 0.7.10
Terraform version: 1.0.2
Terraform Configuration Files
terraform {
  required_providers {
    aws = {
      source = "hashicorp/aws"
      configuration_aliases = [ aws.replica ]
    }
  }
  required_version = ">= 0.13"
}

resource "aws_kms_key" "foo" {
  provider    = aws.replica
}

Expected Behavior
aws.replica should be autocompleted and marked as correct
Actual Behavior
It gets "unresolved reference replica" error

seems to be related to https://github.com/VladRassokhin/intellij-hcl/issues/349
which should be reopened

@0xFate I don't see how this is related to #349 - that issue is about just about everything being broken, whereas this is about supporting a new feature.


Clarify that autoscaling.keda.sh/paused annotation value is not important. Only prescence is checked

Essentially, as a new Keda user I spent a long time trying to figure out why autoscaling.keda.sh/paused: false wasn't working as expected.
I opened this https://github.com/kedacore/keda/issues/5215 in keda-core. Until a resolution is decided there, I think this documentation update would be beneficial to avoid end-user communication.
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)

Fixes #
Potentially fixes - https://github.com/kedacore/keda/issues/5215

Thanks! It is not important indeed, but I personally think the current example value still makes more sense though.
We can add the note you suggested, but I'd prefer to stick with true

As discussed here, let's stay with true and add false once it is implemented.
https://github.com/kedacore/keda/issues/5215

Closing because of the above, but thanks for proposing anyway!


feat: support drop event

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add --typescript flag removal to v5 upgrade guide

Description (required)
As discussed on Discord, this PR adds a new section to the breaking changes of the v5 upgrade guide to document the removal of the --typescript flag from create-astro.
Related issues & labels (optional)

Closes #
Suggested label: 5.0

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

en/guides/upgrade-to/v5.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Allow to skip format check

Introduce the notion of commands through comments. Similar to Dependabot.

Useful link to docs: https://docs.quarkiverse.io/quarkus-github-app/dev/commands.html

nice, thanks @The-Huginn for the pointer


Roll Packages from 86d15a65d091 to 7695c4538f6e (4 revisions)

https://github.com/flutter/packages/compare/86d15a65d091...7695c4538f6e
2024-08-16<EMAIL_ADDRESS>[interactive_media_ads] Adds IMA plugin to labeler yaml (flutter/packages#7420)
2024-08-15<EMAIL_ADDRESS>Roll avd_cipd_verison to latest to use the crashreport tool. (flutter/packages#7422)
2024-08-15<EMAIL_ADDRESS>[flutter_adaptive_scaffold] Add breakpoint extension to get the current active breakpoint (flutter/packages#7347)
2024-08-15<EMAIL_ADDRESS>Roll Flutter from cc13cd1f1191 to 99f00a130923 (24 revisions) (flutter/packages#7419)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


D455 does not show IMU module anymore

Required Info

Camera Model
D455

Firmware Version
<IP_ADDRESS>

Operating System & Version
MacOS

Kernel Version (Linux Only)
10.15.7 (19H114)

Platform
PC

SDK Version
2.40.0

I have updated the D455 to FW version <IP_ADDRESS> and am using the RealSenseViewer from v.2.40.0 (MacOS) and it seems that the IMU Module (Motion) is not shown anymore. I can not start gyroscope / accelerometer streams from the code as well.

Is that an intended behaviour?

Hi @cansik   Dorodnic the RealSense SDK Manager made reference in November 2019 to how IMU data could not be accessed yet on MacOS.
https://github.com/IntelRealSense/librealsense/issues/5261#issuecomment-554916553
The D435i, which like the D455 is equipped with an IMU, is not currently supported on Mac, as indicated by the extract below from the SDK's Releases page.

I checked the features Support Matrix document, which Dorodnic also referenced in his explanation.  IMU hardware timestamps are still listed as (No) WIP - Work in Progress - under the MacOS column of the document.
https://dev.intelrealsense.com/docs/supported-features-matrix

Hi @cansik   Dorodnic the RealSense SDK Manager made reference in November 2019 to how IMU data could not be accessed yet on MacOS.
https://github.com/IntelRealSense/librealsense/issues/5261#issuecomment-554916553
The D435i, which like the D455 is equipped with an IMU, is not currently supported on Mac, as indicated by the extract below from the SDK's Releases page.

I checked the features Support Matrix document, which Dorodnic also referenced in his explanation.  IMU hardware timestamps are still listed as (No) WIP - Work in Progress - under the MacOS column of the document.
https://dev.intelrealsense.com/docs/supported-features-matrix

Ok, thank you very much for clarification!

Ok, thank you very much for clarification!

Hi again.  The new librealsense version 2.42.0 has introduced IMU support for MacOS.
https://github.com/IntelRealSense/librealsense/releases/tag/v2.42.0


Driver Error

Hello, I'm a beginner. Today, when I tried to compile the driver, I found that there were some errors in the compilation process of some drivers, which could not be solved. I hope I can get help from big brother here（ If you send it in the wrong place, please forgive me
The error message is as follows: (Build Error Driver Project List)
1：rpiq
2：SerPL011
3：vchiq
4：vchiq_arm_kern
5：vcos_win32_kern
6：vcos_win32_kern_generic

"rpiq" and "SerPL011" driver project：I also disable the 'Spectre' of 'rpiq & SerPL011', However, other errors are displayed after disabling. This error should be an error during link.
error message list:
1: 严重性	代码	说明	项目	文件	行	禁止显示状态
错误	LNK2001	无法解析的外部符号 __stdio_common_vswprintf	rpiq	C:\Users\Administrator\Desktop\windows-drivers\drivers\mailbox\bcm2836\device.obj	1
2:严重性	代码	说明	项目	文件	行	禁止显示状态
错误	LNK1120	1 个无法解析的外部命令	rpiq	C:\Users\Administrator\Desktop\windows-drivers\build\bcm2836\ARM64\Release\Output\rpiq.sys	1
image:

"vcos_win32_kern_generic" driver project：I also disable the 'Spectre' of 'vcos_win32_kern_generic', However, other errors are displayed after disabling. This error should be an error during link.
error message list:
1：严重性	代码	说明	项目	文件	行	禁止显示状态
错误	C2011	“timespec”:“struct”类型重定义	vcos_win32_kern_generic	C:\Users\Administrator\Desktop\windows-drivers\drivers\misc\userland\interface\vcos\win32\vcos_platform.h	139

I tried to delete this structure, but I couldn't solve it。

Thanks for reporting this. CI builds were also failing.
I've pushed a partial fix here: https://github.com/raspberrypi/windows-drivers/commit/e053007032538b647f9b73f565d3b6ba6bcd9baf
It's partial because of the stdio workaround, so I'll keep this issue open until there's a proper fix available.

Thank you very much for the help of @mariobalanica. I will continue to follow up on this issue. Today, I'm re re installing the environment. I'm still wondering if it's my development environment configuration. I'll try to compile it after I install it

You should be able to compile all the drivers now.

Yes, now the 2004 version of the SDK can be compiled and passed. I'll see if the 1809 version of the SDK can be compiled and passed. I'll send the test results later

Reverted the workarounds as they're not required anymore: a8398a59e5c46a9e576f250b177bff097e33545a


update cohort stats

Signed-off-by: Ke Xu<EMAIL_ADDRESS>

Could you add a description and maybe some screenshots?  It's hard to review code without understanding the context and the changes.


Implementing classes using the new ES6 syntax

Hi,
In my PhD research, I am using fastclick in some experiments.
As a result, I detected the existence of a main "class" in the current codebase (FastClick).
So, I decided to rewrite this code to follow the new syntax for classes provided by ECMAScript 2016, which I think is much more readable. All changes are just syntactical. I was also very careful to preserve the code style.
Do you think it is worth to integrate these changes to the project?
Thank you

Thanks for the pull request @leonardo-silva, we won't be accepting the change as we currently do not have a compilation step which turns modern javascript into ES3/5 syntax.


Accessing Data Collection Endpoints via Private Link

Hello,
this article is stating data collection endpoints as possible targets for private link setup, however documentation of Data Collection Ednpoints: Data collection endpoints states following: '''Data collection endpoints also can't be configured over private links.'''. Please can you provide more explanation?

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

ID: 867ebb3d-7036-104c-a2e0-3c8fd7bfcf47
Version Independent ID: d11c262e-7128-37c3-83b7-52a49b7ebf12
Content: Use Azure Private Link to connect networks to Azure Monitor - Azure Monitor
Content Source: articles/azure-monitor/logs/private-link-security.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @guywi-ms
Microsoft Alias: guywild

@MarekLani
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for your dedication to our documentation. This issue has been moved to an internal work item for triage and prioritization. Thanks in advance for your understanding as we investigate to provide the most accurate documentation updates. #please-close
Internal work item: User Story 219968: GitHub issue 117226
Our support team is available to assist you if you have any pressing questions or problems that need to be resolved. You can create a support ticket by visiting the following link: https://azure.microsoft.com/support/create-ticket/


Feature/cassandraconfig

LGTM. Just double checking the InetAddress transformation is just to resolve DNS names instead of hard-coding IP addresses, correct?

Not initially, no. I just wanted to have something more 'correct' than a String for addresses and InetAddress is what Cassandra accepts. Using domain names would be preferable so that's some good fallout.


Problem with embedding C# syntax between delimiters

Hello.
I am writting a simple language extension to provide syntax highlighting for our custom DSL platform in VSCode. I am trying to embed C# highlighting in between single quotes (') and SQL in between double quotes. The problem is with C# highlighting. It seems that VSCode doesn't recognize ending single quote, and keeps the rest of the file in C# highlighting mode. Source of the extension project is on https://github.com/Hugibeer/RhetosDSL with sample test file to show the behavior.
I am not really sure if this is bug in VSCode, maybe C# support for VSCode is treating ending single quote as a start of the character definition, or is this a question of which extension "came first" to evaluate the single quote. I think it is because Sublime Text doesn't have this issue :) If I am doing something wrong, please help me correct this.

VSCode Version: 1.11.2
OS Version: Windows 10

Steps to Reproduce:

Download code
Open Rhetos.vscode folder and start the extensions
Open file Samples\SampleTest.rhe to see that ShortString is not recognizd as keyword.

Thank you

@alexandrudima can you take a look. Here's an example definition from the grammar:
	"csharp": {
			"begin": "'",
			"patterns": [
				{
					"include": "source.cs"
				}
			],
			"end": "'"
		},

This sounds similar to the issue we had with markdown code block language highlighting leaking outside of the code block itself. We eventually switched to begin/while to fix this

@Hugibeer
Most likely you are hitting the fundamental design limitation of TM grammars when including patterns. Once control is inside an embedded state, it and only it owns the parsing until it decides to pop.
If inside the grammar associated with source.cs there is a rule that consumes ', it will consume ', regardless of the end condition you place. I can explain the difference with Sublime via Sublime shipping a different grammar associated with source.cs than VS Code does.
You can try registering the same grammar for source.cs as Sublime does and most likely, the differences will be gone. Better yet, we should check how TextMate works, since it is the de-facto standard having introduced TextMate grammars.


Implement Stable Sorting Algorithm

Modify the datatable's sorting logic to implement stable sort.
Add accompanying unit test.

What kind of change does this PR introduce? (check one with "x")

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
ngx-datatable relies on each browser's native .sort() implementation, which
is not guaranteed to result in a stable sort in across all browsers.
What is the new behavior?
the ngx-datatable sorting implementation would result in a stable sort across
all browsers.
Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Super interesting, thanks for bringing to my attention and for the PR. I did not know about this issue, did some research and very interesting problem. ( https://medium.com/@fsufitch/is-javascript-array-sort-stable-46b90822543f )
Thanks for your contribution!


perlPackages.Tk: fix build with clang 16

Description of changes
The jpeg configure script fails to detect clang as a functioning C compiler because it uses a test with a main that returns an implicit int, which results in an error with clang 16.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg build perlPackages.Tk

More changes have been proposed upstream:

https://github.com/eserte/perl-tk/pull/94

More changes have been proposed upstream:

jpeg: more fixes for clang 16 eserte/perl-tk#94

Once those are merged, I assume the nixpkgs changes can be dropped. However, I wonder if nixpkgs should be using the nixpkgs libjpeg instead of the vendored one.


update the fuzzer target.

I used the fuzzer to discover #2

Looks great 👍

dang I missed this but you forgot to add the index to the world index, and the bodies to the bodies map. I'll fix it in the next commit but it doesn't affect #2 at all.


Improvement in Moving Forward From Here module

Describe the Issue
Currently, the instructions in Moving Forward From Here module, mention about Managing Packages with NPM subsection, which is a subsection of a completely different course ( i.e. Back End Development and APIs course ).
At the moment, those instructions in Moving Forward From Here module, only mention the name of the "Managing Packages with NPM" subsection and no link has been provided to that particular subsection.
Keeping that in mind, I believe to make it easier for the user to navigate to the Managing Packages with NPM subsection from within the instructions of Moving Forward From Here module, a link to the Managing Packages with NPM subsection should be provided within the Moving Forward From Here module.
Hence, taking the above point into consideration, I believe the last line in the first paragraph of Moving Forward From Here module, should be changed from:

The "Managing Packages with npm" section covers npm in more detail.

To

The Managing Packages with NPM  section covers npm in more detail.

Affected Page
https://www.freecodecamp.org/learn/front-end-development-libraries/react-and-redux/moving-forward-from-here
Expected behavior
The last line in the first paragraph of Moving Forward From Here module, should be changed to:

The Managing Packages with NPM  section covers npm in more detail.

Screenshots

can i can solve the above issue?

Hello! can I solve this bug.

Personally, I would entirely remove that sentence.
It is not common to maintain the links across the challenges, and it does not really provide useful information, as Campers are not expected to go directly to the npm section.
On top of that, I would remove the mention of npm in all sentences - npm != React; there is nothing special about import statements, and there is no direct link between npm and import statements.

Hello, Can I take up issue?

Hello, can I resolve this issue?

@AnonSar Iooks decent. I would leave in the bit about:

The code should look similar, except for the use of import statements (these pull in all of the dependencies that have been provided for you in the challenges).


Stabilization of TestDNSMonitoringNsc

Description

Motivation and Context

How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ ] Have not tested

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Fix moved into https://github.com/networkservicemesh/networkservicemesh/pull/1521


Shows nothing in the FBX output

I am totally a beginner in this area. I am trying to get the 3D models from a android game(for collection
). It seems that the 3D models is split in the asset list. All exported 3D object show nothing at FBX viewer. Can it be fixed? OR It is encrypted? Here is a picture for reference... I am sorry for poor English and knowledge.

Please sort by size then choose the model you want, import it to 3ds MAX

Oh, I see! Thank you very much!


Master

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: v-kents sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

:x: Validation status: errors

File
Status
Preview URL
Details

globalization/toc.yml
:x:Error

Details

:x:Error

Details

globalization/toc.yml

[Error] W:\yr0r-s\globalization\toc.yml is not a valid TOC File: W:\yr0r-s\globalization\toc.yml is not a valid TOC file.
[Error] W:\yr0r-s\globalization\toc.yml is not a valid TOC File: W:\yr0r-s\globalization\toc.yml is not a valid TOC file.
[Error] Unable to load file: toc.yml via processor: ExperimentalTocDocumentProcessor.
[Error] Unable to load file: toc.yml via processor: TocDocumentProcessor.

[Error] W:\yr0r-s\globalization\toc.yml is not a valid TOC File: W:\yr0r-s\globalization\toc.yml is not a valid TOC file.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

globalization/input/font-technology.md
:white_check_mark:Succeeded

globalization/software-internationalization.md
:white_check_mark:Succeeded

globalization/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Allllllll


DevOps Link in Article is Broken (leads to 404 error)

DevOps link under "Multi-Tenant app with database-per-tenant" (anchored link here: https://docs.microsoft.com/en-us/azure/azure-sql/database/saas-tenancy-app-design-patterns#d-multi-tenant-app-with-database-per-tenant) is broken and does not lead anywhere.

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

ID: 54f6795f-45de-12da-60dd-cc7be805548c
Version Independent ID: 7c3571de-62fa-09a2-0cb1-545a6e14f8fe
Content: Multi-tenant SaaS patterns - Azure SQL Database
Content Source: articles/azure-sql/database/saas-tenancy-app-design-patterns.md
Service: sql-database
Sub-service: scenario
GitHub Login: @stevestein
Microsoft Alias: sstein

@PrometheusRising1 Thank you for your feedback. We are looking into this issue and will get back.

@stevestein Please see customer feedback above. When I clicked on the DevOps link in this document, I landed to a 404 Page not found. Could you please help update the document with a correct link. Thanks

@PrometheusRising1 - thanks for pointing out the broken link.
The correct link is: https://docs.microsoft.com/azure/devops/user-guide/what-is-azure-devops
#please-close


Handler for text content

Issue #21 shed exposed an issue that text content is not handled at all. Because of this, any error messages from Trello are not recorded. This means that the user can't know what the error is without a tool like Fiddler.
To resolve this, a handler for text content should be created. Its function would be to merely log any messages that come through.

Romain Meisterhans, I think this would help your scenario. Your thoughts?

Romain Meisterhans (Meisty on BitBucket) wrote:

Yeah I think it would be a good solution to save time with investigation with this type of issues.

Please see version 1.0.2 for this feature. When an error is returned by Trello, you should now see it in the log. Please make sure you read the Logging wiki page.


Update initial refactor

Remove Phone, Email, and Address field from person.
Did not update class diagrams.
Did not remove commands.

Codecov Report

Merging #33 (a7d10c8) into master (35842b8) will decrease coverage by 2.30%.
The diff coverage is 58.82%.

@@             Coverage Diff              @@
##             master      #33      +/-   ##
============================================
- Coverage     72.15%   69.85%   -2.31%
+ Complexity      399      360      -39
============================================
  Files            70       67       -3
  Lines          1232     1118     -114
  Branches        125      104      -21
============================================
- Hits            889      781     -108
+ Misses          311      308       -3
+ Partials         32       29       -3

Impacted Files
Coverage Δ

.../java/seedu/address/logic/commands/AddCommand.java
100.00% <ø> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
66.66% <ø> (-16.67%)
:arrow_down:

...in/java/seedu/address/logic/parser/ParserUtil.java
95.23% <ø> (-1.99%)
:arrow_down:

.../java/seedu/address/model/util/SampleDataUtil.java
20.00% <0.00%> (ø)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <ø> (ø)

...c/main/java/seedu/address/model/person/Person.java
96.29% <75.00%> (-1.33%)
:arrow_down:

...java/seedu/address/logic/commands/EditCommand.java
95.91% <100.00%> (-1.10%)
:arrow_down:

...a/seedu/address/logic/parser/AddCommandParser.java
100.00% <100.00%> (ø)

.../seedu/address/logic/parser/EditCommandParser.java
90.00% <100.00%> (-2.31%)
:arrow_down:

.../java/seedu/address/storage/JsonAdaptedPerson.java
100.00% <100.00%> (ø)

... and 1 more

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 35842b8...a7d10c8. Read the comment docs.


phantomjs hangs while opening the url using selenium?

Which version of PhantomJS are you using? Tip: run phantomjs --version.

phantomjs version - 1.9.7 and also in 2.1.1
2. What steps will reproduce the problem?
from selenium import webdriver
from selenium.webdriver.common.desired_capabilities import DesiredCapabilities

dcap = dict(DesiredCapabilities.PHANTOMJS)
dcap["phantomjs.page.settings.userAgent"] = ('Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Ubuntu Chromium/28.0.1500.71 Chrome/28.0.1500.71 Safari/537.36')
driver = webdriver.PhantomJS("/usr/local/share/phantomjs-2.1.1-linux-x86_64/bin/phantomjs",service_args=['--ignore-ssl-errors=true','--ssl-protocol=any'],desired_capabilities=dcap)
driver.get("https://www.walmart.com/ip/Enfamil-Newborn-Formula-Powder-33.2-oz-Refill-Box/16940597")

print driver.current_url
driver.close()

Which operating system are you using?

ubuntu 14.04
4. Did you use binary PhantomJS or did you compile it from source?
Binary
5. Please provide any additional information below.
But the similar code works  fine in windows .I have problem only in ubuntu.
I am new to these github things. Kindly tell me if any other details needed.

@viveksrinivasanss yes is not a valid option for --ignore-ssl-errors.  should be true or false.
--ignore-ssl-errors=<val>            Ignores SSL errors (expired/self-signed certificate errors): 'true' or 'false' (default)

yeah I tried true as well. Its not working...Is there any other thing I should take care of....can someone replicate the issue?

@viveksrinivasanss both the npm package and official download works for me on 64bit ubuntu 16. the official ubuntu package looks broken.
package details
$ apt-cache show phantomjs
Package: phantomjs
Priority: extra
Section: universe/web
Installed-Size: 919
Maintainer: Ubuntu Developers<EMAIL_ADDRESS>Original-Maintainer: Dmitry Smirnov<EMAIL_ADDRESS>Architecture: amd64
Version: 2.1.1+dfsg-1
Depends: libc6 (>= 2.15), libgcc1 (>= 1:3.0), libqt5core5a (>= 5.5.0), libqt5gui5 (>= 5.0.2) | libqt5gui5-gles (>= 5.0.2), libqt5network5 (>= 5.3.0), libqt5printsupport5 (>= 5.0.2), libqt5webkit5 (>= 5.2.0), libqt5widgets5 (>= 5.0.2), libstdc++6 (>= 4.4.0)
Filename: pool/universe/p/phantomjs/phantomjs_2.1.1+dfsg-1_amd64.deb
Size: 307358
MD5sum: ad67963b0d5e870854125d18c4d33e65
SHA1: 0f5302f161c72c2ce47059b59edb1b13d1c0973a
SHA256: 231199593b127eb1000a44f7e0efc6cf26269dcb3aa0c08f6673a657faffc421
Description-en: minimalistic headless WebKit-based with JavaScript API
 PhantomJS is a headless WebKit with JavaScript API. It has fast and native
 support for various web standards: DOM handling, CSS selector, JSON, Canvas,
 and SVG.
 .
 PhantomJS is an optimal solution for headless testing of web-based
 applications, site scraping, pages capture, SVG renderer, PDF converter
 and many other use cases.
Description-md5: bf355eb2a4481e2b6c41bfbf308d318e
Homepage: http://www.phantomjs.org/
Bugs: https://bugs.launchpad.net/ubuntu/+filebug
Origin: Ubuntu

i get the following error when i execute your script with the official ubuntu binary
Traceback (most recent call last):
  File "/home/jesg/sel-url-test.py", line 10, in <module>
    print driver.current_url
  File "/home/jesg/.local/lib/python2.7/site-packages/selenium/webdriver/remote/webdriver.py", line 486, in current_url
    return self.execute(Command.GET_CURRENT_URL)['value']
  File "/home/jesg/.local/lib/python2.7/site-packages/selenium/webdriver/remote/webdriver.py", line 233, in execute
    self.error_handler.check_response(response)
  File "/home/jesg/.local/lib/python2.7/site-packages/selenium/webdriver/remote/errorhandler.py", line 165, in check_response
    raise exception_class(value)
selenium.common.exceptions.WebDriverException: Message: Error - Unable to load Atom 'execute_script' from file ':/ghostdriver/./third_party/webdriver-atoms/execute_script.js'

so it looks like the official ubuntu package is broken.  is this the error you are seeing?

i dont get any error...it just hangs while it tries to load the url.....hangs forever.......
I used the phantomjs for linux  64 bit from the following link
http://phantomjs.org/download.html
I hope its the correct link to follow.

Same issue here

Looks like there's a similar issue here

The current_url method always crashes on GitHub.com. Here's the SSCCE:
$ phantomjs --version
2.0.0
$ python3 -c 'from selenium import webdriver; driver = webdriver.PhantomJS(); driver.get("https://github.com"); print(driver.title); print(driver.current_url); driver.quit()'
The world's leading software development platform · GitHub
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/selenium/webdriver/remote/webdriver.py", line 493, in current_url
    return self.execute(Command.GET_CURRENT_URL)['value']
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/selenium/webdriver/remote/webdriver.py", line 238, in execute
    self.error_handler.check_response(response)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/selenium/webdriver/remote/errorhandler.py", line 193, in check_response
    raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.WebDriverException: Message: {"errorMessage":"Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: \"script-src assets-cdn.github.com\".\n","request":{"objectName":"","statusCode":200,"headers":{"Cache":"no-cache","Content-Type":"application/json;charset=UTF-8"}}}
Screenshot: available via screen

@essandess the issue you are experiencing should be fixed in ghostdriver 2.0.0.

phantomjs --version says 2.0.0, and this is with the embedded ghostdriver on a native compile.

@essandess phantomjs 2.0.0 embeds ghostdriver 1.2.0 which has issues with sites that have csp (content security policies).
https://github.com/ariya/phantomjs/blob/2.0.0/src/ghostdriver/lastupdate


Refactor Doubles into Money

Coerces into whole numbers
Creating big BigDecimals greater with greater that 14 digits from doubles can have issues with precision but doesn't feel too likely

[#177988195]

I could've refactored this further to demand only an integer constructor that way we would never have non-whole numbers.

Thanks for taking this on! It's a pretty wide-reaching change but overall I really like your approach of introducing a Money class

@andrewedstrom and I worked on a branch that just didn't use Money at all and just switched to int/Integer but that has problems since sometimes we actually want income information to be null 😞
I think this PR is good enough for now but if we think there's a better way that's within reach I'm all ears.


Reduce dependencies on on general-purpose C++ code requiring linking

The codebase begins with an "inheritance" of a C++ utilities collection in the src/util directory. Some of that code requires linking, and is not header-only; I would rather the users of this library not be encumbered by those objects.
After getting rid of many dependencies before the initial commit, we're left with two:

poor_mans_reflection.cpp
stack_trace.cpp

The former should be easier to remove I believe, the latter - well, mostly emotionally painful I guess... I like having stack traces on my exceptions. But for the interest of simplicity we'll drop those until we can just depend on Polukhin's upcoming Boost.Stacktrace,

This was fixed by the last commit, not sure why it didn't auto-close...


Stepping: Allow warning text to be displayed without scrolling to top

This is something that causes stress when developing XSLT in Stroom.  Warning text (eg from stroom:lookup) is displayed at the top of the top window (yellow area on scroll bar) but user must scroll up to see detail.
Would be better if just hovering mouse over the yellow area would display error without having to scroll up, to prevent losing place where editing is taking place.

Closing as this will be covered by #1941


Weekly task will renew daily

Version
What version of Todoloo do you have installed?.
v11.0.2
Describe the bug
A clear and concise description of what the bug is.
I set some tasks under a weekly group, but the tasks will auto renew with daily group.
I use Chinese and not sure if local time affects.
I copy the tasks from one character to another, in the todoloo.lua. not sure if it makes it happen. Because for some characters, they need to do the same weekly tasks you know.
To Reproduce
Include the steps to reproduce the behavior.
I meet this problem at the 2nd day using this addon, and I re-check all the finished tasks. Now they are all un-checked now at the 3rd day. I think they are wrongly auto un-checked, because weekly tasks should renew on Thursday.
Screenshots
If applicable, add screenshots to help explain your problem.

I read the ResetManager.lua, and found the following:
local function ShouldResetGroup(group, previousDailyReset, previousWeeklyReset)
local lastResetPerformed = Todoloo.Config.Get(Todoloo.Config.Options.LAST_RESET_PERFORMED)
if group.reset == TODOLOO_RESET_INTERVALS.Daily and lastResetPerformed < previousDailyReset then
-- reset daily groups
return true
elseif group.reset == TODOLOO_RESET_INTERVALS.Weekly and lastResetPerformed < previousDailyReset then
-- reset weekly groups
return true
end
I guess the bold part should be < previousWeeklyReset then

Hi @mhx3
Well spotted. I've fixed the bug and am working on a release right now. And update sohuld be available within 10-15 minutes.
Thank you for your bug report and your debugging ;)


Matérn-5/2 kernel

Hello,
Is there any easy way of implementing a Matérn-5/2 kernel with celerite2?

It isn't straightforward because of technical reasons about how the core algorithms are implemented. Depending on your use case you might be able to use tinygp's quasiseparable solver which is a generalization of the method used in celerite2.

I have tried to use tinyGP, but the nature of what I am trying to implement makes it very hard to use jax (as far as I understand) in an efficient manner (and if I solve them I would then run into the same padding issue as the one discussed in tinyGP's discussion 108.
This is not the proper repository to discuss/ask this, but have you happen to have a version of the quasiseparable solver that is not written with jax? I have a fork of tinyGP from which I replaced some jax calls with numpy equivalents, but that is far from efficient/optimal

Thanks for all the help!
I guess that I will run into some limitations, maybe even at the level of how the JIT compilation is "stored" (sorry, very new to jax and all of this JIT stuff). I am trying to fit GPs in parallel to N datasets (N ~ 100) datasets each with a different size.

I'm not aware of any non-JAX implementations of this solver. Replacing the JAX functions with numpy definitely isn't a good idea! It'll be far far slower. You'd need to implement the backend in a compiled language which would certainly be possible, but a major project.

It was slower, but not terribly so. By any chance are you planning to write up the proofs for the quasi-seperable solver in any form (either docs or a paper)? It would be easier to follow that than to re-derive or reverse-engineer your implementation :)


Timing issues

Long story short, this microsd card is pretty slow (and tiny) and am having a hard time getting it to play along with the sd-driver (with #48, #49, #50).
For some reason I keep hitting some weird IO error that I haven't been able to pinpoint yet as every time I add a printf to log some values the bug disappears and the program is able to reach the end thanks to the delay it introduces.
Here's a short log of the commands, I haven't been able to track down what command causes this so I hope you may have any hint.
CMD:24 	 arg:0x141a00 	 Response:0x0
CMD:13 	 arg:0x0 	 Response:0x0
R2: 0x0
CMD:23 	 arg:0x2 	 Response:0x0
CMD:25 	 arg:0x141c00 	 Response:0x0
CMD:17 	 arg:0x100800 	 Response:0x0
_wait_token: timeout
Read timeout
CMD:17 	 arg:0x100800 	 Response:0x0
CMD:23 	 arg:0x4 	 Response:0x0
CMD:25 	 arg:0x144000 	 Response:0x0
CMD:24 	 arg:0x144800 	 Response:0x0
CMD:13 	 arg:0x0 	 Response:0x0
R2: 0x0
CMD:24 	 arg:0x100800 	 Response:0x0
No response CMD:13
No response CMD:13
No response CMD:13
No response CMD:13

No response CMD:24
No response CMD:24
No response CMD:24
No response CMD:24

After some more digging I've noticed that the problem is connected to the use of CMD25, I've been able to get the whole program to work reliably by replacing its use with multiple CMD24 at the expense of the write troughtput. I think some more syncing is needed somewhere.
// trying to write a single 2048 byte block
CMD:23 	 arg:0x4 	 Response:0x0
CMD:25 	 arg:0x143000 	 Response:0x0
resp 5 4 2048
resp 5 3 2048
resp 5 2 2048
resp 5 1 2048
// the card is dead from now on, _every_ command is illegal
Illegal command CMD:23

@LemonBoy - If the card is slow, I would recommend using CMD24 instead CMD25, as that is for perfomance.
Also, for timing critical issue it will be helpful if you can share logic analyzer trace with CMD25 vs CMD24. Debug prints won;t help so can have them disabled.

If the card is slow, I would recommend using CMD24 instead CMD25, as that is for perfomance.

Sure, but I think the intermittent CMD25 failures should be treated as a bug.

Also, for timing critical issue it will be helpful if you can share logic analyzer trace with CMD25 vs CMD24.

I don't have access to a logic analyzer at the moment, that's why I tried to debug this problem with the tools at hand but ran out of time.

Also please share the card details, I will see if i can collect one similar one.

It is a no-name 128Mb card that was probably sourced from some chinese website, I think you can easily find some equally shitty ones :)

Please check if PR #59 resolves CMD25 timing issue for you. sorry I still don't have 128Mb card so cant test or debug.

@LemonBoy - Please, let us know if timing issue still exists with master branch.

Sorry for the delay, I've just tested the master branch and it works just fine.
Thanks for looking into this!


FancyZones not saving layout and then not working at all

Microsoft PowerToys version
0.45.0
Running as admin

[x] Yes

Area(s) with issue?
FancyZones, FancyZones Editor
Steps to reproduce
After updating to 0.45.0 using the auto-update, whenever I restarted my computer the FancyZone layouts would reset, I googled this issue and found that it was a solved thread here before (#7790), so I navigated to the FancyZones directory and opened one of the settings files to see if I could find the issue by myself, and it seems I had the same problem (monitor GUID is all 0000's) so I uninstalled PowerToys from my system and then deleted the PowerToys folder in %LOCALAPPDATA% and then reinstalled from the GitHub EXE, and after that the layouts reset, and the whole FancyZones functionality is not working (the overlay doesn't appear nor do apps snap into the size), this is happening on both of my monitors, 1x 1920x1080 Landscape and 1x 1080x1920 Portrait monitors, Before the update it was working just fine. Another thing to note is that I never ran as administrator before (using the "Run as Admin" option in PowerToys) but turning this option on changes nothing.

I open PowerToys and go to the FancyZones tab
"Launch Layout Editor"
I disable FancyZones on my landscape monitor and enable "Rows" on my portrait monitor and edit so that there is 2 rows and 0 pixels around zones ---> Save
I exit out of the FancyZones layout editor
Absolutely nothing happens and FancyZones doesn't enable me to snap windows

✔ Expected Behavior
FancyZones to work and save layouts.
❌ Actual Behavior
FancyZones is resetting previously saved layouts and not working at all.
Other Software
No response

I followed the steps suggested in #7790 and commented with attachments for v0.45.0: https://github.com/microsoft/PowerToys/issues/7790#issuecomment-915095564
Hope this might help.

Seems to be the same problem as reported in issue #13069... Monitor layouts all default to Priority Grid upon restart; custom grids are still there, but need to be manually applied to each monitor after each boot.
Two monitors; PowerToys 0.45.0; Windows 10 Pro, Version 21H1, Windows Feature Experience Pack 120.2212.3530.0

Having the same issue. Fancy Zones "defaulting" to priority layout and forgetting the last setup on reboot. Tried deleting the profile and running power toys using admin mode with no success.  Windows 10 21H1, desktop resolution 5120 x 1440.

I have the same issue, when the computer start all the layouts are set to defaults.

@Happylama25 @shigadeyo @iob @germanbour The team is working hard to solve this issue.

Duplicate of #13069


Cannot target .Net 6

Current solution only targets .Net 5 and does not work when referencing .Net 6.
@gregwiechec seems like this could be a quick update, but we're having issues getting the js to load pulling this up statically.

I just reference this in my solution on .NET 6, and it works like a charm. What error did you get @justinrichwillowtree ?

Interesting.  The reference loads, but the javascript does not inject into the page correctly, resulting in invalid behavior.  Maybe just a configuration issue, but we also worked through updating our CMS from 11 to 12 manually.

EPiServer.Find.Blocks lowest version of 14.2.4 requires Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation >= 6.0.0, however Alloy.HideTabs v2.0.0 requires >= 5.0.12 && < 6.0.0 of the package. I have to choose one of the packages over the other so it really would be nice to have a net6 update so I know I can have both.


Merge release7x again

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.

Not sure how to debug this further, but the failing test passes if we use an anonymous class for this lambda:
https://github.com/gradle/gradle/blob/4fed83ebb6bc991f754eb374823d47803fc18b1e/subprojects/dependency-management/src/main/java/org/gradle/api/internal/catalog/AbstractExternalDependencyFactory.java#L76
The failing test which exercises this code has removed the @UnsupportedWithConfigurationCache annotation in release, which makes this serialization error apparent in 8.0 but not in 7.6.
I was expecting some kind of configuration cache report but did not see one.

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.


fix some typo in md files

Describe what this PR does / why we need it
fix typo issue
Does this pull request fix one issue?
no

Describe how you did it
typo
Describe how to verify it
typo
Special notes for reviews
no

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.郑恺 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@MageekChiu  hi  would you like to help review the PR? thanks


Carthage support only works on "master", not "v1.2.0" tag

ie. using the line github "kishikawakatsumi/KeychainAccess" doesn't work but github "kishikawakatsumi/KeychainAccess" "master" in the Cartfile does.
I'm using Carthage 0.6.5 and Xcode 6.3.1.

I had uploaded pre-built binary for some major releases in last week. (See https://github.com/kishikawakatsumi/KeychainAccess/releases) If you're saying is that it is just downloaded but it will not be build. It's correct behavior.
Carthage skip build phase if it found pre-built binary for the tag you specified. That's why it will be built only when specify "master" because there are not pre-built binary for "master".
If you do not want to use pre-built binary (want to always build from source), you can use --no-use-binaries option with carthage command. Like this: carthage update --no-use-binaries.  I hope that answers your question?

yup, that's exactly what happened. My bad! Thanks very much for the detailed reply!


Subtitles are not updated in PAUSE mode

Stop YouTube video and forward it by the Right Arrow on the keyboard.
Subtitles are not updated.

Hello @arcadeus !
Thanks for noticing the error, it really does exist.
However I'm not sure if I can fix it. The problem is, that the YouTube Iframe Api doesn't have an event for seeking, it only has events for playing, stopping and buffering. Now it works like the following: when the play event is triggered, the script starts an interval check (it runs every 500 milliseconds) and gets the current time from the api. And when the stop event occures, the script cleares the interval.
We could do the same thing and run the current time check even in the stopped state. But I think that would be a bit too much. Imagine, that there 50 embedded videos on a page. Now it only checks the time and renders the subtitle on the video which is currently playing (usually I think a person used to watch only one video at a time). Otherwise it would check all the videos constantly of course and most of the time there wouldn't be any change at all. Only in this case, when the video is paused and someone presses an arrow or seeks with the mouse.
My opinion is, that this is something we can live with, and I think the script shouldn't do anything until the video is not playing.
What do you think, should we fix this? I would appreciate hearing your thoughts about the issue!

"Now it works like the following: when the play event is triggered, the script starts an interval check (it runs every 500 milliseconds) and gets the current time from the api. And when the stop event occures, the script cleares the interval."
Could you please point out the lines in the code where those things are done? I'm trying to understand how it works.

Hello @siloor !
I think that the best solution is:

add a boolean option workInPause (default: false),
don't stop the timer when workInPause is true,
warn users about performance drawbacks.


Improve Node.js Tools for Visual Studio support

Now that Node.js Tools for Visual Studio has graduated to 1.0, improve support for all the new project templates and iron out any inconsistencies.

This includes underlined warnings and code navigation.


Bump KEB to 1.1.4

Description
Changes proposed in this pull request:
Related issue(s)

/kind enhancement


Current Error Handling does not return false on logging errors

There are two types of errors I've run into:

Network Errors:
Here I created an app to log to a remote Fluentd instance.
private static final FluentLogger LOG = FluentLogger.getLogger("audit", "<IP_ADDRESS>", 22224);
However, I forgot to start the remote Fluentd instance before running the app, so I got Network Errors
java.net.ConnectException: Connection refused      at java.net.PlainSocketImpl.socketConnect(Native Method)      at java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:345)      at java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:206)      at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:188)      at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)      at java.net.Socket.connect(Socket.java:589)      at org.fluentd.logger.sender.RawSocketSender.connect(RawSocketSender.java:85)      at org.fluentd.logger.sender.RawSocketSender.reconnect(RawSocketSender.java:98)      at org.fluentd.logger.sender.RawSocketSender.flush(RawSocketSender.java:206)      at org.fluentd.logger.sender.RawSocketSender.send(RawSocketSender.java:195)      at org.fluentd.logger.sender.RawSocketSender.emit(RawSocketSender.java:157)      at org.fluentd.logger.sender.RawSocketSender.emit(RawSocketSender.java:139)      at org.fluentd.logger.sender.RawSocketSender.emit(RawSocketSender.java:134)      at org.fluentd.logger.FluentLogger.log(FluentLogger.java:101)      at org.fluentd.logger.FluentLogger.log(FluentLogger.java:86)      at LoggerInfra.log(FluentdLoggerApp.java:27)      at LoggerCaller.testLogger(FluentdLoggerApp.java:79)      at FluentdLoggerApp.main(FluentdLoggerApp.java:112)

There error was printed to console (expected because of limited error handling #15), but the log() call returned true instead of false which hid the internal failure.
Result of logic in RawSocketSender.java#L204 which simple logs the error message to LOG.error().

A second error is that I ran out of buffer space. In this setup I had my app logging to a local Fluentd instance and buffering to file. The local instance then forwards the logs to a remote instance. However the remote instance was not running, so the local buffer space eventually got filled up. Again the log() call returned true and the stack trace was printed out to terminal.

How about using fluency instead of fluent-logger-java?
We are now considering replacing fluent-logger-java with fluency.

@obokaiwele

1

You can set an ErrorHandler if you want to do something other than just logging like
        FluentLogger logger = FluentLogger.getLogger(null);
        logger.setErrorHandler(new ErrorHandler() {
            @Override
            public void handleNetworkError(IOException ex)
            {
                // Notify me via email or something
            }
        });

As for the return value of log(), it returns true if it can store an event in memory since fluent-logger is going to retry to send the buffered event later.

2

I guess fluent-logger didn't notice the buffer full of the Fluentd.
in_forward/out_forward plugins of Fluentd support ACK response http://docs.fluentd.org/v0.12/articles/out_forward#requireackresponse. Without this feature, clients of in_forward plugin can't notice the failures of sending to in_forward plugin.
Unfortunately, fluent-logger doesn't support the feature and there is no plan to support it for now. But Fluency, which is yet another fluent logger, supports it https://github.com/komamitsu/fluency#enable-ack-response-mode. On the other hand, Fluency doesn't support custom error handler, but I'll implement the feature later.

Unfortunately, fluent-logger doesn't support the feature and there is no plan to support it for now. But Fluency, which is yet another fluent logger, supports it https://github.com/komamitsu/fluency#enable-ack-response-mode. On the other hand, Fluency doesn't support custom error handler, but I'll implement the feature later.

Fluency 1.4.0 now supports custom error handler and user can know and handle a send error.


Is the nativejson-benchmark still maintained?

Since,

the repo has not seen updates since August, 2019.
the benchmark does neither build nor runs due to variety of problems (see last bullet)
https://travis-ci.org/miloyip/ builds have been failing for two years now
there have been some major issues reported, not addressed (e.g. #102)
there have been important fixes discussed and proposed, not addressed or merged either (e.g. https://github.com/miloyip/nativejson-benchmark/issues/102#issuecomment-385033311, https://github.com/miloyip/nativejson-benchmark/issues/102#issuecomment-534021863, #123)

has this project been orphaned?
/cc @miloyip

I'm wondering this myself.  I just tried building it, with the hopes of submitting a PR to update it to use a newer json-c, and ran into the problems talked about in #102.
I also realized that it doesn't actually build any of the thirdparty libraries using their own build procedures, which is a pretty big problem all by itself. :(
@miloyip, I see that you've merged some changes as recently as February.  Do you have time to merge other PRs and make other changes?  Or, are you willing to grant some other people access to do that instead?

So, no answer here for a year and a half... sounds like the repository is kind of abandoned :-(

Seems like it!


Migration instructions from an old version of Stacked

Describe the missing piece of documentation
Hi...I have two projects using Stacked.  I can't imagine ever doing another Flutter project without it. So, thanks!
With that said, one of the projects is 2+ years old.  So old, in fact, I don't even have any Stacked-related dependencies in my pubspec.yaml file.  With that said, are there any directions/documentation on how to migrate to the latest version of Stacked?
Thanks in advance.

Even if there is no official documentation, maybe someone can provide a few bullets/steps on how I should approach this.

Hey @pieterbergmans,
If it was before the package existed then there's no migration guide.
The basics would be

Replace all the BaseViewModels with the ViewModels from Stacked
Create an app.dart file and register the services as dependencies
Register all the routes in the app.dart as well

You can generate an empty stacked project to see the setup


Adopt action nomenclature

This pull request updates the nomenclature from StatelessFunction to Action.
Latest TCKs are passing.
Minor change:
Redundant proto files have been removed. They are fetched from cloudstate master during proto builds. Users can execute the fetch protos script if they want to see the files locally.

@GratefulTony great! Thanks for working on this.

Thanks @GratefulTony i am happy to see you back

also added codestyle check to travis.

I have cleaned out the extra comments and restored the protobuf directory structure so the files have somewhere to go when downloaded. also added the exit hook to the tck script... @marcellanz tck should work for you now that the directory structure has been restored.

thanks @GratefulTony. also, thank you for creating up follow-up issues.

I have cleaned out the extra comments and restored the protobuf directory structure so the files have somewhere to go when downloaded. also added the exit hook to the tck script... @marcellanz tck should work for you now that the directory structure has been restored.

@GratefulTony works for me now, thank you.

@GratefulTony sorry for not acting on this PR sooner, I was absorbed with work by day and night for the last couple of weeks. I'll merge this now and we can going on on any issues arise. Thanks again for your work @GratefulTony 🎉😀

Thanks @marcellanz!

also thanks for your help @dcirne 👍

@GratefulTony sorry for not acting on this PR sooner, I was absorbed with work by day and night for the last couple of weeks. I'll merge this now and we can going on on any issues arise. Thanks again for your work @GratefulTony tadagrinning

Thanks @GratefulTony
Great work guys!!


Improve performance of performing converges

See also #111 which is a research spike where we'll document any reasons converges may currently be slow.

Closing due to issue age.


Question: can gulp watch only for addition/removal of files?

Example usage:
Task: Using gulp-inject to inject js/**/*.js files into index.html.
Problem: If I watch for changes in the js/**/*.js glob, this would mean that I'm running the inject task each time any of the .js files' content is changed, which is unnecessary, since the paths don't change.
Question: How can I create a watcher that would only watch for addition/removal of the files in the glob?
Thanks!

Support questions should be asked on StackOverflow.

Oh, ok, sorry about that!


Fuse transposition with linalg_ext.scatter

Sample IR shown below produces two dispatches. For linalg_ext.scatter we need to support fusion with transposition and collapse / expand. We do this as arg0 could be created via a concatenation of contiguous memory.
func.func @main(%arg0: tensor<2x4x?x16x4x128xf16>, %arg1: tensor<?x1xi32>, %arg2: tensor<?x2x16x4x128xf16>) -> tensor<?x2x16x4x128xf16> {

    %c2 = arith.constant 2 : index
    %dim = tensor.dim %arg0, %c2 : tensor<2x4x?x16x4x128xf16>

    %empty = tensor.empty(%dim) : tensor<4x?x2x16x4x128xf16>
    %transpose = linalg.generic {indexing_maps = [affine_map<(d0, d1, d2, d3, d4, d5) -> (d0, d1, d2, d3, d4, d5)>, affine_map<(d0, d1, d2, d3, d4, d5) -> (d1, d2, d0, d3, d4, d5)>], iterator_types = ["parallel", "parallel", "parallel", "parallel", "parallel", "parallel"]} ins(%arg0 : tensor<2x4x?x16x4x128xf16>) outs(%empty : tensor<4x?x2x16x4x128xf16>) {
    ^bb0(%in: f16, %out: f16):
      linalg.yield %in : f16
    } -> tensor<4x?x2x16x4x128xf16>

    %collapsed = tensor.collapse_shape %transpose [[0, 1], [2], [3], [4], [5]] : tensor<4x?x2x16x4x128xf16> into tensor<?x2x16x4x128xf16>

    %1 = iree_linalg_ext.scatter dimension_map = [0] unique_indices(true) ins(%collapsed, %arg1 : tensor<?x2x16x4x128xf16>, tensor<?x1xi32>) outs(%arg2 : tensor<?x2x16x4x128xf16>) {
    ^bb0(%arg7: f16, %arg8: f16):
         iree_linalg_ext.yield %arg7 : f16
    } -> tensor<?x2x16x4x128xf16>
    return %1 : tensor<?x2x16x4x128xf16>
}

I was playing around with this a bit... This is possible with some tweaks, but need to figure out these two issues

The tensor.collapse_shape needs to be propagated "down" past the scatter. Might be a simple thing to do, but not sure it could be hard since I dont understand scatter op semantics. I tried to do it by hand and I kept getting validation errors
For the transpose operation, the input indexing map is identity and the output indexing map is not. We want to be able to make the output indexing map of this transpose identity. The issue is that we actually do the opposite here. Honestly that pattern is a bit of a hack cause havent really figure out where to do it. It has a spooky action at a distance feel to it. In any case that is an issue.

Lets say we fix the issue and do something like this
func.func @main(%arg0: tensor<2x?x16x4x128xf16>, %arg1: tensor<?x1xi32>, %arg2: tensor<?x2x16x4x128xf16>) -> tensor<?x2x16x4x128xf16> {

    %c1 = arith.constant 1 : index
    %dim = tensor.dim %arg0, %c1 : tensor<2x?x16x4x128xf16>

    %empty = tensor.empty(%dim) : tensor<?x2x16x4x128xf16>
    %transpose = linalg.generic {indexing_maps = [affine_map<(d0, d1, d2, d3, d4) -> (d1, d0, d2, d3, d4)>, affine_map<(d0, d1, d2, d3, d4) -> (d0, d1, d2, d3, d4)>], iterator_types = ["parallel", "parallel", "parallel", "parallel", "parallel"]} ins(%arg0 : tensor<2x?x16x4x128xf16>) outs(%empty : tensor<?x2x16x4x128xf16>) {
    ^bb0(%in: f16, %out: f16):
      linalg.yield %in : f16
    } -> tensor<?x2x16x4x128xf16>

    %1 = iree_linalg_ext.scatter dimension_map = [0] unique_indices(true) ins(%transpose, %arg1 : tensor<?x2x16x4x128xf16>, tensor<?x1xi32>) outs(%arg2 : tensor<?x2x16x4x128xf16>) {
    ^bb0(%arg7: f16, %arg8: f16):
         iree_linalg_ext.yield %arg7 : f16
    } -> tensor<?x2x16x4x128xf16>
    return %1 : tensor<?x2x16x4x128xf16>
}

This gets fused, but only under aggressive fusion. So compiling this with --iree-dispatch-creation-aggressive-fusion=true makes this one dispatch.
@IanWood1  to start with maybe move scatter fusion out of aggressive fusion. As a side maybe we should just try turning aggressive fusion on by default and see where we stand.

Here's a summary here of the changes we found were needed for scatter fusion. I have attached an example scatter. To simplify the explanation, I'll refer to the to the %update operand as a composition of two parts: a "batch" dimension (number of updates) of shape ? and portion "slice" of shape  1x16x8x128:
For clarity, the %update operand is considered as a composition of two parts:

"Batch" Dimension: Represents the number of updates, with a shape of ?.
"Slice" Portion: Represents the slice, with a shape of 1x16x8x128.

 %85 = iree_linalg_ext.scatter dimension_map = [0] unique_indices(true)
    ins(%update, %indicies : tensor<?x1x16x8x128xf16>, tensor<?x1xi32>)
    outs(%output: tensor<?x16x8x128xf16>)
    {...}

Required Changes:
1) Drop the unit dimensions from "slice" portion of the %update operand of the scatter op
For example, change %update's type to tensor<?x1x16x8x128xf16> -> tensor<?x16x8x128xf16>.
2) Fuse with reshapes that modify the "slice" portion of the %update and/or %output operands
For example, fuse with a collapse shape that expands %update's second dim from 16 to 4x4:
tensor<?x16x8x128xf16> -> tensor<?x4x4x8x128xf16>.
This also requires updating the %output operand and introducing a tensor.collapse_shape
3) Fuse with reshapes that modify the "batch" portion of the %update and/or %output operands.
This will be more tricky since it will require changing verifiers and updating the operation documentation. If the "batch" portion of the operand gets expanded from ? to 4x? then %indicies will also have to be expanded to tensor<4x?x1xi32> to account for the change from a list of updates to a 2D tensor of updates.
@MaheshRavishankar @qedawkins

Update: I opened 2 PRs. The first PR https://github.com/iree-org/iree/pull/19560 makes changes to scatter needed to do 1-3. The second https://github.com/iree-org/iree/pull/19450 is for dropping the unit dims on scatter ops. I'm working on reshape fusion and have changes on https://github.com/iree-org/iree/compare/main...IanWood1:iree:scatter_organized.