crashing when files contain qualified imports

Hey folks,
I'm using tern with YouCompleteMe for Vim. I'm experiencing an issue with Tern crashing in any file that has a qualified import, of the type:
import { foo } from '../foo';

it doesn't seem to matter whether the module in question is in node_modules or in my project.
Here's my ~/.tern-config:
{
    "libs": [
        "browser",
        "jquery"
    ],
    "plugins": {
        "node": {},
        "es_modules": {},
        "complete_strings": {},
        "node_resolve": {}
    },
    "exclude": [
        ".git/**"
    ],
    "ecmaVersion": 6
}

Here's a link to a file in the project I'm working on which causes the error: https://github.com/mitodl/micromasters/blob/master/static/js/containers/UserMenu.js
if I comment out the qualified imports I can get completion on things, otherwise nothing.
Here's terns STDERR: https://gist.github.com/aliceriot/7e61da3faf40d23e921cc0ca267c6e11
Anyone have any idea what I'm screwing up? I don't feel very confident about my ~/.tern-config settings.

ah so YouCompleteMe includes Tern v 0.17, when I bumped it to 0.19 things started working fine. I'll file an issue over there, ignore the noise! (srry)


Create support for multilanguage

I know that English is the best known standard internationally, but it would be nice to have the flexibility to use your local language, in my case, Portuguese (Brazil)

I'd be keen to pick this up if the project is accepting contributions :)

@emnesty
Do you want whoever visits your site to be able to switch between multiple languages?
in your case English and Portuguese?

Yes i was thinking of using this component:
https://ui.shadcn.com/docs/components/tabs
And each tab can be a language
By default we should be able to choose a default language, such as English, but the user would have the option to choose others. I'm not a big fan of the model where the system adapts to the language the user is using in the system, because we can have people in Brazil who prefer to read the CV in English for example, so it would be interesting if the user was free to choose.

@emnesty
Inspired by this amazing project, I have created a version with translation features. I hope the code will be helpful as a reference!
Code / Demo

This totally improved the project @dninomiya. Definitely going to use your implementation.


[student dev handbook] Review the Ruby/RSpec checkpoint project

What maintenance needs to be done?
Review the Ruby/RSpec checkpoint project with the following questions in mind:

[ ] Is it up-to-date?
[ ] Does it reflect how we actually use ruby and rspec in practice for professional projects?
[ ] Are there other changes we want to make?
[ ] Should we still have this checkpoint project?  If not, should we replace it with something else?
[ ] Can we move it to Github?

The checkpoint project can be found here:

Part 1
Part 2
Part 3

If no one objects, I can volunteer to migrate these to GitHub

We had the following discussion during our meeting today, we will gather more feedback, and start a draft of the new checkpoint during our November meeting.


[Bug] AttributeError: unexpected attribute 'plot_width'

Hello,
plot_bokeh worked for me last week, but it seems that there were some updates in pandas or somewhere else and it stopped working. This code from the example (or any other)
import numpy as np

np.random.seed(42)
df = pd.DataFrame({"Google": np.random.randn(1000)+0.2,
                   "Apple": np.random.randn(1000)+0.17},
                   index=pd.date_range('1/1/2000', periods=1000))
df = df.cumsum()
df = df + 50
df.plot_bokeh(kind="line", plot_width=None)

produces the following error:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Input In [45], in <cell line: 9>()
      7 df = df.cumsum()
      8 df = df + 50
----> 9 df.plot_bokeh(kind="line", plot_width=None)

File /opt/conda/lib/python3.10/site-packages/pandas_bokeh/plot.py:1785, in FramePlotMethods.__call__(self, *args, **kwargs)
   1784 def __call__(self, *args, **kwargs):
-> 1785     return plot(self.df, *args, **kwargs)

File /opt/conda/lib/python3.10/site-packages/pandas_bokeh/plot.py:439, in plot(df_in, x, y, kind, figsize, use_index, title, legend, logx, logy, xlabel, ylabel, xticks, yticks, xlim, ylim, fontsize_title, fontsize_label, fontsize_ticks, fontsize_legend, color, colormap, category, histogram_type, stacked, weights, bins, normed, cumulative, show_average, plot_data_points, plot_data_points_size, number_format, disable_scientific_axes, show_figure, return_html, panning, zooming, sizing_mode, toolbar_location, hovertool, hovertool_string, rangetool, vertical_xlabel, x_axis_location, webgl, reuse_plot, **kwargs)
    432     xlabelname = (
    433         figure_options["x_axis_label"]
    434         if figure_options.get("x_axis_label", "") != ""
    435         else "x"
    436     )
    438 # Create Figure for plotting:
--> 439 p = figure(**figure_options)
    440 if "x_axis_type" not in figure_options:
    441     figure_options["x_axis_type"] = None

File /opt/conda/lib/python3.10/site-packages/bokeh/plotting/_figure.py:184, in figure.__init__(self, *arg, **kw)
    182 for name in kw.keys():
    183     if name not in names:
--> 184         self._raise_attribute_error_with_matches(name, names | opts.properties())
    186 super().__init__(*arg, **kw)
    188 self.x_range = get_range(opts.x_range)

File /opt/conda/lib/python3.10/site-packages/bokeh/core/has_props.py:369, in HasProps._raise_attribute_error_with_matches(self, name, properties)
    366 if not matches:
    367     matches, text = sorted(properties), "possible"
--> 369 raise AttributeError(f"unexpected attribute {name!r} to {self.__class__.__name__}, {text} attributes are {nice_join(matches)}")

AttributeError: unexpected attribute 'plot_width' to figure, similar attributes are outer_width, width or min_width

I use Python 3.10 in Jupyter Notebooks, pandas_bokeh installed with pip.
It seems that other people started having the same problem quite recently:
https://stackoverflow.com/questions/74280959/attributeerror-unexpected-attribute-plot-width-to-figure-similar-attributes

I've run into this issue as well. It seems Bokeh deprecated plot_width and plot_height, so Pandas-Bokeh will need to be updated accordingly, see https://github.com/bokeh/bokeh/issues/12543

This looks like it was fixed by e6452b3f215ed1347b0246b451a466dedcca860c.
@PatrikHlobil Is there any chance you could release a new version with this fix to make it easier to pull in via pip or other package managers?

This looks like it was fixed by e6452b3.
@PatrikHlobil Is there any chance you could release a new version with this fix to make it easier to pull in via pip or other package managers?

Please

Hi there, sorry I do not have a lot of time maintaining this library. I will have a look at the problem and hope that I can soon release a fix.

Hi there, sorry I do not have a lot of time maintaining this library. I will have a look at the problem and hope that I can soon release a fix.

For what its worth, I have been running commit e71bb528e001d3e32214069b8489083e29daf419 locally (this is the latest commit I see in your repo) for a few weeks now and it is working well, so this may not need more than creating a new release with the existing code.

Any release for this bug anytime soon? ;) Thank you for your work!

Hi I find a temporary fix in stackoverflow https://stackoverflow.com/a/75149903/5614835 stackoverflow and it works
try it out.
go to :  ./lib/python3.10/site-packages/pandas_bokeh/plot.py"
add these before line 439, in plot.py
figure_options['width'] = figure_options['plot_width']
figure_options['height'] = figure_options['plot_height']
del figure_options['plot_width']
del figure_options['plot_height']
I'm using virtual env so these are the steps I followed:


Received Battery Percentage Icon version 1.3 has run into an error

You Found An Error
Battery Percentage Icon version 2.1.3 has run into an error. You can help to fix this by:

Press Ctrl+C on this message

Report the copied error at https://github.com/soleon/Percentage/issues

System.Runtime.InteropServices.ExternalException (0x80004005): A generic error occurred in GDI+.
at Wpf.Ui.Tray.Hicon.FromSource(ImageSource source)
at Wpf.Ui.Tray.Controls.NotifyIcon.OnIconChanged(DependencyObject d, DependencyPropertyChangedEventArgs e)
at System.Windows.FrameworkElement.OnPropertyChanged(DependencyPropertyChangedEventArgs e)
at System.Windows.DependencyObject.NotifyPropertyChange(DependencyPropertyChangedEventArgs args)
at System.Windows.DependencyObject.UpdateEffectiveValue(EntryIndex entryIndex, DependencyProperty dp, PropertyMetadata metadata, EffectiveValueEntry oldEntry, EffectiveValueEntry& newEntry, Boolean coerceWithDeferredReference, Boolean coerceWithCurrentValue, OperationType operationType)
at System.Windows.DependencyObject.SetValueCommon(DependencyProperty dp, Object value, PropertyMetadata metadata, Boolean coerceWithDeferredReference, Boolean coerceWithCurrentValue, OperationType operationType, Boolean isInternal)
at Percentage.App.Extensions.NotifyIconExtensions.SetIcon(NotifyIcon notifyIcon, FrameworkElement textBlock)
at Percentage.App.NotifyIconWindow.UpdateBatteryStatus()
at System.Reactive.AnonymousSafeObserver`1.OnNext(T value)
at System.Reactive.Linq.ObservableImpl.Where`1.Predicate._.OnNext(TSource value)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
OK

I have never run into an error like this.  I am now new to GitHub - hope you can figure this out and tell me how to resolve or get a patch.  I am an account not an "IT" person.

AS you can tell, I am not an IT person.

Hi @EvAcctng56 this issue is already fixed in version 2.1.4. Please update the app to the latest version in Microsoft Store to solve this problem.
Closing, duplicated with issue #25.


Fix gson producing different date formats in diferent os/locale.

What changes were proposed in this pull request?
Previous PR #349 simply fixed bug #347 for windows environment. The root cause of the problem is that gson, by default (probably  due to different os environment/locale which I did not have time to investigate), generates different datetime formats. A small example showing that:

The fix is to set the date format explicitly to make it more deterministic.
How was this patch tested?
Tested manually in two different os environments (windows 7 & Ubuntu 18.04.1 LTS).

Should we revert #349 ?

@uncleGen Yeah. You can do that. Thanks!


add presets to climate

The main change is in climate.py where I added preset support.
To use preset you must add key "presets" into code.json, if the old code without "presets" key were use, it'll work like normal.
"presets": ["none", "eco"],

Then inside "commands" key, you must add the mode beside none here before operation mode.
"commands": {
    "off": "...",
    "cool--operation-mode-cool-when-preset-none": {
        "auto": {},
        "low": {},
    },
    "eco--presets-eco": {
        "cool--operation-mode-cool-when-preset-eco": {
            "auto": {},
            "low": {},
        },
    }
}

Ignore from -- I added that to add a comment.
You can look at the finished new code file at another pull request from me that edit 1162.json
I use home assistant dev container to edit the code, it just formats the file that I press save.
If you want me to just add the change, I'm willing to help.

You might wanted to look at json from https://github.com/smartHomeHub/SmartIR/pull/710

I'm on summer vacation right now. It is an interesting approach.

@vassilis-panos I've updated to the current version with master. Would you merge this branch? what do you want me to do next?

