ITK v5

@LucasGandel @SimonRit
Can this set of fixes be incorporated?  Do you need/want me to cherry-pick some of them off into smaller requests?

@hjmjohnson Sorry for not taking a look at it earlier. Yes, that would be great to split this change into multiple MRs if you can. I can think of three different changes :

unused typedefs + unused variables
ITK_OVERRIDE + const correctness
A more global one changing the library name, cmake version ...

I would remove 45c0361, following my last comment I think 6944b0d is enough.

Broken apart into #138 , #155, #156 , #157, #159,  in the hopes that these can be reviewed more rapidly for incorporation.
The external tool that depends on RTK has been disabled for months due to the excessive overhead needed to maintain private branches.  If this burden persists, we will need to evaluate if we can continue to depend on RTK, or if we will need to permanently fork and diverge from the main tree.

We'll do our best, sorry for the delay and thanks a lot for the work!


Downloading invalid codeforces contest raises error

Downloading a codeforces contest which does not exists or has not started yet raises a ParserError.
Minimal Steps to reproduce
cpcli -c cf:12227 download

Hi, I'd like to take this up.

@nimeshvashistha Hey! Sure, go ahead.


bug: issue with login after db:reset

After a deno start db:reset command, when trying to go to the /account page, there is a 'page not found' error
To Reproduce

login
run a deno start db:reset command
try to go to /account page
you will get a page not found error

In order to solve this, you will have to go to your browser and delete the store session cookies.
After that, you will be logged out.
Now you can log in and normally access the functionalities.
Expected behavior
It shouldn't respond with a 'page not found' and should be able to handle this situation, recognizing that the user is not logged in and redirecting to a login page.

Currently, getSessionId() only checks the session cookie value. Perhaps, it should do a KV check too...

Yeah, I'll apply this fix in the deno_kv_oauth module.

Oh, I realise this is a proposal to add redirect functionality. Yep. I'll add this in if no one gets to this before me.

Let's fix this in #249.

Fixed in #249!


How is external code supposed to "instantiate your own"

Comment on logrus.New() suggests that user can instantiate a Logger, but how is the make(levelHooks) supposed to work when levelHooks is not exposed?
https://github.com/Sirupsen/logrus/blob/master/logger.go#L35

Thanks! Fixed in 93a1736


Changes demo app

The purpose is to switch the demo app to another one that works better as the original one does not work well with url that specify a port as discussed at https://github.com/FreeCodeCamp/FreeCodeCamp/issues/7525

LGTM


Support local testing

It should be possible to develop and test Eggo locally without running on a cluster.

Fixed in https://github.com/bigdatagenomics/eggo/commit/2259d85bd2ffdc7a8b5953a7a2c19bfd6d6afb90


fix: make SQL rel vars unique within a query

Mostly fixes #3713

No, the generated SQL for loop was actually incorrect. Loop was done a bit sloppily, so when I added code that was mutating names after they had been assigned, it stopped working.
It's nice that we have quite a lot of tests that catch stuff like this.


Use proper semantic HTML for inline text formatting

Feature Description
Since #1323 all inline text formatting in a text field is done using  and inline style properties.
This should be changed to proper semantic HMTL elements where applicable - e.g. going from the current:
Hello
world!

To:
Hello
world!

Thus using semantic elements where possible (for bold font-weights, italic and underline) and keep spans where not (for non-bold font weights, text color and letter spacing).
This however requires moving away from the current draft-js-export-html dependency to a custom DraftJS-to-HTML converter (and of course vice-versa).

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria
QA Instructions

@barklund do you think this is important for Beta? Stable?

Related: #5326

Related: #5326

@miina  this feels like a prometheus thing?  Not really sure why it got relabeled pea pod (but that was like 14 months ago so....)


monthSelectPlugin and maxDate or minDate

https://jsfiddle.net/8640edj5/

flatpickr version used: 4.6.1
Browser name and version: Chrome 75
OS and version: Xubuntu 19.04
Months that are less than minDate or more are maxDate active. Maybe it will be better if it is blocked?

+1

#1888

+1

This has been released

@viorelsfetea Quick Q on this - can you 'enable' which months are available just like you can with the regular flatpickr?


Send image from file browser

Currently, when you click the image button to send a picture, an overlay appears with the text "Drag and drop files into Goofy to attach them to your message". Is it possible instead to launch an open file prompt inside Goofy and choose the file from there?
Messenger for Desktop seems to be able to accomplish this.

This is also an annoyance because sometimes drag and drop will just stop functioning. Perhaps that should be a separate issue?

duplicate of #68


Feature/logsource

Created an example source that streams Apache request log entries

Pull Request Test Coverage Report for Build 241

16 of 17 (94.12%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.01%) to 93.949%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/scala/org/codefeedr/plugins/log/ApacheLogFileSource.scala
16
17
94.12%

Files with Coverage Reduction
New Missed Lines
%

src/main/scala/org/codefeedr/plugins/rss/RSSItemSource.scala
1
92.0%

Totals

Change from base Build 226:
0.01%

Covered Lines:
295

Relevant Lines:
314

💛  - Coveralls


Preventing of Loading Hidden LookupEditor

In my Form, I have a Lookup editor and its visibility is set to false. But when I open this form it loads the data of lookup editor. [I see this using chrome console]
How can I prevent not to load them if they are hidden?

if you dont use, remove if from form.

Ok.
Is there anyways to do this without remove them from Form.

if you dont want to load lookup, dont use lookupeditor. Lookup editors is auto registering on constructor when form is generated.

Create new editor based lookupeditor. And override UpdateUtems method

Use [StringEditor] in form, though i don't understand why you want that field in form if you won't use it.

I think it's because he uses the form in multiple places. This is the guy with 1,000 tables

Many many thanks for helping.


Error while plotting EF

Describe the bug
While trying to plot the Efficient Frontier, the following error occurs:
TypeError: can't pickle osqp.OSQP_results objects
Expected behavior
Expected to get a plot of the effecient frontier.
Code sample
fig, ax = plt.subplots()
plotting.plot_efficient_frontier(ef, ax=ax, show_assets=False)

# Find the tangency portfolio
ef.max_sharpe()
ret_tangent, std_tangent, _ = ef.portfolio_performance()
ax.scatter(std_tangent, ret_tangent, marker="*", s=100, c="r", label="Max Sharpe")

# Generate random portfolios
n_samples = 10000
w = np.random.dirichlet(np.ones(len(mu)), n_samples)
rets = w.dot(mu)
stds = np.sqrt(np.diag(w @ S @ w.T))
sharpes = rets / stds
ax.scatter(stds, rets, marker=".", c=sharpes, cmap="viridis_r")

# Output
ax.set_title("Efficient Frontier with random portfolios")
ax.legend()
plt.tight_layout()
plt.savefig("ef_scatter.png", dpi=200)
plt.show()

Operating system, python version, PyPortfolioOpt version
Binder environment
Additional context
i'm facing the same issue like Error while plotting EF #332, Binder link https://mybinder.org/v2/gh/robertmartin8/pyportfolioopt/6158b1e22819d174bae8885fc74eed772890bb55?urlpath=lab%2Ftree%2Fcookbook%2Fbondef.ipynb
I don't know whether this link works.

an interesting issue is i ran these code a week ago, and got a plot of the effecient frontier that time. So i suspect some automatic update issue.

HI @zdlightman,
This seems to be similar to #332, which was caused by running the optimiser before plotting. For technical reasons, this doesn't work (though we are currently working on a fix). Let me know if that works!

Thank you very much for answering, and the tips of formating. i'm really new to Github and major in finance - not CS.
After removing the optimiser from this block, i successfully generated both the random portfolio and the ef.
i was reading your user guide https://pyportfolioopt.readthedocs.io/en/latest/Plotting.html
and i have noticed your reminder
To produce a plot of the efficient frontier, you should instantiate your EfficientFrontier object and add constraints like you normally would, but before calling an optimization function (e.g with ef.max_sharpe()),
i used your original sample code:
'''
fig, ax = plt.subplots()
plotting.plot_efficient_frontier(ef, ax=ax, show_assets=False)
Find the tangency portfolio
ef.max_sharpe()
ret_tangent, std_tangent, _ = ef.portfolio_performance()
ax.scatter(std_tangent, ret_tangent, marker="*", s=100, c="r", label="Max Sharpe")
Generate random portfolios
n_samples = 10000
w = np.random.dirichlet(np.ones(len(mu)), n_samples)
rets = w.dot(mu)
stds = np.sqrt(np.diag(w @ S @ w.T))
sharpes = rets / stds
ax.scatter(stds, rets, marker=".", c=sharpes, cmap="viridis_r")
Output
ax.set_title("Efficient Frontier with random portfolios")
ax.legend()
plt.tight_layout()
plt.savefig("ef_scatter.png", dpi=200)
plt.show()
'''
if the whole
'''# Find the tangency portfolio
ef.max_sharpe()
ret_tangent, std_tangent, _ = ef.portfolio_performance()
ax.scatter(std_tangent, ret_tangent, marker="*", s=100, c="r", label="Max Sharpe")
'''
won't work, which is running the optimiser, how can I get the same picture as you on the website?(e.g. with Max Sharpe portfolio)
and I do recall that I got the whole picture with Max Sharpe portfolio one time last week.
Anyway, thank you again for your kindly answering, the question above is not important. Pypfopt is really a great project.

@zdlightman the second part should be ok, e.g the below script (this is also in the binder notebook at the bottom):
ef = EfficientFrontier(mu, S)
ef.add_constraint(...)
plotting.plot_efficient_frontier(ef, ax=ax, show_assets=False)

ef.max_sharpe()
ret_tangent, std_tangent, _ = ef.portfolio_performance()
ax.scatter(std_tangent, ret_tangent, marker="*", s=100, c="r", label="Max Sharpe")

Just so you know, the reason why there are these issues is that any time you call max_sharpe, efficient_risk, efficient_return, constraints automatically get added to the object. Which is why you can't keep calling the optimisation function in a loop, unless you either 1) reinstantiate the object in each loop 2) deep copy the original (un-optimised) object in each loop.
Hope this helps!


[Radio] Driver

