Revise API Reportback Item accessibility based on user permissions + status

For the mobile app, want to allow users to see their RBs (including pending, flagged, excluded) and other user's public profiles (which includes excluded). Thus the API needs to allow more fine-grained permissions access to reportback items.
Part of the issue seems to be that Northstar uses a single authenticated admin user to hit the Phoenix API, which complicates the capacity to determine proper permissions... although not 100% sure about this.
@aaronschachter @DFurnes

Can we only authorize against the currently logged-in user, or is it possible to provide a UID with the request?

For the currently logged-in user, you'd need to have Northstar authenticate with Phoenix using that user for such calls, but there are other calls that I suspect NS would need to make as an authenticated admin... so it would have to be able to flip-flop between calls to Phoenix as user or admin.

So the only way to authorize a request is against the current session token? Bummer, that makes things a bit more complicated.

Any update on this? If anything, returning all statuses would at least unblock any Profile development in LDT. cc @marahml @jonuy

I believe @jonuy had mentioned the possibility of, for the time being, on the app to just have both an NS auth user and then the anon user (possibly managing two tokens) and using the NS auth user to get all the RB items regardless of status... but not sure if that was going to be an attempted experiment first?

To get to our ideal solution for this, it sounds like we'll need to revamp our user authentication so that user's can auth against Northstar and Phoenix in a similar manner.
I think the work involved for that is here @DFurnes? And it could take a while until we get to that point.
In the meantime, from the mobile app, we really might just have to separately authenticate with the Phoenix API and manage multiple sessions (Northstar and Phoenix).

@jonuy Yup, that's probably the best course of action for now.

The big blocker right now in mobile app is the self can't see their own reportbacks regardless of status. Isn't this just a bug since the Northstar endpoint the mobile app is using should be authenticated as an admin?
I'm wondering if a compromise would be to just keep the reportbacks public regardless of status for now as far as the public profile, instead of each mobile app maintaining two separate authenticated sessions...

@aaronschachter Hm, maybe we can dive into it a little more offline, but I think the Android app is at least able to get a user's own reportback by going straight to the Phoenix API with the rbid. I'm assuming this is the case after a test where I logged out, cleared the local cache of data for the app, and then signed in again with that user to find the reportback info and photo listed in the user's profile.

I had thought that on the mobile app we were only authenticating as the user to log them in, but from that point on keeping a record of their user_id and then the rest of the activity on the app is done as the NS Admin user...
I'd like to avoid having to backtrack on all of the security stuff I recently added to the RBs & RB Items access for the Phoenix API. I fear changing it up will lead to other unexpected issues (potentially).
Instead, I feel managing auth token stuff or switching the token (from user auth to NS admin auth) would be a better approach, cause once a better solution with that gets resolved the RBs & RB Items access wouldn't need to get touched again to re-enable security; it's just the resolved solution for how the app manages access the data.

Sounds like it'd be good to have a meeting on the minds on this one. The iOS app isn't ever pinging at Phoenix reportbacks endpoint for the sake of keeping API requests to a minimum - a single request to user/:user_id/campaigns Northstar endpoint returns all of the data we need to display a user's profile.

Oh right! I'd forgotten that endpoint's what's available. And... from what I can tell from briefly looking at code, it looks like that's what the Android app is doing. But you're saying this endpoint isn't giving you the reportback data from this endpoint?

In the Northstar user/:user_id/campaigns endpoint, reportback_data is only included when Reportback Items which are approved or promoted. Which seems to suggest that Northstar user isn't logged in as an admin (or something's buggy somewhere)

@aaronschachter @jonuy the is to pick this issue up after campaign run stuff is all buttoned up, which I would imagine to be next sprint!

Hey @DFurnes and @weerd. After following up w/ @jonuy and @aaronschachter post-API convo last week, this seems to be most pressing / high-priority launch-blocker. Based on our estimates, looks like wrapping this up by EOD thursday would allow us to hit our target launch week w/ app store submission happening in 1.5 weeks from now.
I know the first next step was discussing a solution in person. Have you two had a chance to sync?
Thanks!

@marahml I'm going to try to wrap up work on DoSomething/northstar#269 today (which should handle Northstar's part in all this). Then @weerd, @chloealee and I are going to sync tomorrow morning to make sure everything's :sparkles: stellar :sparkles:.