@vassilis-panos would you like to revive this PR?


catalyst upgrade must check if .catalyst-auth.json is present / CSCLSROZ-427

if you run catalyst upgrade on a project-folder after cloning the repo it will fail on upgrading .catalystrc (the config file) but continue doing other things.
either silently ignore this part or error out completely.
--> could be a general error if the auth file is missing. should fail and ask to run catalyst init

too old now. closing


[NG] fix form controls grid override retaining col classes

If you wanted to change the default widths on the label and controls in a grid, it would apply the correct classes to the wrapper but not remove them from the control itself.
fixes #2647

Hi Guys,
@gnomeontherun thanks for the fix!
What do you think, when will this be released, because our app with the clarity form upgrade cannot be released, due to this bug.
All the other reviewers are busy right now?
Thanks,
Peter


use a randomly generated 128-bit nonce

Using a timestamp is a bad idea because it's predictable. It's also not guaranteed to be unique across machines.
While we're here, make GenerateNonce and GenerateTimestamp private.

This breaks a test. I'm going to investigate.

Oh, TestMultipleQueryValues is a flaky test because there is an iteration over a URL.Query() map which will be random. I will fix in another PR>

The TestMultipleQueryValues de-flaking turned into #6. But then I realized that with the changes in #6 I could go further and just do the work of adding support for query parameters keys that have multiple values (#6 wasn't actually quite enough to do so, though the PR description stated that).
The support for multiple valued parameters became #8.

Anyway, this branch is ready for review!


Transfer torch tensors to GPU asynchronously

Transferring tensors to GPU can be done asynchronously with x.to(device='cuda', non_blocking=True). In some cases I observe this to speed up a training pass.
Here I compared the current gunpowder.torch.Train transfer calls to the proposed changes in an isolated example, to make sure there is no extra overhead from separating the cast to torch.tensor from the transfer to GPU.
import time
import torch
import numpy as np

x_np = np.random.rand(128, 128, 128, 128)
n_iter = 10

# init cuda
torch.cuda.set_device(0)
torch.zeros((128, 128, 128, 128), device='cuda')

start = time.time()
for i in range(n_iter):
    x = torch.as_tensor(x_np).to(device='cuda', non_blocking=True)
x_back = x.detach().cpu().numpy()
print(f"Asynchronous: {(time.time() - start) / n_iter} s")

start = time.time()
for i in range(n_iter):
    x = torch.as_tensor(x_np, device='cuda')
x_back = x.detach().cpu().numpy()
print(f"Current gunpowder: {(time.time() - start) / n_iter} s")

Added making arrays contiguous to torch.Train and torch.Predict, which is necessary for conversion to torch.Tensor.

Added making arrays contiguous to torch.Train and torch.Predict, which is necessary for conversion to torch.Tensor.

@benjamin9555 Why is the block from line 289-294 deleted?

Also, we don't think it is a good idea to force the arrays to be contiguous by default. Especially for GPU based computation, this can add overhead recreating contiguous arrays in memory before sending them to the GPU. At most, there should be an option to force contiguity.

@benjamin9555 Why is the block from line 289-294 deleted?

Ah, we see now that it is a duplicate from lines 265-270. Good catch!

Also, we don't think it is a good idea to force the arrays to be contiguous by default. Especially for GPU based computation, this can add overhead recreating contiguous arrays in memory before sending them to the GPU. At most, there should be an option to force contiguity.

The current stable version of torch v1.7.1 does still not allow to create a torch.Tensor from a non-contiguous numpy array.
Here's a minimal example that will throw an error:
import torch
import numpy as np
a = np.zeros(2)[::-1]
t = torch.as_tensor(a)

numpy.np.ascontiguousarray will not create a copy in memory if the array is already contiguous, which is what we need, right?

I think the question is whether if you do torch.as_tensor(a, device='cuda'), does it copy it straight to the GPU and not throw an error? I tried to test this locally but pytorch and my local gpu driver are not in agreement on version. If it doesn't error, I think the question is if it is still more efficient to make a local tensor (including copying it to contiguous memory) and asynchronously copy to the GPU after, or to do it all at once by synchronously.  @funkey thoughts?

import torch
import numpy as np
a = np.zeros(2)[::-1]
t = torch.as_tensor(a, device='cuda')

also throws an error. If we assume that torch will eventually allow tensor creation from non-contiguous arrays, then I agree with you, let's make forcing contiguity optional. Otherwise it makes sense to me to fix it and not bother users with it.
I just thought a bit this a bit, converting a non-contiguous numpy array probably involves some overhead, as torch would have to transcribe the numpy indexing to their indexing. I don't know the details of either of them though.

I think the question is if it is still more efficient to make a local tensor (including copying it to contiguous memory) and asynchronously copy to the GPU after, or to do it all at once by synchronously.

Good point.
Right now we are forced to provide a contiguous numpy array to start, therefore the conversion to torch.Tensor on CPU does not copy the data and should thereby be negligible for performance.
To compare the remaining transfer of the torch.tensor to GPU, I have written the minimal example above, on my machine sync and async transfer took the same amount of time in this isolated fashion. However, in a favorable scenario, the forward pass of the neural network can already start once the raw data is on the GPU, even if the labels etc. are still being transferred asynchronously. This can matter, e.g. if you have large multi-channel labels like the shape descriptors.


Fixes Institution Name in Dropdown for Viz Widget, Batch Edit, and Embargo Management

Corrects full name and overrides the authenticated text for the embargo and lease dropdowns


i have an issue while initializing readium js viewer

