Show output from pub get in flutter pub get

Part of https://github.com/dart-lang/pub/issues/2971 .
dart pub get has detailed output about outdated dependencies. This is currently not exposed from flutter pub get.
We are missing out on at least this information:

newer version available
package obsolete
version retracted
known vulnerability (planned)

Before:
> flutter pub get
Running "flutter pub get" in sensors_plus...                     1,691ms
Warning: You are using these overridden dependencies:
! sensors_plus_platform_interface 1.1.1 from path ../../sensors_plus_platform_interface
! sensors_plus_web 1.1.1 from path ../../sensors_plus_web
Running "flutter pub get" in example...                          1,019ms

Note how the warnings output really belongs to the resolution from the example.
After:
> flutter pub get
Running "flutter pub get" in sensors_plus...                     1,803ms
Resolving dependencies...
  matcher 0.12.11 (0.12.12 available)
  test 1.21.1 (1.21.2 available)
  test_api 0.4.9 (0.4.10 available)
  test_core 0.4.13 (0.4.14 available)
  vm_service 8.3.0 (9.0.0 available)
Got dependencies!
Resolving dependencies in ./example...
Got dependencies in ./example.

This PR also uses pub get --example (implemented here) to resolve in './example' if it exists.
Now the output from resolution in ./example is reduced as long as there are no errors.
Because flutter pub get will retry pub get until successful the output of each run is recorded, and only the output of the last run is shown.
Fix: a small bug where flutter create for some templates would invoke pub even if given a --no-pub argument.
Cleanup: pub get will no longer be invoked with --no-precompile (that is now default).
Output from pub get is also show when doing flutter create - that might be distracting - maybe we should turn that off:
> flutter create test_plugin -tplugin
Creating project test_plugin...
Running "flutter pub get" in test_plugin...                      1,931ms
Resolving dependencies in test_plugin...
+ async 2.9.0
+ boolean_selector 2.1.0
+ characters 1.2.1
+ clock 1.1.1
+ collection 1.16.0
+ fake_async 1.3.1
+ flutter 0.0.0 from sdk flutter
+ flutter_lints 2.0.1
+ flutter_test 0.0.0 from sdk flutter
+ lints 2.0.0
+ matcher 0.12.11 (0.12.12 available)
+ material_color_utilities 0.1.5
+ meta 1.8.0
+ path 1.8.2
+ plugin_platform_interface 2.1.2
+ sky_engine 0.0.99 from sdk flutter
+ source_span 1.9.0
+ stack_trace 1.10.0
+ stream_channel 2.1.0
+ string_scanner 1.1.1
+ term_glyph 1.2.1
+ test_api 0.4.9 (0.4.10 available)
+ vector_math 2.1.2
Changed 23 dependencies in test_plugin!
Resolving dependencies in test_plugin/example...
Got dependencies in test_plugin/example.
Wrote 25 files.

All done!

Your plugin code is in test_plugin/lib/test_plugin.dart.

Your example app code is in test_plugin/example/lib/main.dart.

You've created a plugin project that doesn't yet support any platforms.

To add platforms, run `flutter create -t plugin --platforms <platforms> .` under test_plugin.
For more information, see https://flutter.dev/go/plugin-platforms.

Also includes color in the pub terminal output (depending on the flutter terminal color detection):

