Should we apply consistent plane-of-array irradiance for calculating cell temperature and effective irradiance?

Hi there,
I have recently started working with pvlib-python (v0.8.0). I have gone through the basic A simple ModelChain example tutorial. It works well. But after exploring mc attributes, I got myself confused with irradiance for calculating cell temperature and effective irradiance, which I really appreciate your help.
I found that cell temperature could be calculated as:
# equal to mc.cell_temperature
mc.total_irrad['poa_global'] \
* (np.exp(temperature_model_parameters['a']
          +temperature_model_parameters['b']*weather['wind_speed'])) \
+ weather['temp_air'] \
+ mc.total_irrad['poa_global'] / 1000 * temperature_model_parameters['deltaT']

Hence cell temperature is based on EPOA without any reflection, shading, or soiling losses.
Then I found that effective irradiance could be calculated as:
# equal to mc.effective_irradiance
mc.spectral_modifier * (mc.aoi_modifier*mc.total_irrad['poa_direct']+mc.total_irrad['poa_diffuse'])

rather than:
# unequal to mc.effective_irradiance
mc.spectral_modifier * mc.total_irrad['poa_global']

Hence I believe that the reflection losses (i.e. AOI modifier) is taken into account in calculating effective irradiance.
This really bothers me despite small differences in results. I think that the only difference in calculating cell temperature and effective irradiance is that the later contains additional spectral losses? The two should be consistent in including/excluding reflection, shading, and soiling losses according to modeling steps described here or here?
Yours faithfully,
Fei

@FeiYao-Edinburgh,
I suppose the explanation depends on how 'plane of array irradiance' is defined. In pvlib-python, 'poa_global' is the broadband irradiance reaching the face of the module, i.e., reduced by shading and soiling but not adjusted for spectrum or reflections. The tutorial page is a little confusing in that it does not define either plane-of-array or effective irradiance irradiance. I think that page is meant as a general guide rather than a precise description.
In pvlib effective irradiance is the irradiance reaching a module's cells, i.e., 'poa_global' adjusted for spectrum and reflection.
The challenge is with the temperature model coefficients. The values supplied with pvlib have been determined by regressing measured module backsheet temperatures to plane-of-array irradiance measured with an in-plane broadband pyranometer (and also to ambient air temperature and wind speed). As a consequence, the irradiance used to fit the model is not adjusted for reflection or spectral modifiers. That's why pvlib uses 'poa_global' in the temperature functions.
However, IMO when 'poa_global' is not available (e.g., when irradiance is measured in-plane with a reference cell) it is acceptable to use effective irradiance instead of poa_global with the same model parameters. The difference in model output will be on the order of one degree C, which is similar to the uncertainty of the temperature models.

@cwhanse
Thanks for your reply. I believe your key points are (in pvlib-python):

plane-of-array irradiance (poa_global) is the irradiance reaching the face of the module, accounting for shading and soiling.
effective irradiance is the irradiance reaching the modules' cells, furthering account for spectrum and reflections.

This is different from my understanding that the plane-of-array irradiance may include reflection adjustment. I came to think so simply because I found that shading, soiling, and reflection losses have been described before introducing module/cell temperatures in this page? Probably I was wrong?
Nonetheless, I feel relaxed to hear that it is acceptable to use either poa_global or effective irradiance to calculate cell temperature. Personally I am interested in plane-of-array irradiance reductions due to soiling caused by aerosols deposited on solar panels. I believe that is fine to be included for calculating cell temperature, simply because you mentioned that poa_global accounted for shading and soiling is originally designed for calculating cell temperature?
I am looking forward to your further clarification. Many thanks in advance!
Yours faithfully,
Fei

This is different from my understanding that the plane-of-array irradiance may include reflection adjustment. I came to think so simply because I found that shading, soiling, and reflection losses have been described before introducing module/cell temperatures in this page? Probably I was wrong?

Hmm, that's a good point. In pvlib we also use 'poa_global' to refer to the output of irradiance transposition (from GHI, DNI and DHI to plane of array). I can see where there may be some confusion. Since there haven't been shading and soiling models in pvlib until quite recently, the output of the transposition process was the irradiance reaching the module's face. Perhaps pvlib may need a second term to distinguish between transposed irradiance (before shading) and irradiance reaching the module's surface (after shading.)

Personally I am interested in plane-of-array irradiance reductions due to soiling caused by aerosols deposited on solar panels. I believe that is fine to be included for calculating cell temperature according to your reply?

I'm not clear what you mean by "that is fine to be included": do you mean the irradiance reflected away by the soiling?

Perhaps pvlib may need a second term to distinguish between transposed irradiance (before shading) and irradiance reaching the module's surface (after shading.)

Do you think it is fine to add soiling to your sentence to make it as follows?
Perhaps pvlib may need a second term to distinguish between transposed irradiance (before shading and soiling) and irradiance reaching the module's surface (after shading and soiling.)

I'm not clear what you mean by "that is fine to be included": do you mean the irradiance reflected away by the soiling?

Sorry for the confusion. Yes I mean we should account for both shading and soiling to convert transposed irradiance to irradiance reaching the module's surface (to be named). The later is the correct one for further calculating cell temperature?

Perhaps pvlib may need a second term to distinguish between transposed irradiance (before shading and soiling) and irradiance reaching the module's surface (after shading and soiling.)

Yes, exactly.

Sorry for the confusion. Yes I mean we should account for both shading and soiling to convert transposed irradiance to irradiance reaching the module's face (to be named). The later is the correct one for further calculating cell temperature?

Yes. Thinking more about the lab setup that produces the data behind temperature model parameters, the measurements reflect any shading or soiling of the module or pyranometer. So it is appropriate to account for these effects on the irradiance going into the temperature models.

Perhaps pvlib may need a second term to distinguish between transposed irradiance (before shading) and irradiance reaching the module's surface (after shading.)

How about global_incident. I believe this is the terminology used in PVsyst.. Or perhaps global_shaded_incident to be even more specific?

global_shaded_incident

IMO, this term may not be able to reflect soiling.

I came to think so simply because I found that shading, soiling, and reflection losses have been described before introducing module/cell temperatures in this page?

Then would it be great to move descriptions of reflection losses to places like between cell temperature and effective irradiance or within effective irradiance?

I'll first say that other than the irradiance input to the cell temperature functions, we haven't identified where pvlib actually needs a new term. Do you see any other place where there's a gap or snag in the calculation process?
If pvlib gets a new variable name, I think that 'poa_global' should remain as the output of the irradiance transposition step, i.e., before shading, soiling, reflection or spectral adjustment.  The quantities between 'poa_global' and 'effective_irradiance' are possibilities for naming.  I've seen "incident irradiance" used for the irradiance reaching the surface of a module, and "absorbed irradiance" for the irradiance reaching the cells prior to spectral adjustment.

Do you see any other place where there's a gap or snag in the calculation process?

No. I have just started learning and using pvlib-python.

The quantities between 'poa_global' and 'effective_irradiance' are possibilities for naming. I've seen "incident irradiance" used for the irradiance reaching the surface of a module, and "absorbed irradiance" for the irradiance reaching the cells prior to spectral adjustment.

This makes sense in that:

incident irradiance: the irradiance reaching the surface of a module, accounting for shading and/or soiling losses
absorbed irradiance: the irradiance reaching the cells, further accounting for reflection losses
effective irradiance: the irradiance reaching used by cells with spectral adjustment

Yes I agree that it might be enough to make a memo on this issue for the time being.
Thanks for the discussion.


updated phpstorm-eap (2016.1.1)

Changes to a cask
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Hold, needs fix.

Now ready.


Support for Swift 4

Hi! Thanks for great lib!
This library does not support swift 4 now. Any solution?

I made a PR for this #161
However some of the UI is acting funny and it'd be nice to get some help on it.


Repository naming

It seems undesirable that the repository has a dash in the name. Is there a reason for this choice?
It's common practice to have the repository be named the same as the package.

We also have a package (of the same name) for R, we added the -py to delineate the difference between the two

We also have a package (of the same name) for R, we added the -py to delineate the difference between the two

I'm not against the "py" part but I'd like to remove the dash. The dash only exists in the repository name and not in the sub-folder or the package name.

I'm going to reiterate that I strongly suggest renaming the repository to "kgcpy" instead of "kgc-py". This is a very easy fix.
I also suggest changing the package name from "kgcPy" to "kgcpy" to follow the official python convention:

should have short, all-lowercase names. Underscores can be used in the module name if it improves readability. Python packages should also have short, all-lowercase names, although the use of underscores is discouraged.

yeah, agreed, done. this shouldn't break anything since pip is case insensitive, yes?

yeah, agreed, done. this shouldn't break anything since pip is case insensitive, yes?

people now have to do import kgcpy instead of kgcPy, so it is a breaking change


HDDS-4984. Support async profiler 2.0

What changes were proposed in this pull request?
Due to the update of Async profiler 2.0, the output format is changed from svg to html.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4984
How was this patch tested?
Local test.

@adoroszlai  is it? i think it threw an error at me when i specified svg output...
regardless, the html report is easier to use.

So... looks like even with the change, you would still need to point the browser to /prof?output=flamegraph in order to use async profiler 2.0, otherwise the servlet will attempt to generate svg output and that will fail.

how about adding another system property  to specify the async profiler version, and defaults to flamegraph if it's 2.0?

2.0 works with -o svg, although profiler's help does not mention it as a valid output format.
is it? i think it threw an error at me when i specified svg output...

I think I was fooled by some docker-specific behavior (tried to replace /opt/profiler at runtime...).
Now with proper 2.0 installation I see the problem with -o svg.
It might be improved by checking the subprocess for error result instead of waiting forever for the output file.

how about adding another system property to specify the async profiler version, and defaults to flamegraph if it's 2.0?

We could also run the profiler with -v first to get the version number.

+1 as discussed offline, let's merge this first, and we can work on the follow-up later.

Did we open the follow-up issues? Looks like ideal newbie issues....

