Make cache dir optional for users building with no_downloads feature

I've made some changes to the Exe file to make cache_folder optional, which solves some other problems for NIx users. Everything still seems to work as expected, and all the tests pass.
Love to get another release for cargo-leptos Nix users if everything checks out

Hey @benwis! It seems like the Exe stuff is getting a bit "un-clean" and needs refactoring. This is mainly because of previous changes and contributions. What you are proposing makes sense but I think at this point what would be best is a refactoring so that the cache dir is not needed when creating the enum.
If you are ok with it, I can go ahead and do that today. I'm just sorry for realizing this now after you already have spent time on this PR.

Makes sense to me @akesson! For Nix to be happy, all we need is to not download things and not to create a cache dir. As long as it does that I am happy!

Closing this as it was implemented in #81


Document that the action requires to run on ubuntu

I naively added the action to our cibuildwheel matrix (for the shapely wheels), expecting to upload the wheels from each OS-specific build directly.
However, this does not work because the action apparently requires a linux image: "Container action is only supported on Linux" (see eg https://github.com/shapely/shapely/actions/runs/8047467718/job/21976750565). So you need to gather all wheel for the different architectures and upload them from another job running on ubuntu.

Ah yes. I hadn't thought about that, but as it is container based, it currently requires a Linux OS base.
This should get documented in the README as you suggest.

Note to self: c.f. https://github.com/shapely/shapely/pull/1998


Display balance for different wallet

When using the cold wallet the current display of balance becomes sort of obsolete since all rewards will be transfered to the cold wallet instead of the farming wallet.
I suggest to make the display configurable somehow.

The display (and sum) of cold wallet balances is now available in the current :test images.  Will be in the upcoming v0.6.5 release.  Thanks again for the suggestion.

Good day!  Version 0.6.5 has now been released.


Irregular onBlur validation behaviour inside React Bootstrap Modal

Hi,
I have an issue where I have a Formik form inside a React Bootstrap Modal. When I focus on one of the form fields and click the Cancel button to Close the Modal Form, sometimes it triggers onBlur validation of the field that was in focus and at other times it does not. When the onBlur validation is triggered after the clicking the Cancel button, the default behaviour of the Cancel button (to close the Modal) does not work and required an extra click to close the Modal Form.
The behavior where the onBlur validation is triggered is acceptable however, it seems like an anomaly when it is not triggered.
Following is my code:
<Modal
				show={show}
				onHide={handleClose()}
				size='lg'
			>
				<Modal.Header>
					<Modal.Title>Create Category</Modal.Title>
				</Modal.Header>
				<Modal.Body>
					<Formik
						initialValues={ initialValues}
						validationSchema={validationSchema}
						onSubmit={onSubmit}
					>
						{({
							dirty,
							values,
							errors,
							isSubmitting,
							submitForm,
							validateForm,
							setFieldValue,
							resetForm,
							setErrors,
						}) => {
							return (
								<Form>

										<FormikControl
											control='text'
											name='name'
											label='Category Name'
											placeholder='Enter Category Name'
											containerClassName='col-md-6 form-group'
											required={true}
											autoComplete='off'
										/>

										<FormikControl
											control='native-select'
											name='type'
											label='Type'
											options={CATEGORY_TYPES}
											containerClassName='col-md-6 form-group'
											placeholder='Choose Category Type'
											required={true}
										/>

										<FormikControl
											control='number'
											name='commission_charge'
											label='Commission Charge (%)'
											placeholder='Enter Commision Percentage'
											containerClassName='col-md-6 form-group'
											required={true}
											autoComplete='off'
										/>

										<FormikControl
											control='file'
											name='image_url'
											label='Category Image'
											containerClassName='form-group col-md-6'
											required={true}
										/>

										<Button
											variant='outline-danger'
											className='mr-2'
											type='button'
											onClick={event => {
												resetForm();
												handleClose();
											}}
											disabled={isUploadingBanner || isSubmitting}
										>
											Cancel
										</Button>
										<Button
											variant='outline-primary'
											type='button'
											onClick={() =>
												submitInterceptor(submitForm, validateForm, values, setFieldValue)
											}
										> Save
										</Button>

								</Form>
							);
						}}
					</Formik>
				</Modal.Body>
			</Modal>

If anyone can figure out the source of this anomaly, please let me know.
Thanks :)

Are you able to reduce this to a codesandbox? There's a lot going on here that is outside of Formik.

Are you able to reduce this to a codesandbox? There's a lot going on here that is outside of Formik.

Alright, I will create a sandbox and share the link.


Access reference models for "sameAs" validator

How can i access referenced models/names used in "sameAS"
password: {           required,           alphaNum,           minLength: minLength(6)         },         confirmPassword: {           sameAs: sameAs('password')         }
I want to display a generic message on validation error that "confirmPassword" must be same as "password" (how to get this name?).

As a validator is simply a higher order function, there is no way to access the arguments you passed to the validator outside of it's scope.
We were brainstorming that idea, but it's really hard to accomplish without giving up a very important feature, which is a seamless support to arbitrary functions as validators from any library.
Please check https://github.com/monterail/vuelidate/issues/3 as our latest idea how to implement a withMessage modificator.
Currently we lean towards just specifying your message parameters by hand.
You can prepare a translation like Your {field} is not identical to {otherField}. and just provide the parameters by hand like {field: 'repeated password', otherField: 'original password'}.
Notice that this is not even a valid translation, as translation parameters should be independent of the language, so the problem is much deeper. In the end you have to specify the whole message for each usecase anyway. We don't want to implement translations in a library with different responsibilities, so instead an external translation libraries have to be easy to use in combination with Vuelidate.
For us it was an argument to not include this feature for now, as there are many gotchas. Feel free to join the brainstorming, any input is appreciated.

Well, now i understand, thanks for the explanation. I will follow #3 to keep an eye on progress.


Clicking on "Add Service" from "Edit Balancer Service" does nothing.

Server version - V0.270.0-rc1
Create Balancer service with 2 targets.
Edit this Balancer service and try to add another service , by clicking on "Add Service" .
Nothing happens.

v0.27.0-rc1 (ui latest v0.32.0)
Verified fixed


Sensitive information in the body of OAuth2 requests not filtered by default

If I understand https://github.com/zalando/logbook#filtering correctly sensitive information of the body of OAuth2 Request should be filtered by default:

form-url -> client_secret and password

but senstive information is logged:

