Notifications not fired in iOs in foreground and background

Hi team, actually my app dosen't received any notification from this.fcm.onNotification() (in all states).
I tried to curl from console, try to send from firebase console and from our backend. I think our configuration is good because we have previously test with cordova-firebaseX and every notifications was received. We have migrate to your plugin because is look more lightly. I can assure we have totaly remove cordova-firebaseX and we request this.fcm.requestPushPermission();.
Project is on Ionic 4. We load fcm from ngx cause ionic version produce errors during the build (like all methods are not presents) maybe the issue of the onNotification not fired ? This is the only difference from your ionic3 app demo.
I have fcm in debug mode.
import { FCM } from "cordova-plugin-fcm-with-dependecy-updated/ionic/ngx";
Package.json
{
  "name": "",
  "version": "0.0.1",
  "author": "Ionic Framework",
  "homepage": "https://ionicframework.com/",
  "description": "An Ionic project",
  "scripts": {
    "ng": "ng",
    "start": "ng serve",
    "build": "ng build",
    "test": "ng test",
    "lint": "ng lint",
    "e2e": "ng e2e",
    "build:run:android": "ionic cordova build android && npm run run:android",
    "run:android": "native-run android --app platforms/android/app/build/outputs/apk/debug/app-debug.apk --device"
  },
  "private": true,
  "dependencies": {
    "@angular/animations": "^8.2.11",
    "@angular/common": "~8.1.2",
    "@angular/compiler": "~8.1.2",
    "@angular/core": "~8.1.2",
    "@angular/forms": "~8.1.2",
    "@angular/platform-browser": "~8.1.2",
    "@angular/platform-browser-dynamic": "~8.1.2",
    "@angular/router": "~8.1.2",
    "@cometchat-pro/cordova-ionic-chat": "^2.0.5",
    "@ionic-native/camera": "^5.18.0",
    "@ionic-native/chooser": "^5.20.0",
    "@ionic-native/core": "^5.23.0",
    "@ionic-native/crop": "^5.18.0",
    "@ionic-native/deeplinks": "^5.22.0",
    "@ionic-native/file": "^5.18.0",
    "@ionic-native/file-transfer": "^5.18.0",
    "@ionic-native/geolocation": "^5.14.0",
    "@ionic-native/google-maps": "^5.5.0",
    "@ionic-native/image-picker": "^5.18.0",
    "@ionic-native/in-app-browser": "^5.20.0",
    "@ionic-native/keyboard": "^5.24.0",
    "@ionic-native/media": "^5.23.0",
    "@ionic-native/media-capture": "^5.23.0",
    "@ionic-native/splash-screen": "^5.0.0",
    "@ionic-native/status-bar": "^5.0.0",
    "@ionic-native/streaming-media": "^5.23.0",
    "@ionic-native/video-editor": "^5.23.0",
    "@ionic/angular": "^4.7.1",
    "@ionic/cli": "^6.2.1",
    "@mux/upchunk": "^1.0.6",
    "@types/googlemaps": "^3.38.0",
    "cordova": "^9.0.0",
    "cordova-android": "^8.1.0",
    "cordova-browser": "6.0.0",
    "cordova-common": "^3.2.1",
    "cordova-ios": "5.1.1",
    "cordova-js": "^5.0.0",
    "cordova-plugin-add-swift-support": "^2.0.2",
    "cordova-plugin-androidx": "^1.0.2",
    "cordova-plugin-androidx-adapter": "^1.1.1",
    "cordova-plugin-camera": "^4.1.0",
    "cordova-plugin-chooser": "^1.2.5",
    "cordova-plugin-crop": "^0.4.0",
    "cordova-plugin-enable-multidex": "^0.2.0",
    "cordova-plugin-fcm-with-dependecy-updated": "^7.0.1",
    "cordova-plugin-file": "^6.0.2",
    "cordova-plugin-file-transfer": "^1.7.1",
    "cordova-plugin-geolocation": "^4.0.2",
    "cordova-plugin-googlemaps": "^2.7.0",
    "cordova-plugin-googlemaps-sdk": "git+https://github.com/mapsplugin/cordova-plugin-googlemaps-sdk.git",
    "cordova-plugin-inappbrowser": "^3.2.0",
    "cordova-plugin-ionic-webview": "^4.1.3",
    "cordova-plugin-media": "^5.0.3",
    "cordova-plugin-media-capture": "^3.0.3",
    "cordova-plugin-streaming-media": "^2.2.0",
    "cordova-plugin-telerik-imagepicker": "^2.3.3",
    "cordova-plugin-video-editor": "^1.1.3",
    "core-js": "^2.5.4",
    "hls.js": "^0.13.2",
    "ionic-plugin-deeplinks": "^1.0.20",
    "latlon-geohash": "^2.0.0",
    "moment": "^2.24.0",
    "ngx-chips": "^2.1.0",
    "rxjs": "~6.5.1",
    "rxjs-compat": "^6.5.3",
    "shelljs": "^0.8.3",
    "superagent": "^5.1.0",
    "tslib": "^1.9.0",
    "videogular2": "^7.0.1",
    "xml-js": "^1.6.11",
    "zone.js": "~0.9.1"
  },
  "devDependencies": {
    "@angular-devkit/architect": "~0.801.2",
    "@angular-devkit/build-angular": "~0.801.2",
    "@angular-devkit/core": "~8.1.2",
    "@angular-devkit/schematics": "~8.1.2",
    "@angular/cli": "~8.1.2",
    "@angular/compiler-cli": "~8.1.2",
    "@angular/language-service": "~8.1.2",
    "@ionic/angular-toolkit": "^2.0.0",
    "@types/core-js": "^2.5.3",
    "@types/jasmine": "~3.3.8",
    "@types/jasminewd2": "~2.0.3",
    "@types/node": "~8.9.4",
    "codelyzer": "^5.0.0",
    "jasmine-core": "~3.4.0",
    "jasmine-spec-reporter": "~4.2.1",
    "karma": "~4.1.0",
    "karma-chrome-launcher": "~2.2.0",
    "karma-coverage-istanbul-reporter": "~2.0.1",
    "karma-jasmine": "~2.0.1",
    "karma-jasmine-html-reporter": "^1.4.0",
    "protractor": "~5.4.0",
    "ts-node": "~7.0.0",
    "tslint": "~5.15.0",
    "cordova-plugin-whitelist": "1.3.3",
    "cordova-plugin-statusbar": "2.4.2",
    "cordova-plugin-device": "2.0.2",
    "cordova-plugin-splashscreen": "5.0.2",
    "cordova-plugin-ionic-keyboard": "^2.0.5",
    "typescript": "~3.4.3"
  },
  "cordova": {
    "plugins": {
      "cordova-plugin-geolocation": {
        "GEOLOCATION_USAGE_DESCRIPTION": "To locate you"
      },
      "cordova-plugin-whitelist": {},
      "cordova-plugin-statusbar": {},
      "cordova-plugin-device": {},
      "cordova-plugin-splashscreen": {},
      "cordova-plugin-ionic-keyboard": {},
      "cordova-plugin-crop": {},
      "cordova-plugin-camera": {
        "ANDROID_SUPPORT_V4_VERSION": "27.+"
      },
      "cordova-plugin-file-transfer": {},
      "cordova-plugin-file": {},
      "cordova-plugin-telerik-imagepicker": {
        "PHOTO_LIBRARY_USAGE_DESCRIPTION": " "
      },
      "cordova-plugin-ionic-webview": {
        "ANDROID_SUPPORT_ANNOTATIONS_VERSION": "27.+"
      },
      "cordova-plugin-inappbrowser": {},
      "cordova-plugin-chooser": {},
      "ionic-plugin-deeplinks": {
        "URL_SCHEME": "",
        "DEEPLINK_SCHEME": "https",
        "DEEPLINK_HOST": ""
      },
      "cordova-plugin-media-capture": {
        "CAMERA_USAGE_DESCRIPTION": "App would like to access the camera.",
        "MICROPHONE_USAGE_DESCRIPTION": "App would like to access the microphone.",
        "PHOTOLIBRARY_USAGE_DESCRIPTION": "App would like to access the library."
      },
      "cordova-plugin-media": {
        "KEEP_AVAUDIOSESSION_ALWAYS_ACTIVE": "NO"
      },
      "cordova-plugin-streaming-media": {},
      "cordova-plugin-video-editor": {},
      "cordova-plugin-enable-multidex": {},
      "cordova-plugin-fcm-with-dependecy-updated": {}
    },
    "cordova-plugin-googlemaps": {
      "LOCATION_WHEN_IN_USE_DESCRIPTION": "This app wants to get your location while this app runs only.",
      "LOCATION_ALWAYS_USAGE_DESCRIPTION": "This app wants to get your location always, even this app runs in background."
    },
    "cordova-plugin-media-capture": {
      "CAMERA_USAGE_DESCRIPTION": "App would like to access the camera.",
      "MICROPHONE_USAGE_DESCRIPTION": "App would like to access the microphone.",
      "PHOTOLIBRARY_USAGE_DESCRIPTION": "App would like to access the library."
    },
    "cordova-plugin-enable-multidex": {},
    "platforms": [
      "ios"
    ]
  }
}

Hope you can help us .

this is the error i have if i use the ionic/v4 version
ERROR in node_modules/cordova-plugin-fcm-with-dependecy-updated/ionic/v4/FCM.d.ts:2:13 - error TS1005: '=' expected.

2 import type { IChannelConfiguration } from '../../typings/IChannelConfiguration';
              ~
node_modules/cordova-plugin-fcm-with-dependecy-updated/ionic/v4/FCM.d.ts:2:44 - error TS1005: ';' expected.

2 import type { IChannelConfiguration } from '../../typings/IChannelConfiguration';
                                             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
node_modules/cordova-plugin-fcm-with-dependecy-updated/ionic/v4/FCM.d.ts:3:13 - error TS1005: '=' expected.

3 import type { IRequestPushPermissionOptions } from '../../typings/IRequestPushPermissionOptions';
              ~
node_modules/cordova-plugin-fcm-with-dependecy-updated/ionic/v4/FCM.d.ts:3:52 - error TS1005: ';' expected.

3 import type { IRequestPushPermissionOptions } from '../../typings/IRequestPushPermissionOptions';
                                                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
node_modules/cordova-plugin-fcm-with-dependecy-updated/ionic/v4/FCM.d.ts:4:13 - error TS1005: '=' expected.

4 import type { INotificationPayload } from '../../typings/INotificationPayload';
              ~
node_modules/cordova-plugin-fcm-with-dependecy-updated/ionic/v4/FCM.d.ts:4:43 - error TS1005: ';' expected.

4 import type { INotificationPayload } from '../../typings/INotificationPayload'

I can confirm that the same its happening for me. The onNotification callback is not called when the app is closed/killed and tapped from the device tray (iOS) :/