Thanks @elek . I have issued jira and will finish it in a few days. The website of jira is: https://issues.apache.org/jira/browse/HDDS-5009


Stop CI overriding the set config value for Maximum Execution Time, when running from CLI

CI overrides the PHP Maximum Execution Time value to 300 seconds. Whilst this is sensible in the vast majority of situations (browsers), when running a script from CLI, it is likely that execution times may need to be longer. Therefore, don't override the time limit if being run from the CLI - instead default back to PHP's own configuration.
I have been writing an extensive database migration script (copying over GBs of data from an old structure to a totally new one). I'm using CI to format the data and handle the SQL. When performing a very large batch_insert() the execution timed out, even though I'd set php.ini to 3000 seconds rather than 300. I found the culprit to be CodeIgniter setting this value itself.

why not do this:
https://github.com/EllisLab/CodeIgniter/blob/develop/system/core/Input.php#L739


:book: doc fix: ArgoCD pull solution doc should render on browser folder entry

Summary
The argocd pull solution is not displaying nicely when browsing to https://github.com/open-cluster-management-io/ocm/tree/main/solutions/deploy-argocd-apps-pull
It should behave similar to https://github.com/open-cluster-management-io/ocm/tree/main/solutions/deploy-argocd-apps where contents are displayed upon browser entry.

/assign @qiujian16

/lgtm


fabric - create a BP sequence.

force mini_core_ready to be 0, create 30-40 trans to fill all fifo's in the design. then release the mini_core_ready and see how the system reacts.

show the a wave view of all fifo full indication of all fifo in path.

sequence working correctly, now need to generalize it.
picture of waves:

@ShmuelSfez - looking at the fabric level0, I dont see param overrides. is that expected?


Open fails when given no feet

julia> OpenStockAndFlowF{Symbol, Symbol, Int8}(StockAndFlowF())
ERROR: Empty list of legs but no apex given
Stacktrace:
 [1] error(s::String)
   @ Base ./error.jl:35
 [2] Multicospan(legs::StaticArraysCore.SVector{0, Union{}})
   @ Catlab.CategoricalAlgebra.FreeDiagrams ~/.julia/packages/Catlab/rJ36m/src/categorical_algebra/FreeDiagrams.jl:175
 [3] (SMulticospan{0, Ob, Hom, <:StaticArraysCore.StaticArray{Tuple{0}, Hom, 1}} where {Ob, Hom})()
   @ Catlab.CategoricalAlgebra.FreeDiagrams ~/.julia/packages/Catlab/rJ36m/src/categorical_algebra/FreeDiagrams.jl:185
 [4] (StructuredMulticospan{Catlab.CategoricalAlgebra.StructuredCospans.DiscreteACSet{StockAndFlow0, StockAndFlowF}})(::StockAndFlowF)
   @ Catlab.CategoricalAlgebra.StructuredCospans ~/.julia/packages/Catlab/rJ36m/src/categorical_algebra/StructuredCospans.jl:267
 [5] top-level scope
   @ REPL[25]:1

(same error when doing Open(StockAndFlowF()) )

Trying to deal with cases where you're composing StockFlows with an empty list of feet.
Could provide feet with no stocks and flows to every sf, though that doesn't seem like a good solution.

I think you specify the empty interfaces as the empty list [] as opposed to a missing argument.


feat(Trie): completion API

closes #122
@favonia The stuff added to Example.ml might not be according to your aesthetic preference, please advise :)
I think that the complete function should actually return the data in the trie as well, since we might want to further filter the results. For example, we might only want to suggest items that match a certain type. Will update sometime soon.
tagging @jonsterling

@favonia I decided on the name complete at the very beginning, but now I am not so sure that it is the right name.

Is it technically possible to separate it out as an individual module?

As far as I can tell this could be implemented by a consumer of Yuujinchou, so yes.
It's interesting that string based tries are used to implement autocomplete, but this fact is not used here. Maybe a guiding principle for redesign can be to share code between path-based and string-based tries? Maybe an API that converts a namespace into a datastructure that can be used to power an autocomplete engine?

I pushed a branch of forester that has implements this functionality using the existing API, so the changes I made aren't strictly necessary.

@kentookura I will check the code after I finish my current work on asai...


Keyboard hides the WiFi password input on Android

Describe the bug
The keyboard shows on top of the WiFi password entry window and covers the input, making it impossible to see what you are typing or if your key press got accepted.
To Reproduce
Pair with hostpot over BT and attempt to enter new WiFi password.
Expected behavior
The input box should shift up and display above the keyboard.
Screenshots
Link to the screenshot posted in Discord
Smartphone (please complete the following information):

Device: Samsung Galaxy S9+
OS: Android 10
App Version 3.6.0

is this issue still happening?

I am unable to get to that screen to verify. I have one Nebra hotspot here that I tried with, it is connected to Ethernet so when I get to the WiFi screen it tells me it is connected to Ethernet and there is no list of SSIDs that I could try, clicking on refresh does nothing to bring a list of WiFi networks, also unplugging the cable did not help. I tried multiple times, lost more than 30 min rebooting, pairing over BT and trying again and again.

OK, I was able to verify that the issue is still there. My app version is 3.8.2
This is how it looks before you open the keyboard:

This is with Microsoft SwiftKey Keyboard:

And this is with the Samsung Keyboard:


Not all necessary tiles are loaded after a zoom change (demo video)

Good job with this library @johnpryan and all contributors!
I'm experiencing a bug where after zooming in to a new level, not all tiles that should appear on the screen are loaded. Moving the camera is needed to make them load.
Video demonstrating a zoom increase with not all tiles (bottom left in this case) loading until i start scrolling on the map again. https://imgur.com/a/3kfI3qM
The tiles provider used was AssetTileProvider().
Here is the code but I'm not sure how relevant it is:

FlutterMap(
  mapController: mapController,
  options: MapOptions(
    center: lisbon,
    zoom: 13.0,
    minZoom: 13.0,
    maxZoom: 16.0,
    swPanBoundary: LatLng(38.358, -9.634),
    nePanBoundary: LatLng(39.15, -8.458),
  ),
  layers: [
    TileLayerOptions(
      keepBuffer: 4,
      tileProvider: AssetTileProvider(),
      urlTemplate: "assets/map/{z}/{x}/{y}.png",
      maxZoom: 16,
      tms: true,
      backgroundColor: Colors.white,
     ),
     MarkerLayerOptions(
       markers: markers,
     ),
   ],
),

I would love to contribute myself but I'm fairly novice and looking at the source code seems very complex. Hoping to see this and the keep the old tiles until the new ones are loaded improvements for 0.8 as I truly believe that would make this the best flutter map library!

I'm looking for feedback on a similar issue if interested, which may fix it...
https://github.com/johnpryan/flutter_map/issues/79#issuecomment-586135554


PYTHONDONTWRITEBYTECODE

We should play with exporting PYTHONDONTWRITEBYTECODE instead of deleting the pyc files after the fact!
See also https://docs.python.org/3/using/cmdline.html#envvar-PYTHONDONTWRITEBYTECODE and https://docs.python.org/2/using/cmdline.html#envvar-PYTHONDONTWRITEBYTECODE.

Played with this a little bit, and it looks like the Python build scripts create *.pyc files unconditionally.
Additionally, get-pip.py creates *.pyc files too, which I'm guessing are because it's pulling in pre-compiled wheels, so the *.pyc files are coming in as part of what's being downloaded/installed, not being generated by Python or pip.

FWIW *.pyc are generated by pip but you can provide --no-compile to get-pip.py (and/or pip) and it shouldn't create the .pyc files.

Indeed!  I did some experimentation:
diff --git a/3.7/alpine3.10/Dockerfile b/3.7/alpine3.10/Dockerfile
index 05dfb5f..63c4bc2 100644
--- a/3.7/alpine3.10/Dockerfile
+++ b/3.7/alpine3.10/Dockerfile
@@ -152,14 +152,16 @@ RUN set -ex; \
        wget -O get-pip.py "$PYTHON_GET_PIP_URL"; \
        echo "$PYTHON_GET_PIP_SHA256 *get-pip.py" | sha256sum -c -; \
        \
+       export PYTHONDONTWRITEBYTECODE=1; \
        python get-pip.py \
                --disable-pip-version-check \
                --no-cache-dir \
+               --no-compile \
                "pip==$PYTHON_PIP_VERSION" \
        ; \
        pip --version; \
        \
-       find /usr/local -depth \
+       echo find /usr/local -depth \
                \( \
                        \( -type d -a \( -name test -o -name tests \) \) \
                        -o \

Running find /usr/local -name '*.pyc' in the resulting image came up empty, which is interesting. :smile:

Coming back to this with fresh eyes, I'm not convinced adding this makes our Dockerfile more clear (or really much/any more efficient), especially since we'd probably still want a test just to make sure we didn't somehow regress. :see_no_evil:  As such, I'm leaning towards closing this. :see_no_evil:

I think this could greatly simplify Dockerfile's and slightly improve build times. Also, I should note that many developers copy some parts of official Dockerfile's and using best practices would be a good thing. Regarding testing for regression, IMHO that there probably should already be a test to check that images don't contain py[co] files in case somebody breaks cleanup.

Yeah, that's fair -- I suppose it would make more sense to make that one of our post-build integration tests instead of part of the Dockerfile itself.

@tianon Great news. Thank you for all the work)


Issues with powerdot clock

Hi,
I'm trying to use animate with powerdot, while also using powerdot's clock option. The problem is that the clock isn't showing up, just a blueish rectangle where it should come up.
On 2011 animate received a fix to deal with powerdot's clock properly, as stated on this post:
https://www.freelists.org/post/powerdot/Problem-using-clock-with-animate-package,1
Looking at animate.sty revision I see that the fix was introduced in 2011, but it was removed some time later. My current version (Jan 2017) doesn't show the modifications.
I also tried setting pd clockformat and clockrefresh rate as suggested to no avail.
Here's a minimal example:
\documentclass[
                                style=simple,
                                paper=screen,
                                mode=present,
                                fleqn,
                                orient=landscape,
                                clock]{powerdot}

