Blazor mapsui breaks when using .NET9

Mapsui Version
4.x
Mapsui Platform
Blazor
Describe the bug
Attempting to use mapsui blazor with the host target framework set to .NET9 causes mapsui to throw a skia libs not found exception.
Additional context
After updating my project to .NET9, mapsui now breaking, seems the skia libs aren't being pulled in when using this target. Bit of a pain as it's blocking me updating to 9. Wondering if there's a work around I haven't thought of to manually pull the skia libs in myself?
@inforithmics

https://github.com/Mapsui/Mapsui/pull/2744
Herr is the pull request for .net 9 Support it is still WIP because it doesn‘t run on .net 9 yet. So I‘m investigaring.

Cool. Do you plan to patch support into the 4 branch? Would be handy while 5 remains in beta.

Pull request for Asp.NET 9 in SkiaSharp
https://github.com/mono/SkiaSharp/pull/3064


Potential use case: mocking internals

A thought I had, perhaps module attributes could be used to mock unexposed parts of modules. This is a serious current gap for testing and something we are trying to figure out how to best solve in node core... this would not have to be standardized in ecma262 imho, but would be a good one to explore
import feature from './feature.mjs' with {
  internal: {
    express: './path/to/mocked/express.mjs'
  }
}

edit:
dynamic example for completeness
const feature = import('./feature.mjs', {
  internal: {
    express: './path/to/mocked/express.mjs'
  }
});