Should provide such methods:

check() - mark as checked
uncheck() - mark as unchecked
toggle() - mark as checked if unchecked or vice versa
isChecked()
mouseOver()
mouseOut()
click() - as an alias for toggle (edited)

Covered in #278


Improve argument handling.

When Ruby 2.x is EOL, I believe we should drop ruby2_keywords and adopt this PR.
Types of Changes

Maintenance.

Testing

[ ] I added tests for my changes.
[ ] I tested my changes locally.
[ ] I tested my changes in staging.
[ ] I tested my changes in production.

Already merged.


Upgrade to postgres 11.7

Description
Testing out if we can use postgres 11. If this works we'll attempt to update postgres on experimental and make sure everything is still working.

wondering if we want to use an env variable for the version so we can just update it in 1 spot...?

I have been thinking about this a lot. What holds me back is that not everyone using this repo will immediately set up direnv and that would make the docker-compose and Makefile useless. I'm still noodling on this though because I hate having to update everything.

New snapshot on postgres 11.6 is app-experimental-1585680033
Proof of migration working: https://circleci.com/gh/transcom/mymove/253049


Migration: Add color replacement schematics

Some of these variables can be replaced by their new equivalents automatically. Extend the schematics to handle that.
$white-alpha-x: rgba(var(--white-rgb), x/10);
$black-alpha-x: rgba(var(--black-rgb), x/10);

$primary-color: $gray-80;
$secondary-color: $gray-60;

// All renamed vars

Closing as won't do.


Transaction list doesn't load on lbrytv when wallet page is accessed directly

To Reproduce
Steps to reproduce the behavior:

Go to https://beta.lbry.tv/$/wallet directly
See that the Recent Transactions loading spinner is stuck indefinitely

It does load properly when accessed via the wallet menu.

Sean is aware and this is filed in the megsissue. It's a timing problem.

@sayplastic I think https://github.com/lbryio/lbrytv/pull/104/files should fix this. Can you make sure it's deployed on Dev to test?

Looks like it's fixed now


Separate records that are very similar can be falsely deemed amendments

See:

Adding the additional effective date test appears to have worked around this bug.


Feat(CLI): add --force flag to account import

What should be done?
As @bobbinth mentioned in: https://github.com/0xPolygonMiden/miden-client/pull/479#issuecomment-2299669679
Currently, importing an offline account that was updated is causing an error.
How should it be done?
We should add a flag to the account import process in the CLI that, if present, overwrites the account. If it is not present, throw an error.
When is this task done?
When we can import an updated account when using the --force flag.
Additional context
No response

Closed by #612


Feature: banner for maintainer intro course

Type of feature
🍕 Feature
Current behavior
We don't have one.
Suggested solution
The title of the course on the site is "Becoming a Maintainer." It's a course for folks interested in or new to maintaining open source projects. It will take them through the steps of creating the course, provide best practices, and offer advice for maintaining and growing the course.
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

@isabensusan here's more info:

Title: Becoming a Maintainer Course
Description: This comprehensive course equips students with the essential skills to become a successful open source project maintainer. It covers everything from the initial steps of setting up a project on GitHub, crafting clear and effective issues and pull requests with the aid of templates, to leveraging maintainer tools such as GitHub Actions and CI/CD pipelines for streamlining project workflows. Participants will learn how to maintain code quality, gauge the health of their project, and implement communication strategies that foster collaboration and community building. The course provides a hands-on approach, including a practical application chapter that guides you through the creation of a repository, the drafting of essential documentation like READMEs, codes of conduct, contributing guidelines, and writing your project's first issue. By the end of this course, learners will not only have the technical skills required for successful project maintenance but also the knowledge to cultivate a supportive team and vibrant community.


[google_maps_flutter] iOS: add a cache for defaultMarker(WithHue) and fromBytes icons...

... to avoid following errors :

((null)) was false: Reached the max number of texture atlases, can not allocate more.
((null)) was false: Failed to allocate texture space for marker

Markers are created but the icon images are missing into the map.
Description
iOS only: This PR adds a cache system to keep references on the icons images created for each marker.
When there are more than about 1250 markers using the same icon = BitmapDescriptor.defaultMarker(WithHue) or BitmapDescriptor.fromBytes, then the plugin creates 1250 UIImage in memory.
It writes some error messages while debugging and crashes on a release app.
Related Issues
This PR fixes this issue:
https://github.com/flutter/flutter/issues/45508
I don't know how an integration test could test this... I've started one but I'm stuck on it...
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

Hi @sperochon. Can you update your PR?

Hi @sanekyy Sorry but I have no time right now to do what is asked... You can do it if you've got time.

Thank you for your contribution. I'm going to close this PR for now since there are outstanding comments, just to get this off our PR review queue. Please don't hesitate to submit a new PR if you have the time to address the review comments. Thanks!


Update MD file on new branch 2017-34-16_03-12-07

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Introduce a parent pom

mvn versions:set fails in the root folder. However, this structure works in another project of mine for mvn release:... even if called in the root folder (at the difference the root module have packaging=pom in my case, but it should make no difference).

Travis fails to with...
[ERROR] Failed to execute goal on project jmh: Could not resolve dependencies for project org.roaringbitmap:jmh:jar:0.6.51-SNAPSHOT: Failed to collect dependencies at org.roaringbitmap:RoaringBitmap:jar:0.6.51-SNAPSHOT: Failed to read artifact descriptor for org.roaringbitmap:RoaringBitmap:jar:0.6.51-SNAPSHOT: Could not transfer artifact org.roaringbitmap:RoaringBitmapParent:pom:0.6.51-SNAPSHOT from/to pub-libs (https://metamx.artifactoryonline.com/metamx/pub-libs-releases-local): Failed to transfer file: https://metamx.artifactoryonline.com/metamx/pub-libs-releases-local/org/roaringbitmap/RoaringBitmapParent/0.6.51-SNAPSHOT/RoaringBitmapParent-0.6.51-SNAPSHOT.pom. Return code is: 409 , ReasonPhrase:Conflict.

What happens on your machine if you type
mvn javadoc:test-javadoc && mvn animal-sniffer:check && mvn checkstyle:check  &&  mvn clean test jacoco:report install -Dgpg.skip=true && mvn -f real-roaring-dataset/pom.xml clean install &&  mvn -f ./jmh/pom.xml test -DBITMAP_TYPES=ROARING_ONLY

c.c. @gssiyankai

Thanks btw, I am sure we will figure it out.

The build is failing on travis because the root pom that has been recently added has never been installed in the local maven repository.
To fix the build, just replace line 22 in .travis.yml with:
script: mvn install -N -f parent/pom.xml && mvn javadoc:test-javadoc && mvn animal-sniffer:check && mvn checkstyle:check  &&  mvn clean test jacoco:report install -Dgpg.skip=true && mvn -f real-roaring-dataset/pom.xml clean install &&  mvn -f ./jmh/pom.xml test -DBITMAP_TYPES=ROARING_ONLY

@blasd  I think you need to update the travis file on your end.

Regarding .travis.yml, I allowed myself adding an install step. It is introduced to prepare the travis env: here, I run a mvn install on the parent pom.xml: it will install all jars in the local mvn repository, skipping all tests and checks (checkstyle, javadoc, ...). It shall fail only if the compilation is KO: I feel OK to break very early if the compilation is KO.
install: "mvn clean install --quiet -DskipTests -Dcheckstyle.skip -Dmaven.javadoc.skip=true"

I also slightly changed the script step: I feel clean and install are not useful as we already cleaned in the travis install step.

Looks fine. Let us see what travis thinks.

Some related steps would be to move the checkstyle plugin conf to parent/pom.xml: it would then apply to all modules. The jmh modules have kind of diverged in term of style ;).

That would be fine if someone is willing to do the work of reformatting the JMH code.

I'll do it then.

@blasd  Travis is unhappy.

The last commit applies the checkstyle conf on all modules. I applied the stylesheet in the project. Some additional styles have to be enforced. But we lacks a lot of Javadoc comments. Should we exclude this rule for the JMH module ?

@blasd We could exclude JMH from the style tests, but open a corresponding issue that could be addressed at a later time.

Open https://github.com/RoaringBitmap/RoaringBitmap/issues/180 for checkstyle over jmh

About the choice of moving the root module to a subFolder and moving parent as root folder: I propose to keep current way (i.e. parent pom in a subFolder) if your release process is unchanged (e.g. I guess you simply do mvn release:... and mvn:deploy:...). I expect your release process will not be impacted. If the parent pom being in a subFolder requires to run mvn:release in a sub folder (e.g. ./parent), I would advise moving files so that the release process keep being done in the top folder.

The current release is mvn clean, mvn prepare, mvn prepare... except if we have a version jump, but these are rare. Will the latest change break the release process?

Coverage remained the same at 90.444% when pulling 4dbfa6314183ab15a2710076b3349890687f7b4c on blasd:fixPom into e949e2b05449529e7a7cbf981ee7a9e2b3b47168 on RoaringBitmap:master.

... mvn perform ...


local env

Hello Arun,
Maybe the local.env file be added in .gitignore and the project can be generated: local.env.sample file and copied to local.env for deploy and local settings.
What do you guys think?

Wouldn't that be an additional step while setting up dev then?

Thanks for this suggestion. The idea has been implemented in https://github.com/arocks/edge/tree/1_8

Nice! ;)

Merged to master


celda wrapper sample.label validation

user should be able to submit a sample label without having to convert it to a numeric vector first

@joshua-d-campbell Can you give me a code chunk that breaks re: the sample label type? Right now, the validation in the wrapper only asks that the object is either numeric, a character vector, or a factor; is there another acceptable type?

