Swift 3 Migration

Graph, ASAP, Single-Source Shortest Paths (Weighted) migrated to
Swift 3.
Xcode project setting updated.
Other small changed generated by Xcode 8.1

I also made a new commit with several new Swift 3 migrations.

BinarySearchTree
BinarySearchTreeTest
BreathFirstSearch Edge, Graph, Node, Queue
kMeans
kMeans test
Minimum Spanning Tree
Two-Sum Problem both solutions
Other small changes generated by Xcode

@NeilNie I just took a look at this. It seems like you had an older version of the master. Some of the algorithms you've done migrations for has been completed already.
For instance, the Binary Search Tree algorithm was updated a month ago with 6ee811901e79b860c443f8bd8ea3ccfecd9f08ba
The overall code conversion looks about right, though there are a few styling suggestions I want to add.
// not preferred
private class EdgeList<T> where T: Equatable, T: Hashable { ... }

// preferred
private class EdgeList<T: Hashable, Equatable> { ... }

Also, it looks like none of the README files are updated. We'll need those updated with the code changes as well.
I'll also really appreciate if you could break up the changes into their own PRs. It's fairly difficult to review 83 files at once! Otherwise, I appreciate the time you put into migrating so many of these.

@kelvinlauKL Thank you very much for your thoughtful reply. I will keep your suggestions in mind and try to update the READMEs.

@chris-pilcher Could you follow up on this? Travis seems to have broke.

@NeilNie
I apologize for the late response on this PR. All of the changes have since been migrated over to Swift 3, so I'll close this PR now.


Can we make docker port image configurable?

Description
We have to run the container in host network mode, so we cannot remap 8080
Priority
I have no budget and there's no rush, please add this feature for free
More information
I can prepare PR if it is convenient. In fact changes required are small

I don't think any code changes to the repo are necessary for you to facilitate this change on your own.  I confirmed this works in a sandbox.
Change the Dockerfile from this:
FROM tomcat:10.1.16-jre17-temurin-jammy

RUN set -eux; \
	apt-get update; \
	apt-get install -y --no-install-recommends graphviz

RUN sed -i 's/port="8080"/port="8080" maxPostSize="10485760"/' conf/server.xml

ADD build/libs/structurizr-onpremises.war /usr/local/tomcat/webapps/ROOT.war

ENV CATALINA_OPTS="-Xms512M -Xmx512M"
EXPOSE 8080

CMD ["catalina.sh", "run"]

To This (Assuming desiring port 8282 instead of 8080):
FROM tomcat:10.1.16-jre17-temurin-jammy

RUN set -eux; \
	apt-get update; \
	apt-get install -y --no-install-recommends graphviz

RUN sed -i 's/port="8080"/port="8080" maxPostSize="10485760"/' conf/server.xml
RUN sed -i 's/port="8080"/port="8282"/' conf/server.xml

ADD build/libs/structurizr-onpremises.war /usr/local/tomcat/webapps/ROOT.war

ENV CATALINA_OPTS="-Xms512M -Xmx512M"
EXPOSE 8080

CMD ["catalina.sh", "run"]

Yes, the diff can switch the app to another hardcoded port. Thank you.But to make this work:

You need to build and support your own image of structurizr
The port remains to be hardcoded in the image. You can't override it on container start

I'd be interested to see what your solution to this problem is.  I suggest you proceed with the PR you proposed.

I tested the scheme with variable substitution


How to send hex to device?

i find write param type is String, but i need to send hex to device? how to do it?

i find write that support hex, and i send Buffer but not response

Yes. At this point in time the response is solely string. There was an issue opened early in the project where someone retired a byte array. You'd need to do the same thing as that yourself.
I have the plan to allow writing custom parsers in the bootstrapping, but don't have the time to complete it at this point.

Yes. At this point in time the response is solely string. There was an issue opened early in the project where someone returned a byte array. You'd need to do the same thing as that yourself.
I have the plan to allow writing custom parsers in the bootstrapping, but don't have the time to complete it at this point.
Since the result is a string, you should be able to convert it to hex representation yourself.

i use method of Self-Transmitting and Self-Receiving test; but i find the data received is inconsistent with the data sent;  the data is correct in writeToDevice function, but onReceivedData i get a incorrect data；can you help me check it?

Yes. At this point in time the response is solely string. There was an issue opened early in the project where someone returned a byte array. You'd need to do the same thing as that yourself.
I have the plan to allow writing custom parsers in the bootstrapping, but don't have the time to complete it at this point.
Since the result is a string, you should be able to convert it to hex representation yourself.

i use method of Self-Transmitting and Self-Receiving test; but i find the data received is inconsistent with the data sent; the data is correct in writeToDevice function, but onReceivedData i get a incorrect data；can you help me check it?

send file:
let buf = new Buffer('3aa30000000000010002800183a8', 'hex');
RNBluetoothClassic.write(buf)

RNBluetoothClassicModule.java
@Override
  public void onReceivedData(BluetoothDevice device, byte[] data) {
    String msg = String.format("Received %d bytes from device %s", data.length, device.getName());
    Log.d(TAG, msg);

    mBuffer.append(new String(data, mCharset));

    if (!mReadObserving.get()) {
      Log.d(TAG, "No BTEvent.READ listeners are registered, skipping handling of the event");
      return;
    }

    char[] hexArray = "0123456789ABCDEF".toCharArray();
    char[] hexChars = new char[data.length * 2];
    for (int j = 0; j < data.length; j++) {
        int v = data[j] & 0xFF;
        hexChars[j * 2] = hexArray[v >>> 4];
        hexChars[j * 2 + 1] = hexArray[v & 0x0F];
    }

    String message = new String(hexChars);
    Log.d(TAG, 'Check Data:' + message);
    if (message != null) {
      BluetoothMessage bluetoothMessage
        = new BluetoothMessage<>(new NativeDevice(mBluetoothService.connectedDevice()).map(), message);
      sendEvent(BluetoothEvent.READ.code, bluetoothMessage.asMap());
    }
  }

i find reason, bytes[] change UTF-8 type in write function, and this operation will result in the hex code inconsistency required for the machine.

Thanks for looking into it, that would definitely cause you issues attempting to send HEX.  Although I'm not sure how receiving hex will work, since it's doing the exact same conversion to string, although it will be easier for you to convert back.
I don't know how that will affect users already using as it is (as mentioned somewhere, most of the Android stuff I just pulled over from react-native-bluetooth-serial).  I've slowly been re-writing to include more configurability, like adding middleware/handlers to the on and receive that can be applied to the device during the connect request.
I just haven't had time to finish.  also to get the same functionality on Apple, I need to rewrite a bunch of that so that its either All Swift or All Objective C.
Hopefully you can work off your own changes for the time being, but I will definitely leave this open so I have another instance of configured data.


Passing payload on socket.channel

The channel function is not actually passing the payload the Phoenix's socket.connect function.
This fixes it.

Amazing! Thank you 👍


Feet sliding in Talos simulation

Talos_reduced.py.zip
Hello,
I don't know if it's a bug or feature, but I got a problem with sliding feet when trying to simulate a movement in which Talos stands still while reaching a target with its left hand. In this problem the feet should not be moving, but they end up  interpenetrating each other.
I use the DDP method to solve an optimisation problem with various weights (state and control regularization, goal tracking, CoM tracking). Contacts are enabled through 6D contact models. Joints limits are enabled through activation bounds.
I have provided the simulation file I used. I'm using Python 2.7.

It seems your are having numerical integration errors. This is expected in any DAE, not a bug, and note that the contact dynamics is a DAE.
To mitigate this numerical issue, you should increase the proportional -- Baumgarte  -- gain in your contact models (line 74 and 77 of your code).
For more details about these gains you might want to read the Crocoddyl paper: https://cmastalli.github.io/publications/crocoddyl20unpub.pdf
Yours,
Carlos

Please let me know if this solves your problem.

It seems you are using incorrectly the Baumgarte stabilization (i.e. contact gains in line 74 and 77), so it is expected to see sliding motions. Note that the reference placement is the origin of the inertial frame (i.e. world frame). So you have to options:

Do not use proportional gain (zero value)
Define the properly the reference placement, desired placement of the contact

I suggest to follow 1 because, typically, there is good numerical stability with only the damping term.
For more details about these gains you might want to read the Crocoddyl paper: https://cmastalli.github.io/publications/crocoddyl20unpub.pdf
Yours,
Carlos

I will close this issue since it is expected behavior.
Feel free to reopen it, if it is needed for you


[HELP] Support to fallback new functionality

Your purpose
I'm trying to use capmonster_python with userAgent and the new functionality fallbackToActualUA.
userAgent is ok, but fallback no.
Will this new functionality be supported soon?

Hi @lasoama,
Thank you for bringing the fallbackToActualUA implementation for capmonster_python with userAgent to my attention. I have successfully incorporated your solution into the library. Your contribution is invaluable to the project's growth.
Add fallback option to UserAgent tasks


feat(install): Argo CD

xref #2023

I'm guessing you wanted to mention this: https://github.com/kumahq/kuma/issues/2023


How to convert api json response to .xlsx file and then download?