that seems inconsistent (glhf if you forget to put the big mock override attribute on one of your imports) and kind of weird (why not just directly import the mocked file, since you're writing it down directly in static source text)

why not just directly import the mocked file, since you're writing it down directly in static source text

I don't think that would actually help mocking? If express.mjs exports a foo function that access disk, and feature.mjs imports and uses that, there's no way for me to mock that.

Mocking would have to happen at top-of-tree, and module attributes are scattered amidst the tree. I think this would either require a pre-graph loader, or spec-level syntax, totally unrelated to module attributes (iow, mocking is not a per-module concern, it's a per-graph concern, and module attributes are exclusively for per-module concerns)

I think we need a little bit more thought into this. I think mocking is not just replacing one file with another, but rather providing a way to inject full behavior when loading something.
If we consider:
const feature = import('./feature.mjs', {
  internal: {
    express: {
      exports: ['foobar', 'default'],
      async function load (args) {
        // this should be what is resolved to a module.

      }
    }
  }
});

iow, mocking is not a per-module concern, it's a per-graph concern, and module attributes are exclusively for per-module concerns

Can you please expand on this?
My main concern in making it a per-graph means that things could not be easily garbage-collected, which would be needed for the above to work.

@mcollina what i mean is, when you're mocking a specifier, say "lodash" - it's not automatically appropriate to mock all imports of lodash in the entire graph (because your mock might only meet the needs of some of the things depending on this version of lodash), nor is it automatically approrpriate to only mock imports of lodash in a specific subgraph (because you don't want duplicates in your browser bundle, for example, if your mock could be used to replace all instances of lodash in the tree).
Thus, the decision of what to mock, and for which subgraphs, can only be made with knowledge of the entire dependency graph - it's an app-level decision, not a decision that an individual module author can possibly make. Thus, code that goes inline into a module can not possibly meet, in a general sense, the use cases for mocking, because the module author isn't equipped to make that decision (for the cases where they are, they could simply import a different thing, without needing mocks)

I don't think it's necessary to mock the whole graph, but rather only what the single level module imports. From my point of view it's not a graph-level decision, but rather just at the top of the tree.

const feature = import('./feature.mjs', {
  modules: {
    'file://full/url/to/module.js': {
      get async resolve (args) {
        // this should be what is resolved to a module.
      }
    }
  }
});

When you do a dynamic import you want to be able to override any arbitrary module that might get imported as a result of this import. If the module you're trying to overload was previously loaded and cached, you would want it to be re-resolved for this dynamic import, as well as all modules who directly or indirectly would resolve this module.
For testing you need this, you will need that module to be mocked or not mocked by new instances on every import.


Consider not showing venvs from other directories

Testing #11755
I think this work would actually fall on me on the Controller level (so I don't expect you to have to do this), but putting this item here for consideration since the new Python API did expose this. One thing that I always liked about using venvs is that they were restricted to just the folder path that I had open. Now I'm getting way too many non-relevant venvs from all over my system.

I'm no python expert so I'm totally fine if folks disagree and we don't do this. It's just a downgrade for how I use Python + VS Code notebooks.

i like this feature

Looks like a bug in the Python Extension API

Upstream issue https://github.com/microsoft/vscode-python/issues/20068

Found that this is a change in the Python exension api and we need to filter the environments ourselves.

Found that this is a change in the Python exension api and we need to filter the environments ourselves.

Seems surprising that we would have to do it this way- isn't it expensive to collect venvs from all over the disk, just for us to ignore them?

collect venvs from all over the disk, just for us to ignore them?

i think they're caching them, agreed, could probably be expensive (more disc IO , process validations etc..),, but that's what ive been asked to do , that's to filter them)l
@karrtikr @karthiknadig /cc

Yep they're cached from previous session, so it's not expensive. Although as I suggested here  what we can probably do is only return environments in the current VSCode context (only for the currently opened workspaces).
Feel free to reopen https://github.com/microsoft/vscode-python/issues/20068 in that case.

current VSCode context (only for the currently opened workspaces).
Feel free to reopen microsoft/vscode-python#20068 in that case.

@karrtikr . if you think this is what other users of the api need then fine. note this is definitely not something we do not need. please feel free to ignore or change the api accordingly.
im assuming you've made this changes because you've felt other would need it, hence i can't tell you not to do this


Assembly is no longer Strong Named

Re-opening the issue raised in #336
Not supporting Strong Naming creates an issue for businesses or projects who use strong naming to enforce signed assembly verification, and/or who want to put their libraries in the GAC (they still exist :) ).
Pull request to resolve this, incoming.

Prior discussions:

https://github.com/FluentNHibernate/fluent-nhibernate/issues/295
https://github.com/FluentNHibernate/fluent-nhibernate/issues/342
https://github.com/FluentNHibernate/fluent-nhibernate/issues/336

Given the low activity of this project what are the consequences of not merging this? I'm in no position to merge this on behalf of anyone else here, so I'm reluctant to merge something people have been historically uncomfortable with.

Understood James, that's a reasonable concern. :)
The solution that is on PR #451 adds it as a separate project, so the main project remains unaffected and is not strongly signed, but there's a strong-signed version of FNH that can be published for people that need the strong-assembly-naming to be in place.
This is how StackExchange.Dapper does it. Hopefully that's enough of a compromise to bridge the gap between the two sides of the debate on this.
This comment from chester89 seems to indicate that the above implementation would be a preferable solution.


Argument error on Windows

When trying to validate xhtml on Windows using Syntastic it fails. I tracked it down to the file argument to tidy being sent with single quotes around it. I added this to xhtml.vim in syntax_checkers to fix the problem
    if has('win32') || has('win64')
      let makeprg="tidy ".encopt." -xml -e ".expand('%')
    else
      let makeprg="tidy ".encopt." -xml -e ".shellescape(expand('%'))
    endif

@M4T1A5 the reason these problems have not been fixed is because I currently dont have access to a windows box, and I want to get in and find out what is really going on. Removing the shellescape() call seems like it is fixing the symptom rather than the problem.
@mutewinter can you stick in an 'echo makeprg' and see what is actually getting set as the makeprg (without your custom mods)?
If this doesnt offer any clues then it could be an idea to set shellescape on windows and see if that works - if this turns out to be needed for windows then ill write a SyntasticShellEsc() function to set it on and off when doing the shellescape()

This problem seems to be gone on the latest version of Vim that I'm using. I'm currently using Wu Yongwei's pre-compiled gVim 7.3.333.
I'm also using the latest Syntastic.
Feel free to mark this as closed.

This problem seems to persist in the latest version for html.vim.
Thanks to the hints above, I fixed the issue by change " to ' and ' to " for win32 and win64
if has('win32') || has('win64')
    let makeprg='tidy '.encopt.' --new-blocklevel-tags "section, article, aside, hgroup, header, footer, nav, figure, figcaption" --new-inline-tags "video, audio, embed, mark, progress, meter, time, ruby, rt, rp, canvas, command, details, datalist" --new-empty-tags "wbr, keygen" -e '.shellescape(expand("%"))
else
    let makeprg="tidy ".encopt." --new-blocklevel-tags 'section, article, aside, hgroup, header, footer, nav, figure, figcaption' --new-inline-tags 'video, audio, embed, mark, progress, meter, time, ruby, rt, rp, canvas, command, details, datalist' --new-empty-tags 'wbr, keygen' -e ".shellescape(expand('%'))." 2>&1"
endif

Thanks for merging in my pull request with the cleaner solution. (BTW: It was my first pull request ever... so it feels good to have figured it out and get it accepted.) https://github.com/scrooloose/syntastic/commit/469e4efd0a5ebf6d7bf1cbd13112aab9aead6289
This issue appears resolved for xhtml.vim (as @mutewinter mentioned above) and html.vim

@darcyparker no worries - you definitely made it easy for me to accept :)
If anyone has any further issues with this then just comment back here and ill reopen.


Allow selection of dbITerm in the GUI

Description
If you click on an iterm the instance is selected today.  It would be good if you could directly select the iterm.
Suggested Solution
The existing instance selection rtree can be kept but then a secondary search inside the master to see if an mterm is selected should be made.  If one is found then the corresponding iterm can be found.
Additional Context
No response

I can also work on another issue if someone else was planning to do this one.

It's yours.  Thanks

I am almost ready to make a PR, I  just have a couple of questions.
In search.cpp, many functions start with a call to getData(), which returns either top_block_data_ or an element of child_block_data_, but I cant see where top_block_data_ is populated to begin with? This is a private field of the Search class, but it is never modified in search.cpp. Where is top_block_data_ set?
In layoutViewer.cpp on line 882 there is a comment "Look for an instance since no shape was found". I am not sure what is meant by this comment, because it seems everything up to this point in the selectAt() function involves searching for selected objects and adding them to a selections vector. What is the purpose of this comment?

@Iyury1 the comment on line 882 looks to be leftover from long ago when select would only pick the first item it encountered.


[FEATURE]: Unity XT: Storage Capacity tracking feature

Describe the solution you'd like
As part of implementing CSI spec 1.5 , we need to add support for Storage Capacity Tracking feature which helps k8s scheduler to spin off pods only on nodes which helps the pod to meet the storage space criteria as mentioned in  pod spec. If none of the nodes can provide the storage space required by pod, the pods remain in pending state and are not scheduled randomly
Additional context
This feature is available from k8s vesions >=1.24

Closing this issue as this is a duplicate.


Cannot read property 'slice' of undefined

Hi @joshcarr ,
I'm using the minimalist feature of the filter like
<StructuredFilter
	placeholder="Search..."
	options={[
		{category:"Name",type:"text"},
		{category:"Price",type:"number"},
	]}
/>

and I'm getting the error
Uncaught TypeError: Cannot read property 'slice' of undefined
getStateFromProps
Tokenizer
constructComponentWithoutOwner

Could you please help.

addressed this in joshcarr/react-structured-filter/issues/7


[multithreading] Passing args/kwargs to a worker callback function

🚀 Feature
I'd like to be able to pass arguments to a callback function that I've connected to. This should work for at least the following APIs:

worker.{signal}.connect(callback)
@thread_worker(connect={signal: callback})
(since the above two APIs seem to be the most high-level + easy-to-use)

Motivation
e.g., if my callback is viewer.add_image and I want to pass blending='additive' to the callack call
Pitch
Alternatives
an easy way to implement this is to simply add *args and **kwargs where appropriate and pass these when calling the callback. I haven't thought about this much so I'm not yet sure if I would actually like this API (I'm a little hesitant about the *args part; it might cause confusion/decrease code readability)
Additional context
n/a

hi @sumanthratna!  do you happen to have a code example of what you're trying to do?  I looked through viewmask to see where you might be trying to apply this, but couldn't find it.
Currently the thread_worker API has a fixed set of callbacks, like returned or yielded which are called with the value returned or yielded by the decorated function.  By definition, these things will always be a single value (since functions can only return/yield a single value... though it can be a complex one, like a dict or a tuple).  So generally, the best way to accomplish what you're trying to do here is to have your worker yield or return a dict or a tuple, and then make a simple callback function that will unwrap those into the "real" callback that you want.  As an example:
# this works fine if you only have a single argument
@thread_worker(connect={'returned': viewer.add_image})
def make_image():
    return np.random.rand(512,512)

# but if you want something more complex
def my_callback(value):
    viewer.add_image(**value)

@thread_worker(connect={'returned': my_callback})
def make_image_with_parameters():
    return {'data': np.random.rand(512,512), 'blending': additive}

does that accomplish what you're looking to do?

hi @sumanthratna!  do you happen to have a code example of what you're trying to do?  I looked through viewmask to see where you might be trying to apply this, but couldn't find it.
Currently the thread_worker API has a fixed set of callbacks, like returned or yielded which are called with the value returned or yielded by the decorated function.  By definition, these things will always be a single value (since functions can only return/yield a single value... though it can be a complex one, like a dict or a tuple).  So generally, the best way to accomplish what you're trying to do here is to have your worker yield or return a dict or a tuple, and then make a simple callback function that will unwrap those into the "real" callback that you want.  As an example:
# this works fine if you only have a single argument
@thread_worker(connect={'returned': viewer.add_image})
def make_image():
    return np.random.rand(512,512)

# but if you want something more complex
def my_callback(value):
    viewer.add_image(**value)

@thread_worker(connect={'returned': my_callback})
def make_image_with_parameters():
    return {'data': np.random.rand(512,512), 'blending': additive}

does that accomplish what you're looking to do?

hi @sumanthratna! do you happen to have a code example of what you're trying to do? I looked through viewmask to see where you might be trying to apply this, but couldn't find it.

I haven't yet started implementing async or multithreading, but out of my own interest I wanted to experiment with it by using multithreading to add multiple objects to the napari viewer concurrently. e.g., adding a shapes layer, a points layer, and an image layer in viewmask

does that accomplish what you're looking to do?

it does! When I thought of this issue I was thinking of functools.partial but it seems very verbose. If you see the "Alternatives" section in my original post, that would look something like this:
make_image_with_parameters = thread_worker(
    make_image,
    connect={'returned': viewer.add_image},
    blending='additive',
)

I admit, the more I think about this, the more ugly it looks. I'll close this issue since I think it's safe to stick with verbosity, so I'll go with what you suggested. Thank you @tlambert03!

hi @sumanthratna! do you happen to have a code example of what you're trying to do? I looked through viewmask to see where you might be trying to apply this, but couldn't find it.

I haven't yet started implementing async or multithreading, but out of my own interest I wanted to experiment with it by using multithreading to add multiple objects to the napari viewer concurrently. e.g., adding a shapes layer, a points layer, and an image layer in viewmask

does that accomplish what you're looking to do?

it does! When I thought of this issue I was thinking of functools.partial but it seems very verbose. If you see the "Alternatives" section in my original post, that would look something like this:
make_image_with_parameters = thread_worker(
    make_image,
    connect={'returned': viewer.add_image},
    blending='additive',
)

I admit, the more I think about this, the more ugly it looks. I'll close this issue since I think it's safe to stick with verbosity, so I'll go with what you suggested. Thank you @tlambert03!

something like this:
make_image_with_parameters = thread_worker(
   make_image,
   connect={'returned': viewer.add_image},
   blending='additive',
)

ahh, I see. yes in that case (if you always wanted your callback called with blending=additive) then I'd say partial is a very good solution there:  connect={'returned': partial(viewer.add_image, blending='additive')}

something like this:
make_image_with_parameters = thread_worker(
   make_image,
   connect={'returned': viewer.add_image},
   blending='additive',
)

ahh, I see. yes in that case (if you always wanted your callback called with blending=additive) then I'd say partial is a very good solution there:  connect={'returned': partial(viewer.add_image, blending='additive')}


Merge develop to serenity (#9)

Update ci.yml

fix deployment ts (#8)

Update ci.yml

Co-authored-by: kienvc<EMAIL_ADDRESS>

@doquockhanhan : Tuần sau merge


Texture setting support

Adding support for the channel's filter & wrap modes would be fantastic 👍

Yes, I agree this would be a good feature. I have difficulties though imagining a good UX for this, considering we try to keep all setup contained in the shader. A couple of possibilities:

Specify filter and/or wrap mode with the definition of the channel, e.g.

#iChannel0 "file://./tex.png?wrap=clamp&filter=nearest"

Specify filter and/or wrap separatedly from the channel definition:

#iChannel0 "file://./tex.png"
#iChannel0Wrap "clamp"
#iChannel0Filter "nearest"

I like option 1 because there are libraries readily available to deal with it and it keeps the clutter minimal.
I dislike it though because it is difficult to parse for a human.
I like option 2 on the other hand because imho it is easier to parse for a human and makes it easy to disable by simply putting a comment.
I dislike it because honestly, something just feels off about it. It also gives a wrong impression, that iChannel0Wrap would be an available variable. And lastly it complicates the code to parse this.
It's pretty obvious that all of this is pretty subjective, so I'd be happy to hear your thoughts on it, or hear if you have other suggestions on how to do this.

Yep! Completely agree. I prefer something similar to Option 2 for the sake of being able to quickly update settings without breaking the compiler. I considered using structs for a moment, but that defeats why the channels are defined in the first place... so, whatever you prefer :)

🙌


Contradiction in marpa_v_step() return value summary

When the iteration through the steps is finished, marpa_v_step returns MARPA_STEP_INACTIVE.
Return value: On success, the type of the step to be performed. This will always be a
non-negative number. On failure, -2.

MARPA_STEP_INACTIVE is neither a type of step to be performed, nor is it -2.

Commit https://github.com/jeffreykegler/libmarpa/commit/6b03234ebabadcd2ef1d15953abc1806d8fa73de fixes this.  I'm in the process of rewriting the whole doc, and there are other cleanups I will do in the text nearby, but I will leave those until they occur in order.
Absent feedback to the contrary, I will close this issue in a couple of days.

Closed per https://github.com/jeffreykegler/libmarpa/issues/86#issuecomment-1120407186.


Added ssl configuration .

Added relevant ssl enabled configuration which can be set either via clowder or by using env

Closing stale PRs


Generated pom.xml is bogus about Maven imports

it generates:
  <dependency>
    <groupId>javax.ws</groupId>
    <artifactId>rs:jsr311-api</artifactId>
    <version>1.1.1</version>
  </dependency>

Instead of the obvious.

Fixed.


[Work in progress] add support for sending empty payload frame (gateway issue #254)

Changed just the write() method of the AbstractIoSession class in the mina.netty module to not throw IllegalArgumentException, as debugging revelead that this class is the one being used, and not the one in the mina.core module.
Should I also perform the change in the AbstractIoSession from mina.core? When is one class used over the other? Also, what's the purpose of the mina.build module?
I removed the @Ignore annotations from impacted ITs and they all pass.

@danibusu I think we only use the AbstractIoSession from mina-netty.

@jitsni I pasted the stack trace below:

Thread [main] (Suspended (breakpoint at line 430 in AbstractIoSession))
	WsnSession(AbstractIoSession).write(Object, SocketAddress) line: 430
	WsnSession(AbstractIoSession).write(Object) line: 391
	BaseFramingIT.shouldEchoBinaryFrameWithPayloadLength0() line: 97
	NativeMethodAccessorImpl.invoke0(Method, Object, Object[]) line: not available [native method]
	NativeMethodAccessorImpl.invoke(Object, Object[]) line: 62
	DelegatingMethodAccessorImpl.invoke(Object, Object[]) line: 43
	Method.invoke(Object, Object...) line: 498
	FrameworkMethod$1.runReflectiveCall() line: 50
	FrameworkMethod$1(ReflectiveCallable).run() line: 12
	FrameworkMethod.invokeExplosively(Object, Object...) line: 47
	InvokeMethod.evaluate() line: 17
	JUnitRuleMockery$1.evaluate() line: 49
	SpecificationStatement.evaluate() line: 63
	Verifier$1.evaluate() line: 35
	WsnConnectorRule$ConnectorStatement.evaluate() line: 134
	TestWatcher$1.evaluate() line: 55
	RunRules.evaluate() line: 20
	BlockJUnit4ClassRunner(ParentRunner).runLeaf(Statement, Description, RunNotifier) line: 325
	BlockJUnit4ClassRunner.runChild(FrameworkMethod, RunNotifier) line: 78
	BlockJUnit4ClassRunner.runChild(Object, RunNotifier) line: 57
	ParentRunner$3.run() line: 290
	ParentRunner$1.schedule(Runnable) line: 71
	BlockJUnit4ClassRunner(ParentRunner).runChildren(RunNotifier) line: 288
	ParentRunner.access$000(ParentRunner, RunNotifier) line: 58
	ParentRunner$2.evaluate() line: 268
	BlockJUnit4ClassRunner(ParentRunner).run(RunNotifier) line: 363
	JUnit4TestReference.run(TestExecution) line: 86
	TestExecution.run(ITestReference[]) line: 38
	RemoteTestRunner.runTests(String[], String, TestExecution) line: 459
	RemoteTestRunner.runTests(TestExecution) line: 675
	RemoteTestRunner.run() line: 382
	RemoteTestRunner.main(String[]) line: 192

I am thinking of updating the WsnConnector and WsnAcceptor classes and whenever a new WsnSession is created, also set a session attribute like "wsFlag" to true. And then, in the AbstractIoSession.write() method, throw the IllegalArgumentException only if the flag is not set. Do you think this approach is the right one?
I'm not sure about removing the try-catch block. For example, the FileInputStream constructor called in one of the if  blocks may throw FileNotFoundException.

One option is to create a method void validateWriteMessage((Object message) with default behaviour.  WsnSession overrides this method to allow empty message. But seems like heavy weight.
I think the current change is ok. Just remove the whole try block. I don't we use File or FileChannel as message.

@jitsni Updated pull request to incorporate review comments. Thanks!


Declarative Expected Events (errors) in End to End Test

This change enables a new test function, itExpects which works like mocha's it but it allows expected log events to be declaratively specified. This works via a new logger, the ErrorTrackingLogger which is now used in the TestObjectProvider.
In this change i've also clean up some of the test that are currently polluting our errors table in expected errors.
related to #8942

This is cool! Exactly what I had in mind thinking about how to fail a test if an error log gets hit. I'm a little confused about how broadly applicable this is that I want to get cleared up, specifically the fact that it seems coupled to declareCompat.  One of the primary motivations is to get expected error out of our end to end test telemetry. it could maybe be made more generic, but not trying to do that here


PureWindowsPath validation breaks mdocfile for linux

Hi folks - not being familiar with the codebase I'm reluctant to make changes, but I had to replace PureWindowsPath with PurePosixPath throughout data_models.py for it to work on Linux (needless to say this will break it for Windows).

Hey Morgan, thanks for the report!
@shahpnmlab any thoughts on this? I know you pushed for the change to WindowsPath

I just tried this out (on MacOS, not linux) and don't run into the issue when constructing from a string
In [1]: from mdocfile.data_models import MdocSectionData

In [2]: MdocSectionData(SubFramePath='bla.tif')
Out[2]: MdocSectionData(ZValue=None, MontSection=None, FrameSet=None, TiltAngle=None, PieceCoordinates=None, StagePosition=None, StageZ=None, Magnification=None, CameraLength=None, MagIndex=None, Intensity=None, SuperMontCoords=None, PixelSpacing=None, ExposureDose=None, DoseRate=None, SpotSize=None, Defocus=None, TargetDefocus=None, ImageShift=None, RotationAngle=None, ExposureTime=None, Binning=None, UsingCDS=None, CameraIndex=None, DividedBy2=None, LowDoseConSet=None, MinMaxMean=None, PriorRecordDose=None, XedgeDxy=None, YedgeDxy=None, XedgeDxyVS=None, YedgeDxyVS=None, StageOffsets=None, AlignedPieceCoords=None, AlignedPieceCoordsVS=None, SubFramePath=PureWindowsPath('bla.tif'), NumSubFrames=None, FrameDosesAndNumbers=None, DateTime=None, NavigatorLabel=None, FilterSlitAndLoss=None, ChannelName=None, MultiShotHoleAndPosition=None, CameraPixelSize=None, Voltage=None)

i.e. I can construct the section data and the path is coerced to a Windows path from a string
I see the issue when constructing from a path though and agree that should work
In [3]: MdocSectionData(SubFramePath=Path('bla.tif'))
---------------------------------------------------------------------------
ValidationError                           Traceback (most recent call last)
Cell In[3], line 1
----> 1 MdocSectionData(SubFramePath=Path('bla.tif'))

File ~/mambaforge/envs/mdocfile/lib/python3.10/site-packages/pydantic/main.py:176, in BaseModel.__init__(self, **data)
    174 # `__tracebackhide__` tells pytest and some other tools to omit this function from tracebacks
    175 __tracebackhide__ = True
--> 176 self.__pydantic_validator__.validate_python(data, self_instance=self)

ValidationError: 1 validation error for MdocSectionData
SubFramePath
  Input is not a valid path [type=path_type, input_value=PosixPath('bla.tif'), input_type=PosixPath]

I have an idea how I might fix it so this works as expected for the future, glad you got yourself sorted for now :)

Confirmed this works on Linux now - thanks!


[Playable Embeds] Add soundcloud embeds

https://help.soundcloud.com/hc/en-us/articles/115003568008-Embedding-a-track-or-playlist-
yes

you will pr

no because i dont know kotlin

Message ID: @.***>


OW1 Ana

Created by: dubspool
Healing from 75 -> 70
Ammo from 15 -> 12
Biotic Grenade duration effect from 3.5 seconds -> 4 seconds

moved from ow1_emulator_backup#615

Created by: MaxwellJung
Closed because first two issues are resolved, third issue will be made as seperate issue.


Task 1 Confusion

Hi group 3,
Who is doing task 1 of homework 5? Please give a brief overview about task 1. I am confused what exactly we will be doing on it?

@pratibha77118 I followed the hands-on by professor for task 1

I think that should be @s4il3sh  @pratibha77118

@pratibha77118 For task 1, we need to do
Use Fast Fourier Transform (FFT) to analyze CO2 concentration data:
- Calculated actual frequencies from the FFT output in cycles per month, if I were not wrong for the unit.
- Identified peak frequencies in the data by analyzing the FFT results.


classic question: make pock installable via brew cask ;)

I close this issue because it is a duplicate of #31


[Release] Bumped teamcity version to 3.0.0

What does this PR do?
Release TeamCity
Motivation

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

🎉


Weather conditions component & others: Refactoring

Logically segregated the components into different files
Solved the faded weather edit modal issue
LocationConfirmation.js file: Removed extra ternaries
CropSelectorCalendarView.js file: Segregated out the paint growth component

Your review app has been deployed and is available at 'http://,<IP_ADDRESS>,:,80,'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://,<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

editing frost free days is broken

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.


Two CUDA memory issues with Faster/Mask R-CNN

When using CUDA10:
RuntimeError: /pytorch/torch/csrc/jit/fuser/cuda/fused_kernel.cpp:202: a PTX JIT compilation failed
I solved it by switching to CUDA9.0, but then another one pops up:
`RuntimeError: CUDA out of memory. Tried to allocate 54.00 MiB (GPU 0; 14.73 GiB total capacity; 13.64 GiB already allocated; 27.88 MiB free; 279.23 MiB cached)

`
The don't point to a specific error, so I can't track it. Also some applications using Mask/Faster R-CNN actually work correctly. What can I do?

Hi,
Is this when using the torchscripted version of maskrcnn?
The models in faster rcnn use a lot of memory, specially during training, so it would be good to know what you are trying to do (training / inference) so that we know a bit more what the problem is.

This issue appears to be stale, so I'm closing it. If you think it should remain open, feel free to reopen it.

This issue appears to be stale, so I'm closing it. If you think it should remain open, feel free to reopen it.


[UX] Improve UX with Navbar

Currently when user is on 'PIXABAY' Gallery page and refreshes the page, app will display 'GIPHY' page as default.
It would be nice to stay in the same page when user refreshes.

I think a good way is to add a router. For example /pixabay goes to pixabay and / goes to giphy. Or if you don't want to add a router we could set a item in localStorage. What option do you want?

@navaneeth-dev that was my initial thought too. I think '/pixabay' goes pixabay, '/' and '/giphy' goes to giphy.


fix: invalid iam profile for launch template

Fixes #111

I can confirm the role is set correctly now.

I can confirm the role is set correctly now.


More user interactive service with feedback

Each article should have it's own feedback component
Example implementation
From answers.microsoft
Rating for articles

Reporting articles or links (old data,  irrevelant materials, invalid links)

What problems we want to solve

We want to engage more people into maintaining bewebdev.tech
People usually are too lazy to create an issue on github repo to inform us about bugs

Expected solution
Report feature for each link - sends request to github and creates issue with apriopriate labels [auto report] some labels can be added when user selects them in the form on the website.
Rating system - each article should have visible rating, maybe only thumb up or heart icon, thumb down should open report window maybe?

Pretty cool idea. But then we need some way to notify us about the feedback.

You can be notified about newly created issues on github.


License headers should not show up on module-level docs

Please elaborate and provide an example of where you're seeing this.

https://jsr.io/@mtkruto/mtkruto/doc/~/MtkrutoError

No symbol has this doc.

This behaviour is expected. deno_doc sees this text as a plain JSDoc description. Currently, the way to do this is to prefix each line of the comment with // instead of wrapping it in /** ... */. Either way, supporting the @license JSDoc tag would be nice if someone wants to display their license.

Did I understand this wrongly or this module-level doc used to be explicitly enabled with @module?

Yes, I do believe the @module tag is required to denote module-level docs. But do not confuse what you shared (https://jsr.io/@mtkruto/mtkruto/doc/~/MtkrutoError) as a module doc. It's a symbol doc, for the MtkrutoError class.

Oh, I just understood.


allow to sort by $meta-projection

Small fix to allow sorting by meta-projections. This is needed for full-text search and sort results by relevancy. For example:
$query = Models\News::whereRaw([
   '$text' => [ '$search' => $request->Input('q') ]
])->project([ 'score' => [ '$meta' => 'textScore' ] ]);

$query->orderBy('score', [ '$meta' => 'textScore' ]);

http://docs.mongodb.org/manual/reference/operator/query/text/

:+1:


Unable to inherit 'SqlScriptGenerator' class

Is your feature request related to a problem? Please describe.
Consider the following code to inherit SqlScriptGenerator
internal class CustomSqlScriptGenerator : SqlScriptGenerator
{
	public CustomSqlScriptGenerator(SqlScriptGeneratorOptions options) : base(options)
	{
	}

	internal override SqlScriptGeneratorVisitor CreateSqlScriptGeneratorVisitor(SqlScriptGeneratorOptions options, ScriptWriter scriptWriter)
	{
		throw new NotImplementedException();
	}
}

However, this will get compilation error because SqlScriptGeneratorVisitor  and ScriptWriter are marked as internal, which is  impossible to inherit this class.
Describe the solution you'd like

Change CreateSqlScriptGeneratorVisitor(...) become protected instead of internal
Remove ScriptWriter parameter in CreateSqlScriptGeneratorVisitor(...). This can avoid to mark ScriptWriter as public. Instantiation of ScriptWriter will be handled by child class of SqlScriptGenerator. See example below.
Mark SqlScriptGeneratorVisitor as public accessible. SqlScriptGeneratorVisitor will be used to visit the expression tree when generating TSQL script.

Here is the suggested changes:
protected abstract SqlScriptGeneratorVisitor CreateSqlScriptGeneratorVisitor(SqlScriptGeneratorOptions options);

And update Sql###ScriptGenerator become
Example:
public sealed class Sql100ScriptGenerator : SqlScriptGenerator
{
    public Sql100ScriptGenerator()
        : this(new SqlScriptGeneratorOptions())
    {
    }

    public Sql100ScriptGenerator(SqlScriptGeneratorOptions options)
        : base(options)
    {
    }

    protected override SqlScriptGeneratorVisitor CreateSqlScriptGeneratorVisitor(SqlScriptGeneratorOptions options)
    {
        // Create instance of ScriptWriter.
        //Currently this is created by private function in SqlScriptGenerator
        ScriptWriter scriptWriter = new ScriptWriter(options);

        ScriptGeneratorSupporter.CheckForNullReference((object) options, nameof (options));
        ScriptGeneratorSupporter.CheckForNullReference((object) scriptWriter, nameof (scriptWriter));
        return (SqlScriptGeneratorVisitor) new Sql100ScriptGeneratorVisitor(options, scriptWriter);
    }
}

I believe this is the line of code in question:
https://github.com/microsoft/SqlScriptDOM/blob/4b7e0af48237191f14612855acda32c97470fe02/SqlScriptDom/ScriptDom/SqlServer/SqlScriptGenerator.cs#L116


Workflow transition restriction

Hello,
how can I restrict workflow transitions that for example one could change the state from New to Active, but changing from New directly to Resolved, thus bypassing Active, would be forbidden?
https://docs.microsoft.com/en-us/azure/devops/organizations/settings/work/customize-process-workflow

@kubsztal Thank you for the question.
Manually this isn't feasible, however for a custom approach you can use the Work item updated service hook, see WebHooks.
There you would need to run a service that compares the work-item state.  PowerShell's a good candidate for this since you can then use the Azure CLI work-item command.

Thanks, I will check this!


[discussion] Performance - Postgres via UDS

Hi guys,
I am considering some future work during this year to look at, nothing so much important, but related to overall performance, of the light4j platform as well as possible improvement. Although this is standard in JDBC world, one must know that TCP overhead is deadly. So why not replace it with UDS (Unix Domain Sockets) instead?
You can make your small test by yourself, just execute make:
https://github.com/archenroot/ipc-bench
I plan to test some use case:

HARDER with discuss-able effect especially when HA is required- Event store is not build via db clustering capabilities but instead just on 2-4 independent nodes and consume from event topic new events and persist them (I understand I am will need to handle error scenarios), but either I will use kafka or chronicle queue (it does not support in the moment network replication) for higher throughput, both can be seen as buffers, so when 1 node fails, instead of trying to identify and fix, you can initiate fast redeploy and reprocess buffered messages from T - 1 minute (T - time of failure)
- EASIER with real effect - use it for READ only schemas used with CQRS pattern in place, this could have significant performance improvement for heavy READ consuming applications. You can really boost performance here ( I come from background we were handling over 10k requests per second for example), so UDS with PostgreSQL remaining on disk mounted to ram. While postgres supports this natively, I think it is wasted time and better to do it on OS level by creating tmpfs mount and put data files there.

I will leave this open as soon as I am going to possibly incorporate some work old few years, but still not finished (but the POC is fully operational and you can test it on local instances by yourself):
https://github.com/impossibl/pgjdbc-ng/issues/169
The fork with POC:
https://github.com/harbulot/pgjdbc-ng/tree/unixsocket_poc
Just again came to my head as possible improvement to make light-4j real warrior on all levels of operation.
Ladislav

A lot of material to read as I never paid attention on UDS before. I don't mind to improve the performance while limiting to Linux as most cloud environments are in Linux anyway. Also, some new shared kernel container techniques will make UDS across containers which is very crucial. I don't like the VMs and I would deployed container on bare metal if possible to remove the extra layer. Security won't be the issue if the host is not shared.
I am planning to start light-rs which is rust based framework these days to aim another group of customers. This year will be a lot of fn:)

@stevehu - Yeah, I actually use VM in headless mode just to access pure windows apps on linux desktop machines, but when it comes to backend in DC/Cloud, containers are the only option in these days, while I see it still as incubating technologies not serving full potential yet.
Hm, thx for pointing RUST here out, as I am not skilled to code in that language, but it already also took my attention multiple times in the past, I will need to write some real code and compare against java/scala to see how it looks, and more how it works. Even when I read some general description it looks quite promising, thx for hint.... GC per task, not enforced UTF-16, native code, heavy parallel (my personal interest), and much more....
In general the only issue with UDS is that drivers usually implement TCP based (but can differ on db type) communication.

I would like to try the Postgres with UDS driver some time in the future for our own services. If it works between containers, that would be an option for other service to service communication.

@stevehu - yeah.


Training details

Awesome work!
Will you share the training or fintuning code?

Thank you for your insterest, please see the closed issue #2
"Our method is training-free, you can implement this mechanism in any model."

Thank you for your insterest, please see the closed issue #2 "Our method is training-free, you can implement this mechanism in any model."

Thank you for the reply.
May I know what is the baseline of model part?
It seems like MiniGPT-4. In the paper it illustrates a lot of MLLM.

We build our model based on video-llama, since it is a simple but strong video-based MLLM.

Thank you.


Unavailable tcp-services upstream results in 1000s of SYN retries from ingress-nginx-controller and 1000s of correlating log messages in ingress-nginx-controller logs

What kind of request is this (question/bug/enhancement/feature request):
Bug?
Steps to reproduce (least amount of steps as possible):

Provision a Rancher v2.4.5 lab with a downstream cluster (can be single node)
Add a simple nginx Deployment in the default Namespace, a ClusterIP service for port 80 of this workload, and add 8080=default/nginx-service:80 to the tcp-services ConfigMap in the ingress-nginx namespace.
Start a tcpdump on the single node in the downstream cluster tcpdump -pni any port 80 or 8080
Perform looping curl to the service while true; do date --iso-8601=seconds && curl http://<downstream-node-ip>:8080/; done
Scale down the nginx-deployment to 0.

Result:
Observe thousands of messages of the following format within timeframe of c. 1s:
2020-08-26T13:05:13.774467635Z 2020/08/26 13:05:13 [error] 112#112: *6226 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: <IP_ADDRESS>:8080, upstream: "<IP_ADDRESS>:80", bytes from/to client:0/0, bytes from/to upstream:0/0

Before service marked as unavailable:
2020-08-26T13:05:17.001630840Z W0826 13:05:17.001409       6 controller.go:348] Service "default/nginx-service" does not have any active Endpoint for TCP port 80

The tcpdump will also show thousand of SYN packets from the ingress-nginx-controller to the upstream nginx Pod IP, which each receive a RST, ACK.
Other details that may be helpful:
The same behaviour is not shown if you test this but using a simple HTTP ingress Resource.
The upstream issue https://github.com/kubernetes/ingress-nginx/issues/5425 would appear relevant, and the linked nginx issue https://trac.nginx.org/nginx/ticket/2026. In the nginx issue it is asserted that the retries must be originating from the downstream client. However, my testing with packet captures indicates this is not the case and the very high number of requests originate from the ingress-nginx-controller itself after a single request from the client.
Attached is a PCAP (nginx.zip) of the behaviour, alongside log collector output (aseymour-rancheragent-all-0-2020-08-26_14_03_16.tar.gz and aseymour-rancheragent-all-1-2020-08-26_14_03_33.tar.gz) from my reproducer lab. My client IP for the tests was <IP_ADDRESS>, the host I tested connecting to was aseymour-rancheragent-all-0 IP <IP_ADDRESS>, and the upstream nginx Pod IP during my test was <IP_ADDRESS>.
The packet capture shows the a HTTP request in frame 2627, then between frames 2629 and 11804 over 4000 SYN packets sent from the node to the nginx upstream Pod, all receiving a RST, ACK, as a result of the now scaled down nginx Deployment. The nginx-ingress-controller Pod logs from the log collector output on the host, show the 1000s of Connection refused errors in the same window.
Environment information

I reproduced with Rancher v2.4.5, k8s v1.18.6, ingress-nginx v0.32.0
User reported issue with Rancher v2.3.3, k8s v1.16.3, ingress-nginx v0.25.1

gz#11741

Tested and observed the same behaviour with rancher/nginx-ingress-controller:nginx-0.35.0-rancher2 and k8s.gcr.io/ingress-nginx/controller:v0.41.0

Team re-tested with Rancher v2.7.0 / downstream RKE v1.24.8-rancher1-1 / nginx-ingress-controller:nginx-1.2.1-rancher1 and could not reproduce the same behavior, so I think we can close this one out.


Error while cloning site tree into another language

If you want to copy a site tree into another language and your site contains some pages aliases you will get an error for each alias page:
 An exception occurred while executing 'insert into MultilingualPageRelations (mpRelationID, cID, mpLocale, mpLanguage) values (?, ?, ?, ?)' with params ["15", "615", "de_DE", "de"]: SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '15-615' for key 'PRIMARY'
error source: /concrete/src/Multilingual/Page/Section::349
version: 8.4.4

@bitterdev isn't this the same issue as #7781 ?

Yes double post sorry. First i got an empty response from github.


Value Changed Action

Since NapySlider is subclass of UIControl, can you please implement valueChanged action?

and again pod update
happy coding!!!

app crashes on value change
//            let position = Int((positionFromMin * step + min + stepOffset) / step) * Int(step)
            let position = Double(Int((positionFromMin * step + min + stepOffset) / step)) * step
            return Double(position)

crash log
fatal error: Double value cannot be converted to Int because it is either infinite or NaN
2017-06-27 17:10:35.171049+0300 xxx[1529:1244608] fatal error: Double value cannot be converted to Int because it is either infinite or NaN

The blame is on positionFromMin which equals NaN

hmm my fault, seems like a bug in my code set the max and the min to 0

I confirm, when min equals max, it will crash. see #12. I fix that somedays.


build.yml: Added jobs to speed up CI checks.

Summary
Divided jobs risc-v and xtensa to finish workflow under 2 hours.
depends on this PR https://github.com/apache/nuttx/pull/12793
Impact
speeds up workflow
Testing
CI

let's merge this patch directly to unblock ci.


Won't start when I re-create deployment

Hi, I'm playing around with this and after deleting and recreating the etcd cluster I'm getting the following error logs:
time="2016-11-20T11:56:56Z" level=info msg="use in cluster client from k8s library"
time="2016-11-20T11:56:56Z" level=info msg="finding existing clusters..." pkg=controller
time="2016-11-20T11:56:56Z" level=info msg="starts running from watch version: 57524" pkg=controller
time="2016-11-20T11:56:56Z" level=info msg="start watching at 57524" pkg=controller
time="2016-11-20T11:57:01Z" level=warning msg="all etcd pods are dead. Trying to recover from a previous backup" cluster-name=etcd-cluster pkg=cluster
time="2016-11-20T11:57:01Z" level=error msg="fail to do disaster recovery: no backup policy has been defined." cluster-name=etcd-cluster pkg=cluster
time="2016-11-20T11:57:01Z" level=info msg="deleting cluster" cluster-name=etcd-cluster pkg=cluster
panic: TODO: mark cluster dead if no backup for disaster recovery.

goroutine 34 [running]:
panic(0x1878380, 0xc420292ce0)
	/usr/local/go/src/runtime/panic.go:500 +0x1a1
github.com/coreos/etcd-operator/pkg/cluster.(*Cluster).run(0xc4202df8f0, 0xc4203c5c80, 0xc4203c5910)
	/home/ubuntu/code/golang/src/github.com/coreos/etcd-operator/pkg/cluster/cluster.go:183 +0x63a
created by github.com/coreos/etcd-operator/pkg/cluster.new
	/home/ubuntu/code/golang/src/github.com/coreos/etcd-operator/pkg/cluster/cluster.go:119 +0x34e

No idea what's going on.
My manifest is as following:
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: etcd-operator
spec:
  replicas: 1
  template:
    metadata:
      labels:
        name: etcd-operator
    spec:
      containers:
      - name: etcd-operator
        image: quay.io/coreos/etcd-operator
        env:
        - name: MY_POD_NAMESPACE
          valueFrom:
            fieldRef:
              fieldPath: metadata.namespace
---
apiVersion: "coreos.com/v1"
kind: "EtcdCluster"
metadata:
  name: "etcd-cluster"
spec:
  size: 3
  version: "v3.1.0-alpha.1"

panic: TODO: mark cluster dead if no backup for disaster recovery.

It's a TODO.
Would you like to implement it?

@maxott @hongchaodeng The first step is to remove this panic and print out unrecoverable case as an error?

@maxott

No idea what's going on.

This log is not clear? What is your expectation? Let's see how to improve this.
time="2016-11-20T11:57:01Z" level=warning msg="all etcd pods are dead. Trying to recover from a previous backup" cluster-name=etcd-cluster pkg=cluster
time="2016-11-20T11:57:01Z" level=error msg="fail to do disaster recovery: no backup policy has been defined." cluster-name=etcd-cluster pkg=cluster
time="2016-11-20T11:57:01Z" level=info msg="deleting cluster" cluster-name=etcd-cluster pkg=cluster

@hongchaodeng While I'm not afraid of code, there are way too many moving parts I have no idea about that writing code at this point would very likely do more harm than good :)
@xiang90 My confusion relates to:

Trying to recover from a previous backup Where is that backup and how was it defined/created?
no backup policy has been defined Couldn't find anything in the docs or code which gave me any insight on what backup policies there are and how I could configure them.

As I was just semi-blindly trying this out, I may have deployed configurations which set certain state which confused later attempts.
My primary problem may be related to a mismatch between my level of understanding of K8s and the developer's assumption of what I should already know :(
Anyway, I blew away my stack and created a new one, and now everything seems to work. Haven't tried to delete and recreate it, but I don't really have to do that.

OK. This is a mix of documentation  + code enhancement issue. We will improve soon.

I looked into this a bit further when I tried to re-create all resources.
@xiang90 you asked for my expectations. What I would expect is that if I delete an etcd-operator deployment I would expect that all state is removed and there won't be any influence on a future create.
Unfortunately, this is not the case. When I delete the service, there are still entries in the 'system' etcd which are being picked up on a future create. I went into the controller and removed all the respective entries and after that, a new create succeeded.

all state is removed and there won't be any influence on a future

We cannot do this. If a user accidentally remove the operator, we should absolutely not stop all running etcd cluster and other services.

That's a philosophical argument you can make and I would most likely disagree as it goes against expectations. But that's not the point, the fact right now is that, without that you cannot re-create it without a lot of effort and touching parts you shouldn't really.

I would most likely disagree as it goes against expectations.

We will doc this behavior. But we will not change it. I think safety is more important that your expectation. And expectation can be different from people to people and can be changed.

But that's not the point, the fact right now is that, without that you cannot re-create it without a lot of effort and touching parts you shouldn't really.

This is what we should and will fix. We will mark the dead cluster dead instead of panicking the operator itself.

I experienced the same issue with a different flow:
1- I deployed first a cluster in default namespace (same config used bellow)
2- I deleted this cluster
3- Created a new cluster in an other namespace
I think we should be able to create 2 clusters with the same name in separated namespaces.

@calj
One etcd operator only take charges of one namespace.

I think we should be able to create 2 clusters with the same name in separated namespaces.

What's your use case? Why do you need that?

3- Created a new cluster in an other namespace

How did you do that? Provide more info?

+1
This is really confusing that helm reinstall makes errors. Because documentation told us that
deployment should be created first and only after that you can create etcdCluster resource.
The only reason why it works first time is different behaviour when etcd-operator register TDR and when it deal with already registred TDR

@goruha I feel what you described is a different issue? Can you please tell us what you did? What error did you see? And what is your expectation?

I'm closing this issue.
Deleting etcd operator doesn't mean deleting TPR resources.

@hongchaodeng
I agree we should not delete user created TPRs, existing Pods etc. when shutdown etcd operator. The existing etcd clusters should remain running no matter what happens to etcd operator.
But this still needs to be resolved.
panic: TODO: mark cluster dead if no backup for disaster recovery.

goroutine 34 [running]:
panic(0x1878380, 0xc420292ce0)
	/usr/local/go/src/runtime/panic.go:500 +0x1a1
github.com/coreos/etcd-operator/pkg/cluster.(*Cluster).run(0xc4202df8f0, 0xc4203c5c80, 0xc4203c5910)
	/home/ubuntu/code/golang/src/github.com/coreos/etcd-operator/pkg/cluster/cluster.go:183 +0x63a
created by github.com/coreos/etcd-operator/pkg/cluster.new
	/home/ubuntu/code/golang/src/github.com/coreos/etcd-operator/pkg/cluster/cluster.go:119 +0x34e


Cannot create a test case with more than 20 snippets

If I create a test case with 20 snippets everything seems to go well and I can save them, publish the case and run the snippets.
However, if I attempt to add any more snippets beyond the 20 then I get an error response and the edit page just tells me to "review required fields and save again."
I cloned the repo and ran a local copy where I reproduced the issue and I can see in the server log that when I attempt to save the test case Joi validation fails at this line.
The peculiar thing is that the first time I attempt to save the test case the validation error is:

[log,error] message: child "test" fails because ["test" must be an array], stack: ValidationError: child "test" fails because ["test" must be an array]

Looking at the data, test is an Object and not an Array.
Now if I attempt to save again I get a different validation error complaining that the id of snippet 21 is invalid:

[log,error] message: child "test" fails because ["test" at position 20 fails because [child "testID" fails because ["testID" must be a number]]], stack: ValidationError: child "test" fails because ["test" at position 20 fails because [child "testID" fails because ["testID" must be a number]]]

Looking at the data, the testID property of snippet 21 is ''.
Now when I try to delete the last snippet because cannot save the test case with it, I clear out the 'title' and 'code' input fields and save again in hopes that it will be deleted, but I just get the same error. So I cannot even return to the working state where there were only 20 snippets.
So my question is then:
Is it allowed to have more than 20 snippets in a test case or is it expected that it should fail?
If it is not allowed then the user should be informed more clearly and it should be possible to delete cases.

Is it allowed to have more than 20 snippets in a test case or is it expected that it should fail?

After a quick search, I didn't find "20" hardcoded anywhere relevant so it should be "allowed" but that strikes me as a high number of variations to be testing. Maybe there should be some sort of limit.

@maxbeatty I agree that it is a high number of snippets and it would likely make sense to have well defined and properly handled limitation rather than a 'coincidental limit'.


RANDOM CRASH

if 'responses' in response_dict and \

TypeError: argument of type 'NoneType' is not iterable

use reporting template


Make enable/disable Configure parser more robust

When leading - is specified for enable/disable-openssl-xxx, just cut it off.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Desired to fix #13249

Tests are happy though I'm not sure it's a good measure.

I'm also not sure if this is good or not.

I frankly have my apprehensions as well...  our enabling and disabling options are distinctly without a dash to ensure that they are different from compiler / linker flags...  on the other hand, we have double-dash options as well, so we're definitely not kings of consistency, and we do have support for variable assignments on the configuration command line as well...
So, while not entirely opposed to this PR, I do have my doubts...

Well, you could repeat the steps from #13249 and see that there is some weirdness. But emitting a warning may be a better solution.

You could add "--" as an optional separator to mark of configure flags versus those that are just passed through.

I don't think it's reasonable or appropriate for us to claim the entire -enable-* namespace of program arguments.
Rich's idea of a -- separator seems worth pursuing, though.


Uninstall instructions for ClamAV

[#148597917]
Signed-off-by: Krishna Mannem<EMAIL_ADDRESS>

Thank you @kcmannem


Screenshots have objects and npcs floating about

Fabrishot version: 1.10.1
Minecraft version: 1.20.1
Regular shot (F2):

Fabrishot:

Is it because I'm using an older version? I couldn't use the latest yet as I'm also using the Create mod that's still stuck in mc 1.20.1. Any help would be great. Thanks!

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Finally found the culprit, ResolutionControl+ mod, removing it solves the issue!
Thanks for fabrishot btw, you guys are awesome! 😊


Invalid Unicode Upload from Liveview Upload?

I'm attempting to upload a file to supabase with this but I'm getting :invalid_unicode during streaming errors. I've tried with different mp3 files, and still :invalid_unicode.
Here's what the path of an object looks like ""/tmp/plug-1729-wEti/live_view_upload-1729464552-93643243762403-4" -- if I add .mp3 ofter that it won't find the file.
I've tried expanding the path, passing through the file_content instead of the file path, reversing the "1.mp3" with the path in terms of parameter order -- nothing seems to work here.
Client works great. Bucket I use list_buckets() and search for one by name to get the bucket object.
Any ideas?
def handle_event("submit", _params, socket) do
    uploaded_files =
      consume_uploaded_entries(socket, :audio, fn %{path: path}, _entry ->
        IO.inspect(path)
        {:ok, client} = Client.get_client()

        bucket = get_bucket_by_name(client, "audio-files")
        file_content = File.read!(path)
        expanded_path = Path.expand(path)

        IO.inspect(expanded_path)

        {:ok, audio_result} = Supabase.Storage.upload_object(client,  bucket, "1.mp3", path, %Supabase.Storage.ObjectOptions{
          cache_control: "3600",
          upsert: true
        })

        IO.inspect(audio_result)
      end)

      {:noreply, socket}
  end

interesting...

i'll try to reproduce that and do some debugging. thank you for reporting that

@zoedsoupe it's caused by this line in the potion fetcher: https://github.com/supabase-community/supabase-ex/blob/b7bfe786a62f9a0ac1b5676ea4f1aa390116818b/lib/supabase/fetcher.ex#L241
I removed the utf8 encoding in a fork and uploads work for other content types: https://github.com/slexysex/supabase-ex
Unsure if this breaks other functionality as I am only using the storage api


Some configuration and mapping are broken in main branch

Compared with 0.2.x, <C-c> on main seems to be damaged, unable to perform the close function, and cannot pass
smart_open = {
            match_algorithm = "fzf",
            mappings = {
                ["<C-c>"] = "close",
            },
        },

to set

This should work now.  Main has been updated.  Feel free to re-open if the issue is still there.


feat:  Add NacosServiceInstanceConverter util

Describe what this PR does / why we need it
Does this pull request fix one issue?

Describe how you did it
Describe how to verify it
Special notes for reviews

 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.

Can you explain why it was modified in this way?

Can you explain why it was modified in this way?

This design can make the conversion between nacos Instance and spring cloud ServiceInstance clearer.


Sparse depth data after multiple pipe.start()/pipe.stop() events

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
Android 8

Kernel Version (Linux Only)
3.18.71

Platform
Android

SDK Version
2.21

Language
java

Segment
Smartphone

Issue Description
Our use case involves acquiring data stream for short periods of time. Typically for 50 frames per session. For each session we call pipe.start() and pipe.stop().
This seems to work pretty well, until after some amount of sessions depth data gets very sparse. You can see it has very little detail. RBG image does not seem to be affected. The only way to recover from that state is to inplug USB cable.
Would you have any recommendations for this use case or have you had any issues related to depth data getting sparse?

Hi @m1halo
Can you please attach a picture of sparse data?
What I suspect to be happening is camera overheating, forcing the firmware to disable the active projector, reducing depth fill-rate.
If you can reproduce it on a PC, the Viewer communicates such hardware events via a dedicated notification.

Hi @dorodnic, here is the example scan result. We will try to reproduce it on PC as well. One additional observation is that each camera start takes longer and longer (eventually up to several seconds just before sparsity is visible).
Would you have any recommendation on solving this? Ie. lower the frame rate or resolution or does camera require a cooldown time between sessions if it is heating up issue.
[

](url)

Can you capture the IR streams (or use another IR camera) to see if the IR dot pattern is projected or not? You could check the overheating by getting some cold spray and cooling the camera with that to check if it helps.
If you capture longer periods, does it affect? Maybe the exposure control doesn't have time to adjust in such a short time.
What I have noticed is that the camera front glass gets easily dirty which affects the results and wiping it with a cloth helps, but your problem probably is not the same if unplugging helps.
Did you check if the firmware could be updated?

Thanks for tips. We are able to see dot pattern rather steady. With longer capture periods camera gets warm but there is no quality drop (or it might take many hours before it happens). It seems to be happening more often (faster) when those start/stop events happen.
It can be reproduced on latest FW.
Could longer and longer startup time be sign of overheating as well?

Hi @m1halo,

Please also show the left and right IR pictures when the issue happens.
Any kernel/librealsense log when the issue happens?
Are you able to reproduce this issue using realsense-viewer?
Due to longer and longer startup time, I suspect it has some abnormal communication or abnormal system latency there.

Hi @m1halo,
Any update? Is the issue clarified?

Hi @m1halo,
I will close this if no additional question. If you still have the trouble, feel free to reopen it.


AUT-2498: Add auth dashboards for ticf integration

Description:
Add dashboards for prod, integration and staging for authentication's calls to the TICF CRI.
Ticket number:
AUT-2498

If these are new dashboards, which they look like they are, you'll need to add in the corresponding modules for each dashboard into this file as part of the PR as well https://github.com/govuk-one-login/observability-configuration/blob/main/dashboards.tf


Add tests and validation for Gateway/Proxy in DP

Add tests which would verify correct work of API Gateway/Proxy in DP (contour and nginx)

#142


Wallet snap dev

Description:

Related issue(s):
Fixes #
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Created by accident


Endless error-out in install script when applying manifests to supported kubernetes API version with unsupported client

This happens when attempting to apply using a supported version of the Kubernetes server but an unsupported Kubernetes client and feels more like a UX issue to me.
Referencing the command from https://kalm.dev/docs/install:
curl -sL https://get.kalm.dev | bash

This results in an endless loop of:
Awaiting installation of CRDs
error: SchemaError(io.k8s.api.policy.v1beta1.PodDisruptionBudgetList): invalid object doesn't have additional properties

Output of kubectl version follows:
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.5", GitCommit:"32ac1c9073b132b8ba18aa830f46b77dcceb0723", GitTreeState:"clean", BuildDate:"2018-06-21T11:46:00Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.2", GitCommit:"52c56ce7a8272c798dbc29846288d7cd9fbae032", GitTreeState:"clean", BuildDate:"2020-04-30T20:19:45Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}

While the approrpriate error is dumped to stdout:
error: SchemaError(io.k8s.api.policy.v1beta1.PodDisruptionBudgetList): invalid object doesn't have additional properties

This is quickly overridden by the endless Awaiting installation of CRDs. I see a commented sleep 1 which would help with this problem but a better way would be to error out if the kubectl apply -f ... exits with a non-zero exit code. Could raise an MR to fix this behaviour if a maintainer could indicate what the project's preference for this behaviour is?

Thanks for the feedback.  I think there are two things we can do. First, we can update the install script to make it exits when a kubectl command fails without swallowing the errors. Second, update the docs to tell user use fit version kubectl along with their cluster when trying kalm. Merge Request is well welcomed. I will fix the doc later.


Add ignore_malformed to GeoShapeProperty

This commit adds ignore_malformed to GeoShapeProperty mappings i.e. geo_shape fields.
Add XML documentation to explain properties.
Closes #3155

Ported to 6.x


net/shadowsocks: add plugin config support for ss-local

Add plugin config support for ss-local.

Can you please share a link for a practical example? It's been a long a time since I last used shadowsocks.
Thx

Can you please share a link for a practical example? It's been a long a time since I last used shadowsocks.
Thx

OK.  https://github.com/shadowsocks/v2ray-plugin


Add jenkins_user table (#6)

Integration test logs

Logs
Add passing integration test logs here

Example query results

Results
Add example SQL query results here (please include the input queries as well)

@IvenGe this PR is pending on the addition of table doc for jenkins_user.


Add option to provide a VPN gateway to share gateways with multiple VPN connections

what

Add an optional option to provide the Virtual Private Gateway/VPN gateway to be used for the VPN connection
If a VPN gateway is provided, there won't be one created in the model
If a VPN gateway is NOT provided, the old behavior remains of creating one here.

why

In our case we needed to create multiple VPN connections, however we don't need multiple VPN gateways.
Since there is a relatively low default limit on the amount of VPN gateways you can create limitting the amount of VPN gateways we need is useful.

If you want to use this new feature, you need to create the VPN gateway elsewhere in terraform and then pass along the ID.
sample
Sample usage could look like this:
resource "aws_vpn_gateway" "default" {
  vpc_id          = var.vpc_id
  amazon_side_asn = 64512
}

module "my_vpn" {
  count          = 3
  source         = "cloudposse/vpn-connection/aws"
  version        = "0.7.2"
  vpn_gateway_id = aws_vpn_gateway.default.id
  # Other needed VPN settings...
}

Which will create the various VPN connections you need while only using 1 VPN gateway.

Since it is not looking like this is going to be merged, but we need some additional features. The source branch will contain more than just the initially mentioned change.
If people still want to get this moving, you can mention me and/or look at the 0.7.2 tag on my fork which was from before the additional stuff.

@paulvandenburg see my comments from 2 weeks ago to get this moving along -- Thanks!


Updated privileges on SHOW SCHEMAS, SHOW TABLES, and SHOW TYPES

Fixes https://github.com/cockroachdb/docs/issues/5118.

This change is

@roncrdb

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/6e7c9feaf93e514d8e28e31e717a340743e437cb/
Edited pages:

v19.2/show-schemas.md
v20.1/show-schemas.md
v20.2/show-schemas.md

@ericharmeling might also want to get clarification if SHOW TYPES and SHOW TABLES also need privileges on a DB in order to return.

@roncrdb

might also want to get clarification if SHOW TYPES and SHOW TABLES also need privileges on a DB in order to return.

Done.

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/7f3d627e5d14bf91dd04700151b67f07cbf3bec1/
Edited pages:

v19.2/show-schemas.md
v19.2/show-tables.md
v20.1/show-schemas.md
v20.1/show-tables.md
v20.2/show-schemas.md
v20.2/show-tables.md
v20.2/show-types.md


Update logitech-g915-tkl.device to DeviceType=keyboard

Seems like may have diverged accidentally from G915 in https://github.com/libratbag/libratbag/commit/4f36371af3eb43a012feb52eb9a1908ed0b2ac76#diff-1084927271ae24bc399f4cb5ceb16a4735c4b96e7c46462056cef9740a22d724
Feel free to close out if change was intentional.

Thank you! Great that this was noticed before we release a new version of libratbag.


Why the occupied GPU memory keep increasing and finally get GPU memory error?

Which GPU model do you use and how much GPU memory do you have? On Pascal TITAN X with 12 GB memory, there aren't any memory problems and the occupied space is around 8GB with a batch size of 32. If you have memory problems you can maybe try decreasing the batch size.

@btekin , thanks for your answering. I use one Tesla K80 with 12G memory and train with the default setting with batch=32. Initially, the occupied memory is around 11G. After around 70 epoch, the occpied memory increases to out of the memory of 12G and the training process stoped.

It might be due to the multi-scale training. We change the input resolution during training randomly and with the increasing resolution of input images the activation maps are taking up more space. But still 12 GB of memory should be sufficient enough. Did you check if some other process is already running on GPU and taking up space. If so, you can try killing that process and run again. You might also find the discussion in #12 helpful to resolve your memory problem.

Thanks, it seems that I meet the same issue with #12. I will also check the code to figure out why this happens, since it is fine with batch 32 in TITAN X.


Add a characters page

Detailing who's involved for anyone who doesn't play XCOM 2 (or watch it)

It seems that the images don't show up.

I think it works okay for now. THere's some minor alignment issues with the Warlock's description, but he's probably screwing with me in revenge for making him the butt of many jokes 😁


Not all apps can use Azure AD DS

[Enter feedback here]
"Applications that rely on domain services and use older protocols can use Azure AD DS."
I would change that to "might be able to use Azure AD DS", as not all can, i.e. Azure AD Connect is not supported on Azure AD DS (just asked the PG this question).

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

ID: 0a2540a7-f3a9-4440-2c97-163f45993672
Version Independent ID: d08b3a6f-113f-7f5b-2c11-c278ce11e905
Content: Identity and access management (IAM) - Cloud Adoption Framework
Content Source: docs/ready/enterprise-scale/identity-and-access-management.md
Service: cloud-adoption-framework
Sub-service: ready
GitHub Login: @BrianBlanchard
Microsoft Alias: brblanch

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 467631

Thank you for your feedback! This change has been included in an upcoming release to the content.


The updates of DDPG target network parameters in normalized layers

Hello, thanks for sharing. I wonder how to update target network parameters in normalized layers. Are they the same with neural network parameters?

Hi @sasforce! Not sure I understand the question. We are using layer normalization, which does not have any extra parameters to sync between target and current networks. One could, in principle, use batch normalization instead; in which case I would assume the normalization parameters will also need to be updated via the soft update. Does that answer your question?

Thank you for your reply!
I mean I am not really clear about the update of normalization layer parameters like gammas and betas in target networks, whatever BN or LN.

I see. Yes, those are updated the same way the rest of the trainable parameters of the neural net - copy before the beginning of the training, and soft update during training (at least that's how it is implemented in baselines, and from the paper it seems to me that that was the intent)

Thank you very much! I got it.


Linux Assets of the v0.2.0

Hello there
First of all, I would like to say thank you for your job :)
I have a few questions about assets from the last release:

Could you give any tips about this https://github.com/orange-cloudfoundry/credhub_exporter/releases/download/v0.2.0/credhub_exporter_linux_amd64 asset? Is it a binary? I am confused because you usually used another schema of the naming assets.
Would you plan to continue to use the new schema to name assets? Might I ask you to re-upload assets by using old name schema?

I am looking forward to your answers:)

Hi,
I've switched to goreleaser for this version which produces the release assets. The format is now stable and I wasn't planning to re-upload with the old schema.
After verification, there is indeed a problem with the linux binary which is actually a darwin build (Mach-O 64-bit x86_64 executable).  I don't know what happened and I'll fix this right away and re-upload the binaires. The .zip version of the linux build is correct.
Sorry for the inconvenience

The binary as been re-upload and is not matching sha256 if file checksums.txt

Is the new format a possible solution in your use-case as long as it remain stable from now on ?

Thank you for the fast response and for updating the binary. :)
How we use those assets: We have a pipeline which is downloading tar.gz file from URL that has a template like https://github.com/orange-cloudfoundry/credhub_exporter/releases/download/v__version__/credhub_exporter-__version__.linux-amd64.tar.gz than we extract it.
So now I need to change the template - there is not a lot of work.
Then I need to fix a script for this(I mean to skip unpacking step) - and it might be painful because other binaries came to us like a tar file).
To sum up, for me it will be the best solution to continue work with tar files but if you want to continue work without the tar files,  we have to rewrite some part of our pipeline. Please, let me know if you are not planning to use tar files:)

