Control events persistence from the domain model

Right now, the event store is agnostic to what events are being persisted. It is quite a limitation for doing things like #64, domain-driven snapshots, closing periods, etc, as some "fold and archive" or "fold and snapshot" actions need to be controlled by the domain model.
For example, a reservation might get archived after the guest checks out, as the lifecycle of the reservation is done, and it becomes fully idempotent. Similarly, closing a period would collect the period data in a "new period started" event, and then signal the persistence to truncate all the previous events.
So, the idea is to provide a way for the domain model to tell those things to the aggregate store and further to the event store.
I have two ideas atm:

Apply can get an overload or optional parameter to signal the archiving or truncation
A set of attributes for event classes to do the same

Both can be implemented at the same time, but the implementation would be in different layers.

For Apply the change would be to extend the Changes to contain extra information
For attributes, all the work will be done by the aggregate store. The StreamEvent record would need to get new properties.

My 2 cents: I personally like the attribute better, locking in on certain type of events, rather than any event. Although the motivation for archiving is mostly a technical, the implications of it should involve explicit modelling. An explicit event like 'booking archived' would make it simple to handle in downstream consumers, rather than requiring downstream consumers to look for special metadata on any event (theoretically).

Yeah, that's true. I don't agree, though, that it's technical. I'd say that archiving for cheaper storage would work with TTLs. The explicit archive is more business-oriented, like in my example for bookings in the issue description.

Good point - I was in the realm of "if technology was limitless and for free" (where the business would be happy to simply hide archived data behind an 'IsArchived' flag)... Which, to be clear, is an extremely naive approach to IT for any business 🙈

I hear more people advocating the idea of explicit serialisation when the domain event can use value objects (which is a problem right now, including conversions back and forth in Apply/When), and it gets manually converted to and from the persisted event. Similar to #153, but two ways (commands are one way).
If that's done, the convertor can return null (or some pre-defined Ignore value) if the domain event doesn't require persistence.

The proposal in #168 won't solve the issue of snapshotting and archive controlled by the domain model. However, it might enable it. I will do it first, then look for ways to extend it.


Parse text from markdown

Can you please add this line to the header of the 2 modified files:
# Copyright (c) 2023 Fang Xu, Deutsche Telekom AG

Please also modify the README Licensing paragraph and add yourself.

Command not found: black.
need to install black formatter.

@xufangt Some Ruff checks fail. This can often be fixed by running make format. If you want to check formatting localy just run make check.
Already fixed.

need to install black formatter.

Yes. This can be done by make install or poetry lock && poetry install --all-extras

Because of license issues (in conjunction with Stackoverflow) I can not merge this. I am sorry @xufangt .

Because of license issues (in conjunction with Stackoverflow) I can not merge this. I am sorry @xufangt .


New Component (Multi-action button) or Dropdown

Info
We need to duplicate the functionality in this Figma page.
Basically we need a button type for a Dropdown.
Options include modifying the Dropdown component to work with this variant, or creating a new component.
Acceptance Criteria:

Investigate and implement the better of the 2 options listed above

Work needed to be done is captured in issue #1835


openapi.Parameter location is ignored when defined from the openapi.definition decorator

Describe the bug
When a parameter has a defined location such as "path", "header" or "body" and is defined using the openapi.definition decorator, the location seems to be ignored and is shown as "query".
Screenshots

To Reproduce
Use the decorator like so:
@openapi.definition(
    parameter=[
        openapi.Parameter("enterprise", str, "body"),
    ],
)

Expected behavior
The parameter is marked as whatever type it's set to, and isn't overriden by the fallback value.
Environment (please complete the following information):

Version 21.9.1

body seems not in the possible values of the in field in openapi's docs:
https://spec.openapis.org/oas/latest.html#fixed-fields-9
For the request body, https://spec.openapis.org/oas/latest.html#request-body-examples

Not a bug as @ChihweiLHBird stated. You need to use the body kwarg.

This isn't just for body. It happens with path and header too, which are possible values for the in field in OpenApi's docs. I've updated the original issue to reflect it with valid parameters.
This doesn't work:
@openapi.definition(
    parameter=[
        openapi.Parameter("enterprise", str, location="path"),
    ],
)

This does:
@openapi.parameter("enterprise", str, location="path")

This isn't just for body. It happens with path and header too, which are possible values for the in field in OpenApi's docs. I've updated the original issue to reflect it with valid parameters.
This doesn't work:
@openapi.definition(
    parameter=[
        openapi.Parameter("enterprise", str, location="path"),
    ],
)

This does:
@openapi.parameter("enterprise", str, location="path")

Yes, you are right. A pull request was created for this issue. Thank you!

Haha, thank u! :)

Should has been fixed in https://github.com/sanic-org/sanic-ext/pull/27, closing the issue.
Thank you, @prryplatypus


Tower Land Area Calculation Depending on Work Flow

Describe the bug
Molten salt tower model provides different land area calculated values depending on the if the user optimizes the heliostat layout and tower dimensions using either the "Calculate" macro or checking the box to do the optimization on the "Simulate" call.
To Reproduce
Macro Button Optimization Call:

Open the "Tower (Salt), Single owner" model
Change the solar multiple to 2.3 (default 2.4) on the "System Design" page
Click "Calculate" macro button next to "Optimize heliostat layout and tower dimensions" on the "Heliostat Field" page
Click "Yes" to update optimized values
Click "Simulate"

Checkbox Optimization Method:

Create a new model instance using steps 1 and 2 above
Click the checkbox next to "Optimize heliostat layout and tower dimensions" on the "Heliostat Field" page
Click "Simulate"

Expected behavior
These two work flows should provide the same result for land area.
Screenshots
Macro optimized values:

Macro Button Optimization Call Results:

Checkbox Optimization Method:

Operating System and Version

OS: Windows
Version: SAM 2021.12.2

This issue was fixed during major overhaul refactoring of the MSPT cmod: https://github.com/NREL/ssc/commit/3b596633ed7948506f172ce0ea236fa77287b658


[iOS] [Fixed] [CocoaPods | Project] - Support building the project on arm64 macOS

Summary
In scripts/react_native_pods.rb it excludes i386 architecture when Hermes is enabled.
However, when you look at the OS versions RN supports, i366 arch is not needed from the beginning because there is no 32bit iOS devices RN supports.
Also RN does not yet support arm64 simulator at this moment. When we make a new project and try to run it on a simulator on arm macOS, it fails for now. Instead of conditionally excluding arm64, it should always exclude arm64 architecture to solve it. Conditionally changing the build config also causes another issue as described below:

Run pod install on Intel macOS
git-push this change
Run pod install on arm64 macOS

In step 3, it shows diff in the iOS project file.
This PR aims to solve both issues.
Changelog
[iOS] [Fixed] - Support building the project on arm64 macOS
Test Plan
Build the project on arm64 macOS.

Why do you think that RN doesn’t support arm64 simulators? As for me, it runs without errors. What errors do you get? Do you have Hermes and Flipper enabled?
Excluding arm64 is rather doubtful solution. It was the only fix before RN 0.64 release to run RN apps on M1 Macs in iOS Simulator, but now it’s not needed anymore.

Hi @Arkkeeper, thank your for your comment. Interesting to hear that you do not have any issue on M1 macOS.
I see build error when I create new RN project using npx react-native init.. and yes Flipper is installed too.
Also related discussion here: https://github.com/react-native-community/discussions-and-proposals/issues/295 but I see others are having trouble with building on M1.

Ok, now I figured out npx react-native run-ios works if the node installed to macOS is x86_64 version.
If you try to build it by yourself (from Xcode) it fail still.
The reason is because npx react-native run-ios uses /usr/bin/env node. So it builds the iOS app using Rosetta if node is x86_64.

I have arm64 node and I usually build iOS and Mac Catalyst versions of our apps directly from Xcode.
Let's clarify the error that you get trying to build from Xcode, could you please provide any details? Maybe it's something with swift libraries?

@Arkkeeper Interesting. Yes I am seeing exactly the same one posted here.
https://github.com/react-native-community/discussions-and-proposals/issues/295#issuecomment-786956611

Ok, I could now build the app on arm64 native.
Things I needed to change were:

Get rid of LIBRARY_SEARCH_PATHS option
Add a empty Swift file into the app target

A better solution is https://github.com/facebook/react-native/issues/31208#issuecomment-803825310

@Arkkeeper Thanks for your comment. Unfortunately it did not solve the issue on mine..

@chuganzy
try solution described https://github.com/react-native-community/discussions-and-proposals/issues/295#issuecomment-809636024

Thank you @lubomyr, and yes I have tried it here and worked https://github.com/facebook/react-native/pull/31229#issuecomment-808499111
Still there are couple of issues in the current project setup:

It anyway excludes arm64 simulator on Intel macOS when you run pod install: https://github.com/facebook/react-native/pull/31229/files#diff-adcf572f001c2b710d14f409c14763f1a50b08369b3034548f1602685d21f67fL132
Due to 1, it always make diff in the Xcode project file when you run pod install on different arch macOS machines (Intel vs Apple)
If the solution here is the way RN wants to go, then we can make changes in LIBRARY_SEARCH_PATHS and add empty Swift file, but we have to 1 and 2 needs to be solved too.

To move this forward I want to clarify:

If 3. is the way RN wants to go
Why we need LIBRARY_SEARCH_PATHS configured in the first place (I will git blame later and put any findings here if I could!)
Why we need to exclude arm64 on Intel macOS (same as 2. I will git blame.)

Closing as Hermes now supports arm64!


Prevent builds/publish on non-source changes

Right now, regardless of the changes, a new build and deployment will occur every time.
This should be prevented for changes to, for example, documentation.

Closed in #8


Create Terminal does not activate pyenv environments

@brettcannon cc
Pyenv environments need to be activated like conda environments.
e.g.:
pyenv shell [name]
This was never done, is this required for the April release.

They don't need to be activated that way, it would just be nice and make @qubitron happy that he would get his global-install-with-Create Terminal scenario working. 😉

Per @brettcannon does this mean my global-install-with-Create Terminal is now working? 🙏 Not expecting it to but you got my hopes up above!

@qubitron only with pyenv (which means only on UNIX for those that use pyenv).

This works now.


Add Touch Bar category to Contents section at README.md

Touch Bar category is in Applications section, but it is not in Contents section at readme. PR fix it.

@ilyalesik Readme file is generated by CI. We need to change it in code. Please check this file.

I've added it in #383 .

Also it is a good idea to move content generation to CI. I've created an issue for that

Anyway, thank you! 😊


Update craftmanager to 1.0.64

After making all changes to the cask:

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

Closed in favor of #47479.


wait-rsjob spin loop