Since there is so much going in the documentation I can't seem to figure out a straightforward way of implementing this feature in my node-react app:
The API for my project is serving me array of objects containing data in json format page wise. Means, to get the entire data I have to make recursive get calls to the api like this: request.get(/contacts/page/(1...blank response page) and keep on concatenating data in a variable.
Now, I want to be able to convert this json data stored in a variable to a .xlsx file and make it available as a download or download concurrently as the data is fetching from the api.
I just need help in figuring out the steps for this process and nothing else. Thanks!

Supposing that data is an array of JS objects (request, JSON.parse, append to an array), you need to build the workbook then generate a download.  The json_to_sheet utility does the hard part:
/* make the worksheet */
var ws = XLSX.utils.json_to_sheet(data);

/* add to workbook */
var wb = XLSX.utils.book_new();
XLSX.utils.book_append_sheet(wb, ws, "People");

/* write workbook */
var buf = XLSX.write(wb, {bookType:'xlsx', type:'buffer'}); // generate a nodejs buffer
var str = XLSX.write(wb, {bookType:'xlsx', type:'binary'}); // generate a binary string in web browser

See the discussion in https://github.com/SheetJS/js-xlsx/issues/817 for longer explanation as well as a sample jsfiddle and a sample runkit

I am able to work on it and it is downloadable but my requirement is to put some title on top as well .
Requirement is like following format : -
Please have a look on this ,
selection_106
How can i achieve that ? I am getting only data set not title ,

The file which is downloaded from  js code using node-excel-export npm is not readable using xlsx-to-json npm .how can i read sheetjs file

I am able to work on it and it is downloadable but my requirement is to put some title on top as well .
Requirement is like following format : -
Please have a look on this ,

How can i achieve that ? I am getting only data set not title ,

Hey bro did you find any solution on it..


Add a wiki: Mabinogi World Wiki

An anonymous user has submitted a new wiki via our submission form.
Link to origin wiki: https://mabinogi.fandom.com/, https://mabinogipros.fandom.com/, https://mabipro.fandom.com/
Link to destination (independent) wiki: https://wiki.mabinogiworld.com/view/Wiki_Home

The destination Wiki is well ahead of the origin Wiki (460 vs. >30,000); however, I believe the only redirect should be from https://mabinogi.fandom.com/. The other two Wikis look like they were created for Mabinogi private servers with unofficial custom content, not official Mabinogi content, which the destination Wiki does not appear to cover.

Adding in commit https://github.com/KevinPayravi/indie-wiki-buddy/commit/83c54a421d316cc377a2fa1e93fb0ef69fbbe945

Now available in version 2.4.0


Rellying on os:timestamp can crash locker

Rellying on os:timestamp can produce crashes:
11:53:04.269 [error] GenServer :locker terminating
** (FunctionClauseError) no function clause matching in :lists.seq/2
    (stdlib) lists.erl:243: :lists.seq(1505728386,<PHONE_NUMBER>)
    (locker) /Users/druss/dev/<app>/deps/locker/src/locker.erl:564: :locker.handle_info/2
    (stdlib) gen_server.erl:616: :gen_server.try_dispatch/4
    (stdlib) gen_server.erl:686: :gen_server.handle_msg/6
    (stdlib) proc_lib.erl:247: :proc_lib.init_p_do_apply/3
Last message: :expire_leases

So, probably it is rare condition, but may be it would be better to use monotonically increasing time or should I make pull request at least to not crash on occurrence of time springs?

This is the first time I've heard of this crash, but it would be good to have it fixed. With the new time support in OTP-20, ideally locker should take advantage of the monotonically increasing time for timeouts (while falling back on os:timestamp/0 if version < 20). I'd be happy to review and merge a PR.

os:timestamp() can return backwards-rolling time since it just gives you the OS time, which might be modified at any time. Relying on this for monotonically increasing time is not possible.


Fix switcher icons, size and more

hi,
I was happy for many months the 1.2 version with a Config and the Original Minecraft Icons and Sizes to become and okay the Owner have Remove this version but what is with new updates of the Original Mod Design? Or is the 1.0 the only Version what you and the other will bring out?

Hmm. I'm sorry to tell you...
Because the EULA of Minecraft, I can't upload version 1.2 again... (Sorry, I miss some context)
And... I'm still a student. I have to finish school now.
I will try to fix it if I'm free.

Hi, I have a long time not hear. I hope your student time is good. Have you a idear how you can fix it?


Create 0000-01-02-mrhariprasadh.md

create new version for feature hhhhh

going to merge pull request


Reporting the same value in multiple currencies

This is a feature we've talked about and started to implement but never finished or even fully designed.
The CurrencyAwareCalculation*Function interfaces were designed to be able to handle this but have never been wired up to the engine. They are capable of returning values in multiple currencies but it's not clear how that can happen because ReportingRules only returns a single reporting currency at the moment.
The engine performs currency conversion into the reporting currency for the regular calculation functions, and it seems reasonable that the engine should be able to convert one value into multiple currencies if the user requests it.
The things we need to think about are:

Do we actually need this functionality?
How can a user request multiple currencies? Should ReportingRules return a Set<Currency>?
Does this create an extra column in the results? So the results have more columns than the user requested? Or does the user request identical columns with different reporting currencies and the engine merges them for calculations and unmerges them to create the results?
How should we return the results from functions? A specialised Result variant? Using maps everywhere might be heavy when most of the time they will have one currency and therefore one element.

I think we keep it simple: any column the user wants to see in the output has to be requested as a column in the input, and this includes the single reporting currency for that column. If they want the same measure in multiple currencies then each would be a separate column with different ReportingRules.
We can optimise the engine to calculate the measure once and do the currency conversion multiple times later.

When the calculation code was refactored to have multiple measures per function execution, this issue was fixed. Calling a calculation with two columns calculating PV, but with different reporting currencies, only invokes the underlying calculation once.


SQLAlchemy unable to find env variable with lowercase letters

I make a simple setup with sqlite3, where I have env SQLALCHEMY_URL = sqlite:///main.sqlite, however during runtime I get an exception:
Traceback (most recent call last):
  File "C:\Python\Python38\lib\site-packages\starlette\routing.py", line 621, in lifespan
    async with self.lifespan_context(app):
  File "C:\Python\Python38\lib\site-packages\starlette\routing.py", line 518, in __aenter__
    await self._router.startup()
  File "C:\Python\Python38\lib\site-packages\starlette\routing.py", line 600, in startup
    handler()
  File "C:\Python\Python38\lib\site-packages\fastapi_sqla\__init__.py", line 66, in startup
    engine = engine_from_config(os.environ, prefix="sqlalchemy_")
  File "C:\Python\Python38\lib\site-packages\sqlalchemy\engine\create.py", line 754, in engine_from_config
    url = options.pop("url")
KeyError: 'url'

Digging a bit deeper I found that in SQLAlchemy engine_from_config use following code to get variable:
options = dict(
    (key[len(prefix) :], configuration[key])
    for key in configuration
    if key.startswith(prefix)
)

where configuration is os.environ and prefix is sqlalchemy_, but when key is being iterated, it get capital SQLALCHEMY_URL which is not match for key.startswith, therefore before passing key it should be lowercased.
This line of code:
https://github.com/dialoguemd/fastapi-sqla/blob/2aa694edaf647eb5205f4500ef7dbe466d8a4106/fastapi_sqla/__init__.py#L64
can be replaced to this to fix this issue:
engine = engine_from_config(dict((k.lower(), v) for k, v in os.environ.items() if k.lower().startswith("sqlalchemy_")), prefix="sqlalchemy_")

Thanks @XCanG for raising that issue. Fixed by #40 and released in v2.0.2


Refactor should handle primitive / object difference

See microsoft/vscode#198600
Steps to Reproduce:

Use the following code snippet

a = 0

for i in range(10):
    a += 1

Use extract method from refactoring tools on the for loop.
Get the following code

a = 0

def new_func(a):
    for i in range(10):
        a += 1

new_func(a)

This has no effect on primitive types.

This has no effect on primitive types.

To be clear, your point is that at the end of the original snippet a is 10, but at the end of the second snipet a is 0?
Is this what you were hoping to see?
a = 0

def new_func(a):
    for i in range(10):
        a += 1
    return a

a = new_func(a)

Hi, thanks for the response.
Answer to your question is yes. Since primitive types are copied by values, refactor should take care of that I think.

Hi @rentruewang, we've tagged the issue with the bug label, which means it's now in our backlog for fixing. The order in which we address backlog items depends on their priority and our current bandwidth. We will provide an update below this ticket once it's fixed. We appreciate your patience!

@StellaHuang95 Makes sense. Thanks.

Reopening as this has not been fixed yet. It still repros for me in 2024.5 103.


new emoji thinking face added

Thinking face emoji added to 3 files.

Thanks, but I have to decline. This project is called gemoji because it contains information for GitHub emoji (not other emoji). And, GitHub does not support this emoji shortcode.

Thus, to get that emoji into this project, it must first be submitted over on github/gemoji. As it happens, at-mislav is working on it, but it’s taking a while. :anguished:

Gotcha.
thanks for the feedback :)
Shiv
On Jan 31, 2016, at 3:05 PM, Titus Wormer <notifications<EMAIL_ADDRESS>wrote:
Thanks, but I have to decline. This project is called gemoji because it contains information for GitHub emoji (not other emoji). And, GitHub does not support this emoji shortcode.
[screen shot 2016-01-31 at 23 00 25]https://cloud.githubusercontent.com/assets/944406/12705301/0c825a3e-c86f-11e5-843b-c4a315fd7124.png
Thus, to get that emoji into this project, it must first be submitted over on github/gemojihttps://github.com/github/gemoji. As it happens, at-mislav is working on it, but it’s taking a while. [:anguished:]
—
Reply to this email directly or view it on GitHubhttps://github.com/wooorm/gemoji/pull/3#issuecomment-177631332.


NTR: laboratory-based population

This term is requested for the VEuPathDB project and will be imported into the VEuPathDB ontology once available. This will be used instead of MIRO_30000033 as MIRO is inactive.
Thanks,
Chris
editor preferred term: laboratory-based population
textual definition: A population where the collection of individuals is located and maintained in a laboratory, research, or production facility.
definition source for the textual definition: VEuPathDB
logical definition: subclass of ‘population’

Would this fit as a research resource ID (https://scicrunch.org/resources)?

@jake-emerson If you mean, are some of these laboratory-based populations available to be shared and could be given a resource ID, then yes.


gh-actions: add mail notification, when there is some problem not caused by user

JIRA: CI-87

Description

Mails are sent to continuous integration team, when some fails occur in github-actions workflows (steps other than build in Build job and runner step in tests job).
Motivation and Context

There was no information for ci team about fails not caused by user in gh-actions ci workflows
Types of changes

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

How Has This Been Tested?

[x] Already covered by automatic testing.
[ ] New test added: (add PR link here).
[ ] Tested by hand on: (list targets here).

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing linter checks and tests passed.
[x] My changes generate no new compilation warnings for any of the targets.

Special treatment

[ ] This PR needs additional PRs to work (list the PRs, preferably in merge-order).
[ ] I will merge this PR by myself when appropriate.

please remove testfile from PR
please use more semantic naming

Thanks for advice, the changes has been applied


Implement pylint's empty-comment rule (PLR2044)

Implement empty-comment
https://pylint.pycqa.org/en/latest/user_guide/messages/refactor/empty-comment.html
Issue #970
Note, I've written some small programs in Rust but don't call myself experienced :)
Last note, the --fix will leave trailing spaces, but I guess that's fine as it might be tricky to determine what more to remove? Or should the code be a bit smarter and rtrim() after dropping the trailing #?
Summary
Implement pylint's empty-comment rule.
Pylint implementation https://github.com/pylint-dev/pylint/blob/c4281bcff86b66fdf8518e7f57dc3405c8da3a4f/pylint/extensions/empty_comment.py#L43
Test Plan
Manually and by running the unit test from https://github.com/pylint-dev/pylint/blob/c4281bcff86b66fdf8518e7f57dc3405c8da3a4f/tests/functional/ext/empty_comment/empty_comment.py#L4