12:25:18.332 [main] TRACE org.zalando.logbook.Logbook - Outgoing Request: f022e983472f4182
Remote: localhost
POST https://myhostname/services/oauth2/token HTTP/1.1
accept: application/json, application/*+json
accept-encoding: gzip
connection: Keep-Alive
content-length: 281
content-type: application/x-www-form-urlencoded;charset=UTF-8
host: myhostname
user-agent: okhttp/4.7.0

grant_type=password&client_id=myid&client_secret=mysecret&username=myusername&password=mypassword

We are using logbook-spring-boot-starter, version 2.1.0 but I can replicate the behavior in a unit test where I use Logbook.create().
Debugging into DefaultLogbookFactory#create I noticed that the BodyFilter is a NonMergeableBodyFilterPair of org.zalando.logbook.json.CompactingJsonBodyFilter and org.zalando.logbook.json.AccessTokenBodyFilter. A filter for FormUrlEncoded seems to be missing or the documentation is wrong.

Docs are incorrect. In logbook-core there is no BodyFilter registered by default (using service loader), but there should be one.
First, the following filter needs to be declared as a separate class:
https://github.com/zalando/logbook/blob/66a634dea13440649f118e4151bcde71711c8522/logbook-core/src/main/java/org/zalando/logbook/BodyFilters.java#L29-L35
So that it can be registered by name as a service for the ServiceLoader.

Docs are incorrect.

Rather, the docs claim something that is desirable, but that's not the reality. Docs should stay, but behavior should be fixed.

I just released 2.1.2 with a fix.

Thank you very much! 👍


Some regex match more aggressively than expected based on java pattern matcher

Incorrectly mapping ^* prefix to ^.*. For example this shouldn't have any matches:
/api/v1/graph?q=name,sps,:eq,nf.cluster,*ccp*,:re,:and,:sum,(,nf.cluster,),:by

This is mostly likely getting rewritten to an indexOf query.
/cc @nathfisher

On closer inspection this is working as expected. The original test was doing matches rather than find which is what we actually use. The behavior is consistent with an anchored regex.
scala> import java.util.regex.Pattern

scala> Pattern.compile("^*foo*").matcher("a_foo_bar").matches()
res0: Boolean = false

scala> Pattern.compile("^*foo*").matcher("a_foo_bar").find()
res1: Boolean = true

Note, grep and the javascript regex parsers do not seem to like the ^* pattern:
$ echo "a_foo_bar" | grep -E '^*foo*'
grep: repetition-operator operand invalid


RuntimeError: CUDA error: out of memory

Windows 10 Env
Python 3.6.5
run:
python train_pose2vid.py
I got:
Traceback (most recent call last):
File "train_pose2vid.py", line 127, in
main()
File "train_pose2vid.py", line 59, in main
Variable(data['image']), Variable(data['feat']), infer=save_fake)
File "d:\Anaconda3\envs\py365\lib\site-packages\torch\nn\modules\module.py", line 477, in call
result = self.forward(*input, **kwargs)
File "D:\githubs\EverybodyDanceNow_reproduce_pytorch\src\pix2pixHD\models\pix2pixHD_model.py", line 195, in forward
loss_G_VGG = self.criterionVGG(fake_image, real_image) * self.opt.lambda_feat
File "d:\Anaconda3\envs\py365\lib\site-packages\torch\nn\modules\module.py", line 477, in call
result = self.forward(*input, **kwargs)
File "D:\githubs\EverybodyDanceNow_reproduce_pytorch\src\pix2pixHD\models\networks.py", line 123, in forward
x_vgg, y_vgg = self.vgg(x), self.vgg(y)
File "d:\Anaconda3\envs\py365\lib\site-packages\torch\nn\modules\module.py", line 477, in call
result = self.forward(*input, **kwargs)
File "D:\githubs\EverybodyDanceNow_reproduce_pytorch\src\pix2pixHD\models\networks.py", line 417, in forward
h_relu5 = self.slice5(h_relu4)
File "d:\Anaconda3\envs\py365\lib\site-packages\torch\nn\modules\module.py", line 477, in call
result = self.forward(*input, **kwargs)
File "d:\Anaconda3\envs\py365\lib\site-packages\torch\nn\modules\container.py", line 91, in forward
input = module(input)
File "d:\Anaconda3\envs\py365\lib\site-packages\torch\nn\modules\module.py", line 477, in call
result = self.forward(*input, **kwargs)
File "d:\Anaconda3\envs\py365\lib\site-packages\torch\nn\modules\conv.py", line 301, in forward
self.padding, self.dilation, self.groups)
RuntimeError: CUDA error: out of memory
Any help is appreciated!

I met the same problem.
Do anybody know how to solve it?
Thx a lot!

my batchsize is 1.
i don not understand why it,s stil out of memory.

Models use are very big.
I have the same problem with my GPU with 4GB or RAM. :c
Better user google collaboratory. :v

Models used are very big.
I have the same problem with my GPU with 4GB or RAM. :c
Better user google collaboratory. :v

changed the loadSize fineSize from 512 to lower，then can run

changed the loadSize fineSize from 512 to lower，then can run

thx


LogFactory.Setup().LoadConfiguration() with fluent creation of NLog-LoggingConfiguration

Same as #4128 but for dev-branch.
Continuing the work started with NLog 4.7 and NLog 4.7.1 to improve fluent-configuration, by adding new extension-methods to the existing ISetupLoadConfigurationBuilder-interface.
You can now do this:
var logger = LogManager.Setup().LoadConfiguration(c =>
{
   c.ForLogger("*").FilterMinLevel(LogLevel.Info).WriteTo(new ConsoleTarget("logconsole")).WithAsync();
   c.ForLogger().WriteTo(new FileTarget("logfile") { FileName = "file.txt" }).WithAsync();
}).GetCurrentClassLogger();

Was thinking to add more helper-methods for the individual-target-types, but decided to skip it for simplicity (Only included MethodCallTarget as an example)
This allows one to combine file-config with runtime-config, where it loads from NLog.config, and then applies black-hole filtering:
var logger = LogManager.Setup().LoadFromFile().LoadConfiguration(c =>
{
   c.ForLogger("Microsoft.*").TopRule().WriteToNil(maxLevel: LogLevel.Info);
}).GetCurrentClassLogger();

@AndreaCuneo Thank you for the feedback. You are right about improving the support for re-using the same target in multiple logger-rules.
Right now WriteTo starts a Target-Builder, but makes sense that one can call WriteTo and give a pre-created target-builder. And re-use the same pre-created target-builder for several logging-rules.
And it also make sense that one can combine two Target-Builder together. Ex. WithFallback() should also take a pre-created target-builder.
Thank you for raising the bar, and showing that more advanced scenarios should be supported. And not just the simple example from the tutorials.

@AndreaCuneo Now you can do this:
var logger = LogManager.Setup().LoadConfiguration(c =>
{
   var myTarget = c.ForTarget("MyTarget").WriteToConsole();
   var myTargetAsync = c.ForTarget("MyTarget_Async").WriteTo(myTarget).WithAsync();

   c.ForLogger("Microsoft*").FilterMaxLevel(LogLevel.Info).WriteToNil();
   c.ForLogger("*").FilterMinLevel(LogLevel.Error).FinalRule().WriteTo(myTarget);
   c.ForLogger("*").WriteTo(myTargetAsync);
}).GetCurrentClassLogger();

Guess NLog should be smarter, so a Logger-Configuration will never have the same target twice. Thus reducing the issue with duplicate output when not careful with the loggingrules. And if people really need duplicate output to the same target, then one can use other means.

@snakefoot looks grand, and I fully agree on the subscript.
One question: is loose reference by target name supported?
My idea stem from the fact that often which Targets are available is an architecture decision predefined (ie by Architect / Governance ) across a fleet of Applications, for example in Enterprises.
Rules instead could change based on some peculiarities of the given Application.
I think would be useful if the scenario below would be supported out-of-the-box.

// in library code

static ISetupBuilder SetupDefaults(this ISetupBuilder setup) =>
   setup.LoadConfiguration(c =>
   {
       var myTarget = c.ForTarget("Default").WriteToConsole().WriteToDatabase(...).WithAsync();

       c.ForLogger("Microsoft*").FilterMaxLevel(LogLevel.Info).WriteToNil();
       c.ForLogger("*").FilterMinLevel(LogLevel.Error).FinalRule().WriteTo(myTarget);
   });

// in application code
LogManager.Setup().SetupDefaults().LoadConfiguration(c => {
    c.ForLogger("Microsoft.*").TopRule().FilterMinLevel(LogLevel.Error).WriteTo("Default");
});

Sounds like we have moved from "reviewing presented fluent api" to "investigating the wet dreams of a single user".
NLog has no direct concept of giving a name to a group of targets. But one could bring the SplitGroup-target into the game like this:
static ISetupBuilder SetupDefaults(this ISetupBuilder setup) =>
   setup.LoadConfiguration(c =>
   {
       var myTarget = c.ForTarget("Default").WriteToSplitGroup().WriteToConsole().WriteToDatabase(...).WithAsync();

       c.ForLogger("Microsoft*").FilterMaxLevel(LogLevel.Info).WriteToNil();
       c.ForLogger("*").FilterMinLevel(LogLevel.Error).FinalRule().WriteTo(myTarget);
   });

// in application code
LogManager.Setup().SetupDefaults().LoadConfiguration(c => {
    var myTarget = c.Configuration.FindTarget("Default");
    c.ForLogger("Microsoft.*").TopRule().FilterMinLevel(LogLevel.Error).WriteTo(myTarget);
});

But this is an advanced shotgun, that I seldom seen anyone use. The normal users that ask for fluent-api, are people with very simple config, and wants very simple API.
But thank you for all these special API-usage examples. It has revealed several places where more safe-guards must be added, to avoid users shooting themselves in the foot.

@snakefoot sorry, I've confused you with a copy-paste issue and made you lose time. Apologies.
I didn't meant to suggest the 'same name for multiple targets' case, but thanks for the hint nonethess on the SplitGroup target, I didn't know it existed.
The case was only for the the .WriteTo("Default") , by string. I've not seen the overload in the commits, so I wasn't sure if it was me missing it or could have been a small addition.

I think .WriteTo("Default") is a sin because it involves magic-strings that can fail at runtime. It works against the whole idea with fluent-api, that it should be simple predictable with no surprises.

Updated wiki-pages:

https://github.com/NLog/NLog/wiki/Fluent-Configuration-API
https://github.com/NLog/NLog/wiki/Configure-from-code


using hovercard, on mobile view, pressing the hovercard giving error message on chrome console "unable to preventDefault inside passive event listener invocation."

Bug report
Current Behavior

I am using hover card, working without any issues on desktop, however, moving into a mobile view, you have to click the hover card, but clicking it, creates the above error message, and the hover content does not show up.

Expected behavior
I expect the hover card content to show up on click while in mobile view, just like hovering over in desktop shows up the content
Reproducible example
This issue is happening even on the demo of that project, in this url:
https://tailwindcss-radix.vercel.app/
and the Github repo of that demo is:
https://github.com/ecklf/tailwindcss-radix/blob/main/demo/components/hover-card.tsx
Suggested solution
If you can fix this issue, to make the hover card work (with a click) and show the content on mobile view, that will be a good solution.
Additional context
Your environment

Software
Name(s)
Version

Radix Package(s)
@radix-ui/react-hover-card
1.0.5

React
18
Latest

Browser
Brave
Latest

Assistive tech

Node
Latest

npm/yarn
yarn
1.22.19

Operating System
Windows 10 64x

Hey @roniaxe, I think it's by design that the hovercard doesn't open on click since the trigger must be an anchor so when the user clicks on the link it will be taken to another page so it doesn't make much sense to show the content when clicked. However, I believe you have reported a valid bug regarding the error that appears in the console.

Hey @joaom00, Thanks for your reply!
So in mobile view hover card is not an option ? Do you have any idea or workaround of how I can use the hover card to be kind of functional if on a mobile view ?

Although the hovercard opens when the link is focused, and looks like this should be prevented from what I've read of the code, but this is not happening and causing the console error, you can control the open state, with open and onOpenChange props, and handle to open whenever you want, but it seems weird to want to open the hovercard with a click on the mobile. What is your use case btw?

@joaom00 is correct here. HoverCard won't show on touch devices exactly as explained, same thing for tooltips for the same reasons. If you want similar'ish behaviour, you should probably use Popover instead.
I also haven't been able to reproduce the issue on the link you posted using the iOS simulator, or a real device so I will close the issue.


Embeddable player in Dashboard

Feedback on On Demand. Context: https://app.zenhub.com/workspaces/studio-62e817e78dcba15ddb23d5ad/issues/livepeer/studio/1227
@adamsoffer can you update this ticket on any progress related to the embeddable player code in the dashboard? Curious how far we got.

@pglowacky here's the proposed design. We can you leave feedback in the comments if you have any? Can begin implementation this week.
Preview:

FTR we can reuse the lvpr.tv player from
https://github.com/livepeer/player
Should update it to use the SDK player as well

FTR we can reuse the lvpr.tv player from
https://github.com/livepeer/player
Should update it to use the SDK player as well

👆 deleted a duplicate comment

btw we also have docs on embedding the lvpr.tv! might need to update that after migrating it to the SDK player (ideally not breaking the interface tho, just in case users are already using it in the wild):
https://docs.livepeer.studio/guides/playback-a-video-stream#video-player

It seems clear enough but I have a couple of extra questions.

@victorges @adamsoffer the code that can be copied and pasted is the one linked here right?

@adamsoffer in the design for this Embed feature, there is also a Copy link and Download button. Are these features for another time, or are we including them already?

Yup that's right 👍
Yes let's include the "copy link" and "download" call to actions form the design as well
Yes, let's add this to the stream detail view as well.

The design looks great!

Hello all! There is a PR out for this issue. There are a few things, described in the PR, that I need confirmation of.
If you want to take a look, here it is
https://github.com/livepeer/studio/pull/1295


refactor(v2): minor styling improvements

Motivation

Fix borders of collapse doc sidebar button

Replace e-resize type with click when hover on expand sidebar button

Before
After

Add hover state for dark mode switcher

Use Infima buttons on home page

Before
After

Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Preview
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

Thanks
Was not totally satisfied with my homepage button 😅
There are some responsive problems thoughts

There are some responsive problems thoughts. Also wonder if it's not better to keep the text white for dark mode?

Responsive design issues fixed, as for the color it's hard for me to make a decision - I think the default color looks good.

thanks, we can see later to improve if needed :)


Crash in usbmux_packet_alloc assert

We're getting crash dumps from our OS X app pointing to an assertion being triggered in PTUSBHub.m line 66:
#0. Crashed: com.apple.libdispatch-io.opq
0  libsystem_kernel.dylib         0x7fff8d4b5286 __pthread_kill + 10
1  libsystem_pthread.dylib        0x7fff8c6379f9 pthread_kill + 90
2  libsystem_c.dylib              0x7fff93a8f9ab abort + 129
3  libsystem_c.dylib              0x7fff93a57a91 __assert_rtn + 319
4  Application                    0x109abf3c8 usbmux_packet_alloc (PTUSBHub.m:66)
5  libdispatch.dylib              0x7fff947f8e2b ___dispatch_operation_deliver_data_block_invoke + 118
6  libdispatch.dylib              0x7fff947e0700 _dispatch_call_block_and_release + 12
7  libdispatch.dylib              0x7fff947dce73 _dispatch_client_callout + 8
8  libdispatch.dylib              0x7fff947e05cd _dispatch_queue_drain + 1100
9  libdispatch.dylib              0x7fff947e0030 _dispatch_queue_invoke + 202
10 libdispatch.dylib              0x7fff947ed5aa _dispatch_main_queue_callback_4CF + 416
11 CoreFoundation                 0x7fff8a5fd3f9 __CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__ + 9
12 CoreFoundation                 0x7fff8a5b868f __CFRunLoopRun + 2159
13 CoreFoundation                 0x7fff8a5b7bd8 CFRunLoopRunSpecific + 296
14 HIToolbox                      0x7fff8968356f RunCurrentEventLoopInMode + 235
15 HIToolbox                      0x7fff896832ea ReceiveNextEventCommon + 431
16 HIToolbox                      0x7fff8968312b _BlockUntilNextEventMatchingListInModeWithFilter + 71
17 AppKit                         0x7fff872698ab _DPSNextEvent + 978
18 AppKit                         0x7fff87268e58 -[NSApplication nextEventMatchingMask:untilDate:inMode:dequeue:] + 346
19 AppKit                         0x7fff8725eaf3 -[NSApplication run] + 594
20 AppKit                         0x7fff871db244 NSApplicationMain + 1832
21 Application                    0x109a935d4 start + 52

We poll the USB port repeatedly waiting for a device with our client app to connect. The crazy thing about that is the assert being triggered should be:
static usbmux_packet_t *usbmux_packet_alloc(CFIndex payloadSize) {
  assert(payloadSize <= UINT32_MAX); // protocol limitation
  uint32_t payloadLength = (uint32_t)payloadSize;

but the stack shows this coming from the dispatch_io_read in scheduleReadPacketWithCallback (PTUSBHub.m:494), which uses a uint32_t parameter, thus making it (theoretically) impossible to overflow UINT32_MAX:
    // Allocate a new usbmux_packet_t for the expected size
    uint32_t payloadLength = upacket_len - sizeof(usbmux_packet_t);
    usbmux_packet_t *upacket = usbmux_packet_alloc(payloadLength);

Any ideas? Maybe stack corruption in the previous memcpy (should be protected by the assertion, so...)?

Are you sure it's not the second assertion? I.e. assert(sizeof(usbmux_packet_t) == 16);
I'm working on a new version of peertalk where I fixed an issue pertaining to this, namely that depending on compiler version and settings, the actual size of a struct might be larger than what's needed (usually an optimization to align it to words.) This might cause sizeof to report not the logical size, but the effective size of the struct.
Can you repro the crash? If so, run the app through lldb (e.g. lldb /path/to/your.app/Contents/MacOS/your) and see where it actually aborts from an assertion. It's also possible you've got a stack overflow issue in which case the stack trace might be incorrect.
BTW, the fix to the struct size is this:
typedef struct usbmux_packet {
  ...
} __attribute__((__packed__)) usbmux_packet_t;

The second assert should crash on the first call, as sizeof(usbmux_packet_t) is known at compile time (and might be even optimized out). We are getting the traces through Crashlytics, and the issue repeats quite frequently.
I tried to replicate the call sequence to check for any unexpected value of payloadLength causing the crash, but couldn't. But note that nowhere is checked that the received upacket_len is bigger than sizeof(usbmux_packet_t), so a corrupt/unexpected packet of zero or less than 16 size would cause payloadLength to have a value near to UINT32_MAX (adding back 16 in usbmux_packet_alloc will cause this to allocate undersized packets).

b3090d72e56b8714e6ce7d7658038e3a77c4b38f makes some changes that should help in preventing getting into these situations. usbmux_packets are only used during negotiation with the USB mux hub, but isn't used for user communication (i.e. PTChannel doesn't use these packets and doesn't have these limitations.)


Evaluations Data

We would like to provide evaluation data as part of our API.
To this effect, we need to:

[ ] Extend our database schema to include Evalution data
[ ] Create a Golang model conforming to the schema
[x] Develop a scraper with chromedp to scrape the data available from https://coursebook.utdallas.edu/ues-report/

I've completed the scraper component of this work, but there are some concerns regarding IP ratelimits to be addressed. A data model and associated database changes still need to be completed.

I've completed the scraper component of this work, but there are some concerns regarding IP ratelimits to be addressed. A data model and associated database changes still need to be completed.

Upon further investigation of this, I'm not seeing any immediate great solutions for the IP ratelimit problem -- this problem also occurs with scraping courses, but in a much more manageable fashion. Scraping evals leads to a long IP ratelimit every 30-40 evals or so, which obviously isn't sustainable for scraping en masse.
A solution to this issue that I proposed to @iamwood would be to set up an API endpoint for evals that parses and returns specific evals on-the-fly rather than parsing them all en-masse. I'll discuss this alongside some other things once the semester starts rolling more.
Any thoughts on this issue are welcome!

+1 for caching + evals on-the-fly, I think it's a good compromise.

So, after putting together an on-demand scraping endpoint for evals, it seems like we are now being hindered on this front by evals being locked behind captcha verification. I'm not sure if there's any way to circumvent this, but I'm out of ideas for the time being. As such, I'm going to be putting this issue on hold in favor of prioritization of other tasks.


Update ContainerFactory.js

Given that Container's constructor allows x and y to be optional, the JSDoc should match that.
This PR (delete as applicable)

Updates the Documentation

Describe the changes below:
Given that the constructor for Container allows the x and y to be optional, the JSDoc for a GameObjectFactory should match that. This PR fixes this issue.

y is actually set to x if undefined, but it doesn't matter, this will do for now!


Dependency updates

Trivial updates

Hrm, looks like rustls now requires resolver in Cargo, so I guess we have to increase the MSRV from 1.50...

Ah, sure! Convenient that h3 is unreleased.


Automator: update proxy@master in istio/istio@master

Generated by Automator - 2020-12-10T22:55:57+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
b07907be244587c10415ac49a51addfeebe8f067
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
b07907be244587c10415ac49a51addfeebe8f067
link
/test integ-pilot-multicluster-tests_istio

integ-telemetry-k8s-tests_istio
b07907be244587c10415ac49a51addfeebe8f067
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio
618168f894d4fe7b6fc301b2e968911c7ff1c3e9
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


No longer in preview?

It looks like this feature was included in the GA release of Windows Admin Center v1809.  Can I assume that it's out of 'preview' status and fully supported for production use now?
Also, the note at the top of this article says the feature is only supported when Windows Admin Center preview is installed on a Windows 10 computer.  Is it still true for the GA release of WAC v1809 that this feature only works when WAC is installed on a Windows 10 computer?  Or is it available when WAC is installed on a server OS?

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

ID: 4a6e5837-7436-063b-96db-6a6fb481b842
Version Independent ID: d42ec8cb-c614-896e-50fb-34b6e7906c53
Content: Use Windows Admin Center to manage operating system updates with Azure Update Management
Content Source: WindowsServerDocs/manage/windows-admin-center/use/azure-update-management.md
Product: windows-server-threshold
GitHub Login: @Haley-Rowland
Microsoft Alias: harowl

This feature is GA and supported on both desktop and server installations. I will update the page.


Creating 3D objects with threejs

Hi Bill! One suggestion would be to add an example picture of what the end result should look like, as well as some images that show off what other have done using three.js!


store: expand getting the latest backup to return either dir or file

This is needed for darwin support, as we're restoring entire tree
structures.
Issue: https://github.com/adamdecaf/cert-manage/issues/9

Closing in favor of https://github.com/adamdecaf/cert-manage/pull/85


Refactor patron endpoint to get netid from identifiers

And for readability
closes #932

Coverage increased (+0.07%) to 81.604% when pulling 8231f09e5c3ffff2522fd2e066c25d0be48c4e0a on 932-netids into 60ed2f51088a68e18aabd31dfe523426427b6b56 on alma.

Coverage increased (+0.07%) to 81.604% when pulling 8231f09e5c3ffff2522fd2e066c25d0be48c4e0a on 932-netids into 60ed2f51088a68e18aabd31dfe523426427b6b56 on alma.


[BUG] Code samples are shifted one space to the right after the first line in https://docs.microsoft.com

Found via #17311.
While looking at the Key Vault docs I found that all code snippets are shifted one space to the right after the first line, even though the original code does not include said spaces. Take the sample for the decrypt() method as an example:

It should look like the same sample in GitHub, but does not because it has additional spaces as mentioned before.
Not sure if this is true for code snippets for other libraries/languages.

Will create an issue to docs team.

Will create an issue to docs team.

I created one already @sima-zhu

I created one already @sima-zhu

Thanks! https://dev.azure.com/ceapex/Engineering/_workitems/edit/358854/
Brought the issue to docs team. However, the display issue might take a while for docs team to pick up based on the priority.

Thanks! https://dev.azure.com/ceapex/Engineering/_workitems/edit/358854/
Brought the issue to docs team. However, the display issue might take a while for docs team to pick up based on the priority.


TickBlockEvent only thrown for non-overriding Blocks

I know that since @bloodmc's system is able to now wrap around when blocks are being both ticked normally and scheduled block updates are done, it should be possible to throw the TickBlockEvent at appropriate times (perhaps if not for normal ticks, but only for scheduled block updates or random ticks).

@bloodmc
Thoughts on this?

Doesn't TickBlockEvent already have two subclasses, one for random and one for scheduled?

me is right. They were introduced in https://github.com/SpongePowered/SpongeAPI/commit/a13a8d6250298bdcd050deeeed314bfea907e9ea
Implemented by https://github.com/SpongePowered/SpongeCommon/commit/60e57b0ce444babc985b4f98871515ed3eff2e53
This issue was opened months before, it can now be closed (?)

Correct, this should have been closed a while back when the CauseTracker was initially rewritten (dunno how may times it's been rewritten now, but well, it's been at least 2).


[WTF-1686] Document data source retrieval behavior [10.8]

All looks good to me. Have asked @ConnorLand to do a second review from his Front-end knowledge.

LGTM

All looks good to me. Have asked @ConnorLand to do a second review from his Front-end knowledge.

Good on Front End.


training_epochs not defined in Urban Sound Classification using CNN.ipynb

https://github.com/aqibsaeed/Urban-Sound-Classification/blob/5d913af248552e6cb3f635306f64faccd3d247af/Urban Sound Classification using CNN.ipynb uses training_epochs, but this is not defined. Perhaps this was intended to be training_iterations instead?

Fixed.


Support for OPTIONS request for CORS reasons

Hello,
We are using Seasocks on our custom built embedded Linux mobile device to serve up static endpoints that act as a remote control testing API to our device. We are mapping these REST endpoints to C++ functions. This would be something like this:
PUT /arithmetic/add?param1=2&param2=4
In our code, this is done by creating a a RootPageHandler object and adding the URL paths to this.
We are looking to add CORS to our API, so that if our device is sharing a local network connection with a host PC, a web browser on that host PC can call through to our device, and keep the data private and in the local loop. However, the only ability I see to add CORS is through the  web socket handers:
void Server::addWebSocketHandler(const char* endpoint, std::shared_ptr<WebSocket::Handler> handler,
        bool allowCrossOriginRequests) {
    _webSocketHandlerMap[endpoint] = { handler, allowCrossOriginRequests };
}

void Server::addPageHandler(std::shared_ptr<PageHandler> handler) {
    _pageHandlers.emplace_back(handler);
}

We attempted to add a header on our endpoints like this: writer->header("Access-Control-Allow-Origin", "*");, but this appears to be too far downstream and the browser returns the following error: "Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://xxxxx/system/serial-number. (Reason: CORS preflight response did not succeed).
Uncaught (in promise) TypeError: NetworkError when attempting to fetch resource.".
Is it possible to add CORS to the prefllight traffic? Is there an easier way to do this using Seasocks? Thank you in advance

It actually looks like it's in there twice....
and yes, grepping the code we already put Access-Control-Allow-Origin: * in our messages! (Connection.cpp)...!
So, as best I can tell the CORS is working...but
my guess is the "preflight" it's talking about is doing a HEAD or something? can you wireshark capture and see if that's the case (I'm clutching at straws here; we don't support HEAD right now but could...I think. It's bee a while...)

OK, I really apologize for this, but now web requests are working successfully; the failures early on in the process must have been some network hiccups/network IT configuration based.
We just removed the additional Access Control Allow Origin header, and now things are working. I'm confused as to what has changed, but maybe something on our corporate network changed... Very sorry, and thank you for the attention.
On another note: Even with the CORS stuff in there, I think our security experts will soon demand TLS support, so hopefully it's coming to Seasocks soon (I've seen the open tickets).
Thank you!

OK, here's a Wireshark trace from what we're seeing. In this case, we have a PC running Chrome attempting to access our "Monitor" - the embedded Linux device running seasocks with page handlers - with a GET request of 'http://<IP_ADDRESS>/system/serial-number. The Chrome application returns the following error information:

Access to fetch at 'http://<IP_ADDRESS>/system/serial-number' from origin 'http://dpcpo-farga-132dtn1y09cpq-1698599008.us-east-1.elb.amazonaws.com' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: It does not have HTTP ok status.

cors-request-packet-trace-2021-04-26.txt

Hi! Can you attach the packet capture itself instead of the text? It's much easier to debug in wireshark than in a text editor :) thanks!

ah!
I was able to read enough:
Hypertext Transfer Protocol
    OPTIONS /system/serial-number HTTP/1.1\r\n
    Host: <IP_ADDRESS>\r\n

is what was sent, which we don't support I think? Hard to tell

Sorry, here's the full trace (I hope). Unfortunately, getting this secondhand, as I'm having device/networking issues.
The request should have been a GET of this URL: 'http://<IP_ADDRESS>/system/serial-number'.
cors-request-packet-trace-wireshark.zip

We believe that the OPTIONS call is the pre flight check to ensure that the web domain is allowed to call that API. (Sorry, I'm not a HTTP expert; We believe this is part of the standard, so I'll apologize upfront if I have this wrong).

Right! That seems to be what's happening here. We'll need to support OPTIONS to get this to work. I'll rename this issue accoridngly.

Relevant docs: https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS#functional_overview

and https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/OPTIONS#preflighted_requests_in_cors


Customization

Hi,
Is this theme configurable, somehow?
I mean, change logo, and links at the top?
Thank you!

hi @ambs
unfortunately this is currently not really the case.
originally this theme was specifically developed for the COMPAS docs.
i only realized recently that people might want to use it for something else :)
will try to roll out an update with configuration options soonish...

Hi. Yes, it looked like that for me. But the these is quite good.
If I have some time I will try to help. Meanwhile this is a side personal project, so, we have plenty of time :)
Thanks for your answer.


Fix to allow running autopep8 from outside top_srcdir

Sometimes I like to build in a subdirectory but this script is not happy with that.

Cherry-picked to v9.2.x


Enable toggle for cloudtrail lambda

what

Allow user to disable cloudtrail lambda bucket log forwarding when needed

why

the options was not given before.

/test all

/test all

/test all

/test all

/test all

/test all


Add terms and conditions page

Context
Adding terms and conditions terms of use content page, linked to from the footer.
Changes proposed in this pull request
In addition to the new page and associated routes and translations:

updates app-styled-content class to cover more HTML that can be generated by markdown, ensures list items are rendered correctly, and renders headings one size larger to account for the addition of another heading level in the terms and conditions content.
uses the shorter<EMAIL_ADDRESS>markdown syntax for email addresses in the accessibility statement

Guidance to review
Visit http://localhost:3000/terms-conditions http://localhost:3000/candidate/terms-of-use
Link to Trello card
322 - Terms and conditions

Some last-minute changes to content needing to be made before this is ready to merge.

@tijmenb Not sure if this is the best resolution, but have updated route to use /candidate/terms-of-use. I tried moving this into :candidate_interface namespace, but that started causing follow-on issues with the content controller. Perhaps requires a separate PR to address having different candidate/provider content pages?


Make CowArray.freeThisReference @safe

This is needed for my build to pass when dmd flag -checkaction=context is used in my dub project.
Currently, the line
dflags "-checkaction=context"
in dub.sdl makes build error as
/home/per/.local/dlang/linux/bin64/src/phobos/std/uni/package.d(3376,17): Error: `this.data.ptr` cannot be used in `@safe` code, use `&this.data[0]` instead
/home/per/.local/dlang/linux/bin64/src/phobos/std/uni/package.d(2128,29): Error: template instance `std.uni.CowArray!(GcPolicy).CowArray.__ctor!(uint[])` error instantiating
/home/per/.local/dlang/linux/bin64/src/phobos/std/uni/package.d(2137,32):        instantiated from here: `__ctor!()`
/home/per/.local/dlang/linux/bin64/src/phobos/std/uni/package.d(1956,29):        instantiated from here: `InversionList!(GcPolicy)`

I can replace more of the .ptr statements if someone feels the need to.

This seems to be a dmd/druntime issue if -checkaction=context makes a difference.

This seems to be a dmd/druntime issue if -checkaction=context makes a difference.

So, how to move forward?

So, how to move forward?

Improving the @safety is good but we should determine why -checkaction=context makes a difference. I assume other assertion modes are missing safety checks because the error message seems valid.
BTW: Fix for the CircleCI failure: #7548

-checkaction=context fails because the expression is lowered as:
assert(!arr.ptr, _d_assert_fail!"!"(arr.ptr));

The lowering strips the ! to format the actual value instead of true/false

-checkaction=context fails because the expression is lowered as:
assert(!arr.ptr, _d_assert_fail!"!"(arr.ptr));

The lowering strips the ! to format the actual value instead of true/false

So how to progress with this assert statement?
Should we change this commit to
assert(!cast(bool)data.ptr);

for now or should we fix the implementation or lowering of _d_assert_fail!?

-checkaction=context fails because the expression is lowered as:
assert(!arr.ptr, _d_assert_fail!"!"(arr.ptr));

The lowering strips the ! to format the actual value instead of true/false

Ideally the compiler would need to introspect the definition  of _d_assert_fail for this specific overload (template params "!") to determine that the access to arr.ptr in this case is safe. Provided printing the expression arr.ptr is @safe.

So how to progress with this assert statement?

I guess the implementation for checkaction=context could detect XXX.ptr and then @trust the read.

If we were to redefine _d_assert_fail as a template mixin we could help the compiler understand that this use case is @safe.

That wouldn't work because _d_assert_fail calls other functions to format it's argument(s).

That wouldn't work because _d_assert_fail calls other functions to format it's argument(s).

But what wrapping that case in a @trusted lambda?

What about wrapping that case in a @trusted lambda?

Maybe but the compiler should only trust the read, not the entire assert. Otherwise it would create @safe holes, e.g.
void main() @safe
{
     int[] arr = ....;
     assert(arr.ptr is bad());
}

int* bad() @system { .... }

Fix: dlang/dmd#11337

What about sticking to
Assert(!!
for now.


[Multicast] Use group as flow actions for multicast traffic

Add local pod receivers into an OpenFlow "all" type group for each
multicast group, and use such group in the flow action for multicast
traffic. Remove pod from group buckets if the pod has left the
multicast group or is deleted before leaving multicast group.
Improve e2e tests.

Signed-off-by: wenyingd<EMAIL_ADDRESS>

/test-all
/test-multicast-e2e
/skip-windows-all
/skip-ipv6-all
/skip-ipv6-only-all

Codecov Report

Merging #3508 (6becda3) into main (2c7d486) will decrease coverage by 2.24%.
The diff coverage is 27.18%.

@@            Coverage Diff             @@
##             main    #3508      +/-   ##
==========================================
- Coverage   65.58%   63.33%   -2.25%
==========================================
  Files         277      277
  Lines       27264    27345      +81
==========================================
- Hits        17881    17319     -562
- Misses       7483     8198     +715
+ Partials     1900     1828      -72

Flag
Coverage Δ

kind-e2e-tests
52.15% <1.94%> (-3.80%)
:arrow_down:

unit-tests
43.03% <25.74%> (-0.05%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/agent/cniserver/pod_configuration.go
53.51% <ø> (ø)

pkg/agent/interfacestore/interface_cache.go
75.55% <0.00%> (-1.72%)
:arrow_down:

pkg/agent/interfacestore/types.go
69.56% <ø> (ø)

pkg/agent/openflow/client.go
67.75% <0.00%> (-2.19%)
:arrow_down:

pkg/agent/openflow/framework.go
92.85% <0.00%> (-1.35%)
:arrow_down:

pkg/agent/openflow/multicast.go
0.00% <0.00%> (ø)

pkg/agent/openflow/pipeline.go
74.43% <0.00%> (-2.50%)
:arrow_down:

pkg/agent/proxy/proxier.go
60.00% <0.00%> (-2.72%)
:arrow_down:

pkg/agent/multicast/mcast_controller.go
63.96% <50.00%> (-5.99%)
:arrow_down:

pkg/ovs/openflow/ofctrl_bridge.go
56.43% <66.66%> (-1.81%)
:arrow_down:

... and 41 more

/test-multicast-e2e

/test-all
/test-multicast-e2e
/skip-windows-all
/skip-ipv6-all
/skip-ipv6-only-all

/test-all
/test-multicast-e2e
/skip-windows-all
/skip-ipv6-all
/skip-ipv6-only-all

/test-multicast-e2e

/test-all
/test-multicast-e2e
/skip-windows-all
/skip-ipv6-all
/skip-ipv6-only-all

/test-all
/test-multicast-e2e
/skip-windows-all
/skip-ipv6-all
/skip-ipv6-only-all

/test-multicast-e2e

After checking the test code and running several rounds on e2e testbed,
I suggest adding two t.Parallel() in testMulticastBetweenPodsInThreeNodes and testMulticastBetweenPodsInTwoNodes to parallelize tests to further speed up e2e tests, cutting total running time in half approximately.
t.Run(mc.name, func(t *testing.T) {
		t.Parallel()
		runTestMulticastBetweenPods(t, data, mc, nodeMulticastInterfaces)
})

/test-all
/test-multicast-e2e
/skip-windows-all
/skip-ipv6-all
/skip-ipv6-only-all

/test-all
/test-multicast-e2e
/skip-windows-all
/skip-ipv6-all
/skip-ipv6-only-all

@jianjuns @liu4480 @ceclinux @hongliangl May I know if you have other comments on this PR?

@jianjuns @liu4480 @ceclinux @hongliangl May I know if you have other comments on this PR?

Not from me. But I see there is still a pending question in the comments for @tnqn ?

@jianjuns @liu4480 @ceclinux @hongliangl May I know if you have other comments on this PR?

I added one comment. Others LGTM.

/test-all
/test-multicast-e2e
/skip-windows-all
/skip-ipv6-all
/skip-ipv6-only-all

/test-networkpolicy
/test-e2e

/test-all
/test-multicast-e2e
/skip-windows-all
/skip-ipv6-all
/skip-ipv6-only-all

Since this PR switches to use group for multicast, are the previous configurations like mcast_snooping_enable and mcast-snooping-disable-flood-unregistered still required?

No, these configurations are not required. I removed the call in my latest update. But do you think we should also remove the settings in OVSDB for upgrade case?

Since this PR switches to use group for multicast, are the previous configurations like mcast_snooping_enable and mcast-snooping-disable-flood-unregistered still required?

No, these configurations are not required. I removed the call in my latest update. But do you think we should also remove the settings in OVSDB for upgrade case? It doesn't take bad effect even if we don't remove them because we don't use "normal" actions in OpenFlow entries, so the traffic is actually forwarded by Antrea flows but not OVS multicast db cache.

I think no need to remove the setting for ugprade case. It was alpha and harmless as you pointed out.

/test-all
/test-multicast-e2e
/skip-windows-all
/skip-ipv6-all
/skip-ipv6-only-all

/test-all
/test-multicast-e2e
/skip-windows-all
/skip-ipv6-all
/skip-ipv6-only-all

/test-all
/test-multicast-e2e
/skip-windows-all
/skip-ipv6-all
/skip-ipv6-only-all


[WIP, Discuss] Make packages folder optional

Current design:

It works with the storage: none and storage: packages setting

Some internal behavior has been changed to make this possible

Moved "paket.locked" to paket-files folder
Nuspec is now always loaded from .nuget folder (this breaks people
doing manual changes to nuspec files in the "packages" folder)
Packages are always extracted into the .nuget/packages folder (even if
they are cli-tools)

[x] Introduce the required infrastructure

[x] Make sure stuff still works

[ ] Test and add test for the new feature

[ ] Add review comments / change according to feedback

NOTE: Currently based on v2_vs_v3 branch (until that one is merged). This makes the diff easier to review (and easier for me ;))

Whoa this seems to slowly start working. And the garbage collector is even removing everything :)

There is still one bug like:
Paket failed with
-> ArgumentException: The input sequence was empty.
   Parameter name: source
   StackTrace:
        at Microsoft.FSharp.Collections.SeqModule.ExactlyOne[T](IEnumerable`1 source)
        at Paket.NuGet.GetContent@87.Invoke(Unit unitVar)
        at System.Lazy`1.CreateValue()
        at System.Lazy`1.LazyInitValue()
        at <StartupCode$Paket-Core>.$DependencyCache.exprs@207-5.Invoke(Unit unitVar)
        at Microsoft.FSharp.Control.AsyncBuilderImpl.callA@839.Invoke(AsyncParams`1 args)
     --- End of stack trace from previous location where exception was thrown ---
        at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
        at Microsoft.FSharp.Control.AsyncBuilderImpl.commit[a](AsyncImplResult`1 res)
        at Microsoft.FSharp.Control.CancellationTokenOps.RunSynchronouslyInCurrentThread[a](CancellationToken token, FSharpAsync`1 computation)
        at Microsoft.FSharp.Control.CancellationTokenOps.RunSynchronously[a](CancellationToken token, FSharpAsync`1 computation, FSharpOption`1 timeout)
        at Microsoft.FSharp.Control.FSharpAsync.RunSynchronously[T](FSharpAsync`1 computation, FSharpOption`1 timeout, FSharpOption`1 cancellationToken)
        at Paket.DependencyCache.SetupGroup(GroupName groupName)
        at Paket.DependencyCache.OrderedGroups(GroupName groupName)
        at Paket.LoadingScripts.ScriptGeneration.cachedGroups@184-1.Invoke(GroupName g)
        at Microsoft.FSharp.Collections.Internal.IEnumerator.map@74.DoMoveNext(b& )
        at Microsoft.FSharp.Collections.Internal.IEnumerator.MapEnumerator`1.System-Collections-IEnumerator-MoveNext()
        at Microsoft.FSharp.Collections.Internal.IEnumerator.map@74.DoMoveNext(b& )
        at Microsoft.FSharp.Collections.Internal.IEnumerator.MapEnumerator`1.System-Collections-IEnumerator-MoveNext()
        at Microsoft.FSharp.Collections.SeqModule.ToList[T](IEnumerable`1 source)
        at Paket.LoadingScripts.ScriptGeneration.generateScriptContent(PaketContext _arg1)
        at Paket.LoadingScripts.ScriptGeneration.scriptData@315-1.GenerateNext(IEnumerable`1& next)
        at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.MoveNextImpl()
        at Microsoft.FSharp.Collections.SeqModule.ToList[T](IEnumerable`1 source)
        at Paket.LoadingScripts.ScriptGeneration.scriptData@313.GenerateNext(IEnumerable`1& next)
        at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.MoveNextImpl()
        at Microsoft.FSharp.Collections.SeqModule.ToList[T](IEnumerable`1 source)
        at Paket.LoadingScripts.ScriptGeneration.constructScriptsFromData(DependencyCache depCache, FSharpList`1 groups, FSharpList`1 providedFrameworks, FSharpList`1 providedScriptTypes)
        at Paket.RestoreProcess.CreateScriptsForGroups(DependenciesFile dependenciesFile, LockFile lockFile, FSharpMap`2 groups)
        at Paket.RestoreProcess.Restore@471-2.Invoke(Unit unitVar0)
        at Paket.Utils.RunInLockedAccessMode[a](String rootFolder, FSharpFunc`2 action)
        at Paket.RestoreProcess.Restore(String dependenciesFileName, FSharpOption`1 projectFile, Boolean force, FSharpOption`1 group, FSharpList`1 referencesFileNames, Boolean ignoreChecks, Boolean failOnChecks, FSharpOption`1 targetFrameworks)
        at Paket.Dependencies.Restore(Boolean force, FSharpOption`1 group, FSharpList`1 files, Boolean touchAffectedRefs, Boolean ignoreChecks, Boolean failOnChecks, FSharpOption`1 targetFramework)
        at Paket.Dependencies.Restore(Boolean force, FSharpOption`1 group, Boolean onlyReferenced, Boolean touchAffectedRefs, Boolean ignoreChecks, Boolean failOnFailedChecks, FSharpOption`1 targetFramework)
        at Paket.Program.restore(ParseResults`1 results)
        at Paket.Program.main@737-12.Invoke(ParseResults`1 results)
        at Paket.Program.processWithValidation[T](Boolean silent, FSharpFunc`2 validateF, FSharpFunc`2 commandF, ParseResults`1 result)
        at Paket.Program.processCommand[a](Boolean silent, FSharpFunc`2 commandF, ParseResults`1 result)
        at Paket.Program.main()

Probably trying to analyze the wrong (empty) folder.

Ok if this goes green I consider this "ready".
Just to be sure here: While refactoring I did take some care to not break existing functionality (as marked above), so storage: packages or the default are probably still quite stable.
The new feature storage: none is in alpha/beta. Therefore we should advertise as that.
However, I think we can make a stable as long as the new feature clearly is advertised as alpha/beta.

Seems like I broke something :/

I don't understand why I now get a PathTooLongException :/


no history for this page, not attached to Hybrid Modern Auth

this page should be connected to this thread:
Office 365 Enterprise  / Plan for Office 365 /  Hybrid modern authentication
See related Github report from me: https://github.com/MicrosoftDocs/OfficeDocs-Enterprise/issues/434
it's very hard to find for our customers.

@officedocsbot assign @mypil

Closing this issue as it is a duplicate or related to issue #434.

@officedocsbot reopen


Graphs in builtin_types.ipynb are sometimes excessively scaled

In builtin_types.ipynb, the two graphs are huge--that is, with far greater scaling than they should be and otherwise are--under some circumstances.
Specifically, this happens on GNU/Linux systems (I've tested on a couple Ubuntu systems and on Debian 11) when pipenv is used for installation. It doesn't seem to happen when conda is used for installation. I don't know if there are other situations where this occurs.
I suspect a problem with GraphViz itself, since conda installs GraphViz itself but pipenv doesn't, defaulting to the system or manually installed version. I had thought that this affected even the display of graphs that were created without the problem, but I am unable to reproduce that. In a recent test on Ubuntu 23.04 (development version) using pipenv, redrawing the graph makes this, among other changes:
-       "<!-- Generated by graphviz version 7.1.0 (20230121.1956)\n",
+       "<!-- Generated by graphviz version 2.43.0 (0)\n",

This suggests the difference in GraphViz version matters. This may not really be related to operating system, per se, except that some operating systems' package managers provide GraphViz but at much earlier than the latest version. This is possibly related to GraphViz issue 1605, or to GraphViz issue 1855 which was fixed in GraphViz 3.0.0.
Screenshot in VS Code:

Screenshot in JupyterLab:

Notice the scaling of the graph drawing (including the text labels) compared to the text shown in the notebook.
This doesn't seem to ever happen on Windows, nor does it seem to happen on any system when conda rather than pipenv is used to install the dependencies.
Note: The paragraph about it possibly being related to GraphViz versions has been updated, reflecting a change in my viewpoint.

Quoting from #2:

The dev container works around #5, but it won't be reasonable to
close until, at minimum, the issue is documented in the readme.


Understanding the copa update implementation

Hello,
In the mvfst implementation Copa increases or decreases the congestion window by a value of  (ack.ackedPackets.size() * conn_.udpSendPacketLen *                            conn_.udpSendPacketLen * velocityState_.velocity) /           (deltaParam_ * cwndBytes_);. But I noticed that this update calculation is different from the other implementations given in https://web.mit.edu/copa/. I was hoping to understand the reasoning behind the update mechanism used here in mvfst.
Thanks

I think the paper has cwnd in packets, and the addition or reduction in the paper is done per ack.
The code you find in mvfst is done per onPacketAcked call which represents acks more than one packet.
Basically the paper says, on ack, you change this amount: (velocityState_.velocity) / (deltaParam_ * cwnd in packets)
In our case, onPacketAcked acks ack.ackedPackets.size() amount of packets, thus we should change
ack.ackedPackets.size() * conn_.udpSendPacketLen * (velocityState_.velocity) / (deltaParam_ * cwnd in packets) = (ack.ackedPackets.size() * conn_.udpSendPacketLen * conn_.udpSendPacketLen * velocityState_.velocity) / (deltaParam_ * cwndBytes_)

Does that answer the question?

Yes, that answers it. Thanks!


OpenAPI gives "trigger does not match" error

Open API code generation gives erorr
Steps to reproduce.

Place the devonfw_employee.yml in the base directory of the core module
Right click on the yml file and select Cobigen > Generate
It gives the below error popup.

It seems service findEmployee is duplicated on your yml file:

You can check here https://editor.swagger.io/ the validation

It seems service findEmployee is duplicated on your yml file:

You can check here https://editor.swagger.io/ the validation

Removed and tried.. same error.

The problem right now is that for some reason having different matchers for crud_openapi_angular_service_based_app (componentDef) and crud_openapi_angular_client_app (EntityDef) leads to this error.
Putting the same for both makes generation possible but that would obviously not be correct, so we need to make sure that both the triggers are recognized properly.

@jan-vcapgemini was this the issue you already addressed?

@jan-vcapgemini was this the issue you already addressed?

Yes, and I've worked with @MikeSchumacherCapgemini on this issue too.

The only occurence of this problem has been disabled for now.
See #1278 for more information


adding in olmconfig.yaml to the controller

Issue #, if available:

Fixes: aws-controllers-k8s/community#1422

Description of changes:

Adding the olmconfig.yaml to onboard this controller to OperatorHub.
Please let me know if any branding info needs to change

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Relates: redhat-openshift-ecosystem/community-operators-prod#1523
Relates: k8s-operatorhub/community-operators#1591

/test prometheusservice-unit-test


need a way to get engine field of EVP_PKEY

need a way to get engine field of EVP_PKEY, just like EVP_PKEY_get_engine(), version 1.1.x or master?
https://github.com/openssl/openssl/blob/6f0ac0e2f27d9240516edb9a23b7863e7ad02898/crypto/include/internal/evp_int.h#L349-L377
/*
 * Type needs to be a bit field Sub-type needs to be for variations on the
 * method, as in, can it do arbitrary encryption....
 */
