Noisy Logging

https://github.com/revel/revel/blob/ee237a4ce4b7660dcbafda789dd6f524a4a4b47d/revel.go#L124
What is this logging used for?
Thanks

It also creates and logs to a file called none all the time.

It also creates and logs to a file called none all the time.

The same thing happens to me.


update URLSuffix and CloudWatch URL to support China regions

Issue #, if available:
Description of changes:

Change "amazonaws.com" to "{AWS::URLSuffix}" in Outputs to make it available in Beijing/Ningxia region (different API Gateway endpoint)
Add if/else conditions in 4 python files to make it available in Beijing/Ningxia region (different CloudWatch URL)

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

Thanks for the contribution. "{AWS::URLSuffix}" is a good call. We have added this to our backlog and will evaluate it for future releases.

Since there is a forked version this solution available in China regions, we will not add item 2 in scope. We will change "amazonaws.com" to "{AWS::URLSuffix} in the next release.

This change has been included in v4.0.0.


Grouped ToC Docs CI script

Loop over ci-config.json files and run build script. If the working directories of the Docs CI scripts are sufficiently isolated this may be done in parallel to save time.

Part of grouped ToC work. Not necessary anymore.


enhance: Flexible url parameters for RestEndpoint

Motivation

Flexible parameters can even be useful for typical rest patterns. For instance, not requiring anything for a list, or simply having the id itself as a single parameter for a detail endpoint.

Backed out of the getFetchKey change....this would potentially break existing pieces. Best to leave legacy as is and only use new things in new contexts.


ACTION: unbound variable

yaml:
 name: "Helm checks"
        uses<EMAIL_ADDRESS>        with:
          action: "check"
        env:
          ARTIFACTORY_URL: <>
          ARTIFACTORY_USERNAME: ${{ secrets.ARTIFACTORY_USERNAME }}
          ARTIFACTORY_PASSWORD: ${{ secrets.ARTIFACTORY_PASSWORD }}

Running the build results in this error:
home/runner/work/_actions/mbovo/action-helm-artifactory/v1.0.0/main.sh: line 15: ACTION: unbound variable

I'm also stuck with this build error.

The issue stems from this: https://github.com/mbovo/action-helm-artifactory/blob/main/main.sh#L15 and set -o nounset. You can get around it by adding an environment variable for ACTION that reflects the action you want, for example:
env:
  ACTION: push

I used that and was able to get past the error. There is however one warning that remains:
2022-07-06T00:33:02.9661791Z ##[warning]Unexpected input(s) 'action', valid inputs are ['']

Hi @victor10x
You can get past the warning by excluding the "with" block altogether since you're probably using the env/ACTION value. Thanks


Revoke Trust

Closes https://github.com/yunity/karrot-frontend/issues/2341
What does this PR do?
Trust can now be revoked.

Links to related issues
PR on the backend side
Checklist

[ ] added a test, or explain why one is not needed/possible...
[x] no unrelated changes
[x] asked someone for a code review
[x] joined chat.foodsaving.world/channel/karrot-dev
[x] added an entry to CHANGELOG.md (description, pull request link, username(s))
[x] tried out on a mobile device (does everything work as expected on a smaller screen?)

@nicksellen I have not been able to write a working test for the dialog. I don't see much benefit in writing tests for the other lines that are marked as uncovered. What do you think? Merge anyway?

Yup, I agree @pogopaule.

Yup, I agree @pogopaule.

@nicksellen Cool! Could you approve the PR?

I just rebased on top of current master.

Hey, thanks for rebasing and merging! I have been off the radar for a while, but my plan is to work on karrot again in the weeks to come. :)

Hey, thanks for rebasing and merging! I have been off the radar for a while, but my plan is to work on karrot again in the weeks to come. :)

We'll have you any time :+1:  Whenever your rhythm of life comes into sync with Karrot again :heart:


'Name already exists' and I/O error on ACD

When filing an issue, please include the following information if
possible as well as a description of the problem.  Make sure you are
using the latest version of rclone.

What is your rclone version (eg output from rclone -V)
v1.33

Which OS you are using and how many bits (eg Windows 7, 64 bit)
Ubuntu 16 64bit

Which cloud storage system are you using? (eg Google Drive)
Amazon

The command you were trying to run (eg rclone copy /tmp remote:tmp)
sync

A log from the command with the -v flag (eg output from rclone -v copy /tmp remote:tmp)
rclone-log-0921.txt

I am getting these for a few days now.  Remote deletions fail due to I/O error.  It started with a single file with extension JPG in all caps.  Changing it to lower case solved it for a couple of days.
Now it seems to have nothing to do with that, or it may have since I have not manually inspected the directories.  I know I changed the extensions to lower case for some files, but that mission is not complete yet because Windows does not do case sensitive search, and the Ubuntu search does not work at all.
TIA

I used the Batch Rename Utility to change all JPG extensions to jpg.  I cannot delete them from ACD because it does not do case sensitive search.  Is there a command that will make rclone delete all files with .JPG extension in a case sensitive manner?

Looking at your log, it looks like those .JPG files are causing the problem.
If you do
rclone ls --include "*.JPG" remote:path

rclone will show you the JPG files.  rclone is always case sensitive.
If happy with the list you can then do
rclone delete --include "*.JPG" remote:path

Thank you!  The sync tonight should replace all the deleted files.  I'll confirm after checking the log in the morning.

Clean log. Deletions completed. Thank you!


Has anybody gotten webpack-dashboard working with this boilerplate?

So I know this repo hasn't been maintained by the original owner in quite some time, but there still seem to be people kicking around.
I've got my own production version of this boilerplate that took an opinionated departure awhile back, but still maintains core similarity. I've been interested in getting webpack-dashboard set up for awhile to improve dev experience for myself and my team.
However, I've tried a couple times, each of which resulted in like 30+ minutes of work and me deciding it was prohibitive. Just wondering if anyone's had more luck?

@isaachinman My plan was to follow the webpack setup to follow the one used by react-boilerplate (https://github.com/mxstbr/react-boilerplate/pull/886/files) but I haven't gotten to it yet.
Also, would be curious if anyone has already integrated webpack-dashboard with this repo!


Vickel: Watch michaeltwelch01311965(?:@gmail.com)?

Vickel requests the watch of the watch_keyword michaeltwelch01311965(?:@gmail\.com)?. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

michaeltwelch01311965(?:@gmail\.com)? has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


Add go mod support and update travis

Coverage decreased (-1.6%) to 72.897% when pulling 47d87e9d9fd584f3f074c4e6ae18a09b404e77f9 on go1.13 into 1932c8dd22f2283687586008bf2d58c2c5c014d0 on master.


Blocks YouTube on videogameschronicle.com

e.g. https://www.videogameschronicle.com/features/2021-breath-of-the-wild-2

They won't show videos without accepting cookies, so it's a choice between blocking the notice or the videos. If you disable hush and accept cookies it should work.


Adding orin thomas + updating index/map/toc

Docs Build status updates of commit 7ad933b:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

advocates/orin-thomas.yml
:x:Error

Details

advocates/index.html.yml
:warning:Warning

Details

advocates/map.yml
:warning:Warning

Details

advocates/toc.yml
:warning:Warning

Details

advocates/orin-thomas.yml

Line 1, Column 1: [Error-schema-not-found] Unknown schema ''.

advocates/index.html.yml

Line 10, Column 8: [Warning-file-not-found] Invalid file link: './map'.
Line 12, Column 8: [Warning-file-not-found] Invalid file link: './tweets'.
Line 680, Column 8: [Warning-xref-not-found] Cross reference not found: 'advocates.orin-thomas'.

advocates/map.yml

Line 676, Column 8: [Warning-xref-not-found] Cross reference not found: 'advocates.orin-thomas'.

advocates/toc.yml

Line 141, Column 10: [Warning-xref-not-found] Cross reference not found: 'advocates.orin-thomas'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 7ad933b:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

advocates/orin-thomas.yml
:x:Error

Details

advocates/index.html.yml
:warning:Warning

Details

advocates/map.yml
:warning:Warning

Details

advocates/toc.yml
:warning:Warning

Details

advocates/orin-thomas.yml

Line 1, Column 1: [Error-schema-not-found] Unknown schema ''.

advocates/index.html.yml

Line 10, Column 8: [Warning-file-not-found] Invalid file link: './map'.
Line 12, Column 8: [Warning-file-not-found] Invalid file link: './tweets'.
Line 680, Column 8: [Warning-xref-not-found] Cross reference not found: 'advocates.orin-thomas'.

advocates/map.yml

Line 676, Column 8: [Warning-xref-not-found] Cross reference not found: 'advocates.orin-thomas'.

advocates/toc.yml

Line 141, Column 10: [Warning-xref-not-found] Cross reference not found: 'advocates.orin-thomas'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 7873f33:
:warning: Validation status: warnings

File
Status
Preview URL
Details

advocates/index.html.yml
:warning:Warning
View
Details

advocates/map.yml
:white_check_mark:Succeeded
View

advocates/media/profiles/orin-thomas.png
:white_check_mark:Succeeded
View

advocates/orin-thomas.yml
:white_check_mark:Succeeded
View

advocates/toc.yml
:white_check_mark:Succeeded
View

advocates/index.html.yml

Line 10, Column 8: [Warning-file-not-found] Invalid file link: './map'.
Line 12, Column 8: [Warning-file-not-found] Invalid file link: './tweets'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 7873f33:
:warning: Validation status: warnings

File
Status
Preview URL
Details

advocates/index.html.yml
:warning:Warning
View
Details

advocates/map.yml
:white_check_mark:Succeeded
View

advocates/media/profiles/orin-thomas.png
:white_check_mark:Succeeded
View

advocates/orin-thomas.yml
:white_check_mark:Succeeded
View

advocates/toc.yml
:white_check_mark:Succeeded
View

advocates/index.html.yml

Line 10, Column 8: [Warning-file-not-found] Invalid file link: './map'.
Line 12, Column 8: [Warning-file-not-found] Invalid file link: './tweets'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Not running on Apple M1

Hello! I am having troubles to run movingpandas on MacOs Big Sur (v. 11.0) with Apple M1 chip.
First I successfully create an environment:
conda create -n new-env python=3.8
Then, I activate it:
conda activate new-env
Next, the movingpandas are installed without errors:
conda install -c conda-forge movingpandas
However, when I run:
python3 -c "import movingpandas"
the result is:
Traceback (most recent call last):   File "<string>", line 1, in <module>   File "/home/miniconda3/envs/new-env/lib/python3.8/site-packages/movingpandas/__init__.py", line 5, in <module>     from .trajectory import Trajectory   File "/home/miniconda3/envs/new-env/lib/python3.8/site-packages/movingpandas/trajectory.py", line 10, in <module>     from geopandas import GeoDataFrame   File "/home/miniconda3/envs/new-env/lib/python3.8/site-packages/geopandas/__init__.py", line 7, in <module>     from geopandas.io.file import _read_file as read_file  # noqa   File "/home/miniconda3/envs/new-env/lib/python3.8/site-packages/geopandas/io/file.py", line 7, in <module>     import fiona   File "/home/miniconda3/envs/new-env/lib/python3.8/site-packages/fiona/__init__.py", line 84, in <module>     from fiona.collection import BytesCollection, Collection   File "/home/miniconda3/envs/new-env/lib/python3.8/site-packages/fiona/collection.py", line 9, in <module>     from fiona.ogrext import Iterator, ItemsIterator, KeysIterator ImportError: dlopen(/home/miniconda3/envs/new-env/lib/python3.8/site-packages/fiona/ogrext.cpython-38-darwin.so, 2):  Library not loaded: @rpath/libtbb.dylib Referenced from: /home/miniconda3/envs/new-env/lib/libtiledb.dylib   Reason: image not found

Looks like an upstream issue with GeoPandas / Fiona. Can you install and use these libraries without MovingPandas?

Looks like an upstream issue with GeoPandas / Fiona. Can you install and use these libraries without MovingPandas?

No, I cannot. Thank you for the response.

No, I cannot. Thank you for the response.


Upgraded to FTC SDK version 9.0.1 and added Encoder, Gyro and AprilTag samples.

This is duplicate, so needs to be closed.


Fix code markup in the readme

Merging it in, will monitor unexpected failures of terratest pipe in master ( as it is definitely not related to readme :) )