This PR is still draft as locally running cargo dev generate-all add the new PLR2044 rule, but also removes RUF014 which does not seem expected.

This should be resolved by #5832 — sorry about that!

Haven't had a chance to dig in yet but one thing to be mindful of is that this rule is part of a Pylint extension and so isn't enabled by default. We typically mark these as "nursery" as a hack to require explicit opt-in.

Haven't had a chance to dig in yet but one thing to be mindful of is that this rule is part of a Pylint extension and so isn't enabled by default. We typically mark these as "nursery" as a hack to require explicit opt-in.

Ah that makes sense, I just picked the easiest rule to implement. I've queued it up locally as I suppose  there will be more things to address :)

Closing this as stale. Feel free to re-open if you are interested in working on it again!


Proposal (tbd) for allowing function that USE context

This is related to https://monitoring-portal.org/index.php?thread/38672-trouble-figuring-out-variables-macros-in-apply-rules-with-puppet/&postID=239378#post239378 but in no way meant to be a final PR, but rather something to base an actual solution on.
This regex allows to have parameters of the form
param => 'function (var) use (context) { expression }'

where only var will be further parsed by the expression parser in utils.rb.
Side-question: How does one run a single spec?
I've tried
~$ bundle exec rake spec spec/defines/object_spec.rb

but that will still run ALL of the specs.

Sorry, didn't close this by intention!

OK, I found a case (very early already) where the regex above is not working. I will try feeding this info into the corresponding ticket on dev.icinga.com. In case the function body contains an operator (e.g. +), the first regex matches and not the new one I was naively suggesting.
This e.g. is an example that does not work:
name                      => 'function () use (vhost) { "check-http-" + vhost }',

name = function () use ("vhost) { "check-http-"" + "vhost }"

I cannot re-open the PR anymore since it's marked as "Merged" now that I changed the base. Can you create a new PR pleas? Sorry for the trouble


Getting rid of executeActions

@mattklein123 feel free to shoot this down if you find the closures easier to read but tests are easier for me to follow this way.
this does mean more initialize() but I've tried to compensate by automating cleanup.
#1518

Nice LGTM. Can you merge master to get Circle tests running? Also there is some failure on old Travis. Not sure what that is about.

Yeah, it looked like a shutdown complaint from tsan.  I couldn't repro locally  but I believe the problem was that the server was shutting down in TearDown() then the cleanup was happening in the destructor.  This means the server was shutting down without the client/upstream being stopped which seems a likely cause of races.  By sticking them all in the same function I restored prior ordering and hopefully fixed the race.

Oh, closing and reopening doesn't work for retriggering circleCI?  Curses.
FWIW failure appears unrelated to PR.  I wonder if a change to commit and submit -f will work.
Executed 159 out of 159 tests: 159 tests pass.
There were tests whose specified size is too big. Use the --test_verbose_timeout_warnings command line option to see which ones these are.
Copying for image build...
cp: cannot stat '/build/envoy/envoy/source/exe/envoy': No such file or directory
Exited with code 1

@alyssawilk sorry this fix is needed: https://github.com/envoyproxy/envoy/pull/1741. Then you will have to merge master again.

Ah, thanks for the update.  CircleCI is so much faster though - it's going
to be awesome when we've worked the kinks out!
On Tue, Sep 26, 2017 at 10:42 AM, Matt Klein<EMAIL_ADDRESS>wrote:

@alyssawilk https://github.com/alyssawilk sorry this fix is needed:
#1741 https://github.com/envoyproxy/envoy/pull/1741. Then you will have
to merge master again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoy/pull/1732#issuecomment-332220973,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARYFvZu_jpHS2fiQ_cWFqjLTOzLAirJTks5smQ06gaJpZM4Pi52x
.


Use read_file from utils instead of OS cat

Problem fix for an infinite loop on my Macbook. Might be an OS Specific thing.
This call to cat leads to an infinite loop and a never ending process, unless i manually add a newline char at the end of the file This fix uses the existing utility function and seems to work like a charm.

Thanks!


[BUG] ExamplesMethodWithHttpMessagesAsync calls nonexistent API

Describe the bug
A customer on Stack Overflow tried to use ExamplesMethodWithHttpMessagesAsync in sdk/cognitiveservices/Language.LUIS.Authoring/src/Generated/Model.cs, but the method always returns 404 because it's trying to call a LUIS authoring API that doesn't exist. There is no apps/{appId}/versions/{versionId}/models/{modelId}/examples endpoint, only an apps/{appId}/versions/{versionId}/examples endpoint that's called in ListWithHttpMessagesAsync and documented here.
Exception or Stack Trace
{"error":{"code":"404","message": "Resource not found"}}

To Reproduce
Steps to reproduce the behavior:

Call ExamplesMethodWithHttpMessagesAsync with valid parameters
See error

Expected behavior
The Azure SDK should not contain nonexistent API calls
Setup (please complete the following information):

OS: Windows 10
IDE: Visual Studio 2019
Version of the Library used: master branch

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

I've spoken to the LUIS team and it seems like this API was implemented but never exposed. It sounds like they're going to fix it on their end. I'll leave the issue open a little longer for visibility.

Thank you for opening this issue! We are routing it to the appropriate team for follow up.

@praries880 - The LUIS team said they would deploy this API and it should be working in a couple of weeks. I don't think there's anything to do on your end.


Bad Request when testing the app in Bot Emulator

Using Bot Emulator, when I click on Sign In. The new Sign In window shows Bad Request error. I also have an open thread on Stack Overflow at https://stackoverflow.com/questions/53769924/bot-framework-v4-bot-emulator-authentication-bad-request

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

ID: a9a43e68-83a3-1adc-0c55-e277ddb98c6b
Version Independent ID: 5c46848d-bc20-480e-4571-00a283d930a2
Content: Add authentication to your bot via Azure Bot Service - Bot Service
Content Source: articles/v4sdk/bot-builder-authentication.md
Service: bot-service
Product: bot-framework
GitHub Login: @JonathanFingold
Microsoft Alias: v-jofing

Hi @ajaywhiz do you know what version of the emulator you are using? Can be found in Help > About

Sorry, my bad. Problem was with my configuration. I put a wrong ConnectionName. Please close this issue


update CI: run CodeQL scheduled or on dispatch

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (5668b5d) 77.83% compared to head (be9d4ff) 77.83%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #34   +/-   ##
=======================================
  Coverage   77.83%   77.83%
=======================================
  Files           2        2
  Lines         406      406
=======================================
  Hits          316      316
  Misses         90       90

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


Generate const for Route from the @page keyword

Is your feature request related to a problem? In general I prefer to eliminate magic strings.  The current code generated for the @page requires the associated NavLink to use a magic string as in the demo app:
<NavLink class="nav-link" href="counter">

Describe the solution you'd like
    [Microsoft.AspNetCore.Components.RouteAttribute(Route)]
    [Microsoft.AspNetCore.Components.RouteAttribute(Route2)]
    public class Admin
    {
      public const string Route = "/admin";
      public const string Route2= "/otheradmin";

Thus the NavLink would not need a magic string
<NavLink class="nav-link" href="Admin.Route">

Describe alternatives you've considered
We could add
public const string Route = "/admin";

to the class ourselves but the @page directive currently won't let me reference that.
Additional context
Add any other context or screenshots about the feature request here.
I took a quick search trying to find where the code for the .g.cs files gets generated and was unable to locate it.  If this is acceptable change then maybe we more readily get community assistance on this if you point us to the source area.
Thanks!   Blazor Rocks!

It’s a great idea. So great that we already have an issue tracking this :)
Closing as a duplicate of https://github.com/aspnet/AspNetCore/issues/6352

I searched really I swear I did.  The new GH interface even searches automatic.   Sorry I didn't see it. :)


errors when running the code

I am trying to run the code, the following errors appear: FileNotFoundError: [Errno 2] No such file or directory: 'ninja': 'ninja', RuntimeError: Ninja is required to load C++ extensions. Any ideas  to solve it will be highly appreciated. Thank you so much.

My job submission script is:
CUDA_VISIBLE_DEVICES=0,1,2
module load pytorch/1.1.0
python3 train.py

answered in this issue:  #8


Not open settings.

Hi, when I try to open the settings this warning appears and I can't open them (Windows 11 Pro Insider Preview build 22526.1000):
"System.NotSupportedException: You must target Windows 10 in your app.manifest and run without debugging.
in WindowsDesktop.VirtualDesktopHelper.ThrowIfNotSupported()
in WindowsDesktop.VirtualDesktop.GetDesktops()
in SylphyHorn.Services.SettingsService.ResizeList()
in SylphyHorn.Services.HookService.Suspend()
in SylphyHorn.ApplicationPreparation.g__ShowSettings|15_0()
in System.Windows.Forms.MenuItem.OnClick(EventArgs e)
in System.Windows.Forms.MenuItem.MenuItemData.Execute()
in System.Windows.Forms.Command.Invoke()
in System.Windows.Forms.NotifyIcon.WndProc(Message& msg)
in System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)"

@andigenn Thank you for your report.
This is a bug of VirtualDesktop library, rather than of SylphyHorn. I try to fix it now.
Please try the test binary (it worked on build 22533 and 22538), the link is here: https://github.com/hwtnb/SylphyHornPlusWin11/releases/download/v4.0.0-beta.3/SylphyHornPlus-Ver4.0.0-beta.3-fix11Insider.zip
Note: delete cache data (VirtualDesktop.*****.generated.dll) at %UserProfile%\AppData\Local\grabacr.net\SylphyHorn\assemblies before startup.
On recent Windows 11 Insider Preview, SylphyHorn cannot get COM IID at rumtime. It may be difficult to support newer builds if virtual desktop COM API (CLSID or IID) is changed.

Wow! It works fine on Win 11 build 22538.1010 rs_prerelease. Tnx a lot for your work!

@andigenn Thanks for your test.
I have released beta.4.
I will close this issue if no problem occurs.


Support for Apple TV and target

Hi,
Thank you for submitting the pull request.
Could you tell me what is the essence of the change (all I see is a bunch of updated Xcode project files and auto–generated code).

Hi, sorry I'm a bit new in pull requests, I was trying to get it to work on TV but I think the main reason it doesn't work is the pod spec, am I right?

Cocoapods just help automating fetching the necessary files and adding them in your project. In this particular project only the files in the "Classes" folder are used.
I haven't used MCLocalization in an Apple TV project yet, could you tell me what is the problem with them? Some sort of compiler error?

I've created a test tvOS project and, indeed, there was an issue with Podspec – turns out, an additional attribute had to be specified. Seems to be working now.
Thank you for pointing the problem out and please let me know if there are any more issues.


Support clarifications

We have gone through your Project and we would like to use this for our project.
Currently we are using Elastic search version 5.1.1 which faraway from the given one.  Hence we are unable to use this on our project.
Will you able to support on this??
Awaiting for your valuable feedback.

Oculus is an Archived Project
Oculus is no longer actively maintained. Your mileage with patches may vary.

There is no support for anything.


Add docs for projected volume driver

@mburke5678

This change is

@jpeeler Just so I am clear, this is a new section being added to the Kubernetes docs?

@mburke5678 that's right, it's documentation for a new feature

@jpeeler Is this for the 1.6 release?

@chenopis yes

@jpeeler BTW, the Travis CI test is failing because you need to add an entry for the doc in the Table of Contents (ToC):
Error: docs/user-guide/projected-volume/index.md doesn't have an entry in the table of contents under _data/*.yml

Rebased, but still getting an error in travis.

We just fixed an issue w/ Travis, so if you try rebasing again, it should be okay now.

Rebase did solve the Travis issue, but now it seems netlify is failing. Am I supposed to be able to view the logs?

@jpeeler Netlify seems to be working now. Looks good on the docs side. Just be sure to get a Tech LGTM and we can merge this.

@pmorie @mburke5678 Does this look good to you on the tech side of things?

LGTM!


⚠ Samsung has degraded performance

In 8def683, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2701 ms

Resolved: Samsung performance has improved in 2228bc8.


Create helm releaser for ortelius docs repo

add the following to the GitHub actions
.github/workflows/helm-chart-releaser.yml
`name: Release Charts
on:
push:
branches:
- deploy
paths:
- 'chart/**/Chart.yaml'
jobs:
release:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v2
with:
fetch-depth: 0
  - name: Configure Git
    run: |
      git config user.name "$GITHUB_ACTOR"
      git config user.email<EMAIL_ADDRESS>  - name: Install Helm
    uses: azure/setup-helm@v1
    with:
      version: v3.4.0

  - name: Run chart-releaser
    uses<EMAIL_ADDRESS>    with:
      charts_dir: chart
    env:
      CR_TOKEN: "${{ secrets.GITHUB_TOKEN }}"`

itisaby will do this

This has been done


Update jquery.fileupload-image.js for browserify

to make it work with browserify, module "load-image" does not exist

Thanks for the contribution, @ningliu.


mcumgr timeout due to smp_shell_process stalling

smp_shell_process only tries to get one net_buf from the FIFO:
https://github.com/zephyrproject-rtos/zephyr/blob/af10c06885c92321ff947ab83c3abcf8f0281cf3/subsys/mgmt/mcumgr/smp_shell.c#L140
If there have been more than one fragment received, these will only be processed on the next call to smp_shell_process.
This results in timeouts when using firmware image upload using mcumgr, as the system will fail to process the frame, until the mcumgr CLI tool runs into a timeout (default 3s) and resends the frame.
There should probably be a loop in this function, or alternatively a return value that will tell the caller that the function should be called again.
Note: this can only be reproduced with a short delay between fragments sent by the mcumgr CLI tool.
See https://github.com/apache/mynewt-newtmgr/issues/180. A delay <5ms with a baud rate of 460k shows the problem here, but of course YMMV depending on system clock etc.

Thanks @sw! You are certainly right, packets should be processed in a loop. Could you check if #34897 solves the issue?

That was quick. Thank you @mniestroj , that does indeed solve the issue.


[Watson][Color Picker] FATAL_USER_CALLBACK_EXCEPTION_c000041d_ColorPickerUI.dll!ColorPicker.Keyboard.KeyboardMonitor.Hook_KeyboardPressed

Microsoft PowerToys version
0.35.0
Running as admin
Area(s) with issue?
ColorPicker
Steps to reproduce
https://watsonportal.microsoft.com/Failure?FailureSearchText=4d183c04-e61e-8998-680e-ab21f1fb1f96
Occurrences: 35
NOTE: the Total number reported in the bug report is not filtered by version. A similar crash was causing 200-300K event per month that was fixed https://github.com/microsoft/PowerToys/issues/9573
This seems to be a different crash in the same function.
✔ Expected Behavior
None
❌ Actual Behavior
None
Other Software
No response

This crash is no longer present in 0.37.x


[FIX] Accept stream body #25

Change Summary
Accept a stream as a body.
PR Checklist

[x] I have read and signed the Contributor License Agreement.

Is it possible to check the PR and merge it to master?

Please ?

Thanks !


[RFC] Extract default code component properties

What's the problem? 🤔
When copying a demo from the MUI docs, none of the properties are bindable. This makes the component less usable out of the box as it can't be interacted with on the page. Currently the user has to add this functionality themselves after importing by wrapping it with createComponent.
What are the requirements? ❓
No response
What are our options? 💡
No response
Proposed solution 🟢
When a user creates a component that hasn't been wrapped with createComponent we will generate a default argTypes definition. We will try to extract it from:

propTypes if they exist on the component
the typescript types

As soon as the user adds their own createComponent we will ignore this default.
The extraction needs to happen at build time, when the component is defined, not at runtime. Reasons are:

In future optimizations, the runtime won't have type information attached
It will be too expensive, the bundle will need to contain a full parser to extract the information from the source

We can likely try using extract-react-types for this. It looks like it can be made to work in the browser.
Alternatively we should investigate if we can run react-docgen in the browser.
Resources and benchmarks 🔗
No response

Not sure if it relates to it, but I encountered a use case today. I imported MUI pagination component and wanted to use the current page number prop from this custom component into my application to render some assets, but couldn't do it.

I imported MUI pagination component and wanted to use the current page number prop from this custom component into my application to render some assets, but couldn't do it.

Would the following work for you?
import * as React from "react";
import Pagination from "@mui/material/Pagination";
import Stack from "@mui/material/Stack";
import { createComponent } from "@mui/toolpad-core";

interface BasicPaginationProps {
  count: number;
  page: number;
  onChange: (value: number) => void;
}

function BasicPagination({ count, page, onChange }: BasicPaginationProps) {
  const handleChange = React.useCallback(
    (event, newPage) => onChange(newPage),
    [onChange]
  );
  return (
    <Stack spacing={2}>
      <Pagination count={count} page={page} onChange={handleChange} />
    </Stack>
  );
}

export default createComponent(BasicPagination, {
  argTypes: {
    count: {
      typeDef: { type: "number" },
      defaultValue: 10,
    },
    page: {
      typeDef: {
        type: "number",
      },
      onChangeProp: "onChange",
    },
  },
});

Yes, it does. Thanks. App link


How to run BigchainDB local in 4-node

I did those steps to have a BigchainDB 1 node on local:
git clone https://github.com/bigchaindb/bigchaindb.git
cd bigchaindb
make run

But how to start a 4-node BigchainDB, not mentioned in README.md

One option is to follow the "Simple Deployment Template" in the docs.


Java codes

Dictionaries and Hashmaps > Two Strings
Dictionaries and Hashmaps > Ransom Note

Add read me too..thenci will merge

Done now


Blocking Call(sh): Timeout

I have:
{
    "fmt_cmd": ["goimports"]
}

goimports takes 1.032s to run.
I see in the console:
** 2015-01-09 11:46:11.587429 **:
GoSublime
cannot fmt file. error: `Blocking Call(sh): Timeout'
--------------------------------

Writing file /home/pwaller/.../foo.go with encoding UTF-8

I ran into this issue too.
Sometimes goimports will take a while to run (it depends on some factors, like total number of Go packages in GOPATH, whether new imports need to be added, etc.).
As a result, with the default timeout of 1 second, sometimes (but not other times), when I Cmd+S and expect the file to be gofmted, that is not the case.
I rely on Cmd+S to always format my Go code, and these arbitrary potential failures are not okay - it can mean I can accidentally commit unformatted Go code without noticing.
I can increase ipc_timeout to 2 seconds or 10 seconds or 60 seconds to avoid this, but that isn't optimal. If goimports (or whatever fmt_cmd I have set) takes 61 seconds, I don't want it to (largely invisibly) not do anything.
Additionally, for all other users that don't realize this potential pitfall and use default settings (which results in a timeout of 1 second), they have a non-zero probability of running into this too and therefore committing unformatted Go code, etc.
I think the fmt_cmd process should be an exception and executed without any timeout. If it takes too long to run, the user should fix the problem (by using a faster gofmt, or removing many packages from their GOPATH, or make a contribution to goimports project to make it faster, etc.). There should be a 100% guarantee that if you press Cmd+S that it will eventually format your code, without a chance of it deciding not to (because of timeout).

I also ran into this issue, and was wondering why sometimes my files weren't being formatted. I agree with @shurcooL that it should never fail, and that users should rather seek to make the process faster if it is taking too long.

Also ran into this. Agree with @shurcooL (your fix worked for me - thanks!)

I've been running into this too lately :( @shurcooL's fix worked for me.
I agree it's not good that goimports takes so long to execute.. but having it not run at all is even worse! I look really silly committing unformatted code :grin:

If anyone has any suggestion on a way GS can work better here, please feel free to share. As things stand, due to limitations in ST, I can't fmt the file without it being a blocking call. This means that for the entirely of the call to margo the ST ui is frozen. I think goimports should simply be fixed. IIRC there was talk of possibly running daemon so it could cache stuff, but I don't use goimports and haven't kept up with the discussions. If more support is required from me and goimports provides a package (not binary) that will do the caching, then I'm open to including it as part of the margo binary the same way gocode is.

If anyone has any suggestion on a way GS can work better here, please feel free to share.

My suggestion is this. When running the on-save formatting command (configured as either gofmt or goimports, run it without a timeout (or with a really long timeout, like 30 seconds or longer).
If goimports is slow, users should deal with that by making it faster (e.g., by contributing performance improvements to the tool, or by reducing number of unneeded Go packages they have in their Go workspace, or by switching to gofmt).

As things stand, due to limitations in ST, I can't fmt the file without it being a blocking call. This means that for the entirely of the call to margo the ST ui is frozen.

That's fine by mine. It's what I expect, I press save, and the format tool runs until it completes, without timing out.

I've chosen to improve the documentation: https://github.com/DisposaBoy/GoSublime/commit/43858c654c41a233d0fff2bd7922ab587b4ebaa6
If it can be made clearer, feel free to make suggestions.
I can't make the timeout infinite - that's crazy. If it wasn't clear: this setting isn't something special for gofmt, it applies to every blocking ipc call including ones triggered automatically by Sublime Text e.g. completion - and I don't plan to add any more special cases for goimports unless it is to be supported officially.
I decided not to increase it because this interacts badly with other plugins and I've seen Sublime Text crash when frozen for a little while before.
I also considered making the error more noisy, but decided against it because fmt fails all the time, e.g. on syntax errors and it'd be very annoying if it's always in your face.

I can't make the timeout infinite - that's crazy. If it wasn't clear: this setting isn't something special for gofmt, it applies to every blocking ipc call including ones triggered automatically by Sublime Text e.g. completion

Yeah, that wouldn't be appropriate. I meant it should be "infinite" (effectively) only for the command that formats code on save, because it's very important. Not other commands like getting autocompletion.

I don't plan to add any more special cases for goimports unless it is to be supported officially.

What do you mean by officially? It started off as Brad Fitzpatrick's personal project on github.com/bradfitz/goimports, but has since moved to Go external standard library at golang.org/x/tools/cmd/goimports. Are you saying it's only official if it's moved into internal standard library, like cmd/goimports?

I also considered making the error more noisy, but decided against it because fmt fails all the time, e.g. on syntax errors and it'd be very annoying if it's always in your face.

Is it possible to tell the difference when the command fails (because of invalid Go code), or when it's killed by the ipc_timeout setting? If so, you could consider displaying a message like "WARNING: You pressed saved and thought your code would be gofmted, but it wasn't because 1 second passed and the formatting tool hasn't finished yet!". I think even something as annoying as that would be better than silently not formatting one's code on timeout. But it's because I think it's absolutely unacceptable for a Go code editor to silently not gofmt code on save under certain situations.

If it can be made clearer, feel free to make suggestions.

Looks like a good start, thanks for doing that!
I think it's unfortunate that the default GoImports behavior can't be optimal, but I suppose it's fair to require people to configure a higher ipc_timeout when they configure goimports as their gofmt tool.

https://github.com/DisposaBoy/GoSublime/commit/dd981730f13b3a146d256271ce94bad557155802 adds the timeout value to the bcall error message well as mentioning the ipc_timeout setting.
https://github.com/DisposaBoy/GoSublime/commit/668409c1a669b3b339cd1fd548b36e5365a0066c shows a popup if fmt fails for any reason.
@shurcooL

What do you mean by officially?

By officially, I mean officially as part of GoSublime/MarGo.

Is it possible to tell the difference when the command fails (because of invalid Go code), or when it's killed by the ipc_timeout setting?

It's possible to detect the timeout but I don't think it helps, because the fmt can fail for other reasons as well. It's best to simply be aware that it can fail at all.

Ok, thanks. From my perspective, I consider this issue being resolved and could be closed. Thanks for doing that work!


Update environment-variables.md

Needs a new section for config env vars.

Needs a new section for config env vars.

What exactly did you mean here?

The first section shows variables you can use in code that are set by razzle.
Should have another section that shows variables you can set to affect razzle behaviour and config.


Migrate from travis-ci.com to travis-ci.org

Now that Aikido is public we should migrate from travis-ci.com to travis-ci.org. This was a bit of a painful process for or_fcl, so I am holding off until we are done with the upcoming merges.

Fixed.


Ender Carrot Tooltip

Minecraft version & Mod Version:
1.12 - 1.5.11
Single player or Server:
Both
Describe problem (what you were doing / what happened):
The tooltip of the Ender Carrot is missing/invalid. It shows as `item.hores_upgrade_jump.tooltip'
Crash log (if any):
Nope

You should be a QA tester, good eyes!

fixed in 1.12-1.6.0.    https://minecraft.curseforge.com/projects/cyclic/files/2449460


Update Rust to make a release.

Updates the rust version to make it pass github actions.

Thanks so much! I ran out of steam to fix this last night.


Should builder methods remove their prefixes?

With the split of the builders into per-version kinds (see #2842 and #2851), I feel there's a new question around whether we should remove the prefixes from the method names. They used to exist because the builders were combined over multiple HTTP versions, so the options were scoped to a version. But now:
mod http1 {
    impl Builder {
        pub fn http1_title_case_headers() {
            // ...
        }
    }
}

It seems all those can have the http1_ prefix removed, and same for the http2::Builders.
Noticed in https://github.com/hyperium/hyper-util/pull/11#discussion_r1035315089.

Leaving for comments for a few days.

So, http1::Http1Builder? Seems contrary RFC 356. Users who want it can rename it on import: use hyper::client::conn::http1::Builder as Http1Builder.

Oh didn't even know that existed! Yeah seems fine to me.

The APIs are already inconsistent today (e.g. there's http1_title_case_headers but also max_buffer_size on http1::Builder).
Agree with this proposal, the prefixing is now an indirect stutter-naming (conn::http1::Builder::http1_...).


Make alluxio image configurable

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

fixes #381
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #380 (29f6141) into master (0044289) will increase coverage by 0.38%.
The diff coverage is 48.57%.

@@            Coverage Diff             @@
##           master     #380      +/-   ##
==========================================
+ Coverage   17.76%   18.15%   +0.38%
==========================================
  Files          52       53       +1
  Lines        2212     2242      +30
==========================================
+ Hits          393      407      +14
- Misses       1780     1796      +16
  Partials       39       39

Impacted Files
Coverage Δ

pkg/ddc/alluxio/transform.go
0.00% <0.00%> (ø)

pkg/utils/docker/image.go
44.44% <44.44%> (ø)

pkg/ddc/alluxio/utils.go
30.92% <50.00%> (+3.21%)
:arrow_up:

pkg/ddc/alluxio/transform_fuse.go
67.64% <100.00%> (-0.93%)
:arrow_down:

pkg/ddc/alluxio/transform_optimization.go
96.73% <100.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 0044289...29f6141. Read the comment docs.


[Question] Is it possible to load dashboards?

Is it possible to load dashboard in via this provider?
Previously we have used the restapi provider here
Is this possible?
Thanks

Dashboards are saved objects so you may be able to use kibana_object to load them.


next/release

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


"Docs plugin enabled" detection no longer does anything

Because theme-common has a direct dependency on plugin-content-docs, it is no longer possible to make a site without ever installing content-docs (previously discussed in #3360). Not really a big deal because the user can still install theme-classic and plugin-content-blog and pretend she never pulls in content-docs transitively, but it's still not desirable.
We have some checks in our codebase about whether docs is enabled:
https://github.com/facebook/docusaurus/blob/c8d6c7e38685598869863b59f9d05197f8558868/packages/docusaurus-theme-common/src/utils/docsUtils.tsx#L32-L33
https://github.com/facebook/docusaurus/blob/c8d6c7e38685598869863b59f9d05197f8558868/packages/docusaurus-theme-classic/src/theme/NavbarItem/index.tsx#L18-L38
Because we are now always directly importing these hooks from @docusaurus/plugin-content-docs/client instead of letting the docs plugin register theme aliases, these things always statically exist. We should either remove the checks, or dynamically import content-docs/client everywhere (better IMO in terms of bundle size)

This is my major dissatisfaction with the ongoing refactors. Everything gets intertwined and our architecture is getting more and more monolithic despite being allegedly "modular".

Yeah, I'm mainly talking about directly importing @docusaurus/plugin-content-docs in the theme, not the fact that the theme provides the docs components.
The theme aliases somehow work like inversion of control. Instead of having direct dependency relationship, the alias allows them to communicate through a shared protocol where plugin provides the data and theme provides the components. There's no hard dependence in this relationship and unplugging anything naturally works.
I'm not asking to split the theme—just that importing from @docusaurus/plugin-content-docs in theme-common has made this coupling much tighter than before. Lazy-loading could fix it.

https://github.com/facebook/docusaurus/issues/3360 was about a different issue: a blog would not build if the user is using docs: false (despite package being installed => the plugin needed to be registered even if unused)

Ah, I understand now. Scratch the point about skipping installation of a package (maybe we still can in the future with optional peer dependencies, but we have been bitten by peer dependencies, so not in the near future). Bundle size is still a valid claim, though

The theme aliases somehow work like inversion of control. Instead of having direct dependency relationship, the alias allows them to communicate through a shared protocol where plugin provides the data and theme provides the components. There's no hard dependence in this relationship and unplugging anything naturally works.

I understand that, but at the same time it can become quite tedious to maintain the "contract" between content plugin and theme this way.
Remember most of these docs apis are internal and undocumented. Adding indirection only increase maintenance cost from us.
These apis are not really meant to be overridden with aliases. If user is unhappy with a behavior, they can swizzle the component and provide a brand new implementation using documented apis like useGlobalData.

I'm not asking to split the theme—just that importing from @docusaurus/plugin-content-docs in theme-common has made this coupling much tighter than before. Lazy-loading could fix it.

I don't understand why it's worse to import it in theme-common compared to theme-classic?
Maybe you are against both, and I can understand that.
IMHO coupling isn't always a bad thing. Here it's done on purpose to avoid the extra cost of an unnecessary indirection layer. Unless you can give a strong use-case for introducing back this indirection? (pragmatic use-case, not theoretical)

The use-case is that now theme-common is explicitly dependent on content-docs instead of implicitly through theme aliases, and there's no way to rid content-docs/client from the client bundle now.

Then if a user reports using theme-common without using docs (unlikely 🤷♂ ) and complains about bundle size, we can look into it by either introducing back indirection or ensuring tree-shaking works better.
I don't think this is valuable to dedicate much time to this atm, but conceptually I agree.

Yes, that's basically the idea. Quite annoying (because it signals smelly architecture) but not high-priority. Just here to raise awareness and discuss some solutions.

I guess we can close this now that theme-common does not depend on content plugins anymore
(at least it will be the case in v4 soon: see https://github.com/facebook/docusaurus/pull/10316)


Could we use Grad-CAM to get a heatmap?

hi, I'm a beginner, this question may be not professional，but it puzzles me。it seems that Grad-CAM can also get class heatmap, why not use it to visualize abnormal information?

Hey. You can use Grad-CAM to get an "abnormal class heatmap"; it's another gradient-based explanation method. However, you lose some advantages of FCDD, as FCDD's explanations are directly tied to the anomaly score. Feel free to try it, though.

okay, thank you for your patient answer.


Tuxera: Invalid pref. file & can't uninstall

After running brew cask install tuxera-ntfs, I get an 'Invalid Preference File' error in the preference pane, which prevents from authenticating the license key.  In addition, brew cask uninstall tuxera-ntfs doesn't work.
$ brew cask uninstall tuxera-ntfs
==> Running uninstall process for tuxera-ntfs; your password may be necessary
==> sudo: /System/Library/Filesystems/fusefs_txantfs.fs/Support/uninstall-packag
Error: Command failed to execute!

==> Failed command:
["/usr/bin/sudo", "-E", "--", "#<Pathname:/System/Library/Filesystems/fusefs_txantfs.fs/Support/uninstall-package.sh>"]

==> Output of failed command:

==> Exit status of failed command:
#<Process::Status: pid 2020 exit 1>

I suspect this must relate to the 2014 -> 2015 update?  The SHA also isn't being checked in the formula, so perhaps a change went unnoticed?

Seems like that file doesn’t exist anymore. Fixed uninstall. brew update and try again.


[scripts] Add extensions to all script files

This will make it easier to add
shellcheck scripts/*.sh into our pipeline without having to have more
complicated file selection logic anywhere.
Signed-off-by: Steven Danna<EMAIL_ADDRESS>

ℹ Took the liberty of rebasing this, with the intention to merge when green.

...uhm maybe not: https://buildkite.com/chef/chef-automate-master-verify-private/builds/2865#27bebdf0-f8fe-4bc6-a837-e81b8239fe59/135-211 :/

Or maybe not not. Botched up the rebase, better now. 🤞


[AIRFLOW-XXX] Redirect FAQ about airflow[crypto] to How-to Guides.

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-XXX
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a Jira issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

Update Apache Airflow (incubating) Documentation, redirect FAQ about airflow[crypto] to How-to Guides.
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[x] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3864 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3864   +/-   ##
=======================================
  Coverage   77.51%   77.51%
=======================================
  Files         200      200
  Lines       15815    15815
=======================================
  Hits        12259    12259
  Misses       3556     3556

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 abb12be...2963b28. Read the comment docs.

LGTM


consolidate unit testing framework

Since at least 61cdad8, there is a mix of unit tests written with unittest and pytest.
The CI currently only runs the pytest.
In the short term, at least both sets of tests need to be running in the CI. The, a sinfle unit testing framework needs to be selected and used for all tests.

I had missed that pytest runs all unittest tests by default -> this can be closed


mgmt, prepare for May release

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Text in white on home page not quite visible

🐞 Bug Report
Describe the bug
"Station Saarthi: Your platform guide" is written in white color.
Steps to Reproduce
Steps to reproduce the behavior:

Go to website
See error on home page.

Expected behavior
To be more visible, it should be written in black.
Actual behavior
The text is written in white which is not looking good at a light background.
Screenshots or Video

@dhairyagothi pls assign the issue to me

what colour you will do then ??
make it visible use tailwind

what colour you will do then ??
make it visible use tailwind

Ok, I will see to it and will try to make it more clear

@dhairyagothi pls assign this issue to me
I will change it to

You can suggest any other desired change if it does not look good to you.


YWeather Condition & Current sensor not showing anything

Since around version .58 2 of the YWeather sensors have stopped working
Condition & Current
Just updated to .60 and still have the same issue.

Same problem.
But I do not know if it's a problem with the home assistant component or the Yahoo.
Because if trying to go to the image shows an error
sensor.yweather_condition | Sunny |
attribution: Weather details provided by Yahoo! Inc.
friendly_name: Prevision
entity_picture: https://l.yimg.com/a/i/us/we/52/32.gif

Regards

Could be a duplicate of #11120.


Remove immutable from status and auth

Summary
First step on closing #3853
Test plan
Run existing tests

Hm... seems like something's not ok

@erezrokah any idea why e2e tests are failing?

Alright, found the reason


Incorrect use of INT8_MAX instead of UINT8_MAX during login

Hi all,
It appears that the login process is using INT8_MAX (127) for a check involving a seed length that is always a positive number. However, this range is insufficient, as the values being processed during login can exceed the upper limit of INT8_MAX. For instance, the seed length is returning a value of 149, which falls outside the allowable range of INT8_MAX.
Proposed Solution
To address this issue, I propose replacing INT8_MAX with UINT8_MAX (255). This change would properly accommodate the positive-only range required for this check and resolve issues related to exceeding the maximum limit.
As my server always uses ClassicUO Web, which sends an essential packet with a length of 149, this modification would resolve the issue for anyone using CUO Web. Given that CUO Web is likely to become more widely adopted in the future, this adjustment ensures compatibility going forward.
Steps to Reproduce

Attempt to log in with a setup that processes a seed length exceeding 127.
Observe that the seed length check fails due to the INT8_MAX limitation (e.g., during a breakpoint in Visual Studio).

The expected behavior is that the check should handle values within the range 0–255 without any issues.
Where to change: CNetworkInput.cpp
bool CNetworkInput::processUnknownClientData(CNetState* state, Packet* buffer)
{
    // process data from an unknown client type
    ADDTOCALLSTACK("CNetworkInput::processUnknownClientData");
    EXC_TRY("ProcessNewClient");
    ASSERT(state != nullptr);
    ASSERT(buffer != nullptr);
    CClient* client = state->getClient();
    ASSERT(client != nullptr);

    bool fHTTPReq = false;
    const uint uiOrigRemainingLength = buffer->getRemainingLength();
    const byte* const pOrigRemainingData = buffer->getRemainingData();
    if (state->m_seeded == false)
    {
        fHTTPReq = (uiOrigRemainingLength >= 5 && memcmp(pOrigRemainingData, "GET /", 5) == 0) ||
            (uiOrigRemainingLength >= 6 && memcmp(pOrigRemainingData, "POST /", 6) == 0);
    }
    if (!fHTTPReq && (uiOrigRemainingLength > INT8_MAX)) // Should this be UINT8_MAX instead?
    {
        g_Log.EventWarn("%x:Client connected with a seed length of %u exceeding max length limit of %d, disconnecting.\n", state->id(), uiOrigRemainingLength, INT8_MAX); // Same here?
        return false;
    }

Please let me know if my understanding is incorrect or if there are additional considerations for this change.
Thank you!

This is not at bug, but the network logic need to be change for support this new "login logic". Because the CUO web identity send the WebIdentity packet at the initial login phase, and sphere dont have support for this logic.
So, dont need to change the type. Only update the network logic for this


Cleaning up the reserved IPAM, in case of plugin failure in Multus CNI.

If there's malformed configuration in the 'delegates' table in the Multus configuration file and at least one Flannel plugin instance is working correctly, Multus will keep creating interfaces and reserving consecutive IP addresses from the subnet range specified in the Flannel configuration. This will eventually lead to the preservation of entire range from specified subnet and will block the creation of any new K8s pods with Flannel interface.
Moreover, we haven't found any recovery actions after failure, that would do automatic cleanup by itself. We had to remove leftovers after failed pod creation attempts manually by removing failed files and directories from /var/lib/cni/networks/ directory.
Noteworthy 'restartPolicy' option in the pod configuration was set to 'Never'.
Steps to reproduce:

Create Multus configuration file containing at least one properly configured Flannel entry and at least one malformed entry. Example:

cat /etc/cni/net.d/multus-cni.conf
{
    "name": "minion1-multus-demo-network",
    "type": "multus",
    "delegates": [
        {
                "type": "flannel",
                "masterplugin": true,
                "delegate": {
                        "isDefaultGateway": true
                }
        },
        {
                "type": "foobar"
        }
    ]
}

Restart kubelet service.
Create new pod.

This may be naive (I have not looked at the code, just from use of it in testing), but my understanding is that multus with two networks, A and B, would create an interface by calling CNI plugin A then B. On a failure of B, could multus not simply call A to delete the provisioned address?
Note that I'm seeing this issue too, but it's not blocking my testing in the short term.

Yes, you are correct, I will give the solution by next week for this.

@plwhite please check the bug fix #4. In the case of failure in any one of the minion plugins, Multus CNI will clear all the network resources allocated by the previous plugins.

I won't pretend to have done a comprehensive test, but I have tested and it appears to fix the issue. My test case was just to have fouled up my SR-IOV installation so that no interfaces were available, which in the past would lead to flannel cycling blowing up and using all IPs in the flannel network in rapid succession. That error case no longer occurs.
Thanks for the quick turnaround on this one!

Thanks @plwhite. I will close the issue

@plwhite We created a pull request in cni community https://github.com/containernetworking/cni/pull/379 to add mutlus CNI as the 3rd party plugin. As a Multus CNI user welcome your feed back there.


Enable SOVERSION soft links for library targets

Proposed Change:

Enable SOVERSION/soft links for library targets

Please remove internal ticket numbers from git log. This is a public repository.

Thanks for the comments. Please check the updated


feat(NumberTheory/PrimeCounting): Lemma: The nth prime is greater or equal n + 2.

Possibly useful lemma about the nth prime number.
Co-authored-by: Pol'tta / Miyahara Kō<EMAIL_ADDRESS>Co-authored-by: Thomas Browning<EMAIL_ADDRESS>

I misunderstood that a non-reviewer could ask a maintainer to merge with the command “maintainer merge”.
I am not a reviewer and apologize for the review even though I do not have the authority to do the merge.

Your review is welcome! I'm surprised how much you could shorten the proof. Thanks!

@Komyyy Speaking as a member of the reviewer team: we encourage and are grateful for every reviews! It makes our work easier and also let PR authors wait less. If you think this PR (or any other one) is good to go, give it an "approve" and a reviewer / maintainer will get to it soon.

I wonder if we should have n + f 0 <= f n if f : ℕ → ℕ is StrictMono.

I wonder if we should have n + f 0 <= f n if f : ℕ → ℕ is StrictMono.

Or slightly more generally:
lemma StrictMono.add_le {f : ℕ → ℕ} (hf : StrictMono f) (m n : ℕ) : m + f n ≤ f (m + n)  := by
  rw [add_comm m, add_comm m]
  induction' m with m ih
  · rw [add_zero, add_zero]
  · rw [← add_assoc, ← add_assoc, succ_le]
    exact ih.trans_lt (hf (n + m).lt_succ_self)

/-- The `n`th prime is greater or equal to `n + 2`. -/
lemma add_two_le_nth_prime (n : ℕ) : n + 2 ≤ nth Prime n :=
  zeroth_prime_eq_two ▸ (nth_strictMono infinite_setOf_prime).add_le n 0

Apparently the name change in the last commit clashes with the PR comment? What to do?

OK, so the fail is ignored. Let's see...

LGTM.
maintainer merge

Thanks!
bors merge

Wait what?


Documentation ?

Is there any documentation on what any of these libraries does ?

http://netoffice.codeplex.com/


[Suggestion][theme support] remove  class="h-100" from body for better theme support

I suggest removing  class="h-100" from the <body> tag for better theme support.
When class="h-100" is set, the page will be white colored background, once you scroll down (over the device height)
To fix this, simple remove the class.
#987

If you remove that the Editor area is not taking the max height, which I want to keep.

oh well...
i think i just found a better solution than I suggested yesterday.
in css, just add 'height: 100%; to the body tag'.
This will prevent half of the page having a white background even tho a black admin theme is set

have you check the editor if has the max height ?

in css, we should be able to just add 'height: 100%; to the body tag'

As far as I know, height works only if also width is set.


modelNameSuffix

Hi,
awesome project, thanks for putting in the hours!
The openapi-generator-maven-plugin has a configuration property called modelNameSuffix which, as the name suggest, allows the user to append a suffix to the generated model classes.
Do you thing such a feature would be valuable in your project as well?
Example
Swagger:
components:
  schemas:
    FooObject:
      type: object
      properties:
        bar:
          type: boolean

Mapping:
openapi-processor-mapping: v2

options:
   package-name: example.gen

   modelNameSuffix: Dto

Would result in the generation of example.gen.model.FooObjectDto.java

Funny, haven't thought of that.  :-)
Yes, I guess this would be useful. That way we can avoid the postfix "noise" in the openapi description, which is our current strategy at work.. ;-)

@aegliv I think it should work now. Tests pass but I haven't tried it on a real project yet.
You can try the latest openapi-processor-spring snapshot (2021.4-SNAPSHOT) (or openapi-processor-micronaut 2021.2-SNAPSHOT). Just update the version.
The mapping looks like this:
openapi-processor-mapping: v2

options:
  package-name: generated
  model-name-suffix: Resource

It will ignore the suffix if the schema already ends with the suffix to ease migration.

Thanks for the quick implementation! But where did you release your snapshot version? Can't find it on mvn central...

Hi @aegliv,
it is on maven central (https://oss.sonatype.org/#nexus-search;quick~openapiprocessor), the central snapshots repository is: https://oss.sonatype.org/content/repositories/snapshots
if you are using the gradle plugin (2021.3) it should automatically find it (the plugin adds the snapshots repository), if you are using the maven plugin (2021.1) you probably need to add the repository.
    <pluginRepositories>

        <pluginRepository>
            <id>openapi-processor-snapshots</id>
            <name>openapi-processor-snapshots</name>
            <url>https://oss.sonatype.org/content/repositories/snapshots</url>
        </pluginRepository>

    </pluginRepositories>

release with 2021.4

Thanks, works like a charm!
Sry I could not test it earlier but my company's firewall did not let me download your SNAPSHOT version

Thanks, nice to hear that it works for you :-)


Clean up styles.js

Convert the static styles in styles.js into functions that take the current theme as an argument instead of a static import. The default export of styles.js will still be the styles with the defaultTheme provided, so this doesn't break anything.
Remove the need to add dynamic styles manually to providers.

@chrispader mind updating the description here for your open PR?

I can't update the issue description because i'm not the issue owner 🥴

gotta love github

not overdue

@grgia is this done? :)

@chrispader should we close?

yes 👍


refactoring of internal/pool

added xcontext.WithDone()
added xsync.{OnceFunc,OnceValue}

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[x] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report
Attention: Patch coverage is 52.46479% with 135 lines in your changes are missing coverage. Please review.

Project coverage is 53.46%. Comparing base (bb1ee3b) to head (f8b9a73).

Files
Patch %
Lines

internal/pool/pool.go
58.55%
37 Missing and 9 partials :warning:

internal/query/client.go
55.84%
27 Missing and 7 partials :warning:

internal/xsync/once.go
11.11%
23 Missing and 1 partial :warning:

internal/xcontext/context_with_done.go
52.38%
7 Missing and 3 partials :warning:

internal/query/session_status.go
27.27%
8 Missing :warning:

internal/query/session.go
72.22%
5 Missing :warning:

internal/xerrors/check.go
25.00%
2 Missing and 1 partial :warning:

log/query.go
50.00%
1 Missing and 1 partial :warning:

metrics/query.go
75.00%
2 Missing :warning:

internal/query/transaction.go
66.66%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1125       +/-   ##
===========================================
- Coverage   67.41%   53.46%   -13.96%
===========================================
  Files         292      291        -1
  Lines       29053    27949     -1104
===========================================
- Hits        19586    14942     -4644
- Misses       8568    11963     +3395
- Partials      899     1044      +145

Flag
Coverage Δ

53.46% <52.46%> (+0.11%)
:arrow_up:

go-1.21.x
53.44% <52.46%> (-15.53%)
:arrow_down:

go-1.22.x
53.21% <52.46%> (-14.20%)
:arrow_down:

integration
53.46% <52.46%> (+0.11%)
:arrow_up:

macOS
?

ubuntu
?

unit
?

windows
?

ydb-23.3
50.15% <17.25%> (+0.16%)
:arrow_up:

ydb-24.1
53.38% <52.46%> (+0.15%)
:arrow_up:

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

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


Fix FAQ Typo

Removed an unnecessary word from one of the questions in the FAQ section.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@marcusclayton : Thanks for your contribution to the Azure documentation! The author, @bandersmsft, has been notified to review your proposed change.

@marcusclayton Suggestion looks good. Ty!
#sign-off


⚠ EPCC Public Website has degraded performance

In b1faf89, EPCC Public Website (https://www.epcc.edu/pages/proxy.aspx) experienced degraded performance:

HTTP code: 200
Response time: 3269 ms

Resolved: EPCC Public Website performance has improved in 1eb0c5b after 12 minutes.


Suggestion: rename or alias fireImmediately to fireOnStart

I think the naming of the options startImmediately and fireImmediately may cause confusion. A more clear name for the option that sets the callback to fire immediately upon timer start could be fireOnStart or fireImmediatelyOnStart. This is a small detail, but worth considering.

Great suggestion. It would be an API breaking change though, so not sure it's worth breaking the code of everyone using this by changing it. If I make any other major changes to the library, this will definitely go along with them.

Done. Great suggestion.

Lovely <3
------- Original Message -------
On Monday, January 16th, 2023 at 12:33 AM, Justin Mahar @.***> wrote:

Closed #14 as completed.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


How is import.meta.url assigned?

Feature request
More of a series of questions than a feature request. How is import.meta.url assigned? Why are you unable to see the assigned value in a debugger like Chrome's? Why is import.meta.url only available in esm modules? The documentation doesn't explain this and it seems "magical" this variable is only available when you set scriptType to "module" and turn on the experiments.outputModule flag.

import.meta.url is part of the ES Modules spec, it is not a webpack specific feature. I recommend reading the docs about about import.meta https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/import.meta


Fix CleanURL override

https://forum.babylonjs.com/t/is-there-a-switch-here-to-turn-off-cleanurl/50144/3

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15056/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15056/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15056/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15056/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15056/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15056/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15056/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15056/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15056/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
Important - these might fail sporadically. This is an optional test.
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15056/merge/testResults/webgpuplaywright/index.html

This won't change is a pure ESM environment. A high-level getter/setter is needed (See https://github.com/BabylonJS/Babylon.js/blob/master/packages/dev/core/src/Misc/fileTools.ts#L914 for example)


ui: Waivers - Profile list reflects the new Waived Status of a profile

User Story
As an Automate User, I will use this work when I want to see which profiles have been waived.
Problem Statement
In the rare case, if a profile has all waived controls within it, we need to show that the profile itself has been waived.
Definition of Done

A user can see if a Profile is waived based on the new status that the API returns for a profile
A user can filter by status of Waived

Mock Ups

this work involves:

adding the "waived" icon for profiles with a status of waived:

ensuring the logic for showing status of a profile accounts for waived profiles
add status filter buttons to the reporting profiles list page, as shown above. api response already has the info:

curl -s --insecure -H "api-token: $ACCEPTANCE_TOKEN" https://a2-local-inplace-upgrade-acceptance.cd.chef.co/api/v0/compliance/reporting/profiles -d '{"filters":[{"type":"status","values":["passed"]}]}'
{"profiles":[{"name":"apache-baseline","title":"DevSec Apache Baseline","id":"41a02784bfea15592ba2748d55927d8d1f9da205816ef18d3bb2ebe4c5ce18a8","version":"2.0.0","status":"passed"}],"counts":{"total":1,"failed":0,"skipped":0,"passed":1,"waived":0}}

load data by using the load_compliance_reports command from your studio dev env

status filters blocked by https://github.com/chef/automate/issues/3178


Added localization-Chinese-Taiwan - thanks to @ting10688

This localization is from https://github.com/MonitorControl/MonitorControl/issues/528
@JoniVR if everything is all right, please approve! Thanks!


Update Rust deps [dependabot]

Overview
Does not include bumping futures dependency.

See: https://jira.bf.local/browse/ECR-XYZ
Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

Coverage remained the same at 85.884% when pulling 4c643520680977da3cf4f25ec67ecec136b66b35 on vitvakatu:update-deps-01.12.19 into d613433a85f2998a611e1c724086c8a42254d5d4 on exonum:master.


ROUTE_ARGS_METADATA missing after using @Transactional decorator

When @Transactional is being used on a method, the method decorator @Body(), @Param(), and @Query() documentations will be gone later on the Swagger docs.
Below is my inspections:
Here is @nestjs/common injecting ROUTE_ARGS_METADATA to controller methods by using its method name as the key, the ROUTE_ARGS_METADATA itself is mainly stored on the target.constructor which is the class where the method is defined, this is going to be used later for @nestjs/swagger to explore available methods/endpoints and its object structure as well that can be found here
The problem is that @Transactional is overwriting the method without preserving the method name, this makes the createPipesRouteParamDecorator() function of @nestjs/swagger will  not find any kind of ROUTE_ARGS_METADATA that have been previously stored by @nestjs/common, because the key itself to retrieve ROUTE_ARGS_METADATA stored on the class is the method name

Hi @labibramadhan,
Thank you for the contribution!
I believe the issue you describe here, can happen to any decorator declared on the controller method, not just the Transactional.
So according to your suggestion, every decorator need to be changed if used together with NestJS. This does not sound reasonable to me.
Did you try to change the order of decorator declarations?
Meaning  the NestJS decorators close to the method and the transactional at the top - something like this:
@Transactional()
@Post() // first the nestJS decorator is fetching the metadata of the method params
async create(@Body() createCatDto: CreateCatDto) {
  this.catsService.create(createCatDto);
}

Yeah i tried to reorder and put the @Transactional to the first and then everything gone on the Swagger documentation, the @Post or @Get or @Put or @Delete metadata are missing after being replaced by @Transactional decorator. So when i put the @Transactional decorator to the first (applied last), all endpoints documentation doesn't exists on the Swagger documentation, but when @Transactional being put on the last decorator (applied first), the endpoint still exists on the Swagger documentation but missing the payload @Body or @Query or @Param documentation. So the @Transactional decorator is destroying metadata everything that comes before it.

Ok, I will try to find the time to investigate that a bit more, since there is no special code here, just a simple decorator.
It seems to me that no other decorator will work with NestJS and swagger, based on these findings. I am not sure the fix should be done here. But again, need to investigate that...
Cheers

I've also just hit this issue, looks like another 3rd party library may have faced something similar before. Found some links that may help:
nestjs issue: https://github.com/nestjs/nest/issues/1180
nestjs-config 3rd party library PR linked from the nestjs issue: https://github.com/nestjs-community/nestjs-config/pull/34

Seems there is already a PR to fix this from @labibramadhan https://github.com/odavid/typeorm-transactional-cls-hooked/pull/16

#16 Merged - Closing this issue.


Add generate_jwt function

I've added a new function of generate_jwt to the client this returns a token by default valid for 60 sec, this can be overidden using the validity keyword, I've also added placeholder keywords for acl and sub values for future use.
I replaced the code in __headers with this function just to reduce duplication too

This just requires a unit test now.


Incorrect Send Amount

We're now passing along an initial value for the sendableNanoBalance when creating a SendViewModel. This ensures a non-zero value for the sendableNanoBalance, upon which assumption the SendViewController relies. There was a case where if the user had a slow internet connection and we were not able to retrieve the sendableNanoBalance in time before the user entered a value, the view controller would set the balance to the maximum amount by comparing it to the default value of sendableNanoBalance of 0.

The NanoWalletCompany is no longer supporting this app. See https://nanowalletcompany.com/ for details. Closing.


added scroll listener to lock popovers from moving when scrolling

Overview
Updates the position of the components when scrolling so that they do not float with the page.
Fixes #1428
Screenshots (if applicable)
https://www.loom.com/share/8822694e00d64f8ca89cb72c67d51094
Documentation

[ ] Updated Typescript types and/or component PropTypes
[ ] Added / modified component docs
[ ] Added / modified Storybook stories

Thanks! The fix works great


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[rubysrc2cpg] Fix Returns on all Statements in Block-Helper

It was observed that in code such as
      define_method name do
        value = public_send("#{name}_value")
        unit = public_send("#{name}_unit")

        raise ArgumentError, "Invalid unit '#{INFINITY_UNIT}'" if !allow_infinite && unit == INFINITY_UNIT

        next Float::INFINITY if unit == INFINITY_UNIT

        next unless value.present? && unit.present?
        value.public_send(unit)
      end

Each statement was treated as a return statement. This breaks much of the CFG logic. Seems it was mostly a simple fix on how block helper compound statements were constructed.
Examples
Original AST
Fixed AST

Seems that this do-block is not treated as a closure, so this change seems to have some cascading effects on the CFG.


Migration schematics error in angular 18

Hi,
I have angular 18, trying to run output and query migration, its telling its available in 17.1 or 17.3 + versions but project having v18.
[ngxtension] output() is only available in v17.3 and later

Should be fixed with #388.

Waiting for the next release.

Fixed in https://github.com/ngxtension/ngxtension-platform/releases/tag/3.3.0


fix jemalloc conf lg_tcache_max restore default

Proposed changes
same as: #26362
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Unable to use phpwkhtmltopdf Debian 7

Hi,
i'm trying to use your classes, but i'm unable to :/
I installed with "php composer.phar require mikehaertl/phpwkhtmltopdf"
But in my project when i just put "use mikehaertl\wkhtmlto\Pdf;"
i have this error :
Fatal error: Class 'mikehaertl\wkhtmlto\Pdf' not found in /home/seven/webserver/devis/lettre_devis.php on line 21
I so tried to copy the vendor folder to my project, same issue,
i then tried to include the Pdf.php file but then i got this :
Fatal error: Class 'mikehaertl\wkhtmlto\Command' not found in /home/seven/webserver/devis/mikehaertl/wkhtmlto/Pdf.php on line 205
I tried to include command.php into Pdf.php but it result in blank page, i'm pretty sure it's something so easy that i can't see it, but well i can't see it -_-.
Could you help me?
Thanks,
Regards,
Martial.

This sounds like you did not include the composer autoloader.
https://getcomposer.org/doc/01-basic-usage.md#autoloading

Yeah thanks, that was it.
But now it do not save my pdf :/
$pdf = new Pdf('http://localhost/devis/testpdf.html');
$pdf->saveAs('./public/uploads/new.pdf'); (i also tried /tmp/ - /home/user/ ../../public/uploads/)

You should really check the README, which will answer your questions. About your error, check the error message you get from getError().

I'm on a managed cloud server from infomaniak, I installed wkhtmltopdf (supported by infomaniak)
but during the execution of the code always I have this error message: "Fatal error: Uncaught Error: Class 'mikehaertl \ wkhtmlto \ Pdf' not found"
`require_once DIR.'/vendor/autoload.php';
use mikehaertl\wkhtmlto\Pdf;
$pdf = new \mikehaertl\wkhtmlto\Pdf();
$pdf->addPage('/pdf_p1.html');
$pdf->send();`
Thank you

@asmabelhadj123 Something is wrong with the autoloading of composer in your setup. This has nothing to do with our library.

@asmabelhadj123 Something is wrong with the autoloading of composer in your setup. This has nothing to do with our library.

I resolved my problem
the root of wkhtmltopdf is "/usr/local/bin/wkhtmltopdf"


So there is a backdoor miner in this app

https://github.com/meetfranz/franz/blob/master/src/lib/Miner.js

the optional mining feature to receive a free premium account has since been long removed!

Not sure if it was removed, noticed from time to time franz cpu and memory usage gets crazy

Franz was minning in my computer today! (11-19-2019)

Before/After killing the app...


Explicity state encoding/decoding type

Current dependencies on/for this PR:

master

PR #10   (this PR)

This comment was auto-generated by Graphite.


filepicker_image_tag does not work with Cloudfront URLs

It returns 403 and does not match actual URL
Following the instructions I tried to set up cloudfront CDN  (seem my stackoverflow question here )
The cloudfront URL returned does not match the actual URL.
Filepicker URL:
https://www.filepicker.io/api/file/big-long-key
Cloudfront URL:
https://mycloudfronturl.cloudfront.net/big-long-key_filename.png
Right now, the only option is to query the filepicker URL for metadata and then save it myself.
Would suggest either deprecating the filepicker_image_tag for CDN urls or fixing to support the way URLs actually works

Hi @tarr11
Do you configured following this docs https://www.filepicker.com/documentation/cdn/ ?
As you can see here https://www.filepicker.com/documentation/cdn/use_cloudfront the path is the same changing only the domain. This is the actual behavior of the gem.

I believe that test is not modelling the actual behavior of filepicker.  Posting to filepicker returns URLs in this form:
https://www.filepicker.io/api/file/big-long-key
Your test assumes keys of this type:
https://www.filepicker.io/big-long-key

The test reflects the same behavior described here, changing only the path:

https://www.filepicker.io/api/file/N49i6hPRBeropWnCWOLw
https://d2nl1simbk63ds.cloudfront.net/api/file/N49i6hPRBeropWnCWOLw

This is the only thing that the gem does. I recommend you double check the CDN instructions https://www.filepicker.com/documentation/cdn/ and if the problem persist try reach the Filepicker support.

Closing since no more feedback here.

Just to follow up here, the problem is because the documentation recommends going directly from Filepicker's store to Cloudfront.  In my case, i store the files in S3, and then use Cloudfront backed by my S3 URL.  I believe this is a better design choice, since I do not have to rely on Filepicker as a primary store.
The gem works in the case where you go from Filepicker -> Cloudfront, but does not work in the case where you go from Filepicker -> S3 -> Cloudfront.  In this case, the URLs do not have /file/api directory prefix in either S3 or cloudfront, and the filename is not the random ID generated by FIlepicker, and this is what causes the error.