@alejandrombc and others if your are in this case. I have remove all the cordova build and plugins. Then close xcode, delete app and restart phone tu clean cache and local storage. Then create a fresh new build and deploy. This loooong process have help us to solve !

I also had a kinda similar problem, I think it’s something to do with signing process while this plugin is installed. Everything worked  perfect till I removed the app from the ios device, then it stopped working until I used another signing certificate then it worked again well today I removed the app again and installed with same build again it stopped working, do you think your problem is the same in the roots ?

@EmreAkkoc If i understand you change the signing certificate, but have you generate the right key with this certificate in your apple developper and firebase account ? I think it's can explain the difference between both builds.

The fact is I didn’t  change a thing, just removed the app And it stopped working, then I auto signed with xcode and it started again, it’s really hard to reproduce since I am able to get token and all other functionality is intact

@EmreAkkoc the problem you are facing doesn't look like something related to the plugin. So not sure how to help.

this is the error i have if i use the ionic/v4 version
ERROR in node_modules/cordova-plugin-fcm-with-dependecy-updated/ionic/v4/FCM.d.ts:2:13 - error TS1005: '=' expected.

2 import type { IChannelConfiguration } from '../../typings/IChannelConfiguration';
              ~
node_modules/cordova-plugin-fcm-with-dependecy-updated/ionic/v4/FCM.d.ts:2:44 - error TS1005: ';' expected.

2 import type { IChannelConfiguration } from '../../typings/IChannelConfiguration';
                                             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
node_modules/cordova-plugin-fcm-with-dependecy-updated/ionic/v4/FCM.d.ts:3:13 - error TS1005: '=' expected.

3 import type { IRequestPushPermissionOptions } from '../../typings/IRequestPushPermissionOptions';
              ~
node_modules/cordova-plugin-fcm-with-dependecy-updated/ionic/v4/FCM.d.ts:3:52 - error TS1005: ';' expected.

3 import type { IRequestPushPermissionOptions } from '../../typings/IRequestPushPermissionOptions';
                                                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
node_modules/cordova-plugin-fcm-with-dependecy-updated/ionic/v4/FCM.d.ts:4:13 - error TS1005: '=' expected.

4 import type { INotificationPayload } from '../../typings/INotificationPayload';
              ~
node_modules/cordova-plugin-fcm-with-dependecy-updated/ionic/v4/FCM.d.ts:4:43 - error TS1005: ';' expected.

4 import type { INotificationPayload } from '../../typings/INotificationPayload'

As your log suggested, your typescript version was not supporting import type, which was replaced by just import on version 7.0.2.


Accessibility issue when click tab key

Short description
I am doing accessibility check on my site, I found when the geosuggest input is focused if I press tab key, it will trigger onSuggestSelect event which will cause update on site's content.
Expected results
I expected that by using keyboard key tab on input field, it should not trigger and functional events.
Actual results
The onSuggestSelect event is triggered when click tab key.

Reproduction steps

Go to geosuggest input field
write console log in onKeyDown and onSuggestSelect event
Click on tab key from your keyboard
observe the browser console
You will see onSuggestSelect is called when press tab key.

Additional Information

Reporting this as a problem as well. We use this in a form with redux-forms and it's causing the value of the input field to be lost. is there anything we can do besides disabling TAB ? I'd like to keep the site accessible.

In fact is not a bug but a feature. It's explained in the docs: https://github.com/ubilabs/react-geosuggest#ignoretab

@gorkapit thanks for the finding.

Close this since it is a feature designed.


Permission Denied

Hello!
I get a permission denied error in Windows 10 running the Installer script:

Any advice?
Thanks!

Problem solved! Can't run the Installer from inside of the zip file - it has to be extracted first.


Mesh.MergeMeshes does not work correctly with multiple materials

If you create two meshes (e.g. Mesh.CreateBox) and they each have a different material, and then you merge them with Mesh.MergeMeshes([box1, box2], true, true, undefined, undefined, true) the geometry merges as expected, but you end up with a single material which looks kind of like one of the original materials but without the alpha (or something like that). Same thing works fine in the Babylon.js Playground.

Can you provide a playground so I can test it quickly?


Use of set to test uniqueness of grids in wrap_func_objects results in unwanted behavior.

To create a list of unique grids, set is used
https://github.com/Deltares/xugrid/blob/e74fbc1ef44da0596499476ae4b7309e2ab37c99/xugrid/core/common.py#L101
However, this tests if grids refer to the same object, which results in behaviour like this:
import xugrid as xu

a = xu.data.disk()
grids = xu.concat([a, a], dim="foo").grids
print(len(grids)) # prints 1

grids2= xu.concat([xu.data.disk(), xu.data.disk()], dim="foo").grids
print(len(grids)) # prints 2

We want to test if grids are equal, as to be fixed in this issue:
https://github.com/Deltares/xugrid/issues/205

