Hybrid heat boilers in industry show high electricity demand peaks when switching on

A client pointed out that the electricity demand curves for hybrid heat boilers in industry show high electricity demand peaks when switching on. @ChaelKruip and I assume that these peaks compensate for the heat demand that is not produced in the hour before and think that it's modeled like this because of the calculation order of the heat module and electricity merit order but are not sure. @antw , do you know where these peaks come from?
Below you can see the electricity&hydrogen demand and heat output curves for hybrid heat in the chemicals other sector. This is the scenario that we've used for the curves.

Notifying @ChaelKruip @AlexanderWirtz

I'm curious how that heat output curve chart was created, because I'm not seeing any values above 2k.
This matches the maximum hourly demand of P2H (the industry_heat profile is flat):
MAX(V(
  industry_chemicals_other_flexibility_p2h_hydrogen_electricity,
  heat_output_curve
))
# => 1,947.4564100104262

It may be that the spikes are due to refilling the 50MWh buffer; the input capacity of P2H significantly exceeds the hourly demand. If the buffer were empty, it is expected that the electricity load would spike until it is refilled.
There's also chance that the electricity input curve is incorrect. We use a single curve to represent load, rather than separate input and output. This means that if 10 "energy" enters and 10 leaves in the same hour, the net load is 0. In fact the input and output curves should show 10. That might be what's happening here; I'll need to investigate next week.

Thanks @antw for looking into this!!

I'm curious how that heat output curve chart was created, because I'm not seeing any values above 2k.

I've multiplied the electricity input curve with efficiency of the boiler (99.5%) and multiplied the hydrogen input curve with the efficiency of the hydrogen burner (90%). I see now that I've not take into account the buffer behaviour....
I am not actually understanding what is happening, so I am trying to write what I do understand. The spike occurs because the P2H buffer is not full at that point (hour 154 and 155 in your example), so then more electricity can be converted into heat going into the buffer. When the buffer is full, then electricity is converted into heat that can directly be used in industry (from hour 156 onwards).
However, actually we would like to model that heat can directly be used in industry in hour 154 and doesn't go to the buffer first, therefore we have modeled the offset of one hour in the heat output curve.
I have a question concerning the heat output capacity and electricity input capacity, how does this work for P2H in industry? In the front-end the heat output capacity of the P2H boiler can be set and in the scenario this is 5945 MW. I would say that this means that the electricity input capacity should then be 5975 MW (= 5945 / 0.995). However, in the scenario the electricity input capacity seems to be 1957 MW (from hour 156 onwards), even though there is enough excess electricity at that moment in the year, see screenshot below (154 hour / 24 = day 6.4).

I would say that this means that the electricity input capacity should then be 5975 MW (= 5945 / 0.995). However, in the scenario the electricity input capacity seems to be 1957 MW (from hour 156 onwards), even though there is enough excess electricity at that moment in the year, see screenshot below (154 hour / 24 = day 6.4).

I'm not sure where the "1957 MW" is coming from; the capacity of the node says 5975 MW:
V(
  industry_chemicals_other_flexibility_p2h_hydrogen_electricity,
  "input_capacity * number_of_units * availability"
)
# => 5974.874371859297

... and the curve does indeed peak at this value:
MAX(V(industry_chemicals_other_flexibility_p2h_hydrogen_electricity, electricity_input_curve))
# => 5945.0

Note that we seem to be compensating the the heat output conversion (99.95%) incorrectly, the peak of the curve should be 5945.0 / 0.995 = 5974.
The technology is often limited to 1957 MW input because this equals the hourly demand on the node:
DIVIDE(
  V(industry_chemicals_other_hydrogen_hybrid_heat, demand),
  PRODUCT(MJ_PER_MWH, HOURS_PER_YEAR)
)
# => 1957.2426231115257

1957 MWh is subtracted from the buffer, therefore 1957 MW is the maximum electricity load as that will refill the buffer to 100%.
I do think that P2H is handling the heat output capacity wrongly: if the heat demand is 1957 MW then the electricity load to refill the buffer should be 1966 MW.

👍

I'm not sure where the "1957 MW" is coming from

From you example table, the electricity input in hour 156.

The technology is often limited to 1957 MW input because this equals the hourly demand on the node

Okay, clear, I forgot about the heat demand curve.....  😅

I do think that P2H is handling the heat output capacity wrongly: if the heat demand is 1957 MW then the electricity load to refill the buffer should be 1966 MW.

I agree

@marliekeverweij @ChaelKruip I have a fuller-fix ready for staging, which includes correct accounting of the heat output conversion, but I'm having second thoughts about our earlier decision.
It was agreed that it makes sense for P2H to have a buffer, as heat demand may not always coincide with available excess electricity. However, the demand for heat is constant at 1957 MW. If we limit the P2H input capacity to never exceed demand, then the P2H effectively becomes instantaneous (or rather: is instantaneous with a one-hour delay). Energy that enters the buffer in one hour will always be emitted one hour later, because the constant demand says that it must.
I think our decision may be confusing: if the user says they want 5945 MW input capacity (and a buffer sized to match), should we overrule their decision and limit the P2H to 1967 MW? If a user does not want spikes resulting from a higher input and buffer capacity, why not just lower the P2H capacity slider to the point where no such spikes occur? (1967 MW)?

I think our decision may be confusing: if the user says they want 5945 MW input capacity (and a buffer sized to match), should we overrule their decision and limit the P2H to 1967 MW? If a user does not want spikes resulting from a higher input and buffer capacity, why not just lower the P2H capacity slider to the point where no such spikes occur? (1967 MW)?

Good point! I tend to agree with you.
@marliekeverweij @AlexanderWirtz and myself will have a meeting about this tomorrow. We will report back about a way forward if that is okay with you.