Go for the pipeline modifications, we'll stick to the current format


DesiredCapabilities::__construct() must be of the type array, null given

i have problem in first dusk test!
after run artisan dusk, show this error:

Tests\Browser\ExampleTest::testBasicExample
TypeError: Argument 1 passed to Facebook\WebDriver\Remote\DesiredCapabilities::__construct() must be of the type array, null given, called in ../vendor/facebook/webdriver/lib/Remote/RemoteWebDriver.php on line 127

dusk prepare is empty.
dusk driver:
return RemoteWebDriver::create('http://nayket-server.local', DesiredCapabilities::chrome());
APP_URL is nayket-server.local
nginx service is run... and i see home page in chrome browser.
PHP 7.2.9-1+ubuntu16.04

I'm having similar problem with codeception test and yii1.
The facebook/webdriver is 1.6.0.
[PHPUnit_Framework_Exception] Argument 1 passed to Facebook\WebDriver\Remote\DesiredCapabilities::__construct() must be of the type array, null given, called in /var/www/html/dsv/test-app/vendor/facebook/webdriver/lib/Remote/RemoteWebDriver.php on line 127 and defined

Please, fix this.
$ composer show
behat/gherkin                      v4.5.1             Gherkin DSL parser for PHP 5.3
codeception/codeception            2.5.0              BDD-style testing framework
codeception/phpunit-wrapper        6.0.12             PHPUnit classes used by Codeception
codeception/stub                   2.0.4              Flexible Stub wrapper for PHPUnit's Mock Builder
codeception/yii-bridge             dev-master a5841d6 YiiBridge extension
doctrine/instantiator              1.0.5              A small, lightweight utility to instantiate objects in PHP without invoking their constructors
facebook/webdriver                 1.6.0              A PHP client for Selenium WebDriver
guzzlehttp/guzzle                  6.3.3              Guzzle is a PHP HTTP client library
guzzlehttp/promises                v1.3.1             Guzzle promises library
guzzlehttp/psr7                    1.4.2              PSR-7 message implementation that also provides common utility methods
myclabs/deep-copy                  1.7.0              Create deep copies (clones) of your objects
phpdocumentor/reflection-common    1.0.1              Common reflection classes used by phpdocumentor to reflect the code structure
phpdocumentor/reflection-docblock  3.3.2              With this component, a library can provide support for annotations via DocBlocks or otherwise retrieve information that is embedded in a DocBlock.
phpdocumentor/type-resolver        0.4.0
phpspec/prophecy                   1.8.0              Highly opinionated mocking framework for PHP 5.3+
phpunit/php-code-coverage          4.0.8              Library that provides collection, processing, and rendering functionality for PHP code coverage information.
phpunit/php-file-iterator          1.4.5              FilterIterator implementation that filters files based on a list of suffixes.
phpunit/php-text-template          1.2.1              Simple template engine.
phpunit/php-timer                  1.0.9              Utility class for timing
phpunit/php-token-stream           1.4.12             Wrapper around PHP's tokenizer extension.
phpunit/phpunit                    5.7.27             The PHP Unit Testing framework.
phpunit/phpunit-mock-objects       3.4.4              Mock Object library for PHPUnit
psr/http-message                   1.0.1              Common interface for HTTP messages
psr/log                            1.0.2              Common interface for logging libraries
sebastian/code-unit-reverse-lookup 1.0.1              Looks up which function or method a line of code belongs to
sebastian/comparator               1.2.4              Provides the functionality to compare PHP values for equality
sebastian/diff                     1.4.3              Diff implementation
sebastian/environment              2.0.0              Provides functionality to handle HHVM/PHP environments
sebastian/exporter                 2.0.0              Provides the functionality to export PHP variables for visualization
sebastian/global-state             1.1.1              Snapshotting of global state
sebastian/object-enumerator        2.0.1              Traverses array structures and object graphs to enumerate all referenced objects
sebastian/recursion-context        2.0.0              Provides functionality to recursively process PHP variables
sebastian/resource-operations      1.0.0              Provides a list of PHP built-in functions that operate on resources
sebastian/version                  2.0.1              Library that helps with managing the version number of Git-hosted PHP projects
symfony/browser-kit                v3.4.17            Symfony BrowserKit Component
symfony/console                    v3.4.17            Symfony Console Component
symfony/css-selector               v3.4.17            Symfony CssSelector Component
symfony/debug                      v3.4.17            Symfony Debug Component
symfony/dom-crawler                v3.4.17            Symfony DomCrawler Component
symfony/event-dispatcher           v3.4.17            Symfony EventDispatcher Component
symfony/finder                     v3.4.17            Symfony Finder Component
symfony/polyfill-ctype             v1.9.0             Symfony polyfill for ctype functions
symfony/polyfill-mbstring          v1.9.0             Symfony polyfill for the Mbstring extension
symfony/process                    v3.4.17            Symfony Process Component
symfony/yaml                       v3.4.17            Symfony Yaml Component
webmozart/assert                   1.3.0              Assertions to validate method input/output with nice error messages.