Pre-launch Checklist

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] I read the Tree Hygiene wiki page, which explains my responsibilities.
[X] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[X] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[X] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@sigurdm is this ready for review? (it's marked a draft, but maybe that was by accident?)

@sigurdm is this ready for review? (it's marked a draft, but maybe that was by accident?)

Yeah - I marked it as draft until I added the color-handling - but I think it should be good now.

I'm going on 3 weeks vacation today. So don't expect responses before that.

cc @Jasguerrero

I've rebased this, could you take another look?

Fixed the test, had to hack the analyze_snippets_code.dart tool a bit.
There was also a bug if the example folder existed but had no pubspec.yaml - fixed that.

Fixed the test, had to hack the analyze_snippets_code.dart tool a bit.
There was also a bug if the example folder existed but had no pubspec.yaml - fixed that.

thanks! when you're ready to merge, you can add the autosubmit label and the bot will merge when the tree is green.


Add Google social login functionality

What would you like to be added:
Currently, the application supports GitHub social login only. To provide users with more options and flexibility, it is recommended to add Google social login functionality. This will allow users to sign in using their Google accounts in addition to GitHub accounts.
Why is this needed:
Expanding social login options can attract a broader user base and improve the overall user experience. By adding Google social login functionality, users who prefer Google as their authentication method will be able to access the application seamlessly.
Additional Information:

I'm interested in this issue. Would it be okay if I work on it?

@sinterhoo
Sure! If possible, it is good to consider social provider on/off options.

@sinterhoo
Could you share your progress? If there are any issues, I can help.

Apologies for the delay; I've been quite busy recently and haven't made much progress. If this issue needs to be resolved quickly, I'm okay with someone else taking it on. However, if there's no rush, I'd like to work on it myself. It might take a bit longer, but I haven't encountered any issues so far. I'll reach out if I need any help.

I'm just checking on the progress. If you have any questions, feel free to ask.


Notifications: Move GITHUB_SLACK_MAP and GITHUB_JIRA_MAP into YAML files.

This commit moves the data for the GITHUB_SLACK_MAP and GITHUB_JIRA_MAP out of python files and into YAML files. This is being done so that the data is accessible to non-python code.
What does this PR do?
Moves configuration containing a mapping of DataDog GitHub teams to JIRA projects out of python code into YAML files. For consistency, the same is done with the mapping of DataDog GitHub teams to Slack channels.
Motivation
In the near future, go code (outside of the datadog/agent repo) will be using this data for automatic ticket assignment. YAML is a better universal format than Python code.
Additional Notes
N/A
Possible Drawbacks / Trade-offs
It's possible for a future commit to put invalid data (i.e. badly structured) in the YAML file. This PR adds a test that will catch that.
Describe how to test/QA your changes
Execute the unit test suite... specifically the test added in tasks/unit-tests/pipeline_lib_tests.py.

/merge


[🐛 Bug]: Error: failed to retrieve the Cloudflare request context

next-on-pages environment related information
System:
        Platform: linux
        Arch: x64
        Version: #17-Ubuntu SMP PREEMPT_DYNAMIC Thu Jan 11 14:01:59 UTC 2024
        CPU: (16) x64 13th Gen Intel(R) Core(TM) i7-13620H
        Memory: 15 GB
        Shell: /usr/bin/fish
Package Manager Used: pnpm (8.15.3)

Relevant Packages:
        @cloudflare/next-on-pages: 1.9.0
        vercel: N/A
        next: 14.1.1-canary.70

Description
The getRequestContext function throw an error when it called on the server actions.
Reproduction
command
next dev

app/sign-in/actions.ts
'use server'

import { getRequestContext } from '@cloudflare/next-on-pages'

export async function signIn() {
  const { env } = getRequestContext() // throw an error
  const { dbCache } = env
}

next.config.mjs
import { setupDevPlatform } from '@cloudflare/next-on-pages/next-dev'

/** @type {import('next').NextConfig} */
const nextConfig = {}
export default nextConfig

const isDev = process.env.NODE_ENV === 'development'
if (isDev) {
  await setupDevPlatform()
}

wrangler.toml
[[kv_namespaces]]
id = "<redacted>"
binding = "dbCache"

Pages Deployment Method
None
Pages Deployment ID
No response
Additional Information
No response
Would you like to help?

[ ] Would you like to help fixing this bug?

Seems like you forgot to export runtime as 'edge':
https://nextjs.org/docs/app/building-your-application/rendering/edge-and-nodejs-runtimes#segment-runtime-option

Seems like you forgot to export runtime as 'edge':
https://nextjs.org/docs/app/building-your-application/rendering/edge-and-nodejs-runtimes#segment-runtime-option

Next.js server actions doesn't have export const runtime = "edge" options. Should I add it on the page that use the related actions?

Ok it works. We have to add export const runtime = "edge" to the page.tsx that consume the actions.
Thanks @OpenSource03


Update czech translation

Check if this PR fulfills these requirements:

[x] My changes respect the Kodi add-on development rules
[x] I have read the CONTRIBUTING document.
[x] I made sure there wasn't another Pull Request opened for the same update/change
[ ] I have successfully tested my changes locally

Types of changes

[ ] New feature (non-breaking change which adds functionality)
[ ] Feature change (non-breaking change which change behaviour of an existing functionality)
[x] Improvement (non-breaking change which improve functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

In case of Feature change / Breaking change:
Describe the current behavior

Describe the new behavior

Screenshots (if appropriate):

The changes are fine. Specify a new CZ translation maintainer. Thank you

Fine thanks


#7586: Create unstable branch of WH single card nightly FD

This is a regression, I'm ok to skip test for now, but would rather not move it to unstable pipeline.

Tracking here https://github.com/tenstorrent/tt-metal/issues/9158

@AleksKnezevic just to confirm, you'd like to skip this altogether rather than have it pass ND in the unstable branch?>

I would have been ok to skip until I figured out the regression, but someone beat me to it, it's fixed in latest main:
https://github.com/tenstorrent/tt-metal/actions/runs/9386939304/job/25848917507 so I don't want to skip this test.

This is not for ttnn unit tests - this is integration tests for SD
What was the fix? Can't find it...

Can't tell from the link you posted

Here are some hangs on nightly fast dispatch on main from overnight:
https://github.com/tenstorrent/tt-metal/actions/runs/9382086403/job/25832975051 (the skipped one in this PR)
https://github.com/tenstorrent/tt-metal/actions/runs/9383668111/job/25838017615 (test_sharded_matmuls)

The latest one seems to have passed, no: https://github.com/tenstorrent/tt-metal/actions/runs/9387431678 ?

Yeah, so it's ND again.
"Unstable" implies non-deterministic hangs. I can choose a better name, if that's the issue

ok, fair, you can move it to unstable pipeline then, but I would rather that test not get skipped.

Sounds good - not skipping, reverting


Extension now supports all VS19 versions, updated dependencies

Broadened VS19 support, now containing Pro and Enterprise versions beside Community. Updated dependencies. Builds only with VS19 and installable only for VS19. Visit master branch for 2.x version, which is compatible for VS22.

Thanks!


Bulk Box users search not working from email lookup

I have a list of email address that I wish to lookup the account id.
Error:
2 entries failed!
Entry 1 (
) failed with error:
Missing 1 required arg:
name  Name of user to search for
See more help with --help

Entry 2 (
) failed with error:
Missing 1 required arg:
name  Name of user to search for
See more help with --help
I had to insert name label at the top of list versus using no label or login label

Hi @nexus667,
I think your csv file has two lines, one for each email address you are looking up.  You need to add a header line at the top of the CSV with the value name.  Give that a try and let us know how it works for you.
Thanks,
@PJSimon


Very tall Y axis for high BMI on trisomy 21 charts

Porting across https://github.com/rcpch/digital-growth-charts-server/issues/220 to this repo as it seems to be an issue with the chart rendering rather than the data returned from the API.

Birth Date: 08/05/2006
Measurement Date: 01/11/2022
BMI: 36.3 kg/m²

Storybook story to reproduce in the mbarton/trisomy21-bmi-rendering-issue branch: https://github.com/rcpch/digital-growth-charts-react-component-library/commit/6aa9181e7cc89c668191100086b0fc94aafd068c

Ah I see those are the actual values from the reference set:
https://github.com/rcpch/digital-growth-charts-react-component-library/blob/b6331d27ac116991802313028fcd8257e68affd5/src/chartdata/trisomy21_bmi_male_sds_data.ts#L7712
I think I will need @eatyourpeas input here on how that reference was calculated. There's also https://github.com/rcpch/digital-growth-charts-server/issues/217 raised already for the nulls

These values were calculated from our chart-coordinates endpoint and are possibly correct because of the way the maths works at higher values. The BMI curves in trisomy I find much less useful clinically for this reason. @statist7 would better be able to comment whether this is an actual error or whether this is more an issue for rendering where we simply apply an arbitrary upper limit to the visible value of y.

For particular configurations of L, M, S and z, BMI heads off to infinity. The relevant formula is Centile = M(1+LSz)^(1/L) so if 1+LSz = 0 then there is a problem.
This happens when L is negative (which it nearly always is with BMI) and z is sufficiently large and positive. As @eatyourpeas says, perhaps test for and trap large BMI values.


errors on ArchLinux and python3.9

Hello
I am trying to use scalene on some code.
I cannot post the whole code, but here is  what it does, at least:

creating threads, processes, using the 'forkserver' start method, on linux, and multiprocessing.get_context()
overriding threading.Thread.run
creating multiprocessing,queues

I tried to create a simple reproducer code, but I couldn't find what causes the error.
When my code creates new processes, I get these errors:
Exception ignored in: <function _after_fork at 0x559984ec9880>
Traceback (most recent call last):
  File "/usr/lib/python3.9/threading.py", line 1496, in _after_fork
    thread._reset_internal_locks(False)
  File "/usr/lib/python3.9/threading.py", line 827, in _reset_internal_locks
    self._started._at_fork_reinit()
  File "/usr/lib/python3.9/threading.py", line 527, in _at_fork_reinit
    self._cond._at_fork_reinit()
  File "/usr/lib/python3.9/threading.py", line 253, in _at_fork_reinit
    self._lock._at_fork_reinit()
AttributeError: 'ReplacementLock' object has no attribute '_at_fork_reinit'

Is there a way to get more details about this exception?
Thank you

sorry, wrong project. I meant to create this issue for scalene.


Client

Is there a Client - GUI for this awesome db?

Hi @shai-benshimol, there is only a shell tool included. But you can check this project:
https://github.com/falahati/LiteDBViewer


stdlib: add AppNotify.h to the Shell module

This header is a standalone header part of the shell subsystem which
allows application to get notification related to application
suspension.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test

CC: @egorzhdan

Build failed
Swift Test Linux Platform
Git Sha - 320832fe89f9c6c303e4f81f88b0c829bda14dba

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 62d788c08d012a6108b932f303f6a346da619ad3

@swift-ci please test Linux platform

Build failed
Swift Test Linux Platform
Git Sha - 62d788c08d012a6108b932f303f6a346da619ad3

@swift-ci please test Linux platform


aijack cannot pip install

Building wheels for collected packages: aijack
Building wheel for aijack (pyproject.toml) ... error
error: subprocess-exited-with-error
× Building wheel for aijack (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [358 lines of output]
ERROR: Failed building wheel for aijack
Failed to build aijack
ERROR: Could not build wheels for aijack, which is required to install pyproject.toml-based projects

@N0Carrot san
Thank you for your report. Could you share the detailed information about your environment?
If you use ubuntu, try the following commands:
apt install -y libboost-all-dev
pip install -U pip
pip install "pybind11[global]"

pip install aijack

also getting this errror on a mac

I am also getting the same error running pip in wsl

I am also getting the same error running pip in wsl
subprocess.CalledProcessError: Command '['cmake', '/tmp/pip-install-l6cdkfio/aijack_a353f8bfb4da4288b81560abed6b39b3', '-DCMAKE_LIBRARY_OUTPUT_DIRECTORY=/tmp/pip-install-l6cdkfio/aijack_a353f8bfb4da4288b81560abed6b39b3/build/lib.linux-x86_64-cpython-311/', '-DPYTHON_EXECUTABLE=/home/leona/anaconda3/bin/python', '-DCMAKE_BUILD_TYPE=Release', '-DEXAMPLE_VERSION_INFO=0.0.1b2', '-GNinja', '-DCMAKE_MAKE_PROGRAM:FILEPATH=/tmp/pip-build-env-58gmj2kc/overlay/lib/python3.11/site-packages/ninja/data/bin/ninja']' returned non-zero exit status 1.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for aijack
Failed to build aijack
ERROR: Could not build wheels for aijack, which is required to install pyproject.toml-based projects

I reinstalled g++ and it fixed this issue.


[Question]: Functionality Limitation or Bug: Embed into Site Disabled in Agent Begin Component and Begin Component Restoration

Describe your problem
RAGflow -  v0.14.1-206-g85083ad4 slim
I encountered two issues while using the Agent:
When I start entering variable Key and Name in the input field of the Begin Component in the Agent, the embed into site feature becomes disabled.
After deleting the Begin Component on the Agent page, I cannot restore it.
I would like to ask whether this is a bug, a current functionality limitation, or an issue with my operation.
Thank you!

For now, it's been designed this way.
It involves parameter input, which we will support in the near future.


The project should be buildable using the dotnet build command

dotnet build command produces the following error:
Microsoft (R) Build Engine version 16.8.3+39993bd9d for .NET
Copyright (C) Microsoft Corporation. All rights reserved.
Determining projects to restore...
Restored C:\Users\rapoddar\github\csdot\src\test\test.csproj (in 207 ms).
Restored C:\Users\rapoddar\github\csdot\src\csdot\csdot.csproj (in 257 ms).
Restored C:\Users\rapoddar\github\csdot\src\UnitTest\UnitTest.csproj (in 13.76 sec).
File "attribute_generator/code_generator.py", line 261
SyntaxError: Non-ASCII character '\xc2' in file attribute_generator/code_generator.py on line 262, but no encoding declared; see http://python.org/dev/peps/pep-0263/ for details
C:\Users\rapoddar\github\csdot\src\csdot\csdot.csproj(31,3): error MSB3073: The command "python attribute_generator/code_generator.py" exited with code 1.
File "attribute_generator/code_generator.py", line 261
SyntaxError: Non-ASCII character '\xc2' in file attribute_generator/code_generator.py on line 262, but no encoding declared; see http://python.org/dev/peps/pep-0263/ for details
C:\Users\rapoddar\github\csdot\src\csdot\csdot.csproj(31,3): error MSB3073: The command "python attribute_generator/code_generator.py" exited with code 1.
Build FAILED.
C:\Users\rapoddar\github\csdot\src\csdot\csdot.csproj(31,3): error MSB3073: The command "python attribute_generator/code_generator.py" exited with code 1.
C:\Users\rapoddar\github\csdot\src\csdot\csdot.csproj(31,3): error MSB3073: The command "python attribute_generator/code_generator.py" exited with code 1.
0 Warning(s)
2 Error(s)
Time Elapsed 00:00:24.90
--- To Fix the issue need to add the below lines at the top of python file attribute_generator/code_generator.py--
#!/usr/bin/python
# -- coding: utf-8 --

I am running this on windows

Actually we build the project through Github actions you can refer to the file https://github.com/harshsikhwal/csdot/blob/main/.github/workflows/build_and release.yml
we use the same command in the windows environment, you can look at the logs
https://github.com/harshsikhwal/csdot/runs/2431859644?check_suite_focus=true
The default python path is generally taken up
The issue arises when we use python 2 for building
Thanks for pointing out the issue 👍
We have fixed it.


Can I change the sort order within a field only ?

Checklist

[X] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
Is there a way to change ONLY the sort order within a field (rather than the sort order of the fields) ?
I am asking, because I would like to (for example) set AV1 as the best video codec, but without touching the rest of the sort order.
Yet, from my understanding, if I set -S vcodec:av1, I don't only reorder the sort order of the video codes themselves, but reorder the sort order of ALL fields by putting the video codec at the #1 position (which I want to avoid).
Another example would be a user who wants to tell yt-dlp to prefer webm over mp4 but without giving the container format a higher priority in the sort order.

There is no way to do this atm.

Another example would be a user who wants to tell yt-dlp to prefer webm over mp4 but without giving the container format a higher priority in the sort order.

For this specific case, you can use --prefer-free-formats

Thanks for your reply and considering it as an enhancement.
If you ever implement this feature, it would be nice to be able to specify more than one field value, e.g:
something like --sort-field vcodec:av01>vp9.2>vp9>h265>h264


Add JSX to templating languages

Also reformat section and add examples

So what is the point to only adding you implementation?

I'm sorry?

https://github.com/google/incremental-dom/issues/65

I don't see the point in having multiple babel plugins for the same thing, we've already decided to make this the official one.

It's cool that you do not see, but my was done first. So then having you version? I believe you just decided to have your version, no matter other facts. This is nice to hear.

I don't mean to offend, but no that's not the reason. For one, I barely wrote any of the plugin, it was @jridgewell that did the heavy lifting. Also, the first solution is not inherently the more correct one.
JSX-IR is unlikely to be accepted broadly as it abstracts in the wrong direction. JSX has an intermediary representation, the AST. Also JSX elements are equivalent to function calls not objects.
I appreciate that you are so passionate about this, why not work together on the tools rather than in opposition.

Also JSX elements are equivalent to function calls not objects.

I have no dog in this fight, but I think the var node =
...
 use case in https://github.com/babel-plugins/babel-plugin-incremental-dom/pull/5#issuecomment-124947936 suggests that "JSX" does need to be able to return an object, whether that's real DOM (as elementClosed should return) or a virtual DOM object (as React.createElement returns).

Not exactly the point I was trying to make by that statement. Yes to implement certain things with JSX you need objects to exist, that's not the issue here.

Anyways, this discussion should not happen in this issue

I don't see any harm (personally) in listing both JSX solutions and tagging one as official. I remember last year that happened a lot with esnext plugins etc.

Also JSX elements are equivalent to function calls not objects.

That is wrong. Even React/Facebook folks are saying that JSX is just an abstraction over object literals. Function calls are just an React's artifact/debug solution. Also @ivan made a good point about it above.
I might be wrong, but will your plugin provide valid output here?:
let description = null;

  if (props.clip.description) {
    description = [

,

        { props.clip.description }

    ];
  }

return

  // ...
  { description }
  // ...

If not, then that plugin is not correct neither from JavaScript perspective (output) nor from JSX (source).

Also, the first solution is not inherently the more correct one.

That is true. Why your is more correct here? I thing what JSX-IR cannot provide is statics, but it will be solved with tag metadata.

JSX has an intermediary representation, the AST.

This is not problem, we can call JSX-IR final representation or JavaScript represenation :smile:
Anyway, AST and IR are different things. Here is one example: C++ (Source) -> AST (Front-end) -> IR (LLVM) -> [AST (Target AST, not required)] -> JavaScript (Output/Backend/Whatever)

Yes to implement certain things with JSX you need objects to exist, that's not the issue here.

Why? I thought we were talking about correct one implementation. In my mind, it should be correct from both ends, JSX and IDOM.

Anyways, this discussion should not happen in this issue

Why not, seems like a perfect place. Do you know better?

@matthewrobb sounds good to me. But from this point of discussion, I do not see how that official plugin is more correct than JSX-IR renderer.

@NekR I imagine that "official" means that it will be supported and maintained by people already supporting and maintaining components with the JSX/Babel ecosystem.

@matthewrobb oh, I thought official means "official from the IDOM side". Quick question: does Babel/Facebook owns JSX?

@NekR Facebook is the maintainer of JSX but not Babel. Again, I believe the "official" tag here applies to the Babel transform plugin for JSX -> IDOM.

@matthewrobb I cannot find initial comment about "official" anymore, but if "official" means that is official from the babel-plugin-incremental-dom team, then jsx-to-idom could be official from me, right?
I see here only 2 ways how it could be official:

By the Incremental DOM
By the Babel team

I do not see a reason to include in the list only official (and if it's official by the Babel there should not be any reason to hide my implementation on the wiki page). Anyway, from the "official" plugin I expect it to be correct. This one is not in terms of JSX.

This is definitely not the appropriate place for this discussion. Anything said here is going to turn into bike-shedding (if it hasn't already) and it's notifying everyone who is watching this repo.
Take it anywhere else and tag the relevant people. I'm not continuing this here.

@thejameskyle Could you sign the contributor license agreement? Let me know when you have and I can merge this.

While listing multiple plugins gives more choice to developers, it also adds increased confusion. It seems that this implementation is slightly more complete, though I haven't had a chance to really dig into both.
I think going forward it would be a good idea for everyone to get behind a single implementation. There definitely are semantics that are different between how Incremental DOM uses JSX and how React or other virtual DOM implementations use it. It would be good to have a consistent approach and to avoid fragmentation, since it already differs from React, though it superficially looks similar.

This is most right place since it has direct relation to idom. But taking
in account all new comments, I see that all had been just decided already
and nothing can change your mind. So okay, do it how you want.
On Jul 27, 2015 7:07 PM, "James Kyle"<EMAIL_ADDRESS>wrote:

This is definitely not the appropriate place for this discussion. Anything
said here is going to turn into bike-shedding (if it hasn't already) and
it's notifying everyone who is watching this repo.
Take it anywhere else and tag the relevant people. I'm not continuing this
here.
—
Reply to this email directly or view it on GitHub
https://github.com/google/incremental-dom/pull/76#issuecomment-125257370
.

This is most right place since it has direct relation to idom.

I think the point was the discussion of those sorts of issues should be in issues/PRs in the plugin's repo and not here. Incremental DOM and the JSX transformation really do seem to be separate things. As such, the discussion about semantics shouldn't be held in the Incremental DOM repo.

CLA signed

Thanks!


Containerize SWE-bench evaluation

This revamps the swebench.harness module to use fully containerized instances for evaluation.
See this report for further details.
Reference Issues/PRs
This change should address the following issues:
closes #114, closes #113, closes #141, closes #77, closes #104, closes #136
Any other comments?
Thanks to OpenAI's preparedness team (including Oliver Jaffe, Chan Jun Shern, James Aung, Giulio Starace, Dane Sherburn, and Neil Chowdhury) for initiating this.
Thanks to @aorwall for their suggestion regarding this.
And thanks to Cognition Labs for providing some inspiration for these changes too.

LGTM! 💯 Thanks again to all the contributors 🙏🏼


KH2 Gummi Skips not working

Honestly just waiting for this to be fixed...this is my main reason for playing kh2 is to skip the gummi missions if there is any other workaround lemme know thanks :)

There is an updated version of this mod available on the speedrunning discord, sadly no one from the 2 speed run or randomizer communities has really taken ownership of updating this repo.

Any chance you can link me to the discord or the updated mod? i'm struggling to find it apologies.

link to the discord is at the bottom of the main readme

Updated


refactor(css): restructure css files

WIP*
This PR includes:

Restructure LESS files
Rename classnames

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2081-1 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2081-fabric8-planner.badger.fabric8.io

[test]

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2081-10 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2081-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2081-11 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2081-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2081-12 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2081-fabric8-planner.badger.fabric8.io

just an FYI - fabric8-ui has the prefix as .f8-, being that these are all under the fabric8 name. This was done in order to match the organization naming structure. The Code Guidelines has been updated to reflect this.

@mindreeper2420 Thanks for the update Adam. I will make changes accordingly. This PR is still in progress.

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2081-16 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2081-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2081-17 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2081-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2081-19 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2081-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2081-20 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2081-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2081-22 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2081-fabric8-planner.badger.fabric8.io


feat: create frameless buttons

Video 📹
https://user-images.githubusercontent.com/16711614/150616584-2d96b7a9-7c63-4e84-9bfa-4486001e8226.mov
https://user-images.githubusercontent.com/16711614/150616590-8db6c36a-3d02-463d-8c18-3e997a204a1a.mov

Current dependencies on/for this PR:

next

PR #184

PR #185

PR #193

PR #192

PR #197

PR #194

PR #191

PR #201

I like the implementation here but I'm wondering about the purpose - we already have a Button component - can we work the functionality here into that component potentially?

Yes, we do already have a Button module. The issue I was running into with it was frustrating enough though that I wanted to start from scratch:

controlling the colors/size was more difficult and often different from the design guide examples
controlling for behavior when there was no text/icon was more difficult and often the end result was not centered correctly
Drop shadows, frames, etc could not be turned on/off easily depending on props, which meant I needed to use wrappers often.

The one functionality I'd like to bring over from the existing module is the flat prop. Otherwise, I think we'll be better served by this module moving forward.
The only outstanding concern I have with this module (which I still have to test) is whether or not we need to keep using div instead of button, as I'm doing where. I'm a fan of the semantic web but I do remember at one point we had an issue where using button caused some unnecessary submit event firing.
If you'd rather I backfill these changes into the existing button we can definitely do that instead. Let me know what you think.

I'm going to merge in this module for now to unblock myself but definitely open to reverting to the existing module/porting this module features over.

I understand your pain around the existing Button component, but without taking the effort to migrate the existing usages of the old Button component to this one, then we end up in a worse situation. I think we should definitely take time to ensure that we are just using one component rather than two.


Sparse Batch Particle Storage

The current implementation of sparse particle storage only works when the current state is stored as a vector of particles. For GPU implementations where the particles are stored as a dense matrix this isn't appropriate.
This can be overcome by adding a specialised getindex for these containers.

Solved in #22


tests(plugins) wait for UDP server threads to start

This is a band-aid solution to the remaining class of
ephemeral failures happening in our CI. Most threads that
spawn a UDP server use helper:udp_server which does
sleep. A few ones run customized servers, and those were
lacking the sleep.
The most proper solution would be to implement an initialization
handshake (and to refactor the tests touched by this patch to
avoid the repetition that proliferated this bug), but this is
the least-invasive patch to address the problem, submitted in
hopes that it can be applied quickly to address the CI issues.
We will revisit this issue properly when we review the test suite
as a whole in the future.

@kikito Thank you! :)


On iOS UNUserNotificationCenter delegate is set only when askNotificationPermissionOnStart is true

As seen from the code:
https://github.com/mirzemehdi/KMPNotifier/blob/29def58f7c60a06a1b13f7f1a01dd9058c1462ae/kmpnotifier/src/iosMain/kotlin/com/mmk/kmpnotifier/permission/IosPermissionUtil.kt#L28
The UNUserNotificationCenter.currentNotificationCenter().delegate is set only if permission are granted, and this method is only called if askNotificationPermissionOnStart is true.
I don't know what the best fix could be, but something looks wrong (maybe it's just me)

Closing this issue due to inactivity. If this is still an issue or need a help, please feel free to reopen the issue and provide additional context or updates. Thank you.


[DM-27809] Test validate_drp, validate_drp_gen3 and ap_verify pipelines with the squash-sandbox instance

Repeat the dispatch verify step sending verification jobs for the validate_drp, validate_drp_gen3 and ap_verify pipelines to to the SQuaSH Sandbox instance, in addition to the current production instance.  We want both running together for some time to make sure it works before switching over to a new production instance.

@ktlim please let me know if you agree with these changes.

@ktlim please let me know if you agree with these changes.

This looks reasonable, although you might consider having only one loop in each case, calling dispatch for each file twice, instead of having two loops.

This looks reasonable, although you might consider having only one loop in each case, calling dispatch for each file twice, instead of having two loops.

@ktlim that's a good suggestion, thanks!

@ktlim that's a good suggestion, thanks!


Minor improvements to "Fix folder autocreation" PR

Some minor improvements to jgosmann/dmarc-metrics-exporter#16 before merging.

LGTM, Thanks!


feat(connector): add new config value for connector

add new selfDecriptionUrl
add logging for sdFactory in adminstration service
Refs: CPLP-2193

@evegufy the corresponding pr for the backend is https://github.com/catenax-ng/tx-portal-backend/pull/193


Swift Package compatible

Create Package.swift to make Swift package compatible

Please feel free to make another PR to README if there's any further instruction for this.


[verilator] Load elf instead of vmem

Currently we need to pre-process elf files into vmem files to load them into a Verilator simulation (through $readmemh()). It would be nice if we can improve Verilator to directly load elf files.
Steps towards that:

Use libelf to convert elf file into binary. Sample code e.g. https://github.com/opensocdebug/osd-sw/blob/master/src/libosd/memaccess.c#L167 or all across the Internet. (No, not beautiful.)
Load the memories with this binary data.

Either convert the binary into vmem, dump it to a temporary file, and then use $readmemh()
Or (preferred) directly write the mem variable in the prim_generic_ram_1p.sv  & Co. through Verilator. Either by making it /* verilator public*/ or through a task/function in the SV code, similar to the simutil_verilator_memload function in prim_generic_ram_1p.sv & Co.

There might be existing code out in the internet that does that, it might be worth googling first :-)

Supported by #1091


SDXL Support

Hello, are you planning to add support for SDXL ? Best, Andre

I'm looking at it. Made a go a week or two ago but left with the impression it was better to wait till they had worked out some bugs. Revisiting now.

Looks like Controlnet now supports SDXL:
https://huggingface.co/docs/diffusers/main/en/api/pipelines/controlnet_sdxl

BTW I found SDXL a breeze to use outside imaginary. It uses much less RAM by default and the standard image generation/ controlnet use rather easy. There are some questions like guidance scale and controlnet parameters that seem quite random judging from the documentation.

Good to hear. Where are you using it?

Been using it just like Imaginairy for txt2img and img2img. What I struggled with is to finetune parameters for image similarity - seems too many options for what I can handle :)

So you're just using the vanilla sdxl repo?

Wrote just 5kB of Python. But yes it has img2img inbuilt.
from diffusers import StableDiffusionXLPipeline
import torch
torch.cuda.empty_cache()

base_model_id = "stabilityai/stable-diffusion-xl-base-1.0"

pipe =StableDiffusionXLPipeline.from_pretrained(base_model_id, torch_dtype=torch.float16)

pipe = pipe.to("cuda")

num_samples = 3
guidance_scale = 12.5
num_inference_steps = 75
generator = torch.Generator("cuda").manual_seed(0)

images = pipe(
prompt,
negative_prompt=negative_prompt,
num_images_per_prompt=num_samples,
num_inference_steps=num_inference_steps,
guidance_scale=guidance_scale,
generator=generator
).images

Took Controlnet code from here: https://github.com/huggingface/diffusers/pull/4592

Interesting. Maybe I should just integrate with diffusers

Hi. May I ask how things are going with this? Do you have any plans to implement this? I am grateful in advance

Yes I'm integrating with the refiners library, which supports SDXL.  The SD1.5 generations now use this.
Basically the challenge is that refiners uses a different weight format so I need to write realtime conversion script. I've only gotten it done for SD15 and the controlnets so far.  Worked on it tonight, but this is just a side project so it could be a while.

Also have you seen this?
https://github.com/williamyang1991/Rerender_A_Video

Both pretty interesting. who knows when there will be time to integrate

Added in just released 14.0.0


Add a way to capture events values to be able to write tests requiring them

As the issue #581, I need an easy way to capture values from event to reuse them.
For the moment and thanks to @wminshew I did this:
  it('should burn tokens if asked', async () => {
    const response = await (
      await contract.safeMint(await owner.getAddress())
    ).wait();

    expect(response.events[0].event).to.equals('Transfer');
    const [, , tokenId] = response.events.[0].args;

    await expect(await contract.burn(tokenId)).to.emit(contract, 'Transfer');
  });

Note the boring stuff about response / await / await / wait and the required expect on the event name. Maybe something like that could be interesting:
const args = await expect(await contract.safeMint(await owner.getAddress())).to.emit(contract, 'Transfer').args();
const tokenId = args[2];

Interesting, wondering what we would like to return in the case of chaining the matcher, ie:
await expect(tx)
  .to.emit(contract, 'Transfer')
  .to.emit(contract, 'Mint')
  .args()

Just the arguments of the last one, Mint?
Have you thought about this, what would make sense to you in this case?

Yes, it would be perfect ie args() returning the values of the event from the last emit().


RuntimeError: CUDA error

Hi,
I am running on Colab and this appears "RuntimeError: CUDA error: no kernel image is available for execution on the device". Do you have any suggestion?

I fixed.


#91: updating to cocoapods 1.0+

This PR updates to Cocoapods 1.0+ syntax for the Examples app, as well as upgrading to Reveal-iOS 1.6.2 (to avoid bitcode errors).

Great! 👍


[QUESTION] Chaos monkey - Can I Restart Azure SignalR ?

Hello,
In an attempt to test our “resiliency and continuous deployment”, I’d love to be able to do the equivalent of the Restart button in the Azure SignalR Service in the Azure Portal.
For example in an Azure Function so that is could b either a Timed one or a POST.
I guess the Az CLI would probably have something but not sure if it can be ran in a Function.
I’m really curious on the Auth part especially
(It will run in the same situation subscription)
TBF I did not looked a lot, I’m wondering what’s your guidance on that ?

Hi @tebeco,  as you said, Azure SignalR Service does have a az signalr restart command.
For Functions, we don't have a specific guidance about how to manage SignalR service. However there is a general idea. You can think of Azure Functions as a ordinary App Service, that means you can config Service Principal (Client Id and Client Secret) in your Functions config, and inside your Functions, you are able to read them from config, and then authenticate Azure using the SP, and then manage Azure Resources including SignalR via Azure Management SDKs.
Hope this helps.


24_esp32s2_s3_camera_wechat_mini 在S3上面编译不通过idf4.4

都限制了
if(IDF_TARGET STREQUAL "esp32") set(COMPONENT_SRCS driver/camera.c
driver/sccb.c
driver/sensor.c
driver/xclk.c
sensors/ov2640.c
sensors/ov3660.c
sensors/ov5640.c
sensors/ov7725.c
sensors/ov7670.c
sensors/nt99141.c
conversions/yuv.c
conversions/to_jpg.cpp
conversions/to_bmp.c
conversions/jpge.cpp
conversions/esp_jpg_decode.c
)
set(COMPONENT_ADD_INCLUDEDIRS driver/include
conversions/include
)
set(COMPONENT_PRIV_INCLUDEDIRS driver/private_include
sensors/private_include
conversions/private_include
)
set(COMPONENT_REQUIRES driver)
set(COMPONENT_PRIV_REQUIRES freertos nvs_flash)
register_component() endif()

Hi
With above cmake content, it only compiles on esp32 not on esp32s3


update shopify app express and sql lite session storage

WHY are these changes introduced?
After this release of many js npm packages, we should update the  dependencies to their latest versions.

WHAT is this pull request doing?

Test this PR
shopify app init --template=https://github.com/Shopify/shopify-app-template-node#andy-liuu/update-dependencies

Checklist
Note: once this PR is merged, it becomes a new release for this template.

[ ] I have added/updated tests for this change
[ ] I have made changes to the README.md file and other related documentation, if applicable

After this change. I am unable to generate access token in sqlite for custom server. it was working fine before this change.

@fullstackdeveloper918
Thank you for flagging this! Could you please create an issue with reproduction steps? This will help us investigate and resolve the bug more effectively.

There is no need to send any code, video. I am not using my app which I built with yiur app just to make sure wther it is my code issue or template's issue. I am using exact same sample app'code,It is very first step. this issue comes with installation. First we start with normal commands you mentioned in readme file. When we go with normal cloudflare urls which app generate automatically, it gets installed on one store and stores access token in sqlite.
2nd time, I have to change my redirect urls, application urls so I can recieve access token on my server. After updating my server urls it never store any access token in sqlite DB. I am not doing any mistake in settings. I have same access scopes, valid working redirect urls, valid apiKEY and Secret.
You can easily replicate issue by trying app on custom linux server with ubuntu 22.04 version and nginx server.

2nd time, I have to change my redirect urls, application urls

To clarify are you saying that when you are in local development using a cloudflare tunnel the installation works correctly. But when you are running your application in production, and update to the production application URLs that is when the app is not installing correctly?

updating my server urls it never store any access token in sqlite DB.
If you turn on debug logging are you seeing any errors in the terminal? Are you seeing any errors in the UI during install? Does your app show as installed in the list of installed apps in the Shopify Admin?

It gets installed successfully. When I try to open the app inside the admin
panel, it gives an issue called exit frame outside from shopify. This issue
generally comes when we miss the access token. I have checked the SqlLite
and the token was not there. Let me check the debug log as well.
Thanks
@.***>
Pankaj Sharma Director, CyberSify Cloud Computing Pvt. LTD.
950 <9501151756>1 <9501151756>151756 <9501151756> | @.***
https://cybersify.tech
D 264-265B Phase, 2nd Floor, Phase 8B, Industrial Area, Sector 74,
Sahibzada Ajit Singh Nagar, Punjab 160071
On Wed, 13 Nov 2024 at 01:08, Liz Kenyon @.***> wrote:

2nd time, I have to change my redirect urls, application urls
To clarify are you saying that when you are in local development using a
cloudflare tunnel the installation works correctly. But when you are
running your application in production, and update to the production
application URLs that is when the app is not installing correctly?
updating my server urls it never store any access token in sqlite DB.
If you turn on debug logging are you seeing any errors in the terminal?
Are you seeing any errors in the UI during install? Does your app show as
installed in the list of installed apps in the Shopify Admin?
—
Reply to this email directly, view it on GitHub
https://github.com/Shopify/shopify-app-template-node/pull/1343#issuecomment-2471409138,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIJYMG35M6D75QU6VVWLGBL2AJKLHAVCNFSM6AAAAABQ2DI6H2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZRGQYDSMJTHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Wrong loss function?

I notice that the code uses CrossEntropyLoss for local training:
https://github.com/pliang279/LG-FedAvg/blob/7af0568b2cae88922ebeacc021b1679815092f4e/models/Update.py#L28
And it accepts the log-probabilities as input:
https://github.com/pliang279/LG-FedAvg/blob/7af0568b2cae88922ebeacc021b1679815092f4e/models/Update.py#L50
The output of CNN networks is also logsoftmax:
https://github.com/pliang279/LG-FedAvg/blob/7af0568b2cae88922ebeacc021b1679815092f4e/models/Nets.py#L104
But according to the doc of PyTorch, CrossEntropyLoss already has logsoftmax inside:
https://pytorch.org/docs/stable/generated/torch.nn.CrossEntropyLoss.html#torch.nn.CrossEntropyLoss
I think the loss should be calculated via NLLLoss instead if used with input after logsoftmax (https://pytorch.org/docs/stable/generated/torch.nn.NLLLoss.html#torch.nn.NLLLoss).
Or is there any reason why the code uses logsoftmax twice for calculating the loss?

Same issue


run synthetic_full with BEFORE_SYNTH is not working

Hi, i try to start a synthetic_full job of a subclient.
>>> job = subclient.backup('synthetic_full', True, 'BEFORE_SYNTH')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.4/site-packages/cvpysdk/subclient.py", line 1489, in backup
    return self._process_backup_response(flag, response)
  File "/usr/lib/python3.4/site-packages/cvpysdk/subclient.py", line 800, in _process_backup_response
    raise SDKException('Response', '101', response_string)
cvpysdk.exception.SDKException: Response was not success
Incremental level for backup needs to be passed when runIncrementalBackup flag is set to true in backup request
As you see, i set the Incremental level parameter, but the error says that the information is missing.

Hi Daniel,
Some changes have been made to the API end-point. I'll update the SDK code as the per the modified API, and update the repository in this week.
Thanks for bringing this to our attention.
Thanks,
Jatin

Daniel,
The code has been updated with the latest changes, including fix for this issue.


feat: rocks check

Adds a simple implementation of rocks check, which ensures luacheck is installed and runs it for the whole source directory of a given project.
Similarly to rocks fmt, we'll want to expand this to take in a path as an argument for finer control.
I don't believe we should let the user control luacheck with flags, that should be up to the luacheckrc file.

Will fix the failing build in a bit.


Removing this gem breaks the migrations

Because the migrations reference class properties to determine the table name, rails db:migrate no longer works when you remove acts-as-taggable-on from your Gemfile. Is there some other way to uninstall this gem?

Remove the migration files with the Gem

you should not be doing "db:migrate" to set up new environments, use db:schema:load for that purpose, is faster and less error prone. Migrations are for incremental updates, for production and your co-workers local machines that have been upgrading one migration at the time.


Implementing rkt app add and rkt app rm

This follows https://github.com/coreos/rkt/pull/2932
rkt app add
rkt app add <pod-uuid> --app=<app-name> <image-id> <arguments>
This can be just a stage0 sub-command, it will

Extract the image and mount it into "/var/lib/rkt/pods/run/pod-uuid/stage1/rootfs/opt/stage2/app-name/".
Create an image manifest under the directory
Update the pod manifest with this additional app info (need to be thread-safe operation).

All other volume/device mounts, systemd sevice file creation, etc, are left to be handled by rkt app start.
The <arguments> contains fields in the original image that we want to override, the semantics could be similar to actool patch-manifest. e.g.
$ rkt app add 22db0135 \
--app=my-busybox busybox \
--exec=/bin/sleep 1000 \
--mounts=/path/to/host/foo:/path/to/container/foo \
--ports=8080:80

For simplicity, rkt app add only accept image-id, which means the image should already be fetched into the store, otherwise it bails out.
If the pod is not able to be mutable (could be implemented by providing --mutable flag to rkt prepare), then rkt app add bails out.
rkt app rm
rkt app rm <pod-uuid> --app=<app-name>
This needs to be implemented as a stage1 entrypoint coreos.com/rkt/stage1/app/rm in the stage1's pod manifest's annotations. For default stage1 (systemd-nspawn), it will:

Stop the app if it's still running.
Remove the app's systemd service, and reload the stage1 systemd.
Unmount all mounted volumes/dev for this app.
Remove rootfs of this app.
Remove the app section in the pod manifest, remove other files, including image manifest, /stage1/rootfs/rkt/env/*, /stage1/rootfs/rkt/status/*, /stage1/rootfs/rkt/volumes/*, etc.
Run CNI plugins again if necessary?

Basically this should make the pod be identical to the state before it run rkt app add.
Feel free to add anything I am missing.
cc @tmrts @s-urbaniak @euank

@yifan-gu @sgotti how does this fit within the context of https://github.com/coreos/rkt/issues/3098 and https://github.com/coreos/rkt/pull/3100?

@yifan-gu @euank In a mutable pod there are some host&guest resources that needs to be updated with new/removed apps (sysctl, port-forwarding, mount-volumes, etc). Should the pod-sandbox be updated at injection removal/time or on start/stop?

@lucab Arguments for doing these during 'removal':

Usually the 'stop' operation is synchronized, and removal is usually in asynchronized (gc running in background). So leave the cleanup in gc makes performance a little bit better?
We would still need to read out termination messages, logs, so mount-volumes are definitely preferred to be there until removal.

@yifan-gu @sgotti how does this fit within the context of #3098 and #3100?

I think it'll be ok if the stage1s only rely on the pod manifest and ignore the image manifest. So it's ok if the new implementation won't rely on the app manifest (while the current parts using it will be fixed by #3098).
The image manifest extracted inside the pod dir is currently used by the points explained in #3098 for the stage1 and also by other parts in the stage0 (e.g. rkt export) (and perhaps also so by https://github.com/blablacar/dgr?) so it's ok the extract it (in future we'll see if/how it can be totally removed).

Here's my WIP branch for this: https://github.com/kinvolk/rkt/commits/iaguis/rkt-app-add-rm
It's based on https://github.com/coreos/rkt/pull/3149

Based on my understanding of the API spec, we'll need to support the following options:

--exec and args
--set-env and --set-env-file
--annotation and --label
--caps-remove and --caps-retain
--readonly-rootfs
--uid=, --gid=, and --additional-gid=
some SELinux options

We need further clarifications on what the stdin, stdin_once, and tty parameters mean.

Regarding last comment, most of those topics have already been addressed and for the remaining ones we have specific tickets:

SELinux at https://github.com/coreos/rkt/issues/3257
IO and TTY (attach/exec/logging) at https://github.com/coreos/rkt/issues/1799


11 implement git style hooks

Note: Did not run it on the repo yet.
Should I run it and also commit the changes to files?
This is the file we have in ml-ebit. Do we want more formatting done, or is this enough?
closes #11

No need to run the git hook on the whole repo. Let's just apply it to
future commits.
Same styling guidelines as ML should be fine.
On Tue, Mar 10, 2020 at 2:24 PM Chaya Danzinger<EMAIL_ADDRESS>wrote:

Note: Did not run it on the repo yet. Should I run it and also commit the
changes to files?
This is the file we have in ml-ebit. Do we want more formatting done, or
is this enough?
You can view, comment on, or merge this pull request online at:
https://github.com/enjine-com/mcos/pull/30
Commit Summary

implemented code from link in issue
added tests
added pre-commit file
pre-commit config

File Changes

A .pre-commit-config.yaml
https://github.com/enjine-com/mcos/pull/30/files#diff-82c3b36709926de3a1b27bea0d8258cc
(15)
M mcos/optimizer.py
https://github.com/enjine-com/mcos/pull/30/files#diff-ec43181814349f0c1dcc2f949e1a7799
(70)
M requirements.txt
https://github.com/enjine-com/mcos/pull/30/files#diff-b4ef698db8ca845e5845c4618278f29a
(1)
M tests/test_optimizer.py
https://github.com/enjine-com/mcos/pull/30/files#diff-eaa4c9399a1bdc7844365b8c5a09082c
(32)

Patch Links:

https://github.com/enjine-com/mcos/pull/30.patch
https://github.com/enjine-com/mcos/pull/30.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/enjine-com/mcos/pull/30?email_source=notifications&email_token=AAXTIXNHFNSXNSPQUQB3DJDRG2AWJA5CNFSM4LFFHN6KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IT7N7CA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXTIXIUA2RSQCDCFKPZURLRG2AWJANCNFSM4LFFHN6A
.

--
Jin Choi, PhD
Founder/President
(226) 980-6902 - enjine.com
“Bring Your Quant Vision to Life”

Looks good to me


rename master branch to main

What this PR does / why we need it:
We are in the process of renaming the master branch across all repos, and this PR prepares this repo for the renaming.
/kind cleanup
Does this PR introduce a user-facing change? Then add your Release Note here:
The primary Git branch was renamed from `master` to `main`.

Documentation:
NONE

/hold

/override pull-machine-controller-e2e-aws
/override pull-machine-controller-e2e-aws-arm
/override pull-machine-controller-e2e-aws-ebs-encryption-enabled
/override pull-machine-controller-e2e-aws-flatcar-containerd
/override pull-machine-controller-e2e-aws-legacy-userdata
/override pull-machine-controller-e2e-aws-spot-instance
/override pull-machine-controller-e2e-azure
/override pull-machine-controller-e2e-azure-custom-image-reference
/override pull-machine-controller-e2e-deployment-upgrade
/override pull-machine-controller-e2e-digitalocean
/override pull-machine-controller-e2e-hetzner
/override pull-machine-controller-e2e-openstack
/override pull-machine-controller-e2e-openstack-project-auth
/override pull-machine-controller-e2e-ubuntu-upgrade

/override pull-machine-controller-e2e-gce
/override pull-machine-controller-e2e-vsphere
/hold cancel


DescribeDeltaDetailsCommand doesn't call DeltaLog.update

DescribeDeltaDetailsCommand doesn't call DeltaLog.update after DeltaLog.forTable: https://github.com/delta-io/delta/blob/1fcb1dacc8d4ec671cd499676bb61d55d343cc2a/core/src/main/scala/org/apache/spark/sql/delta/commands/DescribeDeltaDetailsCommand.scala#L79
Hence it may return a stale result if DeltaLog.forTable returns a cached DeltaLog.

@zsxwing the change i am assuming is to call a simple deltaLog.update().
How would you test this? I notice that the DeltaLog startTransaction seems to be calling an update. Does this happen when potentially an entirely different spark cluster/client creates a transaction and this detail will return a stale result due it just fetching from the cached log?

Does this happen when potentially an entirely different spark cluster/client creates a transaction and this detail will return a stale result due it just fetching from the cached log?

Right. To simulate this, we can write a test case like this:
create a delta table

val deltaLog = DeltaLog.forTable(spark, ...) // Keep the current DeltaLog
DeltaLog.clearCache() // Remove the above DeltaLog from the cache

sql("desc detail ...") // Cache a new DeltaLog

val txn = deltaLog.startTransaction()
val newMetadata = txn.snapshot.metadata.copy(configuration = metadata.configuration ++ Map("foo" -> "bar"))
txn.commit(newMetadata, DeltaOperations.ManualUpdate)

Now run desc detail again to verify we can see the new added table properties.

@zsxwing  I am a new Delta user and  ready to tackle the issue, Can you assign it to me? thanks

@dragongu done. Thanks for the contribution!

Fixed by #1133

Why was it closed, but not merged?

@dragongu the fix has been merged to master: https://github.com/delta-io/delta/commit/32c3681f37a01d10565c21b70e1296bba0b2ea25
We don't use GitHub merge button to merge PRs so that the status of a PR will always be closed. You can see the bottom in the PR which should have a comment showing which commit contains the change.


Hamburger menu

How do I disable hamburger menu and enable menu bar that is standard accross apps in whole system? I have no idea why is everyone so eager to create yet another version of this user-unfriendly stupidity.

Hi @martinwilczak. Thanks for raising this issue. I understand you're frustrated and would like to find out if you're open to raising this in the Ask Anything channel or on the Mattermost Forum. The community members on both platforms may have experience with this and be able to provide you with some solutions and suggestions that they've implemented.

Closing as it's the wrong repo for this kind of issue.
Thanks @justinegeffen for giving pointers for the right locations to report this :)


CHANGELOG: New release 1.1.0

Signed-off-by: Fernando Fernandez Mancera<EMAIL_ADDRESS>

Closing this. Will re-open later.


Add built-in use of caching

It would be useful for actions/setup-java and to use the same repository cache mechanism used by actions/cache to cache JDKs. This appears to be possible since the addition of https://github.com/actions/toolkit/tree/main/packages/cache in May.
setup-java should still look up the JDK to use (as the JDK determined by a semver spec and other options can change over time, e.g. as new updates get published). Once a specific JDK is determined (e.g. a specific tar.gz or .zip file URI is identified as a key), we can check (and populate) the cache to reuse the actual file hierarchy created for that JDK within that repository's cache across workflow runs. This can likely significantly improve workflow execution times, and bandwidth consumption.

Nice sugestion... as said, improve performance. But i have another point, sometimes the download of the JDK fails and the entire build fail cause of it... so for me it will also reduce workflow failure.


Add: Uruguay Presidencia [UY]

Channel Name
Uruguay Presidencia
Alternative Names (optional)
No response
Network (optional)
No response
Owners (optional)
No response
Country
UY
Subdivision (optional)
No response
City (optional)
No response
Broadcast Area
c/UY
Languages
Spa
Categories (optional)
Legislative
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
No response
Logo
https://cds-assets.cdn.antel.net.uy/imagenes/recursos/6qo1JPEkAV0VQNm20ijOnXCCxcDaX2PMWRnhisZm.jpeg
Notes
No response

Added in https://github.com/iptv-org/database/commit/b3e079a4f019936d827aa323411273f5c7440ee9


chore: release main

:robot: I have created a release beep boop
react-shader-fiber: 1.0.0
1.0.0 (2023-12-22)
Bug Fixes

update readme (31d7fe4)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/pavophilip/react-shader-fiber/releases/tag/react-shader-fiber-v1.0.0 :sunflower:


refactor: Treat scripts as a yarn workspace

Description 📝

Updates our repo to treat our existing scripts/ directory as its own yarn workspace
The motivation behind doing this is to prepare to fix the issue shown below when we begin using pnpm, which hoists packages differently

How to test 🧪

Verify existing scripts still work

For example, verify yarn chnageset works

Verify there are no Cloud Manager regressions as a result

 Author Checklists
As an Author, to speed up the review process, I considered 🤔
👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿  Providing accessibility support

[ ] I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

[ ] All unit tests are passing
[ ] TypeScript compilation succeeded without errors
[ ] Code passes all linting rules

@bnussman-akamai I think passing the --silent flag to the yarn workspace ... commands in the root package.json should resolve the issues with the GitHub and Slack notifications.

@jdamore-linode I had to resort to using YARN_SILENT, but it works as expected. We will be able to re-evaluate this when we switch our package manager.


Class constant assignments cannot be inherited or used as namespaces

Or somehow, T.let does not work for classes.
Input
→ View on sorbet.run
# typed: true
extend T::Sig

class A < Object
end

B = T.let(const_get('A'), T.class_of(A))

T.reveal_type(B)

class C < B
end

class B::D
end

Observed output
editor.rb:15: Can't nest D under B because B is not a class or module https://srb.help/4015
    15 |class B::D
              ^^^^
    editor.rb:8: B defined here
     8 |B = T.let(const_get('A'), T.class_of(A))
        ^

editor.rb:12: The super class B of C does not derive from Class https://srb.help/5067
    12 |class C < B
        ^^^^^^^^^^^
    editor.rb:15: B defined here
    15 |class B::D
              ^^^^

editor.rb:10: Revealed type: T.class_of(B) https://srb.help/7014
    10 |T.reveal_type(B)
        ^^^^^^^^^^^^^^^^
  Got T.class_of(B) originating from:
    editor.rb:10:
    10 |T.reveal_type(B)
                      ^
Errors: 3

Expected behavior
B is usable as both a namespace and a parent class.

I'm trying to apply sorbet to a project that uses active_type. This gem generates classes dynamically. I understand it is not possible to use it directly (4002). I thought T.let could be used to teach sorbet that B is a class that inherits from A, but it appears to not be possible. Current workaround is to hide the definitions from sorbet with const_set, and provide a fake definition (class B < A) out of band via rbi files.

Unfortunately, this is either a feature or an accepted limitation of Sorbet depending on your viewpoint.
For performance, Sorbet resolves all classes and modules, including all mixins and superclass relationships, before it resolves constants assignments declared with T.let. This forces a strict separation between things that are classes/modules and other constants. We have no plans to change this architecture decision.
The current workaround for codebases that must structure their code this way (where the codebase itself can't be changed to stop relying on this pattern) is to use RBI files to declare constants as proper classes, and to hide the constant assignments with calls to const_set so that Sorbet only sees the definition in the RBI file, not in the source file.

Unfortunately, this is either a feature or an accepted limitation of Sorbet depending on your viewpoint.

OK, thanks for explaining.


HACS issue

Hi,
i installed HASS on rpi4 ; went step by step to install HACS but i got error
2021-09-27 00:10:56 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration frigate which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2021-09-27 00:10:56 WARNING (SyncWorker_1) [homeassistant.loader] We found a custom integration hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2021-09-27 00:10:59 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection] [548460526320] Received invalid command: system_health/info
2021-09-27 00:10:59 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection] [548460526320] Received invalid command: system_health/info
2021-09-27 00:10:59 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection] [548460526320] Received invalid command: energy/get_prefs
2021-09-27 00:10:59 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection] [548460526320] Received invalid command: energy/get_prefs
2021-09-27 00:10:59 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection] [548460526320] Received invalid command: energy/get_prefs
2021-09-27 00:10:59 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection] [548460526320] Received invalid command: energy/get_prefs
2021-09-27 00:11:05 ERROR (MainThread) [custom_components.hacs] GitHub API ratelimited - 1008 remaining
2021-09-27 00:11:05 ERROR (MainThread) [custom_components.hacs] HACS is disabled - HacsDisabledReason.RATE_LIMIT
2021-09-27 00:11:05 WARNING (MainThread) [homeassistant.config_entries] Config entry '' for hacs integration not ready yet: None; Retrying in background
not sure if its HACS issue or HASS or frigate issue?
running Home Assistant 2021.9.7
HACS Integration version: | 1.15.2
if i go to HACS -> information>
HACS is disabled (Ratelimited)
GitHub API calls are ratelimited, this will clear in less than 1 hour.
thanks

HACS issue

@blakeblackshear how do u know? thanks

The error message makes it clear (if you're used to looking at these error messages, totally understand not everyone is!) that this is nothing to do with Frigate and is a HACS issue: HACS has not yet started in your HA instance due to a rate limited request to github, give it more time and it should eventually finish loading.

@dermotduffy yes it was HACS issue; there is new issue once HACS was installed properly>
https://github.com/hacs/integration/issues/2226#issuecomment-927694887

It might be connected with mosquitto?
1632734922: New connection from <IP_ADDRESS> on port 1883.
1632734922: Socket error on client , disconnecting.
1632734974: Socket error on client 28gPtUV6GbHgOuRqZySVmk, disconnecting.
1632734991: New connection from <IP_ADDRESS> on port 1883.
1632734991: New client connected from <IP_ADDRESS> as 5vt0kmhIPIbLyDg73Tqtrm (p2, c1, k60, u'homeassistant').
1632735042: New connection from <IP_ADDRESS> on port 1883.
1632735042: Socket error on client , disconnecting.
1632735162: New connection from <IP_ADDRESS> on port 1883.
1632735162: Socket error on client , disconnecting.


Embed k8s controller into maya-agent and look for events RSA-CRDs

FEATURE REQUEST
The feature is to embed k8s controller into maya-agent and look for events RSA-CRDs.
maya-agent should handle events of CRDs in case of new block device is found or removed.
Refer:
https://github.com/openebs/openebs/issues/194

This is not being moved to node-disk-manager along with refactored changes from https://github.com/openebs/maya/pull/224
Thank you @gkGaneshR for your efforts on this one.


Added support for FakeSlots and SyncableInventory

"Fake", "Ghost" or "Phantom" slots are called that way because they
do not allow direct stack interaction as in vanilla, but behave differently
when clicked. This system is often used to set "filters" for items without
actually using up the item in the players hand.
Buildcraft does this by preventing the Slots from being manipulated as usual,
but instead sends a packet to the server containing the details of the click
and the hold item.
This pull requests replicates this mechanism and exposes the system via
FakeSlot, FakeGuiContainer and IFakeSlotListener.
It also includes a straight forward SyncableInventory implementation extending
GenericInventory.
A usage example for both can be seen in an upcoming pull request to OP-Addons.

Feature was implemented in different way.


Fix Helm appsub delete not removing appsubstatus.

Fix finalizer not being added to HelmRelease CR
Signed-off-by: Mike Ng<EMAIL_ADDRESS>

/lgtm


Re-add the v1 to v2 legacy migration to the utilities page

similar to https://alchemix-v2-frontend-git-fix-vault-migration-alchemix.vercel.app/utilities

Completed in #233, prod at #238.


Fix notebook links

Example notebook links were broken.

thank you!! missed those in the docs refactor :)


Fix null dereference in manual update check

The manual update check receives a fresh instance of UpdateChecker from dependency injection, which does not have the timer set. Fix this by changing it to a singleton. In addition, make it explicit that the timer may be null if the automatic update checker is never started.

Thanks for the fix!


feat(pictograms): Update PictogramCategory.js to display images @ 48x48

changes card display from 64x64 to 48x48, the native size of the source svg
Closes #4121
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

Made Vince's requested change.


真机报错

New Issue Checklist

[ ] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
e.g. flutter / ios / android

Platform Version
e.g.  1.5.0 / 12.0 / 9.0

Dio Version
e.g.  2.1.0 / 1.0.17

Android Studio / Xcode Version
e.g. Android Studio 3.3.2 / Xcode 10.2.1

Repro rate
e.g. all the time (100%) / sometimes x% / only once

Repro with our demo prj
e.g. does it happen with our demo project?

Demo project link
e.g. link to a demo project that highlights the issue

Issue Description and Steps
Please fill in the detailed description of the issue (full output of any stack trace, compiler error, ...) and the steps to reproduce the issue.
真机上报错：DioError [DioErrorType.DEFAULT]: SocketException: Connection failed (OS Error: No route to host, errno = 65), address = <IP_ADDRESS>, port = 7002

I‘ve got the same exception too. In the iOS simulator, the dio work well, but in the iPhone the exception was raised. What's more, the same exceptional code in another test app just work well in both sides.


Hey, this isn't really issue. Just curious

So recently i found out that the extension ain't working anymore, as i can resolve from older issues , it's gotta do something with either chrome's new update or windows. So i was just curious about how can i find new keys instead of rolling back. I'd really appreciate if you'd draft that write up you were talking about.
Thank You

For things that are not really an issue please you the Discussions page.


Remove tar as Direct Dependency

https://github.com/edm00se/generator-presto-preso/pull/59#discussion_r299114869

:tada: This issue has been resolved in version 1.6.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix Kudu output operator for some minor fixes

@tweise / @amberarrow  : Could anyone of you please review this minor change.

Need a matching JIRA and the commit message should have the JIRA number as a prefix.

@amberarrow   - Updated the branch with the correct JIRA number in the commit message.

@tweise  - Rebased to the latest. I also removed the extra constructors and added a safety check for timestamp propagation. Could you please review and merge ?


Add a set team mode

If you are super prideful in being on your team (like me haha) you might want to always see your own robot. Also added a couple javascript functions to help switch in between the clock/random/set team modes

oop jk i didnt see the options tab my bad :P

Yeah, was going to say. You can already do this.

Thank you for the submission, though!


get setcookie language value error!

when i scan the url
python dz-ml-rce.py -u "http://xxx.com/forum.php"
i get this error:
get setcookie language value error!

This mean can not get the prefix value(such as 4gH4_0df5_),maybe time out or others,please try to add -s 20 or try it manually.


Failed install frontera in a cluster mode on Python 3.7

Hello,
I install frontera with the following command:
pip3 install frontera[distributed,kafka,hbase]
There is an error as shown in the below image

How should I do to fix the error and setup frontera in a cluster? Thanks for your help,
My environment:
MacOS: 10.14.2
Python: 3.7.2
Scrapy: 1.5.1
Frontera: <IP_ADDRESS>
Detail error:
`Building wheels for collected packages: happybase, thriftpy
Running setup.py bdist_wheel for happybase ... done
Stored in directory: /Users/phongnguyen/Library/Caches/pip/wheels/7a/d7/f7/447e781495238e394f04fe169f14549d31c63993e90152693b
Running setup.py bdist_wheel for thriftpy ... error
Complete output from command /usr/local/opt/python/bin/python3.7 -u -c "import setuptools, tokenize;file='/private/var/folders/0l/3k2zrhsn01b1sy7nktflll640000gn/T/pip-install-hy9tc71e/thriftpy/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" bdist_wheel -d /private/var/folders/0l/3k2zrhsn01b1sy7nktflll640000gn/T/pip-wheel-0zueewe3 --python-tag cp37:
running bdist_wheel
The [wheel] section is deprecated. Use [bdist_wheel] instead.
running build
running build_py
creating build
creating build/lib.macosx-10.14-x86_64-3.7
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/server.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/hook.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/thrift.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/rpc.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/utils.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/tornado.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/_compat.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport
copying thriftpy/transport/_ssl.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport
copying thriftpy/transport/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport
copying thriftpy/transport/sslsocket.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport
copying thriftpy/transport/socket.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
copying thriftpy/protocol/binary.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
copying thriftpy/protocol/compact.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
copying thriftpy/protocol/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
copying thriftpy/protocol/exc.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
copying thriftpy/protocol/multiplex.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
copying thriftpy/protocol/json.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/parser
copying thriftpy/parser/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/parser
copying thriftpy/parser/parser.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/parser
copying thriftpy/parser/exc.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/parser
copying thriftpy/parser/lexer.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/parser
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/contrib
copying thriftpy/contrib/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/contrib
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport/memory
copying thriftpy/transport/memory/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport/memory
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport/buffered
copying thriftpy/transport/buffered/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport/buffered
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport/framed
copying thriftpy/transport/framed/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport/framed
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/contrib/tracking
copying thriftpy/contrib/tracking/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/contrib/tracking
copying thriftpy/contrib/tracking/tracker.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/contrib/tracking
copying thriftpy/contrib/tracking/tracking.thrift -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/contrib/tracking
running build_ext
building 'thriftpy.transport.cybase' extension
creating build/temp.macosx-10.14-x86_64-3.7
creating build/temp.macosx-10.14-x86_64-3.7/thriftpy
creating build/temp.macosx-10.14-x86_64-3.7/thriftpy/transport
clang -Wno-unused-result -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk -I/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include -I/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/System/Library/Frameworks/Tk.framework/Versions/8.5/Headers -I/usr/local/include -I/usr/local/opt/openssl/include -I/usr/local/opt/sqlite/include -I/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m -c thriftpy/transport/cybase.c -o build/temp.macosx-10.14-x86_64-3.7/thriftpy/transport/cybase.o
thriftpy/transport/cybase.c:3127:24: error: no member named 'exc_type' in 'struct _ts'
tmp_type = tstate->exc_type;
~~~~~~  ^
thriftpy/transport/cybase.c:3128:25: error: no member named 'exc_value' in 'struct _ts'; did you mean 'curexc_value'?
tmp_value = tstate->exc_value;
^~~~~~~~~
curexc_value
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:240:15: note: 'curexc_value' declared here
PyObject *curexc_value;
^
thriftpy/transport/cybase.c:3129:22: error: no member named 'exc_traceback' in 'struct _ts'; did you mean 'curexc_traceback'?
tmp_tb = tstate->exc_traceback;
^~~~~~~~~~~~~
curexc_traceback
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:241:15: note: 'curexc_traceback' declared here
PyObject *curexc_traceback;
^
thriftpy/transport/cybase.c:3130:13: error: no member named 'exc_type' in 'struct _ts'
tstate->exc_type = local_type;
~~~~~~  ^
thriftpy/transport/cybase.c:3131:13: error: no member named 'exc_value' in 'struct _ts'; did you mean 'curexc_value'?
tstate->exc_value = local_value;
^~~~~~~~~
curexc_value
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:240:15: note: 'curexc_value' declared here
PyObject *curexc_value;
^
thriftpy/transport/cybase.c:3132:13: error: no member named 'exc_traceback' in 'struct _ts'; did you mean 'curexc_traceback'?
tstate->exc_traceback = local_tb;
^~~~~~~~~~~~~
curexc_traceback
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:241:15: note: 'curexc_traceback' declared here
PyObject *curexc_traceback;
^
thriftpy/transport/cybase.c:3154:24: error: no member named 'exc_type' in 'struct _ts'
tmp_type = tstate->exc_type;
~~~~~~  ^
thriftpy/transport/cybase.c:3155:25: error: no member named 'exc_value' in 'struct _ts'; did you mean 'curexc_value'?
tmp_value = tstate->exc_value;
^~~~~~~~~
curexc_value
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:240:15: note: 'curexc_value' declared here
PyObject *curexc_value;
^
thriftpy/transport/cybase.c:3156:22: error: no member named 'exc_traceback' in 'struct _ts'; did you mean 'curexc_traceback'?
tmp_tb = tstate->exc_traceback;
^~~~~~~~~~~~~
curexc_traceback
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:241:15: note: 'curexc_traceback' declared here
PyObject *curexc_traceback;
^
thriftpy/transport/cybase.c:3157:13: error: no member named 'exc_type' in 'struct _ts'
tstate->exc_type = *type;
~~~~~~  ^
thriftpy/transport/cybase.c:3158:13: error: no member named 'exc_value' in 'struct _ts'; did you mean 'curexc_value'?
tstate->exc_value = *value;
^~~~~~~~~
curexc_value
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:240:15: note: 'curexc_value' declared here
PyObject *curexc_value;
^
thriftpy/transport/cybase.c:3159:13: error: no member named 'exc_traceback' in 'struct _ts'; did you mean 'curexc_traceback'?
tstate->exc_traceback = *tb;
^~~~~~~~~~~~~
curexc_traceback
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:241:15: note: 'curexc_traceback' declared here
PyObject *curexc_traceback;
^
thriftpy/transport/cybase.c:3178:21: error: no member named 'exc_type' in 'struct _ts'
*type = tstate->exc_type;
~~~~~~  ^
thriftpy/transport/cybase.c:3179:22: error: no member named 'exc_value' in 'struct _ts'; did you mean 'curexc_value'?
*value = tstate->exc_value;
^~~~~~~~~
curexc_value
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:240:15: note: 'curexc_value' declared here
PyObject *curexc_value;
^
thriftpy/transport/cybase.c:3180:19: error: no member named 'exc_traceback' in 'struct _ts'; did you mean 'curexc_traceback'?
*tb = tstate->exc_traceback;
^~~~~~~~~~~~~
curexc_traceback
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:241:15: note: 'curexc_traceback' declared here
PyObject *curexc_traceback;
^
thriftpy/transport/cybase.c:3187:24: error: no member named 'exc_type' in 'struct _ts'
tmp_type = tstate->exc_type;
~~~~~~  ^
thriftpy/transport/cybase.c:3188:25: error: no member named 'exc_value' in 'struct _ts'; did you mean 'curexc_value'?
tmp_value = tstate->exc_value;
^~~~~~~~~
curexc_value
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:240:15: note: 'curexc_value' declared here
PyObject *curexc_value;
^
thriftpy/transport/cybase.c:3189:22: error: no member named 'exc_traceback' in 'struct _ts'; did you mean 'curexc_traceback'?
tmp_tb = tstate->exc_traceback;
^~~~~~~~~~~~~
curexc_traceback
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:241:15: note: 'curexc_traceback' declared here
PyObject *curexc_traceback;
^
thriftpy/transport/cybase.c:3190:13: error: no member named 'exc_type' in 'struct _ts'
tstate->exc_type = type;
~~~~~~  ^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
error: command 'clang' failed with exit status 1

Failed building wheel for thriftpy
Running setup.py clean for thriftpy
Successfully built happybase
Failed to build thriftpy
Installing collected packages: ply, thriftpy, happybase, kafka-python
Running setup.py install for thriftpy ... error
Complete output from command /usr/local/opt/python/bin/python3.7 -u -c "import setuptools, tokenize;file='/private/var/folders/0l/3k2zrhsn01b1sy7nktflll640000gn/T/pip-install-hy9tc71e/thriftpy/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /private/var/folders/0l/3k2zrhsn01b1sy7nktflll640000gn/T/pip-record-_zzfdmzn/install-record.txt --single-version-externally-managed --compile:
running install
running build
running build_py
creating build
creating build/lib.macosx-10.14-x86_64-3.7
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/server.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/hook.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/thrift.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/rpc.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/utils.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/tornado.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
copying thriftpy/_compat.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport
copying thriftpy/transport/_ssl.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport
copying thriftpy/transport/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport
copying thriftpy/transport/sslsocket.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport
copying thriftpy/transport/socket.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
copying thriftpy/protocol/binary.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
copying thriftpy/protocol/compact.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
copying thriftpy/protocol/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
copying thriftpy/protocol/exc.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
copying thriftpy/protocol/multiplex.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
copying thriftpy/protocol/json.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/protocol
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/parser
copying thriftpy/parser/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/parser
copying thriftpy/parser/parser.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/parser
copying thriftpy/parser/exc.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/parser
copying thriftpy/parser/lexer.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/parser
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/contrib
copying thriftpy/contrib/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/contrib
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport/memory
copying thriftpy/transport/memory/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport/memory
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport/buffered
copying thriftpy/transport/buffered/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport/buffered
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport/framed
copying thriftpy/transport/framed/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/transport/framed
creating build/lib.macosx-10.14-x86_64-3.7/thriftpy/contrib/tracking
copying thriftpy/contrib/tracking/init.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/contrib/tracking
copying thriftpy/contrib/tracking/tracker.py -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/contrib/tracking
copying thriftpy/contrib/tracking/tracking.thrift -> build/lib.macosx-10.14-x86_64-3.7/thriftpy/contrib/tracking
running build_ext
building 'thriftpy.transport.cybase' extension
creating build/temp.macosx-10.14-x86_64-3.7
creating build/temp.macosx-10.14-x86_64-3.7/thriftpy
creating build/temp.macosx-10.14-x86_64-3.7/thriftpy/transport
clang -Wno-unused-result -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk -I/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include -I/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/System/Library/Frameworks/Tk.framework/Versions/8.5/Headers -I/usr/local/include -I/usr/local/opt/openssl/include -I/usr/local/opt/sqlite/include -I/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m -c thriftpy/transport/cybase.c -o build/temp.macosx-10.14-x86_64-3.7/thriftpy/transport/cybase.o
thriftpy/transport/cybase.c:3127:24: error: no member named 'exc_type' in 'struct _ts'
tmp_type = tstate->exc_type;
~~~~~~  ^
thriftpy/transport/cybase.c:3128:25: error: no member named 'exc_value' in 'struct _ts'; did you mean 'curexc_value'?
tmp_value = tstate->exc_value;
^~~~~~~~~
curexc_value
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:240:15: note: 'curexc_value' declared here
PyObject *curexc_value;
^
thriftpy/transport/cybase.c:3129:22: error: no member named 'exc_traceback' in 'struct _ts'; did you mean 'curexc_traceback'?
tmp_tb = tstate->exc_traceback;
^~~~~~~~~~~~~
curexc_traceback
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:241:15: note: 'curexc_traceback' declared here
PyObject *curexc_traceback;
^
thriftpy/transport/cybase.c:3130:13: error: no member named 'exc_type' in 'struct _ts'
tstate->exc_type = local_type;
~~~~~~  ^
thriftpy/transport/cybase.c:3131:13: error: no member named 'exc_value' in 'struct _ts'; did you mean 'curexc_value'?
tstate->exc_value = local_value;
^~~~~~~~~
curexc_value
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:240:15: note: 'curexc_value' declared here
PyObject *curexc_value;
^
thriftpy/transport/cybase.c:3132:13: error: no member named 'exc_traceback' in 'struct _ts'; did you mean 'curexc_traceback'?
tstate->exc_traceback = local_tb;
^~~~~~~~~~~~~
curexc_traceback
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:241:15: note: 'curexc_traceback' declared here
PyObject *curexc_traceback;
^
thriftpy/transport/cybase.c:3154:24: error: no member named 'exc_type' in 'struct _ts'
tmp_type = tstate->exc_type;
~~~~~~  ^
thriftpy/transport/cybase.c:3155:25: error: no member named 'exc_value' in 'struct _ts'; did you mean 'curexc_value'?
tmp_value = tstate->exc_value;
^~~~~~~~~
curexc_value
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:240:15: note: 'curexc_value' declared here
PyObject *curexc_value;
^
thriftpy/transport/cybase.c:3156:22: error: no member named 'exc_traceback' in 'struct _ts'; did you mean 'curexc_traceback'?
tmp_tb = tstate->exc_traceback;
^~~~~~~~~~~~~
curexc_traceback
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:241:15: note: 'curexc_traceback' declared here
PyObject *curexc_traceback;
^
thriftpy/transport/cybase.c:3157:13: error: no member named 'exc_type' in 'struct _ts'
tstate->exc_type = *type;
~~~~~~  ^
thriftpy/transport/cybase.c:3158:13: error: no member named 'exc_value' in 'struct _ts'; did you mean 'curexc_value'?
tstate->exc_value = *value;
^~~~~~~~~
curexc_value
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:240:15: note: 'curexc_value' declared here
PyObject *curexc_value;
^
thriftpy/transport/cybase.c:3159:13: error: no member named 'exc_traceback' in 'struct _ts'; did you mean 'curexc_traceback'?
tstate->exc_traceback = *tb;
^~~~~~~~~~~~~
curexc_traceback
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:241:15: note: 'curexc_traceback' declared here
PyObject *curexc_traceback;
^
thriftpy/transport/cybase.c:3178:21: error: no member named 'exc_type' in 'struct _ts'
*type = tstate->exc_type;
~~~~~~  ^
thriftpy/transport/cybase.c:3179:22: error: no member named 'exc_value' in 'struct _ts'; did you mean 'curexc_value'?
*value = tstate->exc_value;
^~~~~~~~~
curexc_value
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:240:15: note: 'curexc_value' declared here
PyObject *curexc_value;
^
thriftpy/transport/cybase.c:3180:19: error: no member named 'exc_traceback' in 'struct _ts'; did you mean 'curexc_traceback'?
*tb = tstate->exc_traceback;
^~~~~~~~~~~~~
curexc_traceback
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:241:15: note: 'curexc_traceback' declared here
PyObject *curexc_traceback;
^
thriftpy/transport/cybase.c:3187:24: error: no member named 'exc_type' in 'struct _ts'
tmp_type = tstate->exc_type;
~~~~~~  ^
thriftpy/transport/cybase.c:3188:25: error: no member named 'exc_value' in 'struct _ts'; did you mean 'curexc_value'?
tmp_value = tstate->exc_value;
^~~~~~~~~
curexc_value
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:240:15: note: 'curexc_value' declared here
PyObject *curexc_value;
^
thriftpy/transport/cybase.c:3189:22: error: no member named 'exc_traceback' in 'struct _ts'; did you mean 'curexc_traceback'?
tmp_tb = tstate->exc_traceback;
^~~~~~~~~~~~~
curexc_traceback
/usr/local/Cellar/python/3.7.2_1/Frameworks/Python.framework/Versions/3.7/include/python3.7m/pystate.h:241:15: note: 'curexc_traceback' declared here
PyObject *curexc_traceback;
^
thriftpy/transport/cybase.c:3190:13: error: no member named 'exc_type' in 'struct _ts'
tstate->exc_type = type;
~~~~~~  ^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
error: command 'clang' failed with exit status 1
----------------------------------------

Command "/usr/local/opt/python/bin/python3.7 -u -c "import setuptools, tokenize;file='/private/var/folders/0l/3k2zrhsn01b1sy7nktflll640000gn/T/pip-install-hy9tc71e/thriftpy/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /private/var/folders/0l/3k2zrhsn01b1sy7nktflll640000gn/T/pip-record-_zzfdmzn/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /private/var/folders/0l/3k2zrhsn01b1sy7nktflll640000gn/T/pip-install-hy9tc71e/thriftpy/`

My problem is similar, it's something wrong with cityhash.
below are the terminal messages
 building 'cityhash' extension
    creating build
    creating build/temp.macosx-10.14-intel-2.7
    creating build/temp.macosx-10.14-intel-2.7/src
    cc -fno-strict-aliasing -fno-common -dynamic -g -Os -pipe -fno-common -fno-strict-aliasing -fwrapv -DENABLE_DTRACE -DMACOSX -DNDEBUG -Wall -Wstrict-prototypes -Wshorten-64-to-32 -DNDEBUG -g -fwrapv -Os -Wall -Wstrict-prototypes -DENABLE_DTRACE -arch i386 -arch x86_64 -pipe -Iinclude -I/System/Library/Frameworks/Python.framework/Versions/2.7/include/python2.7 -c src/city.cc -o build/temp.macosx-10.14-intel-2.7/src/city.o -O3 -Wno-unused-value -Wno-unused-function -msse4.2

and then follows with a lot of errors, obviously it can't import certain C language libraries
Command "/Users/Darius/.virtualenvs/frontera/bin/python -u -c "import setuptools, tokenize;__file__='/private/var/folders/ds/mm772gpj1zn9yy1sflxx_stw0000gn/T/pip-install-TmQYm7/cityhash/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /private/var/folders/ds/mm772gpj1zn9yy1sflxx_stw0000gn/T/pip-record-2gcw2r/install-record.txt --single-version-externally-managed --compile --install-headers /Users/Darius/.virtualenvs/frontera/include/site/python2.7/cityhash" failed with error code 1 in /private/var/folders/ds/mm772gpj1zn9yy1sflxx_stw0000gn/T/pip-install-TmQYm7/cityhash/```

Please help point out how to fix the problem, a lot of thanks!

The solution is to switch to thriftpy2. But it seems happybase was abandoned, and we will need either to monkey patch thriftpy or fork happybase and use it for Frontera. I'm more keen to fork it.

So, still stuck in <No module named 'thriftpy'>. Jun 16th, 2020

try master https://travis-ci.org/github/scrapinghub/frontera/builds/618649062, we have switched on thriftpy2 and tests are passing

Yes, it works by installing frontera directly from github:
pip install git+https://github.com/scrapinghub/frontera.git@master
Please make it available for installing it as described in the documentation.


Nav bar

Nav-bar padding and enhancement.

Kindly merge it with hacktoberfest-accepted label.

Could you also colorize the Hover Animations with the respective Navbar colors?

Sure .

Nice work @GauravP23 🎊


Modify example to make it reproducible

Thanks! Seems unnecessary to have folks download the dataset when a jsonl is included with the repo. What do you think about including the code for reproducibility but commenting it out?

Yeah, commenting it out sounds good!