I understand your second thoughts.
I think that ideally the max of the P2H output capacity slider is maxed by the heat demand (1967 MW). Users then know that that capacity is used: what you see is what you get. However, the slider max can not vary based on other sliders setting (heat technology share %, size of industry, etc.). So I think this is the best way to go! Until we better know how heat buffers in industry really behave (see https://github.com/quintel/etmodel/issues/3368)


Update methods in shapely.ops.all

My IDE complained, and this was a quick fix. It seems the methods in __all__ weren't up to date. I also couldn't explain why 'section' was in it, so I've removed that entry.

Coverage remained the same at 81.716% when pulling<PHONE_NUMBER>c2580756b96c2f7bccab3f721705d3 on bartbroere:bugfix/update-ops-methods-in-all into 81895183c1fb496b4522c0ccc278abe25b538cce on Toblerity:master.

Thanks @bartbroere !

I cherry picked this for the maint-1.7 branch too.


Removed usage of centre-channel-color from post comments

Fixes https://github.com/mattermost/mattermost-server/issues/15779

Hello @Raj-Datta-Manohar,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Thanks for the help getting rid of those old CSS variables, @Raj-Datta-Manohar. This looks good to me!


website export demos fix

Coverage remained the same at 82.884% when pulling 44f777cb6f80e3d1197d8e2fd8e2d26fb2d60e1a on xx/website-fix into<PHONE_NUMBER>d70123a35acb47384313c4f28add52 on master.


Enforce valid order signature at type level?

https://github.com/gnosis/gp-v2-services/pull/93#discussion_r538619943
Felix:

I'd slightly prefer if we used UserOrder::default and in there create a real signature that is valid. Creating an invalid order creation and then mutating it into something allows to create semantically invalid data (not so important for the tests but also alluding to my general preference of checking signature at order creation time).

This type was intended to implement the serialization format. I can see some merit in having a checked CorrectlySignedOrder but I don't think enforcing that for this type meshes well with the serialization aspect. The new type can enforce it's invariant but it would also end up being harder to use for it because we would have to disallow mutation because that would invalidate the signature which makes tests more annoying.
In practice, in our non test code we are never going to mutate orders anyway so we are unlikely to gain much from the new type as long as we check the signature in the one place it matters: before adding it to the orderbook.
In test code we might not care about having a valid signature (like in the solver tests) and forcing this invariant would make tests more annoying to write.

In practice, in our non test code we are never going to mutate orders anyway

This is why I think the type should not allow mutation (it's nice if this safety guarantee can be enforced by the compiler).

forcing this invariant would make tests more annoying to write.

Where would we want to mutate orders in tests? I can imaging creating orders for tests as a use case (where we don't care about the signature), but we can have a builder function which computes the correct signature for the test data.


Add NewBigIntegers constructor

In keeping with supporting the creation of arrays of values, a constructor to create an array of BigInteger type should be implemented.
Dim Numbers() As BigInteger

Numbers = NewBigIntegers(1, 2, 3)

Added in v4.0.


Planner related tooltip not updating dynamically

Describe the bug
The tooltip for the planned start (based on an active planner) is not getting refreshed dynamically.
It seems it is just defined once during page reload and then gets outdated. See screenshot below, comparing systemclock 8:11 vs. planned start at 9:00, and the tooltip claiming 2h++ until charge start.

Steps to reproduce

Activate planner with future start date
check tooltip for remaining time till charge start being accurate
wait a few mins
check tooltip again, still showing the previous (and by now outdated) remaining time to start

Can also be replicated with the demo configuration. Just needs the pvPower to be zero.
Configuration details
can be provided if necessary. I was using GSI for planning.

Log details
not needed for UI issue?

What type of operating system are you running?
Linux
Nightly build

[X] I have verified that the issue is reproducible with the latest nightly build

Version
evcc version 0.130.8 (a44ba0d20)

Ha, eigentlich ist in den Tooltips extra eine Aktualisierung eingebaut.  Ich schau mir das an.


Fix Resource closed exception

Change-Id: I4602ce54fde5832b3816c75b7aecf34271c67fa0
Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report

Merging #1262 (29439a1) into master (1d8d32f) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1262      +/-   ##
============================================
- Coverage     71.97%   71.95%   -0.02%
  Complexity     5111     5111
============================================
  Files           473      473
  Lines         21909    21909
  Branches       2345     2345
============================================
- Hits          15768    15764       -4
- Misses         4937     4944       +7
+ Partials       1204     1201       -3

Impacted Files
Coverage Δ

...ava/ai/djl/translate/ServingTranslatorFactory.java
79.45% <100.00%> (ø)

...gine/src/main/java/ai/djl/dlr/engine/DlrModel.java
51.61% <0.00%> (-3.23%)
:arrow_down:

api/src/main/java/ai/djl/util/Platform.java
69.81% <0.00%> (-1.89%)
:arrow_down:

api/src/main/java/ai/djl/repository/Artifact.java
87.80% <0.00%> (-1.63%)
:arrow_down:

...ain/java/ai/djl/tensorflow/engine/TfNDManager.java
87.78% <0.00%> (-1.53%)
:arrow_down:

...ngine/src/main/java/ai/djl/mxnet/jna/LibUtils.java
40.00% <0.00%> (-1.18%)
:arrow_down:

...djl/training/listener/LoggingTrainingListener.java
87.71% <0.00%> (-0.88%)
:arrow_down:

.../djl/training/listener/MemoryTrainingListener.java
76.62% <0.00%> (ø)

...rc/main/java/ai/djl/paddlepaddle/jni/LibUtils.java
48.27% <0.00%> (ø)

api/src/main/java/ai/djl/ndarray/NDList.java
67.71% <0.00%> (+2.36%)
:arrow_up:

... 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 b1194eb...29439a1. Read the comment docs.


error: Keras-Preprocessing 1.0.3 is installed but keras-preprocessing==1.0.2 is required by {'keras'}

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): pypi
TensorFlow version (use command below): tensorflow 1.11.0rc2
Python version: 3.6
Bazel version (if compiling from source): not using
GCC/Compiler version (if compiling from source): not using
CUDA/cuDNN version: not using
GPU model and memory: not using
Exact command to reproduce: Trying to install keras on Travis-CI (linux-Ubuntu 16.04), report:

Describe
Installed /home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/PyYAML-4.2b4-py3.6-linux-x86_64.egg
error: Keras-Preprocessing 1.0.3 is installed but keras-preprocessing==1.0.2 is required by {'keras'}
After I manually downgrade keras-preprocessing to 1.0.2 in requirements.txt and setup.py, it says:
error: Keras-Preprocessing 1.0.2 is installed but keras-preprocessing==1.0.3 is required by {'tensorflow'}
Error Logs
Situation 1: keras-preprocessing 1.0.3 installed: log.

Same Problem for me and almost the same result with Keras Applications (tf needs 1.0.5 and Keras need 1.0.4)
I have no idea how to work my way around it

https://github.com/tensorflow/tensorflow/issues/22601

Nagging Assignee @fchollet: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

anyone has solutions?

@gunan The bug will be fixed in the coming 1.12 version, right?

The issue was keras_preprocessing was depending on keras. It is fixed in the latest version of keras_preprocessing.
While our dependency version is updated, you can always install TF with --no_deps and then install the latest version of keras_preprocessing.
Or you can simply install 1.12.0rc1 with --pre flag to pip install tensorflow

thanks


fix bug of landing not to the current location

set _triplet_lat_lon_finite true to avoid landing to not the current location, see https://github.com/PX4/Firmware/issues/7990

looks good. can you run
make format to resolve the error

Fixed it now

thanks


Suggestion: Find blueprints that link to a given blueprint

A way to search for blueprints that link to a specified blueprint would be useful.

This is the panel "References" on the right hand side.


sse2: correct simde_mm_bslli_si128 for big endian PPC64

Also simplify PPC64 little endian version and add missing simde_assert_m128i_i8 test.

The Travis CI failure isn't my fault.
It passed the first time after I pushed my modifications.

I have a feeling there are a lot of PP64BE issues… I need to figure out a decent way to test it ☹

I have a feeling there are a lot of PP64BE issues… I need to figure out a decent way to test it frowning_face

Yes I ran into two more which (I hope) I just corrected. It took me a while to find them but the good news is that I don't think there are any more. Everything passes on -mcpu=power6, 7 and 8 here.
For me cross-compiling and executing with qemu work fine. I deliberately chose big endian PPC64 because I'm a masochist ;-)  No, seriously I needed to test Altivec and I also wanted a big endian target to catch other bugs. I couldn't get AltiVec to work on s390x so I installed PPC64.
After these adjustments from P5 to P7, AFAICS all the remaining P5 should be P6.
But this is a big change and I am wondering if it is worth the effort.
Is the POWER9 machine you spoke about BE or LE?
Oops the ppc64le machine is now failing on the out of range test r = simde_mm_bslli_si128(a, 19);
This is probably normal but I am wondering why it didn't fail before.  I'll look in the morning. I think I know : I removed an & 15 which seemed unnecessary because of the >= 16 test but the compiler doesn't realize that it won't be executed. There will be the same problem in bsrlli. I think I shall change the vec_sld to vec_slo. It is getting late so I'll push them untested with crossed fingers.

I hope that I have now finished the PPC64BE corrections.

Is the POWER9 machine you spoke about BE or LE?

The OS on there now is LE, but IIRC I could put a BE OS on there.
By understanding is that since POWER8 IBM has really been pushing people to use LE, and BE is mainly a compatibility thing.  At least that's what someone from IBM led me to believe back when P8 was new.
AFAIK s390x is the last real BE platform remaining.

Merged, thanks!


Make API endpoints faster

The API endpoints, especially /cars, are very slow. The /cars endpoint takes ages to respond to the front-end.

The issue was with the EC2 instance and has now been fixed so I am closing the issue


Added regex to Auth/RegisterController because of Stored XSS vulnerability.

I have added regex:[A-Za-z1-9 ] to the Validator function inside the RegisterController.php controller.
Without this regex in the validator a client could register with a username that allowed special characters, resulting in a stored XSS. The user name is stored in database from which even more damage could be done should there be malicious code being called by the XSS.

Hey ! Thank you for your work ! 👍
However, I think the XSS attack can be prevent by escaping data with Blade. The documentation says https://laravel.com/docs/5.6/blade#displaying-data
I close this PR because I don't want to limit user's name to this characters only.

Oke, how would you suggest implementing the prevention of escaped data with Blade? Tried some of the techniques from the link you send but doesn't seem to work.
Another finding: When placing a comment with this content: {{ alert(2) }} an XSS alert box will pop when viewing the public profile of the user that placed the comment.
This also pops in the admin dashboard when checking the comments list.
Any ideas?

If you have a name or a comment with the content <script>alert('3')</script>, It just gonna print <script>alert('3')</script> without execution of the code the same way you see the text on GitHub and not an alert with 3.
It's because I use {{ $user->name }} instead of {!! $user->name !!}
We have to save the user's text in the database but we just need to escape data in the view.
For exemple :

Everything works fine here 👍

add v-pre to any element that can contain user supplied content to prevent {{...}} being parsed as javascript by Vue

@x1mdev 🤔 Ok I got it. Could you open an issue, I'll fix it as soon as possible !
Thank you 👍

@snapey Hum, I think it could fix the problem but I don't want to use Vue on each element :/

I'll open an issue!

You only have to add v-pre on elements that are echoing user supplied input
for instance, a template might have
<comment>
    <div v-pre>{{ $comment->body }}</div>
</comment>


Make core pow implementation more like num-bigint

Note that this approach introduces timing variability. It also doesn't seem to improve performance:
Montgomery arithmetic/modpow, BoxedUint^BoxedUint
time:   [25.614 ms 25.642 ms 25.677 ms]
change: [-1.3757% -1.1214% -0.8876%] (p = 0.00 < 0.05)
Change within noise threshold.

This was mostly pushed up for comparison purposes. Since it doesn't seem to help, closing.


Continue the translation in french. 812 translations remaining

Coverage remained the same at 62.221% when pulling 4db2e4aecf723440a58be2f6a6daf7cc24fdadea on Darklone:french-translation into eb6e97b7d5812bbd8cecf7529675db77009c5d60 on codecombat:master.

Coverage remained the same at 62.221% when pulling 515e57da74a1c56facd9d35df5122f6af6735b6a on Darklone:french-translation into eb6e97b7d5812bbd8cecf7529675db77009c5d60 on codecombat:master.


Do you have write access to the OPS community of practice repository?

Overview
We want to make sure that all OPS volunteers have write access to their community of practice repository so that they can self assign issues.
Action Items

[ ] Check to see if you have write access

You have write access If you see the gear on the top right next to Assignees

You don't have write access if you don't see the gear on the top right next to Assignees

[ ] Tell us

If you have write access, please put your slack name, handle and the phrase "I'm in!" in the comment below

e.g., Bonnie Wolfe, @experimentsinhonesty, I'm in!

If you don't have write access, please put your slack name, handle and the phrase "I need access" in the comment below

e.g., Bonnie Wolfe, @experimentsinhonesty, I need access!

jared, @jafow, I'm in

I am working on the new Hack for LA landing page and have been tasked with writing a description of each COP. I would like to get feedback on my brief description to ensure accuracy and I also would like to include an example project or two for each one. Thank you in advance for your help! Patty
Brief description:
The Operations (Ops) Community of Practice (COP) is a space for operations professionals to discuss all areas of dev-ops, coordinate infrastructure improvement, and share effective practices, and give and receive mentorship. Recent projects include improving AWS hosting, password vaults, and multi-tenant product architecture.
Can someone please give me an example or two of your recent projects?
I welcome feedback on the above description.

Fang Liu, @Fang, I'm in!

Nicole Dubin, @Neecolaa, I'm in!


Repo requirements

[x] repo has unique name
[x] class/id naming conventions comments
[x] PRs & meaningful commit messages
[x] Use of CSS variables

PR: #10


[INLONG-531] delete a cluster in the manager

delete a cluster in the manager
JIRA link - [INLONG-531] created by stingpeng

fixed by https://github.com/apache/incubator-inlong/pull/407


Custom nested Dockerfile location fails using Windows

Is this a BUG REPORT or FEATURE REQUEST?: BUG REPORT
Description
Building image using custom nested dockerfile location fails on Windows.
How to reproduce

install docker-toolbox
in pom supply custom location like <dockerfile>src/main/docker/Dockerfile</dockerfile>
watch build fail with:

[ERROR] Failed to execute goal com.spotify:dockerfile-maven-plugin:1.4.10:build (default-cli) on project x: Execution default-cli of goal com.spotify:dockerfile-maven-plugin:1.4.10:build failed: Could not acquire image ID or digest following build -> [Help 1]

watch log inside docker machine (/var/lib/boot2docker/log/docker.log):

time="2019-01-27T21:32:18.399617116Z" level=error msg="Handler for POST /build returned error: Cannot locate specified Dockerfile: src\\main\\docker\\Dockerfile"

See DockerClient:
    public static BuildParam dockerfile(final Path dockerfile) {
      return create("dockerfile", dockerfile.toString());
    }

Path.toString() will return backslashes and placed as such in query string.
What do you expect
No windows back slashes in call to docker machine.
What happened instead
See steps to reproduce. Build failed.
Software:

Spotify's dockerfile-maven version: [1.4.10]

I currently have the same issue. For now I worked around it with the maven-resource-plugin by moving the respective Dockerfile from the nested location to the "project.build.directory".
<plugin>
    <groupId>org.apache.maven.plugins</groupId>
    <artifactId>maven-resources-plugin</artifactId>
    <executions>
        <execution>
            <id>copy-docker-resources</id>
            <phase>prepare-package</phase>
            <goals>
                <goal>copy-resources</goal>
            </goals>
            <configuration>
                <outputDirectory>${project.build.directory}</outputDirectory>
                <resources>
                    <resource>
                        <directory>${project.basedir}/src/main/docker</directory>
                        <filtering>false</filtering>
                    </resource>
                </resources>
            </configuration>
        </execution>
    </executions>
</plugin>

I think this is kind of critical bug, but the fix should not be complex. Will PR be accepted?


Add Dismissible.confirm_dismiss event handler

Another prop you could add behavior.
I didnt find it useful. If you don't too, then please remove the variable...

https://api.flutter.dev/flutter/widgets/Dismissible/confirmDismiss.html


Implemented optional UTC mode

By default, moment uses the local timezone of the user. In many cases this is desired behaviour, unless you want to share a fixed date, which should be independent of timezone, with users that reside in different timezones.
An example of such a fixed date is a birth date. If I'm in timezone UTC+2 and I choose the date January 1st, 2015, then my birth date should not change to December 31, 2014 if viewed by another user in timezone UTC-7. To prevent that, this pull request offers support for UTC mode.
(In other cases, for example, the date of an event at a specific location, the current behaviour which shows date in the local timezone is desired.)
Implementation
If UTC mode is on, utc() is called on each new date generated by moment(). UTC mode is optional, so it's not a breaking change.
Usage
By default UTC mode is turned off, so then the functionality is the same as before. To enable UTC mode, one can add utc-mode="true" to <mb-datepicker>.

Thank you for your contribution I'll go over your code before I merge


SPA mode

Closes #754.
With this, we can specify a fallback page when using adapter-static. This page contains no content, but will load the SvelteKit app and invoke the router, making it suitable as a 200.html with e.g. Surge, or an index.html with sirv with opts.single.
If a fallback is specified, adapter-static won't forcibly prerender every page, just the ones with prerender = true.
I added a test suite to adapter-static.
Before submitting the PR, please make sure you do the following

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts

Do you think it's worth filing an issue about force disabling SSR as well?

I'd rather wait for that to come up organically — I would rather we investigate alternatives (like static analysis) before providing an 'eject' button that is likely to be misused. People will just forcibly disable SSR as soon as they run into the smallest SSR problem if the option is available to them, rather than working through the problem and preserving SSR-ability where possible. That would be bad stewardship on our part.

I have a requirement package an app using the file:// protocol, meaning there is no web server. Is a SSR-less SPA not in the works? I could use Routify but would rather use Svelte Kit to have both a PWA & a file:// based SPA.

@btakita the issue to track that is https://github.com/sveltejs/kit/issues/907


PollyJS recording API data but still fetching new data from server and replaces the old one it recorded

I am working on an app where I am writing the integration test. I used Pollyjs to mock server response. But yesterday when I ran my tests some of my tests did fail because the data that I was expecting from pollyjs changed.
this.polly.configure({ recordIfMissing: true, logging: true });
I have used the above configuration within the test hook (beforeEach).
I don't understand what is happening but the data that pollyjs returns are changing whenever the server data changes.

Issue is not clear enough, can you add more details?

@vyshnavrajs2019 if the recordings aren't being replayed, its most likely that you have some sort of header value that is dynamic to every test execution. I would take a look at the matchRequestsBy config option and try to exclude anything that could be changing.

Closing this issue due to no response. @vyshnavrajs2019 feel free to reopen if you still need help with this issue.


fix: Always call CaptureSession fully synchronously under Mutex

What
Fixes an issue in the Expensify app

Changes

Tested on

Related issues

How to resolve this issue?


Create first parsing class

create a class that takes a file as an argument and that returns a class server that contains all the configuration

fix


clarify UDF overloading in documentation

Codecov Report

Merging #6913 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #6913      +/-   ##
============================================
- Coverage     83.17%   83.16%   -0.01%
- Complexity    13299    13302       +3
============================================
  Files          1527     1527
  Lines         49375    49375
  Branches       5940     5940
============================================
- Hits          41067    41064       -3
- Misses         5524     5529       +5
+ Partials       2784     2782       -2

Impacted Files
Coverage Δ
Complexity Δ

.../main/java/io/crate/analyze/AnalyzedStatement.java
50% <0%> (-50%)
1% <0%> (-1%)

...rc/main/java/io/crate/analyze/UnboundAnalyzer.java
84.61% <0%> (-12.83%)
2% <0%> (ø)

...n/engine/collect/BatchIteratorCollectorBridge.java
65.78% <0%> (-7.9%)
2% <0%> (ø)

...rate/analyze/ShowCreateTableAnalyzedStatement.java
64.28% <0%> (-7.15%)
5% <0%> (-1%)

.../java/io/crate/planner/node/dql/join/JoinType.java
64.7% <0%> (-5.89%)
6% <0%> (ø)

...ution/engine/distribution/ModuloBucketBuilder.java
73.68% <0%> (-5.27%)
10% <0%> (-1%)

...io/crate/protocols/postgres/types/VarCharType.java
87.5% <0%> (-4.17%)
9% <0%> (-1%)

...tion/engine/distribution/DistributingConsumer.java
75.72% <0%> (-3.89%)
21% <0%> (ø)

...nce/sys/check/node/DiskWatermarkNodesSysCheck.java
57.69% <0%> (-3.85%)
6% <0%> (-1%)

...in/java/io/crate/breaker/RamAccountingContext.java
81.35% <0%> (-3.39%)
20% <0%> (-2%)

... and 26 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 ac5c2cb...5577731. Read the comment docs.

@nslater could you have another look please


V0.2 vm region support

Description

Add region, zone, cluster layers (PM).
Build the snapshots of region, zone, cluster.
Add a basic config of region, zone, cluster structure.

Linked issue(s)/Pull request(s)

issue_number

Type of Change

[ ] Non-breaking bug fix
[ ] Breaking bug fix
[x] New feature
[ ] Test
[ ] Doc update
[ ] Docker update

Related Component

[ ] Simulation toolkit
[ ] RL toolkit
[ ] Distributed toolkit

Has Been Tested

OS:

[ ] Windows
[ ] Mac OS
[ ] Linux

Python version:

[ ] 3.6
[ ] 3.7

Key information snapshot(s):

Needs Follow Up Actions

[ ] New release package
[ ] New docker image

Checklist

[ ] Add/update the related comments
[ ] Add/update the related tests
[ ] Add/update the related documentations
[ ] Update the dependent downstream modules usage

Codecov Report

Merging #258 (534311c) into v0.2 (d51b192) will decrease coverage by 0.88%.
The diff coverage is 35.82%.

@@            Coverage Diff             @@
##             v0.2     #258      +/-   ##
==========================================
- Coverage   64.31%   63.43%   -0.89%
==========================================
  Files         111      114       +3
  Lines        5411     5587     +176
==========================================
+ Hits         3480     3544      +64
- Misses       1931     2043     +112

Flag
Coverage Δ

unittests
63.43% <35.82%> (-0.89%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...mulator/scenarios/vm_scheduling/business_engine.py
16.23% <9.41%> (-2.02%)
:arrow_down:

...simulator/scenarios/vm_scheduling/frame_builder.py
42.85% <50.00%> (+5.35%)
:arrow_up:

maro/simulator/scenarios/vm_scheduling/zone.py
56.66% <56.66%> (ø)

maro/simulator/scenarios/vm_scheduling/region.py
57.14% <57.14%> (ø)

maro/simulator/scenarios/vm_scheduling/cluster.py
57.57% <57.57%> (ø)

...ulator/scenarios/vm_scheduling/physical_machine.py
44.82% <100.00%> (+3.00%)
:arrow_up:

maro/simulator/scenarios/vm_scheduling/common.py
51.42% <0.00%> (-1.52%)
:arrow_down:

... 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 d51b192...534311c. Read the comment docs.

Codecov Report

Merging #258 (534311c) into v0.2 (d51b192) will decrease coverage by 0.88%.
The diff coverage is 35.82%.

@@            Coverage Diff             @@
##             v0.2     #258      +/-   ##
==========================================
- Coverage   64.31%   63.43%   -0.89%
==========================================
  Files         111      114       +3
  Lines        5411     5587     +176
==========================================
+ Hits         3480     3544      +64
- Misses       1931     2043     +112

Flag
Coverage Δ

unittests
63.43% <35.82%> (-0.89%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...mulator/scenarios/vm_scheduling/business_engine.py
16.23% <9.41%> (-2.02%)
:arrow_down:

...simulator/scenarios/vm_scheduling/frame_builder.py
42.85% <50.00%> (+5.35%)
:arrow_up:

maro/simulator/scenarios/vm_scheduling/zone.py
56.66% <56.66%> (ø)

maro/simulator/scenarios/vm_scheduling/region.py
57.14% <57.14%> (ø)

maro/simulator/scenarios/vm_scheduling/cluster.py
57.57% <57.57%> (ø)

...ulator/scenarios/vm_scheduling/physical_machine.py
44.82% <100.00%> (+3.00%)
:arrow_up:

maro/simulator/scenarios/vm_scheduling/common.py
51.42% <0.00%> (-1.52%)
:arrow_down:

... 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 d51b192...534311c. Read the comment docs.


Copy Date-header to decrypted message.

Until there's a good solution how to deal with headers (#24) mail-gpg should copy standard headers, e.g. Date.

I don't understand why, but I couldn't find another way to make the tests pass.
Using encrypted_mail.send(field) also for date resulted in this error:
Error: test: Mail::Message with encryption and signing should decrypt and check signature. (MessageTest): NoMethodError: undefined method `to_datetime'
 for nil:NilClass
.../mail-gpg/.bundle/ruby/2.3.0/gems/activesupport-3.2.17/lib/active_support/core_ext/date_time/calculations.rb:141:in `<=>'
.../mail-gpg/.bundle/ruby/2.3.0/gems/mail-2.5.4/lib/mail/header.rb:165:in `=='
.../mail-gpg/.bundle/ruby/2.3.0/gems/mail-2.5.4/lib/mail/header.rb:165:in `[]='
.../mail-gpg/lib/mail/gpg.rb:143:in `block (2 levels) in decrypt_pgp_mime'
.../mail-gpg/lib/mail/gpg.rb:139:in `each'

I'll continue to fix the tests...

The errors seem a little random to me. I can't really reproduce them locally, only saw them twice but never again. And the results of travis' builds are not consistent either, please compare https://travis-ci.org/jkraemer/mail-gpg/builds/185892181 with https://travis-ci.org/jkraemer/mail-gpg/builds/186052132 — the committed code is identical!
Does anyone have any idea how to pin or fix this? I'm clueless for the moment.

I just released 0.3.0 which does not do header filtering any more.


Fixes to several casks of jetbrains products

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[ ] The commit message includes the cask’s name and version.

This PR basically:

adds missing appcasts
fixes faulty appcasts checkpoints
adds missing auto_updates
Update a few names and urls to canonical versions

As @vitorgalvao deleted the second name of intellij-idea. Should we delete the second name of intellij-idea-ce as well?

@leipert No. In that case it is correct. See name docs.

@leipert Can you also find appcasts for the EAP versions in caskroom/versions?

I rebased the commits onto master and added the last missing appcasts for intellij-idea and intellij-idea-ce.
@vitorgalvao Regarding appcasts for eaps.
Theoretically there are urls available for eaps, but it seems like they are not available for all jetbrains products and sometimes after an release is made, the last eap vanishes...


🚧Update testing app obj movement

Description:
To allow this pull request to be merged, please make sure that the following items of this pull request are complete.
This pull request closes #738
✅The pull request head branch must be a feature branch with the syntax 'feature/-sync-testing'.
✅The pull request base branch must be a 'main' or 'preview' branch..
✅The pull request head branch contains a valid issue number.
✅The pull request title matches the linked issue title exactly.
✅The pull request assignees match the assignees of the issue.
✅The pull request labels match the labels of the issue.
✅The pull request organizational projects match the organizational projects of the issue.
✅The pull requests milestone matches the milestone of the issue.

Additional Info:

Pull requests are automatically synced with the associated issue upon creation by one of the KinsonDigital workflow bots.
The associated issue is the issue number that is embedded in the pull request head branch.
The list above will be automatically updated as the pull request's various settings match or do not match the associated issue.
To manually sync the pull request to the issue, create a comment with the [run-sync] command.

Note You must be an admin member of the organization to use this command.

[x] Sync with the issue.  Use this to enable or disable this pull request from syncing with its associated issue.

Closed without changes
Reason: After researching, it was discovered that all obj movement in the testing app scenes took time into account.  it was also discovered that the game is performing very well and is solid.  Lowering the update iterations discovered that the movement was a little bit jerky and was the cause of any jittery movement behavior.


Update phpunit

Replaces #2232

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 76.82%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
633

Relevant Lines:
824

💛  - Coveralls


Refactor Metalmind Selection Code

The code that chooses what Metalmind to tap/store is one of the oldest pieces of code in the project, and likely needs a refactor to improve performance and be more intelligent about what order the metalminds will show in.

[ ] Improve performance
[ ] Improve Metalmind selection order (Hotbar, Curios, armor, inventory)
[ ] Config values for the above to change order

While we are in the area, we should:

[ ] Fix bug that leaves only a few points left in a metalmind
[ ] Allow grabbing points from multiple metalminds at once as part of one request (which may fix the above)

Ooh, I never thought to say - I think I found a solution to the bug where a few points were left - all you need to do is tap or store at level one, depending on whether you're filling or storing.
I'm still open to a proper solution, though, especially as this workaround doesn't work for copper.

May be worth taking a look at Hexcasting and media works for reference, Hexcasting has to deal with finding media (amethyst and related) in the inventory, and media works implements a better algorithm


vendor prefixes!

add this line vendor-prefixes ?= webkit moz o ms official, only shows webkit...

I'm going to need a bit more info to reproduce that...

button
border-radius 2px
compiled...
button {
-webkit-border-radius: 2px;
border-radius: 2px;
}
should be add -moz- -o- prefixes no?

As far as I can tell, opera never had a prefix for border-radius, and FF hasn't needed one since 3.6. Actually, Chrome hasn't needed one since v4, so we probably shouldn’t even be adding the webkit prefix.

Yes, the webkit prefix should be removed for border-radius.

@slang800 in case that's something you guys wanna do, there's a PR for it now: #306


What is the normal time cost of glfwCreateWindow() and can I speed up?

Versions
glfw: 3.3
OS: win7
Compiler: VS2017 x64
Problem description
glfwCreateWindow() cost 600~700ms on my machine. I'd like to know if this the normal speed, or if I can speed it up?
The long story
I'd like to load and display image on Linux/MacOSX/Windows in pure C. I can load image correctly with stb_image and I've implemented a GDI based version, which shows GUI window instantly. GDI is not usable on linux/mac osx, so I'd like to use libraries like glfw/freeglut.
I've tried freeglut/glfw3.3, both gives 600~700ms time cost for creating window on my machine, which is obviously slow than GDI based implement. Any suggestions? Correct me if I'm wrong since I know very less about how glfw/freeglut implement their creating window functions. Thanks.

I've figured out that this is due to my too old version of nvidia display driver. Switching to 430 then re-run my code, glfwCreateWindow() takes ~130ms which is faster.


Can you add me as a collaborator?

Need a test repo
Sent with GitHawk

This comment interrupts the labels
Sent with GitHawk


Fix README typo

JSON object in package.json file is contributes.keybindings, not contributions.keybindings.

Ah thank you!


adding private IP subnet <IP_ADDRESS>/16 not accepting

when i am trying to add my LAN IP range <IP_ADDRESS>/16, it is taking ages and does not accept,, however adding public IP works fine.

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

ID: dc4264cd-3476-61ae-49dc-c5fa6118722a
Version Independent ID: 8c419a14-db06-3b0d-66d3-ab396924721e
Content: Control access based on network location or app
Content Source: OneDrive/control-access-based-on-network-location-or-app.md
Service: one-drive
GitHub Login: @kaarins
Microsoft Alias: kaarins

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

@officedocsbot close


Added MVP report draft 2

Added MVP draft report num. 2

Merging immediately due to importance and being documentation


NetifSlirp: Fix compatibility with libslirp v4.6.x

NetifSlirp: Fix compatibility with libslirp v4.6.x
Description
Ubuntu 22.04 ships with libslirp 4.6.1 via packages libslirp0 and libslirp-dev. Libslirp v4.6.x has a slightly different configuration struct. The PR adjusts the implementation in MBuffNetifBackendLibslirp.c to support libslirp 4.6.x.
Test Steps
Build FreeRTOS Posix sim TCP Echo Demo with BufferAllocation1 and BufferAllocation2. Test each binary against a TCP echo server.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run uncrustify


Upgrade bundler & references to version 2.2.10 or later to mitigate vulnerability

Change: Fix https://github.com/advisories/GHSA-fp4w-jxhp-m23p
Testing: Deployed built image referencing this brach to an environment with SNMP telemetry; verified presence of end-to-end telemetry and no errors

Sorry Gabe, I misclicked and reset your review. Please re-click the button.


Run with docker

This project and paper are great and I feel very happy to use from all the way from Japan.
But this project did not have docker and docker-compose files, so I made it !
Usage is written in readme.md. I would be so happy if you could merge ;) thank you.

thank you very much , i am using  the dockerfile you offered.

@wantongtang Thank you for using :)

Hi, nyk510, can you tell me the parameter size of wavenet?  thank you very much!


Create Issue Template

From https://github.com/blog/2111-issue-and-pull-request-templates
Things we could ask for:

[ ] Version of Node (and example to find it)
[ ] Config file???

Ended up with these

[ ] Issue Description
[ ] APEX Front-End Boost Version
[ ] Operating System
[ ] Node Version
[ ] Browser
[ ] APEX Version
[ ] Web Server


Feature: Add possibility to select multiple tabs and do context actions + option for all tabs

What's the Problem?
I want a simple and quick way to do an action on multiple tabs or do an action for all opened tabs. Such actions could be for instance duplicating or moving to an new window.
Solution/Idea

add the possibility to use "STRG + click" to select multiple tabs. Then, right-clicking one of the selected tabs and choosing an option on the context menu should apply this action to all selected tabs.
add an option to the context menu of the titlebar and maybe to the vertical tab flyout to do actions for all tabs opened in that window.

Alternatives

do it for each tab individually -> not as straightforward and quick

Priorities

Capability
Priority

This proposal will allow end-users to quickly run actions for multiple tabs
Must

This proposal will allow end users to quickly run actions for all opened tabs
Should

Files Version
<IP_ADDRESS>
Windows Version
Windows 11 21H2 - 22000.376
Comments
No response

Thank you for your feedback.


Move "Security" and "Insights" repository tab to more-dropdown

https://github.com/sindresorhus/refined-github/pull/3346 restored more-dropdown but now we also need to move the Security and Insights tabs to it, like it was before the "Repository refresh"
Possible way to do it:

remove tabs from dom and show/enable the items in the dropdown

Gotchas:

the DOM order of the menu matters, so you probably need to also move them at the end of the dropdown, after "Settings"

Currently the . . . is always shown (even if nothing is inside) This would also fix that.

The dropdown can never be empty, regardless of circumstances. I don’t know if #3347 is related, but I think not. If it’s empty, the creation is failing, not the native resize listener.

What is a repo does not have releases such as https://github.com/yakov116/pulse

Open a new issue. It’s a bug.

This is ridiculously inconvenient! We need some mechanism to be able to configure what should be in the more-dropdown and what should be on top.
I just spent 15+ minutes to find out what the heck and where is my Insights button...

@Xottab-DUTY you can disable the feature and keep the tabs where they started. We won't be adding a configuration option for this, at most we could split each tab into it's own feature.

It's good as is. If you need those tabs, you can disable more-dropdown, it can't be split


Stack size support for inventoryGrid (Minecraft style)

In some games that use a grid based inventory system it would be useful to have the support for item stacks, similar to Minecraft.
This would require storing the current stack size for a given item, as well as the maximal stack size.

Is this feature on the way?

Development has been kinda slow lately due to personal obligations, but I did experiment with implementing this feature by making InventoryGrid inherit InventoryStacked (not sure if I'm going to commit to this approach though, as it seems to increase the complexity of InventoryGrid quite a bit).
It is one of the higher priority features I intend to implement as such inventories are often used. Hope I'll find some time to work on it soon.

Oh, okay, thanks

InventoryGridStacked is added in https://github.com/peter-kish/gloot/pull/99
It extends InventoryGrid and implements some logic for managing item stacks in a grid based inventory (check out the documentation for more details).


Charmrun: Expand ability to disable ASLR and do so by default

I'm not sure why this is using posix_spawn instead of fork/exec. Isn't the personality inherited on an exec call?

I'm not sure why this is using posix_spawn instead of fork/exec. Isn't the personality inherited on an exec call?

The _POSIX_SPAWN_DISABLE_ASLR flag is the only way to disable ASLR at runtime on macOS. I see no reason not to also enable posix_spawn on other platforms that support it, because no functionality is lost by using it instead of fork/exec.

I'm not sure why this is using posix_spawn instead of fork/exec. Isn't the personality inherited on an exec call?

The _POSIX_SPAWN_DISABLE_ASLR flag is the only way to disable ASLR at runtime on macOS. I see no reason not to also enable posix_spawn on other platforms that support it, because no functionality is lost by using it instead of fork/exec.

Ok, but please add this as a comment. Is _POSIX_SPAWN_DISABLE_ASLR only used in macOS?

Is _POSIX_SPAWN_DISABLE_ASLR only used in macOS?

Yes.

This looks good, but there seem to be some unrelated changes now.

This looks good, but there seem to be some unrelated changes now.

The extent of the changes to threads.C are adding // If this crashes, disable ASLR. comments while keeping line lengths under 72 characters through use of local variables.


"Cannot call "ez" on null" when it tries to close the original issue

I just encountered this error:

The issue and all its comments were copied across okay, I just had to manually close the original issue.

I've found that this only happens when moving an issue from a repository on an organization. In my case I'm moving issues from BEEmod/BEE2.4 to BEEmod/BEE2-items. When I previously moved a few issues from another user's fork to BEE2-items this error did not occur.

Updated comment 302818281.

I am also seeing this issue. It will be nice have this fixed.

Same thing happened to me too; After moving dozens of issues, I seem to had hit a quota limit or something, because I got the above reported error. After waiting a few minutes, I could move another batch until I got the same error again. I waited another while, after a few hours I could move another batch. But once I reached 150 moves, it stopped working altogether. I've waited for 3 days already now, but it still won't move a single issue anymore.

@PatrickvL Your issue is a bit different, the issue described here is that it failed to close the original issue - everything copied over fine, you just need to close the original issue manually. I've experienced your bug before, however, when trying to move a large issue (specifically this one) to BEE2-items. I ended up having to copy the issue manually.

I also get this error, but it doesn't even get as far as creating the new issue

Crossposting here from #121:
I encountered this issue. In my case, an issue on repo 1 was assigned to X. I wanted to move this issue to repo 2, in which X no longer was a member. The "move issue" button then returned "Can not retrieve "ez" of null" and the network tab showed a 422 unprocessable entity. Removing the assignment of X in repo 1 and trying to the move again fixed the issue.


Paginator does not appear to work

It just seems to call the same endpoint again, copying the limit param from the first request, but not including the nonce or token params given in the next or prev properties of the returned JSON. Any other optional params to the original request (e.g. arguments to the people search endpoint like updated_since) are also dropped. I ended up having to write my own pagination code. Am I missing something?

Here's how I did it for tags:
  def nation_builder_tags
    return [] unless nation_builder_linked?

    response = nb.call(:people_tags, :index, limit: TAGS_PER_PAGE)
    paginated = NationBuilder::Paginator.new(nb, response)

    list = []

    loop do
      list += paginated.body['results']
      break unless paginated.next?

      paginated = paginated.next
    end

    list
  end

nb is the NationBuilder::Client object.

Interesting. Do you have a code sample to reproduce this? Looking at the code it relies on what comes back from the API to return the parameters, and a simple test works for me:
irb(main):001:0> c = Campaign.find(4) # Campaign is a Nation
irb(main):003:0> response = c.nb.call(:people, :search, limit: 1, first_name: 'Sean')

irb(main):005:0> p = NationBuilder::Paginator.new(c.nb, response)
irb(main):007:0> p.next?
=> "/api/v1/people/search?__nonce=SOMENONCE&__token=ATOKEN&first_name=Sean&limit=1"
irb(main):009:0> p.body['results'][0]['last_name']
=> "Walberg"
irb(main):010:0> p = p.next
irb(main):011:0> p.body['results'][0]['last_name']
=> "Walberg"
irb(main):012:0> p.body['results'][0]['id']
=> 6
irb(main):013:0> p = p.prev
irb(main):014:0> p.body['results'][0]['id']
=> 90

I don't filter the way you mention, so I'm just going on my observations here.


[ios] import RCTCamera.h directly?

Hey there, I'm new to iOS development.
How can I include the RCTCamera.h directly on my project? I need to have direct access to it from my .m files.
#import <RCTCamera/RCTCamera.h>

'RCTCamera/RCTCamera.h' file not found

I've tried updating the "Header Search Paths" of my project but no luck to import the header yet.
Thanks in advance!

FYI I've installed it manually as described here: https://github.com/lwansbrough/react-native-camera#manual-install


The property wasLastModifiedAt disappeared without a trace

We used to have a property called wasLastModifiedAt (see below). It disappeared with the new time changes in the 11.0.0 release. There is no mention of it in the release notes, so I'm wondering if it was an oversight and we still want it.   If we want to restore a property with this meaning, I propose that it be named lastModifiedDateTime. In any event the release notes should mention it. See also #693 and #762
gist:wasLastModifiedAt
	a owl:ObjectProperty ;
	rdfs:subPropertyOf gist:hasActual ;
	rdfs:range gist:TimeInstant ;
	skos:definition "Date that something was last modified."^^xsd:string ;
	skos:prefLabel "was last modified at"^^xsd:string ;
	.

PROPOSAL:

find out whether we want this.
if so, add it
if not, consider updating release note so there is a record of its removal, if it's possible to go back in time in a good way.

Also related to #771 ?


[stable/postgresql] Add external ip and port variables

This lets one expose the service on external ip and a custom port number

@tarasglek do you want to sign the CLA so we can take in your changes?

@tarasglek are you able to continue working on this?

@prydonius
yes sorry, missed the earlier notifications that you guys were looking at this change. I'll submit the pullreq without conditionals as requested

@unguiculus I got rid of if statements.
--set 'service.externalIPs=["<IP_ADDRESS>"],service.port=5433'
works, but I'm not sure how one would pass multiple IPs since , is already used a param separator

@tarasglek Here's an example for externalIPs: https://github.com/kubernetes/charts/blob/e60c6e9d6d0b3cf67291769a940a60f42521fdf5/stable/nginx-ingress/templates/controller-service.yaml#L18

@unguiculus thanks, that's similar to how I implemented it. Can I get this merged?

@tarasglek Well, the way you implemented it forces one to use the one-line format for a list, which is uncommon. I'd suggest you change it.

@unguiculus thanks for explaining. I'm new to yaml. This change is more inline with nginx example

Ping, can this be merged?

@k8s-bot ok to test

Thanks @tarasglek!!


Can't update from a boxed query

Setup
I've made a minimal example in a new project here.
Versions

Rust: 1.42
Diesel: 1.4
Database: PostgreSQL
Operating System MacOS

Feature Flags

diesel: postgres, r2d2

Problem Description
I can't figure out how to apply several dynamic filters to a query, and then update using those filters.
What are you trying to accomplish?
I want a function get_tunnels which selects all rows from a table after applying some filters. Which exact filters get applied is only known at runtime.
Are you seeing any additional errors?
the trait `diesel::Identifiable` is not implemented for `diesel::query_builder::BoxedSelectStatement<'_, (diesel::sql_types::BigInt, diesel::sql_types::Text, diesel::sql_types::Bool), tunnels::table, diesel::pg::Pg>`

This makes sense to me, because All About Updates says you can only pass a query into a diesel::update if it's only had .filter called on it. My query meets this criteria, but because it has to be boxed, the compiler doesn't know that.
Steps to reproduce
Run cargo check in the linked repo
Checklist

[X ] I have already looked over the issue tracker for similar issues.
[X ] This issue can be reproduced on Rust's stable channel. (Your issue will be
closed if this is not the case)

As you already pointed out:

This makes sense to me, because All About Updates says you can only pass a query into a diesel::update if it's only had .filter called on it. My query meets this criteria, but because it has to be boxed, the compiler doesn't know that.

We cannot guarantee for BoxedSelectStatment that this is true for all cases, as we have no way to access the single clauses at compile time. Therefore using a BoxedSelectStatement is not supported as where clause for update statements.
Instead of using a BoxedSelectStatement there, you can just box the update statement itself and apply the filter dynamically afterwards or use BoxableExpression to construct a not statically typed expression, which could be used as filter in a "static" select statement.
Closed as everything seems to work as expected.


Add external issue tracker links to issues.

This patch adds a "ISSUE(...): " syntax to auto-generated, inlined
issues which simplifies the process of linking out to topics that are
being discussed on a remote issue tracker.
I considered a GitHub-specific syntax, but putting a template URL in the
document's metadata seemed simpler and more inclusive.

What do you think about this, @tabatkins? In short: some, but not all, of my specs' inline issues are being discussed on GitHub. I'd like to link to them without thinking too hard about it.
I like the issue syntax in this patch (ISSUE(123):), but I'm not at all convinced that "Issue Tracker Template" makes sense, nor am I really thrilled with the link syntax (which is <http://issue-tracker.test/1234> at the end of the issue).
Is this a crazy idea? If not, suggestions for improvements welcome!

Overall, I agree with this! It sounds like a good addition, and it's a reasonable syntax.  Some nits:

In the rest of the codebase I explicitly use .format(), not %, to format strings, and that's also recommended Pythonic practice.  Using that here also means you have the imo clearer http://example.com/issue{0} syntax; people familiar with brace-based templating languages will be familiar with this.

Do we want multiple arguments?  I can see a spec referencing an issue in a different github repo, for example.  (In CSS we sometimes have issues scattered between Tracker, Bugzilla, and GitHub.) This might be handleable just by allowing multiple named templates, so you can refer to one by name and still just pass the issue number.  We can probably defer this.

The display syntax seems fine to me; we can tweak later if we come up with something better.

Sounds fine to me.

Since we're focusing on specs, it might be reasonable to just have syntax for those three options. I don't think I'd need that for any specs I'm working on, but I can see how it would be valuable. What probably would be valuable is linking to other GitHub repositories (e.g. ISSUE(tabatkins/bikeshed#254): ...). That was my first pass at this (and I think I still prefer it), but it seemed too GitHub-centric. shrug

I went ahead and added support for the GitHub-specific syntax, as it feels like something I'll actually use.
Assuming the bot is happy, WDYT?

I'm fine with specialized syntaxes if they're likely to be useful and they don't claim too much syntax space so we can add to them later.  This looks fine.

Strictified. WDYT?

facepalm Thanks. I didn't even notice that the test had horrible output. :( Fixed in the most recent patch.


Fire HD tablet - FireOS

Massively dumb question - but I want to re-purpose one of my Fire HD tablets as a Vinyl-Cast machine. I'm getting a parsing error when installing the apk on a Fire HD 8 (Fire OS <IP_ADDRESS>) and also a Fire HD 7 with Android 5.1.1. What's the minimum Android requirements for the 1.0.0 release?

 I went and added a comment in the README about minimum Android version needed.
I'm not able to easily test on earlier Android versions and given my experience working with the Android audio APIs, I don't want to enable support for earlier versions without testing. But if you're still interested, I could make an app build that targets earlier Android versions and you could test it out.

Note that as of v1.2.0, the minimum Android level was changed back to Android 6.0 (API Level 23).
Closing issue for now due to inactivity and can reopen if necessary.


Size of first table row is inconsistent between browsers.

Testcase: http://jsfiddle.net/x5kefrd4/1/
On Blink and Webkit-based, the th element has a 2px bottom border. On Gecko and Trident, the th has a 1px bottom border and the first row has a 1px top border.
This means that the first row is 38px on Gecko and Trident and 37px elsewhere - the latter is correct.
There's more to this bug than meets the eye. Firefox's box model display shows the th elements as having 1px of bottom border, but when clicked, it expands to 2px. I'm hoping for some expert eyes on this problem and advice before I file a bug with Mozilla.

Hi @ripdog!
You appear to have posted a live example (http://fiddle.jshell.net/x5kefrd4/1/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 12, column 3: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.2.0
line 13, column 3: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.2.0

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Just check in the latest Safari, Firefox, and Chrome on Mac—everything is 37px for me. Even if there are discrepancies, I don't think it's important enough to worry about right now. Let us know if you find anything else out though—I could easily be overlooking something.

@mdo, I can see the discrepancy in Firefox/Win.

@mdo JSFiddle is broken right now, so a codepen: http://codepen.io/anon/pen/jEdJgq
Changed the JS to display the height of first and second rows. In chrome, output is  37px 37px, firefox 38px 37px.
Another screenshot: http://i.imgur.com/TukZbOd.png

Hi @ripdog!
You appear to have posted a live example (http://codepen.io/anon/pen/jEdJgq.html), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might potentially be causing your issue:

line 8, column 100: Element link is missing required attribute property.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

@patrickhlauke You're comparing the thead's row vs the tbody's row, no? I'd expect some cross-browser variation there with the font-weight and larger border.

@mdo yup, but i'm just confirming the OPs original statement :)

Ah, if that's what the OP meant, then I don't consider this a bug at all. Sounds about right to me.

@mdo @patrickhlauke I'm sorry, I didn't mean to give that impression. The problem is the first table row, not the header. As you can see in my screenshot, it has a different height in different browsers. The header is only involved in that it appears to be a possible cause for the discrepancy.


[ZEPPELIN-5426] mongodb interpreter: support mongosh

What is this PR for?
The mongodb interpreter works by calling into the deprecated mongo shell (https://docs.mongodb.com/manual/reference/program/mongo/), which is going to be replaced by mongosh (https://docs.mongodb.com/mongodb-shell/). This is a largely backwards compatible migration, so that the interpreter works almost unchanged. Some internals have changed nevertheless, this pull request ensures the interpreter works with the new shell without breaking backwards compatibility.
What type of PR is it?
Improvement
Todos
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-5426
How should this be tested?
Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? At some point...

LGTM

Does not work with mongosh 1.10.1 from mongodb 6.0.3

Does not work with mongosh 1.10.1 from mongodb 6.0.3
i got this error : TypeError: db.users.aggr ... ]).table is not a function
it seems that the end of shell_extention.js is no more doing is job :
globalThis.DBCommandCursor seems to be undefined.

I could not access mongo client via Zeppelin , but I can access it via cli
I use mongo rather than mongosh
" E QUERY    [js] ReferenceError: globalThis is not defined :" is the Exception


why don't use rawcam output yuv

Now the project output yuv by isp component, why not use rawcam output yuv directly? I did see "MMAL_ENCODING_I420" likes in "mmal_encodeings.h".

The rawcam component outputs whatever format the CSI2 source device generates, be that Bayer, RGB, YUYV, or something else.
Other than potentially repacking the Bayer formats, there is no image processing applied.
CSI2 devices are almost always line based to avoid needing full frame buffers, therefore planar formats such as I420 would be very unusual.
If you read the CSI2 spec for the supported YUV4:2:0 formats, they specify it as an interleaved format with either:

odd lines being UYYUYYUYY, and even lines being VYYVYYVYY
odd lines being half the length of even lines and only having luma (Y) samples, even lines being UYVYUYVY (same as one of the YUV4:2:2 interleavings).

I don't think I've ever come across hardware which can consume that!
The ISP is designed to accept the majority of these CSI2 formats and convert them into the formats that are more commonly used within computer systems, eg planar YUV, or RGB.
(The ISP is the only hardware block within VideoCore that handles YUYV interleaved YUV4:2:2 for example)

I believe this question has been answered, so closing.
Please reopen if you wish to continue the discussion.


adding a textTrack to the video

Have you read the FAQ and checked for duplicate open issues?:
YES
What version of Shaka Player are you using?:
5.6.0
Can you reproduce the issue with our latest release version?:
yes
Can you reproduce the issue with the latest code from master?:
dont understand whats the meaning
Are you using the demo app or your own custom app?:
my own custom app
If custom app, can you reproduce the issue using our demo app?:
i cant use your demo up its blocking my lnk for security reasons
What browser and OS are you using?:
window 8.1 , chrome
What are the manifest and license server URIs?:
(NOTE: you can send the URIs to<EMAIL_ADDRESS>instead, but please use GitHub and the template for the rest)
this is the track that return when running player.getTextTracks();
      {id: 3, active: true, type: "text", bandwidth: 0, language: "he", …}
active:true
audioBandwidth:null
audioCodec:null
audioId:null
bandwidth:0
channelsCount:null
codecs:null
frameRate:null
height:null
id:3
kind:"subtitle"
label:null
language:"he"
mimeType:"text/vtt"
primary:false
roles:[]
type:"text"
videoBandwidth:null
videoCodec:null
videoId:null
width:null
__proto__:Object

(NOTE: a copy of the manifest text or an attached manifest will not be enough to reproduce your issue, and we will ask you to send a URI instead)
What did you do?
I  want to add subtitles to the  video so i did that
    const letSource =
      'MyUrl.mpd';
    player
      .load(letSource) // this here load the channels it gets !!!
      .then(() => {
        // This runs if the asynchronous load is successful.
        console.log('The video has now been loaded!');

         const [track] = player.getTextTracks();
         player.selectTextTrack(track.id); // => this finds the id and everythings fine but it does nothing
        const isBool = player.selectTextTrack(track.id);
        console.log(isBool); // => this returns undefined
      })
      .catch(this.onError); // onError is executed if the asynchronous load fails.

What did you expect to happen?
expected  isBool  to be a boolean
What actually happened?
isBool is undefined

As you observed, selectTextTrack has no return value.  The documentation agrees with that: https://v2-3-5-dot-shaka-player-demo.appspot.com/docs/api/shaka.Player.html#selectTextTrack
Have you tried enabling the text track with setTextTrackVisibility?  https://v2-3-5-dot-shaka-player-demo.appspot.com/docs/api/shaka.Player.html#setTextTrackVisibility

I have added  track.setTextTrackVisibility(true) to the line after const [track] = player.getTextTracks();
and still nothing I have  only the shaka default textTrack wich is playing nothing

still waiting for reply

@eladonline, JFYI, everyone on the team is on US/Pacific time, so you should not expect replies outside of normal office hours in our time zone.  When your own time zone is offset from ours by more than 6-8 hours, this introduces a lot of latency into communication.  That's just the nature of having a global conversation.  :-)
I think the problem is that it's not track.setTextTrackVisibility(true) as you wrote, but player.setTextTrackVisibility(true).  Please compare your code to the docs I linked above, and also check the JavaScript console.  If you call that method on Track instead of Player, you should see an exception in the console telling you that there is no such method.
Does that help?

joeyparrish Im sorry if you feel I was pressing you to answer it was not my intention
I just wanted to not be forgotten
now to the problem
I changed it to player.setTextTrackVisibility(true)
it made the default doing nothing "shaka track" disappear, but still doesnt play subtitles or making a subtitle item in the  subtitle  menu

No worries.
If it works in JWPlayer and not in your custom app, the next question is whether or not it works in our demo app.  Can you please try in our demo?  You can select "custom asset" from the assets list and enter a URL.  You will need to either allow cross-origin access to your asset or disable CORS restrictions in your browser.
I would also like you to confirm what version of Shaka Player you are using.  Your report says "5.6.0", but that version doesn't exist.  Our latest release is v2.3.6.  What version are you using?

hey sorry fro the wrong information
the currently use version is 2.3.4
(BTW I  updated to 2.3.6 and it threw a lot of errors so I downgrade it again)
I open my chrome with this command
chrome.exe --user-data-dir="C:/Chrome dev session" --disable-web-security
and when trying to run the mpd url on your 'shaka demo' it throws an err 1002

if you want the url of the mpd, I sent it to your supp email.
the head of the email holds this issue id track #1401
so you can try it too

I found your email, but it does not contain a manifest URI.  It appears to be just your email signature and no content.  Can you please send another email with the manifest URI?
As for HTTP_ERROR, that still appears to be CORS at work.  Due to a suspected Chrome bug, to disable CORS, you must use both --disable-web-security and open the debugger, go to the Application tab, click the "Service worker" section, and check the "Bypass for network" box.  Please do that and report your results.
Thanks!

@joeyparrish hey did you find something?

@eladonline, I'm sorry, we haven't looked at this yet.  Thank you for your patience.

Hi @eladonline, I took a look at your manifest and got an error when trying to display subtitles. I'll look into it later this week.
The original problem was that the subtitles didn't display on your custom app, is that right?

Got it. I should have an update for you by the end of the week.

Ok, looks like there is a problem with your subtitle file which prevents us from parsing it.
Here is the snippet from it:
3
00:00:07.000 --> 00:00:09.000
הרוכבים הפלמים
4
00:00:09.000 --> 00:00:11.000
5
00:00:11.000 --> 00:00:13.000
טסט קומיטד
It's not obvious with the comment formatting, but there are a bunch of newlines between cue #5 and it's payload (about 20 newlines in the actual file :)).
According to the VTT spec cue timimg should be directly followed with one newline character and then the text payload.
There is another spot like this in your vtt file: between cue #17 and it's text.
Once you fix the content everything should work :)

@ismena I did as you asked and it works issue can be close, tvm!
Need to mention that if One use shaka player with big companies
it is impossible to stand in the translation criteria
because every company use her tool to make translates and some make more spaces
this propblem is only in shaka player other players work well with this
maybe this issue should turn into a suggestion

@eladonline glad it worked! Let me asked our TL about supporting these kinds of files.
@joeyparrish
Tl;dr: apparently some tools used for translating vtt subtitles add more spaces between timing and payload than the spec allows. We can't parse them. Are we willing to modify the VttParser to be able to recover from this?
From code perspective, it's not a gigantic change, but not a small one either since a lot of our logic was built around the assumption that 2 or more new lines indicate a new text cue.

@eladonline, if you can provide us with samples and the tools that created them, we may be able to support these non-standard VTT files.  I would need to understand how they are created before we change the parser to accommodate.  Please file a new issue if you want to pursue this.  Thanks!


OWNERS: add TiDB cloud o11y team to owners

What problem does this PR solve?

Issue Number: close #209
What is changed and how it works?
Add the following github accounts to owners:

zhangpeijin-milo
z2665
just1900

/lgtm

/cherry-pick release-6.5


Issues on Ubuntu 15.10

Hi,
I'm having some issues with pulling data from a repository on Ubuntu 15.10. Below is a gist of the output from the git pull command and subsequent log messages.
Has anyone else run into this problem ?
https://gist.github.com/micahwalter/7f71ee26b3798b02309e2120e1ae10f9
thanks
Micah

Git LFS is disabled for this repository.

I checked, and your LFS access was disabled. My hunch is that you ran over last month, and then it rolled over without re-enabling access. I can look into that. Have your repo admin check for LFS emails from us about the limit. If you have more questions, hit me up at https://github.com/contact.

FYI this seems to work when I do
$ git lfs clone

vs.
$ git clone

Maybe that's just the way its supposed to be ?

@micahwalter: I wonder if it just worked as I ninja-fixed the LFS usage issue that I described? Let me know if it's still a reproducible problem.


Add t shirt design

We want a t-shirt. Needs design.

we have a t shirt, people like it and even donate for it. It's awesome.


Task chain rather than independent tasks

After the convo and confusion yesterday about tasks, schedules, jobs and airflow. I wanted to make sure that we're actually generating and representing the tasks in Airflow correctly and I'm seconding guessing a bit what we have.
Today we create 1 dag with indepenent tasks as created in: https://github.com/meltano/files-airflow/pull/18
So given a yaml like:
jobs:
- name: g-to-p-job
  tasks:
  - tap-gitlab target-postgres
  - tap-gitlab target-jsonl

That yields something like:

Are we sure we don't want the tasks to be linked e.g. task 1 depends on task 0 upstream instead like in the example below.

@pandemicsyn I was wondering this as well during the demo. If theyre to be run in order like Taylor described i.e. splitting a single command into tasks, then I'd expect them to have dependencies in Airflow like your second screenshot. Otherwise for tap-csv target-postgres dbt:run that gets split into the yaml below, you would have dbt run potentially at the same time or before the EL completes:
jobs:
- name: g-to-p-job
  tasks:
  - tap-gitlab target-postgres
  - dbt:run

@pandemicsyn great call out and I'd made a note to follow up about this. Yes, I would expect them to be sequential like you have them in the second picture. The parallel scenario we would eventually support by having nested arrays eventually I think.

Perfect, sounds like we're all in agreement! I'm gonna go ahead and merge this :shipit:


bcm43xxx: supported ARM CR4 core and bcm43455 chip in particular

Summary
Supported ARM CR4 core in bcm43xxx driver and bcm43455 chip in particular.
Impact
bcm43xxx driver
Testing

@a-lunev could you share some performance and stability data about bcm43xxx? We are very interested in the quality of this driver.

@a-lunev could you share some performance and stability data about bcm43xxx? We are very interested in the quality of this driver.

I have tested the performance neither based on a raw network traffic nor based on UDP.
Concerning TCP network traffic the peak payload throughput I observed is about 22-23 Mbit/s for BCM43362 chip, about 25 Mbit/s for BCM43438 chip, and about 40 Mbit/s for BCM43455 chip. The performance depends on different factors such as the performance of host CPU, radio-frequency interference etc.
The driver stability looks not bad if the network traffic is not high (less than the peak). If the network traffic is about to the peak, the driver (or chip) sometimes hangs up for some reason. In my previous PRs I fixed some stability issues, however as I can see there are still some hidden ones.

BTW, how much frequency do your host MCU run on?

BTW, how much frequency do your host MCU run on?

I tested at 120 MHz, 200 MHz and 400 MHz (ARM CM3/CM7). The higher host MCU frequency, the higher the driver performance. I suppose it may be even better at frequencies higher than 400 MHz.

Thanks a lot for your performance results，we are now debuging on bcm43438 and will refer your reply.
@xiaoxiang781216  ♡

One more question:), @a-lunev how much throughput you can achieve at 120MHz?

One more question:), @a-lunev how much throughput you can achieve at 120MHz?

I prepared configuration file (#5130) for Photon board (120 MHz host MCU, BCM43362 Wi-Fi chip) to measure wlan performance.
Based on the current state of the master branch:

the peak TCP payload throughput (unbuffered mode) is about 13 Mbit/s
the peak UDP payload throughput (unbuffered mode, CRC enabled) is about 17 Mbit/s
the peak UDP payload throughput (unbuffered mode, CRC disabled) is about 20 Mbit/s

Thanks @a-lunev we are trying your new config:).


Added Codomain attention blocks (CODABlocks)

Added one file in layers/codano.py
added test script
one additional dependency - added ''einops" to requirement.txt

@dhpitt can you also check after ICLR deadline?

Are there plans to introduce a full model using the CodaBlocks, or to add a
training recipe for CODANO?
On Wed, Oct 2, 2024 at 11:30 AM Md Ashiqur rahman @.***>
wrote:

@.**** commented on this pull request.
In neuralop/layers/codano.py
https://github.com/neuraloperator/neuraloperator/pull/436#discussion_r1785057905
:

@@ -0,0 +1,607 @@
+from functools import partial
+import logging
+import numpy as np
+import torch
+import torch
+from torch import nn
+import torch.nn.functional as F
+from .fno_block import FNOBlocks
+from .spectral_convolution import SpectralConv1d, SpectralConv2d, SpectralConv3d

+# Codomain Attention Blocks
+class CODABlocks(nn.Module):
It is a layer. I will change the name.
—
Reply to this email directly, view it on GitHub
https://github.com/neuraloperator/neuraloperator/pull/436#discussion_r1785057905,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJWQJOFNATDBOER2LZUTGBLZZQ3T5AVCNFSM6AAAAABOIMDMECVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNBTG44TSOJSGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

We will release the whole model. But that will be done as the next step.

Superseded by #462 (merged)


⚠ Bofrost.be has degraded performance

In c1332c1, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1452 ms

Resolved: Bofrost.be performance has improved in 7b1a4a8.


Aspnetcore IIS module and process lifetime.

I have deployed the new module.  If I stop the is application pool the dotnet process eg somewebsite.exe stays alive.  I have to kill it to do a publish.  I would expect stoping the application pool to also shutdown or kill the process or am I missing something?

How did you stop the application pool? The child process should be terminated automatically by the OS unless it was attached by the debugger or hold some resource. I could not repro this issue locally. Could you please share a simple repro for investigation.

How did you stop the application pool?
By using the app pools node in IIS manager .
I will investigate further and come back.

Sorry Guys.
Happy to report it was because when I recreated the app it defaulted to "Default app pool" and not the one it was previously using which was the one I was restarting.  No wonder it wasn't affecting the process.
Assigning the app to the correct app pool solved the issue.
Please close.


Laravel Eloquent @property don't show up in suggestion

I use Intelephense long time, right now I have some big problem.
When I write the model in laravel and set @property:
/**
 * New model for test
 *
 * @property mixed $test
 * @var mixed $test2
 * @method viod testMethod
 *
 */
class NewModel extends Model

and when I'm doing:
$newModel = NewModel::first();
$newModel->[here no sugesstion for any of DockBlock position]

A few days ago it was working correctly, now I have problems.
Php: 5.3.68 and 7.1.24 (7.1.24 was set to check Intellisense)
VS Code: 1.30
Os: Ubuntu LTS 16.04
All other extensions all off right now.
Help wanted :)

Are you sure you are in the right repository? :-)
I noticed you mention VS Code, but this is the repository of LSP for Sublime Text

Never mind... :P


Fix english translation

Hmm, it’s strange that validation reports error but it works :(
I was hoping it can be done this way because it’s backward compatible.
So it looks support for translations would be a lot harder…