struct evp_pkey_st {
    int type;
    int save_type;
    int references;
    const EVP_PKEY_ASN1_METHOD *ameth;
    ENGINE *engine;
    union {
        void *ptr;
# ifndef OPENSSL_NO_RSA
        struct rsa_st *rsa;     /* RSA */
# endif
# ifndef OPENSSL_NO_DSA
        struct dsa_st *dsa;     /* DSA */
# endif
# ifndef OPENSSL_NO_DH
        struct dh_st *dh;       /* DH */
# endif
# ifndef OPENSSL_NO_EC
        struct ec_key_st *ec;   /* ECC */
# endif
    } pkey;
    int save_parameters;
    STACK_OF(X509_ATTRIBUTE) *attributes; /* [ 0 ] */
    CRYPTO_RWLOCK *lock;
} /* EVP_PKEY */ ;

Also is there a way to get type field?

EVP_PKEY_id() returns the type field.

Simplest way would be like EVP_PKEY_id():
ENGINE* EVP_PKEY_engine(const EVP_PKEY *pkey)
{
return pkey->engine;
}
But maybe this needs reference management with get0/get1 APIs?
EVP_PKEY_get0_engine()
EVP_PKEY_get1_engine()

I'm curious: why do you need to get that field?

Hi Stephen,
Stephen Henson wrote:

I'm curious: why do you need to get that field?

I'm not sure why George ask for EVP_PKEY... function.
For RSA and DSA there is method like RSA_get0_engine and
DSA_get0_engine, so in few steps
engine could be obtained :
...
switch (EVP_PKEY_base_id(pkey)) {
case EVP_PKEY_RSA: { RSA *rsa = EVP_PKEY_get0_RSA(pkey); e =
RSA_get0_engine(rsa); ....
...
OpenSSL 1.1+ lack equivalent function for EC keys and there is no way to
assess engine.
For instance in e_nss I store on engine as extra data some information
like debug level and etc.
As work around I use global variable.

Yes only EC keys are missing this method. Perhaps just adding an EC_KEY_get0_engine() for completeness is all that's required?
Also I've noticed that there's an ENGINE* inside evp_pkey_st as well as inside each 'union { dsa/rsa/dh/ec } pkey'. Is this duplication necessary?

Also I've noticed that there's an ENGINE* inside evp_pkey_st as well as inside each 'union { dsa/rsa/dh/ec } pkey'. Is this duplication necessary?

Depends on what functionality you use.  If the caller ends up using the EVP interface, chances are that the EVP_PKEY engine pointer is enough.  However, if the caller uses the raw algo functions on the raw keys (i.e. for example pick out the RSA* from the EVP_PKEY and uses the RSA functions), then the raw key level engine pointer will be used.
We try to encourage people to use the EVP interface.  I assume the transition is going on... slowly.

@petrovr, I've an answer for you: #2960


action with only adds causes a C++ compile error

Following ecsact file causes a C++ compile error. It's a bit of a weird ecsact file, but it shouldn't have a compile error.
package example;
component ComponentAddRemove;
action AddComponent {
	adds ComponentAddRemove;
}

I found out it's because EnTT doesn't allow an excludes only view but it's planned! See issue: https://github.com/skypjack/entt/issues/929
Instead of working around it we can wait until that feature is available.


Artifact MNAIO image download logs

In this patch we ensure that the download logs for MNAIO images
are stored along with the other job artifacts so that we can easily
review them to determine any issues which arose causing the download
to fail.
JIRA: RE-2072
(cherry picked from commit 3e10bc78177701dd090d683dd4bf6be59cf0d422)
Issue: RE-2072

Thank you for submitting a pull request to this project!
Once all check jobs have passed and the pull request has received a
sufficient number of reviews, the repository's gate jobs can be triggered by
adding a comment of :shipit: to this pull request.
When the gate jobs have completed successfully, this pull request will get
merged automatically.


implement certain file related verbs

INITFILE "TEST"
ERASE "TEST"
STRING "TEST"
DIRECT "TEST",3,3,2,3

INITFILE "TEST",ERR=*next
ERASE "TEST",ERR=*next
STRING "TEST",ERR=*next
DIRECT "TEST",3,3,2,3,ERR=*next

Closed, because implemented by #141


Open websocket lib PR against psanford/wormhole-william

Open #6 against psanford/wormhole-william to begin dialog.

See https://github.com/psanford/wormhole-william/pull/46


Product data should only load for product related pages

Right now it loads on every page load... no need.

https://github.com/socialbaking/karma/blob/5e81ef19ee5b14ea1a4edb21c00930658f42b25f/src/karma/view/index.tsx#L89-L97


How to deal with bias=True Model

Feature request
How to deploy model within bias=True. Example: vinai/PhoGPT-7B5-Instruct
Motivation
.


Update README.md

Hi, my name is Yiwen. I am a student working on an assignment that helps improve the ReadMe documents of the open-source GitHub projects. In this Pull Request, I am proposing some improvements for your ReadMe.
"of"->"for"
"consider"-> "considers"
"is"->"are"
"which"->"that"

Hi @yiwenwang2090,
if you want to contribute please apply your changes to: https://github.com/Ultimate-Hosts-Blacklist/test-launcher/tree/master/ultimate_hosts_blacklist/test_launcher/data/docs too.


fix: grab cornerstone instance when called, not module load

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How to decode numpy arrays that require allow_pickle=True

Hi thank you very much for your work. I am currently using TarWriter to create a custom tar file of a Dataset. In it I encode a numpy which requires the allow_pickle=True to be then decoded with numpy.load. Currently, is there a way to pass this flag to the numpy decoder if necessary? Or is the suggested way to save is as binary file and saving it as a pyd file? I do not think the two should be different once decoded.

you could pass a callable to .decode(customCallable). which takes in two arguments key and value. See this here:
https://github.com/webdataset/webdataset/blob/1e47aa3de66a23fed0d0f28e787c46beb98ac538/webdataset/autodecode.py#L543

WebDataset by default uses allow_pickle=False for .npz and .npy extensions by default. That is the safer and more common default; it's generally not such a good idea to put objects into numpy arrays.
If you do want to decode .npy or .npz data with allow_pickle=True, you can do so by adding a decoder for these extensions that calls numpy.load with allow_pickle=True, or you can just do the decoding in a .map(sample) before the default decoders. Either way, it is something you should do explicitly.
The preferred way is to use .pyd for objects that require pickling, and .npy and .npz for pure numpy arrays.


Temporary file Xdone was not removed + cleanups

This PR removes temporary file Xdone which was not deleted
when running:
$ make test_vim9_script

I also see a temporary file Xdef185, which is not deleted.
I renamed Xdef* as it was used in several functions to help
find where it came from. After my change it's file
XScriptSuccess185 which is not deleted.
In vim9.vim, line 56 is executed, but line 57 is not
executed somehow:
 52 def CheckScriptSuccess(lines: list<string>)
 53   var fname = 'XScriptSuccess' .. s:sequence
 54   s:sequence += 1
 55   writefile(lines, fname)
 56   exe 'so ' .. fname
 57   delete(fname)
 58 enddef

I don't understand why.

Dominique wrote:

This PR removes temporary file Xdone which was not deleted
when running:
$ make test_vim9_script

I also see a temporary file Xdef185, which is not deleted.
I renamed Xdef* as it was used in several functions to help
find where it came from. After my change it's file
XScriptSuccess185 which is not deleted.
In vim9.vim, line 56 is executed, but line 57 is not
executed somehow:
 52 def CheckScriptSuccess(lines: list<string>)
 53   var fname = 'XScriptSuccess' .. s:sequence
 54   s:sequence += 1
 55   writefile(lines, fname)
 56   exe 'so ' .. fname
 57   delete(fname)
 58 enddef

I don't understand why.

I suspect the "cd" in the script makes the file name invalid.
Since the directory is not restored, this only happens once.
The other test files would then be created in the home directory.
Let's restore the directory so that something like this can't happen.
--
If your company is not involved in something called "ISO 9000" you probably
have no idea what it is.  If your company is involved in ISO 9000 then you
definitely have no idea what it is.
(Scott Adams - The Dilbert principle)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


About how to draw the heatmap picture

Thanks for your great work! I got a result KLD=1.252,SIM=0.398,NSS=1.139,But I don't know how to visualize and generate a beautiful heatmap,can you provide some methods?or maybe the python procedures.

Hi, the visualization code is provided in the utils/viz.py. You can enable the visualization by specifying the argument args.viz in training or testing.

Thank you! Is it ok to just replace "default=False" with "default=True" in parser.add_argument('--viz', action='store_true', default=False)?I'm running it right now.

Got a result now,for example:

Good, feel free to modify the code to suite your needs.


Browse, duplicates too aggressive

As Sinetheta wrote, bpni seems to much aimed at creators, not great for seeing others' work.
Anyway, currently the browse function strips away "duplicates" in a non-transparent way. I have three base designs with the word 'chillbox' in the title. None show up in Browse search unless I click 'show duplicate blueprints'. Similarly my pacu farm blueprint also doesn't show up if you search for 'pacu'.

Just wanted to add as I came across the same thing, made a 16x4 "template" of an empyy room to quicly have a start to add prints for starter rooms such as barracks, latrine, mess/great hall... the latter 3 are flagged as dupes.

Not sure how the site is coded, but a seemingly simple band aid for this could be to have the "Show duplicate blueprints" box ticked by default, there doesn't seem to be many if any dupes atm anyways.
<input type="checkbox" value="undefined" aria-checked="true">`


Fix markdown syntax in README.md

I stumbled across this repository while looking for ways to visualize sleep data from the fitbit api. There was some minor errors in README.md that made it display incorrectly, so I thought I would offer a drive-by fix.

Thanks for the fix.


The "SkipNonexistentTargets" parameter is not supported by the "MSBuild" task.

When loading a project with this attribute SkipNonexistentTargets in vscode on Mac OSX.
https://github.com/OrchardCMS/OrchardCore/blob/04694c0e56d0d55963f44c90f3ba1f257045a185/src/OrchardCore/OrchardCore.Application.Targets/OrchardCore.Application.Targets.targets#L14

Hi, I think this can help
https://github.com/OrchardCMS/OrchardCore/issues/1523

Thanks.


update(rules): add rules for PKT mining pools

/kind feature
/area rules
What this PR does / why we need it:
PacketCrypt (PKT) is a bandwidth-hard proof-of-work cryptocurrency. This particular coin makes mining viable in environments with limited compute resources but healthy network pipes.
PKT mining requires announcing to a pool, and there are only a few pools available because they need to be the size of a small datacenter to accommodate all of the incoming bandwidth from the protocol.
Which issue(s) this PR fixes:
This PR adds all 4 of the publicly known PKT pools as of 6/2023 to the miner_domain rules.
Special notes for your reviewer:
I've alphabetized https_miner_domains but can revert that if desired.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: blampe
Once this PR has been reviewed and has the lgtm label, please assign leogr for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1b9a142 add rules for PKT mining pools

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Welcome @blampe! It looks like this is your first PR to falcosecurity/rules 🎉

Hello @blampe and thank you for this PR!
To let these changes in, we need commit messages to adhere to conventional commits, could you please modify it? Thanks

@loresuso sure! Prefixed with update(rules): LMK if add(rules) or something else is more appropriate!

Thanks @blampe!
Some other PRs were merged, if you rebase on master we can let this in :)

Hi @blampe, does it still make sense to split those domains in http and https? those domains needs to be picked up by falco anyway.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: blampe
Once this PR has been reviewed and has the lgtm label, please assign jasondellaluce for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks @blampe!
Some other PRs were merged, if you rebase on master we can let this in :)

@loresuso apologies, I've been traveling for a couple weeks. This has been rebased on main.

Hi @blampe, does it still make sense to split those domains in http and https? those domains needs to be picked up by falco anyway. Having specific protocol related lists make them more complicated to maintain. Whatever protocol they use, they need to be detected. Perhaps adding the port 80, 443 in miner_ports would simplify the overall condition and make it easier and more efficient.

@darryk10 that's a good point. I only added them to the respective http/https lists because there seemed to already be a convention. I don't know what the long-term plans are for those lists and I'm happy to do whatever's most maintainable and aligned with the project's long-term goals. I'm not familiar with the behavior of miner_ports, and my only hesitation is if that might create false positives due to 80/443 being so ubiquitous. (A rule that flagged traffic to these ports would catch non-crypto traffic, for example.)

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

@poiana: Closed this PR.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Saving images on Android in quick succession

If multiple images on android are saved within the same second, they are given the same name (see here) and so overwrite each other. This leaves only the most recent image that was saved in that second.

you could work around this by giving the images names yourself.

you could work around this by giving the images names yourself.


DirectPacketModMeter and PacketModMeter changes

p4info.proto changes
p4runtime.proto changes
spec changes to include section for Intel IPU ES2100

As per our discussion, we are merging this as in. We will make the review comment changes when compiler makes that change too


Integrate With OpenTelemetry

OpenTelemetry is a collection of tools, APIs, and SDKs. You can use it to instrument, generate, collect, and export telemetry data (metrics, logs, and traces) for analysis in order to understand your software's performance and behavior. I think it's time to integrate the EventMesh with it now since OpenTracing and OpenCensus have been integrated into one. In this summer, you are supposed to finish the following tasks:

Finish the OpenTelemetry instrument with EventMesh Java SDK.
Finish the OpenTelemetry EventMesh Connector, let it export the data to EventMesh and receive data from EventMesh.

I'm very interested in this. Maybe I can help

@tydhot welcome and looking forward to your contirbutes. thanks.

@tydhot Would you like to share some new design for us? thanks.

@qqeasonchen hi, maybe you can add my WeChat. My WeChat  id is same as my GitHub id.We can discuss it.

@tydhot sure.

Any update?


Consider converting brainiak to a pkgutil namespace package

This way, other brainiak projects can have decoupled repos / release cycles. Documentation here.
One unresolved issue that @mihaic and I were discussing: namespace packages do not allow any other code to be run in brainiak/__init__.py, which is problematic for checking a user's Python version number. This is if brainiak was installed in any way via pip or setuptools, but will cause problems if, for example, a user has Python 3.2 and is using brainiak code without properly installing it as a python package.

So we have everything in one place, the only change required (version checking aside) is replacing the contents of brainiak/__init__.py with
__path__ = __import__('pkgutil').extend_path(__path__, __name__)


Coinomi added SegWit support

SegWit support was added in the latest beta of the wallet.

Update: SegWit is added in production.


graphql: Add an exists filter operation.

The exists filter operation appears at every nested level regardless
of whether the level represents a link or a property. The operation
takes either a true or false, which translates to EXISTS and NOT EXISTS EdgeQL respectively.
Fixes #1655

I actually ran into a weird bug here it has to do with compiling {exists: true} into EXISTS expr and {exists: false} into NOT EXISTS expr. When the literal true or false in GraphQL is replaced with a variable (which apparently happens by default at the moment) the resulting EdgeQL query + variables has an unused boolean variable. This causes a problem when the variable corresponding to exists changes value, but the underlying EdgeQL doesn't get recompiled, leaving the "original" EXISTS or NOT EXISTS in place and so getting the result intermittently wrong (depending on the cached GraphQL history).
I fixed that by compiling into EXISTS (path_expr) = <bool>$var this way using a variable for exists works out correctly because the EdgeQL query handles the variable change without special case recompiling.
Regarding docs, though, why are GraphQL sections apparently hidden in the documentation site? It's possible to search and find "GraphQL Mutations" and other sections via search, but not directly like other clients' docs. Was it hidden on purpose? I forget if we basically wanted to move it away from the main docs or not.

Regarding docs, though, why are GraphQL sections apparently hidden in the documentation site? It's possible to search and find "GraphQL Mutations" and other sections via search, but not directly like other clients' docs. Was it hidden on purpose? I forget if we basically wanted to move it away from the main docs or not.

Seems to be a regression. Try to debug it.

Regarding docs, though, why are GraphQL sections apparently hidden in the documentation site? It's possible to search and find "GraphQL Mutations" and other sections via search, but not directly like other clients' docs. Was it hidden on purpose? I forget if we basically wanted to move it away from the main docs or not.

Seems to be a regression. Try to debug it.

I'll merge this change (which contains the actual documentation updates) and look into why the site is not rendering that section separately (since it might be a site issue and not the docs issue as such).


libqmi: enableParallelBuilding

Works fine for me with -j48.

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Same as https://github.com/NixOS/nixpkgs/pull/85549#issuecomment-616174998


add support of spago@next

Adds support for spago next.

Thanks, but I'm holding off on doing this for my libraries for now. I want to include version ranges for the spago dependencies, and because of that I want to write a CI check that will ensure they're in sync with the bower ones. Hopefully I'll have time to do that in the coming week.

Anyway this is for another PR, I just split it.


Error running sqla app.py/run_server.py

Using the instructions in the README.rst the example app at python examples/sqla/app.py is broken after.
857d9f5f Merge pull request #1728 from flask-admin/fix-search-placeholder

At f6198b45 I get the following error
hughest@argon [0] $ python examples/sqla/app.py
/home/hughest/tmp/flask-admin/env/lib/python3.7/site-packages/flask_sqlalchemy/__init__.py:835: FSADeprecationWarning: SQLALCHEMY_TRACK_MODIFICATIONS adds significant overhead and will be disabled by default in the future.  Set it to True or False to suppress this warning.
  'SQLALCHEMY_TRACK_MODIFICATIONS adds significant overhead and '
Traceback (most recent call last):
  File "examples/sqla/app.py", line 327, in <module>
    admin.add_view(UserAdmin(User, db.session))
  File "/home/hughest/tmp/flask-admin/env/lib/python3.7/site-packages/flask_admin/contrib/sqla/view.py", line 329, in __init__
    menu_icon_value=menu_icon_value)
  File "/home/hughest/tmp/flask-admin/env/lib/python3.7/site-packages/flask_admin/model/base.py", line 818, in __init__
    self._refresh_cache()
  File "/home/hughest/tmp/flask-admin/env/lib/python3.7/site-packages/flask_admin/model/base.py", line 945, in _refresh_cache
    self._refresh_filters_cache()
  File "/home/hughest/tmp/flask-admin/env/lib/python3.7/site-packages/flask_admin/model/base.py", line 847, in _refresh_filters_cache
    self._filters = self.get_filters()
  File "/home/hughest/tmp/flask-admin/env/lib/python3.7/site-packages/flask_admin/model/base.py", line 1165, in get_filters
    raise Exception('Unsupported filter type %s' % n)
Exception: Unsupported filter type email

At the current master 2d43a263 I get the following error.
hughest@argon [0] $ python examples/sqla/run_server.py
Traceback (most recent call last):
  File "examples/sqla/run_server.py", line 1, in <module>
    from admin import app
  File "/home/hughest/tmp/flask-admin/examples/sqla/admin/__init__.py", line 24, in <module>
    import admin.main
  File "/home/hughest/tmp/flask-admin/examples/sqla/admin/main.py", line 248, in <module>
    admin.add_view(UserAdmin(User, db.session))
  File "/home/hughest/tmp/flask-admin/env/lib/python3.7/site-packages/flask_admin/contrib/sqla/view.py", line 329, in __init__
    menu_icon_value=menu_icon_value)
  File "/home/hughest/tmp/flask-admin/env/lib/python3.7/site-packages/flask_admin/model/base.py", line 818, in __init__
    self._refresh_cache()
  File "/home/hughest/tmp/flask-admin/env/lib/python3.7/site-packages/flask_admin/model/base.py", line 913, in _refresh_cache
    self._search_supported = self.init_search()
  File "/home/hughest/tmp/flask-admin/env/lib/python3.7/site-packages/flask_admin/contrib/sqla/view.py", line 578, in init_search
    for column in tools.get_columns_for_field(attr):
  File "/home/hughest/tmp/flask-admin/env/lib/python3.7/site-packages/flask_admin/contrib/sqla/tools.py", line 123, in get_columns_for_field
    raise Exception('Invalid field %s: does not contains any columns.' % field)
Exception: Invalid field User.phone_number: does not contains any columns.

The example has been updated to reflect the latest changes on 'master', so the version of Flask-Admin that you have installed can't handle those yet.
You can either try an older version of the example, or you can install the latest master and run it agains that. To install the package version that you've got checked-out locally, you can
pip uninstall flask-admin
pip install -e .

where . is the path to the root of the repo, where setup.py is located.

Even with the latest version of master, the example doesn't work, any more ideas please?

Looks like this example is now working. :(


💡[Feature]: Gold Price Prediction using ML

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
leveraging historical data and various influencing factors to forecast future prices. The process begins with collecting extensive datasets that include historical gold prices, economic indicators, geopolitical events, currency exchange rates, and other relevant variables. Data preprocessing steps, such as handling missing values, normalizing the data, and feature engineering, are crucial for improving model accuracy.
Use Case
These predictions can be used in various practical applications. For instance, financial analysts and investors use gold price predictions to make informed decisions about buying or selling gold, optimizing their investment strategies. Central banks and financial institutions also rely on these predictions to formulate policies and manage their gold reserves.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

pls @sanjay-kv update it as level2 bcoz i've to make UI for that

This repo have 150 point limitation.


Django v1.10 filer urls

I get NoReverseMatch
NoReverseMatch: Reverse for 'filer-directory_listing-last'
python 2.7/django 1.10

I get this when using a custom admin site.
Switching back to the default worked.  Don't have time to submit a PR for this right now, but just noting it here for others in the future.


[ACS][Chat] Updated Dependencies

Packages impacted by this PR
@azure/communication-chat
Issues associated with this PR
updated the following dependencies:

common sdk
signaling

to support new identity model
Describe the problem that is addressed by this PR
to add new identity model

What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[X] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[X] Added a changelog (if necessary)

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-communication-chat


logger: Add notifier to next_step?

Depending on the type of model to be trained, the time in between calls to next_step  may vary significantly. In common deep learning scenarios, i.e. the keras callback, next_step  is being called at the end of an epoch which could result in long times (maybe hours) in between calls.
It could be useful to have built-in support for optionally sending a notification each time next_step is being called.
Without changing dvclive, the user could just call a custom library (i.e. https://github.com/liiight/notifiers) after next_step:
class MetricsCallback(Callback):
    def on_epoch_end(self, epoch: int, logs: dict = None):
        logs = logs or {}
        for metric, value in logs.items():
            dvclive.log(metric, value)
        dvclive.next_step()
        notify('pushover', user='foo', token='bar', message=f'epoch: {epoch}')

But having the notification step built inside MetricLogger would have some benefits like access to internals (i.e. _metrics) and configuration options in addition to hiding complexity to the end user.
However, I'm not sure if it is worth to implement this feature inside dvclive or if it would be better to keep dvclive as lightweight as possible.

@daavoo I'm trying to understand the motivation behind this? 😄
could you please elaborate on this? o you want to update the files more often? What notify() means?

@daavoo I'm trying to understand the motivation behind this? smile
could you please elaborate on this? o you want to update the files more often? What notify() means?

Sorry about the lack of clarity.
The motivation comes from working with deep learning models that take a lot of time to train (i.e. hours or days). When working under that circumstances we always ended up writing some sort of "notification" code to complement or integrate into the ml logger.
This notification code takes care of sending a message to some platform (i.e. e-mail, slack / discord / telegram channel, etc) containing information like the number of finished epoch (a.k.a step in dvclive) and associated metrics. We also used it to inform when exceptions occurred during the training loop.
notify() is usually a function that sends useful information as a message to an app.

EDIT: do you have any references in other ml logger frameworks to this functionality?

I think that in other ml loggers we usually have an associated UI with a view that is automatically being updated as the plots/information are being logged (Related with this Studio issue: https://github.com/iterative/studio-support/issues/13)
In addition to that, some ml loggers also provide "notification" utilities:

https://docs.wandb.ai/guides/track/advanced/alert
https://www.comet.ml/docs/python-sdk/Experiment/#experimentsend_notification

Beyond existing functionality in other ml loggers, I have personally found different teams and open source communities solving this problem:

https://github.com/huggingface/knockknock
https://forums.fast.ai/t/training-metrics-as-notifications-on-mobile-using-callbacks/17330

I just discovered another open-source tool focused on this kind of functionality:
https://github.com/labmlai/labml

Related to #91

Another open source tool:
https://github.com/aporia-ai/mlnotify

Interesting integration between DagsHub and New Relic highlighting alerts as one of the main features:
https://dagshub.com/blog/real-time-machine-learning-monitroing-new-relic-dagshub/

Related to https://github.com/iterative/dvclive/issues/91#issuecomment-1035015144, I think the most useful integration here would be making it dead simple to send full reports (similar to the html today) through supported channels.
For example, the slack api could probably be used generate a message with the metrics and plot images, and similar for email (personally, I would prioritize slack because it's more collaborative and probably easier for users to set up).
The local html generated now could just be one report/alert format in that case (and the cml markdown report another).

I'm not sure text-based alerts add enough value (we could instead have a doc or blog post showing how to use dvclive + https://github.com/liiight/notifiers). Full reports with plots seem like a more unique feature, and they extend dvclive's initial value prop of lightweight live monitoring for model training, providing serverless alerting and reporting anywhere without needing to access the training machine. Since a lot of training happens in headless environments anyway, this seems pretty useful to me. What do you think?

I think it's useful and immediate added value for DVCLive.
I'm a little "worried" about how easy would be to maintain because Report Providers sounds like integrations potentially growing perpendicular to ML Frameworks.
So far, looking at slack and email APIs, it doesn't look that bad.

@shcheklein mentioned that it might be worthwhile to look into RSS feed aggregators. There are some parallels in how RSS expects a particular schema of elements (https://validator.w3.org/feed/docs/rss2.html) and can publish them in a consistent format, so maybe it can give some ideas for how to implement.

So it's about tidying up this sort of thing?
from tqdm.contrib.{slack,telegram,discord} import trange

with trange(live.get_step(), epochs, unit="epoch") as pbar:
    for epoch in pbar:
        ...
        live.log("loss", loss)
        pbar.set_postfix(loss=loss)
        live.next_step()

i.e. providing a callback interface?
live.set_callback(
    on_log=lambda name, metric: pbar.set_postfix({name: metric}),
    on_step=lambda new_step: print(f"starting epoch {new_step:>5d}"))

I don't think we are likely to do this now that we have live metrics in Studio and other solutions exist for alerting.


Auto path suggestion does not work after bibliography:  metadata field in YAML

I was wondering if auto path suggestion could be supported in bibliography:  metadata field in YAML. It does not seem to work at this point.

that's correct, none of our YAML completion currently supports paths (but it's definitely on our list to do this!)


Rarity Materials are Broken

epic and myhtic don't worok for both the reforging table and the gem cutting table
it won't even go into the slot

read the book of shadows (ingame guide book)

I just looked through jei, just a higher level table