Package request: Bruno Golden Edition (MIT + Unfree)

Project description
Same as https://github.com/NixOS/nixpkgs/issues/258765 but packaged from the binary release which includes the proprietary code.
From https://www.usebruno.com/pricing:

The source code for Golden Edition features is proprietary.
Perpetual Fallback License that can be used on 2 devices, with 2 years of updates

The binary releases on GitHub do include the proprietary features.
I was able to repackage the x64 .deb version for myself.  I haven't verified the golden features yet other than confirming the option is now there to register on both Ubuntu and NixOS.  This option is not there when packaged from source.  When I get my license from my employer I plan to test it.  I mostly used react-native-debugger as a template.
https://github.com/higherorderfunctor/nixos-config/blob/dotfiles/overlays/bruno-overlay.nix
https://github.com/higherorderfunctor/nixos-config/blob/df336d268b0b5cb6b831e20cac516ad0b8da3e57/overlays/.nvfetcher/generated.json#L22
Metadata

homepage URL:
source URL: https://github.com/usebruno/bruno
license: MIT + Unfree
platforms: being an electron app, everything that's supported by electron

Add a :+1: reaction to issues you find important.

I have tested the golden features with my overlay and they work.


NOISSUE - Add property based testing to users service

What type of PR is this?
This is a feature PR as it adds property-based testing to users API. In addition it is a refactor to error handling on the API

What does this do?
Adds schemathesis, property-based testing to users service

Which issue(s) does this PR fix/relate to?

No issue
Have you included tests for your changes?
Yes, I have included tests for my changes.

Did you document any new/modified feature?
No

Notes

I got below warnings from schemathesis in local and in CI,
Is it possible to provision Magistrala with domain and groups ? , so that schemathesis could test all the cases.
WARNINGS:
  - Most of the responses from `GET /groups/{groupID}` have a 403 status code. Did you specify proper API credentials?
  - Most of the responses from `PUT /groups/{groupID}` have a 403 status code. Did you specify proper API credentials?
  - Most of the responses from `DELETE /groups/{groupID}` have a 403 status code. Did you specify proper API credentials?
  - Most of the responses from `POST /groups/{groupID}/enable` have a 403 status code. Did you specify proper API credentials?
  - Most of the responses from `POST /groups/{groupID}/disable` have a 403 status code. Did you specify proper API credentials?

Is it possible to provision Magistrala with domain and groups ? , so that schemathesis could test all the cases.

This is completely fine. Most of the endpoints schemathesis is probing are not valid hence we get an authorization error because the id is not for a valid entity


Added code coverage.

Changes Unknown when pulling d1569c0eb1539d437175563ecead4b3becaa219b on dblock:coverage into ** on alexa-js:master**.

Changes Unknown when pulling 6b3587b895437351f3aa26d4670e0ae7efc55153 on dblock:coverage into ** on alexa-js:master**.


Revert & improve wrapper creation

Goes with https://github.com/dropoutlabs/PySyft/pull/15

just that one suggestion, LGTM beyond that!


Provide capability to configure HTTP timeouts

