subscribers aren't triggered when subscriptions are reloaded via figwheel

If I make a change to a sub, I have to reload the page or otherwise change the state in order for the sub to get triggered. Ideally, the subs should fire if they're reloaded.

I might need a bit more information.  Does this reloading involve figwheel?  If it does, what are doing in your on-jsload fucntion?

BTW, both re-frame examples now better demo a figwheel workflow.  But that cleanup is only in the  develop branch for the moment.

Indeed.
I had this, which may be old config:
(figwheel/watch-and-reload
  :websocket-url "ws://localhost:3449/figwheel-ws"
  :jsload-callback core/mount-root)

I still don't know what core/mount-root does.  If there is a problem it will be what you have done in there.
Please have a careful look at the various examples. Like this one:
https://github.com/reagent-project/reagent-template/blob/master/src/leiningen/new/reagent/src/cljs/reagent/core.cljs#L47-L48
Notice how mount-root completely re-renders from the top down.
(reagent/render [current-page] (.getElementById js/document "app")
I'm going to close this, because it is not really not a re-frame issue.

Ah, sorry, I thought that was from a re-frame template.
My mount-root is
(defn mount-root
  []
  (reagent/render [my.views/my-app] (.getElementById js/document "app")))

So yes, it rerenders the entire page, at least in theory. In practice, it doesn't even call my.views/my-app, likely because its result is cached. Note that everything works properly if the views.cljs file is the one changing, but if it's a file w/o the views, then the render is effectively a no-op.

After digging in though, you're right, it's not a reframe issue. I thought it had to do w/ the subscriptions, but it actually has to do w/ reagent/render doing its job. I'll check w/ reagent to see if there's a way to blow the cache.

For posterity, I tossed ^:figwheel-always on the views file namespace and that takes care of things.

It is puzzling that was needed.  As I understand it, you are modifying a subscription handler, right, and then mount-root is being called by figwheel.

So here's the real fix:
(defn mount-root
  []
  (reagent/render [#'my.views/my-app] (.getElementById js/document "app")))

Note the #'
See: https://github.com/reagent-project/reagent/blob/ff10d235911a60135d52c18fced9f071a7d8e366/src/reagent/core.cljs#L100-L104

Again, I'm puzzled as to why that would be a real fix.  You aren't calling force-update-all you are force rerendering the entire DOM - a sledge hammer (which is good).
For further background:
https://github.com/reagent-project/reagent/issues/94
https://github.com/reagent-project/reagent/issues/112

Here's a repro. You can ignore the piggieback stuff, just check the links. I'm guessing the problem has to do with memoized ratoms, but i haven't dug too deep.
https://github.com/aaronjensen/figwheel-plus-vim-fireplace

I ran into the issue with shadow-cljs, and while googling the solution I found this issue. Then I found re-frame.core/clear-subscription-cache! function in the source code, which should be on every code reloading while development before React re-rendering. Maybe it will save time for someone.
P.S. See also Why clear sub cache note.

@vganshin I also would suggest creating an app using re-frame-template and noticing the way that it structures things.


[SPARK-8146] DataFrame Python API: Alias replace in df.na

Merged build triggered.

Merged build started.

Test build #34383 has started for   PR 6688 at commit 774c19c.

Test build #34383 has finished for   PR 6688 at commit 774c19c.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class HasCachedBlocks(executorId: String) extends ToBlockManagerMaster

Merged build finished. Test PASSed.


rapids_export verify DOCUMENTATION and FINAL_CODE_BLOCK exist

When the user calls rapids_export with either of these parameters rapids-cmake will now validate that the variables exist, and error out when they don't

rerun tests

@gpucibot merge


Archived experiments

All-
I have successfully (locally) implemented archiving experiments from the dashboard.  It's pretty simple - you click to archive and it's archived in the config for the exp, that's it.  It also reads from the config for an exp so, if for some reason, you want to use the config instead of dashboard to archive an exp you can.  Not sure if thats a good idea or not so let me know but either way I THINK storing the archived project in the project config hash makes sense (im new to Redis, so let me know if you think a has called experiments_archived like the experiments_winner makes more sense)
Question - what exactly happens with archived projects that are still LIVE?  Right now they work the same as any other experiment, with the exception that they are archived.  So the exp doesnt appear on the main dashboard screen although its still "experimenting."  Any thoughts on this?  Any reason not to do this?
Thanks for the feedback.  Once finalized Ill submit a pull request.

Thanks for working on this @noctivityinc, I'll check out your branch over the weekend, feel free to open a PR now to make it easier to do inline code review.

Thanks for working on this @noctivityinc, I'll check out your branch over the weekend, feel free to open a PR now to make it easier to do inline code review.

Hey Andrew-
Definitely take a look but, as mentioned, Im not sure it has any real value since the view is basically the same as “winner selected” or “no winner selected” with the way the implementation was done.
Let me know your thoughts.
Joshua Lippiner
.:t<PHONE_NUMBER>
.:e<EMAIL_ADDRESS><EMAIL_ADDRESS>On Thursday, January 1, 2015 at 6:39 PM, Andrew Nesbitt wrote:

Thanks for working on this @noctivityinc (https://github.com/noctivityinc), I'll check out your branch over the weekend, feel free to open a PR now to make it easier to do inline code review.
—
Reply to this email directly or view it on GitHub (https://github.com/andrew/split/issues/284#issuecomment-68502451).

@noctivityinc I've taken a look at it, and as you said, there's not really much difference, so I don't think it's worth merging at the moment but thanks for investigating it.


Update console.profileEnd type definition

Fixes 24869

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: krzysztof-grzybek sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Xml optional arg serialization

Codecov Report
Patch coverage: 83.78% and project coverage change: -3.20% :warning:

Comparison is base (e3cdc10) 75.83% compared to head (e7eeead) 72.64%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
- Coverage   75.83%   72.64%   -3.20%
==========================================
  Files          31       32       +1
  Lines        3526     3549      +23
==========================================
- Hits         2674     2578      -96
- Misses        852      971     +119

Flag
Coverage Δ

unittests
72.64% <83.78%> (-3.20%)
:arrow_down:

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

Files Changed
Coverage Δ

guardrails/schema.py
81.75% <ø> (ø)

guardrails/utils/casting_utils.py
68.42% <68.42%> (ø)

guardrails/datatypes.py
71.87% <100.00%> (+0.39%)
:arrow_up:

guardrails/validators.py
48.64% <100.00%> (-11.01%)
:arrow_down:

... and 5 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


refactor(input-types): improve input API

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Store is based on abstract-chunk-store

This should be semver MAJOR, since users who pass in custom storage engines with opt.storage will need to update their storage backend to be abstract-chunk-store compliant.

That makes sense. For my use-case I pass in the pieceLength from the outside but that's probably not common.

We could easily support opts.store as well. I agree the .storage pattern is a bit annoying but unfortunately currently needed for the reasons @feross states.


Add Customizable Sidebar Feature

Currently, the sidebar in vscode provides quick access to different files and directories, but it can become cluttered for some users. I propose adding a customizable sidebar feature that allows users to organize the content based on their preferences. This could include options such as collapsing or expanding directories, hiding or showing certain files or directories, and customizing the order of files and directories in the sidebar. A customizable sidebar would provide a more personalized user experience and make it easier for users to navigate large projects.

what i can do new in this design as you think will look good ,any suggestions.

Do you have an example of a product that already does this?

examples
Windows file explorer: In Windows, the file explorer provides a sidebar that allows users to quickly access different files and directories. Users can customize the sidebar by sorting folders and files according to name, size, type, or date modified. They can also pin frequently accessed items to the top of the sidebar for quick access.
Atom text editor: Atom is a popular text editor that provides a customizable sidebar where users can organize content based on their preferences. The sidebar can be collapsed or expanded, and users can hide or show certain files or directories. Users can also sort items in the sidebar by name, type, or recent use, and can customize the order of items.
Sublime Text: Sublime Text is another popular text editor that provides a customizable sidebar. Users can collapse or expand directories, and can hide or show certain files or directories. They can also sort items in the sidebar by name, type, or usage frequency, and can customize the order of items.
isnt it will be better feature if its in vscode

\duplicate https://github.com/microsoft/vscode/issues/27286 https://github.com/microsoft/vscode/issues/164033


Allow newlines in computed binding argument list

This allows breaking in the middle of computed bindings, e.g.:
<div hidden$="[[_someVeryLongFunctionName(
                  someVeryLongArg,
                  anotherCrazyLongArg]]">

Note that [\s\S]* doesn't seem to have worse performance than .*.

Hey @mgiuffrida awesome spot!!
Would you mind adding a new unit test here?
https://github.com/Polymer/polymer/blob/36b9fe034f5e67820a8aaca9020120415397cd15/test/unit/bind-elements.html
Thanks a million!!! :sparkles:

@samccone Done and they pass locally in Cr and FF. PTAL, thanks!
Chrome still limits HTML to 80 chars in WebUI so being able to break up long lines is helpful.

awesome work @mgiuffrida this LGTM :+1:
ping @ebidel for :eyes:

Nice addition

@danbeam for FYI

I can merge this fix into #2580 if that helps.
Would be nice to get rid of the parsing oddities in computed bindings since they're frequently used in Polymer 1.0.
[[format(
    '[[D\'oh!]], "{0}" items left',
    items.count
)]]

There is even one more fix for this area: #2295

BTW, it looks like the intention in the original regular expression was to have a \s* before the ( like this:
var m = expression.match(/([^\s]+)\s*\(([\s\S]*)\)/);
otherwise why use [^\s]+ instead of [^\(]+ ?
Maybe the expression should be:
var m = expression.match(/([^\s]+?)\s*\(([\s\S]*)\)/);

or
var m = expression.match(/([^\s\(]+)\s*\(([\s\S]*)\)/);

If I am not mistaken, this was added in #3017 with \\s*

No, this still seems to be an issue: http://jsfiddle.net/mgiuffrida/x921yb0L/1/
As the expression has changed repeatedly since September, could someone from Polymer own this issue?

We'll take a look at merging this; just need to make sure it's compatible with the first-level regex change in https://github.com/Polymer/polymer/pull/3017/files#diff-870cfe064a36c23b69e8da891fdff6beR92.

I'll give it a shot tonight.

I've opened #3370 and rebased there, my original repo was deleted.

Okay, I think this is right now? I always screw up GitHub pull requests...

triggering new build

LGTM

Thanks @mgiuffrida


Set transport options on requests session

Fix for #20

Codecov Report

Merging #21 into master will decrease coverage by 1.73%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master     #21      +/-   ##
=========================================
- Coverage   89.03%   87.3%   -1.74%
=========================================
  Files           7       7
  Lines         310     315       +5
  Branches       30      33       +3
=========================================
- Hits          276     275       -1
- Misses         22      25       +3
- Partials       12      15       +3

Impacted Files
Coverage Δ

etcd3gw/client.py
85.06% <0%> (-3.53%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks @ThatIsGeek !


Is there any way to use stateful callbacks?

Judging from the source code, the callback is fixed as a single (function) pointer. As such, even C++11 lambda functions would not help.
Do you have any plans (and vision) on how to implement them? My analysis concluded, that the simplest and most efficient way to do it is to store the following struct
struct {
   void (* fptr)(void *);
   void* arg;
}

instead of just function pointer. Overload the operator() for such struct; and if the *arg==0, call the fptr without parameters.
What do you think?

Hi @adamryczkowski
I am always open to improving the functionality of TaskScheduler, given that it maintains small memory footprint and scheduling overhead. The primary target platform is microcontrollers, which [used to] have small amount of RAM to work with, so every byte and every tick counts.
That partially explains why everything about Tasks is so explicit - to minimize the time to get to callback functionality. Since the function pointer is explicit, and no parameters need to be pushed into stack, the callback happens relatively quickly.  Using overloaded functions will (in my opinion) add scheduling overhead, so I would like to understand what is the use case for your request?
If you would like to use the same callback code for different tasks, consider using the Local Task Storage pointer functionality. Every Task can store a pointer to its specific data structure. Callback can get a reference to current task from the scheduler and access task specific variables. There is a chapter about that in the documentation.
Hope this helps. But would be happy to continue the discussion in case I am missing some very useful way of using Tasks that is not achievable as is.
Thanks!
PS: Are you using TaskScheduler, and if yes, what for?

Thank you very much for such detailed answer.
No, I didn't notice the _TASK_LTS_POINTER. I think it is everything I need for now, I'll test it today.
I am looking for a good multitasking framework that will power my future Arduino projects. The first project at hand is my "smart bathroom":
There are 2 Arduinos in my bathroom. The main one is connected to an external CD74HC4067 LED PWM controller that powers powerful 4x4 led stripes (RGB, white and red), motion detector, hall effect based opened doors detector, TCS34725 colour detector (for calibration of LED colours), real time clock, a switch that controls floor heating pump, thermometer and another bathtub Arduino that controls 3 thermometers connected to my bath tub (to measure water temperature and - indirectly - water level), two extra LED stripes + one WS2812B vertical stripe with a nice animation of fire). I also want to test infra-red pilot for more control over the modes.
In the future, beside the autonomous work, The Arduino will be connected (serially via USB, so I could re-program Arduino remotely) with Raspberry Pi, that will do voice recognition and display a nice touchscreen with StarTrek-themed control panel :-).
So there is a loot of simultaneous work to do (lights, bathtub, heating and being managed by Pi).
The second project will be a sort of Jedi-saber for my son. It will be battery-powered, so I want to press for power-efficient design. By design the saber will have ca. 100 WS2812B neopixel stripe, accelerometer, a touch sensor and maybe a flex sensor. I want it to be capable of showing a nice picture when rotated, and show nice animation when touched/wiggled/bent.

Sounds very exciting!  Do you have a web page (or instructables page) to look at it?

Nope. Not yet. But thinking of it - why not? I'll try, but I cannot promise when. It is just that tinkering with electronics is only my hobby, and I barely find time for it.

Yeah. Finding time is the biggest problem... I can relate. Good luck!


KeyError in _structured_raw_data everytime a slash command being used

Everytime someone uses a slash command, I get this error message (but everything is working fine):
Ignoring exception in on_socket_response
Traceback (most recent call last):
File "C:\Users\unluc\AppData\Local\Programs\Python\Python38-32\lib\site-packages\discord\client.py", line 343, in _run_event
await coro(*args, **kwargs)
File "C:\Users\unluc\AppData\Local\Programs\Python\Python38-32\lib\site-packages\discord_components\client.py", line 80, in on_socket_response
ctx = self._get_context(res)
File "C:\Users\unluc\AppData\Local\Programs\Python\Python38-32\lib\site-packages\discord_components\client.py", line 342, in _get_context
data = self._structured_raw_data(json)
File "C:\Users\unluc\AppData\Local\Programs\Python\Python38-32\lib\site-packages\discord_components\client.py", line 310, in _structured_raw_data
if "components" not in raw_data["message"]:
KeyError: 'message'

Can I see some example code of one of your commands that cause this
Please use code blocks

Sure:
    @cog_ext.cog_slash(name = "qr", description = "Szöveg átalakítása QR kóddá", options = [{"name": "szöveg", "description": "A szöveg amit QR kóddá akarsz alakítani", "type": 3, "required": True}])
    async def qr(self, ctx, text): await fun.qr(ctx, text)````

Im gunna need more than that, that code wouldnt cause the issue
That traceback doesnt show anything to do with this library, only the d.py methods. Are you doing anything non-standard anywhere? Does the error occur with a simple ping command (ie await ctx.send("ping")). Is the code open on github?

I got this error even with this code:
@slash.slash(name = "teszt", description = "teszt")
async def teszt2(ctx):
    await ctx.send("a")

That isn't an issue with this library, that's an issue with the discord_components libary

Completely derped and thought discord_components was part of d.py. My bad.


Error when saving lua snippet file

So all of my snippets stopped working, I went to the github page and went to https://github.com/L3MON4D3/LuaSnip/blob/master/DOC.md#lua-snippets-loader
So I created a folder ~/.config/nvim/snippets/
Then I called in my luasnip config file:
require("luasnip.loaders.from_lua").load({paths="~/.config/nvim/snippets/"})

I created a simple all.lua file in that folder. Just like the docs:
ls = require("luasnip")
return {
	ls.parser.parse_snippet("ctrig", "also loaded!!")
}, {
	ls.parser.parse_snippet("autotrig", "autotriggered, if enabled")
}

But when I save the file, this happens:
Error detected while processing BufWritePost Autocommands for "/home/ubur/.config/nvim/snippets/all.lua":
E5108: Error executing lua ...ck/packer/start/LuaSnip/lua/luasnip/loaders/from_lua.lua:161: attempt to call field 'clean_invalidated' (a nil value)
stack traceback:
        ...ck/packer/start/LuaSnip/lua/luasnip/loaders/from_lua.lua:161: in function 'reload_file'
        [string ":lua"]:1: in main chunk

I quit NVIM then go back, the snippets works fine but every time I my snippet files the erroe just come up and I have to quit NVIM again.
Is there a way to fix this? Thank you!

Thank you!


Markdown node docs

Let's also add links to the line type documentation where applicable, e.g.:
Learn more about the JournalBasicFileInfo line type


[Mention The Feature]: Adding a document file in the create blog section !!

Is your feature request related to a problem? Please describe.

Describe the solution you'd like.
Enable users to upload document files in the blog creation section for enhanced content sharing.
Describe alternatives you've considered.
No response
Additional context.
No response
Show us the magic with screenshots
No response
Checklist

[X] I have checked the Existing Issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

@aditya-bhaumik , please assign me under this issue !!

Its been more than a week and no pull request was made so we are closing this issue.


feat: remove deprecated RestAPI provider

Description
Consume terraform-ibm-key-protect:v2.0.0, remove the deprecated RestAPI provider from the module.
Types of changes in this PR
No release required

[ ] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other changes that don't affect Terraform code

Release required

[ ] Bug fix (patch release (x.x.X): Change that fixes an issue and is compatible with earlier versions)
[ ] New feature (minor release (x.X.x): Change that adds functionality and is compatible with earlier versions)
[x] Breaking change (major release (X.x.x): Change that is likely incompatible with previous versions)

Release notes content
Remove the deprecated RestAPI provider requirement, if you are already consuming this module please upgrade to v3.1.0 or greater prior to consuming this release in order to avoid errors in the terraform state.

Checklist for reviewers

[ ] The PR references a GitHub issue.
[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Merge by using "Squash and merge".

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author.
The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).

:tada: This PR is included in version 4.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[BUG] - TypeError: value.toDate is not a function - DatePicker

NextUI Version
2.3.6
Describe the bug
I wanna edit an order so I need to show the current values of the order. The order has supplier, date, total and description. When I want to show the current date, nextjs raise the following error: 'TypeError: value.toDate is not a function'.
I get the current date from DRF API so the date format is datetime.datetime(2024, 6, 14, 0, 0, 0, 1000) and formData.date is: 2024-06-14T00:00:00.001000.
How can I fix it?
My code is:
`
const [formData, setFormData] = useState({
        supplier: '',
        date: '',
        total: '',
        description: ''
    });

            const response = await axios.get(`http://localhost:8000/api/orders/${id}/`, {
                headers: {
                    'Authorization': `Bearer ${Cookies.get('access_token')}`
                }
            });
            const orderDetails = response.data;
            setFormData({
                supplier: orderDetails.supplier,
                date: orderDetails.date,
                total: orderDetails.total,
                description: orderDetails.description
            });

                        <DatePicker
                            label='Fecha'
                            name='date'
                            value={formData.date}
                            showMonthAndYearPickers
                            onChange={handleDateChange}
                            classNames={{
                                calendarContent: 'min-w-[250px]'
                            }}
                        />

`
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Want to edit an order and show the current values.
Expected behavior
Normal behavior of DatePicker/
Screenshots or Videos

Operating System Version
macOS
Browser
Chrome

You need to use @internationalized/date to parse the formData.date before passing to value in DatePicker.
import {parseAbsoluteToLocal} from '@internationalized/date';

@wingkwong Error: Invalid ISO 8601 date time string:
There's a new error.

Sorry it should be parseDateTime.
import {parseDateTime} from '@internationalized/date';

- value={formData.date}
+ value={parseDateTime(formData.date)}

It's work! Thanks!


Add row to compare install size

Fixes #33
These badges will always use the latest version of typescript and flow-bin so you won't need to manually change the links each time.

I think it should be made more obvious that this is just the installed size of the command line tools, not actually affecting the runtime size of generated code.

@nickmccurdy addressed
@styfle thanks!


Don't waste your time with this

Loads of issues
[a] Conflicting PHP  versions, some of the addons what one extension and others another
[b] Uses many deprecated classes
[b] NPM won't compile. These are the errors I got:
npm WARN old lockfile
npm WARN old lockfile The package-lock.json file was created with an old version of npm,
npm WARN old lockfile so supplemental metadata must be fetched from the registry.
npm WARN old lockfile
npm WARN old lockfile This is a one-time fix-up, please be patient...
npm WARN old lockfile
npm WARN deprecated<EMAIL_ADDRESS>request-promise-native has been deprecated because it extends the now deprecated request package, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>Please see https://github.com/lydell/urix#deprecated
npm WARN deprecated<EMAIL_ADDRESS>this library is no longer supported
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>https://github.com/lydell/resolve-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.
npm WARN deprecated<EMAIL_ADDRESS>The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
npm WARN deprecated<EMAIL_ADDRESS>babel-eslint is now @babel/eslint-parser. This package will no longer receive updates.
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>CoffeeScript on NPM has moved to "coffeescript" (no hyphen)
npm WARN deprecated<EMAIL_ADDRESS>Critical security vulnerability fixed in v0.21.1. For more information, see https://github.com/axios/axios/pull/3410
npm WARN deprecated<EMAIL_ADDRESS>This version of tar is no longer supported, and will not receive security updates. Please upgrade asap.
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Please, upgrade your dependencies to the actual version of core-js.
npm ERR! code 1
npm ERR! path l:\laragon\www\core\node_modules\node-sass
npm ERR! command failed
npm ERR! command C:\WINDOWS\system32\cmd.exe /d /s /c node scripts/build.js
npm ERR! Building: C:\Program Files\nodejs\node.exe l:\laragon\www\core\node_modules\node-gyp\bin\node-gyp.js rebuild --verbose --libsass_ext= --libsass_cflags= --libsass_ldflags= --libsass_library=
npm ERR! gyp info it worked if it ends with ok
npm ERR! gyp verb cli [
npm ERR! gyp verb cli   'C:\Program Files\nodejs\node.exe',
npm ERR! gyp verb cli   'l:\laragon\www\core\node_modules\node-gyp\bin\node-gyp.js',
npm ERR! gyp verb cli   'rebuild',
npm ERR! gyp verb cli   '--verbose',
npm ERR! gyp verb cli   '--libsass_ext=',
npm ERR! gyp verb cli   '--libsass_cflags=',
npm ERR! gyp verb cli   '--libsass_ldflags=',
npm ERR! gyp verb cli   '--libsass_library='
npm ERR! gyp verb cli ]
npm ERR! gyp info using<EMAIL_ADDRESS>npm ERR! gyp info using<EMAIL_ADDRESS>| win32 | x64
npm ERR! gyp verb command rebuild []
npm ERR! gyp verb command clean []
npm ERR! gyp verb clean removing "build" directory
npm ERR! gyp verb command configure []
npm ERR! gyp verb check python checking for Python executable "python2" in the PATH
npm ERR! gyp verb which failed Error: not found: python2
npm ERR! gyp verb which failed     at getNotFoundError (l:\laragon\www\core\node_modules\which\which.js:13:12)
npm ERR! gyp verb which failed     at F (l:\laragon\www\core\node_modules\which\which.js:68:19)
npm ERR! gyp verb which failed     at E (l:\laragon\www\core\node_modules\which\which.js:80:29)
npm ERR! gyp verb which failed     at l:\laragon\www\core\node_modules\which\which.js:89:16
npm ERR! gyp verb which failed     at l:\laragon\www\core\node_modules\isexe\index.js:42:5
npm ERR! gyp verb which failed     at l:\laragon\www\core\node_modules\isexe\windows.js:36:5
npm ERR! gyp verb which failed     at FSReqCallback.oncomplete (node:fs:193:21)
npm ERR! gyp verb which failed  python2 Error: not found: python2
npm ERR! gyp verb which failed     at getNotFoundError (l:\laragon\www\core\node_modules\which\which.js:13:12)
npm ERR! gyp verb which failed     at F (l:\laragon\www\core\node_modules\which\which.js:68:19)
npm ERR! gyp verb which failed     at E (l:\laragon\www\core\node_modules\which\which.js:80:29)
npm ERR! gyp verb which failed     at l:\laragon\www\core\node_modules\which\which.js:89:16
npm ERR! gyp verb which failed     at l:\laragon\www\core\node_modules\isexe\index.js:42:5
npm ERR! gyp verb which failed     at l:\laragon\www\core\node_modules\isexe\windows.js:36:5
npm ERR! gyp verb which failed     at FSReqCallback.oncomplete (node:fs:193:21) {
npm ERR! gyp verb which failed   code: 'ENOENT'
npm ERR! gyp verb which failed }
npm ERR! gyp verb check python checking for Python executable "python" in the PATH
npm ERR! gyp verb which succeeded python C:\Python39\python.EXE
npm ERR! gyp ERR! configure error
npm ERR! gyp ERR! stack Error: Command failed: C:\Python39\python.EXE -c import sys; print "%s.%s.%s" % sys.version_info[:3];
npm ERR! gyp ERR! stack   File "", line 1
npm ERR! gyp ERR! stack     import sys; print "%s.%s.%s" % sys.version_info[:3];
npm ERR! gyp ERR! stack                       ^
npm ERR! gyp ERR! stack SyntaxError: invalid syntax
npm ERR! gyp ERR! stack
npm ERR! gyp ERR! stack     at ChildProcess.exithandler (node:child_process:326:12)
npm ERR! gyp ERR! stack     at ChildProcess.emit (node:events:369:20)
npm ERR! gyp ERR! stack     at maybeClose (node:internal/child_process:1067:16)
npm ERR! gyp ERR! stack     at Process.ChildProcess._handle.onexit (node:internal/child_process:301:5)
npm ERR! gyp ERR! System Windows_NT 10.0.19042
npm ERR! gyp ERR! command "C:\Program Files\nodejs\node.exe" "l:\laragon\www\core\node_modules\node-gyp\bin\node-gyp.js" "rebuild" "--verbose" "--libsass_ext=" "--libsass_cflags=" "--libsass_ldflags=" "--libsass_library="
npm ERR! gyp ERR! cwd l:\laragon\www\core\node_modules\node-sass
npm ERR! gyp ERR! node -v v15.12.0
npm ERR! gyp ERR! node-gyp -v v3.8.0
npm ERR! gyp ERR! not ok
npm ERR! Build failed with error code: 1
npm ERR!      C:\Users\Kingdel\AppData\Local\npm-cache_logs\2021-09-11T17_11_22_080Z-debug.log

Do you have python2 installed?
I would do:

nvm use 14 (optional)

... I have wasted 3 hours for this... Should not use this project.... :-( Thanks for nothin...

The real issue are sometime behind the keyboard.
Seriously, nodes and his package manager NPM are actually hard to debug but you should atleast compare version.
But i bet this is because of your Node version. Try downgrade to 12.
Just use NVM and set it to v12 for try.

this fix works :D


ArgumentError: comparison of NilClass with<PHONE_NUMBER>5746 failed

When running rails db:generate_trigger_migration or running HairTrigger::migrations_current? in the console, I get the following:
/home/bkke/.rvm/gems/ruby-3.1.2/gems/hairtrigger-0.2.25/lib/hair_trigger.rb:92:in `sort_by': comparison of NilClass with<PHONE_NUMBER>5746 failed (ArgumentError)

It appears that the regex at hair_trigger.rb:85 fails to match my schema file:
[11] pry(HairTrigger)> previous_schema =~ /ActiveRecord::Schema\.define\(.*?(\d+)\)/
=> nil

While previous_schema variable has [7.0] in the name
[10] pry(HairTrigger)> previous_schema.lines.grep /ActiveRecord/
=> ["ActiveRecord::Schema[7.0].define(version: 2022_12_10_192312) do\n"]

My Ruby version is 3.1.2, Rails version is 7.0.4. HairTrigger version (both HairTrigger::VERSION and gem info hairtrigger) is 0.2.25

nvm, saw a fix in #106


Production mode compilation throws socket.io 404 error

I compiled my app.jsx file using the command, webpack -p (webpack versioned 1.8.9). And i tried to load my index.html page using a node-static server. Webpack now bombards my shell terminal with warning messages as,
20:28:48 [404]: /socket.io/?EIO=3&transport=polling&t=1437749928591-2656
...............................(repetitively prints same stuff)

My webpack.config.js looks like,
var webpack = require('webpack');
var path = require('path');

module.exports = {
  entry: {
    'app.js': './client/app.jsx'
  },
  output: {
    filename: path.join('dist', 'app.js')
  },
  resolve : {
    extensions: ['', '.js', '.jsx']
  },
  module: {
    loaders: [
      {test: /\.jsx$/, loaders: ['babel']},
    ],
  }
};

some installation has made this error disappear.. so, i am closing it now.. sorry, folks...


feat: rate limiter

Summary:
rate limiter
Build & Testing:

[x] I ran cargo test successfully.
[x] I have run ./lint.sh --mode=fix to fix all linting issues raised by ./lint.sh --mode=check.

Checklist:

[ ] I have added relevant unit & integration tests.
[ ] I have updated the documentation accordingly.
[ ] I have performed a self-review of my code.
[x] PR follows the naming convention of <type>(<optional scope>): <title>

Closing this PR for now. This needs a lot of work in terms of design to be ready for production usage.


Adding details about Boolean Data types and FROM subitems

added a couple little things to change

Thank you @zfoster


change advective edmf horizontal advection

Purpose

This makes the horizontal advection more consistent with the vertical advection.
Closes #2255
To-do

Content

[ ] I have read and checked the items on the review checklist.

bors r+

bors r+


Decode Block Scrypt bug Added return thash; at line 56

Scrypt - Block cant be decoded #15 issue

\but\src\primitives\block.cpp at line 58 added, return thash; that was missing

Main scrypt algo would be in service from block height 170,000

Awesome mate!


item input

Mekanism-1.7.10-<IP_ADDRESS>
I can't insert items (ores) in mekanism's machines by using pipes. any sort of pipe, from vanilla hopper to te itemducts nor mekanism logistical transporter, nor ender io item conduits, nor me export bus.
I correctly configured faces and tryed to use different sides but items don't go inside the machine (tested on purification chamber and crusher) output works well and machine-to-machine input/output workd

This is fixed in development.

please can I have the url where get it?

@zortag Dev builds are for donators only.
On that note: A new public release just went out about 30min ago.

thanks to explaining me that


most of the settings don't work anymore

The last built seems broken. Most of the settings don't work anymore like Volume, Locale, Temperature Units, Button Action, Set Location, and Set Time Zone.
I have completely reinstalled your original branch.

Try going to :8080/sdkapp and running Ctrl+Shift+R (CMD+Shift+R on Mac) on the main webpage to redownload all of the scripts and such, they may be caching

I have already done that. I have completely re-downloaded the sources and gone through the full istallation process. I have also tried different browsers on different machines.

Do any errors appear in the wire-pod logs or the browser log?

no, the green dot just jumps back to the previous setting, if I try to change something.

And I'm not sure if it's related, but Vector crashes with error code 915 when I try set location

What device is wire-pod running on? May you pull logs from the robot and get them to me? (can be done by heading to https://www.project-victor.org/noflow-devsetup, connecting the robot, and entering logs in the console)

Its running on an RaspberryPi 4B with 4GB Ram.
Vectors log is attached
vector-logs-2023-02-04-08-54-06.tar.zip

I also Raspberry Pi 4B with 4G ram, I can confirm it works perfectly without any issues. I am able to change any of Vector’s settings without any issues.

I tired to get back to DDL servers and restored the original server_config.json on Vector. When trying to set up Vecot via the Android app, the error 923 came up. neither a reboot nor the complete new setup via wire-pod works now. After a few seconds in which Vector shows his eyes, appears again the error 923.

It seems that the problem was indeed with my Vector. I flattened everything again and installed the latest OSKR firmware 1.8.1. Now everything works again as it should. Please excuse the confusion

It seems that the problem was indeed with my Vector. I flattened everything again and installed the latest OSKR firmware 1.8.1. Now everything works again as it should. Please excuse the confusion

Hello,
I am experiencing same 923 error. How did you loaded latest OSKR firmware 1.8.1?

I used the EscapePod from DDL to reinstall the latest OSKR firmware. There is the possibility to upload .ota files and reinstall Vector with them.
But maybe it's really just a buggy version of vic-cloud or vic-gateway.... (?)

I used the EscapePod from DDL to reinstall the latest OSKR firmware. There is the possibility to upload .ota files and reinstall Vector with them. But maybe it's really just a buggy version of vic-cloud or vic-gateway.... (?)

I think so... Maybe I have to do it with latest EscapePod?
Do you have .OTA of 1.8.1? If so, can you share file/link, please?

I think that the OSKR firmware is hard linked to the serial number of the Vector, so it is not possible to share it. My .ota would probably not work for you.

Have you tried contacting DDL to have them resend you the latest 1.8.1 OSKR image für your robot? Steve Coblentz is always very cooperative on Facebook.

@Hamlet3000 Thanks. I sent a message to him. Did you ask help through facebook directly?

@iPsych yes, I contacted Steve directly through Facebook. He usually replies within a day


Add trailers

This PR adds Trailers support.
Fix #2266
This implementation is based on http2 trailers, so to send a trailer, you must:

announce Trailer in the Trailer header (before WriteHeader) and then (after WriteHeader) add your Trailer value in ResponseWriter.Header ()
or you can use TrailerPrefix, by simply adding headers (after WriteHeader) with the TrailerPrefix (Trailer:)

I could not test it with another server / client implementation.

Codecov Report

Merging #2344 into master will decrease coverage by 0.04%.
The diff coverage is 73.21%.

@@            Coverage Diff            @@
##           master   #2344      +/-   ##
=========================================
- Coverage   85.94%   85.9%   -0.04%
=========================================
  Files         109     110       +1
  Lines        8151    8236      +85
=========================================
+ Hits         7005    7075      +70
- Misses        863     873      +10
- Partials      283     288       +5

Impacted Files
Coverage Δ

http3/client.go
85.94% <0%> (ø)
:arrow_up:

http3/server.go
64.76% <100%> (+0.16%)
:arrow_up:

http3/body.go
90.16% <58.33%> (-9.84%)
:arrow_down:

http3/response_writer.go
77.27% <78.57%> (+1.19%)
:arrow_up:

internal/congestion/rtt_stats.go
91.11% <0%> (-3.76%)
:arrow_down:

server.go
84.67% <0%> (-2.24%)
:arrow_down:

internal/handshake/qtls.go
88.31% <0%> (ø)
:arrow_up:

config.go
100% <0%> (ø)

internal/handshake/client_session_cache.go
96.83% <0%> (+6.58%)
:arrow_up:

Continue to review full report at Codecov.

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

@marten-seemann are there any plans to include trailers into h3 package?
Having this support is a must have to proxy gRPC, and it would be nice to have it.
This PR is already a too old, so, not sure if it should start from scratch.
If there are not plans, I can try and open a PR for this in the near future. Let me know if that would be ok.

@marten-seemann are there any plans to include trailers into h3 package?

It would definitely be a nice-to-have. I'm not actively working on this, but if you want to submit a PR, that would be great!


Downloading training data twice

Currently we are downloading background data twice: Once during the background project, and a second time when querying glitch data. We can use mldatafind to point to the background file directory when querying glitch data

Won't the glitch data generally be a superset of the training background data?


[help/question] Assigning a project to an issue (create or update)

Hi,
I can create/update an issue using this package.
However, I can't find how to assign a project to an issue, like here:

I looked in your doc & github's doc, no solution found so far.
Any idea? I don't mind implementing myself if it's possible but not supported yet.
Thanks.

Interesting, I don't know the answer to this, either... maybe it's not documented yet by GitHub.

@26medias all the columns in a project seem to have identifiers that are unique across Github; if you use createProjectCard with the appropriate content_type and content_id that should assign your issue to the project.  However, I'm failing doing just that as described in #531.

Hey there, if you are still interested in this, the https://platform.github.community/ forum would be a good place to ask about it :)


theme-dark for smart-select

Is your feature request related to a problem? Please describe.
There is no option for the smart-select component to have a dark theme, and it does not inherit a dark theme if it's parent view has the theme-dark option
Describe the solution you'd like
A theme-dark option for the smart-select component

Solved by adding theme-dark as cssClass option in smart-select-params


Added HTMLMetaElement support.

Signed-off-by: Egor Bolgov<EMAIL_ADDRESS>

Thanks ! Can you rebase or merge from master ?

Ok


To create a stunning README.md for our repository

Problem: Our current README.md is plain & has little details. We need a more attractive README.md document.
Solution: Use your creative juices to spruce up our README.md documentation that attract users to join our open source project!
Expectation: To create a stunning README.md with proper markdown formatting that attracts users to join & contribute to our open-source project! Include images as well - e.g. such as the Hacktoberfest 2022 logo/banner. You can refer to other repositories for the general format.
Take note: You can take reference from the README.md of other repositories. But DO NOT PLAGIARIZE or else your PR would be labeled as spam & disqualified from Hacktoberfest 2022.
This is a very beginner-friendly task that allows you to practice your Markdown skills, and contribute to a meaningful cause in a low to no-code manner!

Leave a comment here & tag/ping me if you wish to pick up this task!

Please assign me this issue

@Prakhar9450 Hi Prakhar, assigned it to you!
Surprise me with a stunning README.md, good luck 😊

@Prakhar9450 Hi Prakhar, how is the PR going?

@melvincwng Would this repo need an how to contribute guide?

Hi @Bennykillua, currently I think @Prakhar9450 should be working on the code repository's main README.md.
Not exactly sure what will be the contents of the main README.md, it may or may not contain the contribution guidelines.
Let's wait for his PR first, if it doesn't include the contribution steps, I'll create another issue for you Benny to work on our contribution steps.

Please merge my pr #52

@Prakhar9450 Kindly refrain from plagiarizing from other repository's README.md.
We are not accepting console.log statements/scripts as contributions to this codebase.
This is a community driven website that only accepts useful PRs or enhancements to our website.

@Bennykillua Would you be keen to write our code repository's main README.md?
Include images from Hacktoberfest2022, write a contribution guide (that teaches user how to contribute & make actual meaningful enhancements - not unnecessary console.log scripts) and other details in general.

@melvincwng Yes I can. Just have a quick question like:

what is the repo about? like what is the focus? this will help me write the about me session and answer how someone can contribute.

@Bennykillua Hi Benny, thx for taking up this task.
Will assign it to you.
Yes, that repo you linked, is related to this repo.
In general, this code repository is all about providing a safe space for programmers of all skill levels to submit a PR for Hacktoberfest 2022. Have a specific issue or feature that you want to add to this project? Just submit a PR and if it's valid, it will be merged and considered for Hacktoberfest.
We value actual useful meaningful contributions, not spammy PRs. E.g. I set up the core features of our webpages, @GabrielWarner  is porting over all 89+ members to our webpage, @Bennykillua is writing good documentations for our codebase and @flawlesslys set up the automation pipeline and @theadityaagnihotri helped to fix bugs/minor issues.
In terms of content wise, I'll leave it to your experimentation and interpretation of what I've said. As long as the content is unique, and encompasses the essence of submitting good and valid PRs for Hacktoberfest2022, I'll accept it.
Look forward to your PR

@Bennykillua, pls also insert images & try to make our README.md beautiful.
You can use this example repository as a source of reference/inspiration - https://github.com/tanyagupta0201/Hacktoberfest-2022
But kindly no plagarism, thanks


Use a static port number

After experiencing some confusion, and then seeing this issue, it seems that the PR to change the port from 8080 (the default) to 0 (which means it will be chosen randomly) did more harm than good.
Choosing 12997 due to google searches not turning up anything registered for that port. Or, we could just change it back to use 8080, which yes some people will have issues because it is a fairly common port for people playing around with Python and local docker services and what not. In any case something needs to change in the quickstart docs which do not seem to be in any public GitHub 😀
Cheers!

@googlebot I signed it!

Thanks, but given there were equally as many issues with using a fixed port number I'm going to decline this.
Ephemeral ports for localhost oauth callbacks are the preferred method. From the referenced issue, it looks like the root cause is people are misconfiguring the oauth credentials for the quickstart. That's something that can be addressed in the instructions for the quickstart, rather than re-introducing a bug.

Thanks. I'm all for a solution that updates the docs to provide a working solution. I currently don't understand how it could be possible to specify a callback URL in my Google Account that doesn't provide a port number, so I'm looking forward to seeing how such a thing works.


Add upper bound for pendulum library

Pendulum was recently updated to 3.0, and this was a breaking change causing python tests in python-lyft-etl to fail. We want to set an upper bound for pendulum to prevent this.
Context: https://lyft.slack.com/archives/CA4HS7D8V/p1703024373835889?thread_ts=1702924956.145019&cid=CA4HS7D8V

PTAL @lyft/dpe-airflow, @eschachar

🚀


Admin Stack: Show Prompt if navigating into a nested Stack, even if subRoutes would normally not show Prompt

Bildschirmaufzeichnung vom 2024-04-17, 18:03:01.webm
This adds:

a new ForcePromptRoute, to be used like react-router Route, that also registers it's path in a new context created by RouterPrompt
PromptHandler can now check if the navigating-to-url matches with a path from registered ForcePromptRoute
and show the Prompt

Also adjust StackSwitch:

to use the new ForcePromptRoute
don't use react-router Switch as that would only render the first matching route (but we need ForcePromptRoute to always render so it can register)
instead implement the logic from Switch by only rendering index if no other route matched

@thomasdax98 I believe this is necessary for the stable v7 release.


I cannot set environment variable

I am trying to set the following environment variable with its value:
PAPERLESS_OCR_USER_ARGS={"invalidate_digital_signatures": true}
I need to import many digitally signed documents, so I need to tell the system to just invalidate the signatures. If I have properly understood from my readings, this will keep also a copy of the original, so I should be able to consult my documents and still keep them valid.
Any way, I was trying to set the environment variable with the -e option in you start.sh script, but I am not able to make it work.
I am just adding the line within the Starting paperless block. What am I doing wrong for it to not set the environment variable?

Try setting the env this way under https://github.com/jdoss/ppngx/blob/master/start.sh#L92
-e PAPERLESS_OCR_USER_ARGS="{'invalidate_digital_signatures': true}"

Nope, it does not seem to work. It still throws the following error:
DigitalSignatureError: Input PDF has a digital signature. OCR would alter the document, invalidating the signature.
I am not really sure how that JSON? metadata may be passed through and environment variable, and that is why I am out of clues on how to deal with it.

After close inspection, your way of passing the variable is correct, in the sense that within the container the environment variable is set. The problem is that Paperless does not seem to be acting on it.

That is odd. Is there anything on the Paperless NGX Github issues about PAPERLESS_OCR_USER_ARGS not working?

I have been checking paperless-ngx Issues and have not found anything about it, BUT playing with variations of the format you suggested, I finally made it work. I used the following syntax:
-e PAPERLESS_OCR_USER_ARGS='{"invalidate_digital_signatures": true}'
Thank you very much for your help

No problem! Glad it got sorted out. 😄


Addresses outside the VNET range.

Can you explain a little more what the "Kubernetes service address range " and "Docker Bridge address" are for and what they do?  The subnet for the VNET contains all the node/pod ips, but the example is assigning a lot of IPs to those other 2 IP ranges.  Also, do I need to worry about those IP ranges (service and bridge) colliding with other VNETs I have or IP address I use on-prem?  I'm planning to Peer the AKS VNET to one that has a site-to-site VPN connection.  Thanks.

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

ID: 22ad4873-c73d-4b90-2063-927a1060d2a2
Version Independent ID: 8e6643ad-7333-a786-8712-3d17291e12a7
Content: Network configuration in Azure Kubernetes Service (AKS)
Content Source: articles/aks/networking-overview.md
Service: container-service
GitHub Login: @mmacy
Microsoft Alias: marsma

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

This part of the docs definitly needs a brush up. Do we need to create another VNET/Subnet for the Kubernetes services subnet? Do we need to create a VNET/subnet for the Docker Bridge?
Please also add an example. Thank you.

The Kubernetes service address range is the range from which IP addresses are assigned to Kubernetes services (https://kubernetes.io/docs/concepts/services-networking/service/ describes what a service is). Both these IP ranges should be outside the IP range of the VNet in which you are deploying your cluster. They should also not overlap with any other VNets with which this VNet peers. They should also not overlap with on-prem IPs. The reason to avoid overlap is that if a pod tries to access an IP outside the cluster and if that IP happens to be a service IP then the behavior is unpredictable.
I will make sure the documentation is updated with this info. Thanks!

Thanks, Aanand, appreciate it. For the IP ranges on these things, can they be any private range (10.x, 192.168.x, 172.x), and if so (or not) what reasoning would a customer have for using a specific range?
Sent from Outlookhttp://aka.ms/weboutlook

From: Aanand Ramachandran<EMAIL_ADDRESS>Sent: Tuesday, June 12, 2018 10:13 AM
To: MicrosoftDocs/azure-docs
Cc: Marsh Macy; Assign
Subject: Re: [MicrosoftDocs/azure-docs] Addresses outside the VNET range. (#10049)
The Kubernetes service address range is the range from which IP addresses are assigned to Kubernetes services (https://kubernetes.io/docs/concepts/services-networking/service/https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fkubernetes.io%2Fdocs%2Fconcepts%2Fservices-networking%2Fservice%2F&data=02|01|marsma%40microsoft.com|adff48b5dc604f0cc7e708d5d087dc32|72f988bf86f141af91ab2d7cd011db47|1|0|636644204296227835&sdata=c1p9eDy4ubdGCUKSspFTIZ7pN%2B8azLF0Q1Bfk%2FZuqZw%3D&reserved=0 describes what a service is). Both these IP ranges should be outside the IP range of the VNet in which you are deploying your cluster. They should also not overlap with any other VNets with which this VNet peers. They should also not overlap with on-prem IPs. The reason to avoid overlap is that if a pod tries to access an IP outside the cluster and if that IP happens to be a service IP then the behavior is unpredictable.
I will make sure the documentation is updated with this info. Thanks!
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F10049%23issuecomment-396665952&data=02|01|marsma%40microsoft.com|adff48b5dc604f0cc7e708d5d087dc32|72f988bf86f141af91ab2d7cd011db47|1|0|636644204296237840&sdata=s2oy8%2F5G8lCK%2Bf2AOH%2FgydAy6PBrblPkijAhZcvIAsw%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMxibdVgS18Hb1uHH-lWEK3zMwyxxFraks5t7_bLgaJpZM4UjXR8&data=02|01|marsma%40microsoft.com|adff48b5dc604f0cc7e708d5d087dc32|72f988bf86f141af91ab2d7cd011db47|1|0|636644204296237840&sdata=PTaNGrzTAqckTXZwIdfOb85cmnVSJSyBrZeiNhkZOnU%3D&reserved=0.

All that information and the link to kuernetes docs is very useful.  I set up AKS with advanced networking with the defaults just to see what I got.  Looking through both Resource groups that get created, I can't find any references in either one as to what IP addresses were used for the "Kubernetes service address range " and "Docker Bridge address".  Once I've created these things, how do I go back and check to see what IP ranges were use?  In general I'd want to be able to use Powershell to scrape all IP ranges in a tenant space, or group of resources groups, etc to make sure I don't end up deploying overlaps.

@travis-sobeck - did you check the health blade under your cluster? It displays Pods, nodes etc. and may also show services. I am having some issues with my cluster so not able to confirm. Another option is to look inside the cluster, using kubectl. For e.g. you could list all services in a namespace using "kubectl get services -n ". In the output you will see the IP address of the services which will be from the configured range.
If you want to only check the configured range then its available in portal and i think you may be able to get it in CLI too.

@aanandr I'll poke around today.  I scrapped my Adv Networking instance, waiting on my networking team to get some blocks of IPs I can use for a prod setup and avoid IP collisions.  They want a meeting .. talk about stuff .. :)


THUMOS14 dataset dataset files for Diagnosing Error

Hi~ In the README you mentioned "We provide augmented dataset files for the ActivityNet v1.3 (validation) and THUMOS14 dataset in the data folder. " but I cannot find files related to THUMOS14. Since I am doing experiments on THUMOS14 and it will be of great help to me to diagnose error for my results, could you please share the files related to THUMOS14 dataset?

Thanks for your interest in our DETAD work. You can find the THUMOS14 files in https://github.com/HumamAlwassel/DETAD/tree/master/data
Cheers!


Bug: Requires Return Section in Docstring When None Is the Return Type

I noticed that docstrings require a return section even when the return type is None, but this only occurs when there are parameters in the function. The error does not occur if there are zero parameters in the function. Also, the docstrings use Google style.
Example (Error):
def test(text: str) -> None:
    """Run test.

    Args:
        text (str): Text for the function
    """

Example (Pass):
def test() -> None:
    """Run test.

    """

Hi @rbebb ,
Thank you for reporting this issue.

docstrings require a return section even when the return type is None

I implemented this check because this seems to be the convention of the numpy-style docstring.
I can add a configurable option that allows people to not add a "returns" section when the function returns None (implicitly or explicitly).

but this only occurs when there are parameters in the function

Yep, this is an intended behavior.
By pydoclint's default, the option --skip-checking-short-docstrings is set to True.  This allows people to not put any "args" or "returns" in the docstring, even if the function signature has args or returns.
But as soon as you add an Args: section in your docstring, this docstring no longer is a "short docstring", which is why pydoclint starts to check the existence of a "Returns" section.

Hi @rbebb , I've fixed this issue in a PR.  This fix will be shipped with the newest release in a day or two.

Hi @rbebb , I just published version 0.0.5, which includes this fix.


Each module should have its own installation requirements

This will make dependency management easier in the future, e.g. pip install chirho[interventional,indexed]

Looking into this a bit more it's not obvious that there's anything to be done for existing modules, as they all appear to only depend on pyro. Instead, it seems like the thing to do would be to module options if and only if those modules require additional dependencies. torchdiffeq in the dynamical module in the now revised #299 is a good example.

Closing as I believe this is addressed by my comment above. Feel free to reopen if you believe this was closed by mistake.


The first command takes a long time without feedback

I tried garn enter --help, and it hung for a while before continuing, without any feedback. And I've seen something similar in user tests.
After the first time it's much faster.

On a bad network connection, this has already taken 5 minutes!

It now finished. Part of the problem seems to be gomod2nix:
nix path-info -Sh 'github:nix-community/gomod2nix?rev=f95720e89af6165c8c0aa77f180461fe786f3c21'
/nix/store/yvzwmzwdkbign8h0433m0hvzy01hpzxf-gomod2nix-dev	557.5M

But additionally, I think we should have a helper function that, every time we run a command, logs what the command being run is, if it takes longer than, say, 3 seconds.

Maybe for all these commands that fetch things, we should first check whether they're installed already. If not, log that we're installing them, that we're done installing them, and then run them.


[NO-ISSUE] Improve cert gen in tests

Generate tls secret without using static keystore/trustore files
Reuse the tls secret generation code in tests

we need to find a place to include the license of the dependency, see the binary distro requirement at: https://pkg.go.dev/software.sslmate.com/src/go-pkcs12?tab=licenses

@gtully @brusdev
How about this location: https://github.com/artemiscloud/activemq-artemis-operator/pull/587/commits/f40a98b90ee66e7ac9dc64f73423fb24ce123ba2
I see k8s has a folder LICENSES however I don't know whether vendor or third_party is a proper category for us. I use dependencies instead. wdyt?

I would move the licenses folder in the deploy folder because we should include it in the distribution package. ActiveMQ Artemis does a similar thing, see https://github.com/apache/activemq-artemis/tree/main/artemis-distribution/src/main/resources/licenses/bin

I would move the licenses folder in the deploy folder because we should include it in the distribution package. ActiveMQ Artemis does a similar thing, see https://github.com/apache/activemq-artemis/tree/main/artemis-distribution/src/main/resources/licenses/bin
good point. I'll move the folder into deploy. thanks!

I would create an issue to have a tracker for the new license.

@brusdev PR updated. Pls review again. thx.


func invoke handles json properly

Changes

:bug: func invoke sets the data in the event after unmarshalling the string, so that the event does not treat the data as a json string

/kind bug

Fixes #2241

Release Note

func invoke now correctly handles data with contenttype application/json

/cc @lkingland @matejvasek
Let me know if you think it would be better to make this work more generically for other contenttypes/if you have an idea how to do that. For now, this fix is specific to application/json

I can testify that the fix works on my end.

@lkingland @Cali0707 should this be backported for a patch release?

@lkingland @Cali0707 should this be backported for a patch release?

Thanks for asking.
This is a feature/enhancement, so I see no need to backport.


Need English Documentation

Hello @erlaninfo, I cannot Understand This Readme file. If you like to release good global open source application must be add English Readme.md.
Regards
Thanks.

Thanks for the feedback

@erlaninfo Tell me something about your application...

Not responding..


Bugfix: register signal handler before starting the server

This PR defers the execution of server.run(), which is blocking, until the signal handler has been registered.

This PR is safe to merge. It does not affect any CI builds.


Title change

Title and filename chnge to Remote teaching practices during coronavirus (COVID-19)
This is a quick test PR, from a separate branch.

Test is timing out when linking to an external site.
Works locally for me.
cc @skumarqa2010
  1) Safeguarding and remote teaching during coronavirus page Links through to "Coronavirus (COVID-19): safeguarding in schools, colleges and other providers":
     CypressError: `cy.request()` timed out waiting `30000ms` for a response from your server.

The request we sent was:

Method: HEAD
URL: https://www.saferinternet.org.uk/advice-centre/parents-and-carers


feat: rename velodyne_driver and velodyne_pointcloud package name in launch

Description
In order to create debian package, we are renaming velodyne driver packages by adding prefix awf_ so that we can avoid conflict with the upstream packages.
This PR will point to the branch with renamed packages.
Related Issue:
https://github.com/autowarefoundation/autoware/issues/3222#issuecomment-1475656867

Notes to reviewers
This should be merged with other PRs:

autoware.repos https://github.com/autowarefoundation/autoware/pull/3492
awsim_sensor_kit_launch: https://github.com/RobotecAI/awsim_sensor_kit_launch/pull/7

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

The velodyne driver is no longer used in the Autoware.
We use nebula driver now.
Closing the PR.


fix upstart restart

@jasonxh
fix for upstart restart bug not reloading configs

LGTM! make sure you test it out first


Remove invalid check in CommandLineBuilder

Take two of my previous pull request #419 (closed due to inactivity on my part, sorry)! This version includes a unit test.
When escaping quotes, a check was done to ensure that the number of quotes that will be escaped is even. This doesn't make sense, since there's plenty of command line parsers (including Windows's own CreateProcess) that correctly process any number of embedded quotes (as long as they're escaped properly).
One way that the effects of this check can be seen at a user-interface level is by attempting to define a macro in the project properties for a C++ file which has in its body an odd number of quotes (e.g. a string with an embedded quote). For example, add FOO=""" to the C++ Preprocessor Definitions in the property pages and observe the MSB6001 error that results when you attempt to build.

Hi @cameron314, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@cameron314, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

Instead of completely removing the check, would it be safer to change it so it only counts non-escaped quotes? i.e., don't count quote ::= ' | " when the character before it is \.
My reason for this is build safety. Since the build will do irreversible things to your file system, we should try and protect against malformed command lines which might do undefined stuff to the file system.

@cdmihai: No, this would not be correct either. Take for instance the example I put in the unit test, which passes the string '"' to a tool -- it's completely valid, yet contains only a single double quote, which is not escaped. The CommandLineBuilder does not know what tool it is invoking, nor with what command line parameters (since they often come from user input), and hence cannot impose general limitations like this. All syntactically valid strings must be properly escaped and passed on to the tool.
Anyway, there's plenty of ways to mess up a build that do not involve quotes :-)

To clarify further, the job of the AppendQuotedTextToBuffer method is to take an opaque string and produce the command-line-escaped equivalent such that when that string makes its way into a tool, it comes back to the original string. If the escaping is done properly, there should not be any possibility of corrupting the command line regardless of the contents of the string. There may be certain limitations due to the nature of the command line, but quotes are not one of them since they can always be escaped.

I tend to agree with @cameron314's analysis, @cdmihai. I'd also rather not attempt to reimplement a combination of sh and cmd.exe and CRT escaping rules here--just not trying seems like a better approach.

OK, sold :)
:shipit:

Thanks, @cameron314!

Great, thanks everyone!


innerHTML ignores my angular directives

I'm submitting a ...  (check one with "x")
[ ] bug report => check the FAQ and search github for a similar issue or PR before submitting
[ ] support request => check the FAQ and search github for a similar issue before submitting
[x] feature request

Current behavior
I have the below translation token (includes HTML tag with an angular directive)
"MSG":"You may  Sign in   again"
and my component HTML is

The final translation strips of the routerLink directive
Expected/desired behavior
The final translation should have routerLink directive
ng2-translate version: 6.0.1

[translateParams] also not working with [innerHTML]

I'm having same problem, any update on this?

You cannot have angular directive / components in your translations, it's a limitation of angular because you cannot call the compiler after bootstrap (unless you're lazy loading a module, and that's just in JIT).
I'm sorry but it's not possible with this library. You can do it with angular i18n (the official thing), but not with any external library.

import { AfterViewInit, Directive, ElementRef, Input, OnDestroy, Renderer2 } from '@angular/core';
import { TranslateService } from 'translate-core';
import { Router } from '@angular/router';
@Directive({
selector: '[appTranslate]'
})
export class TranslateDirective implements AfterViewInit, OnDestroy {
@Input() appTranslate: string;
listenClickFunc: Function;
constructor(private element: ElementRef,
private translate: TranslateService,
private router: Router,
private renderer: Renderer2) {}
ngAfterViewInit() {
this.translate.get(this.appTranslate).subscribe((translateText: string) => {
this.element.nativeElement.innerHTML = translateText;
// this.cdRef.detectChanges();
  const navigationElements = Array.prototype.slice.call(this.element.nativeElement.querySelectorAll('a[routerLink]'));
  navigationElements.forEach(elem => {
    this.listenClickFunc = this.renderer.listen(this.element.nativeElement, 'click', (event) => {
      event.preventDefault();
      this.router.navigate([elem.getAttribute('routerLink')]);
    });
  });
});

}
ngOnDestroy() {
this.listenClickFunc();
}
}

`import { AfterViewInit, Directive, ElementRef, Input, OnDestroy, Renderer2 } from '@angular/core';
import { TranslateService } from 'translate-core';
import { Router } from '@angular/router';
@Directive({
selector: '[appTranslate]'
})
export class TranslateDirective implements AfterViewInit, OnDestroy {
@Input() appTranslate: string;
listenClickFunc: Function;
constructor(private element: ElementRef,
private translate: TranslateService,
private router: Router,
private renderer: Renderer2) {}
ngAfterViewInit() {
this.translate.get(this.appTranslate).subscribe((translateText: string) => {
this.element.nativeElement.innerHTML = translateText;
// this.cdRef.detectChanges();
  const navigationElements = Array.prototype.slice.call(this.element.nativeElement.querySelectorAll('a[routerLink]'));
  navigationElements.forEach(elem => {
    this.listenClickFunc = this.renderer.listen(this.element.nativeElement, 'click', (event) => {
      event.preventDefault();
      this.router.navigate([elem.getAttribute('routerLink')]);
    });
  });
});

}
ngOnDestroy() {
this.listenClickFunc();
}
}`

import { AfterViewInit, Directive, ElementRef, Input, OnDestroy, Renderer2 } from '@angular/core';
import { TranslateService } from 'translate-core';
import { Router } from '@angular/router';
@Directive({
selector: '[appTranslate]'
})
export class TranslateDirective implements AfterViewInit, OnDestroy {
@Input() appTranslate: string;
listenClickFunc: Function;
constructor(private element: ElementRef,
private translate: TranslateService,
private router: Router,
private renderer: Renderer2) {}
ngAfterViewInit() {
this.translate.get(this.appTranslate).subscribe((translateText: string) => {
this.element.nativeElement.innerHTML = translateText;
// this.cdRef.detectChanges();
  const navigationElements = Array.prototype.slice.call(this.element.nativeElement.querySelectorAll('a[routerLink]'));
  navigationElements.forEach(elem => {
    this.listenClickFunc = this.renderer.listen(this.element.nativeElement, 'click', (event) => {
      event.preventDefault();
      this.router.navigate([elem.getAttribute('routerLink')]);
    });
  });
});

}
ngOnDestroy() {
this.listenClickFunc();
}
}

@vadost thanks for the workaround, I am doing something similar at present.

@ocombe, First of all thanks for such an awesome and simple to use framework, we found it much more easier than angular's own i18n.
Though I cannot fully comprehend how it is a limitation, as I didnt got into the internal workings of it, but  If it is a limitation, was wondering would it be possible to take care of only few special/popular directives to preserve them as is in the final output?

At runtime (when the library runs), the compiler is not available, which means that you cannot compile the templates.
If you need a directive / component / pipe, you have to put it outside of your translations (by splitting the translations for example).
In the case of You may  Sign in  again you would need to split it in 3: "You may", "Sign in" and "again".
It's a pain, I know, but there's no other way.

now I see why thats an issue, Thanks for explaining this !

@ocombe That would not work for many languages as they need to shift words in sentence like:
 Sign in you may again.
It's very common to do that in Chinese.

@christarczon  Do you really want to put a blank value before every element in your website ?

@duall Create your custom directive! An example you can see from above!

Yes, there is no alternative to $compile.
Custom runtime-created templates will be possible with the next renderer (ivy) in Angular 7, but probably not for 7.0, it will just be "possible" with the new architecture, which doesn't mean that it'll be available immediately when ivy is released

Thanks for your reply. As I have already invested a lot of time in upgrading the static HTML parts of the app (by far the bigger part of it) to angular 5, I was wondering if a workaround along the following lines might be possible to avoid throwing all that work away.
Keep the angular 5 upgraded components that use static HTML templates. Refactor the angularJS element directive that hosts dynamically added attribute directives, as an angularJS component. So the app now functions as an angular 5/angularJS hybrid, at least until angular 7.x brings runtime template creation.
My question is, will an angularJS component in a hybrid app support the use of $compile in the same way that the element directive in a pure angularJS app does?

The workaround proposed by @vadost is not ideal from an accessibility point of view, because the href attribute is missing from the anchor element. Usually the routerLink directive adds the correct href attribute (see router_link.ts). But since custom directives in translations strings are not possible with with library I propose the following workaround using translate parameters.
HTML

TRANSLATION
"MSG": "You may Sign in again."

Based on the locationStrategy used in your project, the path parameter has preceding hash symbol (like here) or not.


Add MTEN plotting tools to repo

Hi @roger-randria,
As discussed during the week, we should add MTEN plotting tools here.
Eoin

Thanks for the request. I'll do my best to get what is needed at least uploaded.

Thanks @roger-randria. That would be a good start!


GH-2303 formatting rules relaxed

GitHub issue resolved: #2303
Briefly describe the changes proposed in this PR:

preserve line break settings for parenthesized expressions as much as possible
use single indent for wrapped expressions

PR Author Checklist (see the contributor guidelines for more details):

[x] my pull request is self-contained
[ ] I've added tests for the changes I made
[x] I've applied code formatting (you can use mvn process-resources to format from the command line)
[x] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change
[x] every commit has been signed off

Note: we merge all feature pull requests using squash and merge. See RDF4J git merge strategy for more details.

The new code conventions xml file fails the xml formatting check. :eyeroll:

It turns out that the formatter maven plugin is a little behind compared to the actual Eclipse IDE formatter. I've tweaked settings to preserve linebreaks for closing parentheses, and this works when I apply in my Eclipse IDE, but it looks as if the maven plugin ignores these particular settings.

The ECA check fails because I switched to a different e-mail address since I started this PR, and it apparently gets confused. The new address has been linked to my Eclipse account and gets accepted, but somehow this PR fails on my old address. We shall cheerfully ignore.

I'm going to be rolling back the indent thing. Too much change.

I'm going to be rolling back the indent thing. Too much change.

I have no strong opinion on that matter.


ActivityIndicator

How to resize the activity Indicator

It is always drawn in its intrinsic size.

accordind to this guy @natanraj it's possible
he said "I just wanted to change color. I am able to change the size etc."
https://github.com/eclipsesource/tabris-js/issues/986

You can change the size of the widget itself, but the spinning wheel renders as large as it can based on the size of the animation that is available on the device.
Changing the size is necessary, for example, to center the widget within the bounds of a UI element.

is it possible to only change the size of the spinning wheel?if yes how can i do that?

No, as @mpost mentioned, it is always drawn in its intrinsic size.
It will scale down to fit smaller elements (for example, a Cell inside a CollectionView).

ok


Comments and documentation is completely absent

Hi,
This is a very useful repository! I like how well it integrates with the example project on graphcms. And it helped me to integrate graphcms into my next.js project, I'm so happy this repo uses all known and standard packages, including Chakra-ui for styling.
Question: why the docs missing? Was it intentional?
For example, repo structure is not obvious. Components seem well organized and have multi-level references, however it is impossible to understand the overall design for components.
The code is not documented either. So things like parseColumnsMdx  https://github.com/GraphCMS/reference-marketing-website/blob/main/utils/_parseColumnsMdx.js#L13 are pretty hard to understand unless one adds multiple debugging statements and logging.
Unit tests would be nice to have, even for a demo purpose without full coverage.
Thanks!

Hi @OperationalFallacy
We don't typically "document" these references. They are aimed at those familiar with all of the moving parts to take, and see how GraphCMS applies in that domain, in this example we are using next-mdx-remote with GraphCMS. Improving documentation is something we could consider though, for sure.
As for the code you have linked to, all that's happening here is we are returning a new object for each of the columns passed in. Inside of this object we are assigning the new object content to contain the markdown (the raw content), and then mdx which is the serialized markdown from next-mdx-remote.
Thanks for checking out this repo.


Upgrading conan to version 1.19.2

In order to be compliant with Windows builds, we need to upgrade conan on Linux as well.
Related to this PR: https://github.com/KanoComputing/mercury/pull/55

Not quite there yet, I get this complain from debuild
dpkg-gencontrol: warning: Depends field of package debian-conan: unknown substitution variable ${python3:Depends}
The thing is that latest conan version needs python3. Any hints on that one?


为什么收起和展开侧边栏会肉眼可见的卡顿

Subject of the feature
Describe your issue here.
Problem
If the feature requests relates to a problem, please describe the problem you are trying to solve here.
Expected behaviour
What should happen? Please describe the desired behaviour.
Alternatives
What are the alternative solutions? Please describe what else you have considered?

Hello @Mount-Ta1. Please provide the complete reproduction steps and code. Issues labeled by need reproduction will be closed if no activities in 3 days.


Add site to monorepo

Add a new Docusaurus site, along with start and build commands.
Closes #547

The preview links in the auto-generated Cloudflare comment above seem to point to Storybook. Just want to confirm that's expected at this point (since we have #550 to setup hosting later on), right?

The preview links in the auto-generated Cloudflare comment above seem to point to Storybook. Just want to confirm that's expected at this point (since we have #550 to setup hosting later on), right?

That is correct :)

The changes look fine. Just wondering if we want to strip out the Docusaurus example pages and images at this point, or do that as part of subsequent issues?

Yes I was wondering that... I was planning on doing it for subsequent issues yeah but happy to do it as part of this one if you think that's better. Shouldn't be a big job

Yes I was wondering that... I was planning on doing it for subsequent issues yeah but happy to do it as part of this one if you think that's better. Shouldn't be a big job

It may be worth doing it now. If we can strip it down to just have our planned 4 pages (Home, Getting started, Components & Contributing), then I think that might help the rest of the team better understand the skeleton that we now have to build upon. Those pages could each be empty or just have a paragraph of Lorem Ipsum. Otherwise, if we keep the out-of-the-box sample content, we might have a lot of handwavy explaining to do "imagine this bit gone, and our page would be here, etc. etc.".
That being said, I don't have particularly strong feelings about this, so happy to defer if you think makes more sense.

I agree with @james-nash. Stripping out the dummy pages will make this PR easier to review.

🔥 blog

Love the commit message!


v2endpoint remove image path correction

the endpoint for single image removal (on the libpod side) should be as follows:
versionedPath/libpod/images/IMAGENAME
The DELETE method then signifies the removal of the image.
Fixes: #6261
Signed-off-by: Brent Baude<EMAIL_ADDRESS>

/lgtm
/hold


feat: add file mode to local file fix

This pull request introduces changes related to handling file modes in the LocalFileFix struct. It adds support for optionally specifying the file mode when creating or modifying a local file, and includes new test functions to verify the behavior of the Apply() method when the file already exists and has different modes. Default file mode retained at 0644.
Main changes:

pkg/fix_local_file.go: Added support for specifying the file mode when creating or modifying a local file.
pkg/fix_local_file_test.go: Added new test functions to verify the behavior of the Apply() method when the file already exists and has different modes. [1] [2]

requesting review from @lonegunmanb

@lonegunmanb I have tried the conversion using fs.Filemode(*lf.Mode) but it is fragile as go's integer literal syntax requires the preceding zero.
This implementation treats 0755 and 755 as identical.


Test hold gesture for speech

Refs: https://github.com/microsoft/vscode/issues/201330

[x] anyOS @eleanorjboyd
[x] anyOS @lszomoru
[x] anyOS @TylerLeonhardt

Complexity: 3
Create Issue

As a requirement you need to have the speech extension (ms-vscode.vscode-speech) installed. Make sure you see microphone buttons in inline chat or panel chat input boxes
For inline chat we have added gestures that allow you to hold down keys for "listening mode". This complements the microphone button that you can already use today. Make sure that

when you press and hold cmd+i inline chat starts and goes into listening mode. When releasing the keys, listening stops and the request is made
start inline chat normally (press and release cmd+i or via the context menu) and now hold cmd+i and make sure listening mode works
make sure the microphone buttons reflects the current state of the microphone, e.g its shines blue'ish once it records
check that the delay after which listening mode starts feels right, not too short, not too long

In addition to the inline chat enhancement there is also a new quick chat mode with a lighter UI. This more experimental but ready for feedback.

press cmd+k and hold i (this is a "holding chord") and make sure a microphone is rendered with the text.
make sure the microphone goes from grey to blue pulsing which indicates it's listening
speak and ensure the spoken words are printed
release the key and make sure the request is made
make sure no request is made when you focus out while recording
make sure no request is made when hitting ESC while recording
press (without hold) cmd+k i, talk, and select cmd+k i again for stop recording and make request

Great work. I love the new quick chat mode 👏🏻


MessageType for every message is MessageType.default

Summary
MessageType for messages should be MessageType.reply and more but it's only MessageType.default
Reproduction Steps

Go through a channel's history using the history method.
Get the list of messages.
The type field of almost every message is MessageType.default when in lots of cases it should be MessageType.reply

Code
async def get_messages(channel: discord.TextChannel):
    messages: list[discord.Message] = await channel.history(limit=10000).flatten()
    # THIS SHOULD FILTER OUT REPLIES BUT IT DOESN'T SINCE THE REPLIES ALSO HAVE MESSAGETYPE.DEFAULT
    messages = [
        message for message in messages if message.type == discord.MessageType.default
    ]
    print(channel.name, len(messages))
    return messages

Expected Results
The type should've been MessageType.reply
Actual Results
The type was MessageType.default
System Information

Python v3.10.2-final
discord.py-self v1.9.2-final
aiohttp v3.7.4.post0
system info: Windows 10 10.0.10240

Checklist

[X] I have searched the open issues for duplicates.
[X] I have shared the entire traceback.
[X] I am using a user token (and it isn't visible in the code).

Additional Information
No response

You are using deprecated version of discord.py-self. It works well in alpha version.

API v6 sends replies as regular messages. Use the alpha.


SaFlag and SigFlag types should be called SaFlags and SigFlags

However, that would be a breaking change again. Are we doing 0.5.1 or 0.6.0 next? How frequently can we release?

I just saw that too, annoying! I kind of like to treat 0.x.y versions such that a breaking change moves to 0.x+1.0, but we don't have a policy yet.

Since we are already adding breakings changes again. I would like to fix this now as well. We can release 0.6.0 in a week to get out a new version.
As long as we do not have a fully clean up version, I do not really want to support old 0.x versions. I'd rather release a new 0.x version every other week or so (with breaking changes, which the 0 in front allows).


Update API version to 8

Blocked by #47
Closes #27
This was tested and it still works fine.

Pull Request Test Coverage Report for Build 298

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.101%

Totals

Change from base Build 280:
0.0%

Covered Lines:
348

Relevant Lines:
395

💛  - Coveralls

If you remove 3e1055359fb081545f83b9758c30e8e6727bfaec, this PR is not blocked by #47


 Did you mean: 'has_local'?

../../ Did you mean: 'has_local'?

../../Library/Application Support/uv/python/cpython-3.13.0rc2-macos-aarch64-none/lib/python3.13/unittest/mock.py:1622: AttributeError
=================================== short test summary info ===================================
FAILED test.py::Test::test_foo - AttributeError: '_patch' object has no attribute 'is_local'. Did you mean: 'has_local'?
ERROR test.py::Test::test_foo - AttributeError: '_patch' object has no attribute 'is_local'. Did you mean: 'has_local'?
================================= 1 failed, 1 error in 0.13s ==================================

oh i guess it's hitting https://github.com/python/cpython/issues/104745 . but maybe triggering that is a bug here?

It's a thread-safety issue in CPython itself - no real surprise this plugin can trigger it.


⚠ Bofrost.at has degraded performance

In 8357d69, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1047 ms

Resolved: Bofrost.at performance has improved in ce780a7 after 15 minutes.


Replace Mono.Nat with Open.NAT

@NicolasDorier
I removed the Mono.Nat code and replaced it with the Open.NAT nuget package. Could you take a look at that? You will see I deleted a lot of code from UPnPLease class because Open.NAT already implements what was implemented by you in UPnPLease.cs
Let me know if you find any error, problem or suggestion.

Is it possible to fix the indentation so the diff is easier to follow ? More important, you are using await without  .ConfigureAwait(false) this has the result to make deadlock if .Result is used in a ASP.NET or WPF context.

You forgot some Await, no worry, I'll fix that. I'm testing it.

as we talked about, I should remove dependency to Mono.Nat and let people decide for themselves what library to take. I close this PR, not because it does not work, but because I agreed with you that UPNP lib have not its place in a Bitcoin Library.


[Security] Workflow pr.yml is using vulnerable action bahmutov/npm-install

The workflow pr.yml is referencing action bahmutov/npm-install using references v1.4.5. However this reference is missing the commit 7e73ecd0864000ec1a8f86106d73e38c09dbaca3 which may contain fix to the some vulnerability.
The vulnerability fix that is missing by actions version could be related to:
(1) CVE fix
(2) upgrade of vulnerable dependency
(3) fix to secret leak and others.
Please consider to update the reference to the action.

This commit was only updating a dev dependency (https://github.com/bahmutov/npm-install/pull/61/files), and a version w/ the update has not been released. Since dev dependencies do not affect the actual runtime of the action, this update will not be made until an actual release has been published.


Externalize query cache into its own service

Fixes #1440

@jamesagnew this is now ready for review

Approved

He javadoc for that annotation says “This annotation is rarely written in
source code, because it is the default“ which suggests there is no
advantage to adding it,
On Thu, Sep 5, 2019 at 10:01 PM James Agnew<EMAIL_ADDRESS>wrote:

@jamesagnew commented on this pull request.
In
hapi-fhir-jpaserver-base/src/main/java/ca/uhn/fhir/jpa/search/SearchCoordinatorSvcImpl.java
https://github.com/jamesagnew/hapi-fhir/pull/1442#discussion_r321550973:

@@ -79,6 +78,7 @@
import javax.annotation.Nullable;
import javax.annotation.PostConstruct;
import javax.persistence.EntityManager;
+import javax.validation.constraints.NotNull;

javax.annotation.Nonnull
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/jamesagnew/hapi-fhir/pull/1442?email_source=notifications&email_token=AANBORVOEKY5RQBCXZFUDUTQIG2ZJA5CNFSM4IPADKYKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCD3QH4I#discussion_r321550973,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANBORT7CHZS5SIFDP63KD3QIG2ZJANCNFSM4IPADKYA
.