here is error log
0 info it worked if it ends with ok
1 verbose cli [ '/usr/local/bin/node', '/usr/local/bin/npm', 'run', 'prepare' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose stack Error: ENOENT, open '/home/theta-1-linux/Desktop/working/readium/readium-js-viewer/package.json'
4 verbose stack     at Error (native)
5 verbose cwd /home/theta-1-linux/Desktop/working/readium/readium-js-viewer
6 error Linux 3.19.0-30-generic
7 error argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "prepare"
8 error node v0.12.2
9 error npm  v2.7.4
10 error path /home/theta-1-linux/Desktop/working/readium/readium-js-viewer/package.json
11 error code ENOENT
12 error errno -2
13 error enoent ENOENT, open '/home/theta-1-linux/Desktop/working/readium/readium-js-viewer/package.json'
13 error enoent This is most likely not a problem with npm itself
13 error enoent and is related to npm not being able to find a file.
14 verbose exit [ -2, true ]

On your filesystem, does this file exist?
/home/theta-1-linux/Desktop/working/readium/readium-js-viewer/package.json

If not, what are the permissions in the containing folder?
Could you try again after moving readium-js-viewer into your home folder?

Hey Daniel
Thanks, it works for me
Thanks & Regards
Rohan M.
<EMAIL_ADDRESS>www.thetatechnolabs.com
This message and any files attached to it are intended for the use of the
addressee only and may contain private and confidential material, the
disclosure of which is prohibited by law. If you are not the intended
recipient, please note that disclosure, distribution or copying of this
message and any attachments is strictly prohibited.
On Tue, Nov 3, 2015 at 12:05 AM, Daniel Weck<EMAIL_ADDRESS>wrote:

If not, what are the permissions in the containing folder?
Could you try again after moving readium-js-viewer into your home folder?
—
Reply to this email directly or view it on GitHub
https://github.com/readium/readium-js-viewer/issues/429#issuecomment-153118664
.


progressColor这个属性可以使用渐变颜色吗？

progressColor = "linear-gradient(#6ED1FF,#1589FC)"类似于这种渐变色，应该怎么使用

https://stackoverflow.com/questions/48741796/how-to-style-svg-progress-bar-with-gradients

You cannot use gradient as a color as it will be later placed inside the stroke.

The post also provided the solution. You may fork this project, complete the function and consider whether to submit a PR.
SVG的Stroke使用渐变色前需要预定义。如果你觉得有必要，你可以fork这个项目然后自己实现。
PS: You should issue in English as far as I'm concerned(我觉得你应该用英文提Issue).


Update README.md

Add the 'json5' library to the dependencies list.

stebler — Today at 10:13
@Star pyjson5 is already in the dependency list the potentially confusing part is that it's not called like that by package managers
on pip pyjson5 is actually a different project
and the one we use is just json5
on Arch Linux it's also python-json5


Changing system clock sets peripheral clock to 48MHz

At boot, the system and peripheral clocks both appear to be 125MHz.
If the system clock is later set (to any speed, even 125MHz), the peripheral clock is set to 48MHz.
I haven't measured the peripheral clock directly, but I have used a modified version of the FatFS "example" app:
https://github.com/msearle5/no-OS-FatFS-SD-SPI-RPi-Pico - which has a "cpu" command to set the system clock and a "spi" command to set the SPI (divided from peripheral) clock. At boot, SPI clocks of 62.5, 31.25, 20.833 etc. MHz can be set implying 125MHz peripheral clock, while after setting the system clock only clocks of 24MHz, 12MHz etc. can be set implying 48MHz peripheral clock.
Setting the system clock with set_sys_clock_khz() calls set_sys_clock_pll() in pico-sdk/src/rp2_common/pico_stdlib/stdlib.c. line 39..64 and line 64 looks as if it's setting clk_peri to 48MHz from the USB clock in all cases, while I would expect it to be set to the same as the system clock.
(Maybe the fault is my expectation, and this is intentional. But if nothing else it was surprising to me, so I would recommend documenting it - the relevant section of the manual here https://raspberrypi.github.io/pico-sdk-doxygen/group__pico__stdlib.html doesn't mention the peripheral clock.)

Not sure where pico-sdk/src/rp2_common/pico_stdlib/stdlib.c's set_sys_clock_pll() is used. The default sdk clock setup happens in pico-sdk/src/rp2_common/pico_runtime/runtime.c: runtime_init() -> clocks_init().

Yes, the default clock setup behaves as I would expect with system and peripheral clocks both 125MHz, as in your diagarm (picoclocks.txt).
The problem shows up only when I set the system clock with set_sys_clock_khz(125000). (stdlib.c:126)
This calls set_sys_clock_pll (stdlib.c:129: "set_sys_clock_pll(vco, postdiv1, postdiv2);").
That function is at stdlib.c:39, and afterwards it appears that the clocks are set up like this:
rosc @ 1-12 MHz

xosc @ 12 MHz
    |
    \-- clk_ref @ 12 MHz
            |
            \-- watchdog tick 1:12, @ 1 Mhz
            |       |
            |       \-- timer/alarm: get_absolute_time() in micro seconds
            |
            \-- pll_sys @ 125 MHz
            |       |
            |       \-- clk_sys @ 125 MHz
            |
            \-- pll_usb @ 48 MHz
                  |
                  \-- clk_peri @ 48 MHz, for UART but not I2C
                  |       |
                  |       \-- DMA pacing timers
                  |
                  \-- clk_usb
                  |
                  \-- clk_adc
                  |
                  \-- clk_rtc 1:1024 @ 46,875 Hz
                            |
                            \-- RTC 1:46875 @ 1Hz

That is the peripheral clock is running at 48MHz, not 125MHz.
It has been seen before (https://forums.raspberrypi.com/viewtopic.php?p=1995296&hilit=48MHz#p1995296 and other posts on that forum) but I have not seen any actual fix for the issue there - by which I mean a method of running clk_peri and clk_sys at the same speed, when that speed is changed at run time and is not 125MHz.
(Using PIO to get a higher speed SPI port may work, for example. But that looks like a workaround that should not be necessary, considering that as far as I can tell this is not required by any actual limitations of the clock hardware - it's the SDK doing something unexpected.)

Ah yes, sorry missed that part on your original question... looks like set_sys_clock_khz does too much stuff. git-blame looks like this function hasn't been touched in ages and is prob a left-over from the early days.
Have you figured out how to set the system clock only? If you haven't changed any other clock setup, a single call like:
clock_configure(clk_sys, CLOCKS_CLK_SYS_CTRL_SRC_VALUE_CLKSRC_CLK_SYS_AUX, CLOCKS_CLK_SYS_CTRL_AUXSRC_VALUE_CLKSRC_PLL_SYS, /* default pll freq: */ 125 * MHZ , /* new sys freq: */ 42 * MHZ);
might do the job.

I don't have a strong opinion on the behaviour. I think the intention here is that if the user is changing the system clock we should switch to a known 48MHz that won't change. Then your uart will keep working etc for all subsequent clk_sys frequency changes. I agree we should document this in the doxygen at the very least. Thoughts @kilograham @Wren6991

Slightly related question: is there ever any point in having a clk_peri run faster than clk_sys?

I changed SDK like in the post here https://forums.raspberrypi.com/viewtopic.php?t=345948 and clock doesn't change clk_peri to 48Mhz anymore and SPI is a derivative of clk_sys again I guess. It works faster when you set it to do so.

@andygpz11 somewhat related to the other clock issue

@kilograham Indeed.  I will look at this after https://github.com/raspberrypi/pico-sdk/pull/1272 has landed.

See also https://github.com/raspberrypi/pico-examples/pull/40


Question on shouldComponentUpdate

I use a weird statement to see if I need to update my component. I guess there must be a better way then this. Any suggestions?
if( nextState._radiumStyleState.main ){
  if( this.state._radiumStyleState.main ){
    if( nextState._radiumStyleState.main[':hover'] !== this.state._radiumStyleState.main[':hover'] ){
	  return true;
    }
  }else{
    if( nextState._radiumStyleState.main[':hover']  ){
      return true;
    }
  }
}

https://github.com/FormidableLabs/radium/blob/master/docs/api/README.md#getstate
You can pass nextState directly to Radium.getState for this exact use case. Please do not access Radium private state, as the names and implementation are subject to change without a major version bump.


Show a summary of known ore sites (e.g., by ore type)

By request from KeyboardHack on the IRC, it would be nice to have a way to put the YARM site list into a summary mode, where all ore sites of the same ore type are added together and you see a total of your stats for each ore type.

Seconded.  I was hunting for such a mod when I started a high scarcity play: all resources low frequency, very small fields, very low richness.
Although I'd like there to be a way to auto-add detected sites for this play, but... that's another issue.

Resolved in v0.9.1 (2023-05-31)


Menu Button still clickable under HUD

When the menu button is clicked and the HUD comes up, if you click again (in the same spot as the button clicked) the button click sound effect still sounds. Minor bug. (Sorry David - menu related pains :/)!

Fixed by commit 54b81a978c43020d27be7e73eced4920c957e0ab.


Add correct MIME type "image/jpeg" on image capture for file uploads

#141 is an awesome feature but was not working out of the gate. I was not using "image/*" and saw that "image/jpg" was also set for accept but since I had "image/jpeg" it didn't work right away. There is no MIME type "image/jpg" and the actual type is "image/jpeg". This commit adds the correct MIME type while keeping "image/jpg" for anyone that used that with this feature. It'd probably be good to add more common image types in the future.
Also changed to .contains instead of == so that if the string matches anywhere in accept it will be enabled. This will streamline using the feature as it is common to have multiple types such as accept="image/jpeg, image/png, image/webp".

Any chance of getting this branch merged?

Closing in favor of https://github.com/hotwired/turbo-android/pull/282


cmake: Move declarations up.

Move several declarations further up in the file.
This is done to make future modifications simpler,
as more variables and functions are available.
Signed-off-by: Håkon Øye Amundsen<EMAIL_ADDRESS>

Codecov Report

Merging #11687 into master will decrease coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #11687     +/-   ##
=========================================
- Coverage   48.23%   48.13%   -0.1%
=========================================
  Files         279      279
  Lines       43291    43278     -13
  Branches    10359    10358      -1
=========================================
- Hits        20881    20833     -48
- Misses      18270    18302     +32
- Partials     4140     4143      +3

Impacted Files
Coverage Δ

subsys/logging/log_output.c
62.38% <0%> (-10%)
:arrow_down:

subsys/logging/log_core.c
70.85% <0%> (-4.69%)
:arrow_down:

misc/printk.c
81.81% <0%> (-3.13%)
:arrow_down:

include/logging/log_msg.h
89.09% <0%> (-1.82%)
:arrow_down:

include/logging/log_core.h
100% <0%> (ø)
: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 ff66b4b...f434e37. Read the comment docs.

FYI, this may be superseded by PR https://github.com/zephyrproject-rtos/zephyr/pull/11551.

recheck

@hakonfam can you please rebase?

@nashif can you take a look at this one?

@hakonfam seems like there's an issue with CI. I assume this is a timeout, but @nashif could confirm this.

recheck


[Shadowrun5thEdition] Skills not translated after adding them to the character sheet

When using a translated version of the character sheet, the Active Skills are only translated in the selection combo box.
Once selected and confirmed, they revert back to English.
I tested this with the French and Spanish character sheet and I don't think the issue is coming from the translation json.
Also, the issue was already there before the last fix to repeating skills, so it should not be related.
Attached capture of selecting the skills in French and how they all appears in English once selected.

Fixed now?

Fixed now?

Doesn't look like it.
I just tested and they still appears in English.

I think I may know what's going on. I'll take a look this weekend.

Pull Request #5951 should fix this. However some translation keys may need adjusting. Let me know if any skills are not translating.

Pull Request #5951 should fix this. However some translation keys may need adjusting. Let me know if any skills are not translating.
It doesn't seem to be fixed unfortunately.
I was waiting for #5951 to be merged, but I just checked the character sheet and the skills are still untranslated.
Unless there is a delay between the commit and the sheet being updated in roll20 proper?

There is. Everything is merged Mondays and pulled on Tuesday. It should be updated now though.

Alright, thanks for your hard work it's mostly fixed.
Two small issues though:

For some reason Firearms and Close Combat Groups are no longer translated anywhere.
Also the skills length are very short, making them hard to read (seems to be fixed now instead of dynamic).

Nevermind the first issue, I just checked the translation files and for some reason there was some regression. I'll correct it and add a pull request.
I don't know if the length issue is related.

I did that to the buttons. I'll adjust to show more text.

They used to be dynamics but giving them the same length as the "Default a skill" one would be an elegant fix (and imho more useable).

@clevett Have you had the time to look into fixing the skills box length?
Or should I open a different bug report for it?

Yep! I increased them a bit in the next pull request #6014 . Ultimately I'd like to provide a more dynamic option that gives them all the space if specializations are not added but for now its a compromise.

Closing this thread since the issue is fixed :)


Remove extra . at the end of helm command

Helm v2.13.1 returns the following error with the extra . after the chart name
"Error: This command needs 1 argument: chart name"
see:
https://github.com/Azure/helm-vsts-agent/issues/4#issuecomment-438218779

Is anyone looking at these PRs?

switching repo to read-only and archive mode


Integrate validation tools

Integrate https://github.com/zegl/kube-score with Monokle;

automatically validate all resources when loading/parsing
show errors/warnings in a dedicated tab!?

(this might be a good plugin contender?)

@devcatalin @WitoDelnat should we close or rephrase this given the nature of our current validation subsystem?

Closing this issue. Validation related issues should probably be created in the monokle-core repo


Nearcast

Initial pull request to submit NearCast processing/viewing code to upstream.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

@leecronce congratulations you're now a contributor to FOREST! Merry Christmas and have a Happy New Year!


Add date and time of data collection to RawDataFile

Read date and time from raw data files and provide this information through the metadata
Maybe use LocalDateTime?
https://www.baeldung.com/java-8-date-time-intro

[ ] mzML (xml element is maybe startTimeStamp)
[ ] mzXML
[ ] .d
[ ] .tdf
[ ] thermo raw --> uses mzML

Needs #633 to fix project save
Project/Set sample parameters

I want to work on this issue  ! Please assign me this

Hello, if no one is working on this issue currently, I would like to try and work on it. I've already set up MZmine locally using this guide https://mzmine.github.io/mzmine_documentation/contribute_intellij.html, and the project is working fine.
It seems like date-time was already added to RawDataFile in this issue #696.
If I'm understanding the codebase correctly, to complete this issue a method should be implemented in src/main/java/io/github/mzmine/modules/io/projectsave/RawDataFileSaveHandler.java that gets the date-time information, please correct me if I'm wrong.


Any in struct

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: termoshtt:x: ytanimuraYou have signed the CLA already but the status is still pending? Let us recheck it.

This branch has been taken out of ap203-codegen. I made a new pull request #133 .


Add hyper-serde to homu.

This change is

There's something weird about this repository, possibly because it was transferred:
Traceback (most recent call last):
  File "/home/servo/homu/_venv/bin/bottle.py", line 862, in _handle
    return route.call(**args)
  File "/home/servo/homu/_venv/bin/bottle.py", line 1740, in wrapper
    rv = callback(*a, **ka)
  File "/home/servo/homu/_venv/lib/python3.4/site-packages/homu/server.py", line 207, in callback
    repo = get_repo(repo_label, repo_cfg)
  File "/home/servo/homu/_venv/lib/python3.4/site-packages/homu/server.py", line 58, in get_repo
    assert repo.owner.login == repo_cfg['owner']
AttributeError: 'NoneType' object has no attribute 'owner'
Traceback (most recent call last):
  File "/home/servo/homu/_venv/bin/bottle.py", line 862, in _handle
    return route.call(**args)
  File "/home/servo/homu/_venv/bin/bottle.py", line 1740, in wrapper
    rv = callback(*a, **ka)
  File "/home/servo/homu/_venv/lib/python3.4/site-packages/homu/server.py", line 116, in queue
    if repo_label in g.repos and g.repos[repo_label].treeclosed >= 0:
AttributeError: 'NoneType' object has no attribute 'treeclosed'

:umbrella: The latest upstream changes (presumably #926) made this pull request unmergeable. Please resolve the merge conflicts.


10 add auth

This adds a very rudimentary and basic auth page where a user can register and login. It also adds a (again, rudimentary) nav-bar at the top of each page. The nav bar shows a "Sign In / Register" link for unauthenticated users, and a "Logout" link for authenticated users.
There's still more to do here, but I'll write other issues to capture those.
This closes #10.

AWESOME


Docker images not working?

Describe the bug
Dockerfiles are not working because ⛔ You can only create a Strapi app in an empty directory. Make sure /srv/app is empty.
I think it is not a problem with the Dockerfile, so I am posting here. None of the examples work here: https://strapi.io/documentation/v3.x/installation/docker.html
Steps to reproduce the behavior

Run docker-compose up --build on any example in Strapi docker repository inside empty directory
See error message ⛔ You can only create a Strapi app in an empty directory. Make sure /srv/app is empty.

Expected behavior
Previously this worked perfectly and all files inside of ./app were mounted properly. Nothing special needed to be done.
Screenshots
N/A
Code snippets
N/A
System

Node.js version: 12.18.0
NPM version: 6.14.4
Strapi version: 3.0.5
Database: mongodb, postgres both tested
Operating system: OSX 10.14, 10.15 both tested

Additional context
Tried all Dockerfiles in the repository but all failed to launch Strapi with this error.

May we see docker-compose.yml & dockerfile anyways (and/or commands you're using to execute) and an ls of the directory you're mapping to the container? I've never observed the issue you're describing, and have run multiple containers.

It's happening in situation when you already have some folders (for example /api and /components) that you want to copy into finished image and you are also using them as volume in docker-compose. That means that they are mounted there even before strapi is initialized ... i think.

Hi @virgil9306, I am closing this issue for inactivity. Feel free to reopen with more information so we can help you figure this out :)


[stable/phpbb] Set dependencies to latest version

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Logs:
Welcome to the Bitnami phpbb container
Subscribe to project updates by watching https://github.com/bitnami/bitnami-docker-phpbb
Submit issues and feature requests at https://github.com/bitnami/bitnami-docker-phpbb/issues

WARN  ==> You set the environment variable ALLOW_EMPTY_PASSWORD=yes. For safety reasons, do not use this flag in a production environment.
nami    INFO  Initializing apache
apache  INFO  ==> Patching httpoxy...
nami    INFO  apache successfully initialized
nami    INFO  Initializing php
nami    INFO  php successfully initialized
nami    INFO  Initializing mysql-client
nami    INFO  mysql-client successfully initialized
nami    INFO  Initializing phpbb
mysql-c INFO  Trying to connect to MySQL server
mysql-c INFO  Found MySQL server listening at guiding-porcupine-mariadb:3306
mysql-c INFO  MySQL server listening and working at guiding-porcupine-mariadb:3306
phpbb   INFO  Configuring the application. Please be patient, it can take several minutes to finish.
phpbb   INFO  Installation completed
phpbb   INFO
phpbb   INFO  ########################################################################
phpbb   INFO   Installation parameters for phpbb:
phpbb   INFO     Username: user
phpbb   INFO     Password: **********
phpbb   INFO     Email<EMAIL_ADDRESS>phpbb   INFO     First Name: User
phpbb   INFO     Last Name: Name
phpbb   INFO     Forum Name: My forum
phpbb   INFO   (Passwords are not shown for security reasons)
phpbb   INFO  ########################################################################
phpbb   INFO
nami    INFO  phpbb successfully initialized
INFO  ==> Starting phpbb...
<IP_ADDRESS> - - [11/Jun/2018:11:54:16 +0000] "POST /install/app.php/install HTTP/1.1" 200 276605
<IP_ADDRESS> - - [11/Jun/2018:11:54:28 +0000] "GET /ucp.php HTTP/1.1" 200 3138

/ok-to-test

/approve cancel

/hold

/hold cancel

/ok-to-test
/lgtm


OkObjectResult from handlers doesnt have the contenttype set to application/json

As we aggred, we remove the unittest since contenttype in OkObjectResult is set in the pipeline.
https://github.com/atc-net/atc/commit/a239c04c89cd6e433c2160280fd942f8069cb5a6


Create Blob API returned nothing

What is the issue?
From the API doc：

Create a Blob
POST /api/blobs/:digest
Create a blob from a file on the server. Returns the server file path.

tried, got 201 status code，but got nothing else returned. bugs?
OS
No response
GPU
No response
CPU
No response
Ollama version
ollama version is 0.3.6

Hi @cool-firer this is expected, as the blob was created. Let me know if you encounter any more issues


Options parameters for changing some ways how maxFiles and files.length after using manuallyAddFile method works

manuallyAddFile method now has 5th parameter - options. I added this one because to keep backward compatibility. Old code will not use this parameter, but all new code can use and add some new functionality.
Currently options can have two params:

dontSubstractMaxFiles - if is true, will not execute this.dropzone.options['maxFiles'] = this.dropzone.options['maxFiles'] - 1 line; This is because such functionality not always is needed.
addToFiles  - if is true, adds manually added file also to files array. This is needed when using code that counts files.

I think this can solve #123 and similar issues with this library.

Thanks @MekDrop for your contribution! :+1:

Needs to add to docs. :scroll:

Creating issue so I can keep track of it! :eyes:

@MekDrop I have merged this PR but I think this options will surely create mess. I would like to take suggestion from @rowanwins on this. Mean while yesterday I have created issue at base library #1574. :lock:

I can see this was in latest release included. But i don't see anything in release info or docs... will it be safe for me to use it in production code?


[CUR-284] fix: hamburger nav visible on overview and unit pages

Issue(s)

The hamburger nav was not visible for curriculum overview and units

Fixes #

Updated the tab component to remove header variant so the hamburger nav will be visible

How to test

Go to https://deploy-preview-1948--oak-web-application.netlify.thenational.academy
Click on any subject phase combination
You should see the hamburger nav for both mobile and desktop views

Screenshots
How it used to look (delete if n/a):

How it should now look:

Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner

Looks good:

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


Allow some mibs to be set conditionally

Per #342 , need to be able to drop some rows if a condition is not met.
This sets up a mib like:
      - OID: <IP_ADDRESS>.4.1.2<IP_ADDRESS>.1
        name: laLoadInt1
        poll_time_sec: 60
        tag: CPU
        condition: MemoryCache=4

The condition attribute means that for this metric, only when the corresponding metric is set for a common index will this metric get passed on. Looking at the issue for example:
standby status
<IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.3.68157449 = INTEGER: 2 ## hot standby
<IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.3.69206025 = INTEGER: 4 ## providing service

cpu usage
<IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.5.68157449 = INTEGER: 10
<IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.5.69206025 = INTEGER: 29

memory usage
<IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.7.68157449 = INTEGER: 13
<IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.7.69206025 = INTEGER: 52

We only want to pass on memory usage and cpu usage when this condition (standby status) = 4
To make this work, set this value in the snmp profile section for memory and cpu:
condition: StandbyStatus=4
Then, if and only if there's a metric with the name of StandbyStatus, and if this metric has a common index with me, and this metric value is 4 will I get passed on.

@314 - does this pattern require that we have all 3 of these metrics in the profile?
standby status
<IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.3.68157449 = INTEGER: 2 ## hot standby
<IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.3.69206025 = INTEGER: 4 ## providing service

cpu usage
<IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.5.68157449 = INTEGER: 10
<IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.5.69206025 = INTEGER: 29

memory usage
<IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.7.68157449 = INTEGER: 13
<IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.7.69206025 = INTEGER: 52

I assume so, but want to make sure I understand correctly that we are still only polling named columns and we aren't grabbing all table columns in memory and then filtering results pre-processing.

also; we're explicitly matching on name correct?
  - MIB: TEST-MIB
    symbol:
      name: standbyStatus
      OID: <IP_ADDRESS>.4.123456
      tag: other_name_here

in this case, we're only looking for standbyStatus and the existence of the tag: other_name_here won't break anything right?

also; we're explicitly matching on name correct?
  - MIB: TEST-MIB
    symbol:
      name: standbyStatus
      OID: <IP_ADDRESS>.4.123456
      tag: other_name_here

in this case, we're only looking for standbyStatus and the existence of the tag: other_name_here won't break anything right?

So ... This is a bit messy. I'm using GetName() This function will return the tag if it exists, otherwise the name. In the above case, you would want to reference the tag of other_name_here because that is what GetName will return.

As implemented, this fix is grabbing all columns (in the profile) and then pre-processing to filter out the non asked for ones. But its not doing a bulk walk and just grabbing <IP_ADDRESS>.4.1.2<IP_ADDRESS> or what have you. It will poll for <IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.3 , <IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.5 and <IP_ADDRESS>.4.1.2<IP_ADDRESS>.<IP_ADDRESS>.7 and get all indexes of these.
You would need standby status because that's what is being checked. The other 2 metrics, cpu and memory are referencing standby status but don't reference each other so either could be dropped.

let me ask this another way (sorry for the confusion).
The hwEntityStateTable's entry has 36 column OIDs.
Given a profile like this:
- OID: <IP_ADDRESS>.4.1.2<IP_ADDRESS>.1
        name: hwEntityCpuUsage
        poll_time_sec: 60
        tag: CPU
        condition: hwEntityStandbyStatus=4

- OID: <IP_ADDRESS>.4.1.2<IP_ADDRESS>.1
        name: hwEntityMemUsage
        poll_time_sec: 60
        tag: MemoryUtilization
        condition: hwEntityStandbyStatus=4

- OID: <IP_ADDRESS>.4.1.2<IP_ADDRESS>.1
        name: hwEntityStandbyStatus
        poll_time_sec: 60
        enum:
          notSupported: 1
          hotStandby: 2
          coldStandby: 3
          providingService: 4

Would ktranslate collect only these 3 columns and then apply filters...
or would it collect all 36 columns, apply the filter for condition, and then finally filter out the 33 column OIDs that aren't in the profile?


Question on training a RAFT-Stereo model capable of handling positive and negative disparity

Hello,
Amazing work on this paper! I am experimenting with RAFT-Stereo to work on stereo pairs with positive and negative disparity values. I wanted to know if raft-stereo can handle stereo pairs with positive and negative disparity. What will be the best way to train such a network? Would a dataset with positive and negative ground truth disparities values suffice or do I need to change some parts of the architecture?

You don't need to modify any aspect of the RAFT-Stereo architecture in order to train it for both positive and negative disparities. Simply training the model on a dataset with positive and negative ground truth disparity is enough.

Thanks @lahavlipson for your quick response!


readme-update

Codecov Report

Merging #17 (c5164cb) into main (ec929a4) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##                main       #17   +/-   ##
===========================================
  Coverage     100.00%   100.00%
  Complexity        43        43
===========================================
  Files             17        17
  Lines            115       115
===========================================
  Hits             115       115

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 ec929a4...c5164cb. Read the comment docs.


PR #114 broke Parcel update lifecycle

It seems that the pr #114 to support react 18 destroyed the update lifecycle of parcels.
See following example: https://codesandbox.io/s/eager-leavitt-5opmv
You removed the whenFinished as parameter here: https://github.com/single-spa/single-spa-react/commit/b555cd6e8a4766bc9cca0556bf0b00464a9f8480#diff-c5eaff085b5e6099025d960035472f3294bb99ba188311b924d7987a6f8f919eR251
But it is still passed in the update lifecycle method and expected to be called.

single-spa minified message #31: Lifecycle function update for parcel parcel-0 lifecycle did not resolve or reject for 3000 ms. See https://single-spa.js.org/error/?code=31&arg=update&arg=parcel&arg=parcel-0&arg=3000

I've confirmed that this is a bug and have begun work on fixing it. Will submit a pull request in the next few days or so

Fix released in https://github.com/single-spa/single-spa-react/releases/tag/v4.2.1


Show changes made in dropbox in logs

Question about/Request from user to log changes made in dropbox.
Current behavior is that changes made from dropbox website or synced dropbox folder do not display in OSF logs. Changes made to dropbox content from the OSF interface do show in logs. All versions of files display on file detail page, regardless of where change was made.
Dropbox has recently added webhooks. Would be great to utilize this functionality on the OSF.

Included in https://github.com/CenterForOpenScience/osf.io/issues/620. Close this one and keep tracking issue there.


Add that 'Limit job authorization scope to current project' should be turned off

When publishing from one project to another, in the same org, the relevant 'Limit job authorization scope to current project' needs to be turned off on the origination project - otherwise the system token is limited to that project. There is one for release pipelines and one for non-release pipelines.
You get to that setting from project settings / pipeline settings

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

ID: 34f46a54-1fe4-6b64-9479-022f3a06cd70
Version Independent ID: 7f2e7006-865c-5ace-cd85-3b2936db9ffa
Content: Project scoped feeds - Azure Artifacts
Content Source: docs/artifacts/feeds/project-scoped-feeds.md
Service: azure-devops-artifacts
GitHub Login: @ramiMSFT
Microsoft Alias: rabououn

@housten Thanks for your feedback! I tested a couple of scenarios. I'm adding that to the docs

Hi @housten,  I'll be updating the docs to provide a more secure approach to accomplish this. Disabling the Limit job authorization scope is not the best security practice. The recommended approach is to configure both your downstream project and pipeline as follows:
On the pipeline generating the artifact (downstream project): select the ellipsis for more options > Manage security > search for your upstream project's build service and allow the following: Update build information, View build pipeline, and View builds.
On the downstream project: Project Settings > Permissions > Users > search for your upstream project's name and then select Expand search > select your upstream project's build service and allow the following: View project-level information.

This isn't quite making sense to me. I am not using a pipeline in the 'upstream' project.
The build in Project_A builds, packages and pushes a Project_A repo's package to the feed in Project_Common.
Project_Common doesn't have a pipeline or process related to Project_A.
How would giving a build service in Project_Common viewing permissions in Project_A have any effect?


Add support for typedoc-style hyperlinks in comments on hover

typedoc supports hyperlink in comments using either the double bracket ([[link]]) or {@link} notation. Neither of these types of hyperlinks are rendered correctly when comments are hovered over in VS Code.
Consider the below example:
/**
 * Friend of [[Bar]].
 */
class Foo {
}

/**
 * Has a [[Baz|good friend]].
 */
class Bar {
}

/**
 * Reference to {@link Foo}.
 */
class Baz {
}

Hovering over each class symbol (Foo, Bar, and Baz) produces the following results:
Foo

Bar

Baz

It would be great if these could get rendered properly using hyperlink-style markdown (e.g. highlighted in blue and, when clicked, navigating to the appropriate class). Without this feature, much of our typedoc can look quite ugly; especially those using the same format as Bar above.
(May be related to #8223; admittedly I don't understand the context of that issue well enough to judge.)

Hello @mjbvz, is there any update to this? I generally assume it's not prioritized; is this something that is better left to an extension, or is this something VS Code plans to try and integrate natively?

I have no plans to prioritize this feature


Fixed for new SLS (#1)

Variables class was removed.

Update run-tests.js

Update package.json

Update package-lock.json

I have a problem that this PR could solve. Why this package is not being maintained?
What is the alternative for serverless-jest-plugin? the mocha plugin is not being maintained as well :|

@meitix  I just use my fork until it's merged with the fix and it works great :)


10.0.22621.317 not supported

10.0.22621.317 not supported

Duplicate of https://github.com/stascorp/rdpwrap/issues/1944


Rename unused parameters to avoid clippy errors

These unused parameters result in clippy errors when compiled with -F unstable,create.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.001%) to 26.517%

Totals

Change from base Build<PHONE_NUMBER>:
-0.001%

Covered Lines:
555

Relevant Lines:
2093

💛  - Coveralls

I'll include this in #18.


Fix ```yaml bug

Fix for this issue: https://github.com/PowerShell/platyPS/issues/333
It looks like previously extra lines were being added in update mode, but this is no longer the case, so the "special case" in the code below was causing ```yaml to be added to the same line as the description rather than a new line.
Repro for this issue:
Take a markdown file, remove a parameter from the parameter list, and run Update-MarkdownHelp.  This will show that the yaml tag ends up on the same line as the description.  After this change this will no longer happen.
Previous:
PARAMETERS
-DefaultProfile
The credentials, account, tenant, and subscription used for communication with azure
Type: IAzureContextContainer
Parameter Sets: (All)
Aliases: AzureRmContext, AzureCredential

Required: False
Position: Named
Default value: None
Accept pipeline input: False
Accept wildcard characters: False

-Disable
Indicates that the key you are adding is set to an initial state of disabled. Any attempt to use the key will fail. Use this parameter if you are preloading keys that you intend to enable later.
Type: SwitchParameter
Parameter Sets: (All)
Aliases:

Required: False
Position: Named
Default value: None
Accept pipeline input: False
Accept wildcard characters: False

After:
PARAMETERS
-DefaultProfile
The credentials, account, tenant, and subscription used for communication with azure
Type: IAzureContextContainer
Parameter Sets: (All)
Aliases: AzureRmContext, AzureCredential

Required: False
Position: Named
Default value: None
Accept pipeline input: False
Accept wildcard characters: False

-Disable
Indicates that the key you are adding is set to an initial state of disabled. Any attempt to use the key will fail. Use this parameter if you are preloading keys that you intend to enable later.```yaml
Type: SwitchParameter
Parameter Sets: (All)
Aliases:
Required: False
Position: Named
Default value: None
Accept pipeline input: False
Accept wildcard characters: False

I need to experiment with this before merging, I will take a look tonight.
Can you possibly add a test that demonstrates that it fails before and doesn't fail after you change?

Sure, I'll look at that today.

Test added - let me know if you need anything else from me.  Thanks so much for taking a look!


Use pooch to fetch data

Following the change introduced in #24, I would suggest to fetch the nuts file using pooch (https://www.fatiando.org/pooch/latest/index.html).
Before downloading, pooch checks if the file is present locally and only downloads if it is missing. This means that the file only needs to be fetched once instead of every time.

Closed by #24.


Improve relation handling

Closes https://github.com/rustprooflabs/pgosm-flex/issues/255.
:warning: Potentially breaking change
My intention of this particular view isn't for it to be queried directly, but that doesn't mean someone somewhere isn't.  This is the reason for bumping version to 0.6.0 instead of continuing with 0.5.2.

Rename backend view places_in_relations to place_polygon_in_relations https://github.com/rustprooflabs/pgosm-flex/commit/5a57c5eef03054aaafa285b0d2d6dbc878bfa1ce

New materialized views
New views to be preferred by default over the base table name (w/out v in the name).

osm.vroad_line
osm.vroad_polygon
osm.vroad_major
osm.vwater_line
osm.vwater_polygon
osm.vpublic_transport_line
osm.vpublic_transport_polygon

New backend views
The materialized views all have related views used.  Not listed here, not expected to be primary use views.

Unit tests pass, Colorado ran successfully.


Fixed the baseUrl option not working problem

Hi, @rileytomasek
Changing the baseUrl is excellent for when you want to cache stuff with a proxy.
I noticed it wasn't working, so I made this PR to fix it.
I hope this helps, and I want to say I like the work you've done in this repo 😄

Thanks for catching this and opening a PR. The repo you linked to is a smart way to handle caching!
I'll publish a new version with the changes right now.

https://github.com/rileytomasek/openai-fetch/releases/tag/v1.1.1


[XML] Allow space after/before the equal sign in(fix #26570)

Fix #26570
Made by command
$ ./build/npm/update-grammar.js atom/language-xml grammars/xml.cson ./extensions/xml/syntaxes/xml.json grammars/xsl.cson ./extensions/x
ml/syntaxes/xsl.json

result

The grammar got updated in the meantime. Thanks anyway @Ikuyadeu !

@aeschli Thank you!


WIP: xfunction_iterator optimization

Fixes #931

Tagging it as WIP since we need to deal with dumy iterators from xscalar.

Did we reach any consensus with this PR?
Maybe the easiest / best solution is to just add a linear index to the xfunction iterator and use it for comparison?
Because a xfunction with only xscalars would always be problematic, right?

I'm gonna implement a find_first_ metafunction that operates on type list, so I can get the first non dummy iterator for comparison.
Adding a linear index can be costly, because the function_iterator can hold other function_iterator instances, and we would perform useless operations of the whole expression tree.


Update URLs

This fixes outdated urls for docs.akash.network, Cloudmos, and images.

All docs.akash.network updated to correct pages on akash.network/docs
Cloudmos URL references have been removed or replaced for Akash Console
Attribute signing documentation link in SDL updated

Thx @88plug !
Updated URLs, naming "Cloudmos" to "Akash Console" and  edit path to .png files.
Ready to merge @anilmurty @brewsterdrinkwater .


[SPARK-10490] [ML] Consolidate the Cholesky solvers in WeightedLeastSquares and ALS

Consolidate the Cholesky solvers in WeightedLeastSquares and ALS.

Jenkins, test this please.

Merged build triggered.

Merged build started.

Test build #43079 has started for   PR 8936 at commit 72f8782.

Test build #43079 has finished for   PR 8936 at commit 72f8782.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43079/
Test PASSed.

Merged build triggered.

Merged build started.

Test build #43116 has started for   PR 8936 at commit 55955e7.

Merged build triggered.

Merged build started.

Test build #43117 has started for   PR 8936 at commit 82aad64.

Test build #43116 has finished for   PR 8936 at commit 55955e7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43116/
Test PASSed.

Merged build finished. Test PASSed.

Test build #43117 has finished for   PR 8936 at commit 82aad64.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43117/
Test PASSed.

Merged build finished. Test PASSed.

LGTM. Merged into master. Thanks!


Subscript returnTypeName does not contain where clause.

As in title, example:

protocol ProtocolWithSubscripts {
    // ...
    subscript<T>(with generic: T) -> Int where T: FloatingPoint { get set }
}

In Subscript documentation it says:
Return value type name used in declaration, including generic constraints, i.e. where T: Equatable

It seems to be regression, as in Sourcery 1.0.3 it was still working.
Is this a new bug or known limitation? It would be great to have at least something similar to the Method .name variable, that returns the whole name, with all generic constraints, etc.
// Using Sourcery 1.6.0, Swift 5.5

We should probably add genericRequirements same as I did for protocols


fix date formatting for non-English locale

close #1159

Time handling in python is really cursed *looks at time.mktime() only working for local time_struct. Thanks a lot.


Use setImmediate instead of requestAnimationFrame

I noticed the queue operations for enhancing elements are bound to paint timings because they use requestAnimationFrame. This can be brutal for performance: http://jsperf.com/raf-vs-setimmediate
This PR swaps out the underlying delay for a simple setImmediate method that uses the browser's asynchronous image error event to attempt to invoke a function as soon as possible, rather than during the next paint.
Perhaps it might be worth a quick check to see if this simple setImmediate() implementation is truly async, and when it isn't fall back to setTimeout(fn, 10).

I'm afraid this PR won't probably ever land for the following reasons:

I really don't use rAF just to ask something asynchronous, I use it to boost up internally as much as possible the whole logic involved if there are elements to resolve in the array
using networks errors to shim any functionality is a very problematic approach if you are under CSP or restricted environment where that might not even fire.

The best part of this poly is its "universal compatibility" and unless you have numbers for the real world where setImmediate, which AFAIK doesn't try to optimize code,  is better than rAF and rAF is the bottleneck, I might consider using it only if asynchronous and going rAF as fallback for every other case.
Thanks in anyway for pointing this out, I will keep it in mind if necesary. Until that time, I rather play it safe here.
Best Regards

P.S. to avoid misunderstanding: I am not asking you benchmarks about setImmediate VS rAF, I'd like to see a Web App done via this polyfill that goes much faster using setImmediate. Anything else is a potentially problematic premature optimization I'm not fund to introduce.

requestAnimationFrame does not try to optimize code. It executes a given function during the next repaint. You are only using it in one place, which is to defer upgrading of registered custom elements so that setting innerHTML does not synchronously instantiate them.

requestAnimationFrame does not try to optimize code. It executes a given function during the next repaint.

which is exactly what I want to do, possibly fit into one single repaint all nodes that needs to show up.
I like how you ignored CSP problems ;-)
Playing dirty with the network to simulate something I don't wactually want to, doesn't look like the best way to go.
Do you have any real-world case where this hack is needed beside a micro-bench?
Thank you


Fix issues arising from the DMD 9029 fix (types match to alias)

This PR represents two minor upfront fixes to make the DMD 9029 fix pass. (With 9029, primitive types can be passed to alias parameters same as named types.)

Tests?

Belong with the DMD pr - this is just fixes.


Captured video  - problems with colors

Bug report, debug log and your config file (FULL LOGS ARE MANDATORY)

https://pastebin.com/wY9HjHfn
Steps to reproduce
RPI4B+AWA_SPI+MS2130+Feintech VSP01222
What is expected?
Real colors
What is actually happening?
Captured video has only blue and yellow colors
System

https://pastebin.com/nwd50gdQ

Hi
Interesting but I don't see any connection to HyperHDR here. In fact quite interesting sepia effect ;) Maybe you tuned-up LUT table or it's broken?
Connect the grabber to USB2.0 port, set MJPEG in the grabber configuration and disable tone mapping.


🐛(HttpRequestHandler) fix bug on android when receiving plain boolean…

… values

fixes https://github.com/capacitor-community/http/issues/144

Thanks for your contributions @Frank608 !


Try to load native linux libraries with matching classifier first

Motivation:
Users' runtime systems may have incompatible dynamic libraries to the ones our
tcnative wrappers link to. Unfortunately, we cannot determine and catch these
scenarios (in which the JVM crashes) but we can make a more educated guess on
what library to load and try to find one that works better before crashing.
Modifications:
Based on #9410 and https://github.com/netty/netty-tcnative/pull/480, this PR adds:

Load native linux libraries with matching classifier (first) and only afterwards try the generic one

Result:
More developers / users can use the dynamically-linked native libraries.

This change is

@netty-bot test this please

thanks for the review, I believe, I have addressed your comments now

@normanmaurer can you give me a hint for what the next steps are? We probably need to create new docker files for the new distributions but is that enough then to get release artifacts?

@netty-bot test this please

@NicoK I need to think about how to process after this to make sure we also release extra native artefacts.


An error when compiling Caffe2 on Ubuntu 16.04 with Cuda 8.0 and Cudnn v5.1

I met an error like the following contents and i have tried to solve this problem using this method https://github.com/caffe2/caffe2/issues/937, but it doesn't work. I search this problem on the Internet but found nothing helpful to me. I need help, thank you!
[ 60%] Building NVCC (Device) object caffe2/CMakeFiles/Caffe2_GPU.dir/operators/Caffe2_GPU_generated_resize_op.cu.o
[ 60%] Building NVCC (Device) object caffe2/CMakeFiles/Caffe2_GPU.dir/operators/Caffe2_GPU_generated_half_float_ops.cu.o
[ 60%] Building NVCC (Device) object caffe2/CMakeFiles/Caffe2_GPU.dir/operators/Caffe2_GPU_generated_softplus_op.cu.o
/home/tianluchao/ProgramFiles/caffe2/caffe2/operators/resize_op.cu(63): error: identifier "__ldg" is undefined
1 error detected in the compilation of "/tmp/tmpxft_00006628_00000000-7_resize_op.cpp1.ii".
CMake Error at Caffe2_GPU_generated_resize_op.cu.o.cmake:262 (message):
Error generating file
/home/tianluchao/ProgramFiles/caffe2/build/caffe2/CMakeFiles/Caffe2_GPU.dir/operators/./Caffe2_GPU_generated_resize_op.cu.o
caffe2/CMakeFiles/Caffe2_GPU.dir/build.make:308: recipe for target 'caffe2/CMakeFiles/Caffe2_GPU.dir/operators/Caffe2_GPU_generated_resize_op.cu.o' failed
make[3]: *** [caffe2/CMakeFiles/Caffe2_GPU.dir/operators/Caffe2_GPU_generated_resize_op.cu.o] Error 1
make[3]: *** Waiting for unfinished jobs....
make[3]: Leaving directory '/home/tianluchao/ProgramFiles/caffe2/build'
CMakeFiles/Makefile2:2484: recipe for target 'caffe2/CMakeFiles/Caffe2_GPU.dir/all' failed
make[2]: *** [caffe2/CMakeFiles/Caffe2_GPU.dir/all] Error 2
make[2]: Leaving directory '/home/tianluchao/ProgramFiles/caffe2/build'
Makefile:138: recipe for target 'all' failed
make[1]: *** [all] Error 2
make[1]: Leaving directory '/home/tianluchao/ProgramFiles/caffe2/build'
Makefile:4: recipe for target 'all' failed
make: *** [all] Error 2

This should be easily fixed by changing the https://github.com/caffe2/caffe2/blob/master/cmake/Cuda.cmake#L3
and
https://github.com/caffe2/caffe2/blob/master/cmake/Cuda.cmake#L4.
Check out https://github.com/caffe2/caffe2/issues/937
Once you done this change, it would be better to make clean and clean the cache.

Hello,
I have the same error, my computer compute capability is 3.0. So, what's the exactly change that i have to make?
Thanks


Support authorization using provisioning file

Hi there,
I would like to supply a provisioning file (.prvc) for authorizing ArcGIS Server, however the cookbooks don't provide an option for supplying the email address which is required when using a provisioning file.

The ArcGIS Chef cookbooks do not support supplying e-mail address during software authorization with .prvc files because we assumed that .prvc files provided by end users would always contain e-mail addresses in them, and that .prvc files without e-mail addresses are used only internally during the software development cycle.

Oh ok. I presumed I needed to supply the email address as an argument, but works ok without it.
Thanks


Refined Storage auto crafting issue

LaserIO version
1.6.6
Minecraft Version
1.20.1
Forge Version
47.1.3
Modpack & Version
ATM 9 0.2.12
Do you have optifine installed?
No
Describe the issue
Trying to auto craft advanced filters in Refined Storage. Example. Patterns for counting filter and all underlying components are installed in Refined. Try to auto craft a counting filter and it will not work because it will not see the basic filter recipe. Tried exact mode in patterns with same result.
Steps to reproduce
Set up an advanced filter (counting, nbt, mod, ect) in Refined storage auto crafter including all underlying recipes.
Try to auto craft the advanced filter and it will not be able to complete.
Expected behaviour
Should be able to auto craft filter if all underlying recipes are included.
Screenshots
No response
Log files
No response
Additional information
No response

I can’t imagine how that would be an issue on my end. Ensure the output of the basic recipe is a basic filter without any nbt tags on it (freshly crafted).


Doesn't work in a mat-sidenav-container

The split pane doesn't work in a mat-sidenav-container. The first split area is not recognized:
split area 1 css:
order: 0
flex: 0 0 calc(0% - 0px)
split area 2 css:
order: 2
flex: 0 0 calc(100% - 11px)

I've been using AngularSplit in a mat-sidenav-container without issue.


Add prometheus metrics

Continuation of https://github.com/kubernetes-incubator/kube-aws/pull/1032
Added the yaml option and rebased onto the current master.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@d15e5cf). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1092   +/-   ##
=========================================
  Coverage          ?   35.34%
=========================================
  Files             ?       59
  Lines             ?     3404
  Branches          ?        0
=========================================
  Hits              ?     1203
  Misses            ?     2039
  Partials          ?      162

Continue to review full report at Codecov.

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

@zach-dunton-sf Thanks for covering this as I hadn't had a chance to do it


Using custom HuggingFace models

Reading the details here, it was my understanding that simply adding
// LLaMa-2 example from HuggingFace (https://huggingface.co/upstage/Llama-2-70b-instruct-v2)
const llama_2_70B_instruct_v2 = new SageMakerModel(this, 'Llama-2-70b-instruct-v2', {
  vpc: vpc.vpc,
  region: this.region,
  model: {
    type: DeploymentType.Container,
    modelId: 'upstage/Llama-2-70b-instruct-v2',
    container: ContainerImages.HF_PYTORCH_LLM_TGI_INFERENCE_LATEST,
    instanceType: 'ml.g5.24xlarge',
    // https://github.com/awslabs/extending-the-context-length-of-open-source-llms/blob/main/custom-tgi-ecr/deploy.ipynb
    containerStartupHealthCheckTimeoutInSeconds: 600,
    env: {
      SM_NUM_GPUS: JSON.stringify(4),
      MAX_INPUT_LENGTH: JSON.stringify(12000),
      MAX_TOTAL_TOKENS: JSON.stringify(12001),
      HF_MODEL_QUANTIZE: 'gptq',
      TRUST_REMOTE_CODE: JSON.stringify(true),
      MAX_BATCH_PREFILL_TOKENS: JSON.stringify(12001),
      MAX_BATCH_TOTAL_TOKENS: JSON.stringify(12001),
      GPTQ_BITS: JSON.stringify(4),
      GPTQ_GROUPSIZE: JSON.stringify(128),
      DNTK_ALPHA_SCALER: JSON.stringify(0.25),
    },
  },
});
// Make model interface aware of the sagemaker endpoint and add the necessary permissions to the lambda function
langchainInterface.addSageMakerEndpoint({
  name: 'Llama-2-70b-instruct-v2',
  endpoint: llama_2_70B_instruct_v2.endpoint,
});

to aws-genai-llm-chatbot-stack.ts would be sufficient (similar to the amazon/FalconLite example).
However with this configuration I get the following error: The following resource(s) failed to create: [Llama270binstructv2upstageLlama270binstructv2035C66C7].
Any directions appreciated 🙏

Hi @uzaymacar,
As of today, HuggingFace models require some research upfront before attempting a deployment.
Reason being, each model has its own unique requirements in terms of:

instanceType (n. of GPU, Memory...)
Environment Variables
Prompting
Request Parameters

This means you can't simply assume to re-use the same instance type and/or env vars for any other model.
The settings you used above are specifically made for FalconLite only.
InstanceType and Env vars
For example, llama2-70B would require an ml.g5.48xlarge instance to start with (be careful with the costs) - check the table under Deploy a model section of this blog post for a map of Llama models / AWS instance type.
And by the look of it no environment vars is needed.
These information as of today are bit sparse, some can be found in HuggingFace Model cards, community tab and/or some blog posts made for the purpose of deploying on SageMaker.
The SageMakerModel constructs simply saves you some heavy-lifting in terms of resources to write but the right settings like instanceType and env variable needs to be researched upfront and can't be assumed for HF models, sometimes you'll have to deal some trial and error too.
Prompting and Request parameter
Lastly, in order for the application to communicate with the model correctly, you need to make sure you have the right  model adapter examples (we added one for llama already but for other models not listed you need to write your own. Usually the format of the prompt is explained in the HuggingFace model cards with examples.
I'll take sometime to collect this directions in one place, hope it clarifies.

@uzaymacar was able to deploy Llama-2-13b-chat-hf, I guess the only thing missing in your code is the HUGGING_FACE_HUB_TOKEN
`     const llama2chat = new SageMakerModel(this, 'LLamaV2Chat', {
vpc: vpc.vpc,
region: this.region,
model: {
type: DeploymentType.Container,
modelId: 'meta-llama/Llama-2-13b-chat-hf',
container: ContainerImages.HF_PYTORCH_LLM_TGI_INFERENCE_LATEST,
instanceType: 'ml.g5.12xlarge',
containerStartupHealthCheckTimeoutInSeconds: 600,
env: {
SM_NUM_GPUS: '4',
HF_MODEL_ID: 'meta-llama/Llama-2-13b-chat-hf',
MAX_INPUT_LENGTH: '2048',
MAX_TOTAL_TOKENS: '4096',
MAX_BATCH_TOTAL_TOKENS: '8192',
HUGGING_FACE_HUB_TOKEN: process.env.HUGGING_FACE_HUB_TOKEN ? process.env.HUGGING_FACE_HUB_TOKEN : ''
},
},
});
// Make model interface aware of the sagemaker endpoint and add the necessary permissions to the lambda function
langchainInterface.addSageMakerEndpoint({
  name: 'Llama-2-13b-chat-hf',
  endpoint: llama2chat.endpoint,
});

`


chore(starknet_sequencer_node): add validation function to active component config

Stack:

#2601
#2600 ⬅
#2599
#2578
#2577
#2576

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is

This change is


literal IPv6 addresses are not parsed correctly in lb_endpoints with STRICT_DNS

I want to have cluster lb_endpoints that may include both hostnames and literal IP addresses at the same time. When the cluster's type is STRICT_DNS, the configuration is refused with this error keeps iterating in the logs
[2021-10-13 13:53:47.264][1][warning][config] [source/common/config/grpc_subscription_impl.cc:126] gRPC config for type.googleapis.com/envoy.config.cluster.v3.Cluster rejected: Error adding/updating cluster(s) default-c02-tcp-443-cluster: malformed url: tcp://fdee:6207::1000:0:0:a3bd:22607

I guess this is because the square brackets are not set around the IPv6 literal (i.e. it should be tcp://[fdee:6207::1000:0:0:a3bd]:22607)? I tried having IPv4 literals and they work perfectly fine. So this is an IPv6-specific issue.
Also I tried it with LOGICAL_DNS but I got the error
[2021-10-13 13:33:38.163][1][warning][config] [source/common/config/grpc_subscription_impl.cc:126] gRPC config for type.googleapis.com/envoy.config.cluster.v3.Cluster rejected: Error adding/updating cluster(s) default-c02-tcp-443-cluster: LOGICAL_DNS clusters must have a single locality_lb_endpoint and a single lb_endpoint

But that was expected since I have more than one endpoint.
Also I am using Envoy 1.19 if that may help.

@alyssawilk thanks for the response. What I meant is that it seems Envoy always blindly assumes that the address field of the core.SocketAddress message is a hostname in the case of the STRICT_DNS and does not add the brackets if the field contains an IPv6 literal. That's why it seems it's not a problem if the address is an IPv4 address.
In my case, I am having more than one lb_endpoint. Some of those endpoints can be IPv6 as well as IPv4 literals along side with the resolvable hostnames.
The weird thing is that if the lb_endpoints only include one or more IPv6 literals, this issue disappears. In other words, this bug only appears if there is a hostname in the lb_endpoints.

I'm willing to work on this, if it's still considered a bug.

Hi,
Is there any update on this issue?
Thanks

This issue was fixed in v1.23. @alyssawilk could you close it?


Add function parameters to ibc.prototype.load()

I just add 2 function parameters to ibc.prototype.load() to help override its behaviour without rewriting the whole function (as advised in the FAQ). The default behaviour stays the same as before.

Whats the advantage of using your arguments instead of rewriting the function?  If they have to write their own filter and registration function they might as well write their own load function.  load() is just a wrapper around these functions.

we have removed the filter_users function in v1.3.0.


how to avoid docker inside docker issue using  testcontainer

Team,
We are using bamboo for our CI. In this case, How can I run my integration tests inside the bamboo agent docker?
TIA

You can either use docker socket mounting or dind. This doesn't seem to be a Testcontainers related question, but more general Docker usage and CI setup.
Or is there anything Testcontainers specific that I'm missing here?

@Muraligowtham please see https://www.testcontainers.org/supported_docker_environment/continuous_integration/dind_patterns/ for more information on what @kiview has said.

Hi @kiview / @rnorth ,
Sorry If my question was unclear.

Hi @kiview / @rnorth,
Sorry if my question was unclear.

Hi @kiview / @rnorth ,
I got the docker inside docker working in my local. Thanks you for the documentation.


[ina.fr] some link structures come up as unsupported, while others work

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2022.07.18 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
Test link:
https://www.ina.fr/ina-eclaire-actu/arletty-carriere-conseils-actrice-marcel-carne
Results as unsupported, while other's seem to work. Maybe, some improvement in regex of test case needed..
Manually extracted and downloaded video via this link eventually:
https://media-hub.ina.fr/video/z3tU1+BVR4+BmI7Y9JILCUg77yNX7/0QvIXegFnUfrHlyAUAYHiClCbJ6GkrC9SftbgzUW5iC+v3tfYyWWjwWOUt0rJCu5N7K/rUD9neBj2n53PFBKPj9eZN8kVoOM6h/sl_iv/nq9SttsGT5I8j7lWy03cRg==/sl_hm/jptv3fl8Y2UpHPIwNABCb0EsLk2LQu1tmwI8afa4LqbUA2D7b7a34J5qRF+zRk/g+pfSRwweBcBARq4UjbPqhw==/sl_e/1659798528
However this link and some other with the same link structure seem to work fine:
https://www.ina.fr/ina-eclaire-actu/video/i12055569/francois-hollande-je-crois-que-c-est-clair
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
./yt-dlp -vU -F "https://www.ina.fr/ina-eclaire-actu/arletty-carriere-conseils-actrice-marcel-carne"
[debug] Command-line config: ['-vU', '-F', 'https://www.ina.fr/ina-eclaire-actu/arletty-carriere-conseils-actrice-marcel-carne']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.06.29 [9d339c41e] (zip)
[debug] Plugins: ['SamplePluginIE', 'SamplePluginPP']
[debug] Python 3.9.13 (CPython 64bit) - macOS-12.4-arm64-arm-64bit
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: avconv -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] Checking exe version: avprobe -bsfs
[debug] exe versions: none
[debug] Optional libraries: sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.07.18, Current version: 2022.06.29
[debug] Downloading _update_spec from https://github.com/yt-dlp/yt-dlp/releases/download/2022.07.18/_update_spec
Current Build Hash f79b9efc8afc0f5169475e1c6b60fa5083497315b83d54b603aecc5813c46f6a
Updating to version 2022.07.18 ...
[debug] Downloading yt-dlp from https://github.com/yt-dlp/yt-dlp/releases/download/2022.07.18/yt-dlp
[debug] Downloading SHA2-256SUMS from https://github.com/yt-dlp/yt-dlp/releases/download/2022.07.18/SHA2-256SUMS
Updated yt-dlp to version 2022.07.18
WARNING: Restart yt-dlp to use the updated version
[debug] [generic] Extracting URL: https://www.ina.fr/ina-eclaire-actu/arletty-carriere-conseils-actrice-marcel-carne
[generic] arletty-carriere-conseils-actrice-marcel-carne: Downloading webpage
WARNING: [generic] Falling back on generic information extractor.
[generic] arletty-carriere-conseils-actrice-marcel-carne: Extracting information
[debug] Looking for video embeds
ERROR: Unsupported URL: https://www.ina.fr/ina-eclaire-actu/arletty-carriere-conseils-actrice-marcel-carne
Traceback (most recent call last):
  File "./yt-dlp/yt_dlp/YoutubeDL.py", line 1431, in wrapper
  File "./yt-dlp/yt_dlp/YoutubeDL.py", line 1506, in __extract_info
  File "./yt-dlp/yt_dlp/extractor/common.py", line 645, in extract
  File "./yt-dlp/yt_dlp/extractor/generic.py", line 4108, in _real_extract
yt_dlp.utils.UnsupportedError: Unsupported URL: https://www.ina.fr/ina-eclaire-actu/arletty-carriere-conseils-actrice-marcel-carne

The webpage format is the same but the video_id has to be extracted from the page, eg from 'og:image' meta element.


Revert "Fix Media bar in Player Sample"

Reverts AvaloniaCommunity/Avalonia.FuncUI#90

it seems that I didn't merge the PR before starting to work on the docs stuff, so the partial stuff leaked in 😭 my bad


SSLs not being renewed

Hot on the heels of #261, it looks like none of the SSLs as being renewed. Some expire in the next few hours.
I cannot see any docs on having to run anything manually to do this but I would have assumed a background process would have done that.
Is there a command to run to force this (or make it happen)?

Can you try renewing in the console? postal console
RenewTrackCertificatesJob.perform

Should I get any output?
root:~# postal console
Loading production environment (Rails 5.0.2)
irb(main):001:0> RenewTrackCertificatesJob.perform
=> []
irb(main):002:0>

It does not appear to have done anything and they've expired now :(
Bar manually pasting in SSL keys/certificates in the database, is there any alternative?

Can you check the renew_after date in tour track_certificates table?

It looks like that date was after the current SSL expiry date. 😨
I popped the date to be before now so it would do something and well, this error might be why.
irb(main):001:0> RenewTrackCertificatesJob.perform
Faraday::ConnectionFailed: Connection reset by peer
	from /usr/lib/ruby/2.3.0/openssl/buffering.rb:178:in `sysread_nonblock'
	from /usr/lib/ruby/2.3.0/openssl/buffering.rb:178:in `read_nonblock'
	from /usr/lib/ruby/2.3.0/net/protocol.rb:154:in `rbuf_fill'
	from /usr/lib/ruby/2.3.0/net/protocol.rb:106:in `read'
	from /usr/lib/ruby/2.3.0/net/http/response.rb:291:in `block in read_body_0'
	from /usr/lib/ruby/2.3.0/net/http/response.rb:276:in `inflater'
	from /usr/lib/ruby/2.3.0/net/http/response.rb:281:in `read_body_0'
	from /usr/lib/ruby/2.3.0/net/http/response.rb:202:in `read_body'
	from /usr/lib/ruby/2.3.0/net/http/response.rb:227:in `body'
	from /usr/lib/ruby/2.3.0/net/http/response.rb:164:in `reading_body'
	from /usr/lib/ruby/2.3.0/net/http.rb:1445:in `transport_request'
	from /usr/lib/ruby/2.3.0/net/http.rb:1407:in `request'
	from /usr/lib/ruby/2.3.0/net/http.rb:1400:in `block in request'
	from /usr/lib/ruby/2.3.0/net/http.rb:853:in `start'
	from /usr/lib/ruby/2.3.0/net/http.rb:1398:in `request'
	from /opt/postal/vendor/bundle/ruby/2.3.0/gems/faraday-0.9.2/lib/faraday/adapter/net_http.rb:82:in `perform_request'
... 8 levels...
	from /opt/postal/vendor/bundle/ruby/2.3.0/gems/acme-client-0.5.5/lib/acme/client/resources/challenges/base.rb:13:in `verify_status'
	from /opt/postal/app/app/models/track_certificate.rb:57:in `verify'
	from /opt/postal/app/app/models/track_certificate.rb:35:in `get'
	from /opt/postal/app/app/jobs/renew_track_certificates_job.rb:6:in `block in perform'
	from /opt/postal/vendor/bundle/ruby/2.3.0/gems/activerecord-5.0.2/lib/active_record/relation/delegation.rb:38:in `each'
	from /opt/postal/vendor/bundle/ruby/2.3.0/gems/activerecord-5.0.2/lib/active_record/relation/delegation.rb:38:in `each'
	from /opt/postal/app/app/jobs/renew_track_certificates_job.rb:4:in `perform'
	from /opt/postal/app/lib/postal/job.rb:43:in `perform'
	from (irb):1
	from /opt/postal/vendor/bundle/ruby/2.3.0/gems/railties-5.0.2/lib/rails/commands/console.rb:65:in `start'
	from /opt/postal/vendor/bundle/ruby/2.3.0/gems/railties-5.0.2/lib/rails/commands/console_helper.rb:9:in `start'
	from /opt/postal/vendor/bundle/ruby/2.3.0/gems/railties-5.0.2/lib/rails/commands/commands_tasks.rb:78:in `console'
	from /opt/postal/vendor/bundle/ruby/2.3.0/gems/railties-5.0.2/lib/rails/commands/commands_tasks.rb:49:in `run_command!'
	from /opt/postal/vendor/bundle/ruby/2.3.0/gems/railties-5.0.2/lib/rails/commands.rb:18:in `<top (required)>'
	from bin/rails:4:in `require'
	from bin/rails:4:in `<main>'

Not sure where it is connecting to but this server has nothing outbound stopping it from connecting.

Looks like an error talking to Let's Encrypt. Not seen that before though.

A second time around it worked just fine.
Could the process get confused and consider that a success and then mark the database as such?

The renew_after date is set with the logic below. The expiry date comes from the certificate itself so in order for it to be in the future, it would have needed to have been renewed successfully.
self.renew_after = (self.expires_at - 1.month) + rand(10).days

Was the expires_at date in the table in the past or future?

The expires at was correct, but the renew after seemed to be in the past, after expiry. That may have just been a coincidence.
However, have found another issue which will have affected this.
CAA DNS record types apply to all hosts under the root domain - we had this set for the CA that we use.
Worth adding to the guide that if you use the same domain and protect the domain from only a set CA, LE won't issue anything (but won't complain either).
CAA 128 issue "letsencrypt.org"
Might be worth having the GUI check for and alert on that maybe? :)
Thanks for the help.

When I run this 'RenewTrackCertificatesJob' I get the following error in letsencrypt log file:
[27379] [2017-07-25T18:48:36.714] INFO -- : Error: Acme::Client::Error::Unauthorized (Must agree to subscriber agreement before any further actions)
[27379] [2017-07-25T18:48:37.692] INFO -- : Error: Acme::Client::Error::Unauthorized (Must agree to subscriber agreement before any further actions)
What could be wrong?

Guessing you didn't follow >> https://github.com/atech/postal/wiki/Click-&-Open-Tracking

also, running 'postal register-lets-encrypt ' is giving below:
"Faraday::ConnectionFailed: execution expired"

Looks like LE might be having issues.

Thanks matt. Do you also have any idea if we can ONLY track email opens without any URL in them? I am able to track open when email has a URL and user clicks on it but not simple emails without URL's.

Nope, but please don't cross post. Someone will reply on the other issue when possible no doubt.

Hello, when I try to perform "RenewTrackCertificatesJob.perform" from postal console I get this error, how could I debug this problem?
irb(main):004:0> RenewTrackCertificatesJob.perform
Faraday::ConnectionFailed: execution expired
from app/models/track_certificate.rb:35:in get'         from app/jobs/renew_track_certificates_job.rb:6:in block in perform'
from app/jobs/renew_track_certificates_job.rb:4:in perform'         from lib/postal/job.rb:43:in perform'
from (irb):4
thanks

@afanjul that means your computer was unable to talk to Lets Encrypt for whatever reason, I'd suggest trying again or upgrading to v2 where this is no longer necessary


link swift-nio-extras in docs

@swift-nio-bot test this please

@swift-nio-bot test this please

@weissi Seems like the Swift 5.2 tests are fundamentally broken here too.

@Lukasa this is the problem:
18:13:48 .FileNotFoundError: [Errno 2] No such file or directory: './docker/docker-compose.1804.52.yaml'

needs a PR like https://github.com/apple/swift-nio-extras/pull/86

5.3 compiler crash is expected


:heart_eyes: compatibility with pos_order_receipt_custom

@Ramil-Mukhametzyanov I fixed the problems

@Ramil-Mukhametzyanov This problem does not related to this PR


Not seeing the bolus - only carb corrections

Was this issue already discussed before?
It looks like i can only see the carbs correction and not the bolus? Any ideas why?

tl; dr; the issue is that the carbs are independently sent to diasend, decoupled from the bolus. This in turn can lead to the situation that the bridge does only see the carbs and not the bolus (which will only appear on diasend later on, when the carbs have already been processed) and send a carb correction to nightscout. Several attempts where made already (e.g. #15) to address this issue but the real solution will be as described in #18.

closing this as it's a duplicate of #18, please follow the progress there.


Backport for JQuery CVE-2019-11358 patch

Taken from:  https://github.com/DanielRuf/snyk-js-jquery-174006/blob/master/jquery-2.2.4.patch

Thanks for the PR. As per the README, development on Shins has ceased. The version of jQuery in the replacement, reSlate, should be up to date.

Thanks for the PR. As per the README, development on Shins has ceased. The version of jQuery in the replacement, reSlate, should be up to date.


Add Pipeline Source to request and response

This field is available since Gitlab 14.3, would be nice to have it supported here

@svanharmelen Will you merge it soon? Then I could spare a workaround in my own code :)


Bugfix amd64 libQt5Core.so.5

Closes #292.

There is still an issue with amd64 (see #295) but this is going to be merged to make sure at least this problem is solved for now.


VLAZ: Watch celebchitchat.com

VLAZ requests the watch of the watch_keyword celebchitchat\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by cigien in Charcoal HQ


Distortion on Mobile Safari

I've noticed problems on Mobile safari. On any Tone app I've tried, including the Tone.js examples, the first time a page loads the audio will be distorted. After reloading the page, everything works fine. This seems to only happen when using the internal speaker. I've tested it without issues via the headphone jack, Bluetooth and AirPlay.
I've experienced this on on an iPhone 6plus/iOS 9.3, iPhone 5s/iOS 9.3, and  iPad Mini 1/iOS 8.3. My iPad Air 2/iOS 9.3, however, works fine. Chrome on iOS also seems to be fine (which is strange since Apple forces all browsers to use their JavaScript engine...).

Perhaps a case of the iOS Samplerate bug? Can you check the sample rate of the context?
fix is here: https://github.com/Jam3/ios-safe-audio-context

Seems like more of a Safari / Web Audio bug than a Tone.js one. Maybe file a bug with Safari.


Final Polish on Quick Profiler

use "blue underline" from settings menu to indicate selected stat.
remove 'mid' value on y-axis of graph.
use image with rounded corners as background of graph.
move the chat window down & out of the way when stats are present.
be consistent with whitespace vs tabs (get rid of tabs).

Tabs to spaces:
That was deliberate.  Both files used a mix of tabs and spaces, and the diffs were a mess.
Why undo it?  We could go back and forth on whether tabs or spaces are better but having a mix is just confusing.
At this point, to restore the file to what it was would basically mean reverting all of my changes to those files and then adding back in the non-space changes by hand, which feels like a lot of work to ultimately leave the files in a worse condition (mixed) than they're currently in (consistent).
Shiny chat changes: I will talk to him.  I am hoping this could go live ASAP so unless his business is ready to go like today or Monday I'd like to go ahead with this change.


Create LICENSE

The name on line 3 is just what got autofilled when I put your repo into https://choosealicense.com/licenses/mit/ . Feel free to change it if you want.

@amagicmuffin Thanks for the commit!
I haven't yet merged it because I have one doubt. The sound assets are not made by me. Does the license apply to the whole of the repository? Or does it apply only to the code?
Also what about the GNU License? And, do I even need a license? lol

I think it would apply to the whole repository. GNU license is just a different license.
Licenses are to give permission for others to use your code legally. Also, I just learned about this like a few weeks ago lol so hopefully I'm not saying anything incorrect