using a construct like get-rsjob | wait-rsjob uses an extreme amount of cpu.  It looks like the wait-rsjob code needs a start-sleep in the do {...} while ($waitJobs.Count -ne 0) loop.  As it is, it just loops hard.
This is from the code downloaded 7/27/2015 3:48PM in the PoshRSJob-master.zip.

In my mind I swear that I made this update earlier but that doesn't appear to be the case. :) I work to get this added this weekend.

Added a 100 millisecond sleep cycle. Give it a run and let me know how things look.


Introduce pipeline documentation

This PR introduces a diagram documentation about the pipeline and, how the steps interact together.

@TheLortex do you think we can merge it?

Thanks for the review! I'll fix the typos and I'll merge :)


Additional weather/fore-info (AQI data)

Acceptance criteria

[x] Precipitation
[x] Cloud
[x] AirQualityIndx
[x] Story meets the criteria in Definition of Done

Comments from Denis 8.8.

AQI compromises can be made to query it only once a day or something similar if needed
downtime at night to update could be done for 5mins or something similar if needed

AQI data is now in production.


Send Mimir's alert to OpsGenie

Towards https://github.com/giantswarm/roadmap/issues/3039
Use mimir to send alerts to alertmanager and to opsgenie.
We probably need a new heartbeat for Mimir as well

Also had to adjust some limits on the max number of rule group per tenant, let's revisit later on when we removed vintage?

I'll call that a victory https://giantswarm.app.opsgenie.com/alert/detail/e5899c93-a463-4046-acd4-63466a300e29-1710278407980/details


Implement continuous integration build and deploy process

The continuous integration build and deploy process enables testing across all engaged BSPs and the following Yocto releases: thud sumo warrior zeus.

This is in progress from another angle (meta-aws-ci)


Library Error

I installed pylsd library, however, I go this error
from lsd import lsd
ImportError: cannot import name 'lsd'

This issue is resolved in #5.
The import statement you mentioned is not correct, it should be from pylsd.lsd import lsd.

Thank you 👍

This issue is resolved in #5.
The import statement you mentioned is not correct, it should be from pylsd.lsd import lsd.

Hi , Does that work for python 3.x env ?


Update button variants to match new variants in bitstyles update

Add stories for each new variants (shape, color, size etc....)
Check new buttons variant with the old ones to see if the classname matches the new visual representation of the new buttons
check lib/bitstyles_phoenix/bitstyles.ex for examples.

Done in https://github.com/bitcrowd/bitstyles_phoenix/issues/122


From Ed Lavender on gear types (DFO/US)

"Missing types. Gear type records are missing for some surveys (DFO-HS, DFO-QCS, DFO-SOG, DFO-WCHG, DFO-WCVI and WCANN). Is this intentional? My understanding is that most of these surveys use a Western IIA trawl but I imagine there may be differences in set up. "
@zoekitchel @mpinsky

@zoekitchel @mpinsky just to make sure we get this corrected? thank you

@James-Thorson-NOAA , you probably know the gear types for WCANN?

Perhaps we leave this blank for now since we don't know.

Sorry but I'm not sure!  Do you want me to email Sean Anderson at DFO and cc y'all to ask?

@James-Thorson-NOAA do you know for WCANN though?
For DFO surveys, we can leave as is, but it'd be great someone can double check the gear was not reported but lost in the code, and I might have re-created a column that pre-existed. If the gear is not consistently reported in the database, I don't think we should infer it in the whole dataset at this point

I don't know what WCANN means ... I assumed that was Western Canada or something ... or maybe it refers to what we often call the West Coast Groundfish Bottom Trawl Survey (WCGBTS)?

Sorry that I'm slow on the context here!  Yes, I know that the WCGBTS (WCANN in that table) has used the same gear the entire time.  From context, I gather that we just need to add a code (WCANN) to a separate table in a Gear field?  If this is correct, I agree with that plan.

it's that the gear column in the integrated dataset itself for now says , so we wanted to see if we can replace that, seems like we can! thanks Jim

@jepa, we can add gear information for WCANN (Four-panel Aberdeen 85/104' bottom-trawl equipped with net mensuration gear; per Aurore's table above). For DFO-HS, DFO-QCS, DFO-SOG, DFO-WCHG, DFO-WCVI, we will leave gear blank.

Done. See my last push. Will push after reviewing all.