I have the same problem with facebook/webdriver version 1.7.1.
A simple RemoteWebDriver::create() results in:
TypeError: Argument 1 passed to Facebook\WebDriver\Remote\DesiredCapabilities::__construct() must be of the type array, null given, called in /app/vendor/facebook/webdriver/lib/Remote/RemoteWebDriver.php on line 145

This may be caused by missing implementation W3C protocol (see #469), and should be fixed in upcoming release 1.8.0.


Automatic versioning with release drafter

Automatic version bumps to major, minor or patch based on the labels used within PR's.
@ludeeus @MartinHjelmare

Also added the workflow for the PR labels to be mandatory

That is one option, you can also just rerun the last run (its the same place as you have to trigger it anyway)

So is this PR fine as-is ? We don't adjust the workflow for the labels (and accept the bit of overhead it brings) ?


c5.large incorrect pricing for Oregon Linux On Demand

c5.large - table shows $0.102 for Linux on Demand cost while Amazon's table  shows $0.085. Amazon table I'm referring to is located here: https://aws.amazon.com/ec2/pricing/on-demand/

@gwolski I tracked down the issue. It looks like Los Angeles was being aliased to us-west-2 and was overwriting the Oregon price. I will push up a fix for this today.


[Question] arm64 Apple support (Apple Silicon)

[REQUIRED] Please fill in the following fields:

Pre-built SDK from the website or open-source from this repo: _____
Firebase C++ SDK version: 6.16.0
Main Firebase Components in concern: Auth
Other Firebase Components in use: None
Platform you are using the C++ SDK on: Mac
Platform you are targeting: Mac

[REQUIRED] Please describe the question here:
I'm porting my app to the newer ARM macs and I had thought the universal builds were for arm64 but it looks like they are 32/64 bit universal instead. Is there anyway to get built frameworks for mac arm64 architecture? Or does anyone know if I try and build from source will it work?
Thanks!

I've added this as a feature request. Internal ticket b/175209675.
We'll work to provide precompiled versions for working on the new Arm Macs, and a framework for running simulators on Arm Macs.  For now you could compile for macs from source by doing a build from this repo with '-DCMAKE_OSX_ARCHITECTURES=arm64;x86_64' defined.  I hope this helps!

Ah great, thanks for the pointers.
I'm not the greatest at cmake so am running into some unrelated config/build issues but I'll let you know if it works when I get it compiling.

I can't seem to figure out how to create the frameworks correctly. I've created the xcode project with this command but after linking I get asserts in the AddCompletionCallback of ReferenceCountedFutureImpl:
cmake -DFIREBASE_XCODE_TARGET_FORMAT=frameworks -DOPENSSL_ROOT_DIR=/opt/homebrew/opt/openssl "-DCMAKE_OSX_ARCHITECTURES=arm64;x86_64" -G "Xcode" -T buildsystem=1 ../firebase-cpp-sdk
I don't think this is an ARM specific issue and is just the way I created the frameworks because it doesn't work when building the x86_64 target either.
It doesn't look like there are any code changes necessary to build universal arm64;x86_64 builds so is it possible to get this put into the next build cycle?

Ah dang.
Thanks for the update!

Is there any update on mac arm support? I saw some changes for mac arm64 cmake builds but not sure if that was related.

Hey just want to check in on this one again. Any closer on building an arm64/x86_64 universal build?

Hi @mtytel,
We cannot publicly commit to any timelines, but this is indeed still on our roadmap.


Deleting and then restoring a .d.ts file leads to type errors

To reproduce:

Use the Webpack Dev Server
Make sure the config includes ts-loader with transpileOnly = true, and the fork-ts-checker-webpack-plugin
Delete a .d.ts file that declares an interface some other .ts file uses
Save a .ts file to trigger recompilation
You should see an type error about the missing interface that was declared in the .d.ts file
Re-add the .d.ts file that was deleted
Re-save the .ts file again to trigger recompilation
You will still see the same error even though the .d.ts file has been restored

It seems like this is a caching bug. If I use ts-loader with transpileOnly = false, and if I don't use this plugin, then the issue goes away

I've replayed your scenario and I don't have this issue. Please send me your tsconfig.json file - it could be the reason - see https://github.com/Realytics/fork-ts-checker-webpack-plugin#modules-resolution

The tsconfig file is pretty simple:
{   "compileOnSave": true,   "compilerOptions": {     "module": "es2015",     "sourceMap": true,     "target": "es5",     "moduleResolution": "node"   } }
I've tried setting "moduleResolution" to "classic", but the same issue persists

Update: I found the issue is solved for me if I comment out lines 96 & 98 in IncrementalChecker.js so the following statement is always executed:
this.programConfig = IncrementalChecker.loadProgramConfig(this.programConfigFile);
Is there any way to update the file list on every run of IncrementalChecker.prototype.nextIteration? If someone else on my team adds a new file to Git and I Pull to get the changes, I'll get a type error from this plugin because it's not aware of the newly added file. The only current fix is to stop and restart Webpack.

Sorry for the delay, I have exams at a college so I'm a little bit busy :/
It's very weird - typescript uses an old program to build a new one but there are checks for program reusability inside typescript compiler.
I would like to help you but as long I can't reproduce the bug, I can't fix it. I've used 2.3.4 too.
Could you create a repository for issue reproduction? Or if it's an open source, just send me a link, please :)

@stowellt ? :)