Deep dive into code with @weerd, @DFurnes, @chloealee, and @jonuy today uncovered that Northstar User Campaigns endpoint (user/:id/campaigns) was making anonymous requests to Phoenix, which is why the reportback_items` wouldn't exist for non-approved/promoted RB items (https://github.com/DoSomething/northstar/issues/270).
The Northstar user/:id/campaigns endpoint is now deprecated (https://github.com/DoSomething/northstar/pull/273). Closing this issue as it should be resolved when we include the Reportback for a given Signup within the Signups endpoint: https://github.com/DoSomething/phoenix/issues/6151


profile page visit error

page - https://openfuel.org/u/@rahulopenxcell

@dan-online looks like some problem while parsing language tags.

@divy-work I'll look into it


⚠ ClamAV has degraded performance

In 66ae475, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 593 ms

Resolved: ClamAV performance has improved in 592f9e4.


Relative URLs for linking content in Markdown

I'd highly appreciate if there would be a way to link to another file within a Markdown file in a repository by using relative URLs.
Bonobo is able to render Markdown files when opening a repository with the Repository Browser. I want to create a link to another file of the repo using a relative URL, e. g. to CHANGELOG.md.
Because of the way Bonobo is built, I can't do this because the resource blob is located before/on top of the repository's GUID. It would be better if resources like blob or commit were located after/under the GUID.
Right now, I'm using absolute URLs which has several downsides. Another option would be to use URL Rewrite rules by analyzing HTTP Referer, which seems quite shaky to me.

Are there any plans to support this or is there no chance at all?

I doubt it, based on a previous issue the author(s) of BonoboGit seems to think that because it's a Git server and not a "doc server" this is not a good thing. Despite every popular Git host on the planet being able to add this.
(On that note, why is Bonobo even bothering to render Markdown files and then not care about this basic feature?)


Fix go.mod

Add a separate go.mod to crit-go/magic-gen since it's a separate
binary which is not part of the library/package.
Generated by:
	cd crit-go/magic-gen
	touch go.mod
	go mod tidy

Perfect. Thanks.


[Errno 21] Is a directory: '/'

Issue Description
reddit = praw.Reddit(user_agent=userAgent)
reddit.login(username, password, disable_warning=True)
submission = reddit.get_submission(submission_id=id)

raises Exception "[Errno 21] Is a directory: '/'" on Android only. Windows seems to be unaffected and working as normal.
System Information
PRAW Version: 3.6.2
Python Version: 2.7
Operating System: Android 5.1

Sounds like this might be a bug with the python implementation on Android. I didn't even know that was a thing. It would be more useful to post the entire stack trace so we can see where the issue is coming from. Nevertheless, it's not likely something we'll fix. However, I'm happy to look at a PR if you think you can come up with a fix for it.


React 19 support

Package

[ ] lucide
[ ] lucide-angular
[ ] lucide-flutter
[ ] lucide-preact
[X] lucide-react
[ ] lucide-react-native
[ ] lucide-solid
[ ] lucide-svelte
[ ] lucide-vue
[ ] lucide-vue-next
[ ] Figma plugin
[ ] all JS packages
[ ] site

Description
We should test the package in React 19 beta and, if it succeeds, accept React 19.0.0 as a peerDependency.
Use cases
I think it requires no further explanation :)
Checklist

[X] I have searched the existing issues to make sure this bug has not already been reported.

Is there a workaround for now?

I have the same issue, someone get new about it ?

I tested the package with React 19 RC and found no issues. I think we're safe to update peerDeps.

When will support for React 19 be available?
If it's just a matter of adding ^19.0.0 to peerDependencies in the lucide-react/package.json, can I submit a PR for that?

@wojtekmaj Awesome tnx for checking

any update on this, thanks


support data file format version 9

Check for changes since data format 8.
Noticed so far on data files:

preamble was reduced to just list !CLIENTS field syntax

notice about minimum interval of 2 minutes between retrievals has gone

!GENERAL section:

new UNIQUE USERS count

!PREFILE section:

includes VATSIM ID
retains flight plans after users logged in
may contain duplicates

!VOICE SERVERS section has been removed (obsolete since AFV)

Noticed on network status file:

servers.live and servers.dev were added
no notice about fetch intervals

Monitor prefiling data over multiple days, especially on days with lower load (i.e. not on weekends). Not sure if prefilings being kept indefinitely and in duplicates is normal behaviour. This could lead to severe problems on graph import as it is right now.

Additional to points already noticed above:

character set appears to be a little messed up (used to be plain undefined 8-bit, now looks like double encoded UTF-8 sometimes)
!CLIENTS section:

structure:

pilots information appears to have remained unchanged
ATC information has new line-break sequence in ATIS message
ATC no longer lists a voice server

data (constants for ratings etc.) still need to be checked

!SERVERS section unchanged
!PREFILE section:

no client type (OK, same as before)
zeros for lat/lon, GS and altitude, protocol version, rating, transponder, facility type, visual range, heading and QNH (both inHg and hPa)

heading being 0 breaks client type guessing
rating being 0 breaks prefiling plausability check

revision counts up from 0 (may be higher if revised before logon)

duplicates are caused if revised

actual departure time = planned departure time
time of last ATIS and time of logon are garbage
flight plans are now really retained for an indefinite time after logon; prefiled flight plan revision stops incrementing after logon

Although VERSION indicates 8 again, it is still format version 9 for compatibility with too restrictive parser: https://forums.vatsim.net/topic/26710-invalid-vatsim-data-file-version/
It was stated that the format version bump was only due to UNIQUE USERS having been introduced. Data may have been wonky for other reasons; together with #5 it seems that maybe data server has just been switched to a complete reimplementation or old data file format is now generated from JSON or whatever...

Character set detection has been split to #6.

Status file complete. Graph import is not part of parser; check and open separate issue if needed. Character set currently breaks line split and increases the amount of broken free-text; not fixed this time, see #6.
Remaining todo:

check parser for network information


Mention how to leverage Mesos Kill Policy for ZDD

I'd be nice to have a section describing how to drain tasks using the recently introduced kill policy.
See https://github.com/mesosphere/marathon/issues/712#issuecomment-245205402 and https://issues.apache.org/jira/browse/MESOS-4547

MLB will use it out of the box, however as explained elsewhere this doesn't help with draining. Connections still get dropped.
You could, however, use a signal handling shim to do it.


Feat: improve data generator tool

Link to JIRA ticket if there is one: https://ctds-planx.atlassian.net/browse/VADC-1388
Improvements

Improved test data generation step

TODOs:

decide where the code should be placed and how to run it. The current exec -it  solution should be seen as temporary


feat: only update changed modules

improves #2
A fix by TS team still needed.
https://github.com/microsoft/TypeScript/issues/40603

Thanks @AndyOGo


Possible regression

I am using MapZen search and running into several address parsing issues. In issue https://github.com/pelias/pelias/issues/464, @orangejulius and @trescube said there may be a regression in libpostal and suggested posting an issue here to notify / investigate.

Responded on pelias/pelias#464

Hey @b-may, libpostal 1.0 was released earlier today. The new parser correctly identifies all the components of the address you posted, and in general implements the changes I mentioned on the Pelias repo, abbreviating things like "Fort"=>"Ft" for smaller cities, etc.
> 103 BEAL PKWY SE, FT WALTON BEACH, FL

Result:

{
  "house_number": "103",
  "road": "beal pkwy se",
  "city": "ft walton beach",
  "state": "fl"
}


Module 'copilot' not found on Require

I did a fresh install for the plugin but when I tried to call the setup function for copilot it says that the module is not define.

I tried reinstalling packer and copilot.lua and failed to make it work, I'm a bit frustrated because I know that I'm missing something here
use {
  "zbirenbaum/copilot.lua",
  cmd = "Copilot",
  event = "VimEnter",
  config = function()
    vim.defer_fn(function()
      require("copilot").setup()
    end, 100)
  end,
}

use {
  "zbirenbaum/copilot-cmp",
  after = { "copilot.lua" },
  config = function ()
    require("copilot_cmp").setup()
  end
}

See what happens if you run vim.cmd('packadd copilot') before  require('copilot').setup()
If that doesn't work, try to debug while not lazy loading either plugin. Just remove the cmd and event fields, and take it out of the defer.
You can also try pasting in this snippet which I took directly from my own config:
  ["zbirenbaum/copilot.lua"] = {
    after = "nvim-lspconfig",
    config = function()
      vim.defer_fn(function()
        require("copilot").setup({
          suggestion = { enabled = false },
          panel = { enabled = false },
        })
      end, 100)
    end,
  },
  ["zbirenbaum/copilot-cmp"] = {
    after = { "copilot.lua", "nvim-cmp" },
    config = function ()
      require("copilot_cmp").setup({
        clear_after_cursor=true,
      })
    end
  },

Thank you for the help, the issue fix it self over night, I think my packer might had some issues running the config function at the wrong time( talking no sense here), I can close this one.
PD: copilot cmp work like a charm after the issue with copilot require solved itself

Hello hello,
I got the same error after running PackerSync earlier. I was able to fix the error by removing these lines:
  cmd = "Copilot",
  event = "VimEnter",

Copilot and copilot-cmp appear to working now, but does this mean the packages aren't lazy loaded anymore?
Also, if it's useful info, the copilot package was initially saved to the share/nvim/site/pack.packer/opt/ directory, which doesn't appear in the list of directories in the error message above. I was also able to "fix" the error by moving the copilot.lua package from /opt/copilot.lua to /start/copilot.lua. After PackerSync, though, the package at /start/copilot.lua is removed, so not a viable solution. Is there some sort of PATH type variable that I need to update?
in packer_compiled here's what I have after commenting out the aforementioned lines:
  ["copilot-cmp"] = {
    config = { "\27LJ\2\n9\0\0\3\0\3\0\0066\0\0\0'\2\1\0B\0\2\0029\0\2\0B\0\1\1K\0\1\0\nsetup\16copilot_cmp\frequire\0" },
    load_after = {},
    loaded = true,
    needs_bufread = false,
    path = "/home/hobo/.local/share/nvim/site/pack/packer/opt/copilot-cmp",
    url = "https://github.com/zbirenbaum/copilot-cmp"
  },
  ["copilot.lua"] = {
    after = { "copilot-cmp" },
    config = { "\27LJ\2\n]\0\0\3\0\6\0\n6\0\0\0009\0\1\0'\2\2\0B\0\2\0016\0\3\0'\2\4\0B\0\2\0029\0\5\0B\0\1\1K\0\1\0\nsetup\fcopilot\frequire\20packadd copilot\bcmd\bvim-\1\0\4\0\3\0\0066\0\0\0009\0\1\0003\2\2\0)\3d\0B\0\3\1K\0\1\0\0\rdefer_fn\bvim\0" },
    loaded = true,
    only_config = true,
    path = "/home/hobo/.local/share/nvim/site/pack/packer/start/copilot.lua",
    url = "https://github.com/zbirenbaum/copilot.lua"
  },

It looks suspiciously similar to the snippets you provided, but I couldn't figure out where I was supposed to paste them. Do they go in packer.lua with all the other use {} imports? Somewhere else?
Thanks so much. Let me know if I can provide more details.


The ADD ID button is getting stacked down by the VC card

Describe the bug
The ADD ID button is getting stacked down by the VC card
To Reproduce

Unlock the app
Go to  home screen and click on Add ID
Download more than four VC cards

Current behavior
The VC cards stacked one by one below and pushing the ADD id button to the bottom of the screen, to access the button it takes us to scroll down.
Expected behavior
As from the previous version of the app the ADD id button used to hover over the screen, so it wont get stacked down
Screenshots
https://user-images.githubusercontent.com/102220709/200237741-6c39ad60-b1ab-42fa-9faf-1b36a260fcaa.mp4
Smartphone (please complete the following information):

Device: iPhone7 & iPhone XR
OS: 15.4.1
Inji app version: 0.4.0 (16) in test flight
MOSIP Version: 1.2.1
MOSIP server: qa-121.mosip.net

Additional context
Issue in Android is registered https://github.com/mosip/inji/issues/216

@kyanthony can we get Walid's input on this?

Tested on:
iOS: 0.4.1 (3)
WORKS AS EXPECTED
@kyanthony @pmigueld @danicaerediano
Devices:
iPhone 11 - iOS 16.1
Screenshot
https://user-images.githubusercontent.com/102940764/203737462-48c45bf3-8f14-469b-b2c6-5499a374cc84.MP4

Verified on:

Device: iPhone 8 & iPhone 11
OS: 16.0.2 (20A380), 15.6.1
Inji app version: 0.4.1 (4) -> Testflight
MOSIP Version: <IP_ADDRESS>
MOSIP server: qa5.mosip.net

Add ID button does not move down as with credential download working as expected, hence closing the bug.
https://user-images.githubusercontent.com/79500509/207039411-8459e58e-e9d2-4944-83cd-7a04ee165bd6.MP4


Fix install-kube-system when node drainer is enabled

Follow-up for #1043

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1060   +/-   ##
=======================================
  Coverage   35.13%   35.13%
=======================================
  Files          59       59
  Lines        3393     3393
=======================================
  Hits         1192     1192
  Misses       2039     2039
  Partials      162      162

Continue to review full report at Codecov.

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

Tested this and seems to have fixed kube system deploys.
/lgtm


Migration didn't properly handle <table_name>

I have a stored sqlite file from an old android app that I am putting into a new android app. Part of the upgrade process I am upgrading it to use Room. I found this because I needed it to read from an existing file. The problem is, every time I try to run the app in an emulator I am getting the Migration failed error. Here is the error...
java.lang.IllegalStateException: Migration didn't properly handle vegetables_table(com.androidtests.roomdemo.models.Vegetable).
                   Expected:
                  TableInfo{name='vegetables_table', columns={grams_in_cup=Column{name='grams_in_cup', type='INTEGER', affinity='3', notNull=true, primaryKeyPosition=0}, name=Column{name='name', type='TEXT', affinity='2', notNull=false, primaryKeyPosition=0}, id=Column{name='id', type='INTEGER', affinity='3', notNull=true, primaryKeyPosition=1}, long_desc=Column{name='long_desc', type='TEXT', affinity='2', notNull=false, primaryKeyPosition=0}, com_name=Column{name='com_name', type='TEXT', affinity='2', notNull=false, primaryKeyPosition=0}}, foreignKeys=[], indices=[]}
                   Found:
                  TableInfo{name='vegetables_table', columns={}, foreignKeys=[], indices=[]}

However, I can open the database in DB Browser for SQLite and it shows all of the columns and data in the vegetables_table schema, You can see from the screenshot that all of the columns are there. This is the exact file that is in my android project, but RoomAsset is saying there are no columns in the database, even though there are columns and data in the database. Any ideas?

I am also facing this same issue when trying to fetch data from any table. Pleas provide some solution if possible. Thanks in advance.
 Caused by: java.lang.IllegalStateException: Migration didn't properly handle master_outlet

This is not an issue, you just didnt correctly format your Entity!
It must have the exact same Fields and configurations as the Database you want to import!
Take a look here: Its teeling you what it needs
Expected:                   TableInfo{name='vegetables_table', columns={grams_in_cup=Column{name='grams_in_cup', type='INTEGER', affinity='3', notNull=true, primaryKeyPosition=0}, name=Column{name='name', type='TEXT', affinity='2', notNull=false, primaryKeyPosition=0}, id=Column{name='id', type='INTEGER', affinity='3', notNull=true, primaryKeyPosition=1}, long_desc=Column{name='long_desc', type='TEXT', affinity='2', notNull=false, primaryKeyPosition=0}, com_name=Column{name='com_name', type='TEXT', affinity='2', notNull=false, primaryKeyPosition=0}}, foreignKeys=[], indices=[]}
Set this table up as Room Entity an you'll be fine.

add filed into database with default value when its notNull=true
example
ALTER TABLE TABLE_NAME  ADD COLUMN newFiled INTEGER NOT NULL DEFAULT 0

I had the same issue but the above comment maybe is the CORRECT ANSWER. Really appreciatable. Thank you.

What happens is that you changed your table by adding fields or by removing or changing the name and time of doing the migration gave this problem, without putting the changes in the table. But he hopes that his table was in the right expectation, so he found different what he expected, I decided like this:
  val MIGRATION_2_3 = object: Migration (2, 3) {
       override fun migrate (database: SupportSQLiteDatabase) {
           database.execSQL ("" "
                   CREATE TABLE new_Song (
                       id INTEGER PRIMARY KEY NOT NULL,
                       name TEXT,
                       TEXT NOT NULL DEFAULT tag ''
                   )
                   "" ".trimIndent ())
           database.execSQL ("" "
                   INSERT INTO new_Song (id, name, tag)
                   SELECT id, name, tag FROM Song
                   "" ".trimIndent ())
           database.execSQL ("DROP TABLE Song")
           database.execSQL ("ALTER TABLE new_Song RENAME TO Song")
       }
   }

At the end of the page teaches the reason for the official website of Room:


Add additional keys to error response

Some services return additional error keys that may be relevant
in debugging, this adds the ability to add arbitrary keys to the
stubbed error response.
cc @kyleknap @jamesls

:shipit:


Nested documents saved as numeric arrays

With an EmbedsMany relation, subdocuments are saved as numeric arrays whenever their parent is a nested document. This is inconsistent with root documents, where nested documents are saved with a push operation, and as such become a unordered list of objects.
This methodology can be found in every CRUD operation with nested documents, so it seems to be a workaround. Is there any reason for this decision?
For completeness, the code I'm talking about can be found in EmbedsMany.php. Line 43 executes a normal push operation, for nested documents (ultimately) line 236 pushes an entire array.

I still have not find the ultimate way to do embedded relations, and especially multiple levels of embedded models. There is always room for improvement.

That is understandable, as it can be quite complex. Thanks for your quick response!

Hey Jens, have you had any more thought around this issue?


[RFC] Drop Python 3.6 support in v10

We are now discussing to drop support for Python 3.6 in CuPy v10. Feel free to leave a comment here if you have any concerns.
Background:

CUDA Python is unlikely to provide a wheel for Python 3.6, although it can be built from the source without any issue. CUDA Python currently requires -std=c++14 so recent versions of gcc, which is not in RHEL/CentOS 7 by default, is needed. We want to avoid requiring CuPy wheel users to manually install non-default GCC.
NumPy dropped Python 3.6 support in June 2020: https://numpy.org/neps/nep-0029-deprecation_policy.html
Python 3.6 support become EOL in December 2021.

@kmaehashi
We deploy our product with Jetson platform. And currently the jetpack only supports to run python3.6 with TensorRT. they don't provide the 3.7 binding for TensorRT in jetpack.
so from my opinion, this may be the case we need to consider.

@twmht
https://docs.nvidia.com/deeplearning/tensorrt/release-notes/tensorrt-8.html#rel_8-0-1

These release notes are applicable to workstation, server, and JetPack users unless appended specifically with (not applicable for Jetson platforms).
...
‣ Added Python 3.9 support. Use a tar file installation to obtain the new Python wheel files. For more information, refer to the TensorRT Installation Guide.

TensorRT-<IP_ADDRESS>.Ubuntu-20.04.aarch64-gnu.cuda-11.3.cudnn8.2.tar.gz contains Python 3.6~3.9 wheels so I guess JetPack 4.6 can use them?

@kmaehashi
oops. my bad. I thought TensorRT bindinds is bundled with JetPack version. I would try this in next week.
thank you.

@kmaehashi
I found out the TensorRT can't be upgrade standalone in Jetpack.
For example, TensorRT must bundle with specific cuda version, and we can't upgrade cuda version and driver within jetpack.
More context: https://forums.developer.nvidia.com/t/how-to-upgrade-tensorrt-to-7-0-version/110231
Also, I found out python3.6 can't work with cupy, since you introduce contextlib.nullcontext() (https://github.com/cupy/cupy/blob/master/cupy/_core/core.pyx), this is only available in 3.7. Is it possible to use python 3.6 with latest cupy?

With Jetpack 4.6, it seems Python binding wheels included in the above tarball works as it is built for CUDA 10.2 (I don't know why it's not built against CUDA 11.3).

cool. I did not try that since they said it's only cuda11.3.

TensorRT 8.0 GA for Linux and only CUDA 11.3

Let me have a try.

And I also found out we may build python bindings for TensorRT by ourself (https://github.com/NVIDIA/TensorRT/tree/main/python)  . I am working on that.


Add support for --dump-preprocessor

While the flag is supported, it wont provide the output of the generated preprocessor_dump.dm file.
Complimentary issue for the cog:
https://github.com/OpenDreamProject/od-cogs/issues/1

Current major blocker for this is that as of now, we're operating with the understanding that the code volume is read only.
A method to extract the output of these types of extra files will need to be added.


[CT-3317] [Regression] Source freshness disablement no longer works as expected

Is this a regression in a recent version of dbt-core?

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

Current Behavior
When leveraging the config: enabled: structure in the source.yml it will properly disable the sources at runtime (ie, dbt run) but does not seem to disable the sources with dbt source freshness.

Expected/Previous Behavior
When leveraging the config: enabled: configuration in the source.yml it will properly disable the source table on dbt run as well as dbt source freshness.

Steps To Reproduce

Upgrade to dbt 1.7.0
Disable a source in your source.yml
Execute dbt source freshness
See the error that the source was not found

Relevant log output
20:05:39  12 of 12 START freshness of intercom.team_admin ................................ [RUN]
20:05:39  12 of 12 ERROR freshness of intercom.team_admin ................................ [ERROR in 0.33s]
20:05:40  9 of 12 ERROR STALE freshness of intercom.conversation_tag_history ............. [ERROR STALE in 1.26s]
20:05:40  11 of 12 ERROR STALE freshness of intercom.team ................................ [ERROR STALE in 1.28s]
20:05:40  10 of 12 ERROR STALE freshness of intercom.tag ................................. [ERROR STALE in 1.49s]
20:05:40
20:05:40
20:05:40
20:05:40
20:05:40
20:05:40
20:05:40
20:05:40
20:05:40
20:05:40  Runtime Error in source team_admin (models/src_intercom.yml)
20:05:40    404 Not found: Table project:intercom_2.team_admin was not found in location US
20:05:40
20:05:40    Location: US
20:05:40    Job ID: 38d47a6c-7713-485e-b159-939d4c15044f
20:05:40
20:05:40
20:05:40
20:05:40
20:05:40  Done.

Environment
- OS: MacOS
- Python: 3.10.11
- dbt (working version): 1.6.7
- dbt (regression version): 1.7.0

Which database adapter are you using with dbt?
bigquery
Additional Context
I tested on BigQuery and ran into this, but may be present in others.

Thanks for reporting this @fivetran-joemarkiewicz !
I took a swing at reproducing this, and it worked for me with dbt-bigquery 1.7.0 🤔
Maybe you can take a look at my simple example below and help me figure what is different between my setup and yours?
Is there any chance that you changed that boolean within dbt_project.yml from true to false but didn't save it?
Here's my project files:
dbt_project.yml
name: "my_project"
version: "1.0.0"
config-version: 2
profile: "sandcastle-bigquery"

vars:
  intercom__using_team: false

seeds/my_source.csv
id,etl_loaded_at
1,"2023-11-05T00:00:00"

seeds/_seeds.yml
version: 2

seeds:
  - name: my_source
    config:
      column_types:
        etl_loaded_at: timestamp

models/_sources.yml
version: 2

sources:
  - name: my_sources
    database: "{{ target.database }}"
    schema: "{{ target.schema }}"
    loaded_at_field: etl_loaded_at
    config:
      enabled: "{{ var('intercom__using_team', false) }}"
    tables:
      - name: my_source
        freshness:
          warn_after: {count: 6, period: hour}
          error_after: {count: 12, period: hour}

Load data into the seed (so it can be sourced), and then check freshness:
dbt seed
dbt source freshness

Look at the output:
(bigquery_1.7) $ dbt source freshness
23:24:19  Running with dbt=1.7.0
23:24:19  Registered adapter: bigquery=1.7.0
23:24:20  Found 1 seed, 0 sources, 0 exposures, 0 metrics, 447 macros, 0 groups, 0 semantic models
23:24:20
23:24:20  Nothing to do. Try checking your model configs and model specification args
23:24:20  Done.

Hi @dbeatty10 thanks for looking into this! In this case the issue is that when the my_source table in your example does not exist in the destination and you wish to disable it (something we offer in a lot of our packages for higher flexibility) you run into the "404 table not found in location" error.
So in your case if you try the same example, but don't seed the file into your destination I am curious if you get the same error I encountered.

@fivetran-joemarkiewicz Gave it a try with my_nonexistent_source.
I got the following when intercom__using_team: true:
(bigquery_1.7) $ dbt source freshness
15:08:55  Running with dbt=1.7.0
15:08:56  Registered adapter: bigquery=1.7.0
15:08:57  Found 1 seed, 1 source, 0 exposures, 0 metrics, 447 macros, 0 groups, 0 semantic models
15:08:57
15:08:57  Concurrency: 10 threads (target='bigquery-service-account-file')
15:08:57
15:08:57  1 of 1 START freshness of my_sources.my_nonexistent_source ..................... [RUN]
15:08:58  1 of 1 ERROR freshness of my_sources.my_nonexistent_source ..................... [ERROR in 0.44s]
15:08:58
15:08:58    Runtime Error in source my_nonexistent_source (models/_sources.yml)
  404 Not found: Table dbt-test-env:dbt_dbeatty.my_nonexistent_source was not found in location US

  Location: US
  Job ID: 5bd047de-cf4e-494f-b0d5-498417494aca

15:08:58  Done.

But then this when intercom__using_team: false:
(bigquery_1.7) $ dbt source freshness
15:09:15  Running with dbt=1.7.0
15:09:15  Registered adapter: bigquery=1.7.0
15:09:15  Unable to do partial parsing because a project config has changed
15:09:16  Found 1 seed, 0 sources, 0 exposures, 0 metrics, 447 macros, 0 groups, 0 semantic models
15:09:16
15:09:16  Nothing to do. Try checking your model configs and model specification args
15:09:16  Done.

@dbeatty10 huh I am seeing the same results as you now. I wonder if there was some hidden version conflict or conflicting config that I had set previously which was resulting in the initial error. Since I can't recreate the error anymore I think this is a false regression and good to close out.
Thanks again!

Thanks for reporting this @fivetran-joemarkiewicz -- we'll re-open this if you see it again or if someone else reports seeing it.


Clize wet-bulb-temperature-integral

Testing:

[x] Ran tests and they passed OK

@arh89 I believe we did find it necessary to ensure data was not lazy when integrating through vertical levels to avoid it being very slow. As such I have retained this and moved it into the plugin.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

See the complete overview on Codacy


Add experimental infix macro support

Experiment is experimental.
Macros now take a second argument which is a single element list with the previous term.
In the old pre-1.0 days we allowed a list of all the previously enforested terms but this didn't behave well so I'm starting out with just a single term. We can argue our way into relaxing this restriction later.
Note that infix macros are contained within implicit delimiters so:
syntax m = (ctx, prev) => {
  return prev;
}
1 + m

throws an error since prev is the empty list.

What were the issues with access to all previous terms?

I seem to remember it being related to concerns about consuming already defined bindings and splitting terms (though that might be solved by just keeping it contained to implicit delimiters). Maybe @natefaubion can remember more?
btw, just realized this PR is wrong. Need to put it on the macro context and have it be consumed.

In the above example, the implicit delimiter is created as a result of the ongoing BinaryExpression enforestation?

The implications for hygiene were probably the biggest concern. Otherwise it was extremely awkward for pattern matching (you couldn't use custom pattern classes), because it was actually matched in reverse. When combined with custom operators, the implementation and edge cases where very opaque. Unless there is a really compelling use case, I'd advise not to go down that route again 😜. The use case at the time was arrow functions, which is moot now, but could also have been solved with single term look behind.

In the above example, the implicit delimiter is created as a result of the ongoing BinaryExpression enforestation?

yep

The implications for hygiene were probably the biggest concern.

Ah! You made me think of an example:
let x = 1;
syntax m = ctx => {
  let decl = ctx.prev();
  return #`function foo() { ${decl} }`;
}
function f() {
  let x = 2; // this gets consumed by m
  m
  return x;  // unbound? bound to the outer let x?
}

This happens even if we only allow a single term look behind. Totally unclear what should happen if a macro moves an already registered binding into a different scope.

Totally unclear what should happen if a macro moves an already registered binding into a different scope.

We could just disallow this. In fact, I think we should only return expressions from prev.

Also, do you want prev to be a getter? Or keep it as a method in case we change our minds about single term lookbehind?

We could just disallow this. In fact, I think we should only return expressions from prev.

Is { var x } an expression? No, but the enforester (where the infix macro expansion would happen) doesn't know that because processing the insides of a delimiter are deferred until later.
There are lots of ad-hoc restrictions we could impose but I'd much prefer a principled approach because of gotchas like that. Not sure if there is one though.
I think we should put this on ice for now. We can come back to it if we figure out "the right way" or we have a sufficient use-case covered by an ad-hoc restriction.

processing the insides of a delimiter are deferred until later.

You mean in the expander?
It's my opinion that if you're making an infix macro you're doing something akin to complicated syntactic surgery. You have to be very careful to expand the previous term and inspect it before you go tossing it in into a different scope. The very example you give should be put in the docs to illustrate how tricky infix macros are.

We'll have to remember to reset prev when reset is called.

On ice for now. Can reevaluate later


apex infra and a circular reference problem

I'm looking at using terraform and apex to manage my infrastructure and I'm running in to a problem:

Deploy apex managed lambdas
Use "apex infra" to deploy the rest of the infrastructure including IAM roles for the lambdas

Each of these are dependent on each other.  I need the roles in place to deploy the lambdas, but I need references to the lambdas for the rest of the infrastructure (e.g. as API gateway endpoints).
Any ideas on how to get around this?

Ah dup of #377. TL;DR no solution at the moment


Is support Linux Arm  ?

Is support Linux Arm  like Java+Qt Jambi  ?

Thanks ,  i will try on PI

Great.
Remember that our Qml.Net.LinuxBinaries is for x64 only. You'll have to manually compile the libQmlNet.so lib and make sure it is in LD_LIBRARY_PATH (or /usr/lib/).

Can we add support to linux-arm to Qml.Net.LinuxBinaries?


Abbreviation of grams

Hey,
I found that the abbreviation for grams which is used by craft-recipe is "gs" - "g" would be the correct abbreviation.
Example:
150 gs should be 150 g

Same goes for mls -> correct would be ml

Fixed in https://github.com/nystudio107/craft-recipe/commit/2d7d8b18769558bcf9a10509784fbb43939799cf


Can't authenticate with GitHub

Hi !
I can't use GitHub to login on the website. After clicking on GitHub Login, I get redirects to a web page saying :
{"code":-1,"message":"Error 1290: The MySQL server is running with the LOCK_WRITE_GROWTH option so it cannot execute this statement"}
The URL look like that : https://codetime-api.datreks.com/auth/github/callback?code=3f4feXXXXXXXXXXXXXXX&state=%2F

@ThomasRobertson server expired, it has been fixed!

I have only just found this project and having this issue @si9ma.

I have the same problem.

Just found this extension in vscode and tried to login with github, but had the same problem

I apologize for the delay and any inconvenience caused. I take full responsibility for the recent issue and have taken immediate action to rewrite the backend to prevent it from happening again.
Your satisfaction is my top priority, and I am committed to doing everything I can to make things right for you. Thank you for your understanding and patience, and please let me know if you have any further concerns.


Negative Integers

I received these comments via email:

I found a bug: it's not possible to enter negative integers directly, they are parsed as floats.

Example:

> -2 .
-2.0  ok
> 3 5 - .
-2  ok

Here's a simple patch, matching negative numbers by slicing up the string:

--- pygmykernel.py.0    2019-07-15 21:16:26.346736400 +0100
+++ pygmykernel.py      2019-07-18 12:48:45.259047600 +0100
@@ -294,7 +294,9 @@

     '''
     if s.isdigit():
-        # it is a decimal integer
+        # it is a positive decimal integer
+        return (True, int(s))
+    elif s[:1] == '-' and s[1:].isdigit():
       return (True, int(s))
     elif (len(s) > 1) and (s[0] == "'"):
        return (True, ord(s[1]))

This works fine for ASCII numbers in my limited testing. It will be enough for me.

I applied above fix to the negative-integer branch.  Testing is invited before I close the issue and merge the change into master.

I've been using Pygmy with this code almost daily for the last 2 weeks.  I haven't seen any bugs with negative or positive integers, nor with floats.


chore: update @typescript-eslint/eslint-plugin & @typescript-eslint/parser to v8

Purpose
Help dependabot with chore updates.
Approach
Updated and tested locally
Dependencies and/or References
chore(deps-dev): bump @typescript-eslint/parser from 7.2.0 to 7.18.0 in /apps/adapt-essentials-asset-fields
chore(deps-dev): bump @typescript-eslint/eslint-plugin from 7.2.0 to 8.0.0 in /apps/adapt-essentials-asset-fields

cc @whitelisab

hi @zzzarius, Contentful Marketplace Team here - I'd be happy to approve and merge this PR if you'd remove any changes that are outside the scope of your specific app (i.e. the circleci and nvmrc changes). It might also need a rebase.


handling new seasons

will this behave correctly when the season is changed directly in database (while the server was not running)?

will this behave correctly when the season is changed directly in the database (while the server was not running)?

Good question. It depends on your definition of correctly.
Not running game server has no state. Therefore it loads the game state from the DB and does the necessary init when started.
I do belive that there is no edge-case but maybe you had something specific in mind?


fix: value for formatting the version number

According to issue #2556, the value "%.5s" in line 90,for formatting the version number given, doesn't verify all numbers needed and cuts the last value of given string, like number version of 23.0.0.  Executing the line
/usr/bin/docker version --format '{{.Server.Version | printf "%.5s" }}'  corresponding to the original line 90 in the script, we have: 23.0.
Which make the condition fails.
Using "%.8s" the condition works properly. Any number like 23.45.01 will be verified.

@brunos1 thx!

Hi @brunos1
Please sign the Oracle Contributor Agreement (OCA) so that we can merge your pull request.


NullRef exception in Reset()

NullRef exception is thrown in Reset() upon pressing the Clear button.

The repo collection seems to contain one null element.

⚠  This can obviously be easily fixed with a NULL check (I will create a pull request for that soon), but further investigation is required to determine what causes the Null element to be added. Maybe a JSON parsing error when reading the file from the cache?

I'll close this issue after merging the PR.


[Cauldron.Interception.Fody] Wrong version of Cauldron.Interception.Fody referenced

Hey,
I have a doubt regarding how Fody chooses which version of the weaver to use.
Let's say, there is a solution having multiple projects, each using different version of Cauldron (Cauldron.BasicInterceptors). Each version of Cauldron brings a corresponding compatible version of Cauldron.Interception.Fody (which is the actual addin of Fody). Now, will Fody choose right version of Cauldron.Interception.Fody, or it would always choose the latest version available in the packages folder?
If we try to build such a solution, then the build fails for the project consuming lower version of Cauldron. I guess this is the expected behavior.
The link AddinSearchPaths mentions that Fody would always take the latest version of the addin.
Could you please help by asserting this assumption.

Yea I have the problem too with different Fody versions. If Fody finds a newer Addin version in the package folder Fody will use it.
Following issue is related to this problem:
https://github.com/Fody/Fody/issues/524

Thanks for the quick reply. I am using PackageReference in my project. But still it doesn't resolve to the correct Addin version. I think, this is the limitation of fody, coz it doesn't know which version of the Addin we are using (maybe because it doesn't parse the csproj/packages.config file to lookup for the version information)

Just found this... https://github.com/Fody/Fody/pull/557
I will look into this... Maybe it fixes these problems.


[FLINK-16202][table] Support JSON_QUERY

What is the purpose of the change
This introduces support for JSON_QUERY in both Table API & SQL.
supersedes #11188
Verifying this change
This change added tests and can be verified as follows:

JsonFunctionsITCase

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? docs + JavaDocs

@flinkbot run azure


DB Merge before pushing and pulling

Is there a way we can merge the databases before db:pull and db:push tasks. thanks

What exactly do you mean? Could you elaborate a bit further? Like what scenario would this be useful in?


System.InvalidOperationException:“The control Lemon.ModuleNavigation.Sample.ModuleAs.ViewA (Content = StackPanel) already has a visual parent ContentPresenter (Name = PART_ContentPresenter, Host = Lemon.ModuleNavigation.Avaloniaui.Containers.NContainer) while trying to add it as a child of ContentPresenter (Name = PART_ContentPresenter, Host = Lemon.ModuleNavigation.Avaloniaui.Containers.NContainer).”

When I start the latest code, this error occurs:

May I ask how to fix it?

Pull latest code and try please

New commit:
https://github.com/NeverMorewd/Lemon.ModuleNavigation/commit/980ed2bd39041c371d87eab7662795d7c74f944c

Thank you, it's okay now.


Duplicate doc id in theoremqa_questions dataset

Find two document "Multinomial_1.json" in theoremqa_questions dataset.

Thanks a lot for raising this! We are looking into it! Stay tuned!

We have updated the documents. Feel free to re-open the issue if you have more questions!


Automate conformance testing on PR using zot

Looks like its not quite passing yet, so made it still pass the test if conformance fails
Ran 63 of 68 Specs in 0.111 seconds
FAIL! -- 60 Passed | 3 Failed | 0 Pending | 5 Skipped

cc @rchincha

@jdolitsky Taking a look at these failures.

Hello, I've been debugging this, here is the summary:
1)
      Get on stale blob upload should return 204 with a range and location [It]
      /home/runner/work/distribution-spec/distribution-spec/conformance/02_push_test.go:216

      Expected
          <string>: 0-21
      to equal
          <string>: bytes=0-21

This one is pretty straightforward, it happens because our GetBlobUpload doesn't return "bytes=" part in Range header.

      POST request to mount another repository's blob should return 201 or 202 [It]
      /home/runner/work/distribution-spec/distribution-spec/conformance/02_push_test.go:268

      Expected
          <string>: "...lobs/upload..."
      to equal               |
          <string>: "...lobs/sha256..."

This one assumes that the registry will do a cross-repo mount. In this case, zot will not mount the blob because it has dedupe disabled. (zot doesn't support cross-repo mounts but it uses its dedupe cache to work around it)
Looking at the test https://github.com/opencontainers/distribution-spec/blob/main/conformance/02_push_test.go#L276
it expects either a 201 or a 202 response (cross mount happened or not), but afterwards it expects the Location to point to the cross mounted blob, but in our case it will return a new sessionID instead with a 202 status code, which is acceptable from the spec point of view, but the test fails.
From my understanding the test should be fixed(more below), but I may be wrong.

      Cross-mounting of nonexistent blob should yield session id [It]
      /home/runner/work/distribution-spec/distribution-spec/conformance/02_push_test.go:295

      Expected
          <string>: /v2/myorg/myrepo/blobs/uploads/910c4c1b-2917-4a9c-8cde-5e05f34766f3
      to have prefix
          <string>: /v2/conformance-ca3bc013-07bb-4e9e-8a0b-9f3ff724d59e/blobs/uploads/

This one is using the http response from test 2)  https://github.com/opencontainers/distribution-spec/blob/main/conformance/02_push_test.go#L297
But in this case, because 2) failed, I think that lastResponse doesn't get updated, and it checks a previous response.
So this one fails because 2) fails.
As a side effect, both 2) and 3) passes if we use dedupe true(because then cross mount will work), just a a side note.
From my point of view this would be a fix, moving the last assertion to the next test:
diff --git a/conformance/02_push_test.go b/conformance/02_push_test.go
index fa3eca3..12d9cdf 100644
--- a/conformance/02_push_test.go
+++ b/conformance/02_push_test.go
@@ -277,7 +277,6 @@ var test02Push = func() {
                                        Equal(http.StatusCreated),
                                        Equal(http.StatusAccepted),
                                ))
-                               Expect(resp.GetRelativeLocation()).To(Equal(fmt.Sprintf("/v2/%s/blobs/%s", crossmountNamespace, testBlobADigest)))

                                lastResponse = resp
                        })
@@ -290,6 +289,8 @@ var test02Push = func() {
                                resp, err := client.Do(req)
                                Expect(err).To(BeNil())
                                Expect(resp.StatusCode()).To(Equal(http.StatusOK))
+
+                                Expect(resp.GetRelativeLocation()).To(Equal(fmt.Sprintf("/v2/%s/blobs/%s", crossmountNamespace, testBlobADigest)))
                        })

                        g.Specify("Cross-mounting of nonexistent blob should yield session id", func() {

Expect a blob session id (/v2//blobs/uploads/) when registry returns 202
Expect blob location(/v2//blobs/) when registry returns 201

@peusebiu pls test with https://github.com/project-zot/zot/blob/main/examples/config-conformance.json. Your observations still valid?

@peusebiu pls test with https://github.com/project-zot/zot/blob/main/examples/config-conformance.json. Your observations still valid?

yes, I used that without any modification

The registry MAY treat the from parameter as optional, and it MAY cross-mount the blob if it can be found.

Alternatively, if a registry does not support cross-repository mounting or is unable to mount the requested blob, it SHOULD return a 202. This indicates that the upload session has begun and that the client MAY proceed with the upload.

Hello,
In the cross mount blob test you expect one of two responses:
				Expect(resp.StatusCode()).To(SatisfyAny(
					Equal(http.StatusCreated),
					Equal(http.StatusAccepted),
				))

so far so good, this meets the dist-spec standard.
But then, on the next line, you expect a blob to be mounted:
				Expect(resp.GetRelativeLocation()).To(Equal(fmt.Sprintf("/v2/%s/blobs/%s", crossmountNamespace, testBlobADigest)))

This line ^ should be run only if the blob was mounted, if registry couldn't mount the blob, or doesn't support cross-mounts, then it should start an upload session and return its location (according to dist-spec).
Move the above line here: https://github.com/opencontainers/distribution-spec/blob/main/conformance/02_push_test.go#L288
to correctly run it only if the blob was mount.
As it is now, this test can not pass on any dist-spec implementation.
A location can not be both equal with:
fmt.Sprintf("/v2/%s/blobs/%s", crossmountNamespace, testBlobADigest) and
fmt.Sprintf("/v2/%s/blobs/uploads/", crossmountNamespace) as the test implies.

this is passing now! could use a review

Do we want to switch to the following which would allow both?

Would do only one, even if doesn't follow the rfc :(

If popular clients don't break (most likely they are not using chunked uploads at all), considering to make the change in zot to do the right thing per HTTP rfc. We have had non-cloud embedded clients requiring the HTTP rfc behavior.
Your "SatisfyAny()" clause should still work.


debug mode adds to the padding or margin of col

when in debug mode, along with the grey color, i also see a slight change in the padding/margin of the Col element. Is this a bug?

Looks like you are right:
https://github.com/JSxMachina/react-grid-system/blob/6c4956a1b77751066adf9137425480a43cc6225a/src/grid/Col/style.js#L32:L37
Would it be worth taking off the border & padding and using outline instead (it doesn't add any width);

Yeah, it currently adds some padding on the top and bottom. The border should not add any width though, as the box sizing is set to border-box.

Changed this in the last version, debug now only adds background and outline.


not enough file handles - build 1.0.11

running couple of days, htting this error  (lots of such an errors):
Ubuntu OS
pageview
null
Error: EMFILE: too many open files, open '/home/user/.config/UA Cyber SHIELD/config.json'
    at Object.openSync (node:fs:585:3)
    at Object.func [as openSync] (node:electron/js2c/asar_bundle:5:1812)
    at Object.readFileSync (node:fs:453:35)
    at Object.e.readFileSync (node:electron/js2c/asar_bundle:5:9160)
    at ElectronStore.get store [as store] (/tmp/.mount_UA-Cybb4Pw16/resources/app.asar/node_modules/conf/dist/source/index.js:276:29)
    at ElectronStore.set (/tmp/.mount_UA-Cybb4Pw16/resources/app.asar/node_modules/conf/dist/source/index.js:170:17)
    at IpcMainImpl.<anonymous> (/tmp/.mount_UA-Cybb4Pw16/resources/app.asar/electron-main.js:34:13253)
    at IpcMainImpl.emit (node:events:390:28)
    at EventEmitter.<anonymous> (node:electron/js2c/browser_init:161:10120)
    at EventEmitter.emit (node:events:390:28) {
  errno: -24,
  syscall: 'open',
  code: 'EMFILE',
  path: '/home/user/.config/UA Cyber SHIELD/config.json'
}
pageview
null
Loaded proxy list
Failed to load targets from https://bitbucket.org/uncork-vacant-shallow/exposable-ibuprofen/raw/6a078f9eee1eca67ca1d274983e256d244888820/40.json. Error: Error: Failed to load received data as JSON. Error: SyntaxError: Unexpected token
                     in JSON at position 0
Loaded targets list
superagent: double callback bug
Error: EMFILE: too many open files, open '/home/user/.config/UA Cyber SHIELD/config.json'
    at Object.openSync (node:fs:585:3)
    at Object.func [as openSync] (node:electron/js2c/asar_bundle:5:1812)
    at Object.readFileSync (node:fs:453:35)
    at Object.e.readFileSync (node:electron/js2c/asar_bundle:5:9160)
    at ElectronStore.get store [as store] (/tmp/.mount_UA-Cybb4Pw16/resources/app.asar/node_modules/conf/dist/source/index.js:276:29)
    at ElectronStore.set (/tmp/.mount_UA-Cybb4Pw16/resources/app.asar/node_modules/conf/dist/source/index.js:170:17)
    at IpcMainImpl.<anonymous> (/tmp/.mount_UA-Cybb4Pw16/resources/app.asar/electron-main.js:34:13253)
    at IpcMainImpl.emit (node:events:390:28)
    at EventEmitter.<anonymous> (node:electron/js2c/browser_init:161:10120)
    at EventEmitter.emit (node:events:390:28) {
  errno: -24,
  syscall: 'open',
  code: 'EMFILE',
  path: '/home/user/.config/UA Cyber SHIELD/config.json'
}
pageview
Error: getaddrinfo EBUSY www.google-analytics.com

............................

Try updating Node.js to the latest or earlier version
P.S. I don't know if it will help, but I advise you to try


Upgrade trunk

This uses the latest version with the MacOS fixes, so we should be able to use crashpad again.

Attaching this as part of #467


Update send-to-flomo extension

Description

Add newline character between content and tags.
Screencast

Checklist

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

Hey @thishym, could you include a description for your update? Also it looks like you haven't included your Raycast username in the contributors. Could you do it as well?

@thomaslombart Thank you for replying, this is my first PR so I apologize for any low-level mistake at first.
I added the description, just add a newline character to make the result looks prettier.
My Raycast username was "joeyming" while pushing and changed it to "thishym" later.
I wrote "thishym" in the contributors, which didn't pass the reviewer check.
Is it ok to change Raycast username? Anything else should I do?
Thanks again.

Sorry, I find there was a typo in my Raycast username and corrected it just now.

Could you add a screenshot and a recording as well? I can't really see what was wrong before, and what is your PR fixing?

OK, this photo shows the changes I made:

I think the error happens due to my raycast username changes. Right now, my raycast username is "thishym".
At first the username was "joeyiming". Even now I fork a raycast extension, the package.json will add this username to contributors. This is weird.
And then I changed it to "thiyhym"(with a typo), and this one didn't pass the reviewer check and bothered us.
Yesterday, I changed it to "thishym".
Sorry, I am not sure what I should make a recording of. The progress after running the extension publish command?

I meant screenshots and recordings of the actual extension you're fixing. What was the bug and what is your PR fixing? 😊

ok, I made a recording, hope this is helpful. ❤


Add failing test for GetParameterToConstructorInjectionRector

The expected behavior would be that the parent constructor is only called once.

That is indeed strange we ca probably handle this in the rector rule

@johanadivare I think that I am too unfamiliar with the code to fix this in the near future.

Fair i had a quick look but i dont see where the double constructor is added

@johanadivare I took a quick look when creating this issue and I think it relates to \Rector\Core\NodeManipulator\ClassDependencyManipulator::addConstructorDependencyWithCustomAssign() and \Rector\Core\NodeManipulator\Dependency\DependencyClassMethodDecorator::decorateConstructorWithParentDependencies(), though I might be mistaken.

@johanadivare I think that I have found the issue and fixed it in rectorphp/rector-src#3094.

@TomasVotruba Should this PR be reopened because it was auto-closed by merging rectorphp/rector-src@9a93704c5aa267807287645f9d07772a554ac960?

It seems this was a bug that was fixed in https://github.com/rectorphp/rector-src/commit/9a93704c5aa267807287645f9d07772a554ac960
If it's not the case, it should be re-opened.

@TomasVotruba Yes, it was fixed by that commit but the test in that commit covers a different rule than this test does so it would cover the change for two different scenarios, which might not be necessary.
I was asking about the reopening because I lack the permission to do so.

I see :)
In that case, next time the "closes x" should link only issue, not the PR. This test fixture should've been part of the fixing PR so it's complete work in one PR.

This test fixture should've been part of the fixing PR so it's complete work in one PR.

The fix was in a different repository, thus different PRs.

I see, got it :


Add Replicate demo and API

Hi @wyhsirius,
Great work on LaVie!
This pull request makes it possible to run LaVie on Replicate (https://replicate.com/cjwbw/LaVie) and via API (https://replicate.com/cjwbw/LaVie/api).
We'd also like to transfer the ownership of the page under Vchitect so you can modify the demo easily!

Thank you for the effort!

Woohoo. Well done!


[React] Pure Component vs Functional Component

i'd lke to contribute, to this issue ? can you help me how to proceed with it?


Animation tree not working at all

Godot version
4.0 beta 6
System information
Windows 11, nothing special
Issue description
The animations in the animation tree are not working
Steps to reproduce
add multiple 2d animations and connect them in an animation tree
Minimal reproduction project
Not the time for that

Duplicate of #69066


[WIP] python.pkgs.mycroft-precise: init at 2020-02-14

Motivation for this change
I wanted to experiment with hotword-triggered voice memos. Unfortunately this turned out to be more complicated than I had hoped.
For now python3.pkgs.mycroft-precise builds and runs. It doesn't work as expected though. Maybe https://github.com/MycroftAI/mycroft-precise/pull/141 will help. I don't know when or if I'll continue to work on this, so I thought I'd put it out there in its current state.
If you'd like to take this over, please do. I maintain enough packages already and I'm not sure if I'll finish this and commit to maintaining it.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

If you want to play with this, clone the community data repository at https://github.com/MycroftAI/precise-community-data and use one of the models. For example if you've cloned the repo to ~/repos/Precise-Community-Data and unpacked the computer model under computer/model, you can use precise-engine to detect the hotword in their sample data:
result/bin/precise-engine ~/repos/Precise-Community-Data/computer/models/computer-en-0.2.0-20190814-eltocino/computer-en.pb < ~/repos/Precise-Community-Data/computer/en/computer-en-117c2f25-4ab5-4620-84c4-84cf1bb7af49.wav

you should see an activation above 0.5 for their samples (currently that is not the case). Alternatively you can try it with your microphone's input:
result/bin/precise-listen -b ~/repos/Precise-Community-Data/computer/models/computer-en-0.2.0-20190814-eltocino/computer-en.pb

a ! should pop up when you say the word "computer" (it currently doesn't though).

/marvin opt-in
/status needs_work

Abusing this PR a bit for a test.
/marvin triage

/status awaiting_changes

Hi!
The marvin-mk2 bot is now discontinued. I have removed the relevant tags from this PR. If you still need someone to look at it, one option would be to ask in this discourse thread.
I am posting this notice to all open PRs with the marvin tag. Please understand that following all of these discussions would take too much time, so I will unsubscribe from this PR unless I have already been involved in it before this message.

I do not intend to pick this up any time soon.

If you're still interested in mycroft itself, may I interest you in #161596?

Thanks for the pointer!


Leave certain positions untouched

There should be way to configure Optimus to leave certain positions in a dimension order of a cube untouched. E.g. dimension 1 must always be time.

@MariusWirtz I am working on this right now for one of our own projects and planning to raise a pull request once this is developed and tested.
I have a question on the input parameter and reordering logic.

Move/keep the "excluded" dimension to the Display Order Position - this is the current behaviour as the ordering always starts with the display order.
Keep the "excluded" dimension to the current Storage order - this is what I would understand is correct.
Provide a map of dimensions to optimus py: {"dim1":1, "dim2":2} etc so there is no ambiguity.
Something else.

I've currently developed the 2nd option, providing a list of comma split dimensions using -d (for dimensions) parameter. probably not the best letter for it 😄 will try to get a pull request this week if that's an acceptable approach.

I think the 2nd option sounds reasonable! So does -d
Feel free to open the PR


Sometimes generating 33 ARI requests in a single second

What version of the package are you using?
User Agent: "CertMagic acmez (linux; 386)"
What are you trying to do?
Look at outliers in Let's Encrypt's ARI request data.
What steps did you take?
Queried LE's observability database for how many times ARI is queried for each certificate.
What did you expect to happen, and what actually happened instead?
I expect CertMagic to query ARI on a regular basis (e.g. every 6 hours, if respecting the Retry-After header), perhaps with some jitter to prevent clustering.
Instead, I see that CertMagic clients are requesting ARI data for a single serial 60+ times in a single day, including bursts of up to 33 requests in a single second.
How do you think this should be fixed?
I suspect this may be related to querying ARI in the "on-demand TLS" code path, and the misbehaving servers may be getting crawled and generating many requests due to that.
The ARI suggestedWindow should be cached for the duration provided by the Retry-After header in the ARI response.
Please link to any related issues, pull requests, and/or discussion
https://github.com/caddyserver/certmagic/pull/286

Oh yeah that makes sense. If you get like 33 requests within a second, then all 33 of them might be triggering ARI via on-demand maintenance. I think certmagic needs to use https://pkg.go.dev/sync#WaitGroup to make sure it only gets fired off a single time per window.

I think this is a thundering herd issue. We probably need https://pkg.go.dev/golang.org/x/sync/singleflight with the domain as key.

every 6 hours, if respecting the Retry-After header

Is this zoned per queried hostname or per client?

Is this zoned per queried hostname or per client?

Neither, it's per certificate. A single certificate may have multiple hostnames, but also a single client may manage multiple certificates.

I'll take a look into this when I'm back at my desk

The ARI suggestedWindow should be cached for the duration provided by the Retry-After header in the ARI response.

CertMagic does honor the Retry-After header, if present, by calling acme.RenewalInfo.NeedsRefresh().
I do agree this is likely a thundering herd, where many calls to update ARI come in before the first one finishes, since it lacks synchronization.
We can synchronize ARI fetching using the configured storage plugin. This will prevent any more than 1 instance in a cluster from fetching ARI at the same time, and after the first one does, the others will load and use its result.
Depending on the storage plugin, it's possible that this locking will be more expensive than actually fetching ARI, but it only happens once in a while, so maybe it's OK.

Thanks for the report! This should fix it but without an offending client to test with I can only guess, but it makes sense to me.
I've synchronized the ARI fetching by the ARI UniqueIdentifier.

Thank you for the investigation and fix! We plan to keep an intermittent eye on ARI traffic patterns for a while, so I'll let you know if I see anything else jump out at me.

As far as I can tell, this isn't fixed by 16e2e0b3443037882be32c731d1e85a90cb69014.  I'm able to repro the extra } error reliably--multiple times per hour--regardless of the lengths of the original and new files, so it's not just a simple truncation issue.  It's always one extra }, even though file sizes often differ by more than one character.
Removing the extraneous closing braces restores sanity, but only briefly.  They keep reappearing in new files.

@Zenexer how did you build Caddy?

I don't have my computer handy, but it was a simple Dockerfile with the usual xcaddy --with syntax, certmagic@master.

% docker compose exec caddy caddy build-info | grep -F certmagic
dep     github.com/caddyserver/certmagic        v0.21.4-0.20240905165329-aad674cda5ba   h1:p4QJmH7xVg2OhcbHBP8dGTdl6raoVQMA8IHjDka6R0Y=

FROM caddy:builder AS builder

RUN xcaddy build \
	--with github.com/caddyserver/certmagic@master

FROM caddy:latest

COPY --from=builder /usr/bin/caddy /usr/bin/caddy

EXPOSE 80
EXPOSE 443
EXPOSE 443/udp

@Zenexer Did you mean to comment on a different issue (#303)? This one is about superfluous ARI requests, not malformed files.

No, I was going under the assumption that the author was correct in linking them--which could be incorrect.

I see. You might actually want this commit: https://github.com/caddyserver/certmagic/pull/300

If that's on master, I would've included it just to be safe.