Adding to some of the things in #205
If objects are equal, but not exactly (i.e. one has a celltree initialized, but the other doesn't), in general you'd want the richer object. But there's no clear definition of that, so I'd say: just take the first if there's many.


Finalise the API for konig::io::fen::FenData.

There are two basic concerns here.
Primarily, the parser API on FenData still returns a nom::error::Error as the Err variant, and it would be preferable to return a FenParseError instead. This probably means digging down into the nom error system and learning how to make the conversion. Potentially, the parse_fen_string function could be modified so that its return type is nom::IResult<&str, FenData, FenParseError>, but I'm unsure if that would brick the type inference in that function.
The other, slightly less important point is what the return type of FenData.as_board should be. If it just returns an impl Board, then I can hide big chunks of detail and keep FenBoard a private struct. Otherwise, it could return either StandardBoard or FenBoard, but how coupled should it be to the standard implementation?
Admittedly, if I follow #11 and add a method to Board to get the Option<Piece> at a particular Index, then I can just stick the simple impl Board return type, which is my general preference.

After some very painful type-wrangling, I've made FenData's as_board method return an impl Board rather than a concrete FenBoard; this also allows me to make FenBoard private again.
The full signature is this:
fn as_board(&self) -> impl Board<Piece = StandardPiece, Index = StandardIndex>

This is nice, but I do have a few concerns. Firstly, this is still pretty closely coupled to the konig::standard implementation, and I think I'd prefer to use something like a FenPiece and FenIndex where I can give them From<StandardPiece> and From<StandardIndex> impls.
For now, those types would probably just end up wrapping StandardPiece and StandardIndex directly, but the point is the separation. The konig::io::fen implementation should play nice with konig::standard, but it doesn't have to depend on it, and the API should making as few promises as possible.

Ok, top-level TODOs for this issue:

Move to FenPiece and FenIndex types, and totally hide the API-level dependency on konig::standard.
Implement the nom::error::ParseError trait on FenParseError, and then put it into the parse_fen_string's return type.


V0.28.0rc0 /keys routes are accessible with admin key

Describe the bug
As per the specification

the endpoint /keys is accessible for the master key holder

Nonetheless it is accessible with the Admin key as well.
To Reproduce
Steps to reproduce the behavior:

Create an instance of meilisearch protected by password
Fetch the admin key
make a request on GET /keys (or POST)
Should throw an invalid API key but instead it works

Expected behavior
Admin key should not have access to this route
Meilisearch version:
v0.28.0rc0

Thanks @bidoubiwa
I don't know if it's already the case but we need to add tests, checking the routes are only accessible with the right key, since it can lead to security issues 🔥 🔥

Admin API Key should have access to it since the defined actions is *.
I did not find any mention in the specification that this should not be the case.
I forgot to update this sentence "Default Admin API Key authorizing all actions on all indexes (except managing /keys resource)." 😫

Ok, thank you @gmourier, I let you update the spec! Thanks, I close this issue
Sorry @bidoubiwa for the confusion!

I think the specification is not up to date yet:

When a master key is set at Meilisearch first-launch, it generate two pre-configured default API Key resources. A Default Search API Key authorizing the search action on all indexes and a Default Admin API Key authorizing all actions on all indexes (except managing /keys resource).

It has been updated https://github.com/meilisearch/specifications/pull/148/files (The spec is not merged on develop yet)


refactor Alert component

Completed the refactoring of the Alert component with Vue 3 and wrote the test file.

@shengxinjing   Can I contribute code to your code repository?

thanks

多写代码提高自己，哈哈哈哈嗝


Use an atomic operation to calculate Issue.Index

Please notice that I've referenced the master version of xorm (xorm.io/builder,github.com/go-xorm/xorm) to build this. I'm not sure if I did it correctly.
Closes #7887
This PR takes advantage of the new feature supported by xorm of building the INSERT from a SELECT.

A thread with an alternate solution and its discussion is: #7898,

Codecov Report

Merging #7944 into master will decrease coverage by <.01%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##           master    #7944      +/-   ##
==========================================
- Coverage   41.46%   41.45%   -0.01%
==========================================
  Files         478      478
  Lines       63966    63974       +8
==========================================
  Hits        26521    26521
- Misses      33991    33997       +6
- Partials     3454     3456       +2

Impacted Files
Coverage Δ

models/issue.go
50.03% <55.55%> (+0.07%)
:arrow_up:

modules/process/manager.go
76.81% <0%> (-4.35%)
:arrow_down:

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

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 9f58187...f224ece. Read the comment docs.

I've just noticed that xorm re-reads the calculated value for me, so calling getIssueByID() is unnecessary. I'll update this PR in a minute.

*Rebased due to changes in go.mod at master.

I haven't tested it but does it select the max index for the specific repo id or across all repo ?

Something wrong with the issue/comment template? Mmm....

On a side note: the concurrency problem exist also for PR since they used GetMaxIndexOfIssue. But first let validate this PR. :smile:

On a side note: the concurrency problem exist also for PR since they used GetMaxIndexOfIssue. But first let validate this PR. 😄

@sapk In fact, that value is being ignored (it always was). I might PR for that later.

@typeless , @sapk, @lunny
The good news is that I've managed to create a test that can make this fail.
The bad news is that I've managed to create a test that can make this fail.
See #7959.
Sadly, for solving our duplicate index problem, the INSERT ... SELECT method only works with SQLite. All MySQL, MSSQL and PostgreSQL fail one way or another:
MSSQL: insert successfully but not returned id (internally it's Cannot insert duplicate key row in object 'dbo.issue' with unique index 'UQE_issue_repo_index'.).
MySQL: Deadlock found when trying to get lock; try restarting transaction.
MySQL8: Deadlock found when trying to get lock; try restarting transaction.
PostgreSQL: pq: duplicate key value violates unique constraint "UQE_issue_repo_index".
With MySQL, a lock is in place but somehow creates a deadlock. With MSSQL and PostgreSQL, if there's a lock, it is not kept from the SELECT part into the INSERT part.
So, we're back to square one regarding this, and the only viable solution at the moment seems to retry the insert, as @lunny proposed.

Close due to failing tests on #7959

Did you try SetExpr with full select max?

Did you try SetExpr with full select max?

@lafriks Yes, that's #7959. I've branched this to add the test. I didn't want to add the test here for performance reasons (the integration tests would have a lot of time added to them for everybody).

I don't that will only supports sqlite3 because I have added tests on https://github.com/go-xorm/xorm/pull/1401 and it will be executed on sqlite/mysql/postgres/mssql both on circleci and drone.
https://cloud.drone.io/go-xorm/xorm/67 and https://circleci.com/gh/go-xorm/xorm/952?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

I don't that will only supports sqlite3 because I have added tests on go-xorm/xorm#1401 and it will be executed on sqlite/mysql/postgres/mssql both on circleci and drone.

@lunny I'm sorry, of course the statement works well in all databases. Your modification generates the right code and it passed all the existing tests on Gitea.
However, it does not fix #7887. It does not work if multiple users attempt to insert at the same time. There are all kinds of concurrency problems. Please see my test in #7959 for details.


For v1.0.29. Direction keys in cropping mode to move selected edge/corner around one pixel at a time. Cropping in general need to make image space. The current screen-space idea doesn't work well in practice at all.

Image-space is done. Still need to do direction keys.

Keep track of last selected line(s) and colour anchor differently so we know which one we're moving with the arrow keys.

a51bffb8120ea7c223bf4c522f39709b7f24cf8f


Stop swiping

Hi,
I search a function for stop swiping, because i have map on carousel so i need to navigate in it. I see u have said we can stop swiping with add one by one and function backwar/forward, but ur exemple not work now , can u have an idea for do that?
Thanks in advance

To stop "touch to swipe" set IsSwipingEnabled to false.

If my work is helping you, please help me back: https://xamarinhq.wufoo.com/forms/nominate-a-xamarin-mvp/
This is what i have done that is community visible:

Alex has been pretty active sharing his knowledge and answering questions in StackOverflow, GitHub and Xamarin Forums.
Alex has been open sourcing Xamarin components and plugin with the community, with more of 10 repositories being maintained on GitHub.
One of Alex's Xamarin.Forms plugin, the CarouselView, is currently being pulled into the Xamarin.Forms core (https://github.com/xamarin/Xamarin.Forms/pull/853). As an stable widget, this control has become the standard for Xamarin.Forms carousel screens with more that 16K downloads in NuGet and more than 100 solved and not open issues.
Another Alex's plugin, the SegmentedControl (https://github.com/alexrainman/SegmentedControl), is a big candidate to be also pulled into Xamarin.Forms core.
Plus Xamarin.Forms UI related plugins, Alex shares plugins to set app icon Badge, Google Analytics and KeyChain access.
Alex shares his knowledge about cutting edge technologies like Couchbase NoSQL and MobileIron integration with Xamarin Apps.


When in use, a deadlock may occur.

After entering the API key and saving it, clicking the generate button will redirect back to the API key input.

The problem was due to OpenAI's recent API change for the gpt-3.5-turbo model. However, since the application is not a chat, I switched back to the "text-davinci-003" model. The issue should now be resolved.


Revert "Revert "Use HTTP and remove Athenz injection""

Was reverted due to mergestop
Reverts vespa-engine/vespa#6103

Superseded by #6129


Exploit Vulnerability and Foxfire

I have been trying to work out why exploit vulnerability wouldn't work with foxfire and found this in the console.
[PF2E Exploit Vulnerability] - Unable to determine damageType of Foxfire. Defaulting to Physical.
Any chance this could be fixed. Thank you.

This has been fixed for foxfire specifically in the latest release... I'm unsure as to whether it works for other strikes granted from feats.


Improve AppVeyor and Travis CI build time

Let's try to make AppVeyor take less time to finish the builds.
Note: I renamed the config file to .appveyor.yml for consistency with Travis, now both filenames start with a dot.

now it's failing :D

Well that's not a huge success for now 😅

I wonder why 5.x job tooks 20+ minutes ...

@keradus If I ever succeed at improving AppVeyor build time, it will only be one or two minutes. The longest step is the PHPUnit test suite itself so it looks like PHP-CS-Fixer has a real performance issue on Windows.

I really don't understand why AppVeyor won't restore Composer cache. Is it something that must be enabled on AppVeyor side?

The only reason I could see is that your file is not in use.
AppVeyor default config file is appveyor.yml. You have renamed it.
One may use custom filename, but first he has to set it up in settings on web. We do not configure anything in CI services directly (keeping whole configuration in file).
Try to un-rename the file.

Thats it
in different PR, we have two builds (2 different PHP versions): https://ci.appveyor.com/project/keradus/php-cs-fixer/build/dev-5016
in this PR, we have only one: https://ci.appveyor.com/project/keradus/php-cs-fixer/build/dev-5022

@keradus This is because I commented out the PHP 5.x test for now, just to make the whole build faster :) This proves the new filename is taken into account though.

:+1:
5.x is indeed that slower on Windows, Idid just run it.
On unix it's 2x slower, on Windows.. argh. Slow part is linting. Please set up SKIP_LINT_TEST_CASES=1 for PHP 5.x

I don't see any config for cache on AppVeyor account settings. Please play with it.
also, you could install hirak/prestissimo globally like we do in Travis, it should increase speed a bit.

Looks like we have no choice but merging the PR to see if the cache works: http://help.appveyor.com/discussions/problems/4048-build-cache-not-restored#comment_39042560.

Wow, definitely faster with SKIP_LINT_TEST_CASES=1!

I don't see any real improvement with hirak/prestissimo but downloading it does take time, not sure it's worth it.

just before prestissimo: https://ci.appveyor.com/project/keradus/php-cs-fixer/build/dev-5023
just after: https://ci.appveyor.com/project/keradus/php-cs-fixer/build/dev-5024
about 14s boost per job

@keradus Installing hirak/prestissimo takes more than 14s. But if the cache works, it should not on subsequent builds so 👍

Installing hirak/prestissimo takes more than 14s.
And yet, overall time of whole build is 14s faster, and that includes time of installing prestissimo. Prestissimo parallelize installing of packages, and we ask for installing 44 packages...

I would also like to try to add C:\tools\php -> .appveyor.yml to the cache items and install PHP only when the directory does not exist. But I'm not sure whether this will speed up builds or just break everything 😀. Shall I do it?

what about 5.x vs 7.x ?
currently, they are installed in same dir.
Won't it mix the cache? or cache is stored per job, not per build ?

(btw, don't cache whole C:\tools\, there were other autoinstalled apps there as well)

I couldn't find information about whether cache is per job or per build. I hope it's per job, otherwise we obviously can't use it for PHP, and we might even face issues with Composer. So what do we do?

If you are willing to play with it on some private playground repo it would be great.
If not, let us not include php dir into cache - as a safe approach.

Sure, I'll see what I can do with my fork of the project.

Good news: cache works fine and is job dependent so caching PHP installation is not an issue, see builds on my fork.
I made more tests with and without hirak/prestissimo and it actually makes the build take more time when Composer cache is available:

in this job: installing the plugin took 16 seconds, updating the project dependencies took 22 seconds: total is 38 seconds;
in this job: no plugin installation, updating the project dependencies took 25 seconds.

Do you still want to use the plugin anyway?

Great to hear that you found some time to play with it ;)

Do you still want to use the plugin anyway?

What if we would cache both:

'%APPDATA%\Composer'
'%LOCALAPPDATA%\Composer'
?

It won't change anything: on master %APPDATA%\Composer is used and Composer never retrieves vendors from cache so I think the directory is never used at all and AppVeyor just stores an empty cache for it.

I let myself modify a bit config for debugging.
I restored the Roaming caching and see the cache - sadly, as you pointed, it's never used so it's empty as we didn't install anything globally on master yet.
But my point is we could follow the approach you made with php package. composer g show -ND will tell us if there is already plugin installed or not, if not - we could install it, if yet, we won't.

Sure, will try that.

I think we're good now :)
%LOCALAPPDATA%\Composer% is used for cache while %APPDATA%\Composer is used for everything else so both are required.

could we apply same approach for composer plugin for Travis ?

Great work here, awesome to see AppVeyor fast finally!

Thanks for your suggestions and help!


[Fix?] Attempt to unstuck Hamming exercise

In an attempt to unstuck the Hamming Distance exercise I'm adding an empty new line at the end of the test.sh file.
Maybe pushing an update will fix and unstuck the exercise from not being able to update.

I can't see anything amiss there


Scheduled backup not work correctly

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
Scheduled backup not work correctly, only two backup are present in gdrive.
If i run moretime in shell configctl system remote backup, only two backup is created.
I set in gui 60 backup retention.
In gui, if i press "setup/check google drive" , opnsense create corretly 60 backup
It seems that the shell command does not read the retention parameter and always sets it to 2
Tip: to validate your setup was working with the previous version, use opnsense-revert (https://docs.opnsense.org/manual/opnsense_tools.html#opnsense-revert)
To Reproduce
Steps to reproduce the behavior:
Setup gdrive backup
Setup cron
Only one backup is present in gdrive after some days
Software version used and hardware type if relevant, e.g.:
Expected behavior
A complete backup chain
Describe alternatives you considered
A clear and concise description of any alternative solutions or workaround you considered.
Screenshots
If applicable, add screenshots to help explain your problem.
Relevant log files
If applicable, information from log files supporting your claim.
Nothing is logged
Additional context
Add any other context about the problem here.
Environment
Software version used and hardware type if relevant, e.g.:
OPNsense 21.7.2_1-amd64
vmware 7.1

I hadn't read about this feature, great thanks!


Parkings overlap highways, hiding them

If a parking area is mapped, all the highways passing throughout it are not shown.

#11 was faster by 2 minutes 🥷


Dev compute microsoft.compute 2019 05 01 preview

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

Automation for azure-sdk-for-java

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/5502
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/5502/commits/7bc603cb312ffecd6931243ce724b84b9408918d

@NarayanThiru can you please rebase on top of latest master?

startbuild

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/5809
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/5809/commits/0ef26a4978b69a2cdcde6184250b5432250cfaa9

@NarayanThiru will close by EOD if no update.

Closing due to inactivity.  Please reopen when ready to engage.


Improve exception handling in example code

The exception handling in the documentation and example code always catches std::exception.  For the purposes of documenting the behavior of the library, it would be better (in my opinion) if the code in the documentation and example programs caught the specific exceptions expected.
For example, rather than catch std::exception like this
try
{
    char i = -1;
    unsigned char j = safe_cast<unsigned char>(i);
}
catch(const std::exception&)
{
}

Do this instead:
try
{
    char i = -1;
    unsigned char j = safe_cast<unsigned char>(i);
}
catch(const std::out_of_range&)
{
}

Doing this will help to document what the actual expected exceptions are expected. If multiple exceptions are expected, then in my opinion, the example code and documentation code should have a catch for each possible exception.

Thanks for your feedback.  I'm currently making changes required to fix bugs and rough edges as highlighted in the boost review.
The error policy design as it is now isn't quite right so it's in the process of being updated.  I'll take a look at this when I make these changes.

The problem I see with this is that there is a wide variety of exception types which could be thrown.  The current method works fine and I don't think anything would be added by making this change so I'm not going to do it.
Note that this applies to the test/example code.  There's no reason why someone who needed this facility shouldn't feel free to do it.  maybe the best would be a documentation note.


Segment properties

Hi there, I've recently begun using @segment/analytics-node. It appears that the track method no longer includes segment properties such as userAgent, userAgentData, page details, and others.
When using this library on the client-side, these events are automatically populated by segment. However, I'm concerned that by switching to the node analytics SDK, we might lose access to these valuable pieces of information.
Any thoughts or ideas on how these properties can be sent further by using @segment/analytics-node?

You would need to pass those properties explicitly; node is a server side SDK, those properties can't be plucked out of the environment like they do in the browser.

Thank you for your time and assistance @silesky ; it means a lot to me to have this confirmation. I'm sending now the IP and userAgent through Node.js. Now, I'm contemplating moving the "identify" and "group" methods to the server-side, but I'm concerned about potentially losing the funnel. The script tag from Segment is placed on the "site page", "authentication page" and "application" where's the main app (game). If I move all the methods to the server-side on the application page, are there any drawbacks like losing the funnel or causing Segment to merge identifiers? It's important to note that on the site page, the user is not logged in is just a simple presentation page where we have the segment script, and there are only limited details available. Only when the user logs in and reaches the app, some details about them are accessible; until then, they are merely marked as a unique visitor by Segment.


PiKVM Version

Which version of the PiKVM is this based on? I love this thing it looks awesome.

v3-hdmi-rpi4-latest.img is OK
At 2022-02-04 11:06:55, "Nick Leffler" @.***> wrote:
Which version of the PiKVM is this based on? I love this thing it looks awesome.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I can confirm—everything works out of the box with the latest version of Raspberry Pi 4 image (following the Pi-KVM guides to do things like access the unit and enable the OLED display):

It mostly works, except RTC wakeup (log spam) and fan control. Those only work on PiKVM v3, for which the software was written.
As those issues don't affect PiKVM v3, the team #wontfix them.

It mostly works, except RTC wakeup (log spam) and fan control. Those only work on PiKVM v3, for which the software was written.
As those issues don't affect PiKVM v3, the team #wontfix them.

Fan control has been added, see discord
BliKVM discord @https://discord.gg/dwx7vKYt
https://discord.com/channels/943534043515977768/943536121365794866/1001152437912080504


Customizable table footer for labelled column subtotals and totals

Hi all.
Does elem-ui table support a customizable multi-row footer ?
Usually report tables have footer showing subtotals and totals based on columns, along with optional 'labels' beside the subtotal or total number value.
Or maybe table element has a named slot for footer ? I'm not sure, because i cant see it in the component guide or in table.vue.
Requesting a new feature if it's not already supported.

Hi im quite new here. Why isnt the original post listed in the issues section ?

Duplicate of #3164.
We are considering adding this feature. Please see #3164 for updates, though you may need Google translate.

Yay, very good news indeed. Yes i told by vue guys in gitter that 3164 exists, and i voted the feature request already (after a google translation indeed).
Thank you for the great library !


Implement "swipe to refresh" gesture

Implemented according to https://developer.android.com/training/swipe/add-swipe-interface.
Closes #152

@lberrymage Can you make sure it's not possible to trigger a refresh while there's already any syncing, etc. happening? I don't see anything preventing that right now.

already merged via #171


Implement support for read-only mode for Input

Add readonly prop support for Input

readonly prop for readonly attribute with true string value

Write test

Codecov Report

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

@@          Coverage Diff          @@
##           master   #465   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          43     43
  Lines         398    399    +1
=====================================
+ Hits          398    399    +1

Impacted Files
Coverage Δ

packages/core/src/Input.js
100% <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 a9a7e6c...ea25612. Read the comment docs.

Thanks for pointing out the readOnly in camel case can be applied to existing Input. Closing this PR.


menudrawer with Toolbar

Since last updates of support libraries i need to use Toolbar instead of ActionBar.
Toolbar with custom layout can be initialized when i use setContentView on Activity and main problem that Toolbar is a part of content, but need to open my VERTICAL TOP menu below Toolbar.
How can it be possible?

Android native drawer is much better now


Support localized TTS Say

Hello,
Anki Python SDK has been unofficially updated to 0.7.2.
This new version allow now to make Vector speak in other language than English. It add the same international languages than Cozmo : French , German , Japanese and of course English. Acapella data for these additional languages were already present in the Vector system partition but not accessible by the Python SDK ... until now !👍
https://github.com/ikkez/vector-python-sdk/releases

Thanks. I will look into that soon. I need to re-organize myself before I relaunch my work on VectorC2.


Make directory creation deterministic

Prior to this commit, the layout of created ext4 filesystems created using tar2ext4 wasn't
deterministic. Recursive directory creation would change from run to run due to the usage
of a map as temporary storage for items from the tar stream.
This commit changes recursive directory creation to follow the same deterministic pattern
as writeDirectory.
At this point, the only possible source of non-determinism in a create file system is the
UUID in the VHD footer. If no VHD footer is included, then the results are deterministic.

@SeanTAllen Were you going to change the UUID generation also in another PR? What's the plan for this, a hash?

I believe that is still a topic for conversation. I will do in another PR once a decision is made.


Missing #include <limits>

When compiling MIOpen with OpenCL backend (release 4.3.0) on Arch Linux with GCC 11 I get several build errors like
MIOpen/src/include/miopen/float_equal.hpp:47:39: error: no member named 'numeric_limits' in namespace 'std'
               std::nextafter(x, std::numeric_limits<T>::lowest()) <= y

This is fixed by adding #include <limits> to float_equal.hpp:
diff --git a/src/include/miopen/float_equal.hpp b/src/include/miopen/float_equal.hpp
index e2441b60c..bd816cb22 100644
--- a/src/include/miopen/float_equal.hpp
+++ b/src/include/miopen/float_equal.hpp
@@ -28,6 +28,7 @@

 #include <algorithm>
 #include <cmath>
+#include <limits>
 #include <numeric>
 #ifdef _MSC_VER
 #include <iso646.h>

@tpkessler Thanks! It seems that GCC 11 started to explicitly require including <limits>, <memory>, <utility>, <thread>, according to gcc-11_header-dep-changes, it might have happened to clang too.
It might should have been fixed for most header files while this one is missed: have we only recently started testing "MIOpen with OpenCL backend (release 4.3.0) on Arch Linux with GCC 11"?


[FEATURE]: Support the Batch API on the Neon adapter

Describe what you want
In @neondatabase/serverless handle transactions like this
import { neon } from '@neondatabase/serverless';
const sql = neon(process.env.DATABASE_URL);
const showLatestN = 10;

const [posts, tags] = await sql.transaction([
  sql`SELECT * FROM posts ORDER BY posted_at DESC LIMIT ${showLatestN}`,
  sql`SELECT * FROM tags`,
]);

Currently, Drizzle does not support transactions on neon adapters. However, if Drizzle supports batch on neon, it can bridge this gap.

Postgres.js also implements a very similar API


Add support for Python 3.9 and 3.10

Issue summary
Gym Retro is a great project that allows to use classic games from consoles like Sega Genesis or SNES for machine learning. Currently it only supports Python 3.8 which is a problem because even conservative distros like Debian stable are already using Python 3.9.
Please add support for Python 3.9 (current Debian stable) and Python 3.10 (next stable)

I wouldn't say this is a problem since you can easily ignore your system's python version by creating virtual environments (which is a good practice too). May I suggest using pyenv with poetry ?

Thank you, I will try it :)

In my case, my script runs with Python 3.10, so would be great to have 3.10 support.
I already tried to downgrade my libs to Python 3.7, but is a major headache, so it would be way simpler to just add 3.10 support to stable-retro.

+1 to what @mateusscheper said.


Ocelot no longer working over HTTPS after certificate renewal

After renewing my LetsEncrypt-SSL-certificate using certbot (DNS acme challenge) and restarting my Ocelot gateway (docker container) the gateway is no longer responding over HTTPS.
Expected Behavior
The new SSL certificate is used after replacing the certificate and restarting the docker container. The gateway responds on port 443 with the new certificate.
Actual Behavior
Replacing my SSL certificate after renewing (manually) via LetsEncrypt has led to my gateway no longer responding over SSL. If I remove HTTPS redirection (app.UseHttpsRedirection()) then the gateway is accessible over HTTP, but not over HTTPS.
However, as I'm using a .dev domain, HTTPS is required (when using a browser).
Responses (UseHTTPSRedirection OFF):
curl vanhoof.dev/api/project/penny -> JSON response from my API
curl https://vanhoof.dev/api/project/penny  -> OpenSSL SSL_connect: SSL_ERROR_SYSCALL in connection to vanhoof.dev:443

wget vanhoof.dev/api/project/penny -> JSON response saved from API
wget https://vanhoof.dev/api/project/penny -> GnuTLS: The TLS connection was non-properly terminated. Unable to establish SSL connection.

Firefox https://vanhoof.dev/api/project/penny -> PR_END_OF_FILE_ERROR
Chrome https://vanhoof.dev/api/project/penny -> ERR_CONNECTION_CLOSED

adding -v to curl https shows:
* TCP_NODELAY set
* Expire in 200 ms for 4 (transfer 0xb88880)
* Connected to vanhoof.dev (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*   CAfile: none
  CApath: /etc/ssl/certs
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* OpenSSL SSL_connect: SSL_ERROR_SYSCALL in connection to vanhoof.dev:443
* Closing connection 0
curl: (35) OpenSSL SSL_connect: SSL_ERROR_SYSCALL in connection to vanhoof.dev:443

Turns out this was my own error.
I had converted the certificate from .pem to .pfx by using:
openssl pkcs12 -export -out MYSSLKEY.pfx -inkey privkey.pem -in cert.pem -certfile fullchain.pem
However, this created an 'invalid' key. The solution was/is to abstain from using the chain cert. ie:
openssl pkcs12 -export -out MYSSLKEY.pfx -inkey privkey.pem -in cert.pem
Using the above command and restarting the Ocelot container with the new .pfx-key fixed the issue.


Research for new PM tool

Since waffle is shutting down we should make a research on other tools and pick one:
https://www.openproject.org/
https://www.zenhub.com/
https://help.github.com/en/articles/about-project-boards

Here is the result of the research:
https://hackmd.aepps.com/GwMxHYE4A4FZILQFMm2AgLEgjI6AmABmgRGOmmEkgCMpIg==?both


Add information for Rust, Option and prost

Additional information regarding why Option is used for protobufs and prost.

I re-wrote the paragraph you provided and posted it above. Is it ok? Or should I expand more?

@seanmooretechwriter Can you introduce your changes in the files directly, I'll perform a last review after that. Now it's hard to see what your new content will replace in the full new paragraphs.
Let's try to avoid comments for potential updates, it's preferable to see them in the the markdown files directly so I can comment line by line.

I consolidated, edited, and ported the content into the file, committed the changes, and pushed.

I'll work purely in the files from now on. Sorry for any trouble.


Add Ztunnel values API in the operator values_types.proto

This PR adds ZtunnelConfig API in the operator values_types.proto. It helps us import the new API struct into our own CRD. All of the new struct fields are from the istio/manifests/charts/ztunnel/values.yaml.
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[X ] Ambient

Hi @yxun. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Note that we are moving away from validating values.yaml against a proto, as

We don't ship an operator anymore
values.yaml is the source of truth, and not the proto
helm template is the validation source of truth, and not the proto
keeping the two in sync is a largely pointless chore.

Hello, Ben
I see your points. We have a tool script which is syncing all Istio community operator apis values.

None of this stops you from maintaining your own proto version of a given chart's values.yaml for your operator going > forward - but we don't intend to, AFAIK, so I don't see the point of this - values_types.proto is effectively an API for a
component we no longer ship, and should be considered deprecated.

Yes, I have added our own values types for our operator. However, that introduced a gap between community Istio and our operator values types. I will check if we still need this upstream change or if we will update our tool script. Thanks.


mechanism to merge shotgun features after atropos but before shogun

To ease EBI submission and make it at the actual experimental level, we may submit shotgun data to Qiita per lane (i.e. one prep per sequencing run and lane in order to have submitted the data as it comes out of the sequencing instrument).
Yet, for some projects, high/desired sequencing depth might only be achieved by running several run/lanes. Hence, one would have to uploaded >1 prep (i.e. >1 sequenced sample; >1 fastq files pair) per biological sample.
The problem is that for now and as far as know, it is only at the biom table level that sequences can be merged in Qiita (through the Analysis functionalities, isn't it?), whereas to harness the full power of Shogun, it is much better to pass it all the reads of a biological of interest, i.e. the replicates that may be scattered in various preps (thanks @qiyunzhu  for the insight here!).
Therefore, a mechanism to merge features amongst Qiita preps after Atropos (or at any step during the shotgun workflow) may be very useful.
Thanks! F

@antgonza and I just discussed. We believe that:

The BIOM tables generated by Bowtie2 are completely mergeable (i.e., the merged table is identical to the table generated from merged data).

The BIOM tables generated by BURST under the "capitalist" mode are not perfectly mergeable. This is because "capitalist" optimizes the resulting profile by minimizing the number of species reported when there are ambiguities. This design is thought to be able to reduce false positives. Although the difference between a merged table and a table from merged data is not significant.

The BIOM table generated by BURST without the "capitalist" mode are perfectly mergeable.

Thanks @qiyunzhu.
@FranckLejzerowicz, summarizing: Qiita relies on options that generate the same answer if the raw sequences were processed together or separately and we plan to keep doing that for the future AKA no need to merge raw sequences, let's just use the resulting BIOMs.


Develop 0.1.2

Release 0.1.2

Changes Unknown when pulling 4a6aaebd1b474aa6e0feafd774a6f2c755db4bbc on develop-0.1.2 into ** on master**.

Changes Unknown when pulling dc25a589dde8a7a3a99d005e94616527ab573c6c on develop-0.1.2 into ** on master**.


Create a public mdw-buildpack repository on GitHub

Hopefully under CenturyLinkCloud.

https://github.com/mdw-dev/mdw-buildpack was created


Define FitResult class

Some thoughts:

The dict that is currently returned is not convenient from a user perspective and hard to debug
See https://github.com/ComPWA/tensorwaves/pull/164#discussion_r524401478: it may solve the parameters, estimator_value problem
Challenge will be to be general enough for different optimisers.
It could serve of a placeholder for iterations. Currently (#164, #154), iterations are updated by each callback instance separately.

What is exactly the purpose of the FitResult? Because in zfit we already define a fitresult API and in general it would be nice to converge on all this APIs if there isn't a good idea why to deviate.
What should the FitResult roughly do?

Not sure yet, it serves more as a note in the ticket system.

Background is this discussion https://github.com/ComPWA/tensorwaves/pull/164#discussion_r524387482

List of things the FitResult has to be able to do (in my opinion):

Store all info, which initialized the fit (initial parameters) and the final state of the process (final parameters). I would also include an estimator value, here. (These things would be common to any kind of Optimizer)
[Optional/Optimizer dependent] Store infos like error estimates. Some Optimizers cannot directly provide this info. Therefore this info would have to be obtained by running a second Optimizer afterwards (or some code which can determine the errors).
[Optional/Optimizer dependent] All setting information of the Optimizer. The use case for this is to be to be able to keep track of different fits that have been performed.

That are all the requirements I can come up with. The current solution with the dict is just a "placerholder" solution, but it already encapsulates the idea. That is the FitResult should be a simple datastructure without any logic. I currently do not see any downside to this. The advantage is that things are very simple and yet flexible. Not sure what you think about this @mayou36
The implementation is another quite difficult question. I even opened a stack overflow thread for this a while ago, with no real conclusion. In ComPWA (c++) we picked a templated version of a FitResult. So this would correspond to a Generic class here. The idea was to include info in the Generic base of FitResult, that is common to FitResults from any possible Optimizer. This would be info like the initial and final parameters, or runtime, etc. Any Optimizer specific info would be injected in the result via the Generic.
A dict would be an alternative solutions (could be used also in class and be a substitute for the Generic).

Hey, sorry for the late reply, I missed your answer! So what we have in zfit is a class FitResult and we solved it the following way
There are mandatory attributes including:

fmin
edm
converged
valid
info  (dict with a lot of, and arbitrary, information
params  (holds all the infos: value, uncertainties etc)
together with two methods that can be executed, the info from the is added to the params dics
hesse  (error with hessian approximation)
errors  (minos-like error estimation, but can be completely independent)

All the additional information is stored in the info and can be arbitrary. This is where uncertainties from the minimizer are stored.
With time, more attributes can be added. Whateve we see is needed.
And the dicts basically solve all the problems of fixed structures (Python FTW).


⚠ Bofrost.lu has degraded performance

In 5a3e872, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 883 ms

Resolved: Bofrost.lu performance has improved in b988435.


docs(Readme): header link redirecting to 404 page

The link in was "https://github.com/vvo/next-iron/session" when it should be "https://github.com/vvo/next-iron-session

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix arm64 macOS failure

The build on arm64 macos is stopped due to running out of disk space:

You are running out of disk space. The runner will stop working when the machine runs out of disk space. Free space left: 76 MB

This is likely due to swap as it only has 7GB of RAM compared to 14GB on the  amd64 runner.

Switch both macos jobs to build the monolithic library to save diskspace.
Switch the arm64 build to release to further decrease memory footprint.

Fixes #10901

@assignUser Should we suggest to add -DFOLLY_HAVE_INT128_T=ON in brew folly cmake?

@zuyu sounds good, though we likely would also need to turn on sse/neon (not sure if that happens automatically?) so we can link against the F14 implementation.

Fyi, sse/neon compiler flags are set in the script https://github.com/facebookincubator/velox/blob/ecebbbf26f49d6dd2946d19decf52e8d389ac1bd/scripts/setup-helper-functions.sh#L76

On our side I know, I meant on folly's side. But after a quick glance at their cmake I don't think they do it so that would need to be added on homebrew as well. Homebrew is pretty aggressive in deprecating support for old macos version so it should be ok to add sse as all relevant hardware supports it? :shrug:


Fix the Windows help in the README

The current README suggests that a user having problems with S3 certificates completely disable their certificate checking globally for RubyGems. This opens up a security hole and would actually get them fired from some companies if they followed those instructions (including one I just worked for). I actually considered not using git-media as a result of this advice, but luckily I did some more searching and found a better solution.
There are instructions in a Gist that explain how to fix the problem; I've added them to the README in this pull request.

Thank you for fixing this glaring problem!


cmd/compile: declaring non-callable append then attempting to call crashes gc

What version of Go are you using (go version)?
go version go1.11 windows/amd64 (also occurs under Linux or on the playground)
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
GOHOSTARCH=amd64
GOHOSTOS=windows

(also happens on playground)
What did you do?
MWE:
package main

func main() {
	var append bool
	append()
}

https://play.golang.org/p/Z36gAU5mgUY
What did you expect to see?
prog.go:5:9: cannot call non-function append (type bool)

What did you see instead?
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x70 pc=0xb7c7ee]

goroutine 1 [running]:
cmd/compile/internal/gc.typecheck1(0xc00032c180, 0x1, 0xc00032c080)
	/usr/local/go/src/cmd/compile/internal/gc/typecheck.go:1270 +0x3ece
cmd/compile/internal/gc.typecheck(0xc00032c180, 0x1, 0xc00032c080)
	/usr/local/go/src/cmd/compile/internal/gc/typecheck.go:238 +0x6ad
cmd/compile/internal/gc.typecheckslice(0xc0003201c0, 0x3, 0x4, 0x1)
	/usr/local/go/src/cmd/compile/internal/gc/typecheck.go:68 +0x50
cmd/compile/internal/gc.Main(0xcc2180)
	/usr/local/go/src/cmd/compile/internal/gc/main.go:518 +0x2059
main.main()
	/usr/local/go/src/cmd/compile/main.go:51 +0x96

Spotted by @weiss91

Fixed on tip. Also backported to 1.11.1

@agnivade Still occurs on play.golang.org (as of now, see
https://play.golang.org/p/Z36gAU5mgUY), 1.11.1/windows, 1.11.1/linux. So not sure about that backport.

Just checked with current master, still crashes. Please reopen.
PS C:\Users\jannis.schnitzer\src\stuff\panic-typecheck> cat .\main.go
package main

func main() {
        var append bool
        append()
}
PS C:\Users\jannis.schnitzer\src\stuff\panic-typecheck> C:\Users\jannis.schnitzer\src\go.googlesource.com\go\bin\go.exe build
# stuff/panic-typecheck
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x70 pc=0xb952c5]

goroutine 1 [running]:
cmd/compile/internal/gc.typecheck1(0xc000340200, 0x1, 0xc000340100)
        c:/go/src/cmd/compile/internal/gc/typecheck.go:1270 +0x3ed5
cmd/compile/internal/gc.typecheck(0xc000340200, 0x1, 0xc000340100)
        c:/go/src/cmd/compile/internal/gc/typecheck.go:238 +0x6b4
cmd/compile/internal/gc.typecheckslice(0xc00032a660, 0x3, 0x4, 0x1)
        c:/go/src/cmd/compile/internal/gc/typecheck.go:68 +0x57
cmd/compile/internal/gc.Main(0xcdfc60)
        c:/go/src/cmd/compile/internal/gc/main.go:518 +0x20d0
main.main()
        c:/go/src/cmd/compile/main.go:51 +0x9d
PS C:\Users\jannis.schnitzer\src\stuff\panic-typecheck> C:\Users\jannis.schnitzer\src\go.googlesource.com\go\bin\go.exe version
go version devel +93e27e01af Mon Oct 15 14:51:30 2018 +0000 windows/amd64
PS C:\Users\jannis.schnitzer\src\stuff\panic-typecheck>

I cannot reproduce. Possibly a windows issue ?
21:33:16-agniva-~/play/go/src$gotip run append.go
# command-line-arguments
./append.go:5:9: cannot call non-function append (type bool)
21:35:35-agniva-~/play/go/src$gotip version
go version devel +296b7aeae0 Mon Oct 15 16:01:56 2018 +0000 linux/amd64
21:35:37-agniva-~/play/go/src$cat append.go
package main

func main() {
  var append bool
  append()
}

@griesemer

I cannot reproduce this either. Also, @Xjs I don't quite believe you're running current master: Your stack dump indicates a crash at c:/go/src/cmd/compile/internal/gc/typecheck.go:1270 which is the middle of a comment. Can you please update (and make sure you're using the newest compiler) and try again, thanks.

@griesemer Indeed, the go tool seems not to have called the freshly-built compile.exe. Can confirm that this error doesn't occur in tip on Windows anymore. Sorry for the confusion. (Can also confirm that 1.11.1's compile.exe does produce this error, though.)

Reason why it didn't get included for 1.11.1 - https://github.com/golang/go/issues/27973#issuecomment-426443001.
Closing it now.


"npm test" fails for firestore-bigquery-change-tracker and gen-schema-view

[REQUIRED] Step 2: Describe your configuration

Extension name: firestore-bigquery-export
Extension version: 0.1.3
Configuration values (redact info where appropriate):

_
_

[REQUIRED] Step 3: Describe the problem
Unit tests are not passing for the helper packages for firestore-bigquery-export
Steps to reproduce:
cd firestore-bigquery-export/firestore-bigquery-change-tracker
npm test
// Tests fail
cd ../scripts/gen-schema-view
npm test
// Tests fail
Expected result
Tests to pass
Actual result
For firestore-bigquery-change-tracker:

For gen-schema-view:

Once the tests are fixed, they should get hooked up to CI: https://github.com/firebase/extensions/issues/260

Are you running against 0.1.3 or 1.1.3? In your screenshot it shows 1.1.3. I can't reproduce this issue on next:


"newShard()" needs to be used throughout code

There are multiple places in our code where shard creation (and restoration of shard metadata) is not consistent, leaving some pointers nil under certain circumstances. This has caused multiple issues, and needs to be fixed, most likely through some refactoring.

CC @corylanou

No longer applicable in new design.


Makefile: Tweak quoting

Without this, an existing CGO_LDFLAGS_ALLOW would get double quoted
leading to a shell parsing error.
Closes #8949
Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

Testsuite passed

Awesome, thanks. Will this be backported to the stable-4.0 branch as well?

yep

@stgraber I tested the build of the master branch in my environment and it works. When do you plan to backport the fix to stable-4.0 branch?


[BUG] WRONG NUMBER REGISTERED IN VACCINATION CENTER WITH MY PHOTO ID ON CO-WIN . I CANT DOWNLOAD MY CERTIFICATE

In vaccination center thay put wrong number on my photo id when they register..  And after vaccination i did not receive any massage.. I cant download my certificate in CoWIN app

Please check with COWIN support https://www.cowin.gov.in/#howcanweHelp


WIFI-628 Adjust wlan naming convention.

Replace dashes with underscores to simplify operation with UCI.
Signed-off-by: Rick Sommerville<EMAIL_ADDRESS>

merged into pending, Thanks !


The update of props is not granular enough.

Describe the bug
const getId = () => {
  console.log("run");
  return "button-id";
};

<button
    type="button"
    onClick={increment}
    data-count={count()}
    id={getId()}
 >
    {count()}
 </button>

Your Example Website or App
https://playground.solidjs.com/anonymous/c4810293-8e55-49c5-a04b-f56408873017
Steps to Reproduce the Bug or Issue

Click on button
See the console

Expected behavior
  _$effect(_p$ => {
      var _v$ = `color: ${color()}`,
        _v$2 = getMyColor();
      _p$.e = _$style(_el$3, _v$, _p$.e);
      _v$2 !== _p$.t && _$setAttribute(_el$3, "id", _p$.t = _v$2);
      return _p$;
    }, {
    e: undefined,
    t: undefined
  });

It seems that the prop updates are wrapped in an effect, giving an impression of a complete replacement. I'm wondering if this aligns with signal's prop updating strategy, which typically emphasizes fine-grained and efficient changes.
Screenshots or Videos
No response
Platform

OS: mac
Browser: Chrome
Version: [e.g. 91.1]

Additional context
No response

This is what we call "merged effects", instead of adding 1 effect to each prop, it adds an effect that tracks all the props. It's an optimization for avoiding having to create too many effects. You are supposed to use a memo for prop functions that you wish to not re-run.

This is what we call "merged effects", instead of adding 1 effect to each prop, it adds an effect that tracks all the props. It's an optimization for avoiding having to create too many effects. You are supposed to use a memo for prop functions that you wish to not re-run.

Thank you for your response. The term 'merged effects' seems to imply a difference from Qwik, where each prop is individually maintained. This appears odd to me that static props still require memoization by the user. However, I'll adhere to this rule.

If you look there is shallow diffing taking place here not to over-update the DOM. But yes we loosened the granularity to improve creation speed. The update speed usually is trivial by comparison. Truly static props are hoisted out of the effect but both data-count and id could update so they are there (they call functions so they could be reactive). The way to think of createMemo is the cost of the execution. If you call a function and it is relatively cheap (most things) then it isn't worth wrapping. If it is wrap. This is pretty independent of how it is used.
Finally Qwik gets there grouping from having a VDOM re-render components on non-granular changes, but having granular changes be our only approach has caused this approach to evolve over the past 6 years. The truth of the matter is that purely granular overhead can be significant. This benchmark shows some of the impact as to get good granular scores it pushes you towards a more granular implementation. Especially note benchmarks around creation/removal: https://krausest.github.io/js-framework-benchmark/current.html

Thanks!! Creation speed escaped my attention.


build(docker): reduce image size

Reduce image size from 1.1GB to 210MB

I tested the CLI, overriding the entrypoint, git and bash commands

fixed the pre-commit


Generating nether worlds in combination with worldborder

I've got an interesting issue with a nether world that's been imported using the most-recent mv2 on a most-recent spigot server. The import goes fine, but when I use worldborder's fill command to pre-generate the world, it doesn't look right--there's not enough surface lava.
Is there any way to increase logging from world generation?
Is this something I should be asking about to the worldborder threads, instead of here (yes, I know that'd be better in some ways, but could they actually help ).
I know that's a bad description, but for the moment, it's what I have. The section for that world from my worlds.yml is below.
 It renders the same, either way. I'm closing the issue.

I generated the same world with a nether and as a nether by itself. It renders the same, either way. I'm closing the issue.


refactor document writing

Codecov Report

Merging #374 into master will increase coverage by 0.20%.
The diff coverage is 28.28%.

@@            Coverage Diff             @@
##           master     #374      +/-   ##
==========================================
+ Coverage   69.46%   69.67%   +0.20%
==========================================
  Files          23       25       +2
  Lines        1261     1253       -8
==========================================
- Hits          876      873       -3
+ Misses        385      380       -5

Flag
Coverage Δ

#unittests
69.67% <28.28%> (+0.20%)
:arrow_up:

Impacted Files
Coverage Δ

src/writer/latex.jl
0.00% <0.00%> (ø)

src/writer/pandoc.jl
0.00% <0.00%> (ø)

src/rendering/htmlformats.jl
72.72% <44.44%> (-6.12%)
:arrow_down:

src/rendering/markdownformats.jl
81.15% <50.00%> (-2.18%)
:arrow_down:

src/rendering/texformats.jl
64.94% <55.00%> (-2.80%)
:arrow_down:

src/rendering/common.jl
78.75% <66.66%> (ø)

src/Weave.jl
47.61% <100.00%> (+1.61%)
:arrow_up:

src/rendering/rendering.jl
88.88% <100.00%> (ø)

src/writer/writer.jl
100.00% <100.00%> (ø)

... and 2 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 db5308b...7e9c4fb. Read the comment docs.


media-breakpoint-down is off by 1px

Bootstrap 4 - mixin/breakpoint
Hy guys,
I don't know if I am using mixin in a bad way, but I just discover that this instruction don't work in the way I supposed.
@include media-breakpoint-down(sm) {
   //my style
}

I assumed that this style will be applied  from 0 to 767 pixel screen, but it will from 0 to 766.
I change the function  breakpoint-max on breakpoint.scss in this way (-0.05 instead of -0.1):
@function breakpoint-max($name, $breakpoints: $grid-breakpoints) {
  $next: breakpoint-next($name, $breakpoints);
  @return if($next, breakpoint-min($next, $breakpoints) - 0.05, null);
}

and it seem to work fine but i don't know if mess with all the other
Nice work,
I love you!
Alessandro

Unsure what you're running into, but it works fine on my end. Can you create a jsbin example that reproduces what you're seeing?

Sorry, I don't know how to use jsbin to do a demo because i'm using the scss mixin and not the css class, but the demo is very simple.
html
<div class="demo">
        DEMO
</div>

scss
    .demo {
        @include media-breakpoint-down(sm) { // this work up to 766
            background: red;
        }
        @include media-breakpoint-up(md) { //this start at 768
            background: green;
        }
    }

In this demo at the screen width 767 the background is not red and neither green. I suppose this problem occur for each breakpoint

@KickinAss Seems like you're using an old version of bootstrap 4 that uses rems and not pixels for breakpoint widths. Can you try upgrading and see if the issue disappears?

Yes, the grid breakpoint is in rem, I started this project one month ago or little more, and downloaded the framework from the official site.
I'll try to upgrade, I hope nothing will break! :)

Ok, no problem now!


Add HEROKU env var to heroku review app

Summary
Heroku deployments have started timing out when running webpack. The only changes we've made recently to our webpack config was adding the thread-loader. Gonna test setting env var we use to disable the thread-loader in the heroku app.json and see if this resolves #1071

The env variable can be anything, webpack just checks for CI currently.

If you're trying to add an env variable, It should be under the env section right?
https://devcenter.heroku.com/articles/app-json-schema#env

Haha, I should read the whole context of the file next time. This IS under the env section. Derp

This is failing on travis, but I'm not sure if its related to this change. I'm seeing the following error message in travis. I noticed a lot of our PRs are also failing on travis.

Error message
/home/travis/build/cerner/terra-core/node_modules/webdriverio/build/lib/utils/ReporterStats.js:381
            if (!suiteStats.tests[uid]) throw Error(`Unrecognised test [${runner.title}] for suite [${runner.parent}]`);
                                        ^
Error: Unrecognised test ["before all" hook] for suite [Aligned All]
at ReporterStats.getTestStats (/home/travis/build/cerner/terra-core/node_modules/webdriverio/build/lib/utils/ReporterStats.js:381:47)
at ReporterStats.testFail (/home/travis/build/cerner/terra-core/node_modules/webdriverio/build/lib/utils/ReporterStats.js:450:30)
at BaseReporter. (/home/travis/build/cerner/terra-core/node_modules/webdriverio/build/lib/utils/BaseReporter.js:159:25)
at emitOne (events.js:116:13)
at BaseReporter.emit (events.js:211:7)
at BaseReporter.handleEvent (/home/travis/build/cerner/terra-core/node_modules/webdriverio/build/lib/utils/BaseReporter.js:300:27)
at Launcher.messageHandler (/home/travis/build/cerner/terra-core/node_modules/webdriverio/build/lib/launcher.js:646:28)
at emitTwo (events.js:126:13)
at ChildProcess.emit (events.js:214:7)
at emit (internal/child_process.js:772:12)
at _combinedTickCallback (internal/process/next_tick.js:141:11)
at process._tickDomainCallback (internal/process/next_tick.js:218:9)
npm ERR! Test failed.  See above for more details.

I’m wondering if a recent release of wdio has a defect. Perhaps we should try locking the version?

Yeah, I'm thinking one of our dependencies must have an update that seems to be causing these errors. The most recent commit to master had passed CI successfully when I checked about an hour or so ago. I re-ran the build for master and now its failing with a similar error message to what I'm seeing on this PR.

Gonna merge this done now so we can get branch deploys working again. Let's fix the CI test issue in https://github.com/cerner/terra-core/pull/1084

@bjankord I have seen this error output before. I don't quite remember what I was doing to create that error but I think it had to do with attempting to test a themeable variable that was not currently present on the page or was named incorrectly.


Strings for Poke4

'Poke4':{
     "MODEL": "Poke4",
     "STRING_SETTINGS": "uTKx3XVyRhlbI7hoHuy/NiYdwlWViPlc4EecZKYTThN/",
     "STRING_UPGRADE": "vzDFqwIEMRfqTPUCV82ahjnz0hXfcZCIxRY8ljuLmTaf"
 }

Confirmed - I can extract payload.bin from update.zip and dump boot.img from payload.bin

Thanks for your strings. They were merged in f2f50ba


Asynchronous read callback does not work on Raspberry

Hi Sandeep,
I am afraid I have another huge problem. I was able to workaround all my other issues and now I am left with this one: For my application I need bleno to respond asynchronously to a read request. Works without problems on OS X, but when running on raspberry I get an empty result (tested with LightBlue iOS) or the app keeps spinning (LightBlue OS X).
Thats probably the interesting part, the http request ist just a dummy to be asynchronous:
TestChar.prototype.onReadRequest = function(offset, callback) {
  console.log(this.uuid+' - onReadRequest');

	http.get({ host: 'github.com', path: '/'},
		function (res) {
  			callback(this.RESULT_SUCCESS, new Buffer('Test'));
// returns TEST when running via OS X, but (null) when running on RPi
		}
	)
};

OS X Debug log:
aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa0 - onReadRequest
  bindings sendCBMsg: 13, {
  "kCBMsgArgAttributeID": 2,
  "kCBMsgArgData": {
    "type": "Buffer",
    "data": [
      84,
      101,
      115,
      116
    ]
  },
  "kCBMsgArgTransactionID": 1
} +256ms

RPi log:
aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa0 - onReadRequest
  gatt read response: 010a0c0000 +619ms
  gatt send: 010a0c0000 +3ms
  hci write acl data pkt - writing:<PHONE_NUMBER>05000400010a0c0000 +2ms
  hci onSocketData:<PHONE_NUMBER>05000400010a0c0000 +75ms
  hci 	event type = 2 +3ms

Here is a RPi log, when the callback is handled immediately:
aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa0 - onReadRequest
  gatt read response: 0b54657374 +65ms
  gatt send: 0b54657374 +2ms
  hci write acl data pkt - writing:<PHONE_NUMBER>050004000b54657374 +2ms
  hci onSocketData:<PHONE_NUMBER>050004000b54657374 +32ms
  hci 	event type = 2 +2ms
  gatt read response: 010a0c0000 +272ms
  gatt send: 010a0c0000 +2ms
  hci write acl data pkt - writing:<PHONE_NUMBER>05000400010a0c0000 +3ms
  hci onSocketData:<PHONE_NUMBER>05000400010a0c0000 +23ms
  hci 	event type = 2 +2ms

Thank you for your time!

I found the problem, sorry I am brand new to NodeJS and it still confuses me sometimes:
My problem was the scope in which I call the callback: this.RESULT_SUCCESS was always undefined, and thus noble emitted an error response instead of the proper result.


Create a minimist wrapper that validates a schema

From @matheuss on July 7, 2017 8:13
It'd, upon a false by default throwOnUnexpectedFlag, throw if you define {boolean: ['version']} and the program is called with --testing
Copied from original issue: zeit/now#5

Closing in favor of zeit/arg 🎉


Does it work with Metabarcoding nanopore data?

Hi,
Does Melon accept FastQ files from long-read metabarcoded data?

Hi,
both fasta and fastq can be used as input, but make sure your fastq files are demultiplexed and quality-controlled.

Hi, thanks for the prompt reply. I will test out the tool then.


Flake: cannot release un-acquired lock in Python's _exit_test

Related: https://github.com/grpc/grpc/pull/23981
Test case "test_in_flight_stream_stream_call" in "_exit_test" fails on Python 2 with following error:
test_in_flight_stream_stream_call (__main__.ExitTest) ... Traceback (most recent call last):
  File "/usr/local/google/home/lidiz/src/grpc/src/python/grpcio_tests/tests/unit/_exit_scenarios.py", line 235, in <module>
    for response in response_iterator:
  File "/usr/local/google/home/lidiz/src/grpc/py27_native/local/lib/python2.7/site-packages/grpc/_channel.py", line 413, in next
    return self._next()
  File "/usr/local/google/home/lidiz/src/grpc/py27_native/local/lib/python2.7/site-packages/grpc/_channel.py", line 706, in _next
    raise self
  File "/usr/lib/python2.7/threading.py", line 289, in __exit__
    return self.__lock.__exit__(*args)
  File "/usr/lib/python2.7/threading.py", line 216, in __exit__
    self.release()
  File "/usr/lib/python2.7/threading.py", line 204, in release
    raise RuntimeError("cannot release un-acquired lock")
RuntimeError: cannot release un-acquired lock
ERROR

There isn't any suspicious logic that could lead to this exception:
https://github.com/grpc/grpc/blob/4365bdfdcc3c9cde3a77d5a024d5aaea28bcf911/src/python/grpcio/grpc/_channel.py#L676-L706
The lock might be deallocated by Python interpretor. There could be a bug in CPython. This exception could apply to other cases.

@gnossen The two burdensome workarounds are 1. register a signal handler; 2. catch interrupt key exception within gRPC library. Both ways allow us to enforce the deallocation order, so the spawned thread (request consumption thread, spin thread, etc.) can be collected before main thread. WDYT?


Update for shogun helm-chart

This adds an extraVolume for the keycloak themes.
It also updates the translation of the layers entity.

We might also want to create an init container that adds the SHOGun login theme. Or we combine the existing shogun-keycloak init-containers into one that does all the needed stuff.


Add Postgres SET ROLE functionality so DB user doesn't need to be owner

As ...
Patti - Platform Admin
I want ...

To be able to use a Postgres user that only has read/write access to the nautobot table for day-to-day operations
To be able to programmatically change to a group role before running migrations (i.e. via SET ROLE ROLE_NAME;) so that I can create/alter/delete tables when running nautobot-server migrate and to have the role reset (i.e. via RESET ROLE;) back afterward

So that ...
So that our security standards with regards to DB roles can be set correctly
I know this is done when...

I can use the credentials of a Postgres DB user that isn't an owner of the tables but still be able to upgrade and run migrations
I can set the name of the role via nautobot_config.py to define the role to use in the SET ROLE command

Optional - Feature groups this request pertains to.

[ ] Automation
[ ] Circuits
[ ] DCIM
[ ] IPAM
[ ] Misc (including Data Sources)
[ ] Organization
[ ] Plugins (and other Extensibility)
[ ] Security (Secrets, etc)
[ ] Image Management
[ ] UI/UX
[ ] Documentation
[X] Other (not directly a platform feature)

Database Changes
Should not require DB model changes
External Dependencies
Should not require external dependencies

This is problematic since any migration that creates new tables would also need to know which other user(s) need to be granted read/write access to those tables. What's the underlying goal behind having a different user create the tables versus populate them?


Add an option to install an extension into a workspace only

Type: Feature Request
As a longtime vscode user, it is getting rather common for me to have many extensions installed and many different workspaces used for various projects in various toolstacks. However, my i5 Chromebook can only handle so much - the more extensions, the slower vscode runs. Thus, I need to be really selective about which extensions I enable. So my workflow is like this: install a new extension, and then go to the N many workspaces I have and disable it individually within each one.
It would be great if it was possible to simply this by simply installing an extension into a specific workspace and that's it.
VS Code version: Code 1.86.1 (Universal) (31c37ee8f63491495ac49e43b8544550fbae4533, 2024-02-07T09:09:01.236Z)
OS version: Darwin arm64 23.2.0
Modes:

Can't you already do this?

Install extension.
Disable it globally.
Enable it just for the workspace(s) you want to use it in.

That's a good idea.


Unused configuration parameters

Description
The configuration parameters bgColor and quality are defined but never used.
Is bgColor intended to set site.webmanifest's background_color property?

I'm in the process of restructuring Hyas (technically I'm practically there, now updating documentation) — see also https://github.com/h-enk/hyas/pull/329
You're right — these settings are not used by the new @hyas/images module, they're temporarily left in place for backward compatibility. So, I will strip them out eventually,

These parameters are part of Hugo's  default imaging processing options — and now also used with the @hyas/images


how to remove temporal lags according to the offset?

Thanks for the work!
I wonder to know that after I get the offset between the audio and video, how can I do to  remove the temporal lags? That is, making the video synced with the audio?
Looking forward to your reply!

I think it is the same discussion as https://github.com/joonson/syncnet_python/issues/53


Peer-to-peer review

Your code looks and works just fine. The extra features are very much interesting too.
The following are mere suggestions:

Update README to contain code for running JavaScript linter
There is redundancy in your UI with respect to the Add button function.

You have 2 Add buttons. One should do since they are close to each other on the page, anyways

Your ReSync feature also has the same redundancy issue as the Add functionality on your UI.

Thank you very much, peers. I really appreciate that.


Llama 3.1 70B high-quality HQQ quantized model - 99%+ quality of fp16

I'm not really sure if that's possible but adding that to ollama could really impact the performance on 4-bit quant option:
99%+ in all benchmarks in lm-eval relative performance to FP16 and similar inference speed to fp16
url:
https://huggingface.co/mobiuslabsgmbh/Llama-3.1-70b-instruct_4bitgs64_hqq

also this:
https://huggingface.co/ModelCloud/Meta-Llama-3.1-70B-Instruct-gptq-4bit

Requires support in llama.cpp.  There have been issues raised (https://github.com/ggerganov/llama.cpp/issues/6368, https://github.com/ggerganov/llama.cpp/issues/4782) but unfortunately no progress.

I would also be really interested in seeing support added for this. Would pay $100 bounty, if that helps.


Merge branch 'main' into cxx17

Link the Issue(s) this Pull Request is related to.
Each PR should link at least one issue, in the form:
Fixes #123
Use one line for each Issue. This allows auto-closing the related issue when the fix is merged.
Summarize your change.
Describe the reason for the change.
Add a list of changes, and note any that might need special attention during review.
Reference associated tests.
If no new tests are introduced as part of this PR, note the tests that are providing coverage.

Hi Darby! Fixes issue #123 That was good old drop-frame support, how does this fix it?
;)
Was this an intentional PR?

Sorry about that! I left the default PR text which apparently references #123. I updated the text to be a bit more descriptive.

Yeah, I don't think we're going to rewrite commit history and force push and break the universe!
Thanks for updating the comment.


There's a new Bugfender SDK version

I've noticed this error in the build log although we are on the latest version: 2.1.2
There's a new Bugfender SDK version. Please check bugfender.com.

Does the iOS library need to be updated?

Yes, you can update the Podfile to the latest version if you'd like. We issue updates for the RN wrapper regularly, so eventually we'll update it here. If you'd like to submit a PR, you're welcome to do so!

I just checked that the dependency is indicated as ~> 1.10, which means that the latest pod version as we speak (1.12.1) is included. I verified that by checking the lockfile.
If your project uses an older version of the pod, you may want to manually issue a pod install to get the latest version.


Persist Location Notes

Prevents an admin comment from overwriting existing Location notes during a Location merge.
Fixes #2415

coverage: 93.471%. remained the same
when pulling 1133f1040fbec449598184babd7eda3864999474 on jdc-2415-merged-location-notes
into b0922d432c45433bf1c1a76fcc6fa195219ac334 on main.


Write error messages to stderr in carthage-verify

I want to use carthage-verify like below to know only errors.
./carthage-verify > /dev/null

🎉 thanks for the contribution!


6.0.0-RC4 has been released

https://twitter.com/snicoll/status/1590331215885877250

We only track stable versions.

https://endoflife.date/spring-framework


Auth token  Sonnen API

EDIT:
I got it to connect. How? I don't know what I really did, but it works now.
There are only 2 errors:

"Auth-Token provided, but could not use official API: 404 - "\n<!DOCTYP...."   looks like it is an API problem. I have the older eco6, not a eco8 or later.
"[REQUEST] <== Error: connect ECONNREFUSED <IP_ADDRESS>:3480".  Wher can I define the port 3480?
I do not have any values in the objecttree:  sonnen/0/status for all 3 values (Consumption, production and relative SOC). Is this related to the error above?

Still do not know where to get the Auth token for my eco6 version.
Thanks
Peter
disregard
Describe the bug
I'm an absolute beginner with ioBroker and coding in general...
I have the Sonnen adapter in my ioBroker installed. There is no connection (Log). I can not find the "Software integration" tab to get the token for the adapter settings. Where I have to look for?
My Sonnen battery is a eco6 (2014):
Version SPS
5.00 (435)
Kapazitaet
0100
SOC
100.0
Betriebsart
10
Watt
0
Temperatur
Thanks a lot,
Peter
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots & Logfiles
If applicable, add screenshots and logfiles to help explain your problem.

ioBroker Log.txt
Versions:

Adapter version: 1.10.0
JS-Controller version:
Node version:
Operating system:
Battery model:

Additional context
Add any other context about the problem here.

Sorry for the late response. I have been on vacation.
I don't think, that ECO 6 needs an API key, so just leave it empty in Adapter config.


NativeModules dependencie added

Hello 👋
#40 for a this issue

Ooops, sorry for late response. I already fixed this issue.


Update instructions and tests of change font-size challenge

Update instructions and test cases of Challenge Change the Font Size of an Element.
closes #6819.

Aside from the above notes, I tested this and it works well.  You can git commit --amend to update this PR - just do a git push -f to update it.

@SaintPeter, Done! Thanks for the help.

@ankitaggarwal011 thanks for your contribution. This works great!


Update team page

New members are missing such as @vcoisne, and some pictures were retaken and have not been updated yet (e.g. mine).

This can obviously wait for the next web version, but it should not be forgotten.

Closing this per objective accomplishment.


Set active for paths like "/foo/#bar" in NavLink.

Unfortunately React sets NavLink with hash paths only active when they have following scheme:
/foo#bar
But I am using this scheme:
/foo/#bar
So this sets also NavLinks active with my scheme.

I feel like this should be in React Router and not here as they handle active route matching.
Or you could just pass in your custom isActive function to react-router-hash-link and it will be passed through with the other props.

I've posted the missing function as an issue but it was rejected.
And yeah, I could pass my function as prop but I thought I'm not the only one with this problem.

I agree <NavHashLink> should only be active when both path and hash match. There are a number of edge cases around path matching that RR handles (trailing slash, when to is a function, etc) and determining if the paths match is out of scope of this library. But adding an additional check to see if the hashes match is within scope.
There is currently a RR bug for determining when a link is active ReactTraining/react-router#6877, when that is fixed I will add an additional check to this lib to make sure the hashes also match.

Closing because this PR adds path matching logic which is out of scope for this library.

This is fixed in v2.2.0


Failed to download resources or network error. Code:-9006

Anybody getting this error after updating to the latest build? This never happened before 4.0.

Disable any VPNs while downloading
Make sure you're connected to a stable internet

If you're super desperate:

Delete YAAGL and the resource files: /Users/username/Library/Application Support/Yaagl OS
re-download the latest release

If nothing else works:
Watch some anime and wait for other responses

Same problem here, cannot download content after logging in. I tried turning on and off VPN and made sure internet was stable, but the problem still existed.

same problem

我的进度条一直在滚动,但是没有任何进展

Do you still have this issue?


Delete extraneous conv layer from example

The second Conv1D layer is strange and does make a significant difference in the accuracy of model. Seems like a typo?
Results with only a single Conv1D layer:

@googlebot I signed it!


JBDS-4320 Show Hyper-V even if it is not detected with link to documentation how to configure and use it with CDK 3

Codecov Report

Merging #668 into master will increase coverage by 0.48%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #668      +/-   ##
==========================================
+ Coverage   95.98%   96.46%   +0.48%
==========================================
  Files          29       29
  Lines        2464     2602     +138
  Branches      208      208
==========================================
+ Hits         2365     2510     +145
+ Misses         99       92       -7

Impacted Files
Coverage Δ

browser/model/hyperv.js
100% <100%> (ø)
:arrow_up:

browser/services/platform.js
90.37% <0%> (-0.05%)
:arrow_down:

browser/main.js
100% <0%> (ø)
:arrow_up:

browser/pages/install/controller.js
100% <0%> (ø)
:arrow_up:

browser/model/devstudio.js
100% <0%> (ø)
:arrow_up:

browser/model/helpers/installer.js
100% <0%> (ø)
:arrow_up:

browser/model/helpers/version.js
100% <0%> (ø)
:arrow_up:

browser/model/installable-item.js
100% <0%> (ø)
:arrow_up:

browser/services/request.js
100% <0%> (ø)
:arrow_up:

browser/model/helpers/hash.js
100% <0%> (ø)
:arrow_up:

... and 17 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 bad6445...1ba69bc. Read the comment docs.

test this

test this

Should be something like this in the end

I think it should at least only show on systems that actually support Hyper-v (meaning Win 8/10 Pro and higher) - the systems where this command
https://github.com/redhat-developer-tooling/developer-platform-install/blob/master/browser/services/platform.js#L104 actually returns something.
It looks weird when it shows up on my Win10 home and the first step is to check if I have Pro/Enterprise edition.


Replace an invisible blank line with <|BLANK LINE|>

Forum #1581
Although we did put a blank line there, it turned out to appear invisible somehow:
https://app.codecrafters.io/courses/interpreter/stages/oe4

@andy1li let's fix the underlying issue - why are blank lines not showing up in the rendered version?

@andy1li another angle to look at here would be -- can we change the test programs so that their expected output is more self-explanatory?
As an example, an alternate option here would be:
print true != true;
print "The result above should be false."
print "36
278
96
"
print "There should be an empty line above this"

@andy1li let's make sure to keep a linear ticket around to investigate the root cause related to markdown rendering too - not super important for now, but can schedule for coming weeks


Terms & Conditions in Footer (or some where else)

Hi @ssddanbrown thank you for creating great software. I love using it.
In Germany (and other places I'm sure) there are legal requirements for website that we include legal terms & conditions so a visiter can find information about the host of the content.
My tech guys have found a solution in which they jus created a page with the content and added a link to the top bar as shown in the following picture:

Am I missing the option to set this up some where or is this not included in your software at this time?
Kind Regards
Wolfram

Thanks for the request and feedback @wojosc.

Am I missing the option to set this up some where or is this not included in your software at this time?

You're not missing anything, It's not possible in the current release but it has been merged as part of the current release cycle so will be part of the next feature release. Please see the following issue for more detail: https://github.com/BookStackApp/BookStack/issues/854#issuecomment-770304122
Since this is effectively a duplicate of #854 I'm going to close off this issue.

That will cover that more than sufficient. Thanks!


Microsoft Python Language Server

Will be great if https://github.com/microsoft/python-language-server is also supported

I tried working on this, I could get the server to start from the skeleton, but was failing to get any diagnostics/completion.
Here are some relevant implementations:

emacs: lsp-python-ms
vim coc-python which wraps a fork of the vscode python plugin
sublime official mpls repo

If you want to get the nupkg for the mpls, which can be extracted and run with dotnet
curl -LO https://pvsc.azureedge.net/python-language-server-stable/Python-Language-Server-linux-x<IP_ADDRESS>.nupkg

See #57