\usepackage{animate}

\pdsetup{
        clockformat=HH:MM,
        clockrefresh=60000
}

\pddefinetemplate[slide]{slide}{
 textpos={.20\slidewidth,.75\slideheight},
 textwidth=.75\slidewidth,
 titlepos={.15\slidewidth,.85\slideheight},
 tocpos={.015\slidewidth,.75\slideheight},
 clockpos={.23\slidewidth,.08\slideheight},
}{}

\begin{document}

\begin{slide}{test}
        \begin{itemize}
                \item 1
                \item 2
        \end{itemize}
\end{slide}

\begin{slide}{animate slide}
\centering\animategraphics{1}{fundo-slides}{1}{1}
\end{slide}

\end{document}

Thanks!

I wrote a package which replaces the Powerdot clock and maintains compatibility with animate:
pdclock.sty

pdclock.sty is not required anymore as it was incorporated into Powerdot.


Fix errors for RSGCN-CWLE/GWLE

Fix train_molnet_wle.py: dictionary key setting
FIx predict_molnet_wle.py: import error

Codecov Report

Merging #424 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #424   +/-   ##
=======================================
  Coverage   83.38%   83.38%
=======================================
  Files         270      270
  Lines       13265    13265
=======================================
  Hits        11061    11061
  Misses       2204     2204


Сonfusion with minimal Python version

Chore
Describe the chore
Different parts of the project specify different minimum python versions:

In the project's README.md ^3.9
In badges in README.md - 3.8 | 3.9 ...
In documentation ^3.7
In CI tests run with ^3.9
In pyproject.toml the version is ^3.8

I consider the source of truth is pyproject.toml, so I think it would be great to bring these places to a unified form:

Put Python version >=3.8 in the documentation
Start running tests with version 3.8

The minimal version this project supports is 3.9. Please refer to the Python docs about supported versions of Python. https://devguide.python.org/versions/

I've updated the readme and the pyproject.toml to reflect the support from 3.9 upwards.


Short solution needed: "ffmpeg aac encode/decode" (ffmpeg)

Please help us write most modern and shortest code solution for this issue:
ffmpeg aac encode/decode (technology: ffmpeg)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

Encode mp3 to aac


AWS shield - sync action is acting on false positive resources

Policy
  - name: shield-protection-elb
    resource: elb
    filters:
      - type: value
        key: Scheme
        value: internal
        op: not-equal
      - type: shield-enabled
        state: false
    actions:
      - type: set-shield
        state: true
        sync: true

Problem - While trying to enable shield protection on resources that are region specific and have a sync = true on the policy the resources that are added to shield are removed when the policy runs on the next region. For eg: you have 10 resources in us-east-1 that are added to shield, when you run the above policy in west-2 to add resources custodian removes the resources in east-1 because they are not available in west-2

we'll need to handle this across regional resources that shield protects.


Chrome headless integration tests

#2880 WIP:
Current test status:

✅ Win 10
✅ Ubuntu 16.04
✅ Ubuntu 18.04
❌ Win Server 2012 R2 https://github.com/aspnet/KestrelHttpServer/issues/3027
❌ Win 8.1 https://github.com/aspnet/KestrelHttpServer/issues/3027

I'll need to investigate if there's any workarounds for #3027.

:up::date:

Verified the tests complete successfully on a fresh Win2012 Server. Ready to merge after a final inspection.


NIFI-5872 - Added compression option to JsonRecordSetWriter

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[x] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Ran a test on my laptop with a workflow generating a lot of data:

without compression, I reached 129M events / 11.15GB out per 5 minutes
with compression (Snappy), I reached 161M events / 1GB out per 5 minutes

It improves the number of processed eps while reducing the amount of data on disk.


[Linux] Frequent crashes when starting application, X-related

Hi,
recently I'm having plenty of (non-deterministic) crashes when starting my egui-based application.
[xcb] Unknown sequence number while processing queue
[xcb] Most likely this is a multi-threaded client and XInitThreads has not been called
[xcb] Aborting, sorry about that.
megra_rs: xcb_io.c:269: poll_for_event: Assertion `!xcb_xlib_threads_sequence_lost' failed.

Not really sure what this means. Usually I have to try two or three times, then the application starts. Any hints ?
I'm on Arch, Xorg, AwesomeWM
Best,
N

I see get the same message, not just frequently, but any time I try to use the lib through epick.
Arch, xorg, dwm here.

I've got exactly same error message above.
I'm on Arch, i3wm

There was something in the underlying winit library, supposedly fixed:
https://github.com/rust-windowing/winit/issues/458

I get this error as well (also on Arch Linux with i3, running the egui_template or the demo app).
Does anybody know how to overcome this?
The web backend works, fortunately, but it would be nice to run apps without a browser and the limitations of WASM.

Because this was tagged with egui_glium I tried to run glium's examples. 80% percent fail with the same error that everybody here gets (the other 20% fail in different ways).
I cloned egui_template again, and tried to execute the example without modifying the code in any way. The app opened once, but not again after.

I have the same issue on every start, deterministically. Currently running Gentoo and available for further debugging.

Similar issue, reported closed: https://github.com/rust-windowing/winit/issues/458

Because this was tagged with egui_glium I tried to run glium's examples. 80% percent fail with the same error that everybody here gets (the other 20% fail in different ways).

Let's  figure out if this is a problem with glium or winit. There is a new glium-free backend in the works at https://github.com/emilk/egui/pull/685 - can you check it out and try to run it?

I cloned egui_template again, and tried to execute the example without modifying the code in any way. The app opened once, but not again after.

try removing the state at /home/UserName/.local/share/egui_template - does it work now? If so, see if you can figure out what part of the config file is causing the problem (maybe restoring the position or the size of the window?)

Just ran the demo app without glium and it worked a charm. Running the demo app with glium, however, works as well. I'm baffled. Could it be there's an issue with eframe? The bug seems too low level to be, but I cannot repro on my side.

Nevermind. Started doing dev on my side when the issue popped up again, and quickly switched my deps to the glow branch. Still got the issue. So it's not only glium.

Since I'm running Gentoo after all I've been compiling a lot of stuff. And I don't know if this is a lead or not, but there seem to be less crashes when my computer is busy doing something.

After some testing with the glium-free backend, it seems this is not a problem with glium. I'd advise to remove the egui_glium tag,

Yes, just tried with the glow backend, same problem.

I get also the same error message using the eframe_template on Kubuntu 21.10:
[xcb] Unknown sequence number while processing queue
[xcb] Most likely this is a multi-threaded client and XInitThreads has not been called
[xcb] Aborting, sorry about that.
egui_test: ../../src/xcb_io.c:269: poll_for_event: Assertion `!xcb_xlib_threads_sequence_lost' failed.

I ran into the same !xcb_xlib_threads_sequence_lost assertion a few days ago.
For me any egui example I try to run crashes 100% of the time, so my system might be a good environment to reproduce this problem.
Since https://github.com/rust-windowing/glutin/issues/1034#issuecomment-1007958758 suggests that it might be window manager related, I'd like to add that I also use i3 (also on Arch Linux).
I did not yet test the glow backend, but I will give it a try if can free up some time later today.
If there is anything else I should give a try just point me at piece of code and I'll try it out.
Likewise if there is any additional information about my system (software versions, graphics hardware...) that might be helpful, just ask.

Nope ... unfortunately the glow_example #685 runs into the same error:
[sebastian@laptop egui_glow] (master)$ cargo run --example pure
    Finished dev [unoptimized + debuginfo] target(s) in 0.16s
     Running `/tmp/egui/target/debug/examples/pure`