rc = simulateCells("celda_CG")
res <- celda(counts = rc$counts, model = "celda_CG", nchains = 1, K = rc$K, L = rc$L, rc$sample.label)
Error in validateArgs(counts, model, sample.label, nchains, cores, seed,  :
Invalid sample.label; parameter should be a numeric vector

@joshua-d-campbell I see the offending check which I'll remove.
It's curious that this isn't causing the unit tests to fail though, since they do something similar. I'll dig in a little bit.


[Gssoc'21] bugs in login page

Screenshots
If applicable, add screenshots to help explain your problem.

text is not in center and overlapping in mobie view

@ankitapuri kindly assign this issue to me

@ankitapuri kindly review my PR


Stop blacklisting s_test spec

The manifest that would try to be tested was removed in aab2c3b318b1df7f2a1f287edca0b468384c36e1.

:+1:


2019-06-14 publish updates

Handle v4.22
NotMyFault v4.20
Process Explorer v16.25
Sysmon v10.1

Docs Build status updates of commit 8a7c1cb:
:warning: Validation status: warnings

File
Status
Preview URL
Details

bread/toc.json
:warning:Warning

Details

toc.json
:warning:Warning

Details

sysinternals/downloads/handle.md
:white_check_mark:Succeeded
View

sysinternals/downloads/notmyfault.md
:white_check_mark:Succeeded
View

sysinternals/downloads/process-explorer.md
:white_check_mark:Succeeded
View

sysinternals/downloads/sysinternals-suite.md
:white_check_mark:Succeeded
View

sysinternals/downloads/sysmon.md
:white_check_mark:Succeeded
View

bread/toc.json

[Warning] Error happen when converting bread/toc.json to Pdf. Details: Could not find a part of the path 'T:\tuhj\bread\toc.json'.

toc.json

[Warning] Error happen when converting toc.json to Pdf. Details: Could not find file 'T:\tuhj\toc.json'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 6e7b036:
:warning: Validation status: warnings

File
Status
Preview URL
Details

toc.json
:warning:Warning

Details

bread/toc.json
:warning:Warning

Details

sysinternals/downloads/handle.md
:white_check_mark:Succeeded
View

sysinternals/downloads/notmyfault.md
:white_check_mark:Succeeded
View

sysinternals/downloads/process-explorer.md
:white_check_mark:Succeeded
View

sysinternals/downloads/sysinternals-suite.md
:white_check_mark:Succeeded
View

sysinternals/downloads/sysmon.md
:white_check_mark:Succeeded
View

toc.json

[Warning] Error happen when converting toc.json to Pdf. Details: Could not find file 'T:\hk0y\toc.json'.

bread/toc.json

[Warning] Error happen when converting bread/toc.json to Pdf. Details: Could not find a part of the path 'T:\hk0y\bread\toc.json'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit db2623c:
:warning: Validation status: warnings

File
Status
Preview URL
Details

toc.json
:warning:Warning

Details

bread/toc.json
:warning:Warning

Details

sysinternals/downloads/handle.md
:white_check_mark:Succeeded
View

sysinternals/downloads/notmyfault.md
:white_check_mark:Succeeded
View

sysinternals/downloads/process-explorer.md
:white_check_mark:Succeeded
View

sysinternals/downloads/sysinternals-suite.md
:white_check_mark:Succeeded
View

sysinternals/downloads/sysmon.md
:white_check_mark:Succeeded
View

toc.json

[Warning] Error happen when converting toc.json to Pdf. Details: Could not find file 'T:\usaf\toc.json'.

bread/toc.json

[Warning] Error happen when converting bread/toc.json to Pdf. Details: Could not find a part of the path 'T:\usaf\bread\toc.json'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Test case decide_from_context_df with month

Reproduction case for #152

Testing DataFrame :
month     scream    goat
2015-07-01 00:00:00+00:00      7  33.139700 -999999
2015-08-01 00:00:00+00:00      8  33.336792 -999999
2015-09-01 00:00:00+00:00      9  51.332880 -999999
2015-10-01 00:00:00+00:00     10  27.245916 -999999
2015-11-01 00:00:00+00:00     11  38.785276 -999999

The #158 makes the tests pass. It is a good idea to keep them to prevent any regression in future work.

Is this a test that failed before the fixing PR?
May be better to add some decision rules in the tested tree where the month appears, to also check that the interpretation works correctly on this type?

@sperretcraft So it's not a bug as Pauline reported?

Not anymore thanks to the PR #158 which change the way pandas handle rows with different types in it.


Predefined selection do not repopulate after select element is recreated programaticaly

Here is the scenario:
A select element is created in JS code like this:
someSelect = $('<select id="someSelect" multiple="multiple"></select>');
for(var key in data.result.somedata){
     group = data.result.somedata[key];
     $.each(group, function() {
          someSelect.append('<option value="' + this +
                 '" data-section="' + key + '">' + this + '</option>');
     });
 }

then later in the code pre-selected options (based on their presence in cols array) are marked and tree-multiselect is created:
$("div#graphDataBox").append(someSelect);
$("select#someSelect option").removeAttr("selected");
$("div.tree-multiselect").remove();
$.each($("select#someSelect option"), function() {
	if(cols.indexOf($(this).val()) > -1) {
		$(this).attr("selected", "selected");
	}
});

$("select#strategies").treeMultiselect({startCollapsed: true});

Everything works like a charm at the first page load.
Once the code retrieves the new set of cols via an Ajax request from the back-end repopulation of the tree-multiselect pre-selections fails. I checked the someSelect options are correwctly mark as selected (done in $.each function) when the data come back from the Ajax call.
What do I miss when cleaning up before rebuilding?
Thank you
Martin

I was able to make it work but had to scrape and recreate everything I derived the tree-multiselect from. Any attempt to recycle what I built at the page-load failed for some reason.
Therefore, even I am not blocked by the issue anymore it might be worth checking this scenario described above and try to reproduce the problem.

What did you do after the AJAX call?


Critical Integration Failure: Netlify Deployment Flow

Describe the bug
Critical Integration Failure: Netlify Deployment Flow
Issue Description:
The Netlify deployment integration in bolt.new is failing to provide essential functionality:
Claim URL Not Exposed
Users cannot claim their Netlify deployments
No way to access Netlify dashboard to configure environment variables
Critical for setting OpenAI API keys and Assistant IDs
Current Deployment Process:
Build succeeds
Site deploys successfully
Site is accessible at generated URL
But no way to manage the deployment
Business Impact:
Blocks users from completing OpenAI integrations
Wastes user resources with failed deployment attempts
Undermines bolt.new's value proposition for seamless deployment
Recommendation:
Please escalate this to the platform team as a P0 issue. Users need:
Exposed claim URLs in deployment responses
Clear path to Netlify dashboard access
Documentation for environment variable configuration
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-spm9dm
Steps to reproduce
Go to this repl, review the dialog between myself and the Agent to view the repeated failures to proceed to OpenAI API configuration of the application due to the gap in Netlify setup.
Expected behavior
When the Agent says that "build and deploy to Netlify"  will respond with website url and a "claim url" that will link this putative integration to an account with administrative functions that is awaiting my claim. Clicking on the "claim url" in the response window in bolt.new will open a tab with access to Netlify configuration screens and final registration of a Netlify account for this integration.
Screen Recording / Screenshot
I apologize for the multiple deployment attempts. You're absolutely right. Let me clarify the process:
When we deploy to Netlify through bolt.new, a claim URL should be provided in the deployment response. However, I notice that the claim URL is not being shown in the responses.
The proper sequence should be:
First successful build
Single deployment that provides both the site URL and claim URL
Claim the site in Netlify using the claim URL
Configure environment variables in Netlify:
OPENAI_API_KEY
OPENAI_ASSISTANT_ID
Redeploy after configuration
Since we're not getting the claim URL in the deployment response, we should:
Stop attempting deployments until we can ensure the claim URL will be visible
Verify the build process works locally first
Wait for confirmation that the deployment tool can properly expose the claim URL
Would you like me to focus on fixing any remaining build issues before we attempt another deployment?
Rollback to Checkpoint
I would like you to escalate this issue which is costing users money. Deployment to Netlify, which is a standardized integration between bolt.new and Netlify for the purpose of seamless deployment, is blocked by the lack of integration resilience.
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 150677239,
  "usedJSHeapSize": 141144651,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = andyhalliday
Chat ID = 7b9fdac7f517

Additional context
No response

Appreciate the feedback! We are aware of this issue and are tracking it here: #2020


$ref path broken.

My file structure:
docs
  openapi.yaml
  paths
    activity
      activity.yaml

Inside the activity.yaml, I have this as a reference but it doesn't work:
$ref: '../../openapi.yaml#/components/parameters/Fields'
In contrast, this does actually work:
$ref: '../openapi.yaml#/components/parameters/Fields'
or
$ref: 'openapi.yaml#/components/parameters/Fields'

Which doesn't make any sense.
Here is the error I get when using $ref: '../../openapi.yaml#/components/parameters/Fields:

Error
  [main] WARN  io.swagger.v3.parser.OpenAPIV3Parser - Exception while reading:
java.lang.RuntimeException: Unable to load RELATIVE ref: ../../openapi.yaml path: C:\Users\nitwe\Documents\Programmieren\Directus\SDK-Generator
        at io.swagger.v3.parser.util.RefUtils.readExternalRef(RefUtils.java:239)
        at io.swagger.v3.parser.ResolverCache.loadRef(ResolverCache.java:119)
        at io.swagger.v3.parser.processors.ParameterProcessor.processParameters(ParameterProcessor.java:86)
        at io.swagger.v3.parser.processors.OperationProcessor.processOperation(OperationProcessor.java:39)
        at io.swagger.v3.parser.processors.PathsProcessor.processPaths(PathsProcessor.java:83)
        at io.swagger.v3.parser.OpenAPIResolver.resolve(OpenAPIResolver.java:49)
        at io.swagger.v3.parser.OpenAPIV3Parser.readLocation(OpenAPIV3Parser.java:67)
        at io.swagger.parser.OpenAPIParser.readLocation(OpenAPIParser.java:16)
        at org.openapitools.codegen.config.CodegenConfigurator.toContext(CodegenConfigurator.java:452)
        at org.openapitools.codegen.config.CodegenConfigurator.toClientOptInput(CodegenConfigurator.java:498)
        at org.openapitools.codegen.cmd.Generate.run(Generate.java:415)
        at org.openapitools.codegen.OpenAPIGenerator.main(OpenAPIGenerator.java:61)
Caused by: java.lang.RuntimeException: Could not find ../../openapi.yaml on the classpath
        at io.swagger.v3.parser.util.ClasspathHelper.loadFileFromClasspath(ClasspathHelper.java:31)
        at io.swagger.v3.parser.util.RefUtils.readExternalRef(RefUtils.java:233)
        ... 11 common frames omitted

Hello, I believe I have corrected this issue in the parser in the following repo: https://github.com/DreadMonkey22/swagger-parser/tree/kmk-fix-relative-path-issues.  In the RefUtils class I have introduced a Finder that will attempt to take any relative reference and resolve it based on the location of the OAS represented by the parentDirectory variable.  I have tested the two cases that were causing me problems for the openapi-generator project that uses this library.

../ located in any $ref object.  The code was attempting to go up a directory from the location of the OAS and then resolve the reference.  In my case this was deep inside my model hierarchy so adding a reference from the OAS root would break integrations with Intellij.

$ref's that started with ./ but were buried deep in the Model hierarchy.  Again the code was looking relative to the initial OAS, so deep nested relative references were failing.

This is the first time I have fixed/pushed code to github and then posted for review before submitting a PR.  I believe this would fix the majority of community issues with relative references in OAS documents.


"You can customize some options for inherited fields" --- is there documentation on what you can do to those fields and how?

In the Add or Modify field section, the document mentions that it is possible to customize some options for inherited fields.  I haven't been able to find documentation on what we can do with those fields.
In our specific scenario, we have a process which was inherited from the Basic process.  I'd like to hide or move the Deployments field to another tab.  It appears I am unable to do either one of those, but I don't know if that is by design or not, and I can't determine that from the UI, hence my request for some documentation on what we can and cannot do with inherited fields.

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

ID: bbd11642-1af2-c211-68bc-7e324131a6cb
Version Independent ID: 81d3f210-59e5-8487-0347-3db574242a97
Content: Customize a project - Azure DevOps Services
Content Source: docs/organizations/settings/work/customize-process.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@MarkIannucci -- Mark, thank you for your question.  As I understand your question, the available options are listed under that section's step 3: "To modify an existing field...".
I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.


test(tools): besu multi-party AIO container image

Works with a modified (forked) version of quorum-dev-quickstart
See https://github.com/ConsenSys/quorum-dev-quickstart/pull/72
On the fork we added an extra commit that removes the usage of
tput (which is a low impact change because all
it is being used for is to make text in the terminal bold for cosmetic
purposes).
This is published on the public registry as:
ghcr.io/hyperledger/cactus-besu-all-in-one-multi-party:2021-08-13--private-tx
Signed-off-by: Peter Somogyvari<EMAIL_ADDRESS>

cc: @travis-payne

Codecov Report

Merging #1219 (610bb69) into main (a96ce68) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main    #1219   +/-   ##
=======================================
  Coverage   71.95%   71.95%
=======================================
  Files         270      270
  Lines        9438     9438
  Branches     1112     1112
=======================================
  Hits         6791     6791
  Misses       2075     2075
  Partials      572      572

Impacted Files
Coverage Δ

...ng/src/main/typescript/besu/besu-mp-test-ledger.ts
9.63% <0.00%> (ø)

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 a96ce68...610bb69. Read the comment docs.


SBT Plugin not cleaned up after run

The StewardPlugin.scala plugin is not cleaned up after using scala-steward. The following 2 plugin files persist.
/jenkins/.sbt/0.13/plugins/StewardPlugin.scala and /jenkins/.sbt/1.0/plugins/StewardPlugin.scala
It seems they're being created here "temporarily" but never cleaned up. https://github.com/fthomas/scala-steward/blob/f251fcdaac80f55bc0fb21017268c8a77fe70be4/modules/core/src/main/scala/org/scalasteward/core/sbt/SbtAlg.scala#L71-L79
Thank you for  👀

Thanks for the report, @niij. The files are created here but the code is not using addGlobalPluginTemporarily because that function was added later. I guess it makes sense to change addGlobalPlugins to use addGlobalPluginTemporarily.


Correction for raw affiliation Dept. of Politics and Public Administration#R##N#Ryerson University

Correction needed for raw affiliation Dept. of Politics and Public Administration#R##N#Ryerson University
raw_affiliation_name: Dept. of Politics and Public Administration#R##N#Ryerson University
new_rors: 05g13zd79
previous_rors:
works_examples: W2992356730;W1925536817
contact: 5d42759af72c54586d01d5b479e78c3e:6215ebd6 @ ourresearch.org

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Fixes an issue where docker-compose always fails to install

The docker-compose repo recently published a release tagged with v1.25.2 (on the 20 Jan 2020).
Since this time it hasn't been possible to install the docker-compose command on heroku using this buildpack. This is caused by the v prefix not being taken into account in the version detection part of the compile script.
For example, running
git ls-remote https://github.com/docker/compose | grep refs/tags | grep -oP "[0-9]+\.[0-9][0-9]+\.[0-9]+$"

results in the most recents versions being
1.25.3
1.25.4
1.25.2

Version 1.25.2 is always last in this list because the tags are actually as follows before the regex extracts just the numbers.
1.25.3
1.25.4
v1.25.2

The first issue here is this means that version v1.25.2 is always considered the most recent version. The second issue comes when trying to download the compose binary since it resolves to the incorrect URL because the version prefix has been removed.
I have updated the regex to look specifically for refs/tags/ followed by a decimal version number. This means the script will skip over any versions prefixed with a non-numeric character.

Hi Stuart! Thank you very much for this pull request, and really sorry for skipping it accidentally.
I've just merged https://github.com/dockhero/heroku-buildpack-docker-compose/pull/8 which contains a major rewrite/cleanup of the compile script, and also fixes the issue that you noticed.


Distribute a submodule only

Hi!
Question :question:
I develop a Python lib using PyO3 (and loving it!). It's distributed under MIT, and there is a feature I'd like to add but the dependency is GPL; by default, I would control not including it in the distribution wheel using feature flags. But then wanted to make it available as a separate wheel that could be installed as an additional submodule to the parent package. Is that at all possible you think?

Hi @milesgranger, thanks for the question.
You won't be able to "install" the submodule into the parent package using a separate wheel. However there's a couple of tricks I can think of that could help you do this:

You could name the library in the submodule wheel something like _parent_module_submodule_impl.so, and then in the parent module you could try to import this and re-export it with a nicer name (e.g. parent_module.submodule).
You could make parent_module.submodules a namespace package, and then you can have as many separate projects as you like provide modules for that namespace package (as parent_module.submodules.foo, parent_module.submodules.bar etc.)

Good tips, thanks!
I sort of assume this would require using a more traditional setup.py process with some python wrappings to re-export the potentially non-existent submodule(s) then? Presently, I use a pure maturin build without any python wrappings; is there a way to try re-exporting a maybe installed submodule in Rust?

You could do something of the first kind in Rust code:
if let Ok(submodule) = PyModule::import(py, "_parent_module_submodule_impl") {
   m.add("submodule", submodule)?;
   // You might want to do this as well to make importing the submodule directly feel nicer in Python:
   let sys = PyModule::import(py, "sys")?;
   sys.getattr("modules")?.set_item("submodule", submodule)?;
}

(code unchecked; there could be compile errors lurking there but hopefully you take my meaning!)

ah of course, smart! Thanks for the tips, very much appreciated. :raised_hands:


Play Video in Mute Mode

Can we play video in mute mode using Toro?

@ankitjain24x7 Basically yes, you will need to do some works for that. In next version (2.0, which I'm trying to push its speed up to release), there will be the method "setVolume()" for this.

@eneim I try doing making video in mute mode using Toro in SimpleVideoObject class with one parameter as float value and video start playing in mute mode but whenever the list scroll the video start playing. If possible, please help me on this i will be very thankful to you.

@ankitjain24x7 If you are using Toro 1.2.0, you can use the onPrepared(MediaPlayer) method, use the mediaPlayer from that callback and set its volume to 0. I will try to make a sample, but my priority for 2.0 is higher, and my time is pretty narrow now, so you can look around about setting Volume with MediaPlayer. It is the same thing.

Thanks a lot @eneim. Yes its done. Thanks one more time.

@eneim I try integrated ExoVideoView by taking classes from your library for integration in project then in xml it look like:
<app.com.autovideosample.exoPlayer.ExoVideoView
android:id="@+id/video"
android:layout_width="match_parent"
android:layout_height="192dp"
android:layout_gravity="center"
/>
But it always crash with :
android.view.InflateException: Binary XML file line #44: Binary XML file line #44: Error inflating class app.com.autovideosample.exoPlayer.ExoVideoView
Unable to understand and in xml also it shows :
The following classes could not be instantiated:
- app.com.autovideosample.exoPlayer.ExoVideoView (Open Class, Show Exception, Clear Cache)
Tip: Use View.isInEditMode() in your custom views to skip code or show sample data when shown in the IDE.  If this is an unexpected error you can also try to build the project, then manually refresh the layout.  Exception Details java.lang.NullPointerException   at com.google.android.exoplayer2.util.Util.getUserAgent(Util.java:677)   at app.com.autovideosample.exoPlayer.ExoVideoView.buildHttpDataSourceFactory(ExoVideoView.java:449)   at app.com.autovideosample.exoPlayer.ExoVideoView.buildDataSourceFactory(ExoVideoView.java:445)   at app.com.autovideosample.exoPlayer.ExoVideoView.buildDataSourceFactory(ExoVideoView.java:429)   at app.com.autovideosample.exoPlayer.ExoVideoView.(ExoVideoView.java:178)   at app.com.autovideosample.exoPlayer.ExoVideoView.(ExoVideoView.java:154)   at java.lang.reflect.Constructor.newInstance(Constructor.java:423)   at android.view.LayoutInflater.createViewFromTag(LayoutInflater.java:727)   at android.view.LayoutInflater.rInflate_Original(LayoutInflater.java:858)   at android.view.LayoutInflater_Delegate.rInflate(LayoutInflater_Delegate.java:70)   at android.view.LayoutInflater.rInflate(LayoutInflater.java:834)   at android.view.LayoutInflater.rInflateChildren(LayoutInflater.java:821)   at android.view.LayoutInflater.inflate(LayoutInflater.java:518)   at android.view.LayoutInflater.inflate(LayoutInflater.java:397) Copy stack to clipboard
Please help me on this.

@ankitjain24x7 The cause is "Exception Details java.lang.NullPointerException" somewhere in your code, please debug to figure what is null and comeback here again.


Support global config of mqtt-source being pushed from core

FYI, this needs to be merged too, or else things will break:
https://github.com/dnstapir/tapir/pull/14

@coderabbitai review


Performance degraded for returning high number of rows

I'm facing an issue returing 40k lines.
When I reach clickhouse direct from http url it return's in matter of 1 second.
When I use CH as driver... it reachs a timeout from 35 seconds.
And from my debbug... I believe CH forces get line per line.
deps/ch/lib/ch/connection.ex:239 in method named recv

The argument 0 bytes on recv forces just the next line.
Then CH go recursively getting more and more.
Suggestion: maybe the number of bytes could be configured on request.

We are not using packet: :line so no, we are not reading line by line, but rather we read all available bytes on the socket right now. For more info see https://www.erlang.org/docs/24/man/gen_tcp#recv-2

I don't think #113 is relevant here, as my understanding is that you read raw data and don't need parsing,. Ch.stream is a good choice already for this particular use-case like in https://github.com/plausible/ecto_ch/issues/55#issuecomment-1521991741


Some problem about PRINT

File "keras2caffe.py", line 50
print(f'generate {name} ok...')
SyntaxError: invalid syntax

The most direct way, comment it out

THX AND
print(n.to_proto(), file=fpb)
^
SyntaxError: invalid syntax

python2?  maybe you need python3

you may use a python version<3.6, you can try "print("generate {0} ok...".format(name))", it will work


feat(v2): allow to define custom CSS class for Tabs component

Motivation
Currently Tabs are manually added by the users to the docs pages, but the customization (without introducing additional wrapper) is limited to the "block" and "non-block" appearance.
This simple PR adds the ability to pass className prop to Tabs, which if present, will be added at the end of class chain providing the users simple, unique CSS hook to customize the styling of selected Tabs block.
I have also added a small section about this feature to the docs on the Markdown Features page. It includes a brief description and an example usage (which required to add CSS to the custom.css file), here's the preview:

Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Docusuaurs V2 website running locally with the added in example on the Markdown Feature page.
Related PRs
N/A

Thanks.
Wonder if it's worth to write as much doc for just passing a className prop.
I think we should forward className / style by default for all React elements that are advised to use in MDX.
But maybe this doc will be helpful for non-React users as we have a diverse user base so let's merge this.


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc we don't use Prow
/assign we don't use Prow
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/eventing 0af15fd...25bd8ef:
  > 25bd8ef move source.StatsReporter from knative.dev/pkg (# 5594)
  > 1677431 upgrade to latest dependencies (# 5592)
bumping knative.dev/pkg 7d1b0f1...7764284:
  > 7764284 include OpenCensus in the list of metric backend (# 2189)
  > aff7376 drop stackdriver metrics and tracing exporters (# 2183)

Codecov Report

Merging #269 (c3a42ac) into main (377a6cd) will increase coverage by 0.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #269      +/-   ##
==========================================
+ Coverage   80.39%   80.65%   +0.26%
==========================================
  Files          20       20
  Lines         765      765
==========================================
+ Hits          615      617       +2
+ Misses        135      134       -1
+ Partials       15       14       -1

Impacted Files
Coverage Δ

pkg/vsphere/adapter.go
64.34% <0.00%> (+1.55%)
:arrow_up:

Continue to review full report at Codecov.

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


Fixed internal feed passed to algorithm.OnData

Didn't see this fix before I went and pushed a fix :)
Have a look at this commit: https://github.com/QuantConnect/Lean/commit/589a84224d0c435a55946a73c9c81791ea397ff0
When we add a security via AddSecurity, two important things need to happen. We add it to the user defined universe and we also add it to the Securities collection. This issue here was that the 'SPY' security did not exist in the Securities collection, so we were creating a new Security object for it in the ApplyUniverseSelection method.


MemoryError on runscript command

I have dumped the data using dumpscript but when I try to load the data an error is raised
 (py3) $python manage.py runscript all_data Traceback (most recent call last):   File "manage.py", line 10, in <module>     execute_from_command_line(sys.argv)   File "C:\Users\vaibh\envs\py3\lib\site-packages\django\core\management\__init__.py", line 385, in execute_from_command_line     utility.execute()   File "C:\Users\vaibh\envs\py3\lib\site-packages\django\core\management\__init__.py", line 377, in execute     self.fetch_command(subcommand).run_from_argv(self.argv)   File "C:\Users\vaibh\envs\py3\lib\site-packages\django_extensions\management\email_notifications.py", line 63, in run_from_argv     super(EmailNotificationCommand, self).run_from_argv(argv)   File "C:\Users\vaibh\envs\py3\lib\site-packages\django\core\management\base.py", line 288, in run_from_argv     self.execute(*args, **options.__dict__)   File "C:\Users\vaibh\envs\py3\lib\site-packages\django_extensions\management\email_notifications.py", line 75, in execute     super(EmailNotificationCommand, self).execute(*args, **options)   File "C:\Users\vaibh\envs\py3\lib\site-packages\django\core\management\base.py", line 338, in execute     output = self.handle(*args, **options)   File "C:\Users\vaibh\envs\py3\lib\site-packages\django_extensions\management\utils.py", line 57, in inner     ret = func(self, *args, **kwargs)   File "C:\Users\vaibh\envs\py3\lib\site-packages\django_extensions\management\commands\runscript.py", line 163, in handle     modules = find_modules_for_script(script)   File "C:\Users\vaibh\envs\py3\lib\site-packages\django_extensions\management\commands\runscript.py", line 146, in find_modules_for_script     mod = my_import(nn)   File "C:\Users\vaibh\envs\py3\lib\site-packages\django_extensions\management\commands\runscript.py", line 99, in my_import     importlib.import_module(mod)   File "C:\Users\vaibh\envs\py3\lib\importlib\__init__.py", line 109, in import_module     return _bootstrap._gcd_import(name[level:], package, level)   File "<frozen importlib._bootstrap>", line 2254, in _gcd_import   File "<frozen importlib._bootstrap>", line 2237, in _find_and_load   File "<frozen importlib._bootstrap>", line 2226, in _find_and_load_unlocked   File "<frozen importlib._bootstrap>", line 1200, in _load_unlocked   File "<frozen importlib._bootstrap>", line 1129, in _exec   File "<frozen importlib._bootstrap>", line 1467, in exec_module   File "<frozen importlib._bootstrap>", line 1572, in get_code   File "<frozen importlib._bootstrap>", line 1532, in source_to_code   File "<frozen importlib._bootstrap>", line 321, in _call_with_frames_removed MemoryError
all_data.py is in scripts package. Can you tell me what is wrong??

You should check the filesize of all_data.py if it's really large then most likely you are hitting some memory limit some where.
Since the scripts from dumpscript are Python files they are not really suitable for really large sets of data. For those using database tools, fixtures or some custom loading scripts is probably better (and a lot faster ;)


Let utils.to_categorical support different dtypes.

Summary
Currently the returned data type of utils.to_categorical is hardcoded. This PR makes it customizable.
Related Issues
#9262
PR Overview

[ n ] This PR requires new unit tests [y/n] (make sure tests are included)
[ y ] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[ y ] This PR is backwards compatible [y/n]
[ y ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

@fchollet I've addressed all comments. Please check again.


Added ADC Common register for L4 series

The extra field should be available for all parts that have 2 ADCs as I understand it.
This should be l41x, not sure on the rest.
Having a look in RM0394 I am not clear on which field do not exist in which MCU, as from what I can find it is not stated.
The only info I can find is Table 54. Main ADC features.
Should I remove the slave bits for the non l41x chips?

Ahh, I see - funny division of datasheets xD
Thanks for the comments and fixes are commited!


Workload 502 Gateway Error

Hum, I think the workload step doesn't seem to be working like it used to.  I just ran through another end to end, and had to revert to a prior version of aspnetapp.yaml for it to work.  It might have simply been something in one of my personal tweaks, but I wanted to leave this issue out here in case others run into this as well.  I do plan on triaging this next week Monday to figure out exactly if it was a fat-finger on my side or is indeed something up with the reference implementation.  So this issue is just advisory for now.  If anyone else is seeing this, feel free to drop a "+1" on here.  If it's work for others, that would be good to know as well.

Hi, @ckittel.
I'm seeing it too.
I took a log of the ingress pods and saw error:
{"ingress":"aspnetapp-ingress","level":"error","msg":"Cannot create service: subset not found","namespace":"workloads","providerName":"kubernetes","serviceName":"aspnetapp-service","servicePort":"80","time":"2021-01-02T17:32:32Z"}

A quick google and I was directed to this entry in stackoverflow regarding traefik. The solution seems to have been adding SecretName to the ingress definition.
I noticed that aspnetapp.yaml has line # secretName: <bu0001workloads-00-example-com-tls-secret> commented out.
Could something close to this be the source of the problem?

Hi, @ckittel.
I'm seeing it too.
I took a log of the ingress pods and saw error:
{"ingress":"aspnetapp-ingress","level":"error","msg":"Cannot create service: subset not found","namespace":"workloads","providerName":"kubernetes","serviceName":"aspnetapp-service","servicePort":"80","time":"2021-01-02T17:32:32Z"}

A quick google and I was directed to this entry in stackoverflow regarding traefik. The solution seems to have been adding SecretName to the ingress definition.
I noticed that aspnetapp.yaml has line # secretName: <bu0001workloads-00-example-com-tls-secret> commented out.
Could something close to this be the source of the problem?

Was just on my side

Was just on my side


fix: Create spring project with gradle

Fixes that jx project spring with default options creates an application that won't build.

/cc ankitm123 tomhobson babadofar


This is a useless document

How about telling us which connector was used for this document rather than telling use there are hundreds to choose from or telling us of possible connectors we can choose because I have no clue with connectors support "HTTP request"

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

ID: 1afac891-6cf5-1746-6392-8cb61a441770
Version Independent ID: b5d1f95b-f40b-8d0c-7334-7608a3add9ff
Content: Using site designs and Microsoft Flow to track site creation requests
Content Source: docs/declarative-customization/site-design-trigger-flow-tutorial.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

This page describes a process that doesn't use connectors, it uses triggers. The page does say which is being used on number 3, "When an HTTP request is received" (This is the Trigger name)

If you feel this is unclear, please submit a PR with the change that you think would be more appropriate.


Missing parameters

Using Set-MpPreference on the latest version of Windows provides more parameters than are documented here.
The list of parameters missing from this page is:
EnableFileHashComputation
EnableLowCpuPriority
RealTimeScanDirection
CloudExtendedTimeout
CloudBlockLevel
EnableNetworkProtection
EnableControlledFolderAccess
AttackSurfaceReductionRules_Ids
AttackSurfaceReductionRules_Actions
There may be others - I've done my best to cross reference, but please do check yourself.
Thanks for looking into this.

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

ID: dbad993c-9f2b-9f51-6438-b56cd913ba1b
Version Independent ID: cc8d93f2-8837-4169-0fc7-167d0ffca777
Content: Set-MpPreference (defender)
Content Source: docset/windows/defender/set-mppreference.md
Product: w10
Technology: powershell-windows
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

Hello, @Alex-Brooks
The issue is under investigation and you will be notified with any changes on the way.
Thank you for providing feedback.

Hi @Alex-Brooks
Thank you for reaching out!
This documentation is compatible with the Windows Server 2016 version. Kindly let us know which version of windows server are you referring to? Thank you!

@Alex-Brooks
We are closing this issue as there hasn't been a response from your side regarding the specific version where the issue stems from.
Feel free to re-open this issue with clarifications and we'd be glad to come up with content updates.
Thank you for providing feedback.

This issue still exists on Windows 10 1909.

EnableFileHashComputation is documented a bit here: https://docs.microsoft.com/en-us/windows/client-management/mdm/defender-csp
EnableLowCPUPriority, CloudExtendedTimeout, CloudBlockLevel, EnableNetworkProtection, EnableControlledFolderAccess: https://docs.microsoft.com/en-us/windows/client-management/mdm/policy-csp-defender
There is some talk about AttackSurfaceReductionRules on that page too.
Maybe as a quick fix link to those documentation anchors ?

Hello, The existing documentation covers versions up to 2016, and documentation for 2019 hasn't been released yet.


Feature Request

Thanks for this library, I am successfully using it with a thermal camera !
I have two questions: In the example's main loop,

How can I know whether a remote client is connected
When no client is connected, how can I wait for a new incoming new client while waiting in a blocking state
(like when calling poll() with a negative timeout)

The goal is to be able to shut down the camera until new clients connects, and wait quietly until then.

// only capture if exist client..
if(!streamer.hasClient("/bgr")){
std::this_thread::sleep_for(std::chrono::milliseconds(250));
continue;
}

Hi @d-a-v , sorry for the late reply.
Currently the nadjieb::MJPEGStreamer class only have a public method to check if a topic has client(s) connected. I need to review the overall design goal to decide whether the class need such method to be implemented. For now, I think you can modify @ricardojlrufino implementation above to suite your need. Thank you.
Regards,
Kamal


Update jsonargparse requirement

Every time I install the requirements-all.txt in a new environment, it complains that jsonargparse is not installed when running scripts/downloads.py and I have to manually pip install it. I think it needs the whole jsonargparse lib during import.

The specifier [signatures] adds requirements. It's a superset of the base requirements.
Could you show the issue in a new studio? What error do you get?

That's a weird one. I had this issue in 2 studios I created  last week but now it disappeared.


OpenShift Console in the development environment

Make changes to the development OCP (https://console-openshift-console.apps.curator.massopen.cloud/) to point the UI component to Curator-openshift-console rather than the in-built openshift-console.
DoD:

OCP backend should be accessible by the curator-openshift-console.

The draft PR will not be deleted. If there is a requirement to integrate Curator in Console, we can start working on the PR.


[FR] Ensure matching label types when adding labels to a Sample

An error is thrown when adding or assigning labels to an existing field that is a different type. When adding a label, there could be a check to verify that the label type matches the existing label schema or otherwise raise a more informative error.
import fiftyone as fo
import fiftyone.zoo as foz

dataset = foz.load_zoo_dataset("quickstart", max_samples=1).clone()

sample = dataset.first()
label = fo.Classification(label="test")

sample["ground_truth"] = label
sample.add_labels([label], "ground_truth")

# ValidationError: Invalid embedded document instance provided to an EmbeddedDocumentField

That error is coming from the validation logic, but yes it would be nice to override MongoEngine's method there to provide actual class names to make it more understandable


Support isConnected for producers, consumers and readers

Java PR link

@Lanayx: Please check if it meet your expectation.


Encrypt ballot produces an error message in the web ui

After filling https://stduhpf.github.io/caritat/#https://github.com/nodejs/TSC/pull/1158 with all my details, when I click on Encrypt ballot, an error message shows up at the bottom of the page:

An error occured: The request is not allowed by the user agent or the platform in the current context, possibly because the user denied permission.

I noticed this only after I committed the faulty vote I reported in https://github.com/stduhpf/caritat/issues/3, not sure if it happens otherwise.

FWIW we are using the git commit author, it's repeated in the JSON file for convenience but it has no implication on your vote. The web UI can't guess what is the name or the email address of the voter, so it's simpler easier to not include it.
Closing as won't fix, thanks for the report.

Sorry commented on the wrong thread, reopening.

Can you share what browser you are using?

I'm using Safari Version 15.2 (176<IP_ADDRESS>.6). The encrypted ballot does get generated, which is what I used in the commit but I wasn't sure if that contains valid data, so I just wanted to reconfirm.

The error was showing up because Safari doesn't want the web app to send data to the clipboard right away. This can be safely ignored, it has nothing to do with the ballot data :) I've pushed a fix to not display the error message before the user clicks the Copy to clipboard button.


Support running fastlane inside a folder that is sibling to fastlane folder

Feature Request
It would be nice to have if fastlane supports running fastlane from command line when current folder is a sibling to fastlane folder. Suppose following folder structure:
- repo
  - fastlane
    - Fastfile
  - sibling
    - sibling_subfolder

I should be able to run the following:
cd sibling
fastlane my_lane
cd sibling_subfolder
fastlane my_lane

Motivation Behind Feature

For people who are used to using bundle, rake, thor or pod commands from cli, it's a little bit inconvenient to not be able to run fastlane similarly. All of these libraries work if command is triggered from subfolders. They search backward towards parent folders, and take the first Gemfile, Rakefile, or Podfile as the source file for library.
Feature Description

I don't know if this is fundamentally opposing to how fastlane has built. But assuming lanes are always starting with pwd being the fastlane folder, it should not cause any problems in terms of backwards compatibility.
Alternatives or Workarounds

I have to cd to the folder which includes fastlane folder to be able to run a lane.
Implementation idea
It seems to me that if we just change this method and also glob over parents, it would provide the mentioned support.
https://github.com/fastlane/fastlane/blob/d56b0602298e1a58743c50b1f94fde7467d53589/fastlane_core/lib/fastlane_core/fastlane_folder.rb#L8-L16
Thanks for amazing tool!

Yes, that could probably work. I can also not think of a reason why this would be counter to what fastlane wants to be, besides maybe that it could be confusing to the user. We probably would also want to make sure to output any non standard path (./fastlane) when it is used as the developer should definitely be made aware of it.
Can you create a pull request maybe?


Fixed kebabCase method and tests

Closes #632

[x] Running yarn lint does not trigger any linter errors
[x] The test of the method you have fixed is passing
[x] You have written a new skeleton method for someone else to work on!
[x] You have written tests to accompany your skeleton method

Hey @BlakeGuilloud I've fixed that merge conflict, thanks!


Random files appearing in my trash bin everytime I open a Jupyter notebook

Environment data

VS Code version: 1.68.0
Jupyter Extension version (available under the Extensions sidebar): v2022.5.1001601848
Python Extension version (available under the Extensions sidebar): v2022.8.0
OS (Windows | Mac | Linux distro) and version: MAC OS MONTEREY 12.4
Python and/or Anaconda version: mamba 0.22.1 py39hde45b87_0 conda-forge, conda 4.12.0 py39h2804cbe_0 conda-forge
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: Local | Remote | N/A : Local and Remote

Expected behaviour
Previously when I was running Jupiter notebooks, they were working fine.
Actual behaviour
When I open Jupyter notebooks in vscode now, these files appear in my trash bin automatically. Each one is 72 bytes, the file names are in the following format, "name of file"-jvsc-1ddefa11-e9b6-4e53-af4e-c472c2608261e3c71dba-91c1-44d1-af5e-b96ffff8817d.ipynb and contain the following code {  "cells": [],  "metadata": {},  "nbformat": 4,  "nbformat_minor": 5 }
They always produce the same file just with a different name. I am worried that this is going to destroy my hard from the constant read and writes. I am running the files from an external APFS SSD drive if that makes a difference.
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Open a jupyter notebook in vscode using mambaforge

Save the file and reopen it.

Open the trash bin and you will see the file I am talking about.

Logs

Output for Jupyter in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Jupyter)

XXX

thanks for the bug, I was able to repro on Windows as well after disabling zmq (which some macs have to do by default).

add "jupyter.disableZMQSupport" : true to your settings
open a new notebook and run a cell
or
open a notebook that has previously run a cell

I see the file temporarily appear in the vscode explorer and then is deleted and ends up in the recycle bin

This is on purpose. We require a 'backing file' in order to start a kernel.

can we skip the trashcan/recycle bin though? It sounds like this wasn't happening previously, though I haven't confirmed that

I don't believe that's our code doing that. We ask jupyter to delete it:
https://github.com/microsoft/vscode-jupyter/blob/38caef1b0c652d6d534f3c9f5b67d357c65ad70a/src/kernels/jupyter/session/backingFileCreator.base.ts#L77

don't believe that's our code doing that. We ask jupyter to delete it:

we should be able to delete (these) local files when dealing with non-zmq for local sessions.

Can also confirm that I experience this bug and I don't think it's possible to enable zmq in mac-arm due to not providing builds for that platform.

+1

Zeromq is native to Mac arm. Is there anyway to enable it?

Zeromq is native to Mac arm according to this https://formulae.brew.sh/formula/zeromq#default Is there anyway to enable it? @DonJayamanne @amunger

+1
mac Apple M1 Pro
vscode:1.73.1

This will get fixed when #5206 is resolved.
Hence closing this as a duplicate of #5206


Supporting Custom Migration Driver

Hi. I understand that the officially supported dialects are MySQL, Postgres, and SQLite, but would it be possible if you could also support custom dialects so we can implement the interface and do migration as well? For instance, we're trying to use Oracle OLTP database for backing data and the SQL semantics are a little different

Hey @wonsoh and thanks for proposing this.
I'm currently working on the schema-versioning and the new migration. This is going to be supported as well.
Can you share how do you expect such an API to look like? Thanks

Hi @a8m
I am looking at this section (https://github.com/ent/ent/blob/master/dialect/sql/schema/migrate.go#L106-L134 ) and this section (https://github.com/ent/ent/blob/master/dialect/sql/schema/migrate.go#L637-L650) since this interface is private and NewMigrate is hardcoded by a switch statement, I cannot pass in my own custom dialect. If we could pass in our custom dialect option by passing in a custom struct that implement  the sqlDialect interface (WithCustomDialect(...)), that would be great.


Unity Tries to Resolve Internal References

Description
I mentioned there is a behavior change in unity where in 4 it would look into only public ctors but in 5 !ctor.IsPrivate check picks internal ctors as well.
I have 2 questions here:

I want to understand why this change has been added and if needed why it is not configurable in an easy way when UnityContainer is created.
Given this change was needed, still code behavior is inconsistent. If it cannot resolve the internal ctor right away it moves to the next ctor but if it fails to resolve down the line for instance if ctor has an input type that it cannot be resolved by unity then it blows up and stops there. It needs to at least fail in either scenario or does the same thing for second scenario.

To Reproduce
Please provide UnitTest in the form of:
public interface Interface1
{
}

public class TestClass1 : Interface1
{
    private readonly string access;

    public TestClass1()
        : this(s => s, nameof(TestClass1))
    {
    }

    internal TestClass1(Func<string, string> access, string data)
    {
        this.access = access(data);
    }

    public override string ToString() => this.access;
}

internal class InternalClass
{
    private readonly string access;

    internal InternalClass(Func<string, string> access, string data)
    {
        this.access = access(data);
    }

    public override string ToString() => access;
}

public interface Interface2
{
}

public class TestClass2 : Interface2
{
    private readonly InternalClass access;

    public TestClass2()
        : this(new InternalClass(s => s, nameof(TestClass2)))
    {
    }

    internal TestClass2(InternalClass access)
    {
        this.access = access;
    }

    public override string ToString() => this.access.ToString();
}

[TestMethod]
public void WorkingResolution()
{
    var container = new UnityContainer()
                .RegisterType<Interface1, TestClass1>();

    // this works
    var res = container.Resolve<Interface1>();

    Assert.NotNull(res);
}

[TestMethod]
public void NotWorkingResolution()
{
    var container = new UnityContainer()
                .RegisterType<Interface2, TestClass2>();

    // this will throw a runtime exception because it cannot resolve InternalClass
    var res = container.Resolve<Interface2>();

    Assert.NotNull(res);
}

Additional context
The context that I ran into issue was having all the code in a separate assembly and registration is done in a separate place. My expectation was to Unity not seeing internal ctors since was not made visible to it by MakeInternalsVisisbleTo attr.

Internal constructors were required for compatibility with Microsoft.Extensions.DI in aspnet.


klayout_tech improvements

A couple of changes to the KLayout technology module:

Added CustomPatterns class, which contains lists of CustomDitherPatterns and CustomLineStyles
Added yaml import/export for CustomPatterns and LayerDisplayProperties
LayerView group members are stored in a list rather than a dict
Added mebes reader options to KLayoutTechnology export since they aren't created by the KLayout API
Fixed some formatting issues when writing .lyt files
2.5D section of .lyt now uses zmin, zmax as it should, rather than zmin, thickness as it did

Thank you Thomas!


Age calculator

831

Title : Age Calculator
Issue No. :#831
Code Stack :
Close #831
Before
https://github.com/apu52/METAVERSE/assets/168094736/52da14a4-ab2c-473c-9dae-8a18ba6f2a56
After
https://github.com/apu52/METAVERSE/assets/168094736/319b9d79-3180-425c-ae4f-b8c111cfaf42
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have created a helpful and easy to understand README.md
[ ] I have gone through the  contributing.md file before contributing

Additional context (Mandatory )
Are you contributing under any Open-source programme?

[x] I'm a GSSOC'24 contributor
[ ] I'm a SSOC'24 contributor

hello @heysagnik please merge it and level

@apu52 please merge this request as it change full UI of the age calculator

@apu52 level it as level 3

Add level on it and merge

@heysagnik please review it


Adding a 404 page

Is your feature request related to a problem? Please describe.
I would like to add a 404 page
Describe the solution you'd like
404 page will be displayed every time a path that is unknown is entered.
Are you willing to contribute to this issue? Yes
Describe alternatives you've considered
Additional context
Once approved I would upload a screenshot as well

@ShambhaviSharma0110 +1 if you put in a page like this, in accordance with the theme of the website

Can I work on this?

@ShambhaviSharma0110 are you going to work on this?

Hello! I am interested to work on this issue if it isn't assigned yet.

@prerna2712 Go ahead with this :rocket: before that read our Starter Guide.
Also, go through above comments.

@maneprajakta I've already made a 404 page I was just waiting to get assigned and make a pr...

@ShambhaviSharma0110 are you going to work on this?

.

@ShambhaviSharma0110 No issues but we do expect some replies.

@prerna2712 check out other issues that we have for project.

yes ofc I'm really sorry for late replies I'll create a pr in just about an hour...

@prerna2712 check out other issues that we have for project.
Okay sure

can I work on this issue?

@arpit456jain thanks for showing interest the issue is assigned already. Go through our starter guide :)

@maneprajakta I had made a 404 page and added the backend for showing this page if 404 error occurs kindly review it.
https://user-images.githubusercontent.com/55352601/113545832-86c69c80-9608-11eb-8449-00a0edbbad49.mp4
If its ok may I create a PR

@arpit456jain
This is nice but if you could do something like this examples
or this

Maintain theme of the website take help of codepen and canva to get the animations you can even get images on canva.

Ok @maneprajakta i will try

@arpit456jain No problem with the debug variable, it's set to False in production, we will review accordingly after a PR is made :)

Ok @Praful932

I am a Gssoc participant , please assign me this issue.

@simranbhalla3 This is already assigned :)
Check out up-for-grab issues or the starter guide

@Praful932 kindly review my PR


Bayonetta 2- Resolution pack causes lighting corruption in Vulkan

Can't recall when this happened exactly, have tried several older Cemu releases (as well as latest Patreon builds) so pretty sure it's a graphics pack update issue. OpenGL is unaffected, this is Vulkan only.
Basically now with Vulkan, selecting a resolution other than default in Bayonetta 2 removes the lighting from the centre of the screen, only visible outside of a circle that pans with the player. I've tested with a combination of other mods, disabled all others and it's only the resolution setting that causes this.
eg:

I was reminded by the reddit thread that this issue was occuring, but after just trying, I can't actually replicate this. I only have a square as shadow, but that might just be a completely different problem.  So that's quite interesting. I would like to know what GPU driver you are using (I'm using the Vulkan developer drivers at the moment)
 and the contents of the log.txt file after you've encountered this problem, which you can upload to a service like pastebin.com.

Latest Nvidia hotfix (445.98). I'll try the Vulkan beta drivers then.

Forgot to mention that I was using 1440p, the shadow resolution pack to 200% and the portal workaround.

No point in shadow resolution being enabled ATM since shadows don't work form Bayo 2

Ok, tried the Vulkan beta drivers, and matched your settings (1440p, 200% shadows, Portal fix) - no change.

log.txt

Ended up being able to replicate it (I actually wasn't using the latest graphic packs, oops...) and fixed a core issue which caused bugs with Vulkan which was related with the ultrawide patch that was added.
Let me know whether this fixed your issue for you too! Thanks for letting us know about it.

Indeed it is fixed! Thanks very much.

Awesome! 😁

Hmm, don't know if this is related to your graphics pack, but now Bayonetta 2 consistently is crashing with an '[22:24:11] Vulkan-Info: Pipeline creation failed to due to invalid shader' now. OpenGL is fine, this is just under Vulkan, have played it extensively under Vulkan without incident, only switched when this shadow bug came up. Was fine for about 1/2 hour, now can't get past this area:

Tried with usual mods/packs, and only with the res pack activated. Same thing. Also deleted all shader/pipeline caches and started fresh, no diff.
log.txt
Log.txt attached. Let me know if you think it's related, otherwise I'll post it in the cemu reddit.

Well, this issue thread is mostly meant for graphic pack issues. It's easier if you were to join the official Cemu discord if you haven't done so already, and that you'd mention your problem in the troubleshooting channel. I might even be able to help you there.
Also, I noticed that you send your log.txt file from OpenGL that didn't crash, so it doesn't really clarify anything.

Ah, whoops. Ok Discord it is.


The graph explorer is not fully loaded

OS: Win10
Build Version: 20200711.5
VS Code Version: 1.47.0
Repro Steps:

Drill down  a Gremlin (graph) account --> Database --> Graph.
Click Graph --> Pop up a dialog --> Click "View Extension" --> Install Cosmos DB Graph extension.
Click on the graph in the tree.

Expect:
The graph explorer shows up as the below:

Actual:
The graph explorer shows up as the below:

More info:
This issue doesn't reproduce on Mac/Linux.

This extension is quite useless until this is fixed. How can you not fix it with extreme urgency? The value that your extension is supposed to provide is completely broken.

@pvillads We prioritize all bugs and bug fixes among all extensions we own and work on.  This issue is slotted to be worked in the next couple of weeks because of the low usage of the extension. Thank you for your reminder and the comments.
Also, we encourage users to use Gremlin in Azure portal where they have a much better coverage and a richer experiences.


Set a preset first

When i want to start farm,  i have communique "Set a preset first!". My friend give me his account to check it there, because he dont have this problem and this communique was poped too. Please help me.


"prefer-readonly" incorrectly reported for a private field that is reassigned-to within a nested class

Bug Report

TSLint version: 5.17.0
TypeScript version: 3.5.1
Running TSLint via: (pick one) CLI

TypeScript code being linted
export class C
{
	private _a = false;

	private readonly _S = class
	{
		private readonly _parent: C;

		constructor(parent: C)
		{
			this._parent = parent;
		}

		doSomething()
		{
			this._parent._a = true;
			console.log(this._parent._a);
		}
	}

	public f()
	{
		(new this._S(this)).doSomething();
	}
}

with tslint.json configuration:
{ "extends": [ "tslint:all" ] }

Actual behavior
TSLint reports:

Private member variable '_a' is never reassigned; mark it as 'readonly'.

Expected behavior
TSLint should not report this error, since the nested class _S is permitted to mutate C's private field _a.

💀 It's time! 💀
TSLint is deprecated and no longer accepting pull requests other than security fixes. See #4534. ☠
We recommend you instead use typescript-eslint to lint your TypeScript code with ESLint. ✅
👋 It was a pleasure open sourcing with you!

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


ASI

Automatic Semicolon Insertion (ASI) is not a feature. Don't rely on it. It's super complicated and there's no practical reason to burden all of the developers in a team for not possessing the frivolous knowledge of how ASI works. Avoid headaches, avoid ASI.
Always add semicolons where needed

To quote standard out of laziness:

No semicolons – It's fine. Really!

There is only a single rule that is a real "gotcha!" and memorizing a single rule is not "frivolous knowledge of ASI".
I'm impartial about whether to avoid or use semicolons. However spreading FUD as the reason to use them should not be how a responsible developer does it. It's largely a stylistic issue. Feel free to advocate for the use of semi-colons but please make sure you aren't doing so under false reasons.
Regardless, I'd take a programmer with frivolous knowledge of how a language actually parses and functions over one who doesn't any day of the week.
Relevant to above statement:
http://www.slideshare.net/olvemaudal/deep-c

Well. Knowing to use ; before ( and [ if they come after a \n, and not to do return\nvalue is not necessarily something to be proud of. I'd rather have code be consistent (just using ; everywhere), than having to bother following those rules.
Not really FUD, the rules are obtuse. The solution proposed by standard, where the linter watches for these corner cases and tells you to patch them is pretty elegant, though.

That's a rather over-complicated way of saying "never start a line with ( or [ "... The specifics aren't overly important in that regard as the simple and easy to remember rule would prevent the specific "gotchas!"
Code would be consistent with or without ;
Again, I don't care which people decide to use. So long as the code base is consistent. I actually prefer using ; , however my reasoning to do so is aesthetic in purpose rather than being based on FUD (we're going to have to agree to disagree on the information surrounding ASI being FUD or not it seems)
I'll submit a PR as suggested sometime tomorrow that proposes using a linter to watch for the "gotcha!" as an easy-fix to any possible ASI issues of not using semi-colons.


♻ Add macro for generating smart filters

This is an initial cut at adding a macro to generate smart filters. I still need to implement more robust error handling in the macro and split the crate up a bit more reasonably, but I wanted to make sure the API fit in with the way these filters are expected to develop.
It does a pretty good job cutting down on boilerplate so far.

One thing to call out is what I mentioned on Discord wrt supporting Filter<DateTime<FixedOffset>> (e.g. link). With my active feature branch containing the smart list UI updates, these are what I am aiming to be able to configure. I think this will be a good exercise to see how a new type should be added, if it would need to be done again down the road

Responding to this portion of your comment first. I'm going to make an addition adding support for this, and it should hopefully demonstrate the process of updating the macro decently well. I'll also try to document it.
The way you'd go about updating this macro should be quite simple. First you'll add a new branch to the match arm generator function, generate_match_arm, that will match on the type you want to add handling for. That arm should call the function you will create in step 2 (just like the two others currently existing). In step 2, you copy generate_string_match_arm or whatever as a base, and define a new function that properly forms the match arm contents the way you'd like. Read the documentation for the quote crate if anything in that function is unclear.

Hey! So I gave this a read-through and don't see any issues really. I am still not very knowledgeable with macros, but I was able to follow at least a bit of what you've done (thank you for the doc comments). One thing I'm curious about that I don't yet understand is how there aren't duplicated impl blocks, e.g. impl Filter<String> for each different filter enum (e.g. MediaSmartFilter, SeriesSmartFilter, etc).
Since it is hypothetically generating analogous code I wouldn't expect an issue, but I'm curious if you've spun up a server and tested this yet?

Admittedly I haven't tested it yet, but I did confirm that it generates the same code I removed. I'll test it properly before I un-draft the PR.
I don't think I fully understand the scenario you describe, but I'll walk through how the macro works and see if that answers it.
The macro operates in two phases.
Phase 1
The macro takes in an enum definition (that includes the #[prisma_table = "whatever"] and #[is_optional] annotations) and peels it apart using the syn crate.
It takes apart the attributes on the outside of the enum and looks for prisma_table, taking its value if it is there and failing if it is not. It removes this from the attributes list so that it can reattach those to the final enum.
The rest of the enum becomes a chunk of data, DestructedEnum, consisting of a Vec of EnumVariant data. Each EnumVariant tracks:

the name of the enum variant (variant_name)
the name of the field inside that enum (variable_inner_name) - this must match the prisma column name being filtered
the type of that inner field (variable_type)
whether or not if found an #[is_optional] attribute

The rest of generation doesn't operate on tokens, it operates on that data.
Phase 2
The "real" enum is constructed by gen_smart_filter_enum. This just recreates the input enum, but with some modifications. For each field, it checks the field's type stored above using should_filter_type, and for any type in that function (i.e., a String, a number type) it will return true and wrap the type so that it's Filter<T>. If it's not a type listed, it assumes it must be a smart filter and uses it as is.
Next, the into_params implementation is generated by gen_smart_filter_impls. This function loops over the variants and fills in match arms depending on the data stored in phase 1:

For String it will generate an arm with generate_string_match_arm
For numbers it uses generate_number_match_arm
For everything else (fallthrough in the match statement) it uses generate_filter_match_arm, and generates code that should fail to compile unless the type was a smart filter.

So I think the answer to your question, in summary, is that it doesn't generate redundant impls because impls only get generated for things when they are annotated with the macro. It doesn't recurse into the enum and generate anything for contained types.

@JMicheli Your explanation / walk-through was absolutely perfect, thank you!

So I think the answer to your question, in summary, is that it doesn't generate redundant impls because impls only get generated for things when they are annotated with the macro. It doesn't recurse into the enum and generate anything for contained types.

Makes sense now 👍

Btw I updated it to handle DateTime<FixedOffset> it was a very simple change since those get handled the same way as numbers, so I just made that branch match on "DateTime" and it worked right off the bat.
I still need to make the error handling a little nicer and I am questioning whether or not the idea of just specifying, e.g., String in the input and then having the macro add the Filter<T> wrapping was a good call. It's less explicit and maybe confusing to read? Less keystrokes though.

Btw I updated it to handle DateTime it was a very simple change since those get handled the same way as numbers, so I just made that branch match on "DateTime" and it worked right off the bat.

Nice! I see it's two lines, which is awesome.

I am questioning whether or not the idea of just specifying, e.g., String in the input and then having the macro add the Filter<T> wrapping was a good call

I think it's a good call, probably? Otherwise the Filter would materialize from 'nowhere' right? I know it literally wouldn't, it would be generated from the macros, but the context for where it came from might be unclear? I'm happy deferring to what you think is best there, I don't think I have a strong opinion besides the fact that I enjoy fewer keystrokes

Yeah the materialization from nowhere was my main concern. But it does simplify writing new smart filters so I think it's okay for now.
I'll change it if my assumption that this makes sense proves to be wrong.
Once I clean up error handling this should be good to merge. Would you prefer to merge it on top of your current filter changes or right into develop?

Sounds good!

Would you prefer to merge it on top of your current filter changes or right into develop?

Let's rebase this PR to experimental and merge it there when ready. I'll rebase my feature branch once this is merged and fix any conflicts


Remove FAQ section about Deno hosting

This section is outdated (looking at you, Heroku) and there's a much better overview in the hosting comparison. Also, the Deno ecosystem has developed much further, and there are plenty of places to host Deno code now.

I'll simply remove it from all translations

Dat would be coolleshlagge!

Could you also remove it in /site/docs/resources/zh/faq.md and /site/docs/resources/es/faq.md, please?

I literally forgot that we recently added Indonesian. LOL.

Could you also remove it in /site/docs/resources/zh/faq.md and /site/docs/resources/es/faq.md, please?

I literally forgot that we recently added Indonesian. LOL.

I hope @fadzikri and @CikiMomogi don't hear about this 😱

jk

Could you also remove it in /site/docs/resources/zh/faq.md and /site/docs/resources/es/faq.md, please?

I literally forgot that we recently added Indonesian. LOL.

lol


decim keyword for epochs fails if lowpass filter is None

(I guess I'm partially responsible for this problem myself in the first place, as I helped get BV importing done when no filter is set ...)
If info["lowpass"] is None, which can happen e.g. if importing underspecified BV files, and epoching uses the decim keyword, an error is thrown b/c new_sfreq < 2.5 * lowpass in line 140 fails if lowpass is None.
This is a simple workaround.
Might be worth considering if low/highpass = None should actually be allowed, or if import filters (specifically for BV) should default to something like 0 (or 1/length) and srate instead.

@agramfort like this?

looks good thanks
let's merge when travis is happy
did you do a git grep lowpass to see if the problem may happen elsewhere?

travis is pep8 ...
There seem to be a few more potential instances, I'll see if I can track them down.

thx

Thanks @jona-sassenhagen !