I apologize for the delay in response. The project is proprietary so I can't send a link. I haven't had time to to setup a repo to reproduce the issue, and unfortunately it might be a while before I will have a chance to do that. I understand if you want to close this bug. I would still like to revisit this in the future at some point, thanks.

No problem :) Thank you for reporting, I will close this issue :) Feel free to re-open issue in the future ;)


Patch for successive halving

Changes this PR implements:

meta keys ident, time_step => model_id, partial_fit_calls
call partial_fit the number of times specified by partial_fit_calls
return all the meta objects returned by _score instead of only some of them

I have a successive halving implementation working correctly with this implementation.
This PR isn't formally tested, and I'm only okay submitting it because it's for https://github.com/dask/dask-ml/pull/288/.

 Feel free to checkout out my branch and copy/paste the _incremental.py and test_incremental.py over – those are the only files changed.

Out of curiosity why were these changes necessary?
Also, did the solution for successive halving in the notebook not work for
some reason?   It isn't super well explained.  Let me know if I can help
explain things there.
On Fri, Jul 27, 2018, 3:49 PM Scott Sievert<EMAIL_ADDRESS>wrote:

Changes this PR implements:

meta keys ident, time_step => model_id, partial_fit_calls
call partial_fit the number of times specified by partial_fit_calls
return all the meta objects returned by _score instead of only some
of them

