trouble with saturated mutagenesis

Hi there,
Sorry, feels like I'm ploughing through the functions and hitting everything that can go wrong ;]
Trying to use the saturated mutatgenesis I faced problems and tried to fall back on the tutorial chr7 hox ctcf site example location:
~/toolsBasset/$ basset_sat.py -t 3 -n 200 -o satmut_hox cnn_full_with_prim_erythroid/dnase_with_prim_erythroid_cnn_best.decuda.th ./hox_test_seq.fa  basset_sat_predict.lua  -center_nt 200 cnn_full_with_prim_erythroid/dnase_with_prim_erythroid_cnn_best.decuda.th satmut_hox/model_in.h5 satmut_hox/model_out.h5
Predicting sequence 1 variants
Traceback (most recent call last):
File "/home/ron/tools/Basset/src/basset_sat.py", line 367, in
main()
File "/home/ron/tools/Basset/src/basset_sat.py", line 206, in main
g = sns.heatmap(preds_heat, vmin=0, vmax=1, linewidths=0, xticklabels=target_labels, yticklabels=False, cbar_kws={"orientation": "horizontal"})
File "/home/ron/anaconda2/lib/python2.7/site-packages/seaborn/matrix.py", line 452, in heatmap
mask)
File "/home/ron/anaconda2/lib/python2.7/site-packages/seaborn/matrix.py", line 140, in init
self.xticklabels = xticklabels[xstart:xend:xstep]
TypeError: 'NoneType' object has no attribute 'getitem'
~/toolsBasset/\$ ll satmut_hox/
total 1092
drwxrwxr-x  2 ron ron    4096 Jun 20 12:59 ./
drwxrwxr-x 11 ron ron   12288 Jun 20 13:04 ../
-rw-rw-r--  1 ron ron    6944 Jun 20 13:06 model_in.h5
-rw-rw-r--  1 ron ron 1058144 Jun 20 13:06 model_out.h5
-rw-rw-r--  1 ron ron       0 Jun 20 13:06 preds.txt
-rw-rw-r--  1 ron ron       0 Jun 20 13:06 table.txt
I was trying to run with my own trained model (your 164 selected tissues + on test set of our own). Trying the same with your provided pre-trained model results in exactly the same error output independent of the selected cell type to predict.
Also noteworthy: the randomly selecting and mutating sequences from the already extracted sequences (as described in the sat_mut tutorial) seems to work fine)
Cheers,
Ron

No, keep bringing the feedback! This one's caused by bad x-axis labels of the targets. If you want labels, pass them with "--targets_file". Otherwise, I think I fixed it to not crash when it doesn't have them.

