potential bug in offline-edit-src.js at line 1336 editStore.init

I thought I would start from the bottom up with the problem description, so I would point out the exact place where I think the bug is, but my explanation may be a bit convoluted.
Using offline-editor-js - v2.13.0 - 2015-08-10.
Line 1336 in offline-edit-src.js executes editStore.init and passes a callback.  The callback resolves a deferred closure variable called "deferred".   However, the target function this.init at line 3205 never executes the callback, so the deferred is never resolved.    The resolved deferred is needed to execute other code that loads features from the database into feature layers.
getFeatureCollections at line 1269 is calling initializeDB and assigning a ".then" which is waiting for the deferred to be resolved by the call back passed to editStore.init.   SInce it never resolves the deferred variable,  the this._editStore._getFeatureCollections function never gets executed and my custom call back is also not executed.
I found this because the anonymous call back that I pass to offlineFeaturesManager.getFeatureCollections is not being executed.
I'm attempting a browser restart using "Approach 2" as described in the https://github.com/Esri/offline-editor-js/blob/master/doc/howtouseeditlibrary.md.
I can see both tile and feature databases in chrome developer tools.   The code is able to successfully open the feature database at line 3208 in the this.init function.
offlineFeaturesManager.getFeatureCollections(function(success, collection) {
  if(success) {
    for(var x=0;x<collection.featureCollections.length;x++) {
      var newFeatureLayer = new FeatureLayer(collection.featureCollections[x].featureLayerCollection,{
                        mode: FeatureLayer.MODE_SNAPSHOT,
                        outFields: ["*"]
                      });

      offlineFeaturesManager.extend(newFeatureLayer,function(result, error) {
        if(result) {
          console.log("Layer has been successfully rebuilt while offline!");
        }
      });

      _layers.push(newFeatureLayer);
    }
  }

My first explanation is not exactly correctly.   It appears that the call back is supposed to happen via the request that is passed back by the indexedDB.open function.
I'm investigating further.

It appears that the "request" functions in this.init at line 3205 , onerror, onupgradeneeded and onsuccess are not getting called.  I put break points in them and the code never entered those functions.  I'm checking to see if there is another handler that it might be trying to call.

When I run the application in IE11, the onsuccess handler at line 3229 is being executed.  When I run it in the latest version of chrome it is not.
I am investigating further.

@pstevlingson this library doesn't support IE in any version due to a wide variety of limitations. As per the README, it only supports the latest versions of Chrome, Safari and Firefox.
Can you post the console.log dump and the offending code block from your application? Even better would be a public repo I can take a look at.
The most common issues related to the editStore not initializing correctly has to do with your application life-cycle.

The block of code is the one below.   Its not executing the call back.   I'm working on getting the console log now.
offlineFeaturesManager.getFeatureCollections(function(success, collection) {
if(success) {
for(var x=0;x<collection.featureCollections.length;x++) {
var newFeatureLayer = new FeatureLayer(collection.featureCollections[x].featureLayerCollection,{
mode: FeatureLayer.MODE_SNAPSHOT,
outFields: ["*"]
});
  offlineFeaturesManager.extend(newFeatureLayer,function(result, error) {
    if(result) {
      console.log("Layer has been successfully rebuilt while offline!");
    }
  });

  _layers.push(newFeatureLayer);
}

}

Document was loaded from Application Cache with manifest http://server03.windlake.com/LeakSurveyCollector/leaksurvey.appcache
Application Cache Checking event
Application Cache Error event: Manifest fetch failed (6) http://server03.windlake.com/LeakSurveyCollector/leaksurvey.appcache
appCacheManager.js enabled
CACHE EVENT: {}
CACHE ERROR: {}
GET http://server03.windlake.com/LeakSurveyCollector/images/tiny-image.png? net::ERR_INTERNET_DISCONNECTED
STATE IS: false
database opened successfully
init editsStore.js
GET http://js.arcgis.com/3.14compact/esri/images/map/logo-med.png net::ERR_INTERNET_DISCONNECTED
looking for tile 16 24695 19243
LIBRARY ONLINE false
looking for tile 16 24696 19243
LIBRARY ONLINE false
looking for tile 16 24695 19244
LIBRARY ONLINE false
looking for tile 16 24696 19244
LIBRARY ONLINE false
looking for tile 16 24695 19245
LIBRARY ONLINE false
looking for tile 16 24696 19245
LIBRARY ONLINE false
looking for tile 16 24695 19246
LIBRARY ONLINE false
looking for tile 16 24696 19246
LIBRARY ONLINE false
looking for tile 16 24695 19247
LIBRARY ONLINE false
looking for tile 16 24696 19247
LIBRARY ONLINE false
looking for tile 16 24695 19248
LIBRARY ONLINE false
looking for tile 16 24696 19248
LIBRARY ONLINE false
looking for tile 16 24695 19249
LIBRARY ONLINE false
looking for tile 16 24696 19249
LIBRARY ONLINE false
looking for tile 16 24695 19250
LIBRARY ONLINE false
looking for tile 16 24696 19250
LIBRARY ONLINE false
Tile Layer initialized for offline. App state is: up
database opened successfully
GET http://server03.windlake.com/LeakSurveyCollector/images/tiny-image.png?=702754016 net::ERR_INTERNET_DISCONNECTED
GET http://js.arcgis.com/3.14compact/esri/dijit/images/ajax-loader.gif net::ERR_INTERNET_DISCONNECTED
GET http://js.arcgis.com/3.14compact/esri/dijit/images/popup.png net::ERR_INTERNET_DISCONNECTED
GET void:/16/24695/19243 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24696/19243 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24695/19244 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24696/19244 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24695/19245 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24696/19245 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24695/19246 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24696/19246 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24695/19247 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24696/19247 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24695/19248 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24696/19248 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24695/19249 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24696/19249 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24695/19250 net::ERR_UNKNOWN_URL_SCHEME
GET void:/16/24696/19250 net::ERR_UNKNOWN_URL_SCHEME
TILE RETURN false, undefined
tile is not in the offline store http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24695/19243
TILE RETURN false, undefined
tile is not in the offline store http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24696/19243
TILE RETURN false, undefined
tile is not in the offline store http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24695/19244
TILE RETURN false, undefined
tile is not in the offline store http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24696/19244
TILE RETURN false, undefined
tile is not in the offline store http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24695/19245
TILE RETURN false, undefined
tile is not in the offline store http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24696/19245
TILE RETURN true, http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24695/19246
found tile offline http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24695/19246
TILE RETURN true, http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24696/19246
found tile offline http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24696/19246
TILE RETURN true, http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24695/19247
found tile offline http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24695/19247
TILE RETURN true, http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24696/19247
found tile offline http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24696/19247
TILE RETURN true, http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24695/19248
found tile offline http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24695/19248
TILE RETURN true, http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24696/19248
found tile offline http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24696/19248
TILE RETURN true, http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24695/19249
found tile offline http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24695/19249
TILE RETURN true, http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24696/19249
found tile offline http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24696/19249
TILE RETURN true, http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24695/19250
found tile offline http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24695/19250
TILE RETURN true, http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24696/19250
found tile offline http://server.arcgisonline.com/ArcGIS/rest/services/World_Street_Map/MapServer/tile/16/24696/19250
Unable to init PhantomLayer
Uncaught TypeError: Cannot read property 'push' of undefined
TypeError: Cannot read property 'length' of undefined
at d.addLayers (http://js.arcgis.com/3.14compact/init.js:308:283)
at declare.initializeApplication (http://server03.windlake.com/LeakSurveyCollector/js/controllers/appcontroller.js:193:26)
at null. (http://server03.windlake.com/LeakSurveyCollector/js/controllers/appcontroller.js:119:26)
at g.hitch (http://js.arcgis.com/3.14compact/init.js:167:296)
at k (http://js.arcgis.com/3.14compact/init.js:189:431)
at m (http://js.arcgis.com/3.14compact/init.js:189:357)
at x.reject (http://js.arcgis.com/3.14compact/init.js:192:143)
at a (http://js.arcgis.com/3.14compact/init.js:190:337)
at k (http://js.arcgis.com/3.14compact/init.js:190:103)
at m (http://js.arcgis.com/3.14compact/init.js:189:357) "TypeError: Cannot read property 'length' of undefined
at d.addLayers (http://js.arcgis.com/3.14compact/init.js:308:283)
at declare.initializeApplication (http://server03.windlake.com/LeakSurveyCollector/js/controllers/appcontroller.js:193:26)
at null. (http://server03.windlake.com/LeakSurveyCollector/js/controllers/appcontroller.js:119:26)
at g.hitch (http://js.arcgis.com/3.14compact/init.js:167:296)
at k (http://js.arcgis.com/3.14compact/init.js:189:431)
at m (http://js.arcgis.com/3.14compact/init.js:189:357)
at x.reject (http://js.arcgis.com/3.14compact/init.js:192:143)
at a (http://js.arcgis.com/3.14compact/init.js:190:337)
at k (http://js.arcgis.com/3.14compact/init.js:190:103)
at m (http://js.arcgis.com/3.14compact/init.js:189:357)
----------------------------------------
rejected at a (http://js.arcgis.com/3.14compact/init.js:190:337)
at k (http://js.arcgis.com/3.14compact/init.js:190:89)
at m (http://js.arcgis.com/3.14compact/init.js:189:357)
at x.reject (http://js.arcgis.com/3.14compact/init.js:192:143)
at a (http://js.arcgis.com/3.14compact/init.js:190:337)
at k (http://js.arcgis.com/3.14compact/init.js:190:103)
at m (http://js.arcgis.com/3.14compact/init.js:189:357)
at x.reject (http://js.arcgis.com/3.14compact/init.js:192:143)
at a (http://js.arcgis.com/3.14compact/init.js:190:337)
at k (http://js.arcgis.com/3.14compact/init.js:190:89)
----------------------------------------
Error
at f.then.b.then [as then] (http://js.arcgis.com/3.14compact/init.js:192:253)
at verifyOnline [as verifyOnline] (http://server03.windlake.com/LeakSurveyCollector/js/controllers/appcontroller.js:413:54)
at declare.constructor (http://server03.windlake.com/LeakSurveyCollector/js/controllers/appcontroller.js:113:18)
at new  (http://js.arcgis.com/3.14compact/init.js:98:398)
at http://server03.windlake.com/LeakSurveyCollector/js/main.js:6:17
at ha (http://js.arcgis.com/3.14compact/init.js:22:473)
at http://js.arcgis.com/3.14compact/init.js:23:202
at ia (http://js.arcgis.com/3.14compact/init.js:23:89)
at fa (http://js.arcgis.com/3.14compact/init.js:23:144)
at g (http://js.arcgis.com/3.14compact/init.js:25:380)"
TypeError: Cannot read property 'success' of undefined
at null. (http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:1162:37)
at k (http://js.arcgis.com/3.14compact/init.js:189:431)
at f.then.b.then [as then] (http://js.arcgis.com/3.14compact/init.js:192:308)
at declare.extend (http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:1152:41)
at http://server03.windlake.com/LeakSurveyCollector/js/services/mapservices.js:106:34
at http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:1275:37
at IDBRequest.objectStoreGraphicRequest.onsuccess (http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:3163:17) "TypeError: Cannot read property 'success' of undefined
at null. (http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:1162:37)
at k (http://js.arcgis.com/3.14compact/init.js:189:431)
at f.then.b.then [as then] (http://js.arcgis.com/3.14compact/init.js:192:308)
at declare.extend (http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:1152:41)
at http://server03.windlake.com/LeakSurveyCollector/js/services/mapservices.js:106:34
at http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:1275:37
at IDBRequest.objectStoreGraphicRequest.onsuccess (http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:3163:17)
----------------------------------------
rejected at a (http://js.arcgis.com/3.14compact/init.js:190:337)
at k (http://js.arcgis.com/3.14compact/init.js:190:89)
at f.then.b.then [as then] (http://js.arcgis.com/3.14compact/init.js:192:308)
at declare.extend (http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:1152:41)
at http://server03.windlake.com/LeakSurveyCollector/js/services/mapservices.js:106:34
at http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:1275:37
at IDBRequest.objectStoreGraphicRequest.onsuccess (http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:3163:17)
----------------------------------------
Error
at f.then.b.then [as then] (http://js.arcgis.com/3.14compact/init.js:192:253)
at declare.extend (http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:1152:41)
at http://server03.windlake.com/LeakSurveyCollector/js/services/mapservices.js:106:34
at http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:1275:37
at IDBRequest.objectStoreGraphicRequest.onsuccess (http://server03.windlake.com/LeakSurveyCollector/js/offline/offline-edit-src.js:3163:17)"
GET http://server03.windlake.com/LeakSurveyCollector/images/tiny-image.png?=542998149 net::ERR_INTERNET_DISCONNECTED
GET http://server03.windlake.com/LeakSurveyCollector/images/tiny-image.png?=195009355 net::ERR_INTERNET_DISCONNECTED
GET http://server03.windlake.com/LeakSurveyCollector/images/tiny-image.png?=131444003 net::ERR_INTERNET_DISCONNECTED
GET http://server03.windlake.com/LeakSurveyCollector/images/tiny-image.png?=902590869 net::ERR_INTERNET_DISCONNECTED
GET http://server03.windlake.com/LeakSurveyCollector/images/tiny-image.png?_=695822580 net::ERR_INTERNET_DISCONNECTED

The tile library seems to work in ie11 without issues.  Do you have the list of limitations in ie (specifically ie11) that cause problems for the offline library?
I'm told our current utility client uses Microsoft rugged books in the field and supports ie11 as their corporate standard browser.   They currently do not support chrome, safari or firefox.

When I run the application in IE11, the onsuccess handler at line 3229 is being executed. When I run it in the latest version of chrome it is not.

To make sure I understand, your app runs just fine in IE11 and not in Chrome?

Do you have the list of limitations in ie (specifically ie11) that cause problems for the offline library?

Nope, sorry. The main issues at the time had to do with limited IndexedDB support, various JavaScript syntax issues. I also don't have any way to troubleshoot IE 11 or Edge issues.

What version of the library are you using? The version number is at the top of each library. The latest and greatest is v2.13.1. Since the line numbers won't match up between versions, I'm not able to match up the error message line numbers from the console.log.
Thanks for the console.log. At least for this round of tweaks, the good news is I don't see any issues related to the OfflineFeaturesManager library initializing.
There is one glaring item that hints it needs further looking into. It's related to tile layer in appcontroller.js line 193. After that there is a cascade of other errors. Can you take a took at that and see if this is the primary event triggering some or all of the other issues?
The deferred errors you are seeing very likely could be because something else broke before or during the asynchronous action began, and that issue then prevented the deferred from being resolved or gracefully rejected.

Unable to init PhantomLayer
Uncaught TypeError: Cannot read property 'push' of undefined
TypeError: Cannot read property 'length' of undefined
at d.addLayers (http://js.arcgis.com/3.14compact/init.js:308:283)
at declare.initializeApplication (http://server03.windlake.com/LeakSurveyCollector/js/controllers/appcontroller.js:193:26)
at null. (http://server03.windlake.com/LeakSurveyCollector/js/controllers/appcontroller.js:119:26)

The appcontroller.js module is failing at line 193 because offlineFeaturesManager.getFeatureCollections is not returning the stored layer in the indexeddb.
Yesterday I was using offline-editor-js - v2.13.0 - 2015-08-10.    I just downloaded v.2.13.1.  I will perform some more testing and give you an update later today.
If there is any chance in the future that you could support ie11 or edge that would be great.    However, the clients might be open to using chrome.

If there is any chance in the future that you could support ie11 or edge that would be great.

No, I don't see any chance of IE11 support. Edge is a possibility as some point in the future if there are more requests. So far, you're the only person who's requested Edge, and only one or two other people have requested IE support in the last two years. The vast majority of implementations for this repo are Android and iOS smartphones and tablets.
Next steps

Can you verify: are you wanting an intermittent offline for a full offline workflow? If you aren't sure of the difference here's a blog post I wrote. The pattern you are showing me above hints at full offline, but I wanted to be 100% certain.

When are you seeing these errors: when the application first loads online, or after an offline restart?

Review the life-cycle patterns in appcache-features.html.

Pay particular attention to the fact that offlineFeaturesManager doesn't get initialized until after feature layers update-end event as shown here.

Place a break point here and let's verify the error message and see if it hints at anything. Looks like we missed printing that error message to console. I'll add that as a minor TO-DO.
I have a strong suspicion that the map is not initialized by the time you run offlineFeaturesManager.extend(). The following error certain points a finger in that direction because the error is happening here and it's in the esri.Map.addLayers() method. If the map isn't initialized then the offlineFeaturesManager library cannot add the phantom graphics layer to it. There's also a good chance the feature layers aren't initialized, as well. Any of these issues could explain what you're seeing.

Uncaught TypeError: Cannot read property 'push' of undefined
TypeError: Cannot read property 'length' of undefined
at d.addLayers (http://js.arcgis.com/3.14compact/init.js:308:283)

I looked into the initialization problem and that does appear to be the root cause of the problem.    I am creating and extending the layer before even adding it to the map.   It looks like I have to add the layer to the map, wait for it to initialize via the "layers-add-result" map event, then the "_map" variable will be set on the layer and "extend" will function properly.
It might be handy to put a check for a null _map somewhere in extend function and warn that the map has to be initialized if its not.  The _initPhantomLayer function seems to be the exact location where it was failing because of the null _map.
That makes sense about Android and iOS.     If the library appears to work with IE11 I'll use it, realizing that if I run into problems I'm on my own.
Thanks for your help on this.

Glad you got it going. I agree on the null map check, that's an excellent idea since this is a common issue. I'll put a fix in place.


Remove reference to non-existent function

The reference function was deleted in b523022401bed89d3d7123d18a17c106e412a6fe

BTW, EventQueue::flush seems like a far more logical place for flush. I only found Display::flush because it was mentioned on IRC.

Thanks!

BTW, EventQueue::flush seems like a far more logical place for flush. I only found Display::flush because it was mentioned on IRC.

Yeah, the current state of flush() reflects the internal behavior: it flushes the whole connection, not a single event queue. But I agree it could be relevant to have an EventQueue::flush() method that does the same thing I guess.

the current state of flush() reflects the internal behavior: it flushes the whole connection, not a single event queue

Ah, that's good to know.
Thanks!


refactor!: remove support for deprecated fields and fix tests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 46.53%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
4230

Relevant Lines:
9091

💛  - Coveralls


hyprpanel fails to launch

I disabled notifications in config.json just in case, but it still fails to launch. Here's the error:
2024-10-31T20:34:33.803+0200 [WARN]  host: Failed loading stylesheet, continuing with defaults: file=/home/kadotux/.config/hyprpanel/style.css
2024-10-31T20:34:33.803+0200 [WARN]  host: Failed adding application watcher: target=/var/lib/flatpak/exports/share/applications err="no such file or directory"
2024-10-31T20:34:33.803+0200 [WARN]  host: Failed adding application watcher: target=/home/kadotux/.local/share/flatpak/exports/share/applications err="no such file or directory"
2024-10-31T20:34:33.941+0200 [ERROR] host.plugin.panel0.hyprpanel-client: Failed initializing window: err="json: cannot unmarshal bool into Go struct field Client.fullscreen of type int" timestamp="2024-10-31T20:34:33.941+0200"
2024-10-31T20:34:34.043+0200 [INFO]  host.plugin.panel0: plugin process exited: plugin=/usr/bin/hyprpanel-client id=14999
2024-10-31T20:34:34.043+0200 [ERROR] host: Lost connection to hyprland IPC bus: err="read unix @->/tmp/hypr/a6ccd36147109d5cb2981122595f06ae93999b55_1730398710/.socket2.sock: use of closed network connection"
2024-10-31T20:34:34.043+0200 [ERROR] host: Received from closed hypr event channel
2024-10-31T20:34:34.045+0200 [ERROR] host: Clients failed: err="panel failed (panel panel0 failed: context canceled): context canceled"
2024-10-31T20:34:34.387+0200 [ERROR] host.plugin.panel0.hyprpanel-client: Failed initializing window: err="json: cannot unmarshal bool into Go struct field Client.fullscreen of type int" timestamp="2024-10-31T20:34:34.387+0200"
2024-10-31T20:34:34.491+0200 [INFO]  host.plugin.panel0: plugin process exited: plugin=/usr/bin/hyprpanel-client id=15024
2024-10-31T20:34:34.491+0200 [ERROR] host: Lost connection to hyprland IPC bus: err="read unix @->/tmp/hypr/a6ccd36147109d5cb2981122595f06ae93999b55_1730398710/.socket2.sock: use of closed network connection"
2024-10-31T20:34:34.491+0200 [ERROR] host: Received from closed hypr event channel
2024-10-31T20:34:34.493+0200 [ERROR] host: Clients failed: err="panel failed (panel panel0 failed: context canceled): context canceled"
2024-10-31T20:34:34.834+0200 [ERROR] host.plugin.panel0.hyprpanel-client: Failed initializing window: err="json: cannot unmarshal bool into Go struct field Client.fullscreen of type int" timestamp="2024-10-31T20:34:34.834+0200"
2024-10-31T20:34:34.933+0200 [INFO]  host.plugin.panel0: plugin process exited: plugin=/usr/bin/hyprpanel-client id=15047
2024-10-31T20:34:34.934+0200 [ERROR] host: Lost connection to hyprland IPC bus: err="read unix @->/tmp/hypr/a6ccd36147109d5cb2981122595f06ae93999b55_1730398710/.socket2.sock: use of closed network connection"
2024-10-31T20:34:34.934+0200 [ERROR] host: Received from closed hypr event channel
2024-10-31T20:34:34.936+0200 [ERROR] host: Clients failed: err="panel failed (panel panel0 failed: context canceled): context canceled"
2024-10-31T20:34:35.300+0200 [ERROR] host.plugin.panel0.hyprpanel-client: Failed initializing window: err="json: cannot unmarshal bool into Go struct field Client.fullscreen of type int" timestamp="2024-10-31T20:34:35.299+0200"
2024-10-31T20:34:35.394+0200 [INFO]  host.plugin.panel0: plugin process exited: plugin=/usr/bin/hyprpanel-client id=15070
2024-10-31T20:34:35.394+0200 [ERROR] host: Lost connection to hyprland IPC bus: err="read unix @->/tmp/hypr/a6ccd36147109d5cb2981122595f06ae93999b55_1730398710/.socket2.sock: use of closed network connection"
2024-10-31T20:34:35.394+0200 [ERROR] host: Received from closed hypr event channel
2024-10-31T20:34:35.396+0200 [ERROR] host: Clients failed: err="panel failed (panel panel0 failed: context canceled): context canceled"
2024-10-31T20:34:35.743+0200 [ERROR] host.plugin.panel0.hyprpanel-client: Failed initializing window: err="json: cannot unmarshal bool into Go struct field Client.fullscreen of type int" timestamp="2024-10-31T20:34:35.743+0200"
2024-10-31T20:34:35.839+0200 [INFO]  host.plugin.panel0: plugin process exited: plugin=/usr/bin/hyprpanel-client id=15093
2024-10-31T20:34:35.839+0200 [ERROR] host: Lost connection to hyprland IPC bus: err="read unix @->/tmp/hypr/a6ccd36147109d5cb2981122595f06ae93999b55_1730398710/.socket2.sock: use of closed network connection"
2024-10-31T20:34:35.839+0200 [ERROR] host: Received from closed hypr event channel
2024-10-31T20:34:35.841+0200 [ERROR] host: Clients failed: err="panel failed (panel panel0 failed: context canceled): context canceled"
2024-10-31T20:34:35.841+0200 [ERROR] host: Restarting too quickly, terminating

What version of Hyprland are you running? They broke API compatibility in v0.42.0, so current versions of hyprpanel require Hyprland >= v0.42.0

I have Hyprland 0.44.1 running actually. I installed hyprpanel with paru from AUR.

Very strange, can you show the output of:

hyprctl version
hyprpanel --version
paru -qs hyprpanel
hyprctl clients -j (you can sanitize/strip the output, all I need to see is the fullscreen and fullscreenClient lines from any client window)

hyprctl version:
`Hyprland, built from branch main at commit a6ccd36147109d5cb2981122595f06ae93999b55 dirty (screencopy: move monitor verif check to the proper place).
Date: Thu Feb 8 01:47:14 2024
Tag: v0.35.0-8-ga6ccd361
flags: (if any)`
hyprpanel --version:
hyprpanel version (devel) built with go1.23.2
paru -qs hyprpanel:
1 aur/hyprpanel 0.2.0-1 [+0 ~0.00] [Installed]     An opinionated panel/shell for the Hyprland compositor. 2 aur/ags-hyprpanel-git r292.f2985e7-2 [+1 ~0.65]     A Bar/Panel for Hyprland with extensive customizability 3 aur/hyprpanel-bin 0.2.0-1 [+0 ~0.00]     An opinionated panel/shell for the Hyprland compositor.
hyprctl clients -j:
hyprctl clients -j | grep fullscreen                                                                 ─╯     "fullscreen": false,     "fullscreenMode": 0,     "fullscreen": false,     "fullscreenMode": 0,     "fullscreen": false,     "fullscreenMode": 0,     "fullscreen": false,     "fullscreenMode": 0,     "fullscreen": false,     "fullscreenMode": 0,     "fullscreen": false,     "fullscreenMode": 0,     "fullscreen": false,     "fullscreenMode": 0,     "fullscreen": false,     "fullscreenMode": 0,     "fullscreen": false,     "fullscreenMode": 0,     "fullscreen": false,     "fullscreenMode": 0,

That says your Hyprland is v0.35.0, please upgrade to a recent version and try again.

That's strange. It is the newest, updated version from Arch repo. And it is 0.44.1: https://archlinux.org/packages/extra/x86_64/hyprland/ Just to make sure, I now replaced it with hyprland-git from AUR
and still hyprctl clients shows:
Hyprland, built from branch main at commit a6ccd36147109d5cb2981122595f06ae93999b55 dirty (screencopy: move monitor verif check to the proper place).
Date: Thu Feb 8 01:47:14 2024
Tag: v0.35.0-8-ga6ccd361

flags: (if any)

however, paru -Ss hyprland | grep Installed shows:
aur/hyprland-git 0.44.0.r46.0baf166d-1 [+94 ~8.64] [Installed: 0.44.0.r99.3852418d-1]
So... I'm pretty sure I'm running 0.44...? Unless I'm completely missing something. Maybe hyprctl doesn't work as expected. Somebody at reddit is claiming so: https://www.reddit.com/r/hyprland/comments/1clpods/how_do_i_check_the_current_installed_hyprland/
anyway, now I'm running 0.44.0, newest package from AUR and hyprpanel gives me the same error.

Something is wrong with your Hyprland install, because hyprctl tells you that you're running v0.35.0:
Tag: v0.35.0-8-ga6ccd361

Make sure you don't have some other version of Hyprland installed at a different path (/usr/local or something), try which hyprctl to see where it's being run from, because you're definitely not actively running v0.44.1

Ok, thanks. That's weird for sure. But I'll try to figure out why I don't have the latest hyprland. I think this issue can be closed, if you want.


Spatial Shader - Unexpected behavior with using OpenEXR 4 channel image as texture

Godot version
4.1.1
System information
Windows 10
Issue description
My spatial shader to implement pre-rendered background needs today 2 texture files, one is the color render of the scene, the other is a depth render.
The depth render has pixel values as unbounded 32 bits floating numbers, it describes distance from the camera in meters.
The color render has pixel values ranging from 0 to 1.0 written in 32 bits floating numbers. Its the render of the scene.
Both images are in the OpenEXR format, both images have 3 channels (RGB), the Depth image repeats the same values in its 3 channels.
The following shader works perfectly (See image 01)

shader_type spatial;
render_mode unshaded, depth_draw_always;

uniform sampler2D _albedo : source_color;
uniform sampler2D _depth : source_color;

void vertex() {
	POSITION = vec4(VERTEX, 1.0);
}

void fragment() {
	float depth = -texture(_depth, SCREEN_UV).x;
	vec4 temp = vec4(0, 0, depth, 1);
	vec4 clip_space = PROJECTION_MATRIX * temp;
	clip_space.z /= clip_space.w;

	DEPTH = clip_space.z;
	ALBEDO = texture(_albedo, SCREEN_UV).rgb;
}

However this is wasteful in disk space (two images at 15MB, a single image with all data size is 8MB), so to improve I went and changed the compositing on Blender to generate a single image, putting the Depth in place of the Alpha channel.
shader_type spatial;
render_mode unshaded, depth_draw_always;

uniform sampler2D _T: source_color;

void vertex() {
	POSITION = vec4(VERTEX, 1.0);
}

void fragment() {
	float depth = -texture(_T, SCREEN_UV).a;  // alpha channel has DEPTH values
	vec4 temp = vec4(0, 0, depth, 1);
	vec4 clip_space = PROJECTION_MATRIX * temp;
	clip_space.z /= clip_space.w;

	DEPTH = clip_space.z;
	ALBEDO = texture(_T, SCREEN_UV).rgb;
}

This refuses to work.
As long as depth and albedo come from the same file, this simply does not work and there is nothing in my power to fix this problem.
For some reason, the value that I retrieve from A channel it returns a garbage value instead. acting as if the value that I pickup is 1.0
See, if I take the old code and do ALBEDO = clip_space.zzz I will have n interesting image with depth of everything. However with the new variation of the shader, it will instead of paint the depth, will paint everything white, as if the value were 1.0
So far, no EXR renders with 4 channels resulted in any kind of success.
I then tried to evaluate the functionality of the files themselves.
shader_type spatial;
render_mode unshaded, depth_draw_always;

uniform sampler2D _T: source_color;
uniform sampler2D depth: source_color;

void vertex() {
	POSITION = vec4(VERTEX, 1.0);
}

void fragment() {
	float depth = -texture(depth, SCREEN_UV).a;  // alpha channel has DEPTH values
	vec4 temp = vec4(0, 0, depth, 1);
	vec4 clip_space = PROJECTION_MATRIX * temp;
	clip_space.z /= clip_space.w;

	DEPTH = clip_space.z;
	ALBEDO = texture(_T, SCREEN_UV).rgb;
}

Now I took the 4 channels image and repeated it in both sampler2D, depth and _T, and again depth failed miserably.
To make sure the problem was NOT with the image, I recreated the EXR file, but this time I switched the channels alpha and red in place, this way the first channel contains DEPTH, and I can just remap the alpha as red for the albedo.
Again I had exactly the same failure, again the value in DEPTH was understood as a uniform value.
As a last measure I created a depth image that had the same thing in 4 channels and used that as input for the depth sampler2D and picked the alpha channel, once again, it failed.
example of the failure state, compared to the previous image, the character is not in the image anymore, thats the failure state, the depth is homogenous 1.0 value.

Just to be sure, I mage another image with the DEPTH in the red layer, green and blue in their normal places, thats all, no red, and it did work correctly with the following code in the fragment:
void fragment() {
	float depth = -texture(depth, SCREEN_UV).r;  // red channel has DEPTH values
	vec4 temp = vec4(0, 0, depth, 1);
	vec4 clip_space = PROJECTION_MATRIX * temp;
	clip_space.z /= clip_space.w;

	DEPTH = clip_space.z;
	ALBEDO = texture(_T, SCREEN_UV).ggb; // missing red channel
}

My tests bring me to conclude that Sampler2D or texture are misbehaving with EXR images with 4 channels and cant grab the correct value present in the A channel. Or DEPTH is broken if there is an image with 4 channels as input in the sampler 2D
Steps to reproduce
Load the optimzed version of the shader.
shader_type spatial;
render_mode unshaded, depth_draw_always;

uniform sampler2D _T: source_color;

void vertex() {
	POSITION = vec4(VERTEX, 1.0);
}

void fragment() {
	float depth = -texture(_T, SCREEN_UV).a;  // alpha channel has DEPTH values
	vec4 temp = vec4(0, 0, depth, 1);
	vec4 clip_space = PROJECTION_MATRIX * temp;
	clip_space.z /= clip_space.w;

	DEPTH = clip_space.z;
	ALBEDO = texture(_T, SCREEN_UV).rgb;
}

Load a render with 4 chanells (attached here test_image.zip), with the depth layer in the alpha channel.
Add the shader into a quad and load that quad in front of the camera as you would a spatial shader (dont forget to flip the quad faces), load the image to the T sampler2D of the material/shader.
Add a cube to the scene and try to have it hide behind the tree.
Minimal reproduction project
example.zip

I think this your problem come from you not noticing
uniform sampler2D _T: source_color;

source_color

https://docs.godotengine.org/en/stable/tutorials/shaders/shader_reference/shading_language.html

It's important to understand that textures that are supplied as color require hints for proper sRGB->linear conversion (i.e. source_color), as Godot's 3D engine renders in linear color space.

when you need "raw texture data" - you dont use source_color - this why you losing your alpha 32-bit value.

example.zip

... share working godot project not just scene to save everyone else time...
I tested your project:

remove source_color
re-import texture with Lossless parameter instead of vram compression.
it seems to work now

on screenshot leaves intersect with cube, before change cube was not visible
Before:

After:

You are so correct I cannot even state how correct you are, it took 7 people and 38 hours of total work hours and before we gave up and created this issue, and you hit the mark perfectly.
Thanks alot!


Test Result Output to Single Table in Snowflake (via Variant Column)

Is there an existing feature request for this?

[X] I have searched the existing issues

Describe the Feature
I would like to save all test output to a single table in Snowflake instead of a single table per test.  We can leverage the variant column type in order to do this and add other metadata as columns in the table.
Describe alternatives you've considered
Na
Who will this benefit?
Anyone using Snowflake and wants to report on dbt tests for quality initiatives.
Are you interested in contributing this feature?
No response
Anything else?
I believe this is a huge enhancement for anyone leveraging Snowflake and other warehouses that support variant/json type columns.  This provides the ability to save a full history of tests and be able to report on test/quality data.

Update: This was a huge help!  I managed to work out a flow where I consolidate all of the data into a single table. Meant to reply sooner but better late than never! For anyone interested, I used the code @jtcohen6  shared and added my own flavor for our use case(cleanup and other metadata):
`{% macro centralize_test_failures(results) %}
{%- set test_results = [] -%}
{%- for result in results -%}
{%- if result.node.resource_type == 'test' and result.status != 'skipped' and (
result.node.config.get('store_failures') or flags.STORE_FAILURES
)
-%}
{%- do test_results.append(result) -%}
{%- endif -%}
{%- endfor -%}
{%- set central_tbl -%} {{ target.schema }}.test_failures {%- endset -%}
{{ log("Centralizing test failures in " + central_tbl, info = true) if execute }}
{%- if test_results | length > 0 -%}
{% set query %}

create or replace table {{ target.schema }}."{{ macro_to_create_a_unique_id() + '_' + central_tbl }}" as (
{% for result in test_results %}
select
    metadata1                                            as metadata1,
    metadata1                                             as metadata2,
    metadata(n)                                          as metadata(n),
    '{{ result.node.name }}'                        as test_name,
    '{{ result.node.unique_id }}'                 as model,
    object_construct_keep_null(*)            as row_data,
    current_timestamp                               as _timestamp
from {{ result.node.relation_name }}

{{ "union all" if not loop.last }}

{% endfor %}
);
{% for result in test_results %}
    drop table if exists {{ target.schema }}."{{ macro_to_create_a_unique_id() + '_' + result.node.name |  trim }}";
{% endfor %}

{% endset %}
{% set final_query %}
insert into {{ central_tbl }}
select * from {{ target.schema }}."{{ macro_to_create_a_unique_id() + '' + central_tbl }}";
drop table if exists {{ target.schema }}."{{ macro_to_create_a_unique_id() + '' + central_tbl }}";
{% endset %}
{% do run_query(query) %}
{% do run_query(final_query) %}
{%- endif -%}
{% endmacro %}`

I posted a version of this code for BigQuery along with some comments/thoughts in this ticket: https://github.com/dbt-labs/dbt-core/issues/4613

Hello, Thank you very much for your solution. I'm writing some generic test in dbt and I want to store the failure in one table as you did. I'm using sqlserver and I got syntax error as :
 ('42000', "[42000] [Microsoft][ODBC Driver 18 for SQL Server][SQL Server]Incorrect syntax near '('. (102) (SQLExecDirectW)")
I removed the replace from your code since the sqlserver does not support it (I tried by the way truncate or alter)
If you can give me a version supported by sqlserver I'll be more than thankful.
Thank you
`  create table {{ central_tbl }} as (
{% for result in test_results %}
select
  '{{ result.node.name }}' as test_name,
  object_construct_keep_null(*) as test_failures_json

from {{ result.node.relation_name }}

{{ "union all" if not loop.last }}

{% endfor %}`

@Mira-Yassine - we're all just dbt users mocking up code here so you might have to experiment to try to figure out what works on SQL Server :). object_construct_keep_null is a Snowflake function to aggregate a bunch of key-value pairs into one OBJECT datatype. https://docs.snowflake.com/en/sql-reference/functions/object_construct_keep_null.html
You'll need to find what the equivalent datatype would be in SQL Server and find the functions to work with it. JSON is the close-enough equivalent datatype in BigQuery, for example (see here: https://github.com/dbt-labs/dbt-core/issues/4613).

@codigo-ergo-sum - Hello, Thank you very much for your hint. That's help me already to know that 'object_construct_keep_null' is a Snowflake function. I'll try to figure out how can I store the test data into one table in my SQL Server.

Good afternoon. Thanks everyone for the helpful discussion.
Could you tell me what the error could be related to - AttributeError: 'str' object has no attribute '_message'.
In dbt_project.yml I write:
on-run-end: "{{ centralize_test_failures(results) }}".
Macro -
{% macro centralize_test_failures(results) %}

  {%- set test_results = [] -%}
  {%- for result in results -%}
    {%- if result.node.resource_type == 'test' and result.status != 'skipped' and (
          result.node.config.get('store_failures') or flags.STORE_FAILURES
      )
    -%}
      {%- do test_results.append(result) -%}
    {%- endif -%}
  {%- endfor -%}

  {%- set central_tbl -%} table_scheme.tests_result {%- endset -%}

--  {{ log("Centralizing test failures in " + central_tbl, info = true) if execute }}

  CREATE OR REPLACE TABLE {{ central_tbl }} ENGINE = MergeTree()
  ORDER BY model_name
  AS (
    {%- for result in test_results %}
    SELECT
      today() as test_date,
      multiIf(
        '{{ result.node.name }}' LIKE 'unique%', 'unique',
        '{{ result.node.name }}' LIKE 'accepted_values%', 'accepted_values',
        '{{ result.node.name }}' LIKE 'not_null%', 'not_null',
        '{{ result.node.name }}' LIKE 'relationships%', 'relationships',
        '{{ result.node.name }}' LIKE 'source_accepted_values%', 'source_accepted_values',
        '{{ result.node.name }}' LIKE 'source_not_null%', 'source_not_null',
        '{{ result.node.name }}' LIKE 'source_relationships%', 'source_relationships',
        '{{ result.node.name }}' LIKE 'source_unique%', 'source_unique',
        '{{ result.node.name }}' LIKE 'dbt_expectations%', 'dbt_expectations',
        'unknown_test'
    ) AS test_name,
      name AS model_name,
      replaceRegexpOne(substring('{{ result.node.name }}', (LENGTH (name) + LENGTH (test_name)+3)), '(__\\d+)+$', '') as column_name,
      toJSONString(tuple(base.*)) AS test_failures_meta
    FROM {{ result.node.relation_name }} AS base
    CROSS JOIN
      (SELECT name
      FROM system.tables
      WHERE database = 'dev_env_datamart') AS models
    WHERE position('{{ result.node.name }}', name) != 0
    ORDER BY column_name, test_name
    {{ "UNION ALL" if not loop.last }}
    {%- endfor %}
  );

{% endmacro %}

Thanks for any advice!

macro_to_create_a_unique_id

Could you share the code for macro_to_create_a_unique_id?

Update: This was a huge help! I managed to work out a flow where I consolidated all of the data into a single table. Meant to reply sooner but better late than never! For anyone interested, I used the code @jtcohen6 shared and added my own flavor for our use case(cleanup and other metadata):
{% macro centralize_test_failures(results) %}

  {%- set test_results = [] -%}
  {%- for result in results -%}
    {%- if result.node.resource_type == 'test' and result.status != 'skipped' and (
          result.node.config.get('store_failures') or flags.STORE_FAILURES
      )
    -%}
      {%- do test_results.append(result) -%}
    {%- endif -%}
  {%- endfor -%}

  {%- set central_tbl -%} {{ target.schema }}.test_failures {%- endset -%}

  {{ log("Centralizing test failures in " + central_tbl, info = true) if execute }}

  {%- if test_results | length > 0 -%}

    {% set query %}

   create or replace table {{ target.schema }}."{{ macro_to_create_a_unique_id() + '_' + central_tbl }}" as (

  {% for result in test_results %}

    select
        metadata1                                            as metadata1,
        metadata2                                             as metadata2,
        metadata(n)                                          as metadata(n),
        '{{ result.node.name }}'                        as test_name,
        '{{ result.node.unique_id }}'                 as model,
        object_construct_keep_null(*)            as row_data,
        current_timestamp                               as _timestamp
    from {{ result.node.relation_name }}

    {{ "union all" if not loop.last }}

  {% endfor %}
   );

    {% for result in test_results %}
        drop table if exists {{ target.schema }}."{{ macro_to_create_a_unique_id() + '_' + result.node.name |  trim }}";
    {% endfor %}

{% endset %}

{% set final_query %}

 insert into {{ central_tbl }}
 select * from {{ target.schema }}."{{ macro_to_create_a_unique_id() + '_' + central_tbl }}";
 drop table if exists {{ target.schema }}."{{ macro_to_create_a_unique_id() + '_' + central_tbl }}";
{% endset %}

{% do run_query(query) %}
{% do run_query(final_query) %}
{%- endif -%}
{% endmacro %}

Hi, @brittianwarner , could you share your macro_to_create_a_unique_id() as well?


Viewing Submissions

https://github.com/xodigital/Formerly#viewing-submissions
Am i being dopey - i cannot get this to work in any shape or form on latest version?

did you sort this out?

Unfortunately not...
On Thu, Oct 22, 2015 at 2:31 PM -0700, "Andrew Wisener"<EMAIL_ADDRESS>wrote:
did you sort this out?

Reply to this email directly or view it on GitHub:
https://github.com/xodigital/Formerly/issues/46#issuecomment-150362147

can you add a code sample

Will do tomorrow .
But used your example and thinned it back a bit to remove non relevant variables.
On Thu, Oct 22, 2015 at 2:36 PM -0700, "Andrew Wisener"<EMAIL_ADDRESS>wrote:
can you add a code sample

Reply to this email directly or view it on GitHub:
https://github.com/xodigital/Formerly/issues/46#issuecomment-150363199

The below is the full error and code snippet in one! Hope it helps. N
It is as per your example with the refinement of country removed.
Template Error
Method "submissions" for object "Craft\FormerlyVariable" does not exist
/chroot/home/heinzher/heinzheroes.com/craft/templates/test.html(2)
1
2 {% set submissions = craft.formerly.submissions.form('heartOfThe').sort('dateCreated desc') %}
3
4 {% for submission in submissions %}
5   Name: {{ submission.nameOfBus }}
6   Town: {{ submission.town }}
7 {% endfor %}
2015-10-23 18:02:37 Apache / Craft 2.4.2696

I have the same issue. Has this been resolved?


Keymaster web app should handle whitespace

If you enter a challenge DID with a leading space, the create response will fail with an "Unknown DID" error.

Fixed in #154


Changesets ignoring the skipCI config key

Affected Packages
pkg:cli
Problem
Setting the config.json this way
{
  "$schema": "https://unpkg.com/<EMAIL_ADDRESS>  "changelog": "@changesets/cli/changelog",
  "commit": true,
  "skipCI": false,
  "linked": [],
  "access": "restricted",
  "baseBranch": "master"
}

when changesets commits in CI, the comit message still contains skipCI
Proposed solution

Hey, thanks for raising this! Couple of clarifications, do you have the repo where you are experiencing this problem, and to check, this error occurs with the changeset action creating commits for you?

Hey @Noviny!
The repo is sadly not on GitHub, but is a lerna repo that we're working on at the moment. Yeah, the error itself occurs when we run the following commands in our CI scripts:
echo "+++ :npm: Publishing to NPM :npm:"
yarn changeset version
yarn changeset publish

Had a really quick look into it, and we deprecated the skipCI flag with our version 2 upgrade, as all use-cases where people wanted commit: true, they also wanted skipCI true, and we add the skipCI flag to all our commits.
Happy to leave this open as 'we should reimplement this to allow fine-grained controls' but for an immediate fix you can set commit to false and run:
echo "+++ :npm: Publishing to NPM :npm:"
yarn changeset version
git add . && git commit -m "versioned packages"
yarn changeset publish

My two take-aways from this is we should add skipCI back as its own flag, defaulting to true (what most people have wanted) and fine grain commit so you can set it per command that can commit.
Thanks for raising this!

Ah right! Yeah we made a workaround using the manual commit in our CI pipelines. Thanks for doing such a great job on this project!

 So when the PR gets merged it has [skipCi] in there and the changesets workflow doesn't run as it should. I couldn't figure out a workaround so instead I've been reduced to having {"commit": false}. I would love to have a way to disable the [skipCi] from being added altogether. I understand the benefit that if it gets pushed to a particular PR then it shouldn't retrigger the build but that messes up our workflow. I couldn't figure out a way in github to not include all commit messages in the condensed commit that gets merged. Any thoughts would be appreciated.

@wweaver it feels like this is the most up-to-date discussion about this issue: https://github.com/changesets/changesets/issues/688 . I didn't have time lately to revisit this but it's on my mind - I see there is already a PR trying to address this (potentially. I'll try to get to reviewing it soon(-ish)


namespace and package not supported

First off, thank you for this impressive package!
Im busy running the transpile step on our chains protobuf files and I am getting this error above.
How would I address this ?  It appears on the surface to be a support issue for the protocol buffers but I may be wrong.
Can someone add some insight here?

Hey Dan! Here is the is a repository with the proto files in question: https://github.com/demondayza/ProtoFilesforTelescopeFeedback

hey @demondayza — I've found a solution locally to get everything to work. But I have a question, did you use the ignite CLI to create these? Why are all the modules in the global namespace?
A big issue with your protos is that your package will not be very portable. I can make an experimental option that will allow these to work, but I still want to point this out to help improve the interoperability of your project
https://github.com/ignite/cli/issues/2871 and this comment specifically https://github.com/ignite/cli/issues/2871#issuecomment-1264690290

is it this one https://github.com/ixofoundation/ixo-blockchain/tree/main/proto ?
I'd love to do two things 1. make an issue to explain how to make ixo protos more interoperable. and 2. make an experimental flag to help you get the JS working for now. How does the sound?

Hey @pyramation there is some legacy work behind the proto files themselves from a previous  developer so we welcome any and all advice to improve them with our current developers . You have the correct repository there absolutely and I encourage you to make a issue we can work with our side.
At the moment we are a couple weeks away from a chain version upgrade so this advice comes at the perfect time!
Essentially we want what's best and this advice will go a long way in helping us do that.
Thanks for the effort so far @pyramation this is what makes the cosmos ecosystem as cool as it is.

awesome! made an issue here https://github.com/ixofoundation/ixo-blockchain/issues/272

also @demondayza let me know if you need help with the telescope package — feel free to add me to it and I can help out.


[BUG] Order of parameters returned for mixtures distributions is wrong

The order for the dataframe should be alternating: mu1-std1-mu2-std2-....

Fixed in #402


Re-open case #2898

Original reporter has posted source in comment. https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/2898#issuecomment-531345595

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

ID: 3cd12943-2c9d-bcfb-8a7f-9b184c34f4e4
Version Independent ID: 6576e82d-b941-b14c-88e5-34d968dcd44f
Content: Front End Pool high availability and management
Content Source: Skype/SfbServer/plan-your-deployment/high-availability-and-disaster-recovery/high-availability.md
Product: skype-for-business-itpro
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@Atreidae Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

The file is probably shared content between 2015 and 2019 versions. This update will require making this difference between the versions unambiguous, without requiring a split in the topic.

@Atreidae Hello and thank you for bringing up this issue.  I'll be investigating this and will be getting back to you on this issue, thanks!

PR #3141

@Atreidae Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Support bitwise queries

i.e. [IntObject objectsWhere:@"(intCol & 1) > 0"] (feature request from twitter)

NSPredicate does support bitwise predicates

Closing as this is also being tracked in #1490.


[CT-2705] dbt deps behaves different between windows and linux  when adding a package

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
i currently have a project that has the expermntal features [materialized views] added to the packages.yml
and i get 2 different responses when running  dbt deps on windows machine and linux machine
the windows reponse ends up adding the packages successfully as per the following
while the linux response is the following
16:02:09  Running with dbt=1.5.1 16:02:13  Encountered an error: Internal Error   Error checking out spec='HEAD' for repo https://github.com/dbt-labs/dbt-labs-experimental-features   error: No such remote 'origin'
Expected Behavior
the windows output is the following
'
14:45:41  WARNING: The git package "https://github.com/dbt-labs/dbt-labs-experimental-features"
is None.
This can introduce breaking changes into your project without warning!
See https://docs.getdbt.com/docs/package-management#section-specifying-package-versions
14:45:41  Installing dbt-labs/codegen
14:45:41  Installed from version 0.9.0
14:45:41  Up to date!
14:45:41  Installing https://github.com/dbt-labs/dbt-labs-experimental-features
14:45:42  Installed from HEAD (default revision)
14:45:42  and subdirectory materialized-views
14:45:42  Installing calogica/dbt_date
14:45:42  Installed from version 0.7.2
14:45:42  Up to date!
14:45:42  Installing dbt-labs/dbt_utils
14:45:43  Installed from version 1.1.1
14:45:43  Up to date!
'
Steps To Reproduce
on both machines i did the following
1- created a conda environment and performed pip install dbt-redshift
2- cloned the project
3- switched to a branch that is in a working state
4- ran dbt clean
5- ran dbt deps
Relevant log output
No response
Environment
- OS: POP_OS 22.04
- Python:3.10.11
- dbt:1.5.1

Which database adapter are you using with dbt?
redshift
Additional Context

Thanks for reaching out @heshamh96 !
Could you share a copy of your packages.yml file?
Also, could you try each of the following options and see if they work for you or not?
The most recent revision of this repo:
packages:
  - git: "https://github.com/dbt-labs/dbt-labs-experimental-features"
    subdirectory: "materialized-views"
    revision: 7200ac20a070ac7ba0eb13707453df1a1fa87380

main branch of that same repo:
packages:
  - git: "https://github.com/dbt-labs/dbt-labs-experimental-features"
    subdirectory: "materialized-views"
    revision: main

Side note: if you are interested in materialized views, they are going to be included in dbt v1.6 🚀  Read more here: https://github.com/dbt-labs/dbt-core/issues/6911

yeah sure ...
packages:
  - package: dbt-labs/codegen
    version: 0.9.0

  #- package: dbt-labs/dbt_utils
  # version: 1.1.0

  - git: https://github.com/dbt-labs/dbt-labs-experimental-features
    subdirectory: materialized-views

  - package: calogica/dbt_date
    version: 0.7.2

i just would like to let you know that after troubleshooting... it's resolved for me
apparently  dbt deps when trying to clone the subdirectory from this second package which is  a package on github
it can't generate because there was a git configuration called
clone.defaultremotename=all
this value was unset on windows configuration
so when i tried to unset it in Linux using the following command
git config --global --unset clone.defaultremotename
it got resolved .... so i think dbt deps is senstive to such configuration ... not sure why honestly ... but as i said it's now fixed

Cool -- glad you were able to fix it @heshamh96 !
I'm going to close this as resolved, but please reach out if anything else is needed and we can open an new issue or re-open this one.


fix/2153 div-overlap

Fixes Issue #2153

Changes proposed
Minor style changes in the component [...subcategory].tsx

Screenshots
Before the fix

After the fix

@Anmol-Baranwal
Please take a look at this PR.


Default material slider thumb size is too small

The radius is currently 6 but it should be 10.

Consequently, the bottom lobe radius of the value indicator should also be updated to 10 from 6.


can't Install t2-cli using npm

Hello team,
I'm using npm to install t2-cli with command:
npm install -g t2-cli
I got this error log when install using.
Mac-Pro-Touchbar:~ victor$ sudo npm install -g t2-cli
npm WARN deprecated<EMAIL_ADDRESS>Package no longer supported. Contact<EMAIL_ADDRESS>for more info.
/usr/local/bin/t2 -> /usr/local/lib/node_modules/t2-cli/bin/tessel-2.js
/usr/local/bin/cargo-tessel -> /usr/local/lib/node_modules/t2-cli/bin/cargo-tessel.js

><EMAIL_ADDRESS>install /usr/local/lib/node_modules/t2-cli/node_modules/usb
> node-pre-gyp install --fallback-to-build

node-pre-gyp WARN Using request for node-pre-gyp https download
node-pre-gyp WARN Pre-built binaries not installable for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v57 ABI, unknown) (falling back to source compile with node-gyp)
node-pre-gyp WARN Hit error EACCES: permission denied, mkdir '/usr/local/lib/node_modules/t2-cli/node_modules/usb/src/binding'
gyp WARN EACCES user "root" does not have permission to access the dev dir "/Users/victor/.node-gyp/8.11.3"
gyp WARN EACCES attempting to reinstall using temporary dev dir "/usr/local/lib/node_modules/t2-cli/node_modules/usb/.node-gyp"
gyp WARN install got an error, rolling back install
gyp WARN install got an error, rolling back install
gyp ERR! configure error
gyp ERR! stack Error: EACCES: permission denied, mkdir '/usr/local/lib/node_modules/t2-cli/node_modules/usb/.node-gyp'
gyp ERR! System Darwin 18.2.0
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "configure" "--fallback-to-build" "--module=/usr/local/lib/node_modules/t2-cli/node_modules/usb/src/binding/usb_bindings.node" "--module_name=usb_bindings" "--module_path=/usr/local/lib/node_modules/t2-cli/node_modules/usb/src/binding" "--napi_version=3" "--node_abi_napi=napi" "--napi_build_version=0" "--node_napi_label=node-v57"
gyp ERR! cwd /usr/local/lib/node_modules/t2-cli/node_modules/usb
gyp ERR! node -v v8.11.3
gyp ERR! node-gyp -v v3.8.0
gyp ERR! not ok
node-pre-gyp ERR! build error
node-pre-gyp ERR! stack Error: Failed to execute '/usr/local/bin/node /usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js configure --fallback-to-build --module=/usr/local/lib/node_modules/t2-cli/node_modules/usb/src/binding/usb_bindings.node --module_name=usb_bindings --module_path=/usr/local/lib/node_modules/t2-cli/node_modules/usb/src/binding --napi_version=3 --node_abi_napi=napi --napi_build_version=0 --node_napi_label=node-v57' (1)
node-pre-gyp ERR! stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/t2-cli/node_modules/node-pre-gyp/lib/util/compile.js:83:29)
node-pre-gyp ERR! stack     at emitTwo (events.js:126:13)
node-pre-gyp ERR! stack     at ChildProcess.emit (events.js:214:7)
node-pre-gyp ERR! stack     at maybeClose (internal/child_process.js:925:16)
node-pre-gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:209:5)
node-pre-gyp ERR! System Darwin 18.2.0
node-pre-gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/t2-cli/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build"
node-pre-gyp ERR! cwd /usr/local/lib/node_modules/t2-cli/node_modules/usb
node-pre-gyp ERR! node -v v8.11.3
node-pre-gyp ERR! node-pre-gyp -v v0.11.0
node-pre-gyp ERR! not ok
Failed to execute '/usr/local/bin/node /usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js configure --fallback-to-build --module=/usr/local/lib/node_modules/t2-cli/node_modules/usb/src/binding/usb_bindings.node --module_name=usb_bindings --module_path=/usr/local/lib/node_modules/t2-cli/node_modules/usb/src/binding --napi_version=3 --node_abi_napi=napi --napi_build_version=0 --node_napi_label=node-v57' (1)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-pre-gyp install --fallback-to-build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/victor/.npm/_logs/2018-11-21T14_56_36_092Z-debug.log
Mac-Pro-Touchbar:~ victor$

I've tried to re-install npm & node to the latest version as mentioned in this link . However it's not helping either.
I've tried on multiple machine, both PC & Mac but the same issue.
Could you look at the issue?
THank you.

Hi,
After doing both steps on this helping link, I can install t2-cli.
I thought one of them is enough. But it requires both.
Anyway, the issue is solved now.
Thanks.


Plus d'entitée climat depuis la version 5.0

Describe the bug
A clear and concise description of what the bug is.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Diagnostics File
While the bug is present, download the diagnostics file and attach it to this report.

@CarpeDiem40,
Thanks for reporting, can you attach your diagnostics file please ?
This can be found here =>

@CarpeDiem40,
You have any update with this file for me ?
Kind regards,
Robbe

@CarpeDiem40,
I'll close this one, since this is probably the same as => https://github.com/Robbe-B/maestro_mcz/issues/106
Try to upgrade your Home Assistant to the latest version.
If this doesn't help, completely remove this integration and try to re-install this integration.
If all this, if the issue is still present, leave a comment here with your diagnostics file.


bugfix: save nixpkgs to db

Coverage remained the same at 96.524% when pulling 4f0112d3e7f4cb15f1ead597ea6b18c9179f7230 on bugifx-db-save into b252634585198ca4ad6b2b8de013d1e53f856393 on main.


Memory leak

HI
At start reserved memory are fine for every worker  but a few days later, the reserved memory exceeds a few hundred megabytes, and on only after kill pid or worker restart memory is released .
How can I manually release memory without erasing pid / restarting worker?
Normal view picture:

Pictures where you can see problem:

For testing:
$worker -> onWorkerStart = function($worker){
Timer::add(0.1, function() use($worker){
foreach($worker->connections as $connection) {
$connection -> send('x');
}
});

I think there's some memory leak in your business codes.
The simplest way is add a timer to check the memory and restart the current process like this.
$worker -> onWorkerStart = function($worker){
    Timer::add(10, function() use ($worker){
        if (memory_get_usage(true) > 128*1024*1024 && count($worker->connections) == 0) {
            // Restart current process.
            Worker::stopAll();
        }
    });
};

The best solution is to find the code that causes the memory leak and fix it.

Hi, at this moment i already automaticly restarting proceses with timer:
$maxMemory = ArrayHelper::getValue(Yii::$app -> params, 'workerman.maxMemory','128');
if ($upInsert -> memory >= $maxMemory || ($upInsert -> connections == 0 && $upInsert -> memory >= 20)){
	posix_kill($upInsert -> pid, SIGQUIT);
	if ($upInsert -> memory >= $maxMemory * 2){
		posix_kill($upInsert -> pid, SIGUSR1);
	}
}

About memory leak, i testet also without my code and got that leak. Mby php7.2 have same problems, because i didn't see it for an older versions. For testing i create empty script:
private function Work(){
	$name = "TEST";
	$consoleDir = Yii::getAlias('@console');
	FileHelper::createDirectory($consoleDir . '/runtime/pids');
    Worker::$stdoutFile = $consoleDir . "/runtime/logs/{$name}_stdout.log";
    Worker::$logFile = $consoleDir . "/runtime/logs/{$name}_workerman.log";
    Worker::$pidFile = $consoleDir . "/runtime/pids/{$name}.pid";
	Worker::$eventLoopClass = '\Workerman\Events\Event';

	$context = ['ssl' => []];
	$worker = new Worker("websocket://<IP_ADDRESS>:8910", $context);
	$worker -> name = $name;
    $worker -> count = 4;
    $worker -> user = 'www-data';
    $worker -> group = 'www-data';
    $worker -> reloadable = true;

    $worker -> onWorkerStart = function($worker){
	    Timer::add(0.01, function() use($worker){
	    	if (!empty($worker -> connections)){
		        foreach($worker->connections as $connection) {
		        	$connection -> send('x');
		        }
	    	}
        });
    };

	$worker -> onMessage = function($connection,$data) use ($worker){
		//after some time this answer fills memory and after client reconect memory not
 		echo "WE GOT ANSER!!! \n";
	};

	Worker::runAll();
}

I think we better not run Yii in Workerman, because Yii is not designed to run persistently in the process, that may make memory leak.

Im runing workerman from yii2-console (CLI) with disabled debuger and logger :
if (php_sapi_name() == "cli" || basename(dirname(getcwd())) == "api" ){
	defined('YII_DEBUG') or define('YII_DEBUG', false);
}

Yii::$app -> log -> targets['file'] -> enabled = false;

if I'm not mistaken, the problem occurred after the php version upgrade to 7.2

I can‘t tell where your business codes have memory leak. May be the Tablets class, you must find it by yourself.

@walkor hi, which profiler or service you will recommend to use on production with workman to find memory leaks?


Question: Ignore SSL certificates

I'm using spring-data-jest to connect to AWS ElasticSearch service, which is inside the VPC. In order to access ES, I ssh into an EC2, which is inside the same VPC and use SSH tunneling (Forwarding the localhost port 9200 to ES endpoint port 443).
It works fine except the fact that SSL certificate is not valid on localhost anymore. I can overcome this when using curl with -k or accessing through browser. Is there anyway to configure Spring data jest to ignore SSL certificate?

You can find some information in section SSL of Jest library : https://github.com/searchbox-io/Jest/tree/master/jest


mostly note to self: callback appear to be missing

went through the code and saw this:
https://github.com/rjrodger/seneca-perm/blob/master/lib/EntityAccessControl.js#L45-L46
I'll look at it when debugging why some 3rd party ACLs tests are not passing.

(link point to a file that does no longer exist)


Add list of applied chargeback rate names as field in chargeback reports

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1369442
Add applied chargeback rate names to chargeback report

@gtanzillo @enoodle Please review
cc @simon3z

@miq-bot add_label providers/containers

👍 LGTM

Euwe Backport details:
$ git log
ommit 96ec8eee944accebe6d5759b2c913bd48c5953b1
Author: Gregg Tanzillo<EMAIL_ADDRESS>Date:   Mon Oct 3 08:11:21 2016 -0400

    Merge pull request #11523 from zeari/chargeback_rate_name

    Add list of applied chargeback rate names as field in chargeback reports
    (cherry picked from commit b06ad9ced9b706446f1290f8e45f96f9ee85c670)


Question: Possible to pass vars directly to dependencies?

Is it possible to call a dependency defined in a terragrunt file with variables defined in that file, instead of having the var file logic defined in the dependency's config?
Right now, it looks like a terragrunt config file:
terragrunt = {
  remote_state = {...}
  dependencies = {
    paths = ["${get_tfvars_dir()}../../../../../modules/security_groups"]
  }
  terraform {
    source = "git::https://mycompany.com/myteam/terraform-modules//asg"
    extra_arguments "all_configs" {
      optional_var_files = [
        "${get_tfvars_dir()}/../../region.tfvars",
      ]
    }
  }
}

Will simply call terraform on the modules/security_groups module with no extra options.
We're wondering if there's a way to pass the optional_var_files values to the dependencies defined here.  Otherwise, the dependencies referenced here will need to assume the folder structure of the var files in the main terragrunt config, which can prove brittle.
Thinking something like:
dependencies = {
  paths = [...]
  extra_arguments "configs" {
    optional_var_files = [
      "${get_tfvars_dir()}/../../region.tfvars"
    ]
  }
}

Or perhaps as a workaround for our particular problem, from within our dependency configuration,  is there a way to get the path of the file that called the dependency?  Neither get_tfvars_dir() or path_relative_to_include() help determine the caller's path when used in our dependency config.

Not sure I understand the question. The dependencies block is solely used when running xxx-all commands (e.g., apply-all) to determine ordering. There's no "calling" a dependency from another dependency...
Could you explain your use case a bit more?

Sure @brikis98.  Consider the following folder structure (we're still on TG v0.18) where we want to call a custom module that resides in its own /modules directory:
modules/
  security_groups/
    main.tf
    terraform.tfvars
prod/
  us-east-1/
    terraform.tfvars
  env.tfvars
global.tfvars

Below is our main terragrunt config for a region.  You can see in this file, we pass in the chain of var files based on the folder structure to stand up this region's infrastructure:
# /prod/us-east-1/terraform.tfvars
terragrunt = {

  dependencies {
    paths = ["${get_tfvars_dir()}/../../modules/security_groups"]
  }

  terraform {
    source = "..."

    extra_arguments "region" {
      optional_var_files = [
        "${get_tfvars_dir()}/../../global.tfvars",
        "${get_tfvars_dir()}/../env.tfvars",
        "${get_tfvars_dir()}/terraform.tfvars"
      ]
    }
  }

}

But we also have a custom, shared module in the root directory that we wish to call as a dependency, that lives in /modules/security_groups:
# /modules/security_groups/terraform.tfvars

terragrunt = {
  terraform {
    extra_arguments "security_groups" {

      # Here, we want to access the same var files that are set in the module
      # that depends this one, but this module has no way to reference them.

      optional_var_files = [
        "${get_tfvars_dir()}/../../global.tfvars", # these don't work
        "${get_tfvars_dir()}/../env.tfvars",
        "${get_tfvars_dir()}/terraform.tfvars"
      ]
    }
  }
}

The above module needs a bunch of region-specific configuration to build security groups: it can't do it on its own without duplicating configs.
Ideally, this security group module would have access the same var files as the main module in /prod/us-east-1/terraform.tfvars.  But since the security group module resides at the root directory, it has no idea about the folder structure of the region module nor its var files.
We're hoping that when terragrunt runs a dependency, there would be a way to pass var files (or maybe just an extra_arguments block) to that dependency, ie:
dependencies {
  paths = ["${get_tfvars_dir()}/../../modules/security_groups"]
  extra_arguments { ... }
}

Or perhaps, a function like get_caller_tfvars_dir that we can call from within a dependency to find the path from where it was called, ie
# /modules/security_groups/terraform.tfvars
optional_var_files = [
   "${get_caller_tfvars_dir()}/../../global.tfvars",
   "${get_caller_tfvars_dir()}/../env.tfvars",
   "${get_caller_tfvars_dir()}/terraform.tfvars"
]

Or am I just doing this all wrong??

I don't follow why you have a modules/xxx/terraform.tfvars. Modules should be defined in Terraform code (.tf files), not Terragrunt .tfvars files. You deploy those Terraform modules by creating Terragrunt .tfvars files in the folder structure that represents your actual live environment: e.g., in folders under prod/us-east-1/xxx. Each of those .tfvars files specifies the variables/settings it needs and/or uses include to pull in common variables/settings from the root .tfvars file for that environment.
See https://github.com/gruntwork-io/terragrunt#keep-your-terraform-code-dry.

Well, right, that's why I started this issue, and having TG config in /modules was just an example of one thing I tried to get dependencies working.  I realize the way I explained above is wrong, and I'm looking for a solution, hence the original question.
We may be talking past each other.  I'll try to explain another way.
Given this folder structure:
modules/
  security_groups/
    main.tf
prod/
  us-east-1/
    terraform.tfvars --> this points to an external module on github, etc
  env.tfvars
global.tfvars

This is a common folder structure we use across our environment.  But in this case, this particular application requires some extra terraform code to be applied when /prod/us-east-1 is applied.
If the user creates files under modules, how can those modules be included into the terragrunt pipeline?  How can we ensure these modules are called when we run terragrunt apply from /prod/us-east-1/?

terragrunt apply is equivalent to terraform apply. It is meant to be used with a single Terraform module. So one option is to solve it exactly the way you would if Terragrunt didn't exist: take whatever module prod/us-east-1/terraform.tfvars is deploying (the thing in the source parameter) and add to it the "extra terraform code under /modules" that you wished deployed each time apply is called. If the thing in the source parameter is an external module, then create a new module of your own that has this structure:
module "external" {
  source = "<THE SOURCE URL IN GITHUB">
  ...
}

module "mine" {
  source = "../security_groups"
  ...
}

terragrunt apply-all will run terragrunt apply on every module it finds in the current folder or any of its children. So another option is to have a prod/us-east-1/foo and prod/us-east-2/bar and to always run apply-all in the prod-us-east-1 folder to deploy them both together. See https://github.com/gruntwork-io/terragrunt#execute-terraform-commands-on-multiple-modules-at-once.

Sorry for the delay in response.  We ended up creating a shared module and used symlinks in our terragrunt directories to point at this symlink, which terraform will honor.  We'll be looking at your suggestion as well.  Much appreciated!


Add free Git Course in PT-BR.

What does this PR do?
Add Resource
For resources
Description
Free Git Course with 200,000 students enrolled
Why is this valuable (or not)
How do we know it's really free?
I'm the author
For book lists, is it a book?
Checklist:

[X] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

Thanks, and welcome to free-programming-books!


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v3.8 to before v5.10.83",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=9c4566a117a6fe404a0e49b27ac71b631945a70f",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=0effb7f51b65840bb129f52ddc765105c48c87ba"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "9c4566a117a6fe404a0e49b27ac71b631945a70f",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "0effb7f51b65840bb129f52ddc765105c48c87ba",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "ARM: socfpga: Fix crash with CONFIG_FORTIRY_SOURCE\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.10.83 by commit 0effb7f51b65840bb129f52ddc765105c48c87ba, it was introduced in version v3.8 by commit 9c4566a117a6fe404a0e49b27ac71b631945a70f. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002470


Add Construction section to all Crocks

Better construction, better docs

This PR does the same thing that was done on this PR but this time for the crocks that are documented thus far.

Coverage remained the same at 100.0% when pulling df9a1145dc70fe1cbd8d98a1df018a341824bf7d on doc/crocks-construction into 427b773f42df2b4e988294333c563ac658faf077 on master.

Okay I addressed most of the comments, will be updating those example for you @bennypowers at a later time, but they will get in there.

Cool. Are you planning on putting them in this PR or a later one?

@bennypowers a later one, if that is cool. I wanna restructure that entire file.
There is a lot of vague and misleading use cases for that whole type


New package: EvidentialFlux v1.1.2

Registering package: EvidentialFlux
Repository: https://github.com/DoktorMike/EvidentialFlux
Created by: @DoktorMike
Version: v1.1.2
Commit: 15333748b79e34bdc6c06f8630410d525455d95a
Git reference: HEAD
Description: Evidential Deep Learning Layers for Flux

This PR should be deleted. I made a naming mistake. I cannot seem to delete it myself though..


Add xsd files to docker images

Adds product and purchase order xsd files to default locations so users running through the examples can create products and purchase order versions. Previously, users would have to copy the files from the Grid directory into the running container and in the correct location.

This PR should close #1141


Add missing shortcut help for 'r'

The keyboard shortcuts list was missing 'r' (refresh).

Thanks!


super call in nested scope.

TypeScript Version:
1.9.0-dev / nightly (1.9.0-dev.20160311)
Code
class A
{
	public Foo($value : any) {...}
}

class B extends A
{
	public Foo($value : any) {...}

	public Bar()
	{
		var that = this;

		// #1: Can't hold a reference on super.
		// var base = super; // Error.

		var baseFooFuntion = super.Foo;

		Object.someMethod(function()
		{
			// #2: Is somehow not calling the method with the correct arguments?!?
			baseFooFuntion(42);

			// And you cant use super.Foo(42); => 'super' can only be referenced in members of derived classes or object literal expressions.
			// So that it produces an error is fine.
		});
	}
}

Expected behavior:
baseFooFuntion(42); call should work.
Actual behavior:
Not working because arguments are not passed correctly.

There isn't really a bug here; it seems like you have several questions about how super ought to work (for example, a naked reference to super is nonsensical in JavaScript). Reading the FAQ on orphaned this (https://github.com/Microsoft/TypeScript/wiki/FAQ#why-does-this-get-orphaned-in-my-instance-methods) and/or asking a question on Stack Overflow should be your next steps.


model with gripper for development

see #62

And we finally have a resolution here via #71. Thanks!


Release Notes with links to GitHub commits and pull requests.

Currently, the Jira Release notes output in the following format:
PROJ-9999 Jira Issue Title
It is great for internal release notes, but not so nice if you use this plugin to integrate Jira to GitHub.
I propose to add a release notes format, in which each item links to the GitHub #issue, commit(s) and/or pull requests created by the GitHub integration plugin.
#9999 Jira Issue Title

Similar to what this project is doing https://github.com/github-tools/github-release-notes

Hi @ey51. I'm from a team at Atlassian that is currently in the process of migrating the ownership over to us from GitHub. As a part of this, we're currently going through the backlog of issues and prs to see what can still be addressed and what should be closed.
I've linked your issue to one of the tickets in our backlog so our team can work on addressing this issue.


[WARN] [1599118979.033952]: Unsupported QA TYPE received: objetive_error

commit: df76904bb6e2b5f78eefe65c5e406ed6efc0f454 (melodic branch)
test MODELS20:
./run_single_sim.sh -p 1 -g 1 -n "f1_v2_r2" -o 2 -r "true" -p 1.7 -c "false"

Suggestion: include a verbatim copy-paste of "all" output, not just a single line.
Context can be important when trying to diagnose these sort of issues.

Thanks for the reminder. I normally do, but I have so many prints in mros_reasoner.py that it is a mess. Tthis corresponds to a feature not implemented, so this is simply a reminder for me.

I think this error comes from an additional observer we added before.  It's solved in https://github.com/rosin-project/metacontrol_sim/commit/9fe54db8af7ac01005fc74e09155eb85e9454ecd

I think this error comes from an additional observer we added before. It's solved in rosin-project/metacontrol_sim@9fe54db

That is the case. Thanks @marioney


Add system image, please.

Hello! I'm trying to build your OS, but there is no Perception.iso file. Can you ad it, please?

I can be placed in the "Releases" section.

Yes, I will be cool. Thank you.

I'm hesitant to share out a disk image until I get something actually fun to play with.


list: add "is_not_empty" requirement to "head" and "tail" (analogous to "vec")

Fixes #1105

Thanks!


Additional jars are loaded twice

Once for check and once for commands. Caching the loaded classloader should be enough.

The AdditionalResourcesTest is currently blocked with this because Quarkus cannot deal with different configuration values in different tests so invalid path and empty path will not be possible until Quarkus will support this.


[L] DUP: Update AWS SDK to V3

Description:
Update AWS-SDK from V2 to V3. When using V2 it is fairly common to see warnings about outdated functionality along with a reminder V2 is nearing end of support. This task is go through the process of updating to V3, resolve all the warnings, and test the app to ensure any breaking changes caused from this update have been resolved.
Amazons guide for Updating to V3
Acceptance Criteria:
Day use pass now uses AWS-SDK V3.
Development Checklist:

[ ] all warnings about v2 to v3 have been resolved
[ ] Maintain functionality
[ ] API, Public, Admin now using V3

Dependencies

Blocked by #336
Blocking

Relevant documentation as reference
Definition of Ready

[ ] Acceptance criteria are included
[ ] Wireframes are included (if applicable)
[ ] Design / Solution is accepted by Product Owner (if applicable)
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

Definition of Done

In progress:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] UI meets accessibility requirements
[ ] Unit tests are written
[ ] Work is traceable in GitHub
[ ] PR linked to ticket number
[ ] If needed/required - Dev adds flag/label to highlight any migration steps necessary prior to PROD deployment

Code review:

[ ] Code is peer reviewed and has passed CI/CD tests

QA:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] Code is potentially shippable to the production environment
[ ] Functional features have been tested and passed by QA
[ ] UI components tested by designer
[ ] Code is deployed to PROD when moved to 'done' column (unless requested otherwise by PO)

PO Review:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] Reviewed and approved by Product Owner

Notes:

MPORTANT: Switching to V3 will cause successful tests to fail in the GH environment. The issue has something to do with the asynchronous startup of DynamoDB. You MUST incorporate a sleep command into the on-pr test routine to WAIT for DynamoDB to be ready by the time tests run. Also use --runInBand to run tests in series to reduce the consecutive dynamodb load. Review here to see implementation

May want to wait to do this until the SAM update is done. We'll move this to the following sprint (as per sprint planning convo)

Notes from refinement May 28, 2024:

needs to be done, but wait until SAM is done (#336 )
V2 SDK is still good until Sept (probably)

Done but can't be released until SAM stuff is done

Just thought I would write a comment on this ticket because I think it should either remain in QA until an issue is discovered from it, or be closed.
There isn't any front end functionality that has been changed from this implementation of this ticket. If there is any issues that arise from this upgrade it would be discovered during the testing of the SAM API ticket.  During that process if any SDK errors arise it will be obvious from the logs, and to this point none have popped up.
@manuji  you have asked me for a list places/things you could check to determine if this work has been completed, we originally created this ticket because of the warnings displayed during building. I can confirm those are no longer appearing.  You should be able to check in your build logs aswell for any warnings of "AWS-SDKv2 requires upgrade". Aside from that I could show you where the version numbers are located. Let me know how you would like to proceed with this.

Moving to blocked based on Chris' comment. @manuji  let us know your thoughts :). cc @LindsayMacfarlane

Hi @Dianadec , as per @Christopher-walsh22 comment, I think it's safe for us to move this to Done. Thank you

Thank you @Christopher-walsh22  and @manuji for your work on this ticket. Based on recommendation (eeek - from 2 weeks ago) - I will now close!


Remove containers created from tests

Running make test local leaves a lot of containers around. Some of them still running. Using --rm gets rid of many of them, but causes circleci to break. CircleCI can be made to work by checking for specific lines in output to avoid btrfs errors. The remaining containers can be removed with a manual make test-clean.
I've created a patch to address this issue. I have already used up my one fork of nginx-proxy so I'm linking to a commit. https://github.com/dmp1ce/nginx-proxy-circleci/compare/f4c0239918c4...c88b58ce6e92?&diff=split&name=c88b58ce6e92
I have verified that circleci still works too. https://circleci.com/gh/dmp1ce/nginx-proxy-circleci
Ideally, all containers created by a test would be cleaned up in a teardown function in BATS. My patch doesn't do that.

@dmp1ce Is the -l 0 part what makes it ignore the btrfs errors on CircleCI? If so, it seems like that should be noted as comments in the test file, since it's otherwise unclear why the -l 0 flags are there.

That's right. It is too bad that there needs to be special code for circleci but I don't think it makes a difference in this case as the output should only be one line. I'll go ahead and add the comments.

maybe it's time to give a try to TravisCI (See #312)

We're very happy with Gitlab CI btw. but I don't know if they offer a free service.

No longer relevant.


Handling of "no submission left" for file upload exercises

Is your feature request related to a problem?

When assessing file upload exercise submissions in exams, and there is no submission left but one clicks on "assess next submission", only an empty page is displayed.
Describe the solution you'd like

Ideally there would be an information "all submissions have been graded" and a button that takes you back to the assessment dashboard of the respective exam.
Describe alternatives you've considered

Alternatively, an error message like "no submission left" would already be better than the current solution. However, to provide consistency with text exercises, the above descried solution would be better.
Additional context

I already added a fix for this in #2994.

potential duplicate of #2702


remove warning

Coverage decreased (-6.03%) to 86.643% when pulling 215a60314b0a2ffc5c03fab865774bc2d5fd594c on hooopo:master into 261f213d8955ca416625d19dbf4e5cff007ef55b on endofunky:master.

Coverage decreased (-6.03%) to 86.643% when pulling 215a60314b0a2ffc5c03fab865774bc2d5fd594c on hooopo:master into 261f213d8955ca416625d19dbf4e5cff007ef55b on endofunky:master.


Error: unable to get local issuer certificate

Repro steps:

Action: azureFunctions.deploy
Error type: UNABLE_TO_GET_ISSUER_CERT_LOCALLY
Error Message: unable to get local issuer certificate
Version: 0.15.0
OS: win32
VS Code Version: 1.31.1

Call Stack
TLSSocket.onConnectSecure (_tls_wrap.js:1055:34)
TLSSocket.emit (events.js:182:13)
TLSSocket._finishInit (_tls_wrap.js:635:8)

@Govindmallurwar are you behind a corporate proxy or something similar? We've been mostly relying on VS Code to fix this for us, which is tracked here: https://github.com/Microsoft/vscode/issues/60773. I believe most situations are fixed already, but perhaps not all.

Could try something like this: https://github.com/Microsoft/vscode-docker/pull/627

I was debugging with Fiddler and was able to repro this bug since it inserts a cert to decrypt https traffic. However, it's fixed for me on VS Code insiders - see here for more info: https://github.com/Microsoft/vscode/issues/52880
Closing as external


updated dog.json

Please consider my pull request.
I can add more breed.

Hey @radhika2503
I am so sorry but this repo has been excluded from Hacktoberfest, yet, I'll take your Pull Request for the sake of Contribution
Thank you so much!


Bug in c header file

Hi. There is a bug in the c header file generation with the reserved-padding. It occurs when only the last bit of a field is not used (reserved) and the one before is. This results in a c-struct with 31 bits instead of 32, which is not good.
I can't test it right now, but my first guess is that it is a off-by-one mistake in the c_header.j2 file. The
{%if tmp.lsb < config.data_width - 1%}

should probably be
{%if tmp.lsb < config.data_width%}

instead. Maybe someone can confirm?
In the generated md-file the padded reserved bit is generated correctly.

Hi, thank you for the report. Need to investigate.
Interesting that you refer the line that was changed within previous wave of fixes:

#25
#26


Question: How can I embed swagger UI?

How can I embed swagger UI? For example like so:
url := ginSwagger.URL("http://localhost:8080/swagger/doc.json")
engine.GET("/swagger/*any", ginSwagger.WrapHandler(swaggerFiles.Handler, url))

I need get all interface doc by httprouter, including, but not limited swagger, Now, is there any other ready-made solution?

Here's a vanilla solution, no server side dependencies:
Have a folder called swagger and use a static route to serve your swagger.yaml or swagger.json file and add this html file in:
<!DOCTYPE html>

    <meta charset="utf-8" />
    <meta name="viewport" content="width=device-width, initial-scale=1" />
    <meta name="description" content="SwaggerUI" />
    <title>SwaggerUI</title>
    <link
      rel="stylesheet"
<EMAIL_ADDRESS>    />

Note: if you're using .json change the swagger.yaml to swagger.json.
Then static serve the folder from your httprouter
router.ServeFiles("/swagger/*filepath", http.Dir("swagger"))

This will serve the swagger ui file from localhost:4000/swagger/swagger.html

Aside: For generation from source code I use
Use swag or similar to annotate your route handlers
swag init -g cmd/api/main.go -o ./swagger

outputs swag files to a folder called swagger

If anyone wants an example repo, comment asking for one, and I'll make one when I have time


Update kontainer driver schemas

This change adds a schema version field so that we can track what resource
version of a dynamic schema a schema was created from, to minimize the amount
of updates we need to do.
Issue:
https://github.com/rancher/rancher/issues/17712

LGTM

LGTM


[5.2] Fix numeric array index causing valid validation

When validating data when using an ID number for the array key in a post request, for example. The use of array_merge would cause the numbers to start from 0 rather than the ID number for the item.

This is a breaking change is it not?

I wasn't intending it to break anything, only thing I'm not completely sure about is the PHP version that the array union operator was added.
<?php

$data = [
    '_token' => 'hjvfdsho',
    'name' => 'Matt',
    '10' => 'hello',
    '20' => 'world',
];

$files = [
    'name' => 'foo'
];

var_dump(
    array_merge($data, $files),
    $data + $files,
    $files + $data
);

Result:
array(4) {
  '_token' =>
  string(8) "hjvfdsho"
  'name' =>
  string(3) "foo"
  [0] =>
  string(5) "hello"
  [1] =>
  string(5) "world"
}
array(4) {
  '_token' =>
  string(8) "hjvfdsho"
  'name' =>
  string(4) "Matt"
  [10] =>
  string(5) "hello"
  [20] =>
  string(5) "world"
}
array(4) {
  'name' =>
  string(3) "foo"
  '_token' =>
  string(8) "hjvfdsho"
  [10] =>
  string(5) "hello"
  [20] =>
  string(5) "world"
}

The only differences between the results are the the union operator keeps the keys. The only other difference is the other in which the array keys are merged when conflicted, but if you are naming multiple form inputs the same your probably going to have problems. I can switch the inputs to the operator around if you prefer.

Holding off on this for now.

Ok @taylorotwell, would you consider an alternative solution such as using a loop? Or even sending a pull request to master?


Concurrent Execution Panic Fix

This fix addresses #96 and 0xPolygonMiden/miden-vm/#602. I've summarized the fix here.

Yes, you got it right! I should elaborate it a bit more on the issue why the code was not panicking when we have a single fragment.


🐛 Bug Report: Missing 'Create' from bucket permission settings

👟 Reproduction steps
Fresh install v1.1.0
👍 Expected behavior
Be able to tick on 'Create'
👎 Actual Behavior
No 'Create' option for permission and not able to upload file to the bucket
🎲 Appwrite version
Version 1.0.x
💻 Operating system
Linux
🧱 Your Environment
Appwrite 1.1.0
Linode ubuntu 22.04

👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

@MaddPenguin thanks for raising this issue! 🙏🏼 We'll look into this ASAP!


Wrong DLL for IWinHttpRequest

The doc at https://docs.microsoft.com/en-us/windows/desktop/winhttp/iwinhttprequest-interface states that winhttp.dll is required for the type library.
However the type library has been moved to winhttpcom.dll in Windows 1803, 1809 and 19h1.

Hi @gerikintercede -- thank you for letting us know! We'll work on updating the doc.


WHACK-22: Remove Jetty

The first commit in this PR is utterly useless, but I kinda like that it's been waiting to be merged since 2016. :)

I've not tested this change, other than that it compiles.

I've reconsidered this PR. Lets remove Jetty, rather than upgrade it. Jetty is a somewhat heavy/complex dependency, that goes mostly unused.
This comment was added to the source code in 2008:

Package not being currently used.
This package was created with the idea that Whack could be
run as a web application running in a webserver (e.g. Jetty).  It would support plugins that
could be dynamically dropped and the server would detect them and use them without a restart.
The plugins could be new external components that were added to the web app. Each external
component could extend the web app and add new configuration pages so that admins could
easily configure the plugins/external components.
Note that conf/jive-whack.xml, security folder and web folder are also not being used.

Let's just remove Jetty, instead of carrying over technical debt.
We can leave the commit that (attempts to) upgrade Jetty from 7 to 12 in, as a starting point for anyone interested in retaining it after all, but I'd be surprised if anyone actually is. If you're reading this and are bothered by Jetty now being gone, please let us know!


System.TypeLoadException: Method 'PromptForCredential' does not have an implementation.

https://github.com/PowerShell/PowerShell/pull/12782 introduced a new abstract member on PSHostUserInterface. This makes it impossible to implement PSHostUserInterface for both PS 7.1 and any earlier PowerShell version. Instead we get a TypeLoadException.
An explanation of this is here: https://github.com/PowerShell/PowerShell/pull/12782#discussion_r464724652
You can find the Host UI implementation (which works today with PS 7.0 and PS 5.1) here.

We're just lucky I was working against the PS master branch (so I could debug the full call stack) -- nothing else caught this

Given the difficulty of turning the abstract call into a virtual one with a sensible behaviour, my recommendation is to revert the change

I agree we should revert the original change.  For password confirmation, I think the cmdlet can just call the existing prompt again and just compare the results.

Maybe we could simply remove the abstract method and do not revert the commit? Or add default implementation?

Maybe we could simply remove the abstract method and do not revert the commit? Or add default implementation?

I'm going to revert the commit so that we don't risk the change being released. Our first job is to unbreak things. We can decide in a new PR whether we reuse the original code or to go with a different implementation.

/cc @Shriram0908 for information.

@rjmholt What about to add optional PSES CI in the repo so that we see issues early?

What about to add optional PSES CI in the repo so that we see issues early?

Not a bad idea really. I'm not sure how we'd achieve that exactly but worth looking into.
/cc @TylerLeonhardt