I have a successive halving implementation working correctly with this
implementation.
This PR isn't formally tested, and I'm only okay submitting it because
it's for dask#288 https://github.com/dask/dask-ml/pull/288.
You can view, comment on, or merge this pull request online at:
https://github.com/mrocklin/dask-ml/pull/1
Commit Summary

use single-threaded scheduler in make_classification
Add basic incremental training function
Clean up incremental model selection
Format with black
MAINT: rename

File Changes

M dask_ml/datasets.py
https://github.com/mrocklin/dask-ml/pull/1/files#diff-0 (4)
A dask_ml/model_selection/_incremental.py
https://github.com/mrocklin/dask-ml/pull/1/files#diff-1 (284)
A tests/model_selection/test_incremental.py
https://github.com/mrocklin/dask-ml/pull/1/files#diff-2 (134)

Patch Links:

https://github.com/mrocklin/dask-ml/pull/1.patch
https://github.com/mrocklin/dask-ml/pull/1.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mrocklin/dask-ml/pull/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AASszFETGrLucj4SwPJ2mZAjCEs6yl_oks5uK5j4gaJpZM4VkxHq
.

Out of curiosity why were these changes necessary?

meta keys time_step => partial_fit_calls

By no means required, but this makes the relationship more explicit.

call partial_fit the number of times specified by partial_fit_calls

This was very implicit; when time_step=0,  model.partial_fit had been called once. I didn't realize that until I dug into the code.
For my test case with successive halving, this meant that a factor of 1.2 more calls to partial_fit were made (486 vs 405).

return all the meta objects returned by _score instead of only some of them

This is only required with a brain fart. I've removed it.

I've updated, and git history is cleaner now. It looks like you've rebased off master – thanks.
Also, I commented in https://github.com/dask/dask-ml/pull/288 about the bugs I thought I found. I deleted the comments to make diff viewing nicer.

The call=False thing seems weird to me.  I'm still not sure I understand why we would call the _partial_fit function and not want to call partial_fit.  Are you just trying to correct for an off-by-one indexing issue?  If so then maybe its less disruptive to change an index somewhere or to change this in your code?

For my test case with successive halving, this meant that a factor of 1.2 more calls to partial_fit were made (486 vs 405).

Can I ask you to share the code that you're building?  Also, can I ask for your thoughts on the successive-halving-ish implementation in the notebook?  Was there something lacking there that required an additional implementation?  I'm surprised that you actually needed to write an implementation here.  I was hoping that you would be able to take the existing implementations in the notebook and quickly apply them to some problem.

Can I ask you to share the code that you're building?

https://gist.github.com/stsievert/facdf6c1427c810cafecd30848fe20d9

can I ask for your thoughts on the successive-halving-ish implementation in the notebook? Was there something lacking there that required an additional implementation?

Nothing required additional implementation. Documentation was a little sparse, so I didn't know exactly what each value in the input meant (e.g., time_step). I had to dig into the code and got surprised by things that were correct but more subtle than I was expecting.

I'm surprised that you actually needed to write an implementation here

Turns out I didn't need too, though I thought I did. I submitted this PR because I thought I caught a small bug, and realized in https://github.com/mrocklin/dask-ml/pull/1#issuecomment-408574936.

The call=False thing seems weird to me. I'm still not sure I understand why we would call the _partial_fit function and not want to call partial_fit.

I knew this would come up. I've removed call from _partial_fit completely, and catch NotFittedErrors in _score now.


[Bug] Some PDFs Not Properly Indexing

Context / Scenario
My team and I built a small feasibility proof of concept utilizing Kernel Memory, which we use often, to index 11 documents (10 .pdf, 1 .docx) and extract some common data fields according (ex: date, location, etc.).
What happened?
After instantiating the instance of kernel memory, we were able to successfully import 9 of the 11 documents using the ImportDocumentAsync() method.
Initially, it appeared all 11 documents were indexed properly, however, 2 of the files resulted in "INFO NOT FOUND".
When investigating further, we used the GetDocumentStatusAsync("docId") method, which indicates it completed successfully.
Below is the output for reference:
Completed: True
Failed: False
Empty: False
Index: default
...
RemainingSteps: [ ]
CompletedSteps: [extract, partition, gen_embeddings, save_records]
All prompts return INFO NOT FOUND including:
"What is the name of this document?"
"What is this document about?"
"What is the word count of this document?"
There does not appear to be any difference between these 2 files and the other 9 indexed correctly (size, file type, naming convention, etc. are all consistent).
Importance
I cannot use Kernel Memory
Platform, Language, Versions
Relevant technologies include:
.NET 8 (C#)
Microsoft.KernelMemory.Core 0.71.240820.1
Relevant log output
No response

could you try importing only those 2 files and checking the content of the vector DB? you can also check the content of document storage, to see the text extracted and all the chunks. That should give you some information to see if the docs are not handled like the other 9.

@apennypacker-launchcg if you can share the two PDFs here, I can try importing them to verify what happens.

Closing for inactivity


Move undefOr2jsAny implicit conversion to Any companion

As discussed on Discord. Sorry, I did the minimal thing, and will appreciate hints for what else to do 😅
The motivation for this change is that this implicit conversion will be automatically be in scope on Scala 3 (as well as Scala 2), which is not currently the case. This forces the use of casts in user code as can be seen in the scala-js-dom Scala 3 PR in https://github.com/scala-js/scala-js-dom/pull/450.

There's probably not much actual unit testing we can do.

How would you feel about adding a test here?
https://github.com/scala-js/scala-js/blob/0d19d982a29d5426e7ad743d8787c2e006520af8/sbt-plugin/src/sbt-test/scala3/basic/Main.scala#L15-L26

Did the appveyor have a legitimate failure, or did it just timeout?

Did the appveyor have a legitimate failure, or did it just timeout?

Spurious failure, apparently. Rerunning the build made it pass.


bitcoind problem

Hello!
I'm trying to edit the bitcoin.config file but I can't do it since I have not permission because I'm not the owner. How can I gain access to edit this file? Or is there another way to do it?

You don't have root access?

I'm not authorized to edit bitcoind folder (that includes bitcoin.config)
I have the root password but I don't know how to gain access.
I've tried to do a "folder sharing" but I get this error: error 255: net usershare add: cannot share path /usr/share/doc/bitcoind as we are restricted to only sharing directories we own.
Ask the administrator to add the line "usershare owner only = false"
to the [global] section of the smb.conf to allow this.

https://github.com/peatio/peatio/blob/master/doc/deploy-production-server.md
setup a production server files are located within the users access.
Else you need a full access vps. Which I recommend because shared hosting is really bad