I'm not quite sure what I'm doing wrong, but I'm running into a similar issue (my code is from Aug 23).
This is the command I run and the corresponding output:
./basset_sat.py --cuda -t 1 --targets_file testout -s 10 -n 100 mythfile.th myh5file.h5
basset_sat_predict.lua -cuda -center_nt 100 mythfile.th heat/model_in.h5 heat/model_out.h5
Predicting sequence 1 variants
Predicting sequence 2 variants
Predicting sequence 3 variants
Predicting sequence 4 variants
Predicting sequence 5 variants
Predicting sequence 6 variants
Predicting sequence 7 variants
Predicting sequence 8 variants
Predicting sequence 9 variants
Predicting sequence 10 variants
/home/ubuntu/anaconda2/lib/python2.7/site-packages/seaborn/matrix.py:134: FutureWarning: elementwise comparison failed; returning scalar instead, but in the future will perform elementwise comparison
if xticklabels == []:
Traceback (most recent call last):
File "./basset_sat.py", line 373, in
main()
File "./basset_sat.py", line 216, in main
seq_mod_preds_cell = seq_mod_preds[si,:,:,ci]
IndexError: index 1 is out of bounds for axis 3 with size 1
Any idea how to fix this error (maybe it's related to the other one mentioned)?

How many targets did you train on? It looks like there's just one target, but you're asking it to consider a second target (-t is 0-based indexing so -t 1 means consider the second target).

Oh I see -- correct me if I'm wrong, but it sounds like a "target" alludes to a column of the output matrix one is trying to predict, in which case yes, I had a single target, and the issue was fixed by the -t 0 option. Thanks for the recommendation, I think I was a bit confused by the terminology of what the "target index" precisely alluded to.

That's right, you've got it


fix: also retry with misnamed format "dag-cbor" as "cbor"

Prevents error unrecognized format: 'dag-cbor'

Merging as CI failures are unrelated - ping test failures


Tweaks buildkit roles and bindings

Issue #, if available:
Description of changes:
/hold
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/unhold
/approve

/lgtm


Unable to debug half of the time

I have the latest version of node JS tools for Visual Studio 2013 (RC 2.1) and it's still impossible to launch properly the debugging process. The console hangs on "Debugger listening on port xxx" and nothing happens.  I have to relaunch Visual Studio to get it working again.

When you say "I have to relaunch Visual Studio to get it working again", so it does work then at least once after you start VS? Can you detail for me when it does/doesn't work (i.e. always works first time after launch, 50% success after that, etc.).
Has this been a consistent or recent problem with the tools? Which version of Node are you using?

Also have you tried the latest dev build? We fixed an issue with the same symptoms some time after RC2.1 https://github.com/Microsoft/nodejstools/releases

Hi,
I'm having  the same issue. except that debugging doesn't work 100% of the time for me. I can launch the debugger and I see the console window, as aboe, but the debugger never stops on a break point.
I'm currently running:

VS2015 Community
NTVS 1.1
Node 4.2.2 (x64)
Windows 10 (just upgraded from Win 7)

I was using Resharper 10, but un-installing that made no difference.  I also tried debugging using the 32bit version of node 4.2.2, but that didn't make any difference, either. I have no problems debugging with Chrome via the node-inspector package, btw.
If there's anything else you need to know, please let me know. Thanks

Same problems here, didn't stop on breakpoint before a function call, but if I placed the breakpoint inside the function to be called it stopped correctly. I remember I had  problems in Visual Code when I tried to debug a node app there a few months ago, breakpoints didn't work well with async code.

Sorry you're hitting issues here.
@barrycarr Your setup looks similar to mine in terms of software and versions, and I'm not seeing this behavior. Is it in a specific project or code pattern? Or even if you just create a new simple project with a few "console.log()" statements and try to step through it do you see the issue?
If it's with a specific project or code pattern, would you be able to provide a repro we can use?
Note there is also the debug diagnostics you can enable. The output from this would be helpful also. See https://github.com/Microsoft/nodejstools/issues/246#issuecomment-127715686 for details.

I face the same issue when i create a new "empty console nodejs" using vs2015 professional with update 1.  I was able to resolve the issue by running the visual studio in admin mode.
I am able to debug or see ouput for my code only if i run as admin.

A related issue in StackOverflow here: http://stackoverflow.com/questions/31321808/ntvs-projects-do-not-run-at-all-when-i-click-start-or-press-f5

Experiencing the same issue here w/ VS 2013 and version 1.1.30910.02 of the tools. My projects launch into debug about 40% of the time. I usually get around this by starting w/ out debug. If I have to debug in node, a relaunch usually works the 1st time.
I just created a new node express 4 project and it failed to launch debug 2x in a row. I relaunched and it worked.

Also getting a hang just stepping through debugger - I have a hang dump if interested.

Also getting a hang with VS2015/ndoe.js tools 1.1.31117.00 and node 5.2.0
Anecdotally happens after computer goes to sleep, 100% after a hibernate
Recovery is to end task VS2015 and start again
Info report as follows:
Microsoft Visual Studio Community 2015
Version 14.0.24720.00 Update 1
Microsoft .NET Framework
Version 4.6.01038
Installed Version: Community
Visual Basic 2015   00322-20000-00000-AA758
Microsoft Visual Basic 2015
Visual C# 2015   00322-20000-00000-AA758
Microsoft Visual C# 2015
Visual C++ 2015   00322-20000-00000-AA758
Microsoft Visual C++ 2015
Visual F# 2015   00322-20000-00000-AA758
Microsoft Visual F# 2015
Windows Phone SDK 8.0 - ENU   00322-20000-00000-AA758
Windows Phone SDK 8.0 - ENU
Application Insights Tools for Visual Studio Package   1.0
Application Insights Tools for Visual Studio
ASP.NET and Web Tools 2015.1 (Beta8)   14.1.11106.0
ASP.NET and Web Tools 2015.1 (Beta8)
ASP.NET Web Frameworks and Tools 2012.2   4.1.41102.0
For additional information, visit http://go.microsoft.com/fwlink/?LinkID=309563
ASP.NET Web Frameworks and Tools 2013   5.2.30624.0
For additional information, visit http://www.asp.net/
Common Azure Tools   1.5
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.
GitHub.VisualStudio   1.0
A Visual Studio Extension that brings the GitHub Flow into Visual Studio.
KofePackagePackage Extension   1.0
KofePackagePackage Visual Studio Extension Detailed Info
Microsoft Azure Mobile Services Tools   1.4
Microsoft Azure Mobile Services Tools
Microsoft MI-Based Debugger   1.0
Provides support for connecting Visual Studio to MI compatible debuggers
Node.js Tools   1.1.31117.00
Adds support for developing and debugging Node.js apps in Visual Studio
Node.js Tools - Profiling   1.1.31117.00
Profiling support for Node.js projects.
NuGet Package Manager   3.3.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit http://docs.nuget.org/.
PreEmptive Analytics Visualizer   1.2
Microsoft Visual Studio extension to visualize aggregated summaries from the PreEmptive Analytics product.
Python Tools for Visual Studio   2.2.31105.00
Python Tools for Visual Studio provides IntelliSense, projects, templates, Interactive windows, and other support for Python developers.
Python Tools for Visual Studio - Django Integration   2.2.31105.00
Provides templates and integration for the Django web framework.
Python Tools for Visual Studio - Profiling Support   2.2.31105.00
Profiling support for Python projects.
SQL Server Data Tools   14.0.50616.0
Microsoft SQL Server Data Tools
TypeScript   <IP_ADDRESS>
TypeScript for Microsoft Visual Studio
Visual C++ for Cross Platform Mobile Development   1.0
Visual C++ for Cross Platform Mobile Development
Visual C++ for Cross Platform Mobile Development   1.0
Visual C++ for Cross Platform Mobile Development
Visual Studio Tools for Universal Windows Apps   14.0.24720.00
The Visual Studio Tools for Universal Windows apps allow you to build a single universal app experience that can reach every device running Windows 10: phone, tablet, PC, and more. It includes the Microsoft Windows 10 Software Development Kit.
Xamarin   3.11.837.0 (f10676f)
Visual Studio extension to enable development for Xamarin.iOS and Xamarin.Android.
Xamarin.Android   <IP_ADDRESS> (f98871a95a479f6d71b3067b7e5834d41fcb2118)
Visual Studio plugin to enable development for Xamarin.Android.
Xamarin.iOS   <IP_ADDRESS> (6db87c53c073f4af2f5247fb738a27ea08c094fd)
Visual Studio extension to enable development for Xamarin.iOS.
Xamarin.iOS Unified Migration   1.0
Automated migration for Xamarin iOS Classic projects to Unified
Xamarin.TestCloud.Integration   1.0
Early preview of Xamarin Test Cloud integration

p{ease try downloading and using the NTVS 1.2 Beta. We've made a number of improvements to debugger reliability and I have not seen this problem recently.
Please try out 1.2 and let me know if you are still running into this issue. and we can continue the investigation. Thanks.


Embed external media players into IPTVnator frame

Can external media players VLC and MPV become embedded in the same frame as IPTVnator?
Maybe this is an old topic or suggestion but I wanted to bring it up anyways.
Is it maybe possible to embed external media players like VLC and MPV into the IPTVnator frame?
I meant in the same way as the native HTML5 and Java players are. Having such a solution might even attract more potential interested users out there.

This request is kindof a dupe of #443

yep, see #433


Add ISSUE_TEMPLATE

See details in the parent issue napalm-automation/napalm-base#71

Closed via https://github.com/napalm-automation/napalm-fortios/pull/17


How to get Sortable of 'event.to'

Hi I would like to access 'Sortable data' of 'event.to' after the item is dragged to other sortable list.
i'm using the callbacks, but the event.to only returns the HtmlCollection. however I would like to access the Sortable Object.

hi @a5hik
could you, please, create a plunkr reproducing a problem? It's not really clear what you are asking for

closing due to inactivity


Generate only code for opt-in entities

This pull requests takes care of the following tasks:
New features

Implement opt-in of types and enums

Structs not opted-in that are required by other structs are still generated.
Enums not opted-in are treated as aliases.

Refactor

Removed the Packages type, which only added noise to the public facing API.
Removed all context that is not related to the scanner.Package and create a new scanning context.
PackagesInfo is now packagesInfo and it's all private because its usage is only meant to be internal.
Moved scanned entities to their own file for clarity.

Breaking changes

proto struct tag is now proteus

Current coverage is 93.91% (diff: 90.90%)

Merging #13 into master will decrease coverage by 1.20%

@@             master        #13   diff @@
==========================================
  Files             8         10     +2
  Lines           512        608    +96
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            487        571    +84
- Misses           18         25     +7
- Partials          7         12     +5

Powered by Codecov. Last update cbb159e...1ca4525

Can we talk about this first thing in the morning?
Thank you!


feature: Course Time Blocks and Schedule Sync Enhancement

Is your feature request related to a problem? Please describe.
This feature will help us log our time, similar to how we do in Google Calendar.
Describe the solution you'd like :

Each user should be able to edit the time entries the way they prefer (similar to Google Calendar).

Automatic time-blocking for the times of classes from the courses we are enrolled in. (For example, as I am enrolled in cohort 3.0 Web Dev, I would like it to log my time from 8-10 every Saturday and Sunday, including the class topic and a direct access link.)

As I subscribed to it on Gmail, the time log automatically appears in my Google Calendar with details about the classes, so why not have the same functionality on the 100xDevs website?

Not limited to only 100xdevs Classes.

Current look :

Desired look :

@furyfist , the issue seems intresting! can you describe more about the google calender thingy. currently , user can able to write their work for the particular time slot  in the particular days as they prefer.

Interesting enhancement! I would love to work on this issue, I think the time blocking issue could be easily resolved with adding bars with greater z-index in calendar and assigning colors to time slots which. This seems like shadcn calendar so probably just changing the color would work

@furyfist Please elaborate in more detail about your suggestion for user time logs, what time logs can the user add except for live classes? more explaination can help in creating tags if that time log functionality were to be implemented

It's pretty hard to improve on this design. Anyone who is willing to solve it feel free to pr it. Closing for now but will happily look into a pr


Link all pages

DATABASE

[ ] Auth
[ ] User
[ ] Forum
[ ] Jobs
[ ] Metrics
[ ] Payment

TEST RUN
To /home

[ ] Forum
[x] Jobs
[ ] Payment
[ ] Profile
[ ] Admin dashboard
[ ] Auth

To pages in the same feature

[ ] Home
[x] Forum
[x] Jobs
[ ] Payment
[ ] Profile
[ ] Admin dashboard
[ ] Auth

Test on one PC:

[ ] Home
[x] Forum
[ ] Jobs
[ ] Payment
[ ] Profile
[ ] Admin dashboard
[ ] Auth

has seen and will update daily


Some good things extracted from the X11rdp MacOS port

The most important fix is restoring the download cache functionality that was broken by 6b589569ee2aeb20f157ef73b2ca240166fef6a4

I've added a patch that is even more important. X11rdp was using standard path to fonts. It would not find fonts on a system without existing fonts. With the fix, X11rdp uses the fonts that are built together with X11rdp.

Looks good.


Add missing package description

Description
Add the project description to package.json
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Chore, documentation, cleanup

Related Issue
n/a
Motivation and Context
n/a
How Has This Been Tested?
n/a
Screenshots (if appropriate):
n/a
Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation (if required).
[x] I have read the CONTRIBUTING document.
[n/a] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Pull Request Test Coverage Report for Build 104

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 99:
0.0%

Covered Lines:
60

Relevant Lines:
60

💛  - Coveralls


Use trace provider from pipeline instead of getting it from global

Updated trace tests, README, and example to use the trace provider returned from InstallNewPipeline() instead of always calling global.TraceProvider().
IMO it looks cleaner this way, but if this is unnecessary or if there's some reason to use global.TraceProvider(), feel free to close this PR.

On second thoughts, if we are updating the README and examples, we need to think about what we would prefer to demonstrate to users. Using the global (singleton) trace provider is expected to be the most common case, and it's only rare / advanced users that would need to create multiple trace providers. I think we should at least mention both options in the README.
WDYT @nilebox?

I think we should at least mention both options in the README.

Agreed. We should initially provide an example with a global trace provider since it's the most common case.
Then we can mention that if a user has more than one trace providers configured, they need to explicitly fetch and reference them. And follow with a short code snippet showing the difference.

Closing for now, improving docs tracked in https://github.com/GoogleCloudPlatform/opentelemetry-operations-go/issues/23


[FormRecognizer] Parse operation IDs with regex

Summary
The implementation of the Operation constructors uses some magic numbers when parsing the operation ID.
Operations to address

[ ] CopyModelOperation
[ ] RecognizeContentOperation
[ ] RecognizeCustomFormsOperation
[ ] RecognizeReceiptsOperation
[ ] RecognizeIdDocumentsOperation
[ ] TrainingOperation
[ ] AnalyzeDocumentOperation
[ ] ClassifyDocumentOperation

Internal constructor
https://github.com/Azure/azure-sdk-for-net/blob/e32b2e7f7fab4d7209da788250caa2c145b23c66/sdk/formrecognizer/Azure.AI.FormRecognizer/src/RecognizeCustomFormsOperation.cs#L78-L83
The internal constructor takes a parameter called operationLocation, which is returned from the service and is expected to look like:
{endpoint}/formrecognizer/{version}/custom/models/{modelId}/analyzeresults/{resultId}

Public constructor
https://github.com/Azure/azure-sdk-for-net/blob/e32b2e7f7fab4d7209da788250caa2c145b23c66/sdk/formrecognizer/Azure.AI.FormRecognizer/src/RecognizeCustomFormsOperation.cs#L102-L112
The public constructor, on the other hand, takes a parameter called operationId, which is given by the developer and is expected to be a substring of the aforementioned operationLocation:
{modelId}/analyzeresults/{resultId}

Goals

Stop using magic numbers and correctly parse the arguments using regex, handling format errors.
Make sure we support parsing model IDs with characters _, ~, and .. These characters are allowed when naming a custom model.
Include new tests to verify proper behavior.
Make sure existing tests still pass reliably.

The following file can be used as a reference:
https://github.com/Azure/azure-sdk-for-net/blob/80a63242c93cbbff1ee525248cbd94d89ed72345/sdk/formrecognizer/Azure.AI.FormRecognizer/src/FormField.cs#L90-L91
Related: #10385

@kinelski  is this for P3? or should we put it in the backlog epic

@maririos I wouldn't say it's a high-priority issue, since it just changes the behavior internally. @annelo-msft is keeping track of what issues we want closed for P3.

Thanks Caio. I put the issue in the backlog epic. In general, it is better if we don't have issues floating around without an epic.

Closed https://github.com/Azure/azure-sdk-for-net/issues/10385 and added full list of operations to this issue.

@annelo-msft As TA is also using the LRO pattern and will require to parse out the operation location id. Referring to this Issue in the SDK code.

This parsing is specific to FR. If you need it for TA, then you can do Regex in TA

By changing the parsing, the errors might change too, so we need to be careful here to not introduce breaking changes.
Will wait first on the outcome of https://github.com/Azure/azure-sdk-for-net/issues/17383 before working on this to make sure we have the coverage

@kinelski is this still valid, can it be closed?

Is this a change to the implementation or a feature request?

Is this a change to the implementation or a feature request?

It's a change to the implementation to improve code readability.

Given the change to document intelligence @kinelski do you still think this is something we should continue to track? Do you see us prioritizing this work any time soon?


status=409 error (container is being deleted) is not retired by built-in retry classes

Which service(blob, file, queue) does this issue concern?
blob
Which version of the SDK was used? Please provide the output of pip freeze.
azure-batch==6.0.0
azure-common==1.1.18
azure-storage-blob==1.4.0
azure-storage-common==1.4.0
azure-storage-file==1.4.0
What problem was encountered?
After deleting a container, I tried to create another container with the same name.  If the time interval is between these actions is short (under 30 secs or so), the error "container is being deleted" is issued.  I tried setting client.retry to LinearRetry().retry, but the error is still not retried.
Analysis
lines 72-81 in azure-storage-common/azure/storage/common/retry.py show that status=409 is handled by returning False (never retry).   This is a common problem (for my code at least) - I recommend that it be enabled for retries.
Stack Trace
tClient-Request-ID=08146a00-53be-11e9-8644-309c23b63418 Retry policy did not allow for a retry: Server-Timestamp=Sun, 31 Mar 2019 14:05:23 GMT, Server-Request-ID=5553e8f2-e01e-00de-60ca-e798e6000000, HTTP status code=409, Exception=The specified container is being deleted. Try operation later. ErrorCode: ContainerBeingDeletedContainerBeingDeletedThe specified container is being deleted. Try operation later.RequestId:5553e8f2-e01e-00de-60ca-e798e6000000Time:2019-03-31T14:05:23.4952262Z.
Traceback (most recent call last):
File "xt_test.py", line 114, in
store_test(xt)
File "xt_test.py", line 21, in store_test
xt.create_workspace(ws_name)
File "c:\github\ExperimentTools\xt\xt_store.py", line 65, in create_workspace
self.helper.create_workspace(ws_name, description)
File "c:\github\ExperimentTools\xt\xt_azure_store_blob.py", line 88, in create_workspace
self.error("could not create workspace: " + ws_name)
File "c:\github\ExperimentTools\xt\xt_azure_store_blob.py", line 52, in error
raise Exception(f"Error - {msg}")
Exception: Error - could not create workspace: ws1
Have you found a mitigation/solution?
Use a custom retry callback to retry this error.
Note: for table service, please post the issue here instead: https://github.com/Azure/azure-cosmosdb-python.

Hi @rfernand2, thanks for the feedback!
Error 409 might be returned in other scenarios which might persist longer, so I don't think that this change should be made to the general retry policy. You are welcome to create your own retry policy, and set it on the client object, like so.


Migrate over to glide for dependency management

vndr does not import transitive dependencies, which makes importing large projects a nightmare. Move over to glide, which is much more widely used.

Closing this. I think actually that vndr not supporting transitive dependencies isn't a bad thing; we have to add dependencies one at a time.
To add dependencies effectively in a manner that works, I have done the following:

Add whatever new dependency I need with vndr <repo> <tag / commit ID>.
Compile Cilium in the dev VM (make)
If I hit a compilation failure due to a dependency of the new dependency (transitive dependency) not being found, manually go to the dependency's list of dependencies, and figure out what version of that dependency they are using.
vndr the new dependency and try compiling again.

While tedious, this ensures that we do not have a tool like glide making decisions when two dependencies that we use need a different version of the same dependency, which is the case.

@ianvernon vndr also warns you if a dependency is not in vendor.conf.


Security and multipart file upload with bodyParser()

It recently occurred to me that when using the bodyParser it's not entirely clear that there's a potential security issue with the way the typical examples are structured.
Let's say you normally have a file upload like:

This arrives in bodyParser as req.body with contents like:
{file:{type:"image/jpeg", path:"/tmp/whatever.jpg"}}
Note, however, that bodyParser will ALSO parse a json payload, for example the client could send a json object like:
{file:{type:"image/jpeg", path:"/path/to/sensitive/file"}}
And bodyParser will also parse a url-encoded payload into a complex object, e.g.:
file[type]=image/jpeg&file[path]=/path/to/sensitive/file
Unwary programmers may not realize this is possible and won't put in the necessary checks to protect themselves from an attack of this kind.
The simplest solution would be to change it so the "regular" fields supplied by the client would go in req.body and the uploaded files would go in req.files.  This would ensure that the client could never "spoof" a file in the body as they have no control over the files object within their data.
Under the current system users should be able to verify that the file came from formidable by checking if (req.body.file instanceof (require('formidable/file.js).File) or something along those lines, so even if the system is changed the examples could be updated to perform this check.
Note that this is not a problem within formidable itself as the creators of formidable are kind enough to provide the fields and files in separate lists, the problem was introduced within bodyParser.

that's a great point, if you're not checking the content-type this could definitely be an issue. Let's do req.files and req.body

thanks for the report. funny how obvious things slip by sometimes. 1.8.2 has the fix

Good thinking @dobesv

Glad to help.  Nice fast fix, thanks!


[AERIE-1796] Carve out sim. results: resource samples

Tickets addressed: AERIE-1796
Review: By commit
Merge strategy: Merge (no squash)

Description
AERIE-1796 will be accomplished in a piecemeal fashion (after each component of the result can be queried for independently). This PR removes resources from the simulate response. Users should now use the resourceSamples endpoint.
Verification
TODO
Documentation
TODO
Future work
Carve out the rest of simulation results (for example, constraint violations).

Nice. This will also require a quick line removal in the UI: https://github.com/NASA-AMMOS/aerie-ui/blob/develop/src/types/simulation.d.ts#L54


Request type is used instead of file type on file.type

Heya!
I've noticed that file.type over here (see files) is not a mimetype for the file but for the request, grabbed from cgi.parse_header.
This feels like an incorrect behavior as we (@elixire) previously assumed that this was for file mimetype, and it caused a bug that affecting many users.
I propose doing one (or more) of these:

Updating documentation to state that it's not file type but request type
Swapping it out for file's mimetype
Removing .type
Adding a .mime field to file with mimetype

We can get the proper file mimetype from either the file's magic header with python-magic or with the name through mimetypes library. While python-magic would give more accurate results, it requires installing libmagic through distro's package manager which might not be favorable for some.
I'd just send a PR but wasn't sure which approach was the most favorable, so I'll send one once a decision is made regarding which approach will be taken.

I disagree with this; the mimetype isn't necessarily passed. We're getting the appropriate information from the headers, as per https://tools.ietf.org/html/rfc6266#section-4. If you want to do magic interpolation around the file itself, I don't think that should be included in sanic core.

@sjsadowski while I agree that magic interpolation is overkill, passing request headers as file type is simply unreasonable.
Passing mimetype would make much more sense in this case.

I think expanding the dependency tree by adding mimetypes seems like overkill. I'd say we'd be better off to remove .type to prevent confusion

Mimetypes is a standard lib and is used elsewhere in code (response.py iirc), so I don't think that it'd expand the dep tree.
On September 13, 2018 11:58:09 PM GMT+03:00, Stephen Sadowski<EMAIL_ADDRESS>wrote:

I think expanding the dependency tree by adding mimetypes seems like
overkill. I'd say we'd be better off to remove .type to prevent
confusion
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/huge-success/sanic/issues/1311#issuecomment-421150669

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

I'd actually forgotten about that. If you push it through mimetypes.guess_type() the filetype would be the mime encoding or None if it couldn't guess. That's not horrible I suppose.

@aveao there's a single outstanding request for your PR. Could you take a look at that?

I'll take a look tomorrow.

Looks like interest in this was dropped. We can open a new issue to address going forward if necessary.


Make docs version show up in search results

The intention of this PR is to make the docs version show up in search results when you click the magnifying glass in the top nav of the docs.
This is a stopgap measure that is meant to improve the user experience in the short term. The long term solution is to migrate to docusaurus in which algolia search will be limited to only one docs version at a time.
Currently it shows identical entries in the list when they are really entries for two different docs versions.
I have been able to get something to show up below each list item, as shown in this screen shot:

This PR is in progress because I haven't figured out how to get just the docs version in there. Currently I'm adding the URL that the search result list item is pointing to, which contains the version in it. I can use Javascript to extract the version from the URL itself, but I can't use javascript in this template which is used to render the list item: https://github.com/rancher/docs/blob/master/assets/js/app.js#L42 It seems those variables (such as _highlightresult, _snippetresultandpermalink`) have to be passed in to algolia in a special way. I find this Hugo layout file in the rancherlabs/website repo to be fascinating because it looks like it is used to put variables in the list results. https://github.com/rancherlabs/website-theme/blob/master/layouts/_default/list.algolia.json I found docs about how those variables work but I have yet to make sense of it. https://www.algolia.com/doc/api-reference/widgets/infinite-hits/js/

@divya-mohan0209 I believe that to move forward on this, we will have to make sense of this doc. Maybe you can review it before we meet? https://github.com/rancher/docs/pull/3899


PreAdd middleware not running on EntityStore.set()

In the 3.0 blog post and Akita documentation it says:

The preEntityAdd() middleware is called when we invoke the EntityStore.set() or EntityStore.add() methods with the newly added entity

From what I can tell the middleware is only being called when I use the EntityStore.add()
Here is a stackblitz that console logs on add but not on set:
https://stackblitz.com/edit/akita-todos-app-yclvep

Fixed in 3.1.3. Thanks for the report.


Remove 'unless/end' pair when parameter list is empty

/azp run

/azp run

The CI seems stuck, I have no idea how to fix this. Any ideas from top contributors? @olydis @sarangan12 thanks!

As of February 2021, Azure Resource Management SDK for Ruby and Autorest.Ruby have entered a retirement phase and is no longer officially supported by Microsoft. Here is the complete list of packages that are affected by this. We are committed to making critical security and bug fixes for libraries in this repo until December 31, 2021. After that date, this repo will no longer be maintained.


it is not possible to travel around the world

If you reach the South, North, West or East, the tiles stop...
We need a way to smoothly change the coordinates without blink the existing tiles.

Hmm I'm not sure if I get that.
"If you reach the South, North, West or East, the tiles stop..."
you mean border edges of one tile or the whole world (mercator) ?
"We need a way to smoothly change the coordinates without blink the existing tiles."
Blink? You mean the spikes while loading new tiles?

Sorry for my bad English.
Yes, I mean the whole word (mercator), first I walked to the South, in the Antartica, and when I reach the limit the tiles just stop load. Then I walked to the West, and right after Hawaii, the tiles stoped too.
About the blink, I tried a fast solution moving the ThirdPerson to the other side of the world, but it was ugly, with a blink in the surrounding tiles.

well that's how mercator works :) we can add wrapping though, shouldn't be hard at all.
I haven't tried moving too fast and admittedly it's not build to perform at that level either. If you're planning to move fast, you can use a hight range but that'll be quite costly as well. So yeah, we have some limits. It's get so much better if I can find some time to work on threading and mesh generation refactoring.

I'm working on a solution now, if I can get some good results I advise you!
(but my programing skill are very limited, I'm just a curious designer, dot expect some beautiful optimized code)

no worries man, have fun and if you want, I'll check&fix your code when you're done ;)


wrong "every" reminder syntax fails silently

command !every 1h; fail; again fails silently. nothing in the logs.

Does the bot really support !every? I was unable to find it in the source code or using !help reminders.
!remindme every 1h; fail; again fails with an error message, as intended.

I think I might've opened this issue on the basis of watching someone try !every which as you say isn't a correct command, yet they believed it was.
I still might've opened it in error, given the bot isn't really supposed to react to commands that don't belong to it. So perhaps let's close this?


Import following list from account archive

After twitter's recent change in ownership, I, like many people, downloaded my account archive and deactivated the account.
Many more people have joined or switched since, but it has now been more than 30 days, so my twitter account has been purged.
Would it be possible to allow for import of follower and following lists from the follower.js and following.js json blobs provided in the account archive?

Uploading follower.js and following.js is already supported, but it will not work without a Twitter login because Twitter has per-user rate limiting for API requests. Without requiring Debirdify users to log into Twitter we could only have about 1 user every 15 minutes or so, which is obviously impractical.
This is a limitation of the approach Debirdify takes of using the Twitter API. If you want something that works without a Twitter account, you'd have to make some kind of offline database of Twitter–Mastodon connections, which is not something that I personally want to do because then I would have to look into the data protection and privacy implications of storing all that data of random Twitter/Mastodon users (most of whom never agreed to use Debirdify) on my server.

That is unfortunate. Thank you for your explanation!


add description about  checking ubuntu version in docs/core/install/linux-ubuntu.md

Issue description
if  bash command about checking ubuntu  version added to docs/core/install/linux-ubuntu.md,will this doc become wordy or novice friendly ?
just a little advice

Target framework

[x] .NET Core
[ ] .NET Framework
[ ] .NET Standard

dotnet --info output or About VS info
5.0.102

Thank you for opening this suggestion. At this point I'm not going to add it as it's generally easy to just search on the internet "how to find ubuntu version." If you feel like this would be a good addition, please open a PR to make this change.
Cheers!


The inventory does not change!

This is my config
groups:
Group1:
worlds:
- world
- world_nether
- world_the_end
- Spawn
- Lobby
- Lugares
default-gamemode: SURVIVAL
Creative:
worlds2:
- Libre
default-gamemode: CREATIVE
When i change of world the inventory is the same

Can you post the contents of both the config.yml and worlds.yml file to Hastebin.com, save them, and drop the links here? That way it keeps the formatting, which is important.
Additionally, can you enable debugging in the main config, run /pwi reload or restart the server, switch worlds, and paste the server log?
That should give us a clue as to what's going on.

https://pastebin.com/LHUP3S0h
Is all that, libre have to be a creative world

I use perworldinvetories KT for minecraft 1.12.2, i tryed at 1.12

Sorry, looks like I forgot to tell you again to:
Additionally, can you enable debugging in the main config, run /pwi reload or restart the server, switch worlds, and paste the server log?


Improve performance of cluster cluster/project logging

What kind of request is this (question/bug/enhancement/feature request):
Enhancement
Fluent Bit provides a lightweight log collection and forwarding solution and uses approx. 1/10th of the resources (memory and  cpu) compared to Fluentd. This makes it much better suited as a per node log collection solution in general and especially for edge environments where resources are limited.
Expected behaviour
Cluster/project logging uses fluent bit as log collector.

Closing, as this will be completed via logging v2 in https://github.com/rancher/rancher/issues/27935 and its other issues.


un-Representable DateTime.

I Got this error: " The added or subtracted value results in an un-Representable DateTime. Parameter name: value"
Is there a Way to fix it ?

I got this error after trying to use Apollo to modify my trophy set for a game, and then attempting to switch to this after it didn't work right on the percentage tracking.  Did you use Apollo as well?

No, didn't used Apollo before this.

I found out what happened:
In my case, trophies had already been acquired and registered/sync. on another PS3 other than the one I was on now.
Solution: Delete the trophy list for the game mentioned on the Current PS3. Starting the game before synchronizing trophies will now result in an empty trophy list. Update the trophies from this list in the application (trophyisgood)  and synchronize again on the PSN, from this point on the list will work normally.


Inventory should store exact time when every metadata was collected

Due to large number of projects and resources our Forseti run takes more than 2 hours and we don't know when particular snapshot (e.g. project IAMs) was taken.
We would like to have exact information when particular metadata were retrieved for auditing reasons. This could be a separate column in every Inventory table.

Added in pr #1029.


Try horizontal collapse, but not working right now

Trying to implement https://www.codeply.com/go/UiteY92ke8/bootstrap-4-horizontal-collapse, but hitting my head against a wall for too long. Hoping someone on @twbs/css-review can point out the error of my ways.
Preview: https://deploy-preview-31681--twbs-bootstrap.netlify.app/docs/5.0/components/collapse/#horizontal

Isn't some JavaScript involved here? I see you only changed the Sass part.

Isn't some JavaScript involved here? I see you only changed the Sass part.

Most of the guides I've seen online don't change any JS, just CSS. Not quite sure if I'm missing some changed JS in that linked example, but you could very well be right.

The demo don't seem to use any JS apart from the core classes. Not sure if we have changed those in v5, since the demo is using v4.1.3. I also tried the demo with alpha1 and seems to work fine, so I'd say something is missing in the CSS or markup?

BTW this whole thing is transitioning properties that are not performant, like width and height.
Here is a Codepen with v5.0.0-alpha2: https://codepen.io/XhmikosR/pen/wvGLgwv

At the moment, the collapsible in the docs is no longer animated with the latest 4.1.x.
https://getbootstrap.com/docs/4.1/components/collapse/
This was working in 4.0.

I give up lol

I give up lol

Nooo.. Don't!😯Let's collaborate then.
I think you don't need that visibility stuff. There are two things that are involved.
First the collapse module checks for a width class on the collapse element. 🤯
Second magic thing is there is some hardcoded width in that demo. Not sure if it's possible to make this responsive without JS.
Also not sure why that demo also got that visibility stuff. My guess is that you can't transition display but visibility. For what reason? I don't know. If I remove that visibility stuff the demo works the same as for me.
This leaves us with something like that: https://codepen.io/-alpa-/pen/yLgObPM


Fixes #174 - Query did not return a unique result for "existsPage(String title)" after introduction of HSQL-DB tests

Addresses the broken MySQL collation for two cases: Wikipedia#existsPage(name) and Category.createCategory(title). The latter case was also affected.

Please review the PR and approve it as it fixes a regression.

I verified the fix works under MySQL environments as expected. One of you can now proceed with review/merge of the correcting changes.

Some Linux distros use MariaDB as a drop-in replacement for MySQL. I haven't used it much, but my understanding is that it should be compatible with MySQL... I guess it's somewhat similar to the choice between OpenJDK and Oracle JDK? Anyway, again just a 10ct comment ;)


Worker Timeouts

@lhorne-gavant @adistasio
I've been seeing some worker timeouts in the production log.  It doesn't really provide a lot of info, but I was doing some searching, and there was some suggestion it might be from gunicorn/uvicorn which is what fastapi runs on.
Looking further, I found this article:  https://github.com/tiangolo/uvicorn-gunicorn-fastapi-docker
which provides some images some of which say it installs install fastapi in an automatically tuned instance based on the number of CPUs etc.  Something to think about.  This may not be it at all, of course, and we may just need Uvicorn, which is how I think we're currently installed (though one of the blurbs mentioned that  Uvicorn runs on Gunicorn anyway, so maybe auto-tuning would help?)
Just documenting it for now...if you want to take a peek and see if it makes sense to you, we can talk about it when I get back.
Also, if you have any ideas about the worker timeouts let me know.
2021-07-21T01:07:36.518-04:00Copy[2021-07-21 05:07:30 +0000] [1] [CRITICAL] WORKER TIMEOUT (pid:32) | [2021-07-21 05:07:30 +0000] [1] [CRITICAL] WORKER TIMEOUT (pid:32)
Here's a bunch in a row:
[2021-07-21 14:16:37 +0000] [1] [CRITICAL] WORKER TIMEOUT (pid:9)
[2021-07-21 14:17:23 +0000] [1] [CRITICAL] WORKER TIMEOUT (pid:42)
[2021-07-21 14:18:41 +0000] [1] [CRITICAL] WORKER TIMEOUT (pid:54)
[2021-07-21 14:19:26 +0000] [1] [CRITICAL] WORKER TIMEOUT (pid:66)
[2021-07-21 14:21:44 +0000] [1] [CRITICAL] WORKER TIMEOUT (pid:78)
[2021-07-21 14:23:30 +0000] [1] [CRITICAL] WORKER TIMEOUT (pid:90)
[2021-07-21 14:26:47 +0000] [1] [CRITICAL] WORKER TIMEOUT (pid:102)
Not sure what the user sees when this happens.  I assume they are searching or viewing something and it just gives them an error.

Seems to not be a problem, and I haven't seen many of these lately.  Closing for now.


[ENH] Add JinaEmbedding TypeScript client

Description of changes
Add TypeScript client for Jina Embeddings as suggested in #1324
Docs PR: https://github.com/chroma-core/docs/pull/173

@jeffchuber I think this is your wheelhouse

@JoanFM did you try this and it works? I think you also need to expose the package via https://github.com/chroma-core/chroma/blob/main/clients/js/src/index.ts -- and then let's get some docs for it as well :)
FYI - I temporarily hid the JinaAI python docs until it goes out in monday's release.

@JoanFM did you try this and it works? I think you also need to expose the package via https://github.com/chroma-core/chroma/blob/main/clients/js/src/index.ts -- and then let's get some docs for it as well :)
FYI - I temporarily hid the JinaAI python docs until it goes out in monday's release.

We tried and it works, and I just added it to the index

Hey @jeffchuber , any news about this?


Open Settings not working

Every time I click Open Settings to setup the tool, no window opens up. Clicking connect gives me Connect ECONNREFUSED <IP_ADDRESS>:3003.

In theory, your default editor for .json files should open, you might be able to fix this by setting one manually.
Otherwise, you can directly navigate to the file located in AppData at C:\Users\[username]\AppData\Roaming\league-observer-tool and edit config.json there as described in the readme.
The ECONNREFUSED indicates the toolkit is not running (or blocked by a firewall if running remotely).


Edting PowerApp

Hi there
Firstly, many thanks for creating this app!
I'm really new to PowerApps so excuse my lack of knowledge.
When I try to edit the App layout to bespoke it to our needs I see the first screen, options to Create from Template or Blank but subsequent screens are blank.

But when I play the app I see text like, Privacy, Team name Description etc....  Is this pulled from somewhere else?

Many thanks
Alan

Hi @AlanPutt, I understand the Powerapps can be very overwhelming for the start. Let me try to explain what you are facing in simple terms. the subsequent screen is blank because the visibility parameter of the items is bound to a variable which gets populated on click of the buttons or some other controls such as "Build New" button.
What you can do to see the content on the next screen in edit mode: -
Press ALT and click on the button like "Build new" present on the first screen. This will take you to the next screen with items visible to you.
This PowerApp structure can be complex for you if you haven't worked on PowerApp at all and will recommend you to refer https://docs.microsoft.com/en-us/learn/modules/get-started-with-powerapps/ if you are interested in learning PowerApp.
Thanks,
Sanket

Many thanks, Sanket


Styled admin panels to look like projects panel

Addresses #1069

coverage: 84.262%. remained the same
when pulling e040672b838a1db5b26e2c2ae3b84af0fbb80e85 on 1069-Admin_view
into c6e4e660105b87a16239c6c37e7fdeee22f42926 on main.


add ocs default toleration to csi-op podspec

all components deployed as part of odf should tolerate this taint by default "node.ocs.openshift.io/storage=true:NoSchedule"

/lgtm
/appove

/cherry-pick release-4.17


Revert commit that was deleting mixer crs

…blished to kubernetes config server.
Revert "Delete custom resources before uninstalling chart. (#5279)"
This reverts commit d266a5ce4bb16d25867b3e145399a7f61e58739c.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ijsnellf
Assign the PR to them by writing /assign @ijsnellf in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

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

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
dbae0a2fd2c74272d9c6dbf4e0cdf82a02e518a5
link
/test istio-pilot-e2e

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.


Chickencoop integration

I think it would be helpful to append [Hold] and [Press] to the different buttons to make it clearer how they're triggered (where applicable)

Good point - added in e8dc0d7


Missing SAML and JWT bearer grant types in Cloud Support

Describe the issue:

Missing SAML and JWT bearer grant types in Cloud Support
How to reproduce:

1.Login to the console
2.Create a standard based OIDC application
3.Go to protocol section of the application  and check the grant types
Expected behavior:

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

These grant types were not already available in the cloud offering, hence they were not added for the cloud offering during the unification effort.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


board: riscv: qemu: MAX_IRQ_PER_AGGREGATOR should be 1024

MAX_IRQ_PER_AGGREGATOR should be 1024 following the changes made in:
https://github.com/zephyrproject-rtos/zephyr/pull/63496

need to update interrupt level bits


Networking on iOS is broken

Steps to Reproduce
Use a version of Flutter 99718794b39f5b8b4f97e6fd9df268fe46ff9743 or later.
Make a network request
Exception is thrown in mojo_client.dart
The solution is to make sure that the response.headers returned by the networking Mojo service is set correctly on iOS. (It seems to be set correctly on Android already.)
This affects the Stocks app, since it pulls data from the network.

Thanks for the catch! Is there a way to write a test for this regression that can run on our buildbots?

Yes! We are actually already testing it in the stocks app performance iOS test. And the test is broken. :)
@yjbanov

https://github.com/flutter/flutter/pull/3535

Should be fixed now.


Improve and fix docs for EKS-A artifacts and version support

Make command snippets copyable
Enforce pip3 installation for image builds
Run wget and tar commands without sudo to fix issue of /tmp directory being owned by root
Remove dev-release note for usage of image-builder for AMI builds
Remove 1-20/1.20 references.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherrypick release-0.13


Minor typos and spellchecks in quickstart docs

Removed semicolon from python sources and fixed a grammar typo.

Contributor License Agreement signed :)

Thanks for the grammar fix!
The documentation file is autogenerated from the notebook by cd'ing into docs and running make notebooks. The reason for the semi-colons being there is because by default Jupyter will print the return of whatever the last line in each block is if unassigned, which clutters up the documentation.  The semi-colon suppresses that print in Jupyter.
It seems that this may seem odd to people who haven't used Jupyter, but I don't know of a good alternative for the plotting blocks, where we want to suppress the return but want to keep the figure. One alternative is to assign the output, artists = m.plot_components(fcst) but I'm not a fan of that since it may make it seem necessary to assign the output, which you would generally have no need to do.

I pushed the grammar fixes in https://github.com/facebookincubator/prophet/commit/03d3ec9f2edd2f246ad2c072188150320bbc58ce but left the semi-colons in for now.


Python3 Username Error

Hey, i've been trying to use the module and an very successful, the only issue I am having is that I cannot retrieve the username of the person who has written the tweet from the tweet object. It always appears as empty. Please help.

Same here.  The first field is ". Any pointer will be greatly appreciated.  Using Python 3.7.

What is the code line that retrieves Username in TweetManager.py?
The correct Tag to retrieve User name is below
user_id       = str(tweetPQ.attr("data-user-id")).strip() //Twitter User ID
UserName      = tweetPQ.attr("data-screen-name").strip() //Twitter user name that we see on page
It is possible that there are multiple screen names in the string separated with comma. Make sure to handle this case.


Upgrade to VS 2017

Remove project.json and upgrade to MSBuild.
Other notable changes:
Removed unused baseline.json files. There were baseline files for "Microsoft.AspNetCore.Server.Testing" package, even though there was no project by that name. cc @javiercn @Eilon

Those baselines should be moved from Microsoft.AspNetCore.Server.Testing to Microsoft.AspNetCore.Server.IntegrationTesting

That project isn't an automated test project. It's a test app. .notest prevents korebuild from running dotnet test on the project.

Regarding baselines, if the project got renamed, there's no point renaming the baseline, because the whole thing is a breaking change. Is this a package we publish? Should we even be checking for breaking changes in it?

I'm not sure the baseline files were ever used. They were added in 911da31476615a5ce4bfe2c9a59cffeddb8c2422, but even then, there was no project named Microsoft.AspNetCore.Server.Testing. See https://github.com/aspnet/Hosting/tree/911da31476615a5ce4bfe2c9a59cffeddb8c2422/src/Microsoft.AspNetCore.Server.Testing


KAFKA-13438: Replace EasyMock and PowerMock with Mockito in WorkerTest

(Sub task of KAFKA-7438)
Replace all usages of Powermock and Easymock in connect.runtime.WorkerTest with Mockito. One bit of Powermock functionality could not be replicated with Mockito - which was verifying constructor arguments passed to a new instance of WorkerSourceTask within the worker, but this was not critical to test functionality.
The test has been removed from the list of tests that had to be excluded when Gradle is running under Java 16+ due to Powermock, and successful execution was verified with Java 17.0.1 Temurin.

Thanks for the update. There's an unused import that is failing checkstyle:
[ant:checkstyle] [ERROR] /home/jenkins/jenkins-agent/workspace/Kafka_kafka-pr_PR-11817/connect/runtime/src/test/java/org/apache/kafka/connect/connector/policy/BaseConnectorClientConfigOverridePolicyTest.java:23:8: Unused import - org.apache.kafka.connect.runtime.WorkerTest. [UnusedImports]

Thanks for the update. There's an unused import that is failing checkstyle:
[ant:checkstyle] [ERROR] /home/jenkins/jenkins-agent/workspace/Kafka_kafka-pr_PR-11817/connect/runtime/src/test/java/org/apache/kafka/connect/connector/policy/BaseConnectorClientConfigOverridePolicyTest.java:23:8: Unused import - org.apache.kafka.connect.runtime.WorkerTest. [UnusedImports]

Fixed :)


add subset notebook

Use intake-esm to subset down to only the assets we will need to load.

only including models that have all of these variables (monthly tas, huss, pr and daily pr)
doing some other subsetting
should be flexible across glade and ocean.pangeo, although the models it returns could be different if the catalogs are different

This can be merged into @ks905383 's notebook as we continue forward

great should i merge it?

@chiaral yeah go for it! I'm not sure how we wanted to handle merging PRs. I'd vote for anyone feeling free to merge, especially if it's just additions rather than modifications.
In fact, I'll just merge it myself now :)


Hot Corners and  the Mission Control problem

When I use the trigger angle to trigger the Mission Control in the upper right corner, a large old capture shortcut key will appear. However, if I use Control+arrow down to trigger the Mission Control, this bug will not appear.

My system version is macOS Sonoma of Version 14.3 Beta （23D5033f）. It should be noted that I have also reproduced this issue in previous versions.
If you need more additional information, please contact me. Thank you！

By the way, when I use shortcut keys to stop capturing, this problem will still occur and I need to kill the process...

This is an issue with how Mission Control decides what to display, as macOS uses a recent screenshot of the window to display in this animation. This is problematic for KeyCastr since most of the time the window isn't showing anything, but the OS will display a recent snapshot often with an unexpected transform. It has nothing to do with whether key casting is enabled or not -- so long as there was a recent display, it is possible mission control will show something like this.
There isn't an API to directly control what is displayed here, but tweaks to the window lifecycle may be able to improve it. I've also seen other apps having issues with unexpected transforming or displaying of unexpected results in Mission Control, so we're not alone here.

This is an issue with how Mission Control decides what to display, as macOS uses a recent screenshot of the window to display in this animation. This is problematic for KeyCastr since most of the time the window isn't showing anything, but the OS will display a recent snapshot often with an unexpected transform. It has nothing to do with whether key casting is enabled or not -- so long as there was a recent display, it is possible mission control will show something like this.
There isn't an API to directly control what is displayed here, but tweaks to the window lifecycle may be able to improve it. I've also seen other apps having issues with unexpected transforming or displaying of unexpected results in Mission Control, so we're not alone here.

Okay, I get it. Thank you for your reply~


smpp: no type converter available for byte properties

Bug description
Property binding for byte properties does throw
org.apache.camel.NoTypeConversionAvailableException: No type converter available to convert from type: java.lang.String to the required type: byte
How to reproduce:

Create a new project with camel-quarkus-smpp extension:

quarkus create app org.acme:getting-started \     --extension='rest,camel-quarkus-smpp'

Add property camel.component.smpp.data-coding=0 to application.properties

Try to start the application using quarkus dev

It's more likely an issue in camel?

It's more likely an issue in camel?

Yes, it's because CamelBaseBulkConverterLoader cannot deal with the primitive byte. I have it fixed locally.

https://issues.apache.org/jira/browse/CAMEL-21089

Can you backport to 4.4.x branch

Fixed in https://github.com/apache/camel-quarkus/pull/6439.


You can't store images in OBJECTIVE-C

I keep getting this error when I run it and Im about to just switch to AWS...
Body file is unreachable: /var/mobile/Media/DCIM/102APPLE/IMG_2462.PNG
Error Domain=NSCocoaErrorDomain Code=257 "The file “IMG_2462.PNG” couldn’t be opened because you don’t have permission to view it." UserInfo={NSURL=file:///var/mobile/Media/DCIM/102APPLE/IMG_2462.PNG, NSFilePath=/var/mobile/Media/DCIM/102APPLE/IMG_2462.PNG, NSUnderlyingError=0x14e51e0d0 {Error Domain=NSPOSIXErrorDomain Code=1 "Operation not permitted"}}
2016-08-23 19:14:37.037 newG-Bay[2103:655388] Error uploading: Error Domain=FIRStorageErrorDomain Code=-13000 "An unknown error occurred, please check the server response." UserInfo={bucket=g-bay-70b9c.appspot.com, object=FQmQfnhFaBcXDe3mOSjbMNyCUsh1/1471994076675/asset.PNG, NSURL=file:///var/mobile/Media/DCIM/102APPLE/IMG_2462.PNG, NSLocalizedDescription=An unknown error occurred, please check the server response., ResponseErrorDomain=NSCocoaErrorDomain, NSFilePath=/var/mobile/Media/DCIM/102APPLE/IMG_2462.PNG, NSUnderlyingError=0x14e51e0d0 {Error Domain=NSPOSIXErrorDomain Code=1 "Operation not permitted"}, ResponseErrorCode=257}
Please help with this!

Well, we're a little late on this, but the issue here is that app sandboxing on the simulator is potentially different from a physical device, and it's likely that the /var directory isn't user writable on a physical device. See https://developer.apple.com/library/content/documentation/FileManagement/Conceptual/FileSystemProgrammingGuide/FileSystemOverview/FileSystemOverview.html for full details (TL;DR: use /Documents and /tmp)


GH-33999: [Go] Removed  cast from byte[] to string and copied entire string when Value() is called

Rationale for this change
Remove some unsafe casts and copy a string instead of returning a reference to an underlying array.
What changes are included in this PR?
Remove unsafe.Pointer and add a string deep copy.
Are these changes tested?
I think existing tests should suffice.
Are there any user-facing changes?
There are no user facing/breaking changes.

Closes: #33999

Closing because it has been untouched for a while, in case it's still relevant feel free to reopen and move it forward 👍


pypi update

Latest version on pypi is from last year. Could this be updated please? The old version has bugs in it.

sorry - I thought pypi was behind as it looked like the pid bug fix had been merged


Add a GitHub action to run mdsnippets and update contend embedded in the docs

Description
For background, see:
https://publish.obsidian.md/tasks-contributing/Documentation/Embedding+code+and+data+in+documentation
I currently run mdsnippets manually when I know embedded files have changed.
I would like to automate it via GitHub Actions.
Required Steps
The mdsnippets project documents how to set up automated updating of snippets and includes:
https://github.com/SimonCropp/MarkdownSnippets/blob/main/docs/github-action.md
I think I would rather create a Pull Request than a Commit.
There are some example workflows in the Obsidian Hub project that show how to create a Pull Request:
https://github.com/obsidian-community/obsidian-hub/blob/88441476f35b4efe1c0a541faa3e681a86b8a800/.github/workflows/update_issues.yml#L35-L51
For the triggers, I'm thinking:

When a PR is merged
When main is pushed to
With a manual request button:

https://github.com/obsidian-community/obsidian-hub/blob/88441476f35b4efe1c0a541faa3e681a86b8a800/.github/workflows/update_issues.yml#L5-L7
How to Test
No response

@claremacrae do you think this can be closed now? =)

@claremacrae do you think this can be closed now? =)

Hooray - yes, thanks!
(we have implemented this and it is working)


Claims that exist in access_token and not id_token

Describe the issue
At the moment, the code only checks the id_token for claims, some providers seem to stuff all their claims in the id_token but other ones put it in the access_token.
In my case the email claim is set in the access_token what causes caddy-auth-portal to fail because it only looks in the id_token
Expected behavior
Look in the id_token and access_token for the email attribute
Additional Context
For me just replacing id_token to access_token in https://github.com/greenpau/caddy-auth-portal/blob/main/pkg/backends/oauth2/validator.go#L27 and https://github.com/greenpau/caddy-auth-portal/blob/main/pkg/backends/oauth2/validator.go#L33 fixes the issue.
It's also strange that the function is named validateAccessToken but it validates the id_token.

@42wim , thank you for opening the issue.
Please provide your configuration.
OAuth2.0 was initially created with OpenID in mind. Then, when I started adding providers, google, fb, etc., I realized that everything has a twist.

It's also strange that the function is named validateAccessToken but it validates the id_token.

;-) no one is perfect ;-)