Fix for sorting IPs (#2334 #2995)

Suggestion for sorting IPs, possibly fixing #2334 and #2925. Didn't have a chance to test this locally yet, so please check. Method used from https://stackoverflow.com/questions/48618635/require-sorting-on-ip-address-using-js. @deviantony

ping @itsconquest for functional review

Great! Is this ready to be merged or does @itsconquest still need to review this?

Hi there @mdbraber I am going to review this functionally.
I just need to know, is your contribution:

fix sorting of containers by IP address
fix sorting of networks by IPAM subnet
or both of the above?

Cheers

Hi there @mdbraber I am going to review this functionally.
I just need to know, is your contribution:

fix sorting of containers by IP address
fix sorting of networks by IPAM subnet
or both of the above?

Cheers

Both of the above

@mdbraber have you had a chance to have another look at this?


issue on wget "https://raw.githubusercontent.com/PINTO0309/Tensorflow-bin/master/tensorflow-2.6.0-cp37-none-linux_aarch64_download.sh"

[Required] Your device (RaspberryPi3, LaptopPC, or other device name):
RaspberryPi3 B+
[Required] Your device's CPU architecture (armv7l, x86_64, or other architecture name):
armv7l
[Required] Your OS (Raspbian, Ubuntu1604, or other os name):
Raspbian
[Required] Details of the work you did before the problem occurred:
I tried $ wget "https://raw.githubusercontent.com/PINTO0309/Tensorflow-bin/master/tensorflow-2.6.0-cp37-none-linux_aarch64_download.sh" but it show error 404 can't find the path
[Required] Error message:
HTTP request sent, awaiting response... 404 Not Found
[Required] Overview of problems and questions:
Not sure why the path link can't found

wget "https://raw.githubusercontent.com/PINTO0309/Tensorflow-bin/main/tensorflow-2.6.0-cp37-none-linux_aarch64_numpy1195_download.sh"

Hi the access for the tensorflow-2.6.0-cp37-none-linux_aarch64_numpy1195_download.sh is denied unable to proceed . can you open the access? Thanks a lot

The aarch64 installer will not work on armv7l operating systems, you will need to download the armv7l files.

Hi, sorry for disturbing but I faced another issue although I had use the armv7l files but I face issue while running sudo-H pip3 install tensorflow-2.5.0-cp37-none-linux armv7l.whl. I had tried apt.get purge python3-wrapt but I can't run the apt.get command


Support more geospatial files.

There are files that have coordinates in lat/lon but no listed projection.
Closes #676.

This still isn't working for the specific data that I have

I'm seeing segfaults....
(large_image_dev) ➜  large_image git:(more-geospatial) python
Python 3.8.12 | packaged by conda-forge | (default, Sep 16 2021, 01:59:00)
[Clang 11.1.0 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import large_image
>>> large_image.__file__
'/Users/bane/Software/kw/large_image/large_image/__init__.py'
>>> large_image.open('/Users/bane/Software/OASIS/data/og_Data_scripts_Sept2021/PRISM_PPT/2017/PRISM_PPT_20170714.tif')
Created LRU Cache for 'tilesource' with 81 maximum size
TIFFReadDirectory: Warning, Unknown field with tag 42113 (0xa481) encountered.
[1]    7133 segmentation fault  python

This still isn't working for the specific data that I have

It works for the sample file you shared privately.  There is a structurally similar file here: https://data.kitware.com/api/v1/file/61969eb32fa25629b9f3638b/download

I'm getting segfaults for that file as well so something is probably wrong with my local installation. I'll investigate this soon here

I'm seeing segfaults....
(large_image_dev) ➜  large_image git:(more-geospatial) python
Python 3.8.12 | packaged by conda-forge | (default, Sep 16 2021, 01:59:00)
[Clang 11.1.0 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import large_image
>>> large_image.__file__
'/Users/bane/Software/kw/large_image/large_image/__init__.py'
>>> large_image.open('/Users/bane/Software/OASIS/data/og_Data_scripts_Sept2021/PRISM_PPT/2017/PRISM_PPT_20170714.tif')
Created LRU Cache for 'tilesource' with 81 maximum size
TIFFReadDirectory: Warning, Unknown field with tag 42113 (0xa481) encountered.
[1]    7133 segmentation fault  python

Based on that segfault, it isn't trying to parse it as geospatial -- it is using some other tile source.  What do you see with something like
import large_image

print(large_image.tilesource.isGeospatial(<path>))

Same segfault
>>> import large_image
>>> p = '/Users/bane/Software/OASIS/data/og_Data_scripts_Sept2021/PRISM_PPT/2017/PRISM_PPT_20170714.tif'
>>> large_image.tilesource.isGeospatial(p)
TIFFReadDirectory: Warning, Unknown field with tag 42113 (0xa481) encountered.
[1]    13148 segmentation fault  python

Same segfault
>>> import large_image
>>> p = '/Users/bane/Software/OASIS/data/og_Data_scripts_Sept2021/PRISM_PPT/2017/PRISM_PPT_20170714.tif'
>>> large_image.tilesource.isGeospatial(p)
TIFFReadDirectory: Warning, Unknown field with tag 42113 (0xa481) encountered.
[1]    13148 segmentation fault  python

Odd -- what version of the GDAL wheel do you have installed?

I was on 3.3.2 and so I reinstalled to 3.4.0 it's working now... 🤷🏻♂

I was on 3.3.2 and so I reinstalled to 3.4.0 it's working now... 🤷🏻♂

Were you importing any other libraries before large_image (this may or may not be germane, since the problem might occur even without such an import)?  There was a curious issue with importing torch before gdal, which turned out that one of libraries that we were building for GDAL happened to use -ftls-model=initial-exec which exhausted some local resource for static thread local storage which caused a failed import.  In other bug reports on this topic it causes segfaults.

Were you importing any other libraries before large_image

Nope, just large_image. I think I had a mismatch of versions between my python gdal bindings and the gdal binaries (on Mac so I can't use the large_image_wheels)


Shape error of update_confusion_matrix_variables with tensorflow

Hello,
I encountered a problem when using sample weights in confusion metrics with tensorflow. I think the source of the error to be line 543 of update_confusion_matrix_variables in metric_utils :
    if sample_weight is not None:
        sample_weight = ops.broadcast_to(
            ops.cast(sample_weight, dtype=y_pred.dtype), y_pred.shape
        )

which works if replaced by:
    if sample_weight is not None:
        sample_weight = ops.broadcast_to(
            ops.cast(sample_weight, dtype=y_pred.dtype), ops.shape(y_pred)  # use of ops.shape()
        )

this would also match line 551-553 in the same function below:
    if label_weights is not None and not multi_label:
        label_weights = ops.expand_dims(label_weights, 0)
        label_weights = ops.broadcast_to(label_weights, ops.shape(y_pred))   # uses also ops.shape

If this change is okay, I can make a pull request.

Thanks for the report. What is the error stack trace?
The fix looks good to me, you can open a PR.

The error would look like this
Epoch 1/50
Traceback (most recent call last):
  File ".\anaconda3\envs\cnn_keras\lib\code.py", line 90, in runcode
    exec(code, self.locals)
  File "<input>", line 1, in <module>
  File "C:\Program Files\JetBrains\PyCharm Community Edition 2021.3.2\plugins\python-ce\helpers\pydev\_pydev_bundle\pydev_umd.py", line 198, in runfile
    pydev_imports.execfile(filename, global_vars, local_vars)  # execute the script
  File "C:\Program Files\JetBrains\PyCharm Community Edition 2021.3.2\plugins\python-ce\helpers\pydev\_pydev_imps\_pydev_execfile.py", line 18, in execfile
    exec(compile(contents+"\n", file, 'exec'), glob, loc)
  File "./train.py", line 151, in <module>
    hist = model.fit(
  File ".\anaconda3\envs\cnn_keras\lib\site-packages\keras_core\src\utils\traceback_utils.py", line 123, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File ".\metrics.py", line 219, in update_state
    return super(BalancedBinaryAccuracy, self).update_state(
ValueError: Cannot convert a partially known TensorShape (None, 12) to a Tensor.

No, this is fine. The fix you proposed is correct for this issue. Thanks!


add close_range wrapper

We need to wrap this new Linux call to avoid closing of systemd sockets. New version of ssh already uses it, without this fix hack didn't work.

Thanks for this. I'm looking at the logic for this, and it's a bit hard to follow. Let me see if I can restructure it.
I'm going to use ex_first and ex_last for first and last excluded fds in this pseudocode:
# ranges do not intersect
if (
  ex_first > ex_last or
  first > last or
  last < ex_first or
  first > ex_last
):
    return _real_close_range(first, last, flags)

# we know the ranges intersect
# is there a non-intersecting sub-range before the excluded range?
if first < ex_first:
    if ret := _real_close_range(first, ex_first - 1, flags):
        return ret

# is there a non-intersecting sub-range after the excluded range?
if last > ex_last:
    if ret := _real_close_range(ex_last + 1, last, flags):
        return ret

# XXX only needed for debug
# was the range specified entirely within the excluded range?
if first >= ex_first and last <= ex_last:
    pass

return 0

Does this look right to you?

I'm also pondering whether a fallback function should be added for cases where close_range isn't available. I could update the DLWRAP macro to allow a fallback function to be provided, which at the very least could call abort().

close_range seems possible, but really annoying to simulate - I will just add code to DLWRAP which prints an error message calls abort() if the target function isn't found, don't worry about that.

I'm also pondering whether a fallback function should be added for cases where close_range isn't available. I could update the DLWRAP macro to allow a fallback function to be provided, which at the very least could call abort().

A scenario in which someone manages to call our close_range without existence of original seems unlikely, but possible

I think it would only happen if the software calling it was, itself, using dlsym to find the function, or perhaps if the binary were compiled on a newer system and run on an older one.
Ideally, we should try to continue in such a case, but otherwise, printing an error and calling abort is way better and easy to implement.
On October 28, 2023 12:39:44 PM GMT+01:00, Ivan @.***> wrote:

I'm also pondering whether a fallback function should be added for cases where close_range isn't available. I could update the DLWRAP macro to allow a fallback function to be provided, which at the very least could call abort().

A scenario in which someone manages to call our close_range without existence of original seems unlikely, but possible

Thank you for the revised PR, that's much better!
I'd still like a range check on sd_min_fd and sd_last_fd.
I can merge your PR as-is and add the check myself in a separate commit, or you can add it - which would you prefer?
I also considered whether the "range entirely inside" bit should be in an #ifdef NDEBUG block, but with optimizations enabled the compiler should automatically exclude it regardless. I'll check in Compiler Explorer (godbolt.org) to be sure.
I'm also really glad to see someone else has found this hack useful!
On October 28, 2023 12:39:44 PM GMT+01:00, Ivan @.***> wrote:

I'm also pondering whether a fallback function should be added for cases where close_range isn't available. I could update the DLWRAP macro to allow a fallback function to be provided, which at the very least could call abort().

A scenario in which someone manages to call our close_range without existence of original seems unlikely, but possible

I'd still like a range check on sd_min_fd and sd_last_fd.

You mean you want variables names inside wrap_close_range to called so?
I can do usd_min_fd and usd_last_fd like it was before.
We have to reassign variables to unsigned int to avoid warnings or add warnings ignore rule.

I'd still like a range check on sd_min_fd and sd_last_fd.

Oh, yes, in ranges do not intersect if we need to check them signed because sd_last_fd is -1 by default.

Merged, thank you.

You changed flags arg from int to unsigned int. It was int on purpose. Function in unistd.h uses int. There is miss match between syscall and lib function. It won't compile.
gcc -O2 --std=gnu11 -Wall -Wextra -Wno-unused-function -shared -fPIC -s -DNDEBUG sdsock.c addr.c -lsystemd -ldl -o libsdsock.so
In file included from sdsock.c:24:
sdsock.c:39:8: error: conflicting types for ‘close_range’; have ‘int(unsigned int,  unsigned int,  unsigned int)’
   39 | DLWRAP(close_range, int, (unsigned int a, unsigned int b, unsigned int c), (a, b, c));
      |        ^~~~~~~~~~~
dlwrap.h:38:6: note: in definition of macro ‘DLWRAP’
   38 | TYPE NAME ARGS { return _impl_##NAME ARGN; }
      |      ^~~~
In file included from sdsock.c:6:
/usr/include/unistd.h:1208:12: note: previous declaration of ‘close_range’ with type ‘int(unsigned int,  unsigned int,  int)’
 1208 | extern int close_range (unsigned int __fd, unsigned int __max_fd,
      |            ^~~~~~~~~~~
make: *** [Makefile:25: libsdsock.so] Error 1

Ugh, sorry, that's annoying. I built it on an older system that doesn't have close_range so the error wasn't triggered. I'll fix that.
On October 28, 2023 9:40:06 PM GMT+01:00, Ivan @.***> wrote:

You changed flags arg from int to unsigned int. It was int on purpose. Function in unistd.h uses int. There is miss match between syscall and lib function. It won't compile.
gcc -O2 --std=gnu11 -Wall -Wextra -Wno-unused-function -shared -fPIC -s -DNDEBUG sdsock.c addr.c -lsystemd -ldl -o libsdsock.so
In file included from sdsock.c:24:
sdsock.c:39:8: error: conflicting types for ‘close_range’; have ‘int(unsigned int,  unsigned int,  unsigned int)’
  39 | DLWRAP(close_range, int, (unsigned int a, unsigned int b, unsigned int c), (a, b, c));
     |        ^~~~~~~~~~~
dlwrap.h:38:6: note: in definition of macro ‘DLWRAP’
  38 | TYPE NAME ARGS { return _impl_##NAME ARGN; }
     |      ^~~~
In file included from sdsock.c:6:
/usr/include/unistd.h:1208:12: note: previous declaration of ‘close_range’ with type ‘int(unsigned int,  unsigned int,  int)’
1208 | extern int close_range (unsigned int __fd, unsigned int __max_fd,
     |            ^~~~~~~~~~~
make: *** [Makefile:25: libsdsock.so] Error 1

@vanym Try now?

Try now?

It compiles, but you forget to change %u to %d.
Also I think discarding ret variable not worth it, we are doing hack that replaces original functions, who knows, maybe someone will pass non -1 as error.

About closefrom and musl (alpine) support. Wrapper looks wrong, it should call _real_closefrom(sd_last_fd + 1) after all. Also there is should be debug output like in other wrappers.
It was my next step to build libsdsock in alpine docker, so I'll do that fix. Also there another build problem in alpine. It uses elogind instead of systemd, I'll research this too.

You're right about closefrom, please check the fix I pushed?

I'll do that fix

I did in my close-range branch 956f62b. Looks like Github doesn't notify about that here since it already merged.
Meanwhile you also try to fix it in e4038eb "more warnings" commit and here 8605e95. It also incorrect, if you pass number 3 to it first check will fail, second also fail and it will do nothing

Oh, you already did change it in third b2992ea commit. Looks like it'll work correctly.
Why you use "calling ... instead of ..." in wrap_close_range but did it differently in wrap_closefrom. Also there is no debug output about closing fds before sd_min_fd.
I think errno = 0  should be conditionally, because it's unlikely (btw I did unlucky misstypo in code comment) to happen and why to change it without need inside closefrom that do not change it by itself

I cherry-picked in your commit to incorporate those changes. I also modified the code to save/restore errno instead of just clearing it, just in case the caller is making assumptions.

Can you please make an issue for whatever needs to be done for alpine? I'm a little confused since I didn't think it used systemd?

I cherry-picked in your commit to incorporate those changes. I also modified the code to save/restore errno instead of just clearing it, just in case the caller is making assumptions.

Okay, thanks for compromise


Where can I find the pre-defiend skin region covering cheek,nose, and forehead?

In the definition of loss texture, one of the regularization losses, you have mentioned a pre-defined skin region covering cheek, nose, and forehead. And you told readers to see suppl. material. Where can I get this pre-defined skin region?

Actually, maybe I found a spelling mistake in your paper. In section 4.3, "where R is a pre-defined skin region covering cheek, noise, and forehead"

Hi, thanks for the correction.  In the facemodel_info.mat, there is an element "skinmask", which is the skin region we used for training.


 fields yeld NPE on getX operations.

GHPullRequest@645aa696[base=org.kohsuke.github.GHCommitPointer@7ce026d3,head=org.kohsuke.github.GHCommitPointer@7ce69770,additions=1,merged=false,mergeable=,deletions=1,assignee=,state=open,number=1903,comments=2,labels=,title=XXXXX,milestone=,url=PR_URL,id=122473208]
Will throw NPE on .getMergeable()

https://developer.github.com/v3/pulls "The value of the mergeable attribute can be true, false, or null. If the value is null, this means that the mergeability hasn't been computed yet, and a background job was started to compute it. Give the job a few moments to complete, and then submit the request again. When the job is complete, the response will include a non-null value for the mergeable attribute."
This should be reflected in field. Worse case scenario with Boolean, so null can be one of potential values or with proper enum YES,NO, NOT_DEFINED( or something similar)

Without stack trace, I'm afraid I cannot fix this.

I can include stack trace but it will just indicate place in our code which triggers NPE due to unboxing of Boolean: https://github.com/kohsuke/github-api/blob/master/src/main/java/org/kohsuke/github/GHPullRequest.java#L55
Getter/field type is wrong. Either getter must indicate full use of three potential value( as opposed to boolean) - null, True, False/(or True, False(false, null) or it should be changed to enum.


Cfg.py: move succlabels to a larger scope

Also, switch to use .get() instead of raw subscripting, in case there are no labels for a given successor.

You are right, using get is better than raw subscripting in general. Thank you!


Volume control breaks at 0%

When using the template skin and Spotify, setting the volume to 0% will instead cause the skin to show the volume as 100%, at which point the volume controls on the skin cease to work. Other things, such as play/pause, continue to work and control Spotify, though.
I believe the following line may be the issue, after a quick glance.
https://github.com/keifufu/WebNowPlaying-Redux-Rainmeter/blob/437d1944e3a6c1433a61bcea198a4525da8da0ba/src/WNPReduxAdapterLibrary/WNPRedux.cs#L166

Before looking into this i want to make sure you're using Spotify web and not the desktop player, right?

I am using the desktop player, with Spicetify and the associated plugin, if that changes anything.

The Spicetify extension is no longer maintained as WebNowPlaying >2.0.0 can read from desktop players without it. The windows API used for this doesn't allow controlling volume and rating though.

I see, this issue would be with the extension rather than here if it were not deprecated. I have just tried to replicate the issue with the web player and it works fine, sorry!

All good. I'd personally recommend not using Spotify but that's biased.
If you're using a Chromium based browser you can make Spotify web a PWA, so it'd feel more like a desktop app.


Set mbedTLS version to 2.16.6

Description
Security

Fix side channel in ECC code that allowed an adversary with access to
precise enough timing and memory access information (typically an
untrusted operating system attacking a secure enclave) to fully recover
an ECDSA private key. Found and reported by Alejandro Cabrera Aldaya,
Billy Brumley and Cesar Pereida Garcia. CVE-2020-10932
Fix a potentially remotely exploitable buffer overread in a
DTLS client when parsing the Hello Verify Request message.

Bugfix

Fix compilation failure when both MBEDTLS_SSL_PROTO_DTLS and
MBEDTLS_SSL_HW_RECORD_ACCEL are enabled.
Fix a function name in a debug message. Contributed by Ercan Ozturk in
#3013.
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@dcgaws there are no new config macros, so we are OK.


Aws stripe disks fix

Fixes https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/issues/1905, based on an initial implementation from @HarpPDX.
Right now, the AWS stripe disks feature wrongly assumes that the boot drive is always at nvme index 0, while in reality the boot drive index can be at any nvme index. This diff fixes that assumption and also adds support for Clear Linux.

Thanks for the fix.
There are conflicts in this PR. If you are able to resolve those conflicts, we can merge this change.

@s-deitz I have rebased with master. Hope everything works now.

Thanks. That worked and I have merged these changes.

Thanks @s-deitz. Appreciate the quick response.


make setup mounts project root as read-only

https://github.com/neuromation/cookiecutter-neuro-project/blob/06aee7e4a384ea1d481b80f738331f97b2de10f6/{{cookiecutter.project_slug}}/Makefile#L68
So: either we don't need this mount, or, if we do write there, it should mount as rw

Why is this a problem?
This mount is used for reading files with pip and apt requirements. AFAIU, you don't need to update these files or the content of this folder while performing set up, but you need to read these files. Is it correct?

outdated


Mark tasks done in TW as done in Asana

Forgot to implement a very crucial thing!

Done. https://github.com/manishrjain/asanawarrior/commit/b531899c107bd250c589300aff8b92c59a0d4a88


[feat]: Display Tor address on LAN setup page when accessed over LAN

Prerequisites

[X] I have searched for existing issues that already suggest this feature, without success.

Describe the Feature Request
I have to go find my Tor address whenever I want to download my Root CA. Why? It should just display on this page:

Describe the Use Case
Eases the already frustrating experience of installing root CAs
Describe Preferred Solution
Tor address should display on the LAN setup page.
Describe Alternatives
Alternatively, could display a link to the Tor address in the About section
Anything else?
No response

I think it was for security reasons that we initially suggested to only download the cert over Tor, remind me why @dr-bonez . If there isn't a significant attack vector / value of doing this, then I agree it would be useful to display on this page.

If they are accessing on lan, it's probably an insecure connection if they've not downloaded the CA yet. But that also means they've sent their master password over an insecure connection anyway so I'm not sure how much it matters.

Right, and the requested feature is not that we allow downloading of cert over LAN, just that we make it easier to find the Tor address from this page.

For security reasons you must set up LAN over a Tor connection. Please navigate to your Embassy Tor address and try again.

For security reasons you must set up LAN over a Tor connection. Please navigate to your Embassy Tor address and try again.

I'm for it

Right, and the requested feature is not that we allow downloading of cert over LAN, just that we make it easier to find the Tor address from this page.

Whoops, I left misunderstood the original ask 😅
For sure to all of this.


ERROR [ file 'atac.bds', line 1304 ] :	Function parse_align_log_to_html(string,string[],string[],string[]) cannot be resolved

Hi,
I am trying to run the atac dnase pipeline and am getting this error.
bash-4.1$ bds atac.bds -species hg19 -nth 4 -se -bam1 file1.bam -bam2 file2.bam
Picked up _JAVA_OPTIONS: -Xms256M -Xmx728M -XX:ParallelGCThreads=1
Compiler messages:
ERROR [ file 'atac.bds', line 1304 ] :	Function parse_align_log_to_html(string,string[],string[],string[]) cannot be resolved
ERROR [ file 'atac.bds', line 1305 ] :	Function parse_dup_to_html(string,string[],string[],string[]) cannot be resolved
ERROR [ file 'atac.bds', line 1306 ] :	Function parse_flagstat_to_html(string,string[],string[],string[],bool) cannot be resolved
ERROR [ file 'atac.bds', line 1307 ] :	Function parse_pbc_to_html(string,string[],string[],string[]) cannot be resolved
ERROR [ file 'atac.bds', line 1308 ] :	Function parse_xcor_to_html(string,string[],string[],string[],string[]) cannot be resolved
ERROR [ file 'atac.bds', line 1312 ] :	Function parse_idr_to_html(string,string) cannot be resolved
ERROR [ file 'atac.bds', line 1327 ] :	Function parse_ataqc_txt(string,string[],string[]) cannot be resolved
Compiler messages:
ERROR [ file 'atac.bds', line 1304 ] :	Function parse_align_log_to_html(string,string[],string[],string[]) cannot be resolved
ERROR [ file 'atac.bds', line 1305 ] :	Function parse_dup_to_html(string,string[],string[],string[]) cannot be resolved
ERROR [ file 'atac.bds', line 1306 ] :	Function parse_flagstat_to_html(string,string[],string[],string[],bool) cannot be resolved
ERROR [ file 'atac.bds', line 1307 ] :	Function parse_pbc_to_html(string,string[],string[],string[]) cannot be resolved
ERROR [ file 'atac.bds', line 1308 ] :	Function parse_xcor_to_html(string,string[],string[],string[],string[]) cannot be resolved
ERROR [ file 'atac.bds', line 1312 ] :	Function parse_idr_to_html(string,string) cannot be resolved
ERROR [ file 'atac.bds', line 1327 ] :	Function parse_ataqc_txt(string,string[],string[]) cannot be resolved
I downloaded and extracted the pipeline, pointed it with PATH and LD_LIBRARY_PATH to Miniconda3 and Java, ran ./install_dependencies.sh, ran ./install_genome_data.sh for hg19, all of those were successful. When I go to run the pipeline I get this error. Have you seen this before?
Thank you!

There was a major update for the report generation and those compilation errors seem to come from it. Please git pull the code and try again with bds /path_to_code/atac.bds to check any complication errors.

Done! The error is fixed. Thank you!


[PROPOSAL] Renaming project?

Thank you all for this nice guide! It's a good introduction for beginners to git.
Since it seems to become more a general git-guide than a "how to write good commit messages"-guide it may be better to rename the project.
It also may be better to reorganize some things, because the jump from "good commit message" to "rebase vs merge" was a bit confusing for me as a first-time reader. Maybe adding some kind of TOC or something...
What do you think about this proposal?

Hey, @twity1337. I think a TOC would be great and would help people what they want to find faster (feel free to open an issue on that).
Even though there is a lot of more information about git and commits in general, I don't have any intention to turn this guide into a general purpose git guide. I think renaming the project would cause the project to be bad at being a general guide. This would require additional effort to be at a "good enough" state, which we may not achieve. That's why I think that keeping it focused in the commit messages (although not strictly restrict to it) it's better.


Add option to define custom parameter for Inbound and Connectors

Description:
In some inbounds and connectors, there may be requirements to pass custom properties to the 3rd party SDKs. We need to make an improvement to the form UI to allow the user to define the parameters as key value pairs.

The latest release v1.1.0 has resolved the issue. Please reopen if the problem persists.


[DDW-26] react polymorph button with loading spinner

This PR adds ButtonSpinnerSkin to simple skins in order to show the LoadingSpinner component within Button when page data is loading.

Adds loading prop (boolean) to Button component.

Adds "with LoadingSpinner" story to Button stories to show new skin and functionality.

@DominikGuzei
You'll need to merge PR 75 before this one. This branch branches from feature/ddw-27-react-polymorph-loading-spinner in order to use the new LoadingSpinner component.


[Question] Do you have a vue-float-menu that works with Vue 2 via CDN?

Hi there
Congrats on this component, do you happen to have one that I could test out with Vue 2 via the CDN

@Mashiane Yes, you can use unpkg.com for this. here is the link
https://unpkg.com/react-float-menu


Export/Import unprotected sites / allow list

Allow an easy import/export of allow listed sites, even better would be a shared format with your privacy essentials browser extension.

Hello @IOI-655321  , thanks for taking the time to submit this feature request. Sorry for the very late response, we are currently going through old feature requests.
After internal discussion with the team, we've agreed that there's no plan in the foreseeable future to support this request. Should this change in the future, we'll update this conversation appropriately.


Is there any callback to find out when synchronization (local DB <> Firebase) has been completed?

Is there any callback to find out when synchronization (local DB <> Firebase) has been completed?
Because based on such issues https://github.com/chat-sdk/chat-sdk-android/issues/666 is would be really helpful

I haven't found anything on this but check the last comment i made on that issue
it works for now

Did you find any solution? My problem is if the user sends a message and the message was still in the sending/uploading state and the user go to any other activity, the firebase and receiver receive the message but the status of the message on the sender side remains sending/uploading. Any help would be highly appreciated

Did you find any solution? My problem is if the user sends a message and the message was still in the sending/uploading state and the user go to any other activity, the firebase and receiver receive the message but the status of the message on the sender side remains sending/uploading. Any help would be highly appreciated

Using greendao I don't think so but using Database refs it is possible asynchronously
ChatSDK already handles this though
Look into when it changes the state of the message or just general uses of the message state and dig through them

The database is synchronizing whenever new data comes in from Firebase. To find out when that is you can use the event object.
Disposable d = ChatSDK.events().sourceOnMain()
            .filter(NetworkEvent.filterType(EventType.UserPresenceUpdated, EventType.ThreadUserRoleUpdated))
            .subscribe(----lamda here----);

Event types:
public enum EventType {
    ThreadAdded,
    ThreadRemoved,
    ThreadDetailsUpdated,
    ThreadMetaUpdated,
    MessageAdded,
    MessageUpdated,
    MessageRemoved,
    MessageSendStatusUpdated,
    ThreadUsersUpdated,
    ThreadUserRoleUpdated,
    UserMetaUpdated,
    UserPresenceUpdated,
    ContactAdded,
    ContactDeleted,
    ContactsUpdated,
    TypingStateUpdated,
    Logout,
    ThreadRead,
    MessageReadReceiptUpdated,
    NearbyUserAdded,
    NearbyUserMoved,
    NearbyUserRemoved,
    NearbyUsersUpdated,
    Error
}

GreenDao doesn't provide any way to notify us when it's data changes annoyingly. But you can use these notifications.


Start month

Hi, excellent work, first of all. I just suggest that the initial month of the calendar should be the one of the initial date: starting always from January is mostly impractical

Hmm, it should automatically scroll to the selected month on open. That's a bug.

We're experiencing this too. :)

I got the same problem. Hope you find tha way to eliminate this issue soon

Woohoo! Thanks for the quick response. After your fix, its work now.

Seems that it ignores the scrollTo as it is not sized yet (offsetHeight = 0).  I worked around for now with:
_showDialog: function() {
  this.$.dialog.toggle();
  this.async(function() {
    this.$.picker.$.calendar.updateSize();
  }.bind(this));
},

samer here.

@jasongardnerlv Thanks, that helped! Unfortunately it scrolled to August whereas the selected date was July :0 (I use de locale if it makes any difference)

This should have been fixed since 1.0.3. Are you still seeing an issue? If so, can you provide steps to reproduce?

Closing for now. Let me know if anyone's still experiencing issues.


🐛 BUG: Pages middleware plugins always return status 200

Which Cloudflare product(s) does this pertain to?
Pages
What version of Wrangler are you using?
2.6.2
What operating system are you using?
Linux
Describe the Bug
Our Remix app deployed on Pages uses a 302 redirect to our identity provider if a user doesn't have an active session. We recently added @cloudflare/pages-plugin-honeycomb using a _middleware.ts file. Since introducing this dependency, we no longer get redirected to our identity provider. Instead, we see a blank page returned with a 200.
@cloudflare/pages-plugin-honeycomb is compiled using wrangler with npx wrangler pages functions build --plugin --outfile index.js. The compiled code includes the following:
return new Response(
  [101, 204, 205, 304].includes(response.status) ? null : response.body,
  { ...response, headers: new Headers(response.headers) }
);

This appears to be a pattern used by wrangler when compiling pages functions.

pages-plugin-template.ts
pages-template-worker.ts

If you try and run this in Chrome, you can see the status gets stripped from the new Response, causing it to default to 200.

It appears that it's not possible to spread a Response object meanings its values are never copied to the new Response.

This issue does not affect miniflare. I suspect it has to do with the specific implementation details of the fetch shim used by miniflare vs the legit fetch implementation used by V8.

2.6.2

This is an old version of Wrangler.
The bug was fixed in 2.7.0

Whoops. I probably should have noticed that when I was copying the links to the code! Sorry about that!


Supporting srcset attribute analogously to src attr in image_tag

There is a relatively new srcset attribute in img tag that can be useful for HiDPI/Retina devices and is the second easiest way to support them. To properly use it in Middleman it should be handled analogously to src attribute (supporting relative/absolute paths computing). Unfortunately it also means hacking it in Markdown engines and other high-level HTML-abstractions plugged into Middleman.
I'm considering an example with extended syntax using attribute list definition from Kramdown:
![Alternative text](/folder-relative-or-absolute/image.jpg "Title attribute"){<EMAIL_ADDRESS>2x"}

This should generate (in case of relative paths enabled and Markdown source 1 folder deeper than image folder):
<img alt="Alternative text" title="Title attribute"<EMAIL_ADDRESS>2x" src="../folder-relative-or-absolute/image.jpg">

The generated path in srcset attribute of img tag should be computed relative/absolute-wise the same way as src is. Currently it’s not, so the ../ part is missing, I only get /. To make things more difficult the syntax of srcset is quite flexible and contains more than a path, in the simpliest case.
Any hope?

I'm doing some issue-gardening 🌿🌷🌾 and came across this issue.
Did the suggestion by @tdreyno work? If so, can this be closed?

Wow, I haven’t seen dperrymorrow’s commit. Will check if it works (in MM 3.4.1) any time soon. Unfortunately I’m a occasional user (and don’t use MM4 in any project yet, at least not untill some crucial gems are compatible).


[NeuralChat] RAG evaluation

Data augmentation: retrieval dataset construction, include (1) Context to Question and Mine Hard Negatives, (2) Context, Question to Ground Truth.
Retrieval evaluation: MRR (Mean reciprocal rank) and Hit (Hit Ratio) under different vector database, embedding model, retrieval type, search type.
RAG evaluation:  answer relevancy, faithfulness, context recall, context precision using Ragas framework.

@Liangyx2  @XuhuiRen any update?

@Liangyx2 @XuhuiRen any update?

No, only to solve the CI error if occurs.


Add Foundry CLI installation instructions

@rhinchey-cs Is it possible to give me write access to this repo so I can merge?


Move field displays from bottom to top?

Hello, is there any way to modify where a field is in the card?  I only really want the fields Sunrise/Sunset/SolarNoon (see image below), but would love for them to all be on the same line instead of solar noon on the bottom.  Is there any way to change this, maybe with card-mod?

Yes please! Dashboards space is very limited and moving solar noon between sunset and sunrise would be ideal.

I wanted to request the same feature. I don't even need the noon label. It would be enough for me if the noon time was written in smaller font size between sunrise and sunset.
In my case, the reason is also that my dashboard is relatively full and this adjustment would make the card much more compact.


tps verify: Warn if public/samples/X.in mismatches with gen/manual/X.in

This check is important to ensure that samples cases provided to the contestant matches with the one in the grading server.
However, this is open for debate whether we want the file to match exactly. Is there a use case where we want the sample cases to be different in public-grader and secret-grader format?

Closing this as we can use symlinks if we do want them to be the same.


Notification standards

There is a PR on notification standards that has been stale for quite some time: https://github.com/NearSocial/standards/pull/19.
The notification standards are an important part of documentation for developers. This task consists in fixing the PR and merge it into NearSocial.

Thanks for flagging this @milenakrawczyk ! I came across it going through the standards repo in NearSocial.
@Ishatt can you assist with making sure we don't lose track of this issue? Zooming out, and working with the NearSocial Standards repo will be super important as we start to standardize primitives like this and component forks.


Using local/development storage?

I'm using the Azure Storage Emulator. How do I tell Python to connect to my development emulator instead of the official server?

See my comment here:
https://github.com/Azure/azure-sdk-for-python/issues/431

Added that to the top of my code - No change.

Can you share your code?

from lib.post import PostLoader
from azure.storage import TableService, Entity
import json, urllib2, urllib
import os
from datetime import datetime
from time import sleep
os.environ["EMULATED"] = "true"
Read config
tiwt_keys = open('keys.json')
keys = json.load(tiwt_keys)
tiwt_keys.close()
graph_url = "https://graph.facebook.com/v2.2/"
table_service = TableService(account_name=keys['azure_storage_account'], account_key=keys['azure_storage_key'])
That's enough to cause the problem

Change that last line to:
table_service = TableService(account_name=None, account_key=None)

Now it complains that the HTTP values are not in the correct format. Either way, authentication is still required for the emulator

When it sees None or empty for account name and key, and ENVIRON set, it will set the correct flag and use the emulator credentials.
https://github.com/Azure/azure-sdk-for-python/blob/master/azure/storage/storageclient.py#L89
It sounds like you may be getting a different unrelated error now.  Are you running this inside a worker role in PTVS?  If not, did you manually start the emulator first?
You should paste the exact error you are getting.

This is what I get: Traceback (most recent call last):
File "main.py", line 99, in
main()
File "main.py", line 67, in main
table_service.create_table('posts')
File "F:\Python27\lib\site-packages\azure\storage\tableservice.py", line 261,
in create_table
dont_fail_on_exist(ex)
File "F:\Python27\lib\site-packages\azure_init.py", line 527, in _dont_fai
l_on_exist
raise error
azure.WindowsAzureError: Unknown error (The value for one of the HTTP headers is
not in the correct format.)

InvalidHeaderValue
The value for one of the HTTP headers is not in the
correct format.

What's a worker role PTVS thing?

https://github.com/microsoft/ptvs/wiki/Cloud-Project
I'm able to create a table successfully when the emulator is running.  If I don't start it, I get an error that it refused connection.  Not really sure what could be causing the error you are seeing.  If you can print out the contents of the request, that might help.  All the calls will go through this:
https://github.com/Azure/azure-sdk-for-python/blob/master/azure/http/httpclient.py#L218

So I'm not using PTVS. Can you elaborate on printing out the contents of the request?


Build

style consistency and overview of grammar

build to main: no conflicts/errors: styles consistency & grammar overview changes integrated


API bestiary

bestiary object with info on kills and deaths

Yeah I can see how that would take a while, although with enough time you could have it show which mob needs the least amount of kills to level up so you can work out what the next best mob to grind for would be

from discord suggestions


Rust: error: 'cargo-clippy' is not installed for the toolchain 'nightly-x86_64-apple-darwin' but is available on the path

Describe the bug
When setting language.rust.enable = true, all the toolchain packages are available, including cargo-clippy. However, cargo does not seem to know it's there.
To reproduce
https://gist.github.com/duckpuppy/ac757f17ad717429860a79fe031d838b
$ cargo init
     Created binary (application) package

$ cargo clippy
error: 'cargo-clippy' is not installed for the toolchain 'nightly-x86_64-apple-darwin'
To install, run `rustup component add clippy`

$ cargo-clippy
    Checking test-rust-clippy v0.1.0 (/Users/paikens/src/personal/test-rust-clippy)
    Finished dev [unoptimized + debuginfo] target(s) in 0.49s

$ which cargo-clippy
/nix/store/hggdp3ccc41wallkn3lnm4yh4mz6hm5q-devenv-profile/bin/cargo-clippy

$ which cargo
/nix/store/hggdp3ccc41wallkn3lnm4yh4mz6hm5q-devenv-profile/bin/cargo

Version
devenv: 0.6.2

hey @duckpuppy can you try again with latest main now that #667 has landed?
if it still doesn't work, can you try again with languages.rust.channel = "stable";?


About X-Decoder

Thank you for your great work.
I’m trying to run X-Decoder in a local environment.
For Train and Evaluation, what kind of PC specs do you expect when running on a Single GPU?
Also, for DATASET.md, do I need to download the entire dataset?
Finally, regarding the demo, when do you plan to release it?
I look forward to hearing from you.

training is not able to run on a single gpu. it is run on 32 V100 gpus.
you can run inference code on a single gpu.
for training/evaluation, yes you need to download the whole dataset.
we have already release the demo, please refer to hugging face: https://huggingface.co/spaces/xdecoder/Demo/tree/main


WIP: Introduce support for AES hardware acceleration in WASM targets

You can enable this by enabling the Rust feature wasm when building to a WASM target. For now the code does not check on target or provides any kind of fallback, When using this feature on non-WASM targets or when using a browser that does not support web crypto, it will therefore fail now.

Included in #10 . Thanks for the initial code though!


CSV upload transition: if an upload user tries to log in, provide help text to direct them to SR

Problem statement
As a CSV Uploader user, I need to see the following messages pre, during and post transition of this service to SimpleReport:
1 - When the feature is not yet Live on SimpleReport
Inform the user of the upcoming change about the upload feature and that the submission history will continue to be available until further notice.
2 -  When the feature goes Live on SimpleReport but also available on ReportStream during the transition period.
Inform the user the feature has migrated to SimpleReport and provide them with the link to the new location. Let the users know that the service will be removed from ReportStream by x date and that the submission history will continue to be available until further notice.
3 -  After the feature has fully migrated to SimpleReport and has been removed from ReportStream
Inform the user the feature has migrated to SimpleReport and provide them with the link to the new location. The submission history will continue to be available until further notice.
Questions
Q: Is it possible to flag our CSV upload user's credentials so we can show specific help text if they try to log in?
A: Yes.
Need to know
Link to the transition documentation created by the SimpleReport design team
https://skylight-hq.slack.com/files/U01PJ2ZPD2L/F03G1P1NAV8/csv_uploader_transition_-_communications_plan_shared
Planned order of events:
1 - Pre-transition banner goes up on ReportStream about upcoming transition to SimpleReport (aim to complete by next Sprint (6/8/22)
2a - CSV Uploader goes Live in SimpleReport - July 1 ETA
2b - Transition banner goes up on ReportStream informing the user that the site is now also available on SimpleReport and a link to go there. The user can continue to upload on ReportStream for now.
3 - CSV Uploader is disabled on ReportStream, directing the user to SimpleReport. July 14 ETA
Acceptance criteria

[ ] Help content should inform users of the location change before, during and after the transition.
[ ] If a deactivated CSV upload user tries to log in, we direct them to help content that gets them started with SR.
[ ] Help content should inform users of the deactivated feature
[ ] Help content should inform users of how to get connected/in touch with SR

To do

[ ] ...

@drew-usds Is this a must have, given there will be an alert message on pages post log in?

@drew-usds Scratch my previous question. This ticket is for after Simplereport has Uploader up and running. I'm moving it to our backlog.

We'll have to ensure that the user logging into ReportStream is in an Okta group that has access to ReportStream. Access to login is not permission to operate anything within RS, though, so we can still protect all in-app operations from being access while allowing them in so we can pass them over to SimpleReport.

Design choice: Do something once a user logs in **
not when they try to log in.

@drew-usds I know you missed refinement and we did not get to this in our sync yesterday so wanted to tag you here and we can add to our refinement or perhaps do another sync today/friday to get to the tickets in the slack thread "Update for 4/21: we will did 3815 but did NOT get to 5109, 5208, 5145, 4338 and 4790 perhaps 4338 @Andrea Chappelear (MO, she/her) perhaps you can look at this a bit more @drew.gardner think you can look at 4790 async. perhaps we can bump to Tuesday but if we can do another meeting Friday to finish up"

Per our backlog refinement today, @RakshindaAslam is taking an action item to sync with Product owners cross SR/RS to discuss plan for the transition - to help inform what content/design work is needed for this ticket so we can unblock # 5295. Try to complete sync/get info within week or so to meet July 1st CSV uploader transition deadline. cc: @a-chappelear @bgantick @HeatherRienks @drew-usds

Regarding comment above, a meeting is now set for 5/20/22 with RS/SR team to discuss transition.

Updates 5/25: I'm starting to work on the content for the email series (# 5140) and will adapt some of that content for this ticket. Will provide updates this week before I leave on PTO (Friday 5/27).

Updates!
Figma files for the banner (s) can be found here:
Pre-move, https://www.figma.com/file/VrSkTA87t8PPeXPugQ0i8O/ReportStream?node-id=3209%3A22143
....and should it be determined to not want an auto-redirect, but instead have another banner that will redirect users via the link, https://www.figma.com/file/VrSkTA87t8PPeXPugQ0i8O/ReportStream?node-id=3210%3A21918
From a design perspective, this uses the same alert banners from USWDS and are already in place on RS. The blue is for pre-move messaging and switches to the yellow version once the event has passed and it needs to be more "alert"-ish.
Content is based on the draft work from the email(s) mentioned above, but guessing this may alter slightly (ie. we might want to link to help content that is on SR when available.) Easy to update as needed!
@RakshindaAslam @BerniXiongA6 @drew-usds


@Cadl.Rest.segmentSeparator doesn't work with Azure.Core.ResourceCollectionAction

...and it mostly but doesn't fully work with ResourceAction...
import "@azure-tools/cadl-azure-core";

@Cadl.Rest.resource("whocareses")
model WhoCares {
  @key
  id: string
}

@Cadl.Rest.segmentSeparator("x")
op almostWorks is Azure.Core.ResourceAction<WhoCares, {}, {}>; // <-- produces a /whocareses/{id}/xalmostWorks route. The slash in front of the 'x' is questionable....

@Cadl.Rest.segmentSeparator("/")
op works is Azure.Core.ResourceAction<WhoCares, {}, {}>; // <-- produces a whocareses/{id}/works route

@Cadl.Rest.segmentSeparator("/")
op doesntWork is Azure.Core.ResourceCollectionAction<WhoCares, {}, {}>; // <-- produces a /whocareses:doesntWork route. Not what I would expect...

Playground Link

est: 3

@johanste in the case of segmentSeparator("x") there is logic that says the only allowed separators are / and : and thus if the segment doesn't start with one of those, it appends /.  I could change that logic, but would it make more sense to prevent anything but / or : as allowed values for segmentSeparator. Thoughts?

I don't think that appending '/' is the right thing to do (not sure when I'd ever use that). I know that some services use a '+' or ',' in some cases (not Azure). At least one Azure service has (tried to use) '.'. If we want to restrict, we can restrict (and error out if it is not one of our approved ones), but appending '/' is not right. We may want to special-case empty string.

@daviwil @johanste so, the problem with ResourceCollectionAction is the order in which things get evaluated:

@segmentSeparator(":") on ResourceCollectionAction
@collectionAction on ResourceCollectionAction
@segmentSeparator("/") on Foo is ResourceCollectionAction

By adding @collectionAction(WhoCares) to doesntWork it then does work, but it doesn't seem like it will be easy to fix this.

It sounds like a premature application of a decorator where last-in-should-win (I think).

@markcowl after looking into this and discussing with David and Johan, this is no longer est: 3. It's probably more like est: 8-13.


Pacific Power Association (PPA) stations

Closes #40
Add Pacific Power Association (PPA) stations.

These stations may no longer be in operation...


Add XY field

Please add coordinates values to both sides' datasets. Thanks, Moran

Not sure where the geolocation is fed into the map.

ok, done in both hit and place.


Least squares psf

Add a (pretty slow) least squares fit method to the PSF module.

Removed Docile stuff.

UndefVarError: NelderMead not defined

Aha, Optim's default documentation doesn't match its last tagged version.
The last release, v0.4.4, uses different syntax than the master version
that I am running on my computer.
I'll switch to the obsolete syntax, I suppose.
On Mon, Mar 7, 2016 at 6:21 PM, Jeffrey Regier<EMAIL_ADDRESS>wrote:

UndefVarError: NelderMead not defined
—
Reply to this email directly or view it on GitHub
https://github.com/rgiordan/SloanDigitalSkySurvey.jl/pull/5#issuecomment-193560103
.


Fix ammo pickup widget showing in drone

When in drone and looking at a dead body the ammo pickup widget will show up sometimes (see screenshot below). Note this might only happen on World Partition levels where we move a AWorldPartitionStreamingActor (with a UWorldPartitionStreamingSourceComponent) to where the player is looking to force World Partition to load that cell. It seems there's 2 bugs here:

When in drone you should never see the ammo pickup widget
We should be considering the player close to the ammo source if it's the AWorldPartitionStreamingActor that's overlapping with the pickup sphere. (Use L_LargeMap for world partition)

I couldn't replicate this bug.
Did you play the game in multiplayer? If not, then please try to replicate the bug on this branch with VLOG enabled, and write down repro steps.
https://github.com/HaywireInteractive/OnAllFronts/tree/bugfix/Issue-907_AmmoPickupWidgetShowingInDrone

Yes it was multiplayer on City Sample, and using separate process for server. But I tried again on L_Large and couldn't repro. Next time I'm on City Sample I'll try to reproduce and see if I can do it consistently.

I just reproduced this. I was in L_SmallNoObstacles, client net mode, running under one process, with pm.GiveAllSoldiersDrone 1. I first moved in drone a bit away from soldiers, then pressed enter to start combat and eventually this showed up:


Strange problem

Hi Daniel
Not sure if you're interested in helping me debug something, but I made use of this implementation as part of a proof assistant I am implementing.
Firstly, I think line 48 (where you check if you should raise a variable) is incorrect. Should it not read:
 LocalVar j -> if j >= lower then LocalVar (i + j) else LocalVar j

?
Secondly, I am now encountering a problem where it fails to make progress on this constraint:
(Lam (Ap (FreeVar 0) (LocalVar 0)),Ap (MetaVar 4) (FreeVar 0))

Pretty-printing this:
(\x. P x)  ~ ?4 P

For some reason, it spins around on this and doesn't find the obvious solution  ?4 := (\y.\x. y x). I'd appreciate any help you can give. It's also possible I somehow introduced a bug that wasn't present in your original implementation. If that's the case, I'm sorry to have bothered you!

Ack, I'm sorry, I just saw this now!

I'm not stuck anymore, as I switched to pattern unification instead (works for what I'm doing). Still would be interested to figure out what was going wrong, but no urgency!

I think pattern unification is probably sufficient for a proof assistant yeah, I would like to figure it out as well. I'll have to spend some time looking at code my longtime nemesis (past me) wrote.

Hello there! I am now also looking at this code as a source of inspiration, thanks for making it public :)
I think that the problem with non-termination is because of this line:
https://github.com/jozefg/higher-order-unification/blob/21382f44205aa3d8b115fe2b2eba47489da4b492/src/Unification.hs#L199
I believe this line exists to avoid identifying meta-variables with local (bound) variables. This is a bit confusing (why do we need an isClosed check here if we're concerned about bound variables). But what we are trying to check here is that we did not capture any of the bound variables which we previously converted into fresh free variables upon entering the scopes here:
https://github.com/jozefg/higher-order-unification/blob/21382f44205aa3d8b115fe2b2eba47489da4b492/src/Unification.hs#L164-L173
So if you remove isClosed check then the example of @liamoc works well. However, it is not correct to remove it entirely. Instead one could introduce another constructor (e.g. FreeLocalVar or FreshVar or something like that) and check for absence of those.


Confused about the implementation

Hey there, I got a question about the way this tool is implemented: Why do I need an API token? From the looks of it, you're scraping the lyrics directly from the official webpage anyway. Wouldn't it make more sense to fetch lyrics through Genius' API?

Genius API doesn't have an endpoint for returning lyrics. The API is used for searching songs and getting the link to the lyrics page (where we then scrape the lyrics from).

I noticed something a while ago that can potentially be used to directly fetch the lyrics and that is Genius's internal edit lyrics API. It returns the lyrics as plain text which is displayed to the user for editing. I'd welcome a PR if anyone's interested in looking into that.

Wait what? Genius, the website whose main selling point is lyrics, has an API that returns anything BUT lyrics? Why even have an API then???

I noticed something a while ago that can potentially be used to directly fetch the lyrics and that is Genius's internal edit lyrics API. It returns the lyrics as plain text which is displayed to the user for editing. I'd welcome a PR if anyone's interested in looking into that.

What endpoint is that?

Wait what? Genius, the website whose main selling point is lyrics, has an API that returns anything BUT lyrics? Why even have an API then???

I believe the purpose of the API is to bring traffic to Genius web pages. Having an endpoint that returns lyrics as well would drive the traffic AWAY from Genius and hurt whatever they have for a business model. There might be legal issues with lyrics copyright as well.

What endpoint is that?

https://genius.com/api/songs/3365691/lyrics_for_edit_proposal?text_format=html%2Cmarkdown
It takes a song ID and returns the lyrics for it as plain text. Quite nice 😈. Let me know if you're interested in creating a PR for it. Otherwise, I would do it.

I believe the purpose of the API is to bring traffic to Genius web pages. Having an endpoint that returns lyrics as well would drive the traffic AWAY from Genius and hurt whatever they have for a business model. There might be legal issues with lyrics copyright as well.

Okay, that actually makes some sense. Although then I still wonder why they even bothered with an API in the first place. Oh well.

Let me know if you're interested in creating a PR for it. Otherwise, I would do it.
Unfortunately, I don't actually work with JavaScript that much. I only discovered this page because I was looking for other people implementing the Genius API and was wondering how they did it because my own efforts had me brickwalled at actually getting the lyrics. I didn't know it wasn't even part of it.

Thanks for the link though! I'll try and see if that works for my usecase :)

Hmm, from the looks of it, that URL only works while logged into Genius (like the actual website). In a private tab it doesn't work, and thus also not from a serverside application. What a shame...

Try this. Open the URL in a tab (non-private), open the network tab in chrome dev tools and catch the request, right-click the request, and choose "copy as cURL". Then paste it into your terminal and hit enter. If it works (which I'm sure it will), you only need to replicate that cURL request in your code somehow (it might just be really hard to implement programmatically in case they're using a rotating token for auth). Let me know how it goes if you try it.

I just can't get this to work. I tried every terminal I have installed (git bash, PowerShell, regular old cmd), but no matter what I do, I never get a result. It waits roughly half a second, then simply exists without any results:

Either way, however, it looks like this won't be a stable long-term solution anyway. I don't think these guys will keep the cookies unchanged indefinitely.

Okay, I got it to work. It seems the response is cached by the browser and when the endpoint returns 304, the browser just shows the cached version. When you copy that 304'ed request as cURL, it doesn't work in the terminal (since no cache there).
To fix it, hard refresh the page when you catch the request in the network tab so it fetches a new response instead of falling back to the cache. The status code will be 200 for a fresh request. Then copy and use that in the terminal.

Either way, however, it looks like this won't be a stable long-term solution anyway. I don't think these guys will keep the cookies unchanged indefinitely.

You're probably right.

Oh, that actually did the trick! It was the cache after all. Thanks!
I'm still not 100% convinced that this is a stable solution, but it's the best shot I got so far, so I guess I'll go with it until something better pops up. Thanks for all your time and help! :)

Glad to help :)

FYI, one more thing I just found out: Looks like the only thing required to make this work is the cookie, and within the cookie all you need is the user_credentials value.

Nice, that simplifies things quite a bit. user_credentials cookie seems to have a Max-Age of 3 months btw.

Not perfect, but at least better than once a day or something lol


exec: "git": executable file not found in %PATH%

Description
No response
Gitea Version
1.19.1
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
Git-2.40.0-64-bit
Operating System
No response
How are you running Gitea?
无法识别 'git' 命令：exec: "git": executable file not found in %PATH%

Database
SQLite

Can you run "git" in your "cmd" window?
Windows has two PATHs: one is user setting, one is system setting.
If you can't figure out the PATH environment, you can set the git path in app.ini manually.

https://github.com/go-gitea/gitea/blob/35e562d7bd5e984a5cb97f74f5753d2a9998c1d3/custom/conf/app.example.ini#L681-L686

what user is gitea being executed as? does that used have git in its %PATH% variable

Not resolved, but copied data from other computers to run.

Could you try setting [git] PATH="" to the exact path to the git binary?
docs here: https://docs.gitea.io/en-us/config-cheat-sheet/#git-git`

I am pretty sure it's a config problem.
It depends how the author runs Gitea (as current user or as service), and Windows has 2 PATH settings: one for current user, one for global (all users).
I guess the author has succeeded running Gitea on another server, instead of changing the global PATH or changing the [git].PATH in app.ini.

After the prompt, the 'custom/conf/app. ini' configuration file was not generated.

Yes, because there are errors. You need to write the config to your app.ini manually if your system default values do not work.

It's a bit strange that I generate data on my own computer, copy it to the server, and it runs normally without having to set [git] in the app.ini file .

No git path


Some suggestions

We are in the team development, because the front of the staff on the interface, often to find a long time to find the corresponding interface, I hope the author can provide a search function.
A markdown document describes the multiple interfaces, hoping to join the anchor can quickly jump.
I hope daux.io can better!
Thank you very much!

Thanks for your suggestions, search is one thing we would want to include; follow #18 for more informations


MissingPluginException(No implementation found for method authenticate on channel flutter_web_auth_2)[Bug]:

Description
Hi I get this error when calling the authenticate method is a flutter web app hosted on firebase.
"MissingPluginException(No implementation found for method authenticate on channel flutter_web_auth_2)"
This works fine when I run the app locally on my laptop but not when it is hosted. The url i pass to the FlutterWebAuth2.authenticate works correctly when I manually paste into a browser.
Any help much appreciated.
Minimal Reproduction
final result = await FlutterWebAuth2.authenticate(
url: updatedUrl, callbackUrlScheme: "my-app");
This is in a flutter web app hosted on firebase. The url works correctly when manually pasted into a browser and also works correctly when run locally.
Exception or Error
MissingPluginException(No implementation found for method authenticate on channel flutter_web_auth_2)
Expected Behaviour
I expect the authenticate method to open a google oauth2 consent url
Screenshots
No response
Additional context
No response
Device
desktop
OS
mac OS 14.2.1 (23C71)
Browser
chrome and safari
Flutter version
3.19.0
flutter_web_auth_2 version
3.1.1
Checklist

[X] I have read and followed the entire troubleshooting guide and it has not provided the solution I need.
[X] I have provided all the information I can.

Could you maybe try the latest 4.x beta version? If that also does not work, I will investigate a bit more

Thanks for your response - I managed to fix this by doing a flutter clean and rebuild of the app and deploying again on v3.1.1 Apologies - I should have done this before raising the issue. Many Thanks

No problem, and thank you for reporting back! :)


Transform literature list into markdown and expand it - #20

please review my work

You can request a review by the whole group. If you scroll down in the Reviewer List you'll find htw-pv3.
Save your file as .md and not .txt! That way github will parse it into a nice looking html site. After that you are good to go

The advantage of the markdown-file is good to know. I already wondered why(?)

@PhuongThaoBui I am planing to work over your suggestion. Need time...

The file was original a txt-file. I don't want to delete the file and write completely over, so keep it. Ideally the file should transform into a .md-file. @FaidoEwald : You can delete the file, if it is necessary to transform into .md-file
Some points should be worked on, are
[checkbox:unchecked] transform into .md-file
[checkbox:unchecked] expand the literature about sonnja and the PV-literature


A quicklook tool, with interactive targetpixel image registered to selected position in lightcurve

We propose a long-awaited feature: the ability to inspect the target-pixel file (TPF) at a cadence-of-interest, identified interactively from inspection of the associated lightcurve.
The motivation for this feature arises in many scenarios: "Is this blip I see in my lightcurve astrophysical or some instrumental noise source?" may be the most common.  Inspecting the cause of centroid shifts may be another niche, but powerful application-- did a supernova go off in the outskirts of your galaxy's target pixel postage stamp?  If so, you may witness a spatially segregated temporal rise in flux that pulls the centroid location away from the galaxy center and towards the supernova-- a sure sign that you have discovered a genuine transient.  A similar centroid-offset phenomenon can distinguish unresolved background eclipsing binaries from singleton transiting-planet-host stars.  Finally, the effortless feedback of pixel data instills an intuitive feel for the connection of lightcurve and pixels, perhaps even more-so than the existing k2flix gif-animations, that are not seemlessly registered to their associated lightcurves.
The successful design of this feature hinges upon the TPF image being displayed effortlessly and instantly.  That is: latency is near-zero---much less than human reaction time--- and no typing or clicking-through is needed.  We also demand interactivity:  a user may wish to zoom in on a region of interest while maintaining the linked, high-performance display.
These user experience objectives drove us to select a design that adopts the javascript framework bokeh.  We investigated a matplotlib design, but we observed much higher latency, with less customization out-of-the-box.
The baseline design is simple.  The setting is a Jupyter notebook.  The user executes:
tpf.quick_look()

which yields a lightcurve plot that is wider than it is tall, adjacent to a nominally square image window for the TPF.  A slider for cadence sits below the lightcurve window.  The slider instantly updates a vertical bar on the lightcurve window, and updates the pixel image in the TPF window.
The design offers many avenues for additional functionality and custom behavior.
First, the baseline design automatically computes the default (raw, simple aperture photometery [SAP]) flux using a bare to_lightcurve() method.  This SAP lightcurve may be adequate for some purposes, but likely the y-range will be overly large, and astrophysical signals of interest may be lost beneath the large motion-induced noise.  Instead, the user may instruct the use of a pre-processed lightcurve that has already dropped outliers and/or applied motion-detrending and custom background subtraction:
lc_raw = tpf.to_lightcurve()
lc_cln = my_custom_cleaning_function(lc_raw)
# lc_cln may have fewer elements than lc_raw!
tpf.quick_look(lc=lc_cln)

In this case, the user will witness a nice, cleaned lightcurve with the raw pixel-level data on its right.  Sliding the toolbar over missing cadences may simply jump over those cadences in the TPF, or show them anyway.  We haven't decided what the default behavior should be, but probably the former?

⬆ a prototype of the quicklook tool.
A few recommended upgrades:

[ ] All the normal stylistic features-- axes labels, titles, distinctive fonts and colors, etc.
[ ] A longer slider bar, ideally the same with as lightcurve window.  The steps jump by more than 1, so a given pixel cannot be "dialed into".

Bonus goal:

[ ] Option for interactively updating the Screen Stretch on the TPF image.

I'd like to suggest tooltips as an important extra feature.  The tooltip should provide the user access to the cadence number, the index number, the exact time value, the exact flux value, and the quality flags raised for the cadence.

Updates from a discussion with Geert:

add tooltips!
use [1%, 95%] log stretch by default
try aperture mask clicking feature
see if we can just have [Play/Pause] [Previous Frame] [Next Frame] buttons

Now with screen stretch slider on log scale, and support for single increment steps using left arrow ◀ and right arrow ▶ keys.

Now with hover-over tool-tips that indicate quality flags, flux, cadence number, time.


Update quoted tweet

Follow up for https://github.com/vercel/react-tweet/pull/117

The quoted tweet background color changes with hover
Clicking anywhere in the quoted tweet takes you to that tweet URL in a new tab
All quoted tweets components are now exported by react-tweet
Updated utils and types

Our Vercel projects are not synced to this repo since it was moved so I'll make sure to update them before merging this PR.


Change minimum supported Python version

Changes minimum supported Python version to 3.6.1 (see #64) and updates dependencies.

Codecov Report

Merging #83 into develop will increase coverage by 1.67%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop      #83      +/-   ##
===========================================
+ Coverage    91.31%   92.98%   +1.67%
===========================================
  Files            9        9
  Lines         1312     1312
  Branches       244      244
===========================================
+ Hits          1198     1220      +22
+ Misses          70       48      -22
  Partials        44       44

Impacted Files
Coverage Δ

src/snps/resources.py
94.73% <0.00%> (+0.40%)
:arrow_up:

src/snps/snps.py
93.04% <0.00%> (+1.44%)
:arrow_up:

src/snps/ensembl.py
70.73% <0.00%> (+39.02%)
:arrow_up:

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 13eae77...672f380. Read the comment docs.


"OAuthException: (#12) bio field is deprecated for versions v2.8 and higher"

Reading from facebook changelog for the v2.8 (https://developers.facebook.com/docs/apps/changelog):
User Bios - The bio field on the User object is no longer available. If the bio field was set for a person, the value will now be appended to the about field.

Please open this issue on the league/oauth2-facebook repo. Thanks! /cc @SammyK

@slk500 Make sure you're using the latest version of the package since this was fixed in v1.4.4.


⚠ Bofrost.lu has degraded performance

In 26085e1, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1656 ms

Resolved: Bofrost.lu performance has improved in aea42b5 after 44 minutes.


Release tests for first release to Play Store

Discuss on the features and test for basic release on the Play Store.

https://support.google.com/googleplay/android-developer/answer/113469?hl=en&ref_topic=7072031

release tests are done.


Testing the 'test-app'

Can you provide more information how test the running test application written in Java?

Take a look at the Makefile
replace the test-app-golang folder by the test-app-java at step 06
you can change the app version manually
here
and repush your repo.

Can we see the ruining test app in Kubernetes cluster?
http://helloworld:8080/ Gives ERR_CONNECTION_REFUSED
Services and ingress are created.Pods are deployed as well.

the default port for ingress is 80 (and 443 for SSL) so you can view your app
http://helloworld.local


I have a problem installing hostapd --need help--

I am using kali linux 2.0, i have dell latitude e6410..
I am haing touble to install hostapdfile:///root/Pictures/Screenshot%20from%202018-05-12%2015:54:53.png

pls help

file:///root/Pictures/1.png

/root/Pictures/1.png

root@kali:~/Desktop/roguehostapd-master# python setup.py install
/tmp/tmp_netlink_MR3VY0/test_netlink.c:2:29: fatal error: netlink/netlink.h: No such file or directory
#include <netlink/netlink.h>
^
compilation terminated.
[!] The development package for netlink is missing. Please download it and restart the compilation.If you are on Debian-based system: 'apt-get install libnl-3-dev libnl-genl-3-dev'.
warning: no files found matching '*' under directory 'roguehostapd/'

Hi can you follow the instructions to install libnl?

i tried      apt-get install libnl-3-dev
and        apt-get install libnl-genl-3-dev
and then i tried to install roguehostapd and i get this=
root@kali:~/Desktop/roguehostapd-master# python setup.py install
/tmp/tmp_openssl_2XHNaZ/test_openssl.c:2:25: fatal error: openssl/ssl.h: No such file or directory
#include <openssl/ssl.h>
^
compilation terminated.
[!] The development package for openssl is missing. Please download it and restart the compilation.If you are on Debian-based system: 'apt-get install libssl-dev'.
warning: no files found matching '*' under directory 'roguehostapd/'
then i tried= apt-get install libssl-dev  ..i got this
root@kali:~/Desktop/roguehostapd-master# apt-get install libssl-dev
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
libssl-dev : Depends: libssl1.0.0 (= 1.0.1k-3) but 1.0.1k-3+deb8u1 is to be installed
E: Unable to correct problems, you have held broken packages.
--please help--

i checked
root@kali:~/Desktop/roguehostapd-master# apt-cache policy libssl1.0.0
libssl1.0.0:
Installed: 1.0.1k-3+deb8u1
Candidate: 1.0.1k-3+deb8u1
Version table:
*** 1.0.1k-3+deb8u1 0
100 /var/lib/dpkg/status
1.0.1k-3 0
500 http://old.kali.org/kali/ sana/main i386 Packages
root@kali:~/Desktop/roguehostapd-master# apt-cache policy libssl-dev
libssl-dev:
Installed: (none)
Candidate: 1.0.1k-3
Version table:
1.0.1k-3 0
500 http://old.kali.org/kali/ sana/main i386 Packages
and then i installed
root@kali:~/Desktop/roguehostapd-master# apt-get install libssl1.0.0=1.0.1k-3
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following packages will be DOWNGRADED:
libssl1.0.0
0 upgraded, 0 newly installed, 1 downgraded, 0 to remove and 60 not upgraded.
Need to get 2,120 kB of archives.
After this operation, 8,192 B disk space will be freed.
Do you want to continue? [Y/n] y
WARNING: The following packages cannot be authenticated!
libssl1.0.0
Install these packages without verification? [y/N] y
Get:1 http://old.kali.org/kali/ sana/main libssl1.0.0 i386 1.0.1k-3 [2,120 kB]
Fetched 2,120 kB in 7s (266 kB/s)
Preconfiguring packages ...
dpkg: warning: downgrading libssl1.0.0:i386 from 1.0.1k-3+deb8u1 to 1.0.1k-3
(Reading database ... 325236 files and directories currently installed.)
Preparing to unpack .../libssl1.0.0_1.0.1k-3_i386.deb ...
Unpacking libssl1.0.0:i386 (1.0.1k-3) over (1.0.1k-3+deb8u1) ...
Setting up libssl1.0.0:i386 (1.0.1k-3) ...
Processing triggers for libc-bin (2.19-18) ...
root@kali:~/Desktop/roguehostapd-master#
and
root@kali:~/Desktop/roguehostapd-master# apt-get install libssl-dev=1.0.1k-3
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following NEW packages will be installed:
libssl-dev
0 upgraded, 1 newly installed, 0 to remove and 60 not upgraded.
Need to get 1,249 kB of archives.
After this operation, 5,347 kB of additional disk space will be used.
WARNING: The following packages cannot be authenticated!
libssl-dev
Install these packages without verification? [y/N] y
Get:1 http://old.kali.org/kali/ sana/main libssl-dev i386 1.0.1k-3 [1,249 kB]
Fetched 1,249 kB in 10s (117 kB/s)
Selecting previously unselected package libssl-dev:i386.
(Reading database ... 325236 files and directories currently installed.)
Preparing to unpack .../libssl-dev_1.0.1k-3_i386.deb ...
Unpacking libssl-dev:i386 (1.0.1k-3) ...
Setting up libssl-dev:i386 (1.0.1k-3) ...
root@kali:~/Desktop/roguehostapd-master# python setup.py install
warning: no files found matching '*' under directory 'roguehostapd/'
root@kali:~/Desktop/roguehostapd-master#
root@kali:~/Desktop/roguehostapd-master# apt-get install libssl-dev
Reading package lists... Done
Building dependency tree
Reading state information... Done
libssl-dev is already the newest version.
0 upgraded, 0 newly installed, 0 to remove and 60 not upgraded.
root@kali:~/Desktop/roguehostapd-master# python setup.py install
warning: no files found matching '*' under directory 'roguehostapd/'
root@kali:~/Desktop/roguehostapd-master#
--Still cant install roguehostapd--
--have i done something wrong ?. Please help--

brother you question sloution?