Provide capability to configure HTTP timeouts by configuring KEDA_HTTP_DEFAULT_TIMEOUT environment variable. (see https://github.com/kedacore/keda/pull/1251#issuecomment-739836264)
This is a follow-up for https://github.com/kedacore/keda/issues/1133

@tomkerkhove I think this is done?

Yes you did this through - https://github.com/kedacore/charts/pull/90


[FEATURE] Pretraining models

Hi! Thanks for this great work.
are there pretraining code scripts and the corresponding hyperparameters used for pretrained models?
some pretrained models have some hints such as input size, augmentation, and dataset trained on.
but what I wanted is the codes for pretraining and reproduce these model weights, and possibly train also on my own data.

Most of the models you are able to access here are not trained using timm. The best way to find the hyperparameters is to look at the model file, which then contains information such as URL to the original paper that introduced the model. Note, however, that it is not uncommon for other (more recent) papers to use the timm codebase, which makes adaptation very simple. Further, you should be able to find good training hyperparameters for a relatively large number of models if you browse, e.g., the discussion page or look at @rwightman's gists.

@opencomvis moving to discussion, as Torben mentions you can find quite a few different recipes (hparams) in past issues and discussions here... there are templates for good results, several recipe templates were provided in ResNet Strikes Back (https://arxiv.org/abs/2110.00476) and there are discussions here which translate them to specific args.
Unfortunately every model, pretrain vs finetune, dataset combo varies so it's not possible to provide hparams that work best in all situations, you always need to do some amount of searching so having templates is generally more useful.


Refactor(sideeffect): Remove  attribute for cleaner state management

This pull request refactors the SideEffect class by removing the unnecessary _side_effect_on_action attribute. This change simplifies the internal state management of the SideEffect class, making the codebase cleaner and easier to maintain.
Changes

Removed the _side_effect_on_action attribute from the SideEffect class.
Updated the relevant logic to ensure smooth functioning without the _side_effect_on_action attribute.

Refactored the SideEffect class to remove the unnecessary _side_effect_on_action attribute, simplifying the internal state management. Great work!


initialRegion not correctly handled.

React Native Maps 0.16.3, React Native 0.47, Google maps on Android and iOS.
Unfortunately, I'm finding that the initialRegion prop of MapView is still ignored in both android and iOS with Google maps.
On Android, I get a whole-world view, with latitude and longitude of 0. On iOS, the same latitude and longitude, but zoomed right in.
I have verified that my provided initialRegion is valid.
Appreciate everyone's work.

This is likely the same as discussed in #1559 - but I'm finding it on iOS as well. Using region over initialRegion works.

Looking into this a bit more - logging out initialRegion from MapView._onReady and MapView._onLayout, and event.nativeEvent.region from MapView._onChange.
I'm not very familiar with the guts of react-native-maps, so if anyone can chime in and give pointers, that'd be helpful.
Google Maps on iOS:
On first load:
MapView._onReady logs a correct initialRegion
MapView._onLayout does not appear to fire.
--> initialRegion is honoured and the map displays correctly.
When the map component is unmounted then re-mounted:
MapView._onLayout is fired with the correct initialRegion.
MapView._onChange fires followed by and logs a modified region which includes longitude of zero and latitude of essentially zero.
MapView._onReady fires and logs a modified initialRegion which is identical to the region of onChange.
--> initialRegion is not honoured, instead the map shows the blue screen of just sea at this point of the earth.
Google Maps on android:
On first load:
MapView._onReady logs a correct initialRegion
MapView._onLayout does not appear to fire
MapView._onChange logs a region with latitude and longitude of zero
--> The map reflects that initialRegion has been ignored.
In my implementation, I store the region in redux after user map moves. If I move the map, all is fine and my stored values reflect the map.
When the map component is unmounted then re-mounted with the updated, post-move initialRegion out of redux:
MapView._onReady is fired with the correct initialRegion
MapView._onLayout is fired with the correct initialRegion
MapView._onChange fires and logs a modified region which includes longitude of zero and latitude of essentially zero.
(Note _onReady fires before _onLayout, the reverse of iOS)
--> Again, the initialRegion prop is ignored as above and the map reverts back to the world view centred on latitude, longitude of zero.
What is happening to change the map region internally as the map is mounted?

Update: I can partially fix the issue in Google Maps on iOS by commenting out two lines in ios/AirGoogleMaps/AirGoogleMap.m:
- (void)setInitialRegion:(MKCoordinateRegion)initialRegion {
  // if (_initialRegionSet) return;
  // _initialRegionSet = true;
  self.camera = [AIRGoogleMap makeGMSCameraPositionFromMap:self andMKCoordinateRegion:initialRegion];
}

partially, because removing this check ensures the initialRegion is set and the map displays correctly, but then the user can no longer move the map.

I'm guessing that _initialRegionSet is preserved as true between unmounting and remounting a component, preventing update of the camera position.
In android, the equivalent is in AirMapView.java - but commenting out the checks has no effect:
  public void setInitialRegion(ReadableMap initialRegion) {
    if (!initialRegionSet && initialRegion != null) {
      setRegion(initialRegion);
      initialRegionSet = true;
    }
  }

If there's someone else who can continue on from here, that'd be great.
As a temporary workaround, I have used the onMapReady callback of MapView, and call animateToRegion to display the initialRegion. This means I temporarily get the erroneous display, then the region I want.

@heysailor did you do a clean build after upgrading the library to 0.16.3? There were some native changes included that won't take effect unless the native modules are rebuilt.
I was seeing the same issues you described but I no longer see them on 0.16.3

NOTE:
This change was ultimately reverted because the correct fix had been merged but not yet released.

I've just tested 16.3 and it seems #1577 still persists

Same issue here, RN 0.46.4 & maps 0.16.4

Same Issue here: RN 0.48.4 and maps 0.16.4

Same on (my fork)[https://github.com/aaronbuchanan/react-native-maps]

Same issue here: RN 0.48.4, expo 21.0.3, maps 0.15.4 and 0.17.1
As soon as I start my app, most of the time map shows Lat/Lng 0,0. Sometimes, it shows correct initialRegion. Can't tell when it shows correct region and wrong region, (0,0).
I was using react-native-maps in Expo, which was 0.15.4.
So I installed recent maps, and tried to use that but same happens.
(before: import {MapView} from 'expo', now: import MapView from 'react-native-maps')
Since my app's initial loading time is rather long, users have to see the wrong region more than ten seconds.

I succeeded avoiding this issue by deferring heavy load jobs to start 1 second after MapScreen component is mounted.
Here is my guess:
It seems that map is rendered at Lat/Lng 0,0 at first time and quickly change it to initialRegion. In normal case, it is quick enough that we cannot see unwanted region. However, if there are heavy initial tasks from the beginning, the job to change map's region to initialRegion is delayed, which makes us to see unwanted region for a while.
My MapScreen component uses MapView and its componentDidMount() calls an action creator this.props.reload(), which invoke an API to get huge data. I just deferred this one second after cDM like that.
componentDidMount() {         setTimeout( () => {             this.props.reload();         }, 1000)     }
Now, I cannot see unwanted region (0,0) anymore. According to the log, map's initial region is (0,0) but seems to change its region very quickly to my initialRegion.

I have a little bit different problem. On first component mount, initial region is applied. On second, it is not applied. Anyone experiences the same?

That's fixed already, but let us now if not.

We are still facing this issue.
Using:
react-native: 0.54.4
react-native-maps: 0.21.0
react-native-maps-super-cluster: 1.4.0
We are user react-native-maps-super-cluster, so maybe it's a problem with that, but it seems odd that it fits the errors above perfectly.
On Android everything is fine, but on ios it seems like, it is zooming to the current region first and then just stops there, ignoring the intiialregion all together.
I've logged and checked everything to make sure the correct region is used for initialRegion.
We also tried the componentDidMount( requestAnimationFrame, trick, but that didn't work either.
Right now we block the first 3 seconds of regionChanges and have a timeout of 3 seconds after componentDidMount to do the animation to the correct location.
That works, but it doesn't look good at all, because the map keeps jumping around quite a bit.
Like I said: on android everything works perfectly fine.

I'm experiencing the same issue. I'm logging my initialRegion and it has the correct values on first render / when mounting. But it still renders the world map.
I'm using the AnimatedMapView so maybe it's related to that.

hi
same issue

I added an initial state of "{}" empty object instead of null , Since is was useState to set Latitude and Longitude using setState

I know this has been closed but If anyone is still struggling with this issue as I was, I managed to fix it.
In my case I was using animateCamera to rotate the map the match the users heading as they moved. I think in my situation there was a race condition where the compass state would change which would fire the animateCamera function before the Map was fully mounted. This would stop the re-render of the map from its initialisation location of lat: 0 and long:0 leaving the user looking at the middle of the Atlantic Ocean.
I fixed this race condition by adding a ref to the mapView, created an animateMap local state (which I will use in the settings page so the user can turn off the rotation on the map if they want) and then added the following use effect which is fired when the mapView renders:
useEffect(() => {
if(theMap){
setAnimateMap(true);
}
}, [theMap]);
In essence, the initial state of animateMap is set to false on initialisation. The useEffect is triggered when anything changes with the theMap ref and the state of animateMap is updated to true if the theMap ref exists (it has mounted). The following code then handles the changes from compassHeading and only animates the map if the animateMap state is true.
Hope that helps as I struggles to find a lot of help for this one.


Build comments

Requires:
<wp:comment>
	<wp:comment_id>5</wp:comment_id>
	<wp:comment_author><![CDATA[coolwebsitedotcom]]></wp:comment_author>
	<wp:comment_author_email>willspiro@gmail.com</wp:comment_author_email>
	<wp:comment_date>2018-07-02 17:49:32</wp:comment_date>
	<wp:comment_content><![CDATA[This is a cool post...]]></wp:comment_content>
	<wp:comment_approved>1</wp:comment_approved>
</wp:comment>

This is an issue because blog template often do not expose the commenters email address, and we need a <wp:comment_author_email> value. Maybe set a default value here?

https://github.com/williamspiro/hubXml/pull/20


create spans directly using protobuf structures

otel-cli was always a problem child user of opentelemetry-go. From the very start the otelclicarrier.go file was working around intentional limitations in the SDK. This set of changes moves otel-cli to directly create spans using the protocol buffer structures from OTLP, and then sends them along using the opentelemetry-collector backend.
It doesn't look like this reduced dependencies significantly, which is expected since the OTel SDK is mostly defensive coding to make API use consistent.
Somewhat inspired by #173, which may be doable through the SDK, but this makes features like that request a breeze.

Unit tests are clean. Functional tests aren't passing yet. Will work on that more tomorrow.


sum() returns an array

What happens?
I loaded some data in a notebook and ran sum() over a column. It returned an array instead of a number.
See notebook here: https://observablehq.com/d/8821f2b8f94550a2
And see the screenshot attached.

To Reproduce
You can fork the observable notebook here:
https://observablehq.com/d/8821f2b8f94550a2
Browser/Environment:
Version 120.0.6099.234 (Official Build) (arm64)
Device:
Macbook Pro
DuckDB-Wasm Version:
Not sure of the version, it's within Observable Notebooks
DuckDB-Wasm Deployment:
Not sure of the version, it's within Observable Notebooks
Full Name:
Nael Shiab
Affiliation:
CBC/Radio-Canada

It's maybe a decimal value represented as arrow which needs some work (https://github.com/apache/arrow/issues/28804). Try casting to int or float.

Ah! It works! Thanks, @domoritz! But just for my understanding, is this a bug or something I should be aware of?

Decimal handling needs to be improved in Arrow but there are also issues with how DuckDB was maps types to arrow types.
This particular issue is a bit tricky since there is no decimal type in JavaScript. One could argue that you should do the decoding yourself here but I think Arrow could be more ergonomic and return e.g. a float.
We are tracking decimal issues in Arrow (see https://github.com/apache/arrow/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc+decimal+label%3A"Component%3A+JavaScript") and it's second on my mental queue (after addressing some bundling issues). I think the issue can be closed here until we improve Arrow.

Alright! Thanks a lot for the explanations, @domoritz . :)


Make state immutable in reducers/effects

Call Object.freeze on state when passing it into to the reducers or effects
so that state cannot be modified except when being returned from an emitter.
Add tests to prove state is immutable in these scenarios.

So! The current tests run in node which means you can't call send.
So I made a test file that you can browserify and run in the client.
However, setting up automated testing for browser tests is super opinionated so I didn't set up a way to actually, you know, run the tests.
We use karma for scripto for client testing (and xvfb for doing it headless on CI).  However, karma is very heavy and requires a TON of modules and plugins.  but it also is extremely reliable, and gets you coverage reports easily.  But, since it's also XVFB for automated testing, I still can't get it set up correctly on travis-ci (but can do it on circle-ci).
I've also done testling, which is super minimal, but doesn't get you coverage easily and still requires XVFB.
Zuul is great, but you can't run it in an automated fashion without setting up sauce labs support (I have an extremely old branch where I tried to implement local test running but never finished).
So like, um, I have no clue how you want to commit to running these tests, but like this will open it and run it...
echo '<title>test</title><body><script src="bundle.js"></script>' > index.html
../node_modules/bin/browserify browser.js > bundle.js
open index.html

(Or xdg-open if you're on linux. I have no clue how to do this on windows lol)

Ooh, love this patch - minor version I reckon. Just needs tests fixed and we can merge.
Browser tests are def tricky - perhaps just add these tests for now, but exclude them from CI? - that way Travis stays green and we can save ourselves some work in the future.
I've done a bit of work for tests / coverage here - https://github.com/yoshuawuyts/choo/pull/24. It uses smokestack which can integrate with saucelabs; I reckon using sauce might be the easiest?

Another alternative to consider might be tape-run. Basically runs your code through a headless electron instance. No integration with saucelabs ofc though, so v8 only. I run it like this:
browserify --debug test/index.js | tape-run --render="tap-spec"

Oooooh I hadn't seen tape-run and it's instructions for using electron! That would greatly simplify some other only-chrome stuff in my life :).
Interestingly, you can't easily mark a test as "skip" when you run tape tests through the node executable. Adding {skip: true} at either level makes node simply exit after running browser test (since it comes first in the glob), and even wrapping it in an if statement causes node to exit after processing the file.
I renamed it so it comes later in the glob pattern (and since node will exit after the first file it will never execute), but this feels really hacky?
Better ideas or solutions?
Also added some documentation to explain that state is frozen within a reducer or an effect just to be explicit about it.

So I tried to use Zuul since then it can run in sauce labs for free using "open sauce" which means you can target a ton of browsers, but Zuul uses an older version of browserify-istanbul, which relies on an older version of esprima which is unable to parse template tag syntax.
I opened a PR to update to the latest version of browserify-istanbul, but that's a dead-end until then defunctzombie/zuul#289
I do have this running under karma in my project, but the list of requirements is pretty heavy (although some sub-deps of choo require karma already so we're installing it as it is).
Here's the minimal list I've found I need to just support tape and istanbul via browserify:
    "browserify": "^13.0.1",
    "browserify-istanbul": "^2.0.0",
    "istanbul": "^0.4.3",
    "karma": "^0.13.22",
    "karma-browserify": "^5.0.5",
    "karma-chrome-launcher": "^1.0.1",
    "karma-coverage": "^1.0.0",
    "karma-script-launcher": "^1.0.0",
    "karma-tap": "^1.0.4",
    "tape": "^4.5.1",

That feels kind of heavy, but it works.  Except for in travis-ci.

This patch is looking great by the way - testing, merging and publishing as minor :sparkles:

Ah yeah, so the problem here is that Object.freeze mutates an existing object, rather than create a new copy - the fact that a single action can trigger both an effect and reducer is problematic - I think the cleanest resolution is to only allow for a single call

@yoshuawuyts @toddself Hey, just a heads up -- I've lost the reference to where I read about this, but I distinctly remember reading that Object.freeze has some pretty bad performance characteristics in some browsers (maybe Safari? not sure).  Would you consider guarding this with a check for process.env.NODE_ENV === 'production', so that it could be easily stripped out of a production build?

the fact that a single action can trigger both an effect and reducer is problematic - I think the cleanest resolution is to only allow for a single call

@yoshuawuyts interesting; could you say a bit more about this?   If I'm understanding you correctly, this would effectively remove "actions" as a concept independent of effects and reducers--is that right?

Object.freeze has some pretty bad performance characteristics in some browsers (maybe Safari? not sure).

Oh yeah, that's a good call. We could def add an argument that can be passed on start - I've found it's generally  not a good idea to have env var switches live within packages.

If I'm understanding you correctly, this would effectively remove "actions" as a concept independent of effects and reducers--is that right?

Ummm, maybe? The way I view actions is as a single trigger on any of the stores - a 1:1 mapping if you will; quite similar to a function call. Having it trigger multiple things at once seems... well confusing haha

Hope this answers your questions!

The way I view actions is as a single trigger on any of the stores - a 1:1 mapping if you will; quite similar to a function call. Having it trigger multiple things at once seems... well confusing haha

Interesting.  In my work, I've been thinking of "actions" as a separate abstraction, basically representing any "verb" within the system (some of which map directly to user actions like selectSomeItem or cancelSomething, others of which are more internal like fetchSomeData or updateBlahStatus).  Coming from that point of view, I feel that it can sometimes make sense for an action to trigger multiple things, because I think there isn't always a 1:1 mapping between "verbs" and stores.  But if there's going to be a 1:1 mapping, then would it be simpler to eliminate actions altogether, in favor of an API like (state, effects, stores) => { ... stores['namespace:reducerName'](args) ... }?

I have it set up similarly but rather use effects to trigger multiple reducers to react to input.
The user might perform some action which causes several parts of the state to update, so i trigger an effect which in turn triggers all the single modification reducers to fire that are necessary.

On Jun 24, 2016, at 05:28, Anand Thakker<EMAIL_ADDRESS>wrote:
The way I view actions is as a single trigger on any of the stores - a 1:1 mapping if you will; quite similar to a function call. Having it trigger multiple things at once seems... well confusing haha
Interesting. In my work, I've been thinking of "actions" as a separate abstraction, basically representing any "verb" within the system (some of which map directly to user actions like selectSomeItem or cancelSomething, others of which are more internal like fetchSomeData or updateBlahStatus). Coming from that point of view, I feel that it can sometimes make sense for an action to trigger multiple things, because I think there isn't always a 1:1 mapping between "verbs" and stores. But if there's going to be a 1:1 mapping, then would it be simpler to eliminate actions altogether, in favor of an API like (state, effects, stores) => { ... stores'namespace:reducerName' ... }?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Yeah, I can see how that makes sense, although again, if there's going to be a 1-1 mapping between actions and effects||reducers, then it seems to me like the "actions" concept is unnecessary...

It's more of one-to-many.  One effect can trigger several reducers, but
a reducer only ever changes one part of the state and will never call another
reducer.
On Fri, Jun 24, 2016 at 06:36:12AM -0700, Anand Thakker wrote:

Yeah, I can see how that makes sense, although again, if there's going to be a 1-1 mapping between actions and effects||reducers, then it seems to me like the "actions" concept is unnecessary...

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/yoshuawuyts/choo/pull/46#issuecomment-228347197

Here is an example:
https://github.com/toddself/boxcar/blob/master/effects/move.js
On Fri, Jun 24, 2016 at 06:36:12AM -0700, Anand Thakker wrote:

Yeah, I can see how that makes sense, although again, if there's going to be a 1-1 mapping between actions and effects||reducers, then it seems to me like the "actions" concept is unnecessary...

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/yoshuawuyts/choo/pull/46#issuecomment-228347197

It's more of one-to-many.  One effect can trigger several reducers, but a reducer only ever changes one part of the state and will never call another reducer.

effects-reducers is 1-to-many, but I thought @yoshuawuyts was proposing above that actions-{reducers or effects} should be 1-1

(...) actions-{reducers or effects} should be 1-1

Yup you're both right. One action can trigger either one effect or one reducer; effects can emit multiple actions. Not the current behavior, but think it would make more sense

Yup you're both right. One action can trigger either one effect or one reducer; effects can emit multiple actions. Not the current behavior, but think it would make more sense

@yoshuawuyts I can definitely see your point on the clarity/simplicity of doing this, but it still leaves me with the question about what value is then being added by the "action" concept?  Since each action would simply map directly to either a reducer or effect, they really are equivalent to function calls (like you said above) -- so at that point, why is the indirection through send(...) necessary or preferable to an API where you just call methods directly?

why is the indirection through send(...) necessary or preferable to an API where you just call methods directly

Because multiplexing. A function call requires knowledge of which function you're calling, and importing it explicitly from the location it's declared. By using a message bus, function calling becomes centralized and thus more loosely coupled. This enables better tracing, logging and upgrading parts of the application without touching other parts. It also clearly signals that it's a different method of calling functions; one where callbacks or returning eventual values doesn't fly - thus making for a better enforcement of the unidirectional data flow (despite there always being ways of cheating around it).

A function call requires knowledge of which function you're calling, and importing it explicitly from the location it's declared. By using a message bus, function calling becomes centralized and thus more loosely coupled. This enables better tracing, logging and upgrading parts of the application without touching other parts.

Ah, yeah: I totally agree on the value of a message bus internally.  What I was thinking/wondering about was not plain function calls, but rather an API where instead of getting send as a parameter, views (and effects, etc.) would get something like { effects, stores }, and where stores['thing1'](data) or effects['thing2'](data) would be the exact equivalent of send('thing1', data) or send('thing2', data)...

It also clearly signals that it's a different method of calling functions; one where callbacks or returning eventual values doesn't fly

... but I hadn't thought of this point, which I can see being a downside of the thing I'm describing above.  I guess now that I've written it out more explicitly, the thing I was suggesting adds more magic and more internal machinery for little gain.
Thanks for engaging with my speculations and questions!

@yoshuawuyts agreed on the fact that Object.keys(reducers).concat(Object.keys(effects)) should be a unique set otherwise it makes it hard to reason about order of operations when you trigger something that fires and effect and a reducer.

@toddself Given that this is moved to https://github.com/yoshuawuyts/choo/tree/browser-tests and barracks I think it's safe to close right? :sparkles:

💯


Clean Resuable Action to remove unused files and folders

This is already addressed in release v1.0.0-alpha


SubCommunities are not displayed appropriately in browse hierarchy

How to reproduce:

Install the DSpace "spring-rest" webapp locally.
Create a hierarchy that includes sub-communities (either via AIP import, or by migrating data created via a DSpace 6.x UI)
Point the Angular UI at this local "spring-rest" webapp

The result is that all subcommunities are listed on the homepage of the Angular UI (as if they were top-level communities).  Additionally, if you browse to their parent community, the subcommunites are not listed.
This bug is not reproducible when using our REST API demo site, as that site currently doesn't have any subcommunities.

Fixed by #330


VIS Loading Values for ever

Hello!
Iam not sure if it is better to write in englisch or in german. So I decided to write in englisch (its not my native language).
I posted the problem already in IOBROKER Forum but without any result
https://forum.iobroker.net/topic/47588/vis-loading-values-vis-startet-nicht-mehr/6
Problem:
I have a Problem in respect to the Visualisation. As soon I update "Socket.io" or "WEB-SERVER" Adapter my Visualisation wont work anymore. I just get a screen with LOADING VALUES and thats it.
Versions:

Adapter version:
Adapter    "admin"         : 5.1.23   , installed 5.1.23
Adapter    "alexa2"        : 3.9.3    , installed 3.9.3
Adapter    "daswetter"     : 3.0.7    , installed 3.0.7
Adapter    "discovery"     : 2.7.0    , installed 2.7.0
Adapter    "flot"          : 1.10.6   , installed 1.10.6
Adapter    "history"       : 1.9.13   , installed 1.9.13
Adapter    "ical"          : 1.11.2   , installed 1.11.2
Adapter    "icons-mfd-png" : 1.0.2    , installed 1.0.2
Adapter    "info"          : 1.9.8    , installed 1.9.8
Adapter    "javascript"    : 5.2.8    , installed 5.2.8
Controller "js-controller" : 3.3.15   , installed 3.3.15
Adapter    "knx"           : 1.0.45   , installed 1.0.45
Adapter    "openhab"       : 1.1.0    , installed 1.1.0
Adapter    "pushover"      : 2.0.5    , installed 2.0.5
Adapter    "rickshaw"      : 1.0.0    , installed 1.0.0
Adapter    "simple-api"    : 2.6.1    , installed 2.5.3 [Updateable]
Adapter    "snmp"          : 0.5.0    , installed 0.5.0
Adapter    "socketio"      : 3.1.4    , installed 3.0.12 [Updateable]
Adapter    "vis"           : 1.4.3    , installed 1.2.9 [Updateable]
Adapter    "vis-fancyswitch": 1.1.0   , installed 1.1.0
Adapter    "vis-history"   : 1.0.0    , installed 1.0.0
Adapter    "vis-hqwidgets" : 1.1.7    , installed 1.1.7
Adapter    "vis-jqui-mfd"  : 1.0.12   , installed 1.0.12
Adapter    "vis-map"       : 1.0.4    , installed 1.0.4
Adapter    "vis-metro"     : 1.1.2    , installed 1.1.2
Adapter    "web"           : 3.4.7    , installed 3.0.10 [Updateable]

JS-Controller version:  3.3.15

Node version: v12.22.6

Operating system: Debian

Anybody who can assist me ?`
Many thanks
BG

Pleas eupdtae weband vis to the latest available version - else check changelogs because it coud just be version interoperability ... and your vis is way outdated

i tried to do this, but as soon i leave the old versions, the problems occurs

the above listed adapter version are working right now but as soon i update to the latest versions the problem appears

In fact you need to update ALL together and then do a hard reload pot incl clearing caches ... most users use uptodate versions and no report of such an error appears so far. Also the one warning in forum id no reason for it not loading.

host.IOBrokerVers4
2021-09-07 11:47:36.890
error
startInstance system.adapter.vis.0: required adapter "web" has wrong version. Installed "3.0.10", required ">=3.0.12"!

host.IOBrokerVers4-Server
2021-09-07 11:47:36.666
error
Cannot write files: /opt/iobroker/iobroker-data/files/vis/cordova.js: EACCES: permission denied, open '/opt/iobroker/iobroker-data/files/vis/cordova.js'

host.IOBrokerVers4-Server
2021-09-07 11:47:36.604
error
Cannot write files: /opt/iobroker/iobroker-data/files/vis/css/add_kian.css: EACCES: permission denied, open '/opt/iobroker/iobroker-data/files/vis/css/add_kian.css'

host.IOBrokerVers4-Server
2021-09-07 11:47:36.546
error
Cannot write files: /opt/iobroker/iobroker-data/files/vis/css/app.css: EACCES: permission denied, open '/opt/iobroker/iobroker-data/files/vis/css/app.css'

host.IOBrokerVers4-Server
2021-09-07 11:47:36.487
error
Cannot write files: /opt/iobroker/iobroker-data/files/vis/css/backgrounds.css: EACCES: permission denied, open '/opt/iobroker/iobroker-data/files/vis/css/backgrounds.css'

host.IOBrokerVers4-Server
2021-09-07 11:47:36.429
error
Cannot write files: /opt/iobroker/iobroker-data/files/vis/css/doc.css: EACCES: permission denied, open '/opt/iobroker/iobroker-data/files/vis/css/doc.css'

host.IOBrokerVers4-Server
2021-09-07 11:47:36.372
error
Cannot write files: /opt/iobroker/iobroker-data/files/vis/css/montserrat-regular-webfont.woff: EACCES: permission denied, open '/opt/iobroker/iobroker-data/files/vis/css/montserrat-regular-webfont.woff'

host.IOBrokerVers4-Server
2021-09-07 11:47:36.313
error
Cannot write files: /opt/iobroker/iobroker-data/files/vis/css/styles.css: EACCES: permission denied, open '/opt/iobroker/iobroker-data/files/vis/css/styles.css'

host.IOBrokerVers4-Server
2021-09-07 11:47:36.256
error
Cannot write files: /opt/iobroker/iobroker-data/files/vis/css/vis-editor.css: EACCES: permission denied, open '/opt/iobroker/iobroker-data/files/vis/css/vis-editor.css'

host.IOBrokerVers4-Server
2021-09-07 11:47:36.199
error
Cannot write files: /opt/iobroker/iobroker-data/files/vis/css/vis.css: EACCES: permission denied, open '/opt/iobroker/iobroker-data/files/vis/css/vis.css'

host.IOBrokerVers4-Server
2021-09-07 11:47:36.141
error
Cannot write files: /opt/iobroker/iobroker-data/files/vis/edit.html: EACCES: permission denied, open '/opt/iobroker/iobroker-data/files/vis/edit.html'

host.IOBrokerVers4-Server
2021-09-07 11:47:36.071
error
Cannot write files: /opt/iobroker/iobroker-data/files/vis/edit.src.html: EACCES: permission denied, open '/opt/iobroker/iobroker-data/files/vis/edit.src.html'

That i found in the logs after updating to the last versions.
currently this adapter version are installed:
Used repository: stable
hash unchanged, use cached sources
update done
Adapter    "admin"         : 5.1.23   , installed 5.1.23
Adapter    "alexa2"        : 3.9.3    , installed 3.9.3
Adapter    "daswetter"     : 3.0.7    , installed 3.0.7
Adapter    "discovery"     : 2.7.0    , installed 2.7.0
Adapter    "flot"          : 1.10.6   , installed 1.10.6
Adapter    "history"       : 1.9.13   , installed 1.9.13
Adapter    "ical"          : 1.11.2   , installed 1.11.2
Adapter    "icons-mfd-png" : 1.0.2    , installed 1.0.2
Adapter    "info"          : 1.9.8    , installed 1.9.8
Adapter    "javascript"    : 5.2.8    , installed 5.2.8
Controller "js-controller" : 3.3.15   , installed 3.3.15
Adapter    "knx"           : 1.0.45   , installed 1.0.45
Adapter    "openhab"       : 1.1.0    , installed 1.1.0
Adapter    "pushover"      : 2.0.5    , installed 2.0.5
Adapter    "rickshaw"      : 1.0.0    , installed 1.0.0
Adapter    "simple-api"    : 2.6.1    , installed 2.6.1
Adapter    "snmp"          : 0.5.0    , installed 0.5.0
Adapter    "socketio"      : 3.1.4    , installed 3.1.4
Adapter    "vis"           : 1.4.3    , installed 1.4.3
Adapter    "vis-fancyswitch": 1.1.0   , installed 1.1.0
Adapter    "vis-history"   : 1.0.0    , installed 1.0.0
Adapter    "vis-hqwidgets" : 1.1.7    , installed 1.1.7
Adapter    "vis-jqui-mfd"  : 1.0.12   , installed 1.0.12
Adapter    "vis-map"       : 1.0.4    , installed 1.0.4
Adapter    "vis-metro"     : 1.1.2    , installed 1.1.2
Adapter    "web"           : 3.4.7    , installed 3.4.7

it is working again. i just changed the config of the web adapter.
binded ip: allow all ips -> <IP_ADDRESS> then i tried it again and no connection was possible.
after that i switched back to allow all ips and then it worked.
can be closed

In fact for the "permission denied" try to run "iobroker fix". should fix that too


Fix test issues due to an out of order merge

Fixes some issues I am seeing after the following PRs were merged:

#10899 - Incorrectly added tsc.ts as a dependency of harness
#10448 - .types baselines are incorrect following #10676 since Travis CI for #10448 passed prior to the merge.

// CC: @mhegazy, @vladima

:+1:

thanks!


Added .travis.yaml and travis badge to readme.md

@domejiraphon @liyingxin97 @YongchangLiu98 I guess I probably shouldn't approve this because I made changes, is one of you able to do so when you have a chance?


Assignment G3_1 (Close #1)

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #1000

Regex validating Names was incorrect. It was checking usernames start with alphabets along with characters.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!

@googlebot I fixed it.

@googlebot I fixed it.

@googlebot I fixed it.

@googlebot I signed it!

@googlebot I fixed it.


Support range literals with no upper bound.

This syntax is only valid in 2.6, so the tests unsurprisingly fail on 2.3 and 2.5.
Not sure if it's better to throw away these fixtures, or change the CI setup to run against 2.6.

I need to rejiggle the CI script, which is currently hideously naieve.

push incoming.

oh joy there's a failing CI on 2.3 :(

@georgebrock ignoring the failing CI, WDYT of the bash patch?

docker.

@georgebrock this is now blocked by #91

@samphippen The script changes look good to me.
Nice use of bc(1) for the version comparison, I was thinking of doing something awful with Gem::Version.new(RUBY_VERSION).

@georgebrock please provide final LGTM, I got this green on all 3 rubys we CI.

I just did a mega rebase and I want to see if I didn't break anything.


APK Verification

The program cannot distinguish any other file from an APK file. It simply tries to run operations on the file it is given, even if it is not a APK file. This results in so many errors on the screen that I want to cry. I will fix this as soon as possible.

Solved!


Header Improve (scroll style and signup/login button)

If we need to add scroll style, we need to make header fixed. And then we will mess the layout. So we leave it in the future.


Set the selectedIndex of the tab bar programmatically

Does anyone know a way to set the selected index of the tab bar. Currently it is only possible to set the initialSelectedIndex property but I would like to be able to set this property dynamically.

Thanks @gappture - There is a PR for this outstanding and hopefully it will be fixed soon. #465


 Put.io is a cloud storage and download manager that enables you to fetch content from various sources and store it in your private and secure cloud space.
Screencast
https://share.cleanshot.com/MMXc9V6j
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi, is anyone available to review the most recent changes?

I'm still not sure about the wget / aria2c commands, is there no way to have them fixed? Potentially it could be dangerous since you can run all such of user commands

Are you ready for another review @EpaL

I guess we can close this PR since https://github.com/raycast/extensions/pull/6936 was merged?

Yes! 🙂


sslverify not used, so https doesn't gain security

I've been upgrading the Chef cookbook yum-jus to incorporate the changes done here, where the cookbook was already setting sslverify=true. This errored on CentOS 5.11 due to some incompatibility with the certificate, so I compared it to this repo, and found you are missing sslverify.
Without this, any MITM can still decrypt the SSL connection with any certificate self-signed or not.
sslverify=true works for the IUS mirrorlist on CentOS 6.6 and 7.1

Hello andytson,
Thanks for taking the time to reach out to us regarding this issue.  We are looking to take steps to use https as much as possible, but not all of our mirrors have been configured for https.  Currently, we are mixing both http and https in an effort to get to a pure https configuration.  The mirrorlist is https, but it returns a list of http mirrors (&protocol=http).  Even with a list of https mirrors (&protocol=https) EL 5 errors out (maybe due to lack of SNI support??).
There was a bunch of internal conversations about how to move toward https or if we should even bother.  We were torn between moving toward a pure ssl configuration (where possible), using no ssl (like we were before) or trying to introduce some ssl.  We felt the mirrorlist URL was making it pretty clear that one should not except end to end encryption.  If that is not the case, we may need to revisit this decision.
Also not that sslverify default is false for EL 5, but true for EL6 and EL7.
-Ben

Hi,
Sorry I wasn't aware that sslverify was set to true for EL6+7. That should take care of it.
Regarding security, to prevent tampering of packages, all that's needed is that the gpgkey is transmitted securely (either through importing https://dl.iuscommunity.org/pub/ius/IUS-COMMUNITY-GPG-KEY, or installing jus-release from a trusted https location)
Obviously it's not possible to provide complete end-to-end https OTB without dictating the terms for 3rd party mirrors, which would reduce your mirrorlist.

Additionally metalink formatted mirrorlists can help prevent forced staleness of mirrors https://fedoraproject.org/wiki/Features/YumMetalinks

EPEL does that now using https://mirrors.fedoraproject.org/metalink?repo=epel-7&arch=x86_64

Hey andytson,
Thanks again for your interest in this issue.  I agree that getting the ius-release package or our public gpg key securely is the important part.   Along those lines, what changes are needed for Chef?  Are you having Chef configure the repos manually?  Why not just have Chef install the ius-release package?
In regards to metalink, I think for now we are going to stick with mirrorlist.  We already have a process in place for dealing with stale mirrors.
-Ben

Hi Ben,
Chef (Company) is the maintainer of the Chef cookbook (https://github.com/chef-cookbooks/yum-ius), I just contribute some of the things you change to the ius-release.
Chef by convention manages the .repo files themselves so they are configurable (e.g. defining a private mirror, or enabling ius-archive), so don't use the ius-release RPM any more (they used to).
They use the gpgkey accessible from http://dl.iuscommunity.org/pub/ius/IUS-COMMUNITY-GPG-KEY, however my changes are introducing the https protocol for it, assuming the https url remains available.
https://github.com/chef-cookbooks/yum-ius/pull/6
I think the biggest reason they took this approach was because there was no single unchanging url to grab the ius-release RPM from (e.g like https://mirror.webtatic.com/yum/el7/webtatic-release.rpm).
Before they did this, my workplace (like others) had to keep adjusting the release number every change to ius-release.
Additionally Chef (software) yum_repository resource doesn't support re-parsing existing ius.repo file to insert configuration changes.
Regarding the metalink format, I was more meaning this as an alternative to help prevent MITM of http mirrors offering old signed packages from ius-archive with known vulnerabilities in place of the latest. I don't believe this can be prevented through processes (nor can bad-actors with access to mirrors temporarily or selectively providing stale packages), only either through secure hashes inside metalink or via ssl verified https repositories.

Hey @andytson, we've got something in the works for that.  I created a redirect a while back, but we haven't tested it much or published anything about it.
$ curl -IL https://dl.iuscommunity.org/pub/ius/stable/CentOS/7/ius-release-latest.rpm
HTTP/1.1 301 Moved Permanently
Date: Fri, 05 Jun 2015 20:08:36 GMT
Server: Apache/2.2.15 (CentOS)
Location: https://dl.iuscommunity.org/pub/ius/stable/CentOS/7/x86_64/ius-release-1.0-14.ius.centos7.noarch.rpm
Connection: close
Content-Type: text/html; charset=iso-8859-1

HTTP/1.1 200 OK
Date: Fri, 05 Jun 2015 20:08:37 GMT
Server: Apache/2.2.15 (CentOS)
Last-Modified: Fri, 05 Jun 2015 04:16:40 GMT
ETag: "15c0808-2048-517bd8f0ffa00"
Accept-Ranges: bytes
Content-Length: 8264
Connection: close
Content-Type: application/x-rpm

Hey @andytson,
Also see our recent mailing list post about some new permanent ius-release URLs:
https://lists.launchpad.net/ius-community/msg01495.html
-Ben

yep, thanks. Ironically started getting errors on some of my works old projects due to the el6 13 to 14 release number change.

I'm disappointed this is closed, and as a result will either use the https mirrors or will drop use of IUS.
I can make a proof of concept mirror that shows old packages in ius repository, but that is very trivial.
I'll also be notifying an OS dist about this issue for them, as I believe this kind of compromise shouldn't be allowed to be possible in this day and age.

Hey @andytson,
Sorry for any confusion.  It seems like the conversation had ended.  At this point, what are your concerns?
-Ben

I'm concerned that without using https mirrors or metalinks format mirrorlists, it's possible for a MITM to serve up old ius packages with their own generated repodata, log the traffic downloading the old package, and use a known security exploit. These packages would be able to be installed because they are signed still by the gpg key
I think also due to 3rd party mirrors, the metalinks will be the best solution to avoid bad mirrors as well. These might be able to fool you by serving different repos when you're checking them versus real users.

I'm disappointed this is closed, and as a result will either use the https mirrors or will drop use of IUS.

Are you prepared to stop using CentOS as well?
CentOS
mirrorlist=http://mirrorlist.centos.org/?release=$releasever&arch=$basearch&repo=os&infra=$infra

plaintext mirrorlist
mirrorlist service run over HTTP
offers HTTP, FTP, and RSYNC mirror links, without the explicit option to choose one protocol

IUS
mirrorlist=https://mirrors.iuscommunity.org/mirrorlist?repo=ius-centos7&arch=$basearch&protocol=http

plaintext mirrorlist
mirrorlist service run over HTTPS
offers HTTP, HTTPS, FTP, or RSYNC mirror links, with the explicit option to choose one protocol, meaning you can force it to only show you HTTPS mirror links

The metalink format is intriguing.  Before I wrote umirr, I experimented with MirrorManager.  I got it up and running, but I honestly just couldn't figure out how to configure and use it.  The documentation for it is nearly non-existent, and what is available is severely out-dated.
If you are truly concerned about MITM attacks, change the requested protocol to HTTPS.  We can re-evaluate the options in the future, such as adding metalink support to umirr or switching to MirrorManager.
One final question, would having signed repository metadata alleviate your concerns?

Regarding CentOS, sorry I didn't want to name and shame them but "I'll also be notifying an OS dist about this issue for their own repos" was meaning them.
Signed repository metadata looks nice, especially since metalink is sha* hashes only when this is gpg, however the gpg signature is fetched from the repo server rather than mirrorlist server, so someone could capture a current repodata and still serve it some time in the future where the packages in the repodata then have known vulnerabilities. It also, based on reading that link you mentioned, would require an EPEL package dependency on ius-release (other than epel-release).
Metalink solves this issue by the sha* hashes being stored in the mirrorlist file (formatted as metalink) on the mirrorlist server on https, therefore with ssl verification, no MITM that can't compromise SSL CA root/intermediaries can serve any non-approved repodata (of which can have multiple versions to allow for your 24 hour sync time).

Sorry I wasn't aware that sslverify was set to true for EL6+7 by default. That should take care of it.

Now that EL5 is no longer a concern, I'm going to close this issue.
Switching to metalink format and using signed repository metadata are still topics I want to investigate further, but I don't see the need to keep this issue open for that.


Does not work with --prod flag (Angular 7)

Got errors like :
ERROR in Error during template compile of 'AppModule'
Function calls are not supported in decorators but 'SpeechRecognitionModule' was called.
forRoot() should only "return", without any function call (actually const providers: Provider[] = SpeechRecognitionModule.buildProvidersFromConfig(config); and some array pushes).
I also got :
ERROR in : Error: Internal error: unknown identifier undefined

@MaksJS
Thank you for reporting!
I’ll check the repeatability.

@kamiazya you can reproduce it by running ng build --prod in the root of the projet

@MaksJS
Thanks!
I could reproduce it in the way you said.
And I found a nice commit in your fork repository.
Please send me a pull request!

I've sent you a pull request. I've refactored all buildProvidersFromConfig logic into return statements of forRoot() and withConfig().

@MaksJS
Thank you for sending PullRequest!
I merged and released this commit, I got the following another error.
ERROR in : Invalid provider for undefined. useClass cannot be undefined.
           Usually it happens when:
           1. There's a circular dependency (might be caused by using index.ts (barrel) files).
           2. Class was used before it was declared. Use forwardRef in this case.

I will continue to address this issue in this issue as well.
If you know the solution to this problem, please continue to support me.

@kamiazya I've debugged with console.log in<EMAIL_ADDRESS>        CompileMetadataResolver.prototype._validateProvider = function (provider) {
            console.log('type', provider);
            if (provider.hasOwnProperty('useClass') && provider.useClass == null) {
                this._reportError(syntaxError("Invalid provider for " + stringifyType(provider.provide) + ". useClass cannot be " + provider.useClass + ".\n           Usually it happens when:\n           1. There's a circular dependency (might be caused by using index.ts (barrel) files).\n           2. Class was used before it was declared. Use forwardRef in this case."));
            }
        };

type { useValue: true, provide: undefined }
type { useValue: false, provide: undefined }
type { useValue: 1, provide: undefined }
type { useClass: undefined, provide: undefined }

We got undefined for all the providers, it seems like there is a circular dependency. I think it's a problem with the way the classes are imported / exported in source files. There is no problem with your package "speech-synthesis" which I'm also using.

@MaksJS
Now, I released a version that solved the problem of circular dependency!
I was greatly helped by detailed debugging and reports.
Thank you!
The build is now successful even with the --prod flag, as shown below.
$ ng add @kamiazya/ngx-speech-recognition
Skipping installation: Package already installed
    ✅ Added dependency<EMAIL_ADDRESS>    ✅ Dependencies installed
    ✅ SpeechRecognitionModule Imported to /src/app/app.module.ts
UPDATE package.json (1363 bytes)
UPDATE src/app/app.module.ts (446 bytes)
[###########################################################################################################################################] 1900/1900
$ ng build --prod

Date: 2019-06-03T23:12:38.038Z
Hash: 2f457f8b83fd05caf766
Time: 16798ms
chunk {0} runtime.26209474bfa8dc87a77c.js (runtime) 1.41 kB [entry] [rendered]
chunk {1} es2015-polyfills.1e04665e16f944715fd2.js (es2015-polyfills) 56.4 kB [initial] [rendered]
chunk {2} main.59dad101e63add5094a3.js (main) 162 kB [initial] [rendered]
chunk {3} polyfills.8bbb231b43165d65d357.js (polyfills) 41 kB [initial] [rendered]
chunk {4} styles.3ff695c00d717f2d2a11.css (styles) 0 bytes [initial] [rendered]

@kamiazya It works great now, thanks!


Match tsx 'import' color with typescript/javascript/jsx

Description
javascript, javascriptreact and typescript have a purple import statement, while typescriptreact is red. This PR changes the typescriptreact import statement to purple to match the other languages.

Screenshots
Before:

After:

Nice catch 🙌 And thanks for the screenshots :)


Cannot completely rename the input box of the frame.

I found that I cannot completely rename the input box of the frame.
I added a feature to completely delete it if it is an empty string, but after blurring, it still shows "Frame 1".
Originally and after modification

@hqwuzhaoyi already solved at #7376


Upgrade from 1.6.2 to 2.0.3 leads to WARN: image icon for 'check' not found

After upgrading from 1.6.2 to 2.0.3 we experienced the following error message:
WARN: image icon for 'check' not found or not readable
There were no further code changes. With the previous versions, we had no problems regarding the check icon. Downgrading back to 1.6.2 helped.

It sounds to me like you have the icons attribute set on the doument and there's an icon macro in your document that refers to the icon named "check":
icon:check[]

The converter is going to look for an image icon that matches that name. If it can't find the icon, it will log a warning. This is an enhancement that was introduced in 2.0.0.alpha.2, which is cited in the CHANGELOG:

add support for image-based icons, resolved from iconsdir and having the icontype file extension (#1770)

The message should be telling you exactly what file the converter looking for. The message should actually look like this:

asciidoctor: WARNING: image icon for 'check' not found or not readable: /path/to/project/images/icons/check.png

If you need more assistance with using this feature, please ask your question in the project chat at https://chat.asciidoctor.org.


Fix cohort info styling

Description
The current state:

With this PR, the custom styling of the cohort information section is removed and replaced with Stack, Label, and Text components without custom styling.

I also removed the "divider" code since that didn't add anything useful. It's only used in error-analysis (the horizontal black line):

Areas changed

npm packages changed:

[ ] responsibleai/causality
[x] responsibleai/core-ui
[ ] responsibleai/counterfactuals
[ ] responsibleai/dataset-explorer
[x] responsibleai/error-analysis
[ ] responsibleai/fairness
[ ] responsibleai/interpret
[ ] responsibleai/localization
[ ] responsibleai/mlchartlib
[ ] responsibleai/model-assessment

Python packages changed:

[ ] raiwidgets
[ ] responsibleai
[ ] erroranalysis
[ ] rai_core_flask

Tests

[ ] No new tests required.
[ ] New tests for the added feature are part of this PR.
[ ] I validated the changes manually.

Screenshots (if appropriate):
see above
Documentation:

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

woohoo!

Codecov Report

Merging #1277 (d3b16d1) into main (80de61d) will decrease coverage by 52.36%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #1277       +/-   ##
===========================================
- Coverage   65.57%   13.20%   -52.37%
===========================================
  Files          91       30       -61
  Lines        4505     1477     -3028
===========================================
- Hits         2954      195     -2759
+ Misses       1551     1282      -269

Flag
Coverage Δ

unittests
13.20% <ø> (-52.37%)
:arrow_down:

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

Impacted Files
Coverage Δ

...ponsibleai/responsibleai/_tools/shared/versions.py

...ranalysis/erroranalysis/analyzer/error_analyzer.py

...ranalysis/erroranalysis/_internal/matrix_filter.py

...nsibleai/responsibleai/_managers/causal_manager.py

...bleai/responsibleai/_tools/causal/causal_result.py

...ranalysis/erroranalysis/_internal/cohort_filter.py

erroranalysis/erroranalysis/_internal/__init__.py

erroranalysis/erroranalysis/__init__.py

responsibleai/responsibleai/_config/base_config.py

responsibleai/responsibleai/_internal/constants.py

... and 51 more

Continue to review full report at Codecov.

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


Add color and vehicle type to VeRI chips

This fixes #65.

Codecov Report

Merging #66 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #66   +/-   ##
=======================================
  Coverage   88.19%   88.19%
=======================================
  Files          29       29
  Lines        1575     1575
  Branches      209      209
=======================================
  Hits         1389     1389
  Misses        165      165
  Partials       21       21

Impacted Files
Coverage Δ

pelops/datasets/veri.py
88.23% <100%> (ø)
: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 2368b79...cf52b70. Read the comment docs.

@d-grossman: I've added in two new values to the misc dictionary in VeRI chips. They are set to strings (https://github.com/Lab41/pelops/pull/66/files#diff-e20cb6b674d6b9fac3c868dac28b4504R57) for the Training set (the only place where we have the information) and to None for the test set chips.
Will this information suffice for training? Do you want different/other keywords?

I just rebased onto the current master so I can run tests in Docker.

It builds and tests in docker now! Very nice!

I have added a "more helpful" error for when the xml file is malformed:
ValueError: multi-byte encodings are not supported

The label file 'train_label.xml' comes malformed from the
source. The first line needs to be changed to:
'<?xml version="1.0" encoding="UTF-8" ?>'
if it is not already.
See: https://github.com/Lab41/pelops/issues/72

@d-grossman If you could give this one more look over, I think it's ready to go.

I have added a patch to work around #76.


sumsung s10 phone load video crash,anyone can help me ?

Describe the bug
i load a video into my  ar scene crashed on my s10 phone,but the same code is work on huawei mate20x
To Reproduce
Steps to reproduce the behavior:
1.load video
`Material.builder().setSource(activity, GetSceneformResource(activity, VIEW_RENDERABLE_MATERIAL))
.build()
.thenAccept(
{ material ->
                        var vertices = arrayListOf<Vertex>()
                        vertices.add(Vertex.builder().setPosition(Vector3(-0.5f, 0.0f, 0.0f))
                                .setNormal(Vector3(0.0f, 0.0f, 1.0f))
                                .setUvCoordinate(Vertex.UvCoordinate(0.0f, 0.0f))
                                .build())

                        vertices.add(Vertex.builder().setPosition(Vector3(0.5f, 0.0f, 0.0f))
                                .setNormal(Vector3(0.0f, 0.0f, 1.0f))
                                .setUvCoordinate(Vertex.UvCoordinate(1.0f, 0.0f))
                                .build())

                        vertices.add(Vertex.builder().setPosition(Vector3(-0.5f, 1.0f, 0.0f))
                                .setNormal(Vector3(0.0f, 0.0f, 1.0f))
                                .setUvCoordinate(Vertex.UvCoordinate(0.0f, 1.0f))
                                .build())

                        vertices.add(Vertex.builder().setPosition(Vector3(0.5f, 1.0f, 0.0f))
                                .setNormal(Vector3(0.0f, 0.0f, 1.0f))
                                .setUvCoordinate(Vertex.UvCoordinate(1.0f, 1.0f))
                                .build())

                        var triangleIndices = arrayListOf<Int>()

                        triangleIndices.add(0)
                        triangleIndices.add(1)
                        triangleIndices.add(2)
                        triangleIndices.add(1)
                        triangleIndices.add(3)
                        triangleIndices.add(2)

                        var submesh = RenderableDefinition.Submesh.builder().setTriangleIndices(triangleIndices)
                                .setMaterial(material).build()

                        var ulCorner = ModelRenderable.builder().setSource(
                                RenderableDefinition.builder().setVertices(vertices)
                                        .setSubmeshes(Arrays.asList(submesh)).build()).build()

                        ulCorner.thenAccept({ renderable ->
                            renderable.getMaterial().setExternalTexture("viewTexture", texture)
                            if (!mediaPlayer.isPlaying) {
                                mediaPlayer.start()
                                texture.surfaceTexture.setOnFrameAvailableListener {
                                    surfaceTexture: SurfaceTexture ->
                                    this.renderable = renderable
                                }
                            }
                        })

`
Expected behavior
Screenshots
Desktop (please complete the following information):

OS:  macos
GPU: [e.g. NVIDIA GTX 1080]
Backend: [OpenGL/Vulkan]

Smartphone (please complete the following information):

Device: sumsung model:SM-G9730
OS: Android Pie 9.0

Additional context
can anyone help me ?

the log i had catch these:
2020-11-11 14:14:48.562 6814-7090/com.whyhow.sucailib A/whyhow.sucaili: java_vm_ext.cc:542]   native: #10 pc 000570d5  /data/app/com.whyhow.sucailib-yU_hMat5RSxHXi7sm2XW3Q==/lib/arm/libfilament-jni.so (???)
2020-11-11 14:14:48.640 6814-7090/com.whyhow.sucailib A/whyhow.sucaili: runtime.cc:574] #28 pc 000570d5  /data/app/com.whyhow.sucailib-yU_hMat5RSxHXi7sm2XW3Q==/lib/arm/libfilament-jni.so (???)
2020-11-11 14:14:48.705 7267-7090/? E/DEBUG:     28 pc 000570d5  /data/app/com.whyhow.sucailib-yU_hMat5RSxHXi7sm2XW3Q==/lib/arm/libfilament-jni.so
2020-11-11 14:14:48.736 7267-7090/? E/DEBUG: /data/app/com.whyhow.sucailib-yU_hMat5RSxHXi7sm2XW3Q==/lib/arm/libfilament-jni.so: f0ea41d366fe91d0561682a81042f26b73aa901a
2020-11-11 14:14:48.736 7267-7090/? E/DEBUG: /data/app/com.whyhow.sucailib-yU_hMat5RSxHXi7sm2XW3Q==/lib/arm/libfilament-jni.so: f0ea41d366fe91d0561682a81042f26b73aa901a

What version of Filament are you using? Could you also show the entire crash logs please? Important bits are missing

the version is 1.7.0, Can only catch so many logs，sorry for this .and thank you for your time.

What version of Filament are you using? Could you also show the entire crash logs please? Important bits are missing

the version is 1.7.0, Can only catch so many logs，sorry for this .and thank you for your time.

Please try with Filament 1.9, we fixed several JNI bugs that could explain your crash from what I can see.


Provide staticly-linked binary?

Tried in the server (centos 7.9) and failed :(
$ fqgrep TCAAGTAGTAACTGCCGCGACCTTAGGGCAAGACCTTATGATA  KPN1big_1.fq.gz
fqgrep: /lib64/libc.so.6: version `GLIBC_2.25' not found (required by fqgrep)

Oh no, that's not good, let me see if I can fix that.  Installed via bioconda, from the packages on the releases page, compile from source?

If from the uploaded binaries, I wonder if it would work if I added:
        env:
          RUSTFLAGS: -C target-feature=+crt-static
          OPENSSL_STATIC: 1

to https://github.com/fulcrumgenomics/fqgrep/blob/49882dca71daf7916e8e82acdbba16fa355733d6/.github/workflows/publish.yml#L27-L28

Installed via bioconda, from the packages on the releases page, compile from source?

It might still fail.
I'm pretty sure Conda caches some files in their server, including binaries (my tools) and packages supposed to be compiled from the source. It may check and compare the local environment with that of the cache server; if they matched, the cached binary is downloaded to avoid compiling from the source, which may fail to run. https://github.com/waveygang/wfmash/issues/166
I think https://github.com/sharkdp/fd and https://github.com/BurntSushi/ripgrep are good examples of uploading portable binaries.

@shenwei356 found a little time to work on this, testing this over here: https://github.com/fulcrumgenomics/tmp-fqgrep-githubactions.  Once I get it working, I can fold it back into here.  The ripgrep example is super helpful.  Any chance you can test one of the binaries on CentOS 7.9 when I get them built?

Any chance you can test one of the binaries on CentOS 7.9 when I get them built?

Glad to help :)

I have a few here: https://github.com/fulcrumgenomics/tmp-fqgrep-githubactions/releases/tag/v16

It works fine on CentOS 7.9, and it's much faster than seqkit grep.
$ seqkit stats t.fq.gz
file     format  type   num_seqs      sum_len  min_len  avg_len  max_len
t.fq.gz  FASTQ   DNA   1,000,000  150,000,000      150      150      150

# https://github.com/shenwei356/memusg
$ memusg -t -s 'time fqgrep CCGATATGATTCG t.fq.gz -F -t 4 -c'
7

real    0m1.827s
user    0m2.140s
sys     0m0.485s

elapsed time: 1.920s
peak rss: 253.62 MB

$ memusg -t -s 'time seqkit grep -s -P -p CCGATATGATTCG t.fq.gz -j 4 -C'
7

real    0m2.984s
user    0m4.207s
sys     0m0.078s

elapsed time: 3.190s
peak rss: 24.15 MB

I failed to test other cases cause the help usage of the binary is different from the readme of the repo.
And it's hard to correctly use the option -e, --regexp <regexp>...  following the help message.
E.g. the command below suspends without any message or output, and the CPU usage is zero.
fqgrep -e CCGATATGATTCG  t.fq.gz  -c


Long hashtable values incorrectly wraps

I have the following generic logic:
import-module psyaml

## local variables
$my_arr = @()
$my_hash = @{}
$blahs = @('one', 'two', 'three')
$something = @(@{Name = 'xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx'})

## generate output
foreach ($someValue in $something) {
    foreach ($blah in $blahs) {
        $my_hash += @{$blah = $someValue.Name}
    }
}
$my_arr += @{'zzzz' = $my_hash}

## convert to yaml
$my_arr | ConvertTo-YAML

I end up getting output that looks like:
---

  zzzz:
    one: >
    xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
    xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx

    three: >
    xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
    xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx

    two: >
    xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
    xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx

I found the reason why the output is split into multiple lines:

https://github.com/Phil-Factor/PSYaml/blob/master/PSYaml/Public/ConvertTo-Yaml.ps1#L118-L139

Personally, I would say that an application should not bake into their program, the linter, or concepts of the linter.  This should be like a post-processing, coupled into unit tests, and things of that nature.  In this specific case, it should be up to the users, how long they want their values to be.  Then, the linter is really up to them, and not this program / module.

+1 to this. Wrapping should be both optional and configurable. And when wrapping is done, the indentation should be correct according to the YAML spec (e.g. https://yaml-multiline.info/)

+1 too. I have to use other tools to fix the multiline indentation with this library.

+1
My use case is manipulating kubeconfig  YAML files that sometimes have very long BASE64 strings embedded in them. Merely reading and rewriting these files with PSYaml breaks them.
Snippet of the initial state with 375k byte BASE64 value on one line with no breaks:
apiVersion: v1
clusters:
- cluster:
    certificate-authority-data: LS0...

Yes, 375 thousand bytes with no line break. It's a BASE64 CA cert bundle, and that's simply what those look like. Execute:
$tmp = ConvertFrom-Yaml (Get-Content '.\kube.yml' -Raw)
Set-Content -Path '.\kube_test.yml' -Value (ConvertTo-Yaml $tmp)

Renders the following (snippet) with the BASE64 value folded every 80 bytes.
  apiVersion: 'v1'
  clusters:
  -
      cluster:
        certificate-authority-data: >
          LS0...

This can't be read by kubectl because the spaces embedded in the deserialized string aren't valid BASE64 (see the multiline doc linked above for the interpretation of the '>' syntax). The correct behavior in my case would be  to simply drop the string in unchanged. There is no YAML block style (according to the link above anyway) that fits my requirements because any deserialization from block scalar format will inject unwanted characters. I need an option to not mung my input strings at all.
As it stands my workaround will be to tell users that they can't embed certificate data and will have to use the file reference syntax. And I'll tell them to find/write me a YAML support library or come here instead of complaining to me ;)


Finished search bar

Search bar is created and context structure is setup for the frontend

Also note that the scope is this PR is already hella big now, rename it to something like "Finish necessary frontend components" instead of "Finished search bar".


Shape converter not available in docker

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Is your feature request related to a problem? Please describe.
Self explanatory, Shape converter not available in docker. Many other annotation tools, softwares, etc. to import do not support masks. I need to convert these to polygons but the shape converter is not available in local docker installation.
Describe the solution you'd like
Please add shape converter to local docker installation of CVAT
Describe alternatives you've considered
I could make my own conversion tool to post process it, or download and reupload annotations. but it takes very long and is very cumbersome. Not a good experience overall. Especially since app.cvat.ai (the website) already supports this.
Additional context
No response

Right now that is enterprise feature. It might be open sourced in the future.


Revisions to harmonization draft

Following our discussion on 2023-08-16:

[x] inquire with CEDS regarding SSA Energy and Industry VOC (@gidden)
[x] propose update to historical china CO2 data (@coroa)
[x] update unharmonized data with extraction sector split and proposed updated sector mapping (@merfort)
[ ] check and verify new harmonized values and confirm that we wish to move forward with sector split (@coroa)
[ ] update unharmonized data with OAE emissions directly to their relevant sectors (e.g., transport and industry) instead of reporting as separate OAE sector (@merfort)

Mattermost discussion regarding CO2:

@gidden
Let me start a separate thread on harmonization issues - and please let me know if you would like me to put this on github as well for posterity.
I am looking at the results for the cp044-OAE_off run. Note that there are now significant discrepancies in the near term for CO2. This was not the case for the previous set of scenario results .
It looks like the primary reason for the change between versions is due to revisions in industry (further away) and transportation (previously underestimated, now closer).

@nicobauer
OK, what about the change from 2015 to 2020. REMIND increases emissions from energy and industry by slightly more than a giga ton of CO2 (1050 Mt more precisely). A look into CEDS shows for the period 2015 to 2019 an increase of 650 and for EDGAR from 2015 to 2018 an increase of 550. Industrial process increased about 100Mt in both cases, but there some tendency towards stagnation. Now a look into BP statistics indicates an increase of energy related emissions by 960MtCO2. The increases are consistent with the energy part (net of process emissions) compared with CEDS and EDGAR.
In REMIND the emissions are mostly driven upwards by the energy supply sector (about 700Mt). In CEDS the increase from 2015 to 2019 is nearly 490Mt and in EDGAR from 2015 to 2018 it is also about 480Mt.
Conclusion: there are differences in the accounting of emissions that imply differences in CO2 emissions 2015. Assuming a change in emissions from 2015 to 2020 of about +1GtCO2 seems reasonable. The increase that I see in the figure for the harmonized data seems too small.

Hi all, I uploaded a new version of the emissions (REMIND-MAgPIE-CEDS-RESCUE-Tier1-2023-10-17.csv) in the google drive folder. In this version the OAE calcination emissions are reallocated to the 5 end-use subsectors. As explained above, just to have that information, these emissions are still reported in a separate variable
(CEDS+|9+ Sectors|Emissions|CO2|OAE Calcination Emissions, I renamed it from named CEDS+|9+ Sectors|Emissions|CO2|CDR OAE Calcination Emissions since the term "CDR" is actually misleading). Please make sure to exclude this variable from the harmonization and downscaling process, since that would lead to a double accounting.
Furthermore, I reverted the split between Energy Sector|Extraction and Energy Sector|Supply emissions back so that there is now again just one Energy sector.
That should solve the last two remaing issues.
Please let me know, if there is anythings else.

Hi all,
The new version of the emissions have finally made it through our harmonization and reporting pipeline. Find the results on AWS: http://rescue-task1.3.s3-website.eu-north-1.amazonaws.com/harmonization/2023-11-03/ . I kept the split into Modelled and Non-Modelled Energy Sector emissions.
I think that should conclude this exercise.


Guidelines for security administration

When we deploy an application, we also set up a maintenance process for it, because forgotten applications can easily turn into a security risk.  The main thing we're looking at is the application's third party dependencies.  These need to be updated if they have released security patches since our application was written.
The .NET Core documentation is very good on the programming-related aspects of security, for example how to use ASP.NET Core Identity, but it doesn't give any information relating to this kind of maintenance.  Ideally there would be a list of component versions which should no longer be used, because of known security weaknesses that have since been fixed.
I see there has been a security bulletin (aspnet/Announcements/issues/203) and it has been tagged Security.  Is that now a fixed policy, so searching for uses of that tag will turn up all known security issues?  Also presumably aspnet/Announcements is only for the web-related aspects of .NET Core.  Is there something similar for other components?
I imagine this is going to be an issue for lots of companies who are making the change from older versions of .NET.  As I understand it (my background is Linux more than Windows) with older versions, there was a single package rather than multiple components, and fixes would generally be installed automatically by Windows Update.

This issue was moved to dotnet/docs#1397


Repackaging - Forcepoint CSG (MMA to AMA Migration)

Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

/Package


Allow to disable a column in a layout + time picker optional seconds

Hello, it's me again^^
I noticed that a picker must have 3 columns, but in some case you may want only two (for example in a time picker without the seconds column).
With this commit, setting a column's layout proportion to 0 will disable the column completely (without needing to do weird proportions like [100, 100, 1]).
That created an opportunity for a "showSecondsColumn" property in the TimePickerModel that would enable/disable the seconds column, so I implemented it too. :) Apparently, this is a feature that is requested by several people: #61 #45
Finally, I fixed the warnings created by my previous commit.
Have a good day,
Martin

+1 for this PR, any reason it's not merged yet?

Please merge this if validated, it's very useful

I also need this feature. We don’t need the seconds feature for setting a notification time for example.

It look great, sorry for all for my late PR review, thanks

It looks great, sorry for all for my late PR review, thanks

Great, thanks for merging that! Would it be possible to update the pub package as well? :)

Hi like this feature. Is it possible to disable it from user modification but still display the current value?


Remove prune function from smoke test

Coverage decreased (-0.05%) to 90.893% when pulling 683332a879cf5363e1b0431c0019a83f422e0118 on SAP:improve_smoke_test into b35d0c55147e2a33c09c51c250ef4b7be5f4a2b0 on cloudfoundry-incubator:master.


Add Config File Support

💡 Summary
Add the capability to specify ScubaGoggles parameters via a config file.
Motivation and context
There are some additional parameters that would be useful to add support for via a config file:

Custom DKIM selectors. GWS, unlike M365, allows you to specify custom DKIM selectors. If an agency uses a custom selector, ScubaGoggles will produce false positives for the DKIM checks, as the API does not currently allow you to determine what the selectors are.
Top-level OU. In most cases, ScubaGoggles is able to correctly determine the top-level OU, but there are certain edge cases where it can't. It would be useful to allow the users to specify the top-level OU name to cover cases where we can't figure it out.
"break glass" account names. For common controls, we assert that there should be between 2 and 4 super admin accounts, but note that exceptions are allowed for break class accounts. Unless we allow the users a way to specify the names of the break glass accounts, ScubaGoggles is actually unable to make this exception.
Groups/OUs allowed to have Sites enabled. The baseline says exceptions can be made on a per-group and per-OU basis, but currently the ScubaGoggles test fails if any OU/group is non-compliant.

There are likely other useful features we could add, but those are the ones that immediately come to mind.
Implementation notes
I recommend mirroring the syntax used on ScubaGear: https://github.com/cisagov/ScubaGear/tree/main/sample-config-files.
Acceptance criteria

[ ] Config file support is added
[ ] Config file support is documented in the readme
[ ] #347
[ ] #348
[ ] #349
[ ] #350
[ ] #351
[ ] #296

FYI that link for the syntax for ScubaGear isn't working :/


Add ClientResultSample

How about adding error completion & error handling like this ?

How about adding error completion & error handling like this ?

try catch looks good to me. But SendErrorButton from client looks confused in current scenario. Let's figure out suitable ones in other samples.


does the current openai-copatible-API support function calls?

like format
    response = client.chat.completions.create(
        model=model,
        messages=[
            {
                "role": "system",
                "content": "You are a helpful assistant that helps decide whether the customer comment violates the community guidelines. Reviews should not be offensive, abusive, or contain any personal information.",
            },
            {
                "role": "user",
                "content": "customer comment: " + content,
            }
        ],
        tools=[
            {
                "type": "function",
                "function": {
                    "name": "respond_to_comment",
                    "parameters": {
                        "type": "object",
                        "properties": {
                            "is_safe": {
                                "type": "boolean",
                            },
                        },
                        "required": ["is_safe"],
                    },
                },
            }
        ],
        max_tokens=20,
    )

Hello, at the moment, function calling isn't supported, but it's on our roadmap. It's worth noting that there are few open-source models with this capability currently available. As far as I know, fine-tuning is necessary to enable function calling.