[xcb] Unknown sequence number while processing queue
[xcb] Most likely this is a multi-threaded client and XInitThreads has not been called
[xcb] Aborting, sorry about that.
pure: xcb_io.c:269: poll_for_event: Assertion `!xcb_xlib_threads_sequence_lost' failed.
Aborted (core dumped)

Just for kicks I patched my mesa library using https://bugs.freedesktop.org/attachment.cgi?id=143767 and it actually seems to solve the problem. All examples I tested work now.
So if this is actually a mesa bug creating an error where there should be, is there anything that can be done to work around it on the application side?

I can confirm the same issue on Arch under both i3 and Plasma. I tried patching mesa but did not succeed in doing so at all.

Hmm I just updated to egui 0.17.0, using glow instead of glium. Seems like it doesn't happen anymore ? But there's other things that changed in my system configuration as well, so I can't really tell what fixed it.

Should we perhaps close this in favor of https://github.com/rust-windowing/glutin/issues/1034 ?

Not opposed to that ...

Closed in favor of https://github.com/rust-windowing/glutin/issues/1034


:package: feat(bundlers): Add rspack plugin

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #118

PR #119

PR #120

PR #121

PR #122

PR #123

PR #124   👈

This comment was auto-generated by Graphite.


refactor(shell): Use alternate to close links

What does this PR try to resolve?
Simplifies the code for includign hyperlinks, especially the fact that this allows for including the variable in the format string.
How should we test and review this PR?
Hand tested each case as we don't have SVG support for hyperlinks and we don't have SVG tests for these cases
Additional information

Tested. Thanks!
@bors r+

:pushpin: Commit af9f1346a269e1fb14d0fa44852da0daecb22d50 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit af9f1346a269e1fb14d0fa44852da0daecb22d50 with merge 36cf66a476b0b90fe1a920592edae2b2d5ed2147...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 36cf66a476b0b90fe1a920592edae2b2d5ed2147 to master...


Kernel should be loaded regardless of case used

Currently the kernel service provide loads the site kernel from kernel.php. To keep it consistent with Style-CI it should load from Kernel.php as well as supporting the lower case alternative for backwards compatibility.

Fixed by #237


Ability to use token embeddings?

Feature request
Hello, first thank you for the great work for this library and making it open source. I wanted to ask a possibility to get the token embeddings directly from the input. I was not able to find that option. If it's possible it would be great to imitate sentence transformers behaviour
model = SentenceTransformer("all-mpnet-base-v2")
model.encode("This is a test sentence", output_value="token_embeddings")

output-value can be added to the api parameters if possible. Moreover, if possible other types of poolings can be utilized on runtime (I don't know if this is possible in the library)
Motivation
I want to use late chunking and for that, the token embeddings are required.
Your contribution
I can try to help.

Hi, are there any updates on this?


Support numberDecimal and numberSigned keyboard type

Following on from #14554, TextInputType does not appear to support explicitly requesting numberDecimal, numberSigned, or a combination of those.
Without this, it's not possible to show a numeric soft keyboard that allows only number, decimal, and minus (negative).

Android documentation seems to imply that InputType.number only includes digits, and you can combine it with numberDecimal and/or numberSigned if you also want additional symbols: https://developer.android.com/reference/android/widget/TextView.html#attr_android:inputType
However, this behavior doesn't seem to be enforced by some firmwares, as some devices show additional symbols even if only InputType.number is specified without additional flags. I tested this on Nexus 5X (7.1.1) and Nexus 6P (6.0), and both of them show additional symbols that can't be disabled.
So we have a couple of options for Flutter's TextInputType:

(Preferred) Keep TextInputType.number mapped to InputType.number as it is today, and add new enum values TextInputType.numberDecimal and numberSigned, to allow your app to explicitly enforce additional symbols on the keyboard if needed.

Translate TextInputType.number to combination InputType.number|numberDecimal|numberSymbol. This way, you will lose device specific behavior, and Flutter will always enforce all numeric symbols to appear on the keyboard (which is why this might not be such a great idea). At least on some devices, this will be a breaking change, although probably not a huge one.

In either option, you won't be able to reliably disable extra symbols no matter what, since Android API doesn't seem to allow this, at least on some devices.
I'm leaning towards 1), but thoughts are welcome.

Does not appear to allow numberDecimal and numberSigned simultaneously, unless you're thinking of making the enum values bitfields?  Note also there are other flag bits, e.g. textAutoCorrect/TYPE_TEXT_FLAG_AUTO_CORRECT
https://developer.android.com/training/keyboard-input/style.html#Spelling
https://developer.android.com/reference/android/text/InputType.html

Seems fine to me.

I see a comment on top of TextInputType:
/// On Android, behavior may vary across device and keyboard provider.
This seems to imply that option 1 was assumed in the original design, but I'll get opinions from the team to narrow down our options.
JFYI, Flutter supports autocorrect flag as separate parameter in widget constructor (not a bit field).

For option 1, we would probably introduce a full set of enum values in TextInputType (number, numberDecimal, numberSigned, numberSignedDecimal).
For option 2, we would simply always enforce number|numberDecimal|numberSigned on Android, in TextInputPlugin.inputTypeFromTextInputType().

Option 1 sounds more versatile.  /cc @jason-simmons

I prefer #1, but adding numberSignedDecimal is a red flag to me. What if Android adds another flag, like TYPE_NUMBER_FLAG_HEX ? Do we now add another 4 enum types? And when they add TYPE_NUMBER_FLAG_INFINITY we add another 8? This doesn't scale...
Here's a crazy idea for an option 3:
class TextInputType {
  const TextInputType(this.index) : negative = null, symbols = null;
  const TextInputType.numeric({ this.negative: false, this.symbols: false }) : index = 2;

  final int index;
  final bool negative;
  final bool symbols;

  static const TextInputType text = const TextInputType._(0);
  static const TextInputType multiline = const TextInputType._(1);
  static const TextInputType number = const TextInputType.numeric();
  static const TextInputType phone = const TextInputType._(3);
  static const TextInputType datetime = const TextInputType._(4);
  static const TextInputType emailAddress = const TextInputType._(5);
  static const TextInputType url = const TextInputType._(6);

  static const List<TextInputType> values = const <TextInputType>[
    text, multiline, number, phone, datetime, emailAddress, url,
  ];

  operator == (dynamic other) { ... } // compares all the fields
  int get hashCode => ...; // handles all the fields
}

Just came about this when trying to set just TextInputType.number on my emulator of a Pixel2 I still get  ',' '.' and can add as many '-' or spaces

You're getting the minimal possible keyboard provided by the underlying
device & OS that still has numeric features, which could mean that it might
have more than what was requested.
On Fri, Jun 8, 2018 at 2:42 AM escamoteur<EMAIL_ADDRESS>wrote:

Just came about this when trying to set just TextInputType.number on my
emulator of a Pixel2 I still get ',' '.' and can add as many '-' or spaces
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/14916#issuecomment-395709137,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBIjqnGwawfYihTWIgnFkaxB0ntTxAiks5t6kcagaJpZM4SVJHX
.

Add this property to your text field
 keyboardType: TextInputType.numberWithOptions(decimal: true,),


feat: tracked entity input (DHIS2-16023)

Implements DHIS2-16023
Requires https://github.com/dhis2/analytics/pull/1594
Figma
Spec
Netlify
Cypress bot message
Cypress dashboard

Key features

Adds support for the new input type Tracked entity

Description
The following is only a quick summary of what this PR contains, for full details see the spec and Figma links above.
A new input type ("Tracked entity") was added to the input panel for creating TE line lists. After selecting a tracked entity type (TET), associated dimensions can be selected through the new TET dimension menu item (e.g. "Person dimensions"). There are two new global dimensions, "Registration org. unit" and "Registration date". The program dimensions panel now supports cross-program selection for TE.
All of the above are featured toggled to v41. Because of that, the nightly.yml workflow has temporarily been modified to run the CI Cypress tests against dev.
Because of the cross-program aspect of TE, all uses of dimension id's now need to go through extractDimensionIdParts and formatDimensionId to be prefixed/unprefixed correctly.

TODO

[ ] Cypress tests
[ ] Update docs
[ ] KFMT
[ ] Revert the changes to nightly.yml

Known issues

[x] "Update" button - blocked by the backend and by Analytics needing sth like withTrackedEntityType for the Analytics request
[x] Selected dimension count in Program selector - blocked by LIBS-536 and needs a solution for how to count dimensions that doesn't have the program id as a prefix
[x] Tracked entity dimensions (e.g. "Person dimensions") - needs a backend endpoint, set up the request and implement the ui (similar to "Your dimensions" but with a different icon)
[x] Tracked entity dimensions getting wiped at various times, such as when changing TET id or going back and forth between panels. Rewrite the hooks to use proper dependency lists instead?
[x] Registration org unit and registration date - needs spec
[x] Filter TET dimensions by program usage - needs spec
[x] Dimensions returning NaN values DHIS2-16382
[x] Legend set by data item doesn't work, backend not returning pid.sid.did metadata (where the legend set is fetched). Backend issue DHIS2-16379
[x] Loading an AO with dimensions that use program or stage prefix doesn't work. Backend doesn't return the necessary metadata for the program/stage name. DHIS2-16437
[x] Saving an AO with both an ou dimension and a pid.ou dimension causes the items for each to get mixed up and be added to both dimensions. This is caused by the backend not keeping the items separated. DHIS2-16419
[x] Some TETs throws 409 errors DHIS2-16397
[x] "Not empty" condition doesn't work DHIS2-16459
[x] Program status and org unit table headers needs formatting DHIS2-16478
[x] Time dimensions not being prefixed with program id (@jenniferarnesen )
[x] Using multiple conditions doesn't return the correct response DHIS2-16510
[x] Sorting a numeric column doesn't return a sorted result + numeric dimensions can't use conditions DHIS2-16508
[x] Booleans display undefined instead of True and False + conditions can't be used DHIS2-16513
[x] Layout is empty / missing chips after loading an AO that includes time dimensions. Missing metadata from the backend for /eventVisualizations (Slack discussion / DHIS2-16437)
[x] Saving an AO with repetition, reopening and clicking update removes the repetition (Slack discussion)
[x] Multiple period items can't be used DHIS2-16708
[x] Empty values get sorted out when using "not contains" and "not equals" DHIS2-16459
[x] Multiple periods with items are filtered as OR instead of AND DHIS2-16726
[x] Repetition table headers missing suffix because of missing stageOffset DHIS2-16742
[ ] Org units can't be used without items DHIS2-16381 (uncomment code)
[ ] Different results for repeated events between enrollment and TE DHIS2-16709
[ ] Multiple option set options are filtered as AND instead of OR DHIS2-16743
[ ] "Your dimensions" throws an error when used in an analytics request DHIS2-16749
[ ] Program indicators are currently disabled in the backend (TBD)
[ ] When opening a legacy saved AO the repetition is missing DHIS2-16733 / (skipped test in repeatedEvents.cy.js)

Screenshots
New input type "Tracked entity"

New dimension menu item for tracked entity type dimensions

New global dimensions "Registration org. unit" and "Registration date"

Program dimensions panel with cross-program selection

New suffix logic for cross-program selection using program or stage depending on the type of duplicate

Please fix the lint warnings if possible. (ie., "React Hook useEffect has missing dependencies")


The content of the lazy static should require RefUnwindSafe

It is possible that a panic happens while a lazy static is being modified, and leave the lazy static in an invalid state.
The user could then catch the unwind and continue to use the object in its invalid state.
That's the reason why the UnwindSafe and RefUnwindSafe traits were invented.

Related comment: https://github.com/rust-lang/rust/issues/37136#issuecomment-254030747

Hi, I'm sincerely sorry for taking so long to respond.
Could you elaborate a bit on which kind of modification is the problem here? I'm not sure if the issue refers to the Once-based initial modification, or a content-dependent one involving other internal mutability.

It's about the content of the static having internal mutability.
Imagine you call a method on the static object. This method takes a &self but modifies the content through interior mutability. However unfortunately a panic occurs during the method call, and the internal state of the object becomes invalid.
Later another piece of code accesses the same static object and observes this invalid state.
However keep in mind that the situation about panic safety is a bit blurry in Rust right now. The Send and Sync traits more or less mean "panic safe" already.
I'm not sure if the traits should actually be enforced in addition to Send/Sync, but I thought I'd open an issue anyway.

Ah, I see.
Am I right in thinking that this issue can in theory happen with a plain old static as well?
If so then I would just mirror what is done in that case - and afaik RefUnwindSafe is not enforced for them.
I'll leave this open until the situation around panic safety becomes more clear. (Also, adding the bound would be a breaking change as well, so I'd rather not experiment needlessly :))

Am I right in thinking that this issue can in theory happen with a plain old static as well?

Well, I don't see how you could modify the content of a non-mut static.

A plain non-mut static can contain internal mutability types, like for example the atomics from the std lib.

I may not be up to date, but I thought that anything requiring a constructor (eg. a Mutex) couldn't be put in a static.
Atomic types do implement UnwindSafe fwiw.

It's still a unstable feature, but its allowed now. See here.
The main issue is more the initialization code needed might exceed what is possible within a constant or constexpr, but thats an issue about being able to initialize the thing, not about the unwind semantic if it where constructed in a static.

My understanding here is that we don't need to worry about the unwind safety during initialization because Once will poison on panic.
For the value itself, we inherit its unwind safety from the standard library:
impl<'a, T: RefUnwindSafe + ?Sized> UnwindSafe for &'a T {}
impl<T: RefUnwindSafe + ?Sized> UnwindSafe for *const T {}
impl<T: RefUnwindSafe + ?Sized> UnwindSafe for *mut T {}

If the value T inside the Lazy contains interior mutability then it won't be unwind safe unless an explicit implementation exists on T. Attempts to use unwind unsafe values in panic::catch_unwind will fail.
So I think the current state of affairs is ok?

@tomaka Hmm, you can't get in a situation where you use the value after you've caught and ignored the panic, because you can't catch and ignore the panic in the first place unless the value is unwind safe, right?

Oh, you're saying that you can't access the lazy_static in the first place if it's not UnwindSafe.
I admit that this issue is very old and I don't remember at all my reasoning here.


Update number of model in HF Hub

In 4. Sharing models and tokenizers - The Hugging Face Hub, it mentions that the HF Hub contains over 10,000 models. However, the Hub now hosts over 800,000 models, so this number might need to be updated. :smile:

Maybe it make sense to use online badge from official README.md, otherwise you need to update it from time to time.

@sergiopaniego

Makes sense! I love that solution.
I've updated it accordingly https://github.com/huggingface/course/pull/738/commits/ef02dbe4ecd30e8f5f88e17fa98e6bf3b85b8072.
I'm not sure but I think the content is not being actively updated. cc @lunarflu


[macos][docker][bk] allow macos agent to pass linux agent into docker

[Sorry if I'm raising this against the wrong repo]
Currently when I use the docker plugin I configure it to mount-buildkite-agent: true:
steps:
  - label: ":ballot_box_with_ballot:"
    command: ./src/generate-and-store-number.sh
    plugins:
      - docker#v3.5.0:
          image: "starkandwayne/buildkite-base:latest"
          propagate-environment: true
          mount-buildkite-agent: true

The buildkite-agent mounts my host machine's buildkite-agent binary into the docker container.
Alas, if my host machine's architecture is different from the linux container (macos -> linux for example), then the binary will fail:
+ buildkite-agent meta-data get generated-number
./src/fetch-and-display-number.sh: line 14: /usr/bin/buildkite-agent: cannot execute binary file: Exec format error

This is probably not a production issue -- I wouldn't run my agent on my macOS for production pipelines -- but it is an issue for two use cases I'm affected by:

local iteration of pipelines with bk local run
initial user experience of buildkite.com before they commit to running agents on linux-based cloud servers

Could it be possible for bk local run and buildkite-agent start inject a linux buildkite-agent into docker, rather than arbitrarily passing in the host machine's binary?

Hi @drnic! This is tricky one. It sounds like this might be a good feature request for the docker-buildkite-plugin to be aware of the host buildkite agent's platform/architecture as well as the target docker buildkite platform/architecture and do some fancy mapping when they don't match.

I'll close the issue here as it's not really a buildkite agent issue. :-)

@sj26 can I ask a favour, could you please reopen and move the isssue to the docker-plugin project?

Hmm, I can't seem to transfer the issue across github orgs, but I've opened an issue over there.


[Feature Request] Allow specifying post_file via relative/absolute path

It's really cool that you can specify a file to use with post requests from the command line! However, currently, one can only specify the -p, --post_file param using a bare filename (no path) which must exist in the same directory in which you are running bees (due to the scp command which assumes that the destination of the file on the bee file system is ~/%(post_file)s). It would be more ergonomic to be able to specify a relative or absolute path to a file, and have that file read and sent to the bee swarm.
I think the line which needs to be changed is the one which tells ab where to find the file: https://github.com/newsapps/beeswithmachineguns/blob/4b8783dc36d327ad95b711db39054d64bd718344/beeswithmachineguns/bees.py#L447

I started my own fork of this, don't think this is maintained anymore but I am maintaining this - https://github.com/radzhome/beeswithmachineguns2

I started my own fork of this, don't think this is maintained anymore but I am maintaining this - https://github.com/radzhome/beeswithmachineguns2

If you like this kind of open source work we are looking for people https://github.com/newsapps/beeswithmachineguns/issues/230


Update json4s-jackson to 3.6.9

Updates org.json4s:json4s-jackson from 3.6.8 to 3.6.9.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.json4s", artifactId = "json4s-jackson" } ]

labels: library-update, semver-patch

Superseded by #123.


Use 1ES Windows VM for CI

Current OOM issue prevents most of the PR's to succeed in CI. As a temporary measure, we need to switch to VMs that have more RAM.
Since https://github.com/Azure/azure-sdk-tools/issues/2497 is fixed, we can use Windows VM.

Both public and internal builds have passed.


Getting an Error: could not locate the bindings file

Tried to use pouchDB server but keep getting this error everytime I run the server and I can't connect to the database at all.
I'm not sure if this has something to do with me using NVM. This is running on a raspberry pi.
I also tried to install leveldown but still getting the same problem.
(node:30270) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 5): Error: Error: Could not locate the bindings file. Tried:
 → /home/pi/.nvm/versions/node/v7.5.0/lib/node_modules/pouchdb-server/node_modules/leveldown/build/leveldown.node
 → /home/pi/.nvm/versions/node/v7.5.0/lib/node_modules/pouchdb-server/node_modules/leveldown/build/Debug/leveldown.node
 → /home/pi/.nvm/versions/node/v7.5.0/lib/node_modules/pouchdb-server/node_modules/leveldown/build/Release/leveldown.node
 → /home/pi/.nvm/versions/node/v7.5.0/lib/node_modules/pouchdb-server/node_modules/leveldown/out/Debug/leveldown.node
 → /home/pi/.nvm/versions/node/v7.5.0/lib/node_modules/pouchdb-server/node_modules/leveldown/Debug/leveldown.node
 → /home/pi/.nvm/versions/node/v7.5.0/lib/node_modules/pouchdb-server/node_modules/leveldown/out/Release/leveldown.node
 → /home/pi/.nvm/versions/node/v7.5.0/lib/node_modules/pouchdb-server/node_modules/leveldown/Release/leveldown.node
 → /home/pi/.nvm/versions/node/v7.5.0/lib/node_modules/pouchdb-server/node_modules/leveldown/build/default/leveldown.node
 → /home/pi/.nvm/versions/node/v7.5.0/lib/node_modules/pouchdb-server/node_modules/leveldown/compiled/7.5.0/linux/arm/leveldown.node: unable to import leveldown

I think the problem is that leveldb was not compiled. Can you try to run pouchdb-server --in-memory and see if that works?

How do I properly compile leveldb?
The pouchdb-server seems to work while running it in memory.

I don’t know about that. Maybe see if you can find help with the leveldb community? At least they claim the support Raspberry Pi in their README: https://github.com/Level/levelup

Thanks for your quick response.
I have been able to get it to work by simply reinstalling node and deleting my node_modules folder then reinstalling pouchdb-server. It recompiled leveldb for me.

Great. I'm going to close this issue.


Add print_verbose() built-in function to print in verbose mode only (3.x)

3.x version of https://github.com/godotengine/godot/pull/52877. Unlike https://github.com/godotengine/godot/pull/52877, this also adds C# support for print_verbose().
This can be used as a shorthand for:
if OS.is_stdout_verbose():
    print("...")

Unlike print_debug(), this works in release builds too and can be toggled off in debug builds.
Preview
extends Node

func _ready():
	print("normal", " and ", "verbose")
	print_verbose("verbose", " only")

Verbose mode disabled
normal and verbose

Verbose mode enabled
Enable verbose stdout in the advanced project settings:

Alternatively, you can start Godot with the --verbose/-v command line argument.
normal and verbose
verbose only

I'm not a big fan of print_verbose because it always allocates. It can (and probably should) be replaced with a macro in C++ to avoid that, but for most language bindings it's unavoidable. C# and GDScript don't support lazy evaluation of arguments (neither does C++ tbh, but macro magic).


Multiple kinesis streams not working properly

I noticed that when subscribing to multiple kinesis streams e.g. rightArm and leftArm, one of them becomes receives any activity, while the other one is receiving more activity than normal, like if it was receiving activity for both streams at the same time. Nevertheless, this activity data is always marked as being a single label, so it is still imposible to determine where it is coming from.
Is this a known limitation of the system?
I also noticed all the neurosity system is implemented based on firebase DB actions. Is the firebase neurosity underlying architecture also open source? Can we learn somwhow how is that implemented to see if there is a way to overcome this issue?
Thanks in advance

Also, I noticed that this does not only happens using the SDK but also while listening to kinesis in the neurosity portal. So it must be a known limitation.

Hi @juank-pa,
Yes, this is a known limitation.
The Neurosity Console uses the SDK to train and read Kinesis data. We do use Firebase for cloud streaming. The SDK is open source as you can see, but unfortunately, our OS is not., We do recommend for folks to create their own models using the raw data from the SDK .
I hope this is helpful.
Best,
Alex


Chat with others creating similar tools

@schell , it would be great to chat with @kjetilkjeka about his company's internal work and see if we can combine forces. What is best? Our maintainer zulip? Here? Email?

@kjetilkjeka - I reached out via email :)


[Event Request] Codeunit 5815 "Undo Sales Shipment Line" - procedure CheckSalesShptLine - OnCheckSalesShptLineOnBeforeCollectItemLedgEntries

Please add an Event OnCheckSalesShptLineOnBeforeCollectItemLedgEntries in Procedure CheckSalesShptLine in Codeunit 5815 "Undo Sales Shipment Line"
local procedure CheckSalesShptLine(SalesShptLine: Record "Sales Shipment Line")
    var
        TempItemLedgEntry: Record "Item Ledger Entry" temporary;
        IsHandled: Boolean;
        SkipTestFields: Boolean;
        SkipUndoPosting: Boolean;
        SkipUndoInitPostATO: Boolean;
    begin
        IsHandled := false;
        OnBeforeCheckSalesShptLine(SalesShptLine, IsHandled, SkipTestFields, SkipUndoPosting, SkipUndoInitPostATO);
        if not IsHandled then
            with SalesShptLine do begin
                if not SkipTestFields then begin
                    if Correction then
                        Error(AlreadyReversedErr);
                    if "Qty. Shipped Not Invoiced" <> Quantity then
                        if HasInvoicedNotReturnedQuantity(SalesShptLine) then
                            Error(Text005);
                end;
                if Type = Type::Item then begin
                    if not SkipTestFields then
                        TestField("Drop Shipment", false);

                    if not SkipUndoPosting then begin
                        UndoPostingMgt.TestSalesShptLine(SalesShptLine);

                        IsHandled := false;
                        OnCheckSalesShptLineOnBeforeCollectItemLedgEntries(SalesShptLine, TempItemLedgEntry, IsHandled); <-- NEW EVENT
                        if not IsHandled then
                            UndoPostingMgt.CollectItemLedgEntries(
                                TempItemLedgEntry, DATABASE::"Sales Shipment Line", "Document No.", "Line No.", "Quantity (Base)", "Item Shpt. Entry No.");

                        UndoPostingMgt.CheckItemLedgEntries(TempItemLedgEntry, "Line No.", "Qty. Shipped Not Invoiced" <> Quantity);
                    end;
                    if not SkipUndoInitPostATO then
                        UndoInitPostATO(SalesShptLine);
                end;
            end;

        OnAfterCheckSalesShptLine(SalesShptLine, TempItemLedgEntry);
    end;

    [IntegrationEvent(false,false)]
    local procedure OnCheckSalesShptLineOnBeforeCollectItemLedgEntries(SalesShptLine: Record "Sales Shipment Line"; var TempItemLedgEntry: Record "Item Ledger Entry" temporary; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 43428.


Google Colab GPT-2 tweets no different than the input

I seem to load the csv file from using the python code, I am using 1000 tweets and the only results are just printing out tweets I gave it vs running putting out GANs tweets. Do I maybe need to use more tweets?

Yup, 1,000 tweets is on the low side. There is no hard and fast rule, but I’ve heard folks get decent results once they reach 1MB size, which is about 30,000 tweets.

Will this py file work by itself? Id love to run this in colab with a GPU for faster trainning

Yup, 1,000 tweets is on the low side. There is no hard and fast rule, but I’ve heard folks get decent results once they reach 1MB size, which is about 30,000 tweets.
@sdelgadoc
I Will try larger dataset, do you know if this py file work by itself? Id love to run this in colab with a GPU for faster trainning

Yes, this script could run in a Colab. However, I doubt you’ll see an increase in performance. The performance constraint is Twitter’s data transfer rate not processing power.  So, I doubt believe that adding a GPU will make it run faster.

Yes, this script could run in a Colab. However, I doubt you’ll see an increase in performance. The performance constraint is Twitter’s data transfer rate not processing power. So, I don’t believe that adding a GPU will make it run faster.

Wonderful thank you for the help


Add images to Existing Job

Hi is it possible to add new images to an existing job/task and if it isn't then why not?
Cheers,

A lot of duplicates of this issue, please, refer to the original #95


Add screen

Updated mapper, tests and fixtures to support screen calls @f2prateek

LGTm, squash the changes into one and I can merge.


Auto Update

I am constantly hounded for updates. That is great and I love that this is actively developed, but for all that is holy please just handle this quietly without my intervention.
Mac OS X

I've also noticed that the updates are not matched up with the git branch. I assume this is because someone compiles and uploads to the nylas website; however, a CI system should be set up so updates are pushed through as soon as they're ready (especially since some of us, like myself, are using this as our main client).
Additionally, an Atom style update system should be implemented. Where the update automatically downloads and installs, and just displays a prominent icon that lets the user know that it will be applied upon program restart.

CI is not possible (at least not a public one) right now because they don’t build from GitHub, some part not being open-sourced right now (see #3270). But indeed it should be a thing later on.


Nonstackables Unstack in Shulker Boxes (overwrites items)

Nonstackables will overwrite themselves or other items in a Shulker Box.
To replicate:
Configure normally non-stackable items to stack
Place a stack of those inside of a Shulker Box (more than 27 or a combination of items totaling more than 27 slots in vanilla)
Pick up and then place the Shulker Box
Open the Shulker Box
The nonstackables will be unstacked and some additional items will be overwritten.
This does not happen in creative, only survival.

Thanks for the bug report! I'm unable to reproduce this bug in my server but I am aware of other times that this occurred that have been patched. What version of Simple Stack are you running on? What Minecraft version are you running on? Are there any errors being thrown to the console? Thanks!

This may have been a server specific issue, I apologize for jumping to report it.
The server runs Minecraft 1.16.5 and Spigot. Yesterday we removed all plugins then reuploaded them, this seemed to have somehow fixed the issue (we used the exact same files we had removed). Kind of confused but extremely happy it is now working.
While the issue was happening, the console never reported any errors or warnings.
Thank you for your swift response and keep up the good work!!


TypeMerge does not work, I don't know exactly why

Here is a repo where it can be tested: https://github.com/xharkonen/merge
this is completely ignored while the other merge works fine, maybe related to interfaces being used.
          TransactionDecorator: {
            selectionSet: '{ id }',
            fieldName: '_decorators',
            key: ({ id }) => {
              console.log('transactionDecorator id', id);
              return id;
            },
            argsFromKeys: (ids) => {
              console.log('transactionDecorator', ids)
              return { ids }
            },
          },

_decorators resolver is never being called.
Any idea what's wrong? It is a bug?

wrong place


Can not capture the child class of "Exception" in app.exception_handler globally

Hi All,
I just start to learn FastAPI. I am trying to capture the StarletteHTTPException globally by a following simple sample.

import uvicorn

from fastapi import FastAPI
from starlette.requests import Request
from starlette.exceptions import HTTPException as StarletteHTTPException
from starlette.responses import JSONResponse

app = FastAPI()

@app.exception_handler(StarletteHTTPException)
async def exception_callback(request: Request, exc: Exception):
    print("test")
    return JSONResponse({"detail": "test_error"}, status_code=500)

if __name__ == "__main__":
    uvicorn.run("test:app", host="<IP_ADDRESS>", port=1111, reload=True)

It works. I open the browser and call the endpoint / and try to access http://<IP_ADDRESS>:1111/ , it will return the json {"detail":"test_error"}  with HTTP code "500 Internal Server Error" .

However, when I only changed StarletteHTTPException to Exception in the @app.exception_handler,

import uvicorn

from fastapi import FastAPI
from starlette.requests import Request
from starlette.exceptions import HTTPException as StarletteHTTPException
from starlette.responses import JSONResponse

app = FastAPI()

@app.exception_handler(Exception)
async def exception_callback(request: Request, exc: Exception):
    print("test")
    return JSONResponse({"detail": "test_error"}, status_code=500)

if __name__ == "__main__":
    uvicorn.run("test:app", host="<IP_ADDRESS>", port=1111, reload=True)

The method exception_callback could not capture the StarletteHTTPException when I accessed the http://<IP_ADDRESS>:1111/  . It reported 404 error.

But it seems that StarletteHTTPException is the child class of Exception. I think the excepted behaviour is: StarletteHTTPException error could be captured by the method exception_handler decorated by Exception.  Pls review.
Thank you !
Environment

OS: Windows
FastAPI Version 0.63.0:

This is somehow related to: https://github.com/tiangolo/fastapi/issues/2683
Anyway, this question should be on Starlette side. We just use their code here.

Thank you for your replying @Kludex
It seems that similiar issue was raised at https://stackoverflow.com/questions/61596911/catch-exception-in-fast-api-globally serveral months ago.
Could we have some workaround for this issue currently ? Through the middleware or other method ? Thank you

Could we have some workaround for this issue (capature all the exceptions globally) currently ? Through the middleware or the other method ? Thank you

Sure @zhfkt , you can create your own router, then handle the errors inside it.
from fastapi import APIRouter, FastAPI, Request, Response
from fastapi.routing import APIRoute

from typing import Callable
from fastapi.responses import JSONResponse

class ExceptionRoute(APIRoute):
    def get_route_handler(self) -> Callable:
        original_route_handler = super().get_route_handler()

        async def custom_route_handler(request: Request) -> Response:
            try:
                response: Response = await original_route_handler(request)
            except Exception as e:
                if isinstance(e, ValueError):
                    return JSONResponse(
                        content={"detail": "value_error!"}, status_code=500
                    )
                if isinstance(e, KeyError):
                    return JSONResponse(
                        content={"detail": "key_error!"}, status_code=500
                    )
            return response

        return custom_route_handler

app = FastAPI()
router = APIRouter(route_class=ExceptionRoute)

@router.get("/raises_value_error")
async def test():
    raise ValueError("test")

@router.get("/raises_key_error")
async def test():
    raise KeyError("test")

app.include_router(router)

Now when you hit these endpoints, they will be captured by our little exception handlers.
INFO:     <IP_ADDRESS>:47988 - "GET /raises_value_error HTTP/1.1" 500 Internal Server Error
INFO:     <IP_ADDRESS>:47988 - "GET /raises_key_error HTTP/1.1" 500 Internal Server Error

@ycd This is an interesting solution. But it also throws for 422 errors and turns them into 500.  Particularly problematic in unit tests.  My goal is to send myself an email when a generic exception happens.  I wished there was a non-blocking SMTPHandler we could add to app.logger.addHandler(mail_handler) like we used to in Flask. I wonder if that's possible in Fastapi.

It is not precisely an answer to the original question, but this case maybe can help someone:
I want to capture all the exceptions that do not have been captured by upper handlers and allow all the middleware to continue operating normally (when there is a 500 error, this is handled through ServerErrorMiddleware that again launches an exception, ruining the entire flow towards up)
from fastapi import FastAPI, Request

from fastapi.responses import JSONResponse
from starlette.middleware.exceptions import ExceptionMiddleware

app = FastAPI()

@app.exception_handler(500)
@app.exception_handler(Exception)
async def generic_exception_handler(request: Request, exc: Exception):
    print("test")
    return JSONResponse({"detail": "test_error"}, status_code=500)

# Hack ExceptionMiddleware to handle (500 or Exception) errors
generic_exception_handlers = {k: v for k, v in app.exception_handlers.items() if k == 500 or k == Exception}
if generic_exception_handlers:
    _app = app.middleware_stack
    while True:
        if isinstance(_app, ExceptionMiddleware):
            _app._exception_handlers.update(generic_exception_handlers)  # noqa
            break
        elif hasattr(_app, 'app'):
            _app = _app.app
        else:
            break

@app.get("/raises_value_error")
async def test():
    raise ValueError("test")

@app.get("/raises_key_error")
async def test():
    raise KeyError("test")

Thanks everyone for the conversation and thanks @Kludex for the note! 🚀
If your problem is solved @zhfkt , you can close the issue. ☕


chore(main): release 2.5.0

:robot: I have created a release beep boop
2.5.0 (2024-11-12)
Features

new publish script (0e77559)

Bug Fixes

added .npmrc for auth (4d2f6a8)
create .npmrc to configure npm auth (daec2ed)
fixed CI env (1330b27)
pkg files config (7eb2700)
switched to pkg json script for publish (3c77b25)
used env vars (a263d16)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/nico-i/eslint-config/releases/tag/v2.5.0 :sunflower:


To decide about the (new and improved) experimental section

After discussing with @unintendedbear and looking to the current results we have thought a new experimental section that could improve the paper a lot and ease future steps.
First, we think that the fitness we are using is complicated to understand and requires a lot of computational time. Besides we do not find any reference to justify its use. So, we are going to the go back to the classical 10-cross validation, as it has been used widely: Divide the dataset in A and B (10 times) and use A for the training (whole evolution) and then B at the end of the evolution to validate (not DURING the evolution as we are doing now). So, we need to decide the fitness using some of the classical error metrics (see this wikipedia link to decide which error to use as fitness ), or the coverage (as we are doing now), or the fitness used by [1] (positives, negatives and depth).
Also, it would be a great improvement to the paper to compare with other method in the literature. Guys in [1] deal with each individual as a rule, and the model is the whole population. We could do some experiments with that structure and decide which method is better.
So, the steps would be:

Chose the most appropriate metric for fitness and validation
Execute with classic 10-cross validation and the chosen error metric as fitness and validation. We will have 10 final validation metrics, and therefore the average of our method.
Execute with every individual as a rule, instead as a tree (I will modify the code for this), and compare with the previous method.

What @zeinebchelly and @JJ think?
[1] Ivan De Falco, Antonio Della Cioppa, Ernesto Tarantino: Discovering interesting classification rules with genetic programming. Appl. Soft Comput. 1(4): 257-269 (2002)

I think it makes more sense to use the 10-cross validation instead;
specifically if we think in terms of complexity.
Concerning the metric, maybe we can focus on FPR? What do you think?
And I totally agree with adding a section showing the performance of the
developed method in comparison to other state-of-the-art methods.
2016-11-23 14:24 GMT+01:00 Pablo García Sánchez<EMAIL_ADDRESS>
After discussing with @unintendedbear https://github.com/unintendedbear
and looking to the current results we have thought a new experimental
section that could improve the paper a lot and ease future steps.
First, we think that the fitness we are using is complicated to understand
and requires a lot of computational time. Besides we do not find any
reference to justify its use. So, we are going to the go back to the
classical 10-cross validation, as it has been used widely: Divide the
dataset in A and B (10 times) and use A for the training (whole evolution)
and then B at the end of the evolution to validate (not DURING the
evolution as we are doing now). So, we need to decide the fitness using
some of the classical error metrics (see this wikipedia link to decide
which error to use as fitness
https://en.wikipedia.org/wiki/Positive_and_negative_predictive_values
), or the coverage (as we are doing now), or the fitness used by [1]
(positives, negatives and depth).
Also, it would be a great improvement to the paper to compare with other
method in the literature. Guys in [1] deal with each individual as a rule,
and the model is the whole population. We could do some experiments with
that structure and decide which method is better.
So, the steps would be:

Chose the most appropriate metric for fitness and validation
Execute with classic 10-cross validation and the chosen error
metric as fitness and validation. We will have 10 final validation metrics,
and therefore the average of our method.
Execute with every individual as a rule, instead as a tree (I will
modify the code for this), and compare with the previous method.

What @zeinebchelly https://github.com/zeinebchelly and @JJ
https://github.com/JJ think?
[1] Ivan De Falco, Antonio Della Cioppa, Ernesto Tarantino: Discovering
interesting classification rules with genetic programming. Appl. Soft
Comput. 1(4): 257-269 (2002)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/geneura-papers/2016-PPSN/issues/21, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AJgODv-hEi4vR--qXr_EUirgxMcfSaXBks5rBD6RgaJpZM4K6koM
.


Add example for device_map

As per title.
@IlyasMoutawwakil Would this be catched in the GPU CI?

Closing in favor of an other PR


remove suivm branch. Code was merged into latest

Description
As titled, really not a lot to add, except that some people may rejoice at the news...
Test Plan
No test

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

hey @lxfind calm down!!! :)


New version: ente-io.ente.CLI version 0.1.16

Pull request has been created with komac v2.2.1 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:
(Automated response - build 886.)


Suggestion: Link the comment icon to the comment section of a post

Scroll to comment of a post when clicked

@horpeyloaded It is already fixed, can you check again?


migrate nhooyr.io/websocket to github.com/coder/websocket

nhooyr.io/websocket is deprecated and stopped working (my CI builds failed)
the new repo is github.com/coder/websocket
related info:
https://pkg.go.dev/nhooyr.io/websocket
https://coder.com/blog/websocket
also we need a version bump to fix the examples directory.

Hi @cpouldev
Thank you for contributing. Can you format the code to pass the test? Thanks
gofmt -w -s .


IP Information boolean to display LAN and WAN

Added to NetworkManagerHUD
True displays your LAN and WAN IP address for debugging purposes in console log, useful if you are host/server.
Should especially help with new Mirror users, and for debugging connection problems with them.
(many struggle to know how to get the IP's or the differences between them)
Tooltip and Summary included, boolean is false by default, and this feature is completely optional.
Feel free to offer any code improvements and grammar changes.

Related to this thread:
https://github.com/vis2k/Mirror/issues/2404
Edit: A few hours after posting this, and Punfish commenting that its not needed ( cheeky bugger :D  ), we had another trying to connect PC and Android device via localhost
https://discord.com/channels/343440455738064897/467961162558865408/781547322327826443
Honestly its a frequent thing, people not knowing the difference, the above HUD mod should help us say, okay now click start host/server on one device, that LAN/WAN IP it says in debug log, is what you type in the other devices client box, job done  :D

What about showing this in the actual runtime GUI instead?
might be useful in builds too?
also isn't there an easier way to get both addresses? seems like a lot of magic :)

What about showing this in the actual runtime GUI instead?
might be useful in builds too?
also isn't there an easier way to get both addresses? seems like a lot of magic :)
why do we need to show WAN?

It could potentially be useful in the server's Debug GUI to show the WAN IP.. I think it already shows the LAN?
Did a quick google and I could find this;
           string hostName = Dns.GetHostName(); // Retrive the Name of HOST            Console.WriteLine(hostName);           // Get the IP            string myIP = Dns.GetHostByName(hostName).AddressList[0].ToString();            Console.WriteLine("My IP Address is :"+myIP);            Console.ReadKey();

Haven't tested nor confirmed but I believe this returns the LAN IP. in 5 lines that should be easier. WAN however is a tad more of a pest to find
3. WAN is useful because not everybody understands the difference, and honestly it is a bit of a hassle to find it personally, having to go to a IP check website or whatever. This would be a quick & Easy display for both LAN and WAN and automatically makes users know there is a difference.

How about this, a dropdown that has localhost, lan and wan.
Whichever is selected fills in the gui hud in scene, and the network address box.

@MrGadget1024  @vis2k

Live GUI drop down sets the input field and network manager address.
(you can still set a default in inspector before scene start)
WAN variable added so users don't keep requesting it.
Extra null checks added.
Thanks to Dragonslaya for the gui, and James for the further adaption ideas.

Updated with latest.
Still need to do platform test on iOS and Android, so don't approve just yet.

Summarised:
iOS needed a separate LAN method  (added and tested).
Windows, Mac and Android all work and tested.
(Although i needed to adjust previous method to get "FirstOrDefault" LAN rather than AddressList[0], as that sometimes gave me an ipv6 on windows for LAN)
Ready for someone to review and hopefully merge  :O

Majority of changes/recommendations done, re-submitted.


API key link is broken

Hey there;
While trying to access the "The API project" link listed under the "Usage" section, I've received a 404 error message.
Can you please verify?
Thank you.
Stella Lunardeli.

@arctouch-stellalunardeli I suspect the correct link is https://github.com/Q42/accessibility-data-pipeline/tree/main/api ... @huijkman can you confirm?

https://github.com/Q42/accessibility-data-pipeline/tree/main/api is indeed the correct link. Feel free to shoot me a message if you have any questions regarding the data pipeline and API.

It took some time till I had the appropriate rights for this repo to fix this in the reamde. https://github.com/Q42/Q42Stats.Android/commit/d655e6c76068d3a24eae1a19dddd0dd8cf237d6d


ARTEMIS-1876 InVMNodeManager shouldn't be used if no JDBC HA is configured

When database persistence and no shared store option is being used,
Artemis is choosing to use InVMNodeManager, that is not providing
the same behaviour of FileLockNodeManager.

@andytaylor It is causing several regressions on Oracle tests, so it needs to be ported on 1.x as well.
ATM I'm waiting the tests to finish and I will push it for 1.x as well

@andytaylor The tests seems fine to me, so it is ok :+1:


Op 7652 no groups found v3.10

We prefer small, well tested pull requests.
Please refer to Contributing to Spinnaker.
When filling out a pull request, please consider the following:

Follow the commit message conventions found here.
Provide a descriptive summary for your changes.
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Add inline code comments to changes that might not be obvious.
Squash your commits as you keep adding changes.
Add a comment to @spinnaker/reviewers for review if your issue has been outstanding for more than 3 days.

Note that we are unlikely to accept pull requests that add features without prior discussion. The best way to propose a feature is to open an issue first and discuss your ideas there before implementing them.
https://devopsmx.atlassian.net/browse/OP-7652

This PR  triggered this build in this execution, please merge to master only if this succeeds:  https://spin.prod.opsmx.com/#/applications/gcp-gitprtrigger/executions/details/01FYTYYREN19EVTSDS4G4J5KT6

This PR  triggered this build in this execution, please merge to v3.10 branch only if this succeeds:  https://spin.prod.opsmx.com/#/applications/isd310/executions/details/01FYTYZ02M273SJBTQS78WQGM5


clientSecret is no longer needed (and cannot be obtained)

In the Google API examples the user is asked to enter both the clientID and the clientSecret. But there is no longer need for the client secret, and you cannot get such a thing form the API console.
The fix should simply ignore this field and pass 0 as clientSecret.

Wiki and working examples update.  Thanks for the report.


Make the building UI more obvious

During almost every user test we've conducted with new users, people are confused about what happens after they hit save. They tend to think they broke the project or did something wrong, then go back to look for an error message. After we've pointed out the building.... message (which they overlook), it becomes more clear.
To do:

[ ] Make build UI more obvious on the page (using the same pattern, possibly, as the  Project Saved UI that displays on the bottom right of the screen)
[x] Add some concept of active building, either through a spinner or loading gif. This has been requested by multiple users who see the static Building... word as a frozen action (can't tell something is actively happening).

Is there more we want to do for this?


Server support?

On https://modrinth.com/mod/basicskyblock, it says the mod is compatible for both client and server. However, I haven't been able to make it work on a server.
Is this possible? If not, will there be server support in the future?

Just saw the creater of the basic skyblock mod is a different developer.


Develop

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 7 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 78.958%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
788

Relevant Lines:
998

💛  - Coveralls


fix(angular.copy): should be able to angular.copy Float32Array

angular.copy can now copy a Float32Array when only provided src. I added some tests to confirm this.
fixes #10210

@googlebot done.

+1 @caitp
I was just looking at this too and this is just what I was thinking.
If we are going to support copying Float32Array then we should support all type arrays. Unfortunately IE9 doesn't support them so we need to tread carefully but @caitp's ideas are the direction to go.
@DropsOfSerenity - if you would like to update your PR accordingly that would be great.

[ ] Update the error doc to be more generic
[ ] Create a isTypedArray() method instead of specific one. Perhaps we could compare src.constuctor.name with /(Uint8(Clamped)?)|(Uint16)|(Uint32)|(Int8)|(Int16)|(Int32)|(Float(32)|(64))Array/`
Update the copy method accordingly
Update the docs

Excellent, I'll get to it, on a side-note I don't know why the CI build keeps failing, all the tests pass over here.

Don't worry. We are having flakiness issues connecting to SauceLabs and BrowserStack when running tests on Travis

@caitp @petebacondarwin
Alright thanks for the great advice, I've added a new commit that supports all TypedArrays. I did try using constructor.name instead however that resulted in a memory leak for reasons I don't understand.
One other approach I was thinking of could be to use instanceOf, but then I'd have to add a check to make sure the Type being compared to actually exists (in the case of ie9- it doesn't)
Either way this solution seems to be working well, updated the tests to check all TypedArrays.

@caitp sweet looks like CI finally passed :)

party =) I'll take a look at this later today, gotta finish up a quick patch I'm writing first

I lied, I guess I did review the PR before finishing my patch :3 It looks good to me, just fix up the nits and squash the commits

@caitp Ok i squashed with my changes. :3

good stuff, looks ready to merge to me

i guess this was really a feature rather than a fix >_- oh well, not changing that message :>

This change is great... have you considered exposeding istypedArray as angular.isTypedArray?

glad you like it @Hypercubed! but... exposing these utilities is problematic in many ways =\ i don't think it would be smart... Besides, @@toStringTag makes it unreliable anyhow


accuracy in the experimental results

Why are the results of my 5 experiments on cora dataset showing higher accuracy than the results shown in the paper  by using the author's code. Who can tell me why?

Could you let me know if you found the reason? Also, do you know how we can make it constant?


Nested Routes and Stores

First I wanted to say thank you for the great library.
I have an application that has Store A and Store B under respective routes A and B.  The fetch of B is dependent on the id's returned by the fetch of A.  Is there any way, even a hackish way I could get server side rendering to generate the markup for this case.
I've tried a few things and it looks like it does call for the data when marty calls 'React.renderToString' but it doesn't wait for the fetch to return.
 <Route name="A" path="/a"/>
    <Route name="B" path="b" handler={ require('./BComponent') } >

    <DefaultRoute handler={ require('./AComponent') } />
</Route>

export default Marty.createContainer(BComponent, {
  listenTo: BStore,
  fetch: {
    b() {
      return BStore.for(this).getBs(this.props.a.ids);
    }
  }
});

Hi.
Have you tried to call the waitFor method?
I'm not sure it will work but it's worth a try. :smile:

Sorry for delay in replying to this. dependsOn might be what you want
class SomeStore extends Marty.Store {
  getFoo(barId, fooId) {
    return this.fetch({
      id: barId,
      dependsOn: this.getBar(barId),
      locally() {
        return this.bars[barId].foos[fooId]
      }
    })
  },
  getBar(barId) {
    return this.fetch({
      id: barId,
      locally() {
        return this.bars[barId];
      },
      remotely() {
        return BarAPI.getBar(barId)
      }
    })
  }
}

I'm gonna close this now, please re-open if this doesn't solve your problem

We tried both waitfor and dependsOn, what ends up happening on the server is it returns the request but you can see the server still processing the nested request, it makes the necessary async calls but the response/html has already been returned by marty.

@thecaddy Try 0.9.15 version. It could be the same issue as #314 (nested fetches).

Ah ok great.  We'll take a look.  We coded a workaround for now but we'll dedicate some time to take a look.


Bug: a missing english translation for Copy source file preview link message

If I click "Copy source file preview link" on an image, it returns a message in Chinese as below.
"已复制内容 "/infinite_image_browsing/file?filename=%2Fhome%2Fredacted%2Fstable-diffusion-webui%2Foutputs%2Fsave%2Fredacted.png" 到粘贴板"

It's probably an untranslated function, don't worry about it. The message above is just to let you know that the copy has been completed.

Already fixed.
. If you have any further problems or requests, please create a new issue.


README should clearly state that this is an application boilerplate, and not the underlying framework library

The issue don't have a lot of information. I try to understand from the title. I made a PR https://github.com/angel-dart/angel/pull/120
If it is wrong, please let me know, i fix that.


Compositor: Warning removal: unused variable

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/ThunderNanoServices/556/rdkcentral/ThunderNanoServices

Commit: 27c65e243d14a142d5a6387ce0c5896c93dc5b5a

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/ThunderNanoServices/556/rdkcentral/ThunderNanoServices

Commit: 27c65e243d14a142d5a6387ce0c5896c93dc5b5a

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: Re-identify search which was lost in the tool.

Commit: 27c65e243d14a142d5a6387ce0c5896c93dc5b5a


[Snyk] Upgrade @salesforce/apex-node from 0.11.0 to 0.12.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @salesforce/apex-node from 0.11.0 to 0.12.0.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 21 days ago, on 2022-05-18.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report

Merging #1027 (36ee870) into develop (3961072) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1027   +/-   ##
========================================
  Coverage    58.37%   58.37%
========================================
  Files           51       51
  Lines         1576     1576
  Branches       332      332
========================================
  Hits           920      920
  Misses         613      613
  Partials        43       43

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 3961072...36ee870. Read the comment docs.


Update UG

Changed some wording to be clearer and/or more direct, and fix some behaviour mismatches.
Group similar commands together.
Update find and findLocation to support substring search

Codecov Report

Merging #166 (21aa79d) into master (c6ab4cb) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 21aa79d differs from pull request most recent head 4963406. Consider uploading reports for the commit 4963406 to get more accurate results

@@            Coverage Diff            @@
##             master     #166   +/-   ##
=========================================
  Coverage     56.86%   56.86%
  Complexity      463      463
=========================================
  Files            92       92
  Lines          1762     1762
  Branches        213      213
=========================================
  Hits           1002     1002
  Misses          712      712
  Partials         48       48

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add QR code option

Should be done now @mickhansen

double scroll on smaller desktop screens is a bit odd, but not a major issue.


[Mx4PC] - new release for the Data Transfer tool, planned for release on November 17

We've prepared a new release of the Private Cloud data transfer tool and are planning to release it tomorrow, on November 17. It's not time-sensitive and ahead of schedule, so it's OK to review and publish it at any convenient time, next week is OK as well :)
This MR includes release notes and updated documentation (how to use IRSA authentication).

Reviewed and approved. Merging by request from the team.