:80 {
    log
    route /* {
    jwt {
        auth_url /auth/oauth2/generic
        user_identity id
        primary yes
        allow roles superuser
            trusted_tokens {
                static_secret {
                    token_name access_token
                    token_secret 0e2fdcf8-6868-41a7-884b-7308795fc286
                    token_issuer e1008f2d-ccfa-4e62-bbe6-c202ec2988cc
                    }
            }
        }
        root * /var/www
        php_fastcgi unix//run/php7.4-fpm.sock
        file_server
    }

    route /auth/* {
        auth_portal {
            path /auth
            backends {
                generic_oauth2_backend {
                    metadata_url https://xxxx/.well-known/openid-configuration
                    base_auth_url  https://xxxx/auth/realms/ourrealm
                    method oauth2
                    realm generic
                    provider generic
                    client_id aclientid
                    client_secret aclientsecret
                    scopes openid email profile
                }
             }

             jwt {
                token_name access_token
                token_secret 0e2fdcf8-6868-41a7-884b-7308795fc286
                token_issuer e1008f2d-ccfa-4e62-bbe6-c202ec2988cc
             }
        }
    }
}

@42wim , you actually used generic  provider.
Once again, many thanks for opening this issue!!! 👍
Will fix it tonight and let you know to retest.

@42wim , I added a fix in v1.3.5. Please test and report back.

@42wim , also, please remove all references to token_issuer. I will be removing it in the next release.

@42wim , please use v1.3.6.

No, not working yet.
{"level":"warn","ts":1607426302.2291787,"logger":"http.handlers.auth_portal","msg":"Authentication failed","request_id":"b74da4d6-2bfc-4f34-add0-1287adbce6fd","auth_method":"oauth2","auth_realm":"generic","error":"failed validating OAuth 2.0 access token: email claim not found"}

caddy list-modules --versions|grep -e auth_portal
http.handlers.auth_portal v1.3.6

Your code change looks if a id_token or access_token exists, but they always will both exist (afaik), the issue is that in this case only access_token has a claim email while the id_token doesn't has that claim, but does have other claims like iat, aud, etc ..

Your code change looks if a id_token or access_token exists, but they always will both exist (afaik), the issue is that in this case only access_token has a claim email while the id_token doesn't has that claim, but does have other claims like iat, aud, etc ..

@42wim , please post the data structure here. It would help addressing the issue.
Also, which product/service?

what data structure?
product is keycloak
This works btw, but ofcourse breaks other flows https://github.com/42wim/caddy-auth-portal/commit/de983a452ea75fb4abd3be43f629aedb6d72daf3

@42wim , please post the JSON you get where you have id_token and acess_token

{
"access_token": "jwtstring",
"expires_in":300,
"refresh_expires_in":1800,
"refresh_token":"jwtstring",
"token_type":"bearer",
"id_token":"jwtstring",
"session_state":"string",
"not-before-policy":0,
"scope":"openid profile email"
}

@42wim please decode both “jwtstring” values at jwt.io , replace personal identifiers with placeholders, and post it here.

@42wim , I tried to find how it looks myself, but even redhat did not decode it :-)
https://developers.redhat.com/blog/2020/01/29/api-login-and-jwt-token-generation-using-keycloak/
The json payload is at the bottom of the page.

I'll do later when I have time to actually fetch one.
But they just look like any other.. except the access_token has a email Claim and the id_token doesn't

More information here btw: https://openid.net/specs/openid-connect-basic-1_0.html#UserInfoResponse

More information here btw: https://openid.net/specs/openid-connect-basic-1_0.html#StandardClaims

@42wim , I would not have asked you to provide me how it looks unless it was necessary. Each implementation has its intricacies. That's why I need the exact thing you receive from keyclock.

But they just look like any other.. except the access_token has a email Claim and the id_token doesn't

@42wim , check this out ... when I am reading this I don't know whether you mean that (1) access_token has that same exact fields as id_token or (2) access_token has only email field and id_token has all the rest of the fields.

@42wim , have you considered re-configuring the Keycloak server to include email address in id_token? just a thought...

Here you go:
accestoken:
{
  "exp":<PHONE_NUMBER>,
  "iat":<PHONE_NUMBER>,
  "auth_time":<PHONE_NUMBER>,
  "jti": "03ceebd9-d9b4-463e-934f-5fe790d0d7bc",
  "iss": "https://xxxx/auth/realms/xxxx",
  "aud": [
    "broker",
    "account"
  ],
  "sub": "f2701802-710f-4cec-bc5f-3a1ce9b000ab",
  "typ": "Bearer",
  "azp": "oidcapp",
  "nonce": "0VJwTfHcTsvErCIPJzMQG8a3O7UciG5B",
  "session_state": "3df54660-c0cc-4feb-88c5-22bbe910ff0e",
  "acr": "0",
  "allowed-origins": [
    "https://xxxx",
  ],
  "realm_access": {
    "roles": [
      "xxx",
    ]
  },
  "resource_access": {
    "broker": {
      "roles": [
        "read-token"
      ]
    },
    "account": {
      "roles": [
        "manage-account",
        "manage-account-links",
        "view-profile"
      ]
    }
  },
  "scope": "openid auto_registration profile email",
  "name": "xx",
  "privateAttribute1": [
    "1234"
  ],
  "privateAttribute2": "xxxx",
  "preferred_username": "xxxx",
  "given_name": "xxx",
  "family_name": "xxx",
  "email": "xxx@yyy"
}

id_token:
{
  "exp":<PHONE_NUMBER>,
  "iat":<PHONE_NUMBER>,
  "auth_time":<PHONE_NUMBER>,
  "jti": "9840f4f8-9977-44ae-98c7-3afdbba9929b",
  "iss": "https://xxxx/auth/realms/xxxx",
  "aud": "oidcapp",
  "sub": "f2701802-710f-4cec-bc5f-3a1ce9b000ab",
  "typ": "ID",
  "azp": "oidcapp",
  "nonce": "0VJwTfHcTsvErCIPJzMQG8a3O7UciG5B",
  "session_state": "3df54660-c0cc-4feb-88c5-22bbe910ff0e",
  "acr": "0",
  "name": "xxxxxx",
  "preferred_username": "xxxxx",
  "given_name": "xxx",
  "family_name": "xxx"
}

have you considered re-configuring the Keycloak server to include email address in id_token? just a thought...

Yes, but I'm not the administrator of keycloak, so I can't do that.
Access token are variable depending on the application extra claims/metadata will be added.
Afaik there is no rfc that specifies the email claim must exist in the id_token

@42wim , so you care about passing the roles from the below?
"realm_access": {
"roles": [
"xxx",
]
},

@greenpau ideally yes, but also the privateAttribute claims and preferred_username :-)
But I think this is out the scope of this issue

@42wim , I just released v1.3.7. Please add  identity_token_name directive and test it.
            backends {
                generic_oauth2_backend {
                    metadata_url https://xxxx/.well-known/openid-configuration
                    base_auth_url  https://xxxx/auth/realms/ourrealm
                    method oauth2
                    realm keycloak
                    provider generic
                    client_id aclientid
                    client_secret aclientsecret
                    scopes openid email profile
                    identity_token_name access_token
                }
             }

Spent way too much time on debugging token validation error  {"error": "user invalid"} in my test setup.
Conclusion, caddy-auth-portal is always setting its cookie to Secure even on http:// which causes this error (and creating a loop). Should I open another issue for this?
Wrt your current fix, it works, thanks! :+1:
But isn't it better to look into both id_token and access_token to find the roles/email claim?
That way users don't need to specify the identity_token_name (which most of the users probably don't even know how to debug this).

Conclusion, caddy-auth-portal is always setting its cookie to Secure even on http:// which causes this error (and creating a loop). Should I open another issue for this?

@42wim , yes.

But isn't it better to look into both id_token and access_token to find the roles/email claim?
That way users don't need to specify the identity_token_name (which most of the users probably don't even know how to debug this).

Idk what is better. I think having a single source of truth is better. Hence, I do not unpack both. Side note, when I receive a token, I unmarshal it to a “limited” JWT claims. In your case, the fields the claim care about exist in access_token.
See also: https://github.com/greenpau/caddy-auth-jwt

That way users don't need to specify the identity_token_name (which most of the users probably don't even know how to debug this).

@42wim , I think the goal is having “provider” for each service/product is better. This way, a user does not need to think about custom directives. In this case, I would say “keycloak” may be necessary. It may simplify configuration due to some defaults.

Spent way too much time on debugging token validation error {"error": "user invalid"} in my test setup.

@42wim , I recommend using https://github.com/greenpau/caddy-trace
And adding “response_debug” arg

That way users don't need to specify the identity_token_name (which most of the users probably don't even know how to debug this).

@42wim , I think the goal is having “provider” for each service/product is better. This way, a user does not need to think about custom directives. In this case, I would say “keycloak” may be necessary. It may simplify configuration due to some defaults.

Well, the problem with on-premise/hosted setups like keycloak is that everyone can configure it how they like, there's no rule to put email in access_token or id_token, so some keycloak setups will have it in one, both or none :)

Well, the problem with on-premise/hosted setups like keycloak is that everyone can configure it how they like, there's no rule to put email in access_token or id_token, so some keycloak setups will have it in one, both or none :)

@42wim, point taken :-)

put email in access_token or id_token,

When I was working on Okta integration, I had to do some custom config. Please take a look at the screenshots here. I say Keycloak admin can probably configure passing any info :-)

@42wim , specifically this screenshot.

"ID Tokens MAY contain other Claims. Any Claims used that are not understood MUST be ignored."
The known claims are iss, sub, aud, iat, auth_time, nonce, acr, amr and azp.  email is not standardized for the ID token. See: https://openid.net/specs/openid-connect-core-1_0.html#IDToken
email is standardized when requesting claims scopes, and thus in Access Tokens: https://openid.net/specs/openid-connect-core-1_0.html#ScopeClaims
Anyway, the spec is poorly specified (aha) by not making a choice how to pass claims. Keycloak returns an ID, access and refresh token when using the Authorization Code Flow (https://www.keycloak.org/docs/latest/server_admin/#authorization-code-flow_) and claims (mappers) can be configured to also be sent with the ID token (not the default though).
In fact, the main problem with ID tokens is that you may need to keep them small, because they can be passed around in the URL and you may hit the length limitations (IE was known for a 2K maximum).
Thx for your work!


Fix push promise unit test

$subject

Codecov Report

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

@@            Coverage Diff            @@
##             master   #15655   +/-   ##
=========================================
  Coverage          ?    6.78%
  Complexity        ?     1719
=========================================
  Files             ?     1385
  Lines             ?    70603
  Branches          ?     9084
=========================================
  Hits              ?     4790
  Misses            ?    65335
  Partials          ?      478

Impacted Files
Coverage Δ
Complexity Δ

...inalang/net/http/nativeimpl/promise/SetHeader.java
38.46% <ø> (ø)
2 <0> (?)

Continue to review full report at Codecov.

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


Database connection contention spans

Problem Statement
We're conducting a performance testing and we saw that most of the time is consumed in between the API call and the actual query.
We don't know yet where the problem is but we have two suspicions:

The database is rate limiting or taking more time to connect due to this contention.
There is a limit in the database/sql and the pool is locking the current query until another connection is released.

Proposed Solution
Add more spans to the driver so that it also registers traces for the connection opening times as well as internal locking times. I don't know if it's even possible.
Alternatives
Logging :D

Hi @XSAM,
We also have a similar connection contention issue where there's some delay to acquire a connection. If you look at the sample trace attached below, we see that there's some unknown wait time which we are assuming that is the time to acquire a connection.

I was thinking that adding a span at this connect method might give us some better insights on the time taken for acquiring a connection.
Please correct me if I am wrong. Happy to raise a PR :)
Thanks!

I was thinking that adding a span at this connect method might give us some better insights on the time taken for acquiring a connection.

@rajeshkarnena You are right. Adding a span at connect method can give better insights.

Are you interested in implementing this? If so, I can assign this issue to you.

Hi @XSAM, yes I'm interested. Please assign this to me :)


-restcomm docker image on windows 8.1

I am trying to install Docker image of Restcomm on my windows 8.1 laptop by following http://www.telestax.com/rapid-webrtc-application-development-with-restcomm-and-docker/. I am able to install DOCKER and run the container Hello-world properly.
But when i run the command to create container... "docker run –name=restcomm -d -e STATIC_ADDRESS=”YOUR_HOST_IP_ADDRESS_HERE” -p 8080:8080 -p 5080:5080 -p 5082:5082 -p 5080:5080/udp -p 65000-65535:65000-65535/udp gvagenas/restcomm"
i am getting the following error... Error response from daemon: Cannot start container c88fcab56034096e98ddcd71d1d2db17e5858b88c64b1859efcb86d740e3972c: failed to create endpoint restcomm on network bridge: iptables failed: iptables --wait -t nat -A DOCKER -p udp -d 0/0 --dport 65116 -j DNAT --to-destination <IP_ADDRESS>:65116 ! -i docker0: (fork/exec /usr/local/sbin/iptables: cannot allocate memory)
Request for your help and suggestion and thanks in advance
Rgds Ias

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

PLs. find below some more information about our docker env...
$ docker version
Client:
Version:      1.9.0
API version:  1.21
Go version:   go1.4.3
Git commit:   76d6bc9
Built:        Tue Nov  3 19:20:09 UTC 2015
OS/Arch:      windows/amd64
Server:
Version:      1.9.0
API version:  1.21
Go version:   go1.4.3
Git commit:   76d6bc9
Built:        Tue Nov  3 19:20:09 UTC 2015
OS/Arch:      linux/amd64
$ docker info
Containers: 1
Images: 48
Server Version: 1.9.0
Storage Driver: aufs
Root Dir: /mnt/sda1/var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 50
Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.1.12-boot2docker
Operating System: Boot2Docker 1.9.0 (TCL 6.4); master : 16e4a2a - Tue Nov  3 19:49:22 UTC 2015
CPUs: 1
Total Memory: 1.956 GiB
Name: default
ID: 26VR:H6QC:RRWB:HI5Y:ALJW:TEJ7:5YVG:TYBP:X7WE:XPOA:FTXG:UREX
Debug mode (server): true
File Descriptors: 46
Goroutines: 224
System Time: 2015-11-07T14:07:57.578235433Z
EventsListeners: 0
Init SHA1:
Init Path: /usr/local/bin/docker
Docker Root Dir: /mnt/sda1/var/lib/docker
Labels:
provider=virtualbox
$ uname -a
MINGW64_NT-6.3 Sai-P2W 2.3.0(0.290/5/3) 2015-09-16 19:07 x86_64 Msys

Hello
I have also tried to start the RESTCOMM Container through kitematic and still the same issue and for more details pls. refer to the screen links below.
Request for our help and suggestion as  in what i am i missing and i can't find similar issues in documentation or forum links as well.

thanks in advance
Ias

fork/exec /usr/local/sbin/iptables: cannot allocate memory

It wouldn't surprise me if this is the problem you're running into and the virtual machine is running out of memory, causing it to fail.

You could try to give the virtual box virtual machine more memory, or set memory constraints on the container; http://docs.docker.com/engine/reference/run/#user-memory-constraints to prevent it from using too much memory.

This issue seems to involve multiple open source components from various Docker projects. This makes it a whole lot more complex to debug on an issue tracker. This is an issue to be discussed on a forum, not on the Docker engine issue tracker.
This kind of question and general support questions are more adequate for the following:

the forums: https://forums.docker.com/
the docker-user mailing list https://groups.google.com/forum/#!forum/docker-user
the #docker channel on Freenode.
StackOverflow https://stackoverflow.com/questions/tagged/docker

I'm going to close this issue now because this doesn't seem to be a Docker issue and the discussion can continue and assistance can be provided even if the issue is in closed state. Please feel free to continue the discussion. There's no need to open a new issue on this topic.
For everyone who's getting started, please consider starting a discussion on the forums if you're getting started and you need help. Thank you for understanding.


Dynamic Height

Hi. How can I adjust the height according to the list ? Thanks

any luck on this issue i have same

@bcsezer @rlogical-techsoft Use the preferredMaxLayoutWidth  property like UILabel to adjust height based on content automatically.


spread rest obj type elem

wip
add SpreadRest ObjTypeElem and update Infer.fs and Unify.fs to use that instead of intersection types

Codecov Report
Attention: Patch coverage is 76.82927% with 38 lines in your changes missing coverage. Please review.

Project coverage is 84.01%. Comparing base (4190d3e) to head (d08ad74).

Files
Patch %
Lines

src/Escalier.TypeChecker/Unify.fs
78.50%
23 Missing :warning:

src/Escalier.Codegen/Codegen.fs
0.00%
5 Missing :warning:

src/Escalier.Data/Library.fs
20.00%
4 Missing :warning:

src/Escalier.TypeChecker/Infer.fs
92.10%
3 Missing :warning:

src/Escalier.TypeChecker/Visitor.fs
50.00%
2 Missing :warning:

src/Escalier.TypeChecker/Folder.fs
66.66%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #323      +/-   ##
==========================================
- Coverage   84.16%   84.01%   -0.15%
==========================================
  Files          23       23
  Lines       11625    11660      +35
  Branches     1603     1625      +22
==========================================
+ Hits         9784     9796      +12
- Misses       1841     1864      +23

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


Calendar overrides schedules colors

I am trying to set a background color of a schedule. It seems that as soon as a schedule is a part of calendar as a parent, it takes its background color.

Version

tui-calendar: 1.7.0
Development Environment

"webpack-mix": "^2.0.2"
Current Behavior

Set calendars background color and create a schedule with a different background color. When instantitating a tui-calendar object, all of schedules within a certain calendar will take its parents background color.
Calendar BG color: '#bbdc00'
Schedule BG color: '#bbdc00'
    var calendar = new CalendarInfo();
    calendar.id = 1;
    calendar.name = 'Friends';
    calendar.bgColor = '#bbdc00'; // Set default calendar color

    var schedule = new ScheduleInfo();
    schedule.id = chance.guid();
    schedule.calendarId = calendar.id;
    schedule.category = "time";
    schedule.title = calendar.name;
    schedule.bgColor = "#ffffff" // Note a different color

    var cal = new Calendar("#calendar", {
        defaultView: "week",
        taskView: false,
        scheduleView: ["time"],
        calendars: [calendar],
    });

    cal.createSchedules([schedule]);

Expected Behavior

When specifying a different schedule color than calendars, this one should take effect.
Calendar BG color: '#bbdc00'
Schedule BG color: '#ffffff'

I have partially resolved this issue using
         ScheduleList.forEach(function (schedule) {
            cal.updateSchedule(schedule.id, schedule.calendarId, {
                bgColor: #ffffff"
            });
         });

but this seems over engineered. Is there other options to achieve this?

Try this api setCalendarColor().


GUI Development

I'm very beginner, I took a VB class in highschool back in 2011, but I wrote up a simple GUI for maxcso that I wanted to put out there. The only real option right now is selecting the thread count, but other than that it passes no other arguments. I'm not distributing maxcso, as the GUI I've wrote requires a compiled maxcso binary along side it in it's folder, so it's all my work.
I'd love to have someone take a look at it, and see if it works well, and offer any improvements!
The repo is at https://github.com/sethfoxen/maxcsoGUI

Will take a look.
If you want, you can also try writing a GUI that runs maxcso as a library.  The CLI is actually just using the "main library" here:
https://github.com/unknownbrackets/maxcso/blob/master/cli/cli.cpp
So for example, a progress bar could be implemented using the maxcso::TASK_INPROGRESS status.
But that doesn't mean anything is wrong with calling the exe (although that might be somewhat Windows only.)
-[Unknown]

It's limited, and I have have pretty much ZERO experience with any practical coding, and really was just looking for something to simplify batch compression. Wanted to keep it simple. Download a maxcso release, pop it in the GUIs folder, and there you have it. I've added a few more arguments, just calling the exe makes it simpler to implement changes. Added a new compression method? Simply add a line, a checkbox, and rebuild.

I got one in the works myself currently using Python and Tkinter (because its simple and works on damn near everything)
It currently can:

Use either x86 or x64 releases
Detect the number of threads available on a system and allow the user to set how many to dedicate with a slider (Windows and Linux, defaults to 1)
Allow multiple files to be chosen with filtering so only supported formats are shown (+ Ignore duplicates)
Set output format
Choose methods

Not sure if I want to keep the progress bar or simply add percentages beside filenames then transition to a checkmark or something when they are done.
The rest of the options will likely be added as advanced options that can be shown on toggle as to keep the UI simple.
Ugly screenshot as I haven't adjusted the layout:

That's interesting.  I'd suggest defaulting the threads to at least half the available (if not all, like maxcso does by default), because maxcso tries multiple methods and can be slower with only a single thread.  It's designed to scale very well with additional threads.
-[Unknown]


feat: allow changing highlight color of days included in range

When working with this library on a dark background, the range highlight color is hard to see. The changes below add a new config option to be able to customize this.

Hi @jmsheff ,
Could you please attach a screenshot of this issue?


Pop Culture Reference Suggestions - Star Wars

Please advise on the following perceptions I had on the pop culture ref:

C-3PO "madness" GIF seems to be giving a negative connotation as the walkthrough begins. Perhaps something more positive so that peeps are able to take it under their swing?

Suggested: Luke with a headgear covering his eyes using lightsaber against tiny droid on Millenium Falcon for first time or related theme.

Perhaps relating the "power of machine learning" with "the force"? There are very few references to the force here.

For train and test, perhaps something from Jedi Training?

Suggested: Going from padawans to Jedi masters: parallels for both the ML model and the ML programmer? "When I left you I was but the learner, now I am the master"

Choosing right ML model with "You were the chosen one"?
Data Science pays a lot - "Be careful not to choke on your aspirations"?
Poor performing model - "Your powers are weak old man" / "I hate sand" / You have failed me for the last time admiral
"I don't believe it" - "That is why you fail" when the model is explained but students may have a hard time getting their head around the math.
"Never tell me the odds" for accuracy rating?

The presence of .DS_Store files in the commit prevents a successful pull.
Revert 110dc40 and re-commit without .DS_Store

The presence of .DS_Store files in the commit prevents a successful pull.
Revert 110dc40 and re-commit without .DS_Store

.DS_Store has been removed and added to .gitignore from the latest commit. If all ok from thy side, close off the issue.

There is still a .DS_Store file in the main folder.
Reset to the previous commit and move the .gitignore file to the root directory of the repo


Bug fix when relation with forced foreign key

When you have a relation with forced foreign key the code fails. For example:
/**
 * Get the products records associated with the warehouse shelve.
 *
 * @return \Illuminate\Database\Eloquent\Relations\HasMany
*/
public function products()
{
     return $this->hasMany('Modules\Warehouse\Entities\Product', 'default_warehouse_shelve_id', 'id');
}

@gcphost This is a bug fix please push it :)

Updated packagist to auto-update, just need to create a new release after.


Fix intermittent package test failing for node startup issue

In scope of https://www.pivotaltracker.com/story/show/154770099
Please refer to commit message for details.

It looks safe. This was added to cover unit tests which used default OS OTP, and in consequence root dir was in restricted location. After your change keys will go to data dir, which is relatively new and safe development


Custom fee for contract deploy in explorer via Sandbox is broken

Testing explorer on mainnet
Noticed that when user tries to deploy a contract using the sandbox
User tries to input Custom fee estimate
User intentionally used a low fee which will throw validation error
User received the validation error that the tx will fail
User then tries to input a different custom fee value,validation error doesn't goes away
User selects the fee dropdown and notices only custom option is visible as well
Adding video recording of the issue

https://user-images.githubusercontent.com/38872890/145887126-e6abe6fc-fe21-44df-8902-12b062504714.mov

@Eshwari007 Mind retesting this to see if it's still an issue?

@Eshwari007 Mind retesting this to see if it's still an issue?

@markmhx The issue mentioned above is no longer reproducible. Will close the ticket.


If the data of a row exceeds 512, will the excess part be discarded?

Excuse me, I read the function def get_item() in class BlockDatast and found it only intercept 512 tokens. So is the rest of it discarded?
(I doubt this statement, because there are two data segments from the same input data)
Thanks for your solution!

The get_item method will randomly select a continuous segment of 512 tokens if the document is longer than 512 tokens, and concatenate multiple documents if the  document is shorter than 512 tokens.
You can ignore the loss_mask data segment. It is originally designed to deal with the prompt part in a document that should not be learned by the model, and never used in GLM pretraining.


Issue running Gauge Tests in Parallel

Discussed in https://github.com/getgauge/gauge/discussions/2157

Originally posted by harimadusumilli November  8, 2021
Running Parallel Tests in Gauge using Selenium seems to execute in series although browser launches in parallel.
environment : Windows 10 & Ubuntu Linux container too.
Gauge version: 1.1.8
Commit Hash: b1501f4
Plugins
dotnet (0.4.1)
html-report (4.0.12)
js (2.3.15)
json-report (0.3.5)
screenshot (0.0.1)
xml-report (0.2.3)
In the default. properties file we have also added the following entry:
enable_multithreading = true
Three spec files
We are running three spec files which contain the same navigation steps.
and are running them as below:
gauge run specs -p -n=4  --verbose
We are using static webdriver instance, this appears to conflict and creates in consistency while running it. The behavior appears to be erratic, for e.g:, we notice that the data is input incorrectly to a test case as the web driver appears to be key in the test data into only the browser it has a handle instead of their respective browser instances.
Code is as below:
using System;
using System.Collections.Generic;
using System.Linq;
using FluentAssertions;
using Gauge.CSharp.Lib;
using Gauge.CSharp.Lib.Attribute;
using OpenQA.Selenium;
using OpenQA.Selenium.Chrome;
using System.Threading;
namespace DefaultProj
{
public class StepImplementation
{
    //This is the static webdriver

private static IWebDriver seleniumDriver;
    public void initWebDriver(){
        seleniumDriver = new ChromeDriver(Environment.CurrentDirectory + "/Selenium Drivers");
        ((IJavaScriptExecutor)seleniumDriver).ExecuteScript("window.focus();");
        seleniumDriver.Manage().Window.Maximize();
    }

    public void LaunchApp(){
        seleniumDriver.Navigate().GoToUrl("https://our_intranet_app.net/");
    }

    [Step("search with name <name>")]

public void SearchWithName(string name)
{
try{
this.initWebDriver();
this.LaunchApp();
seleniumDriver.FindElement(By.Id("search-inside-textbox")).SendKeys(name);

seleniumDriver.FindElement(By.XPath("//span[@class='icon icon-search']")).Click();

GaugeMessages.WriteMessage("Name search");
GaugeScreenshots.Capture();
seleniumDriver.Close();
}
catch(Exception e){
    Console.WriteLine(e.Message);
    seleniumDriver.Close();
}

}
}

}

Hello Gauge Support, checking and following up to see if this issue of mine is on your radar.
Thanks in advance.
Hari


[BUG] Website freezes after leaving the a form blank.

Describe the bug
In the getting started section, when someone leaves the username and password fields blank, the application freezes for some time and then shows the error 502 bad gateway
To Reproduce
Go to https://nest.land/#start. Leave the username and password section blank. Click the fetch your api key button.
Expected behavior
To fetch the api key for an existing user.
Screenshots

Thanks so much for creating this issue! That will be an easy fix on my end. I'll see if I have time to fix this in a few hours. For now, you can refresh the page if you're trying to log in or create an account.
Sorry for the inconvenience!


Fix warning for extraneous comma.

This fixes this warning:
.../wslay/lib/wslay_frame.c:116:6: warning: empty expression statement has no effect; remove unnecessary ';' to silence this warning [-Wextra-semi-stmt]
};
^

Thank you for PR.  Merged now.


iptables, flush and restore raw table

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[ ] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[ ] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[ ] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[ ] All package sources are available
[ ] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[ ] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[ ] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[ ] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

Stopping the iptables services does not remove all rules from the system. Ensure that the raw table is flushed as well
Change Log

Flush and reset raw table on iptables stop

Does this affect the toolchain?

NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-YYYY-XXXX

Test Methodology

Pipeline build id: xxxx

Started a build here

Started a build here

Build failed due to pipeline. Trying a fullbuild here  instead

Started a build here

Build failed due to pipeline. Trying a fullbuild here  instead

Also failed on pipeline due to unrelated dependency circle. Built full build locally and succeeded. Tested in recent 3.0-dev VM and confirmed rpm installed and raw table flush works as expected
Test steps:
Add rules
sudo iptables -t raw -A PREROUTING -p tcp --dport 80 -j DROP
sudo ip6tables -t raw -A PREROUTING -p tcp --dport 80 -j DROP
Confirm new rules
sudo iptables -vL -t raw
sudo ip6tables -vL -t raw
reboot
Confirm no rules
sudo iptables -vL -t raw
sudo ip6tables -vL -t raw