Use of Number.isInteger limits browser support

The Number.isInteger() method is not supported in Internet Explorer 11 and earlier versions. Looks like the use was added in version 1.18.11

MDN has this as a polyfill
  return typeof value === 'number' &&
    isFinite(value) &&
    Math.floor(value) === value;

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Number/isInteger#Polyfill

This was introduced in #345 and I'm sorry to say that the last couple of months doing ES6-only has made me completely oblivious to this - sorry.
@jeff-phillips-18 @spadgett does either one of you want to send a PR for this?

@rodneyrehm I will send a PR shortly

(...for anyone who cares (probably very few folks), also experiencing this problem on phantomjs 2.1.1.)

fixed in v1.18.12

To be honest, not sure this is right thing to do. Support of older browsers should be concern of consumer. I'd rather maintained list of required polyfills for some older (but still used) browsers.
Adding polyfill in own script isn't that hard, and if developer supporting older browsers, he will have file with other IE-related polyfills anyway.
In foresight, I agree that this is very small change, but in future more and more such changes can occur and we might end up with something bloated, like jQuery, packed with dozens of workarounds.

Thanks @rodneyrehm! Needed this fix for phantomjs as well.


Add a new proxy list

http://<IP_ADDRESS>/proxies.txt
https://api.openproxylist.xyz/http.txt
https://raw.githubusercontent.com/monosans/proxy-list/main/proxies/http.txt
https://raw.githubusercontent.com/zeynoxwashere/proxy-list/main/http.txt
https://raw.githubusercontent.com/Volodichev/proxy-list/main/http.txt
https://raw.githubusercontent.com/cybblog/proxy-list/main/https.txt
https://api.proxyscrape.com/v2/?request=displayproxies&type=http
https://raw.githubusercontent.com/UserR3X/proxy-list/main/http%2Bs.txt
adding some more here

u forgot to add ,

use https://jsonformatter.curiousconcept.com/#


Some node finder refactoring and optimization

Scope & Purpose
Some refactoring to reduce code duplication and reduce dynamic memory allocations in the optimizer.
Enterprise component: https://github.com/arangodb/enterprise/pull/514

[x] Strictly new functionality (i.e. a new feature / new option, no need for porting)

Testing & Verification
This change is a trivial rework / code cleanup without any test coverage.
Jenkins: https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/11127/
Jenkins Performance: http://<IP_ADDRESS>:8080/job/helper-perf-singleserver-simple-linux/1543/

https://<IP_ADDRESS>/view/PR/job/arangodb-matrix-pr/11159/


Replace cfdisk with mkdosfs for 32 KB cluster size

I used the cfdisk command found in this document previously, but after an update TwilightMenu++ shows a warning that game loading is slow if the SD card does not have a 32 KB cluster size. Using this mkdosfs command creates a proper 32 KB cluster size FAT32 partition and TwilightMenu++ no longer shows the warning. Games load fine.

If its only one step, you might as well merge it with the other step

Thank you for the feedback, I merged it into the first step now :)

Something irks me about the title.
@urmum-69 What do you think the title should be?

Maybe just
## Section I - Formatting your SD card

or so would be a better title? The determining slot isn't really the important part of it so it doesn't really need to be in the title I think.

@Epicpkmn11  thank you for the suggestion for the title. I merged your proposal. Sorry for the late reaction on my side


chore: upgrade lint-config

Fixing with https://github.com/Wiredcraft/loopback-connector-couchbase3/issues/10


can't find where classes are declared

Hi! I forked the project so I could edit the classes for the signin button. For example, I'd like to change btn-default to btn-primary. But when I search the repo, I don't find a single result for "btn-default". How is that getting applied if it's not in the repo?
Cheers!

ah found it. I think the answer is: yes, github search is broken

I'm trying to fork the package but there was no .bash_profile or .bashrc in my home directory
I created them both. In .bashrc export PACKAGE_DIRS
And I added this to my .bash_profile [ -r ~/.bashrc ] && source ~/.bashrc
so I need to do anything else? meteor is still not picking up my local packges.


Connection to Adafruit lost

Adafruit has made updates to their API, namely SSL keys have changed. Libraries needs to be updated to communicate.
https://github.com/adafruit/Adafruit_MQTT_Library/releases/tag/2.4.3

After just restarting MQTT was working correctly again. Closing until error occurs again.


Conf file refactor

Summary
Refactor conf file.

[x] add cmd to resources config file
[x] add default cmd to extra dep

Details and comments
95% of the work was already done in the MR #34
Closes #19
Epic #9

@IceKhan13 I added just 2 little stuff, do you see something else to do in this issue ? The entire case seems to be cover

I think we need to address number of issue in current configuration setup:

rename file appropriately. qe_config.json is not the best option
we have this snippet in 3 runner files (coverage, tests, lint) which is not good. We should hide it inside configuration class or refactor somehow https://github.com/qiskit-community/ecosystem/blob/ac1c4a05ec1150f33593ce64dfeffca89465aa3a/ecosystem/runners/python_coverages_runner.py#L41-L50
Optional: move from json to yaml (as all configs usually in yaml)
Maybe store configuration class inside Repository class and store in db too. So this configuration will be deafult: if repository has config file -> we use it, if not -> we use config stored in DB

I'll add it to issue.

Let me know what do you think

@IceKhan13 I did a draft of point .2 :

functions :
https://github.com/qiskit-community/ecosystem/blob/b8060b44476244a4c7cace312323da4506eb4aee/ecosystem/runners/runner.py#L15-L46

call :
https://github.com/qiskit-community/ecosystem/blob/b8060b44476244a4c7cace312323da4506eb4aee/ecosystem/runners/python_tests_runner.py#L44-L48

What do you think of that ?
PS : I just changed the test runner class not the style and coverage (yet)

You see both of functions are using runner as argument. Maybe it's better to put it as method in PythonRunner class?
And maybe name them somehow get_config and inject_tox_file or something like this, so it actually tells you what it is doing :)
So then you will call it like
config = self.get_config()
self.inject_tox_file(config)

or you can do even better and hide config at all from workload. Because you only use it in inject_tox_file.
def inject_tox_file(self):
     ...
    config = self.get_config()

Or something along those line :)

or you can do even better and hide config at all from workload. Because you only use it in inject_tox_file.
def inject_tox_file(self):
     ...
    config = self.get_config()

yes and no, config is also use for coverage file .coveragerc and lint file .pylintrc. So I was thinking having 1 function checking for any file:
https://github.com/qiskit-community/ecosystem/blob/4a4ada41909ea901ef63d98b21183738fb873f84/ecosystem/runners/runner.py#L46-L80
or we can let the render part in each of the Style, Coverage and Tests class

So, nevermind what I said yesterday night (coding when sleepy is not a good idea ^^')
So I merge the 2 functions as you suggested and make it working for anyfiles :
https://github.com/qiskit-community/ecosystem/blob/6daf43bad541fa5d92cc131a64b74ec2c472e49e/ecosystem/runners/runner.py#L50-L81
I edit all of the 3 PythonRunner, what do you thing @IceKhan13 :)

this line :
https://github.com/qiskit-community/ecosystem/blob/d304378bf5af7b3d9b798ce1ec3180e88ca71be5/ecosystem/models/configuration.py#L123
for Bug #42

So, nevermind what I said yesterday night (coding when sleepy is not a good idea ^^')

If you feel sleepy, just go and get some good sleep 😄 Sleep is very important! You already did a lot of progress on this project!

So, nevermind what I said yesterday night (coding when sleepy is not a good idea ^^')

If you feel sleepy, just go and get some good sleep 😄 Sleep is very important! You already did a lot of progress on this project!

Don't worry, it was night and I was finishing a movie, I saw your comment and I always I was too curious to not trying to correct it right now x)

@IceKhan13 by the way, this one is finished for me


Ctrl-click: support FS paths such as C:\ and C:/

Description of the new feature/enhancement
As of 1.8.1521.0, Windows Terminal supports some file URLs:
file://C:/<path>
file:///C:/<path>

But support for regular FS paths seems to be lacking:
C:\<path>
C:/<path>

Being able to Ctrl-click into files and folders would be extremely convenient.
The reason I'm also asking for / support, in addition to /, is because certain CLI tools may output these partially "unixified" paths. For example, Go stacktraces have paths that look like this:
C:/<path>:<row>:<col>

(Support for row/col is a separate issue: #10428. This is about FS paths which may use both slash styles.)

I think #2671 covers this.

Should I add examples to that issue?

Feel free to do so if you want to. We know what the feature is about. Just lack a solution that handles all scenarios (cmd, powershell, WSL & possible other shells). See https://github.com/microsoft/terminal/issues/9793#issuecomment-818736920 .

Closing as duplicate, apologies for your time.


Test files do not have strict: true applied in editor

The tsconfigs aren't applied to tests, which can cause issues during development. I'm not sure how to fix the issue in a monorepo, but including tests in tsconfig.json and having a tsconfig.build.json extending it and excluding tests should be the way to go.

I tried to do it the other way where there is a tsconfig for tests. I saw that pattern on another mono repo and copied it.

Fixed with https://github.com/solidjs/solid/pull/784


ROX-13582: Central telemetry configuration. Metadata response update.

Description
This and following PRs deprecate #3819 to simplify review.

Central telemetry configuration with no actual use (see following PRs);
/v1/metadata response with telemetry configuration.
Telemetry environment variables reused:

ROX_TELEMETRY_FREQUENCY — the data flush interval (i.e. API call to Segment);
ROX_TELEMETRY_ENDPOINT — the telemetry storage endpoint, implies default segment endpoint if left empty;

New environment variable:

ROX_TELEMETRY_STORAGE_KEY_V1 — segment write key, disables telemetry if left empty.

rhacs.redhat.com/tenant central deployment label is read for the Tenant ID, which will serve as the group ID for clients, such as central, central API users, UI users etc;
Central installation information provides the central ID.

Checklist

[x] Investigated and inspected CI test results
[x] Unit test and regression tests added
[ ] ~Evaluated and added CHANGELOG entry if required~
[ ] ~Determined and documented upgrade steps~
[ ] ~Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

Testing Performed
Unit tests.
/v1/metadata response:
{
  "version": "",
  "buildFlavor": "development",
  "releaseBuild": false,
  "licenseStatus": "VALID",
  "userId": "3wf49KZA31rMxvgqCKVi8o3ZvINhjDqx04G5r16+Y3g=",
  "telemetryEndpoint": "",
  "storageKeyV1": "..."
}

Current dependencies on/for this PR:

master

PR #4070   👈

PR #4071

PR #4072

This comment was auto-generated by Graphite.

/retest

@0x656b694d Are we good to merge this? I'm not sure about those test failures, they look unrelated to me. Did you investigate them already?

/retest


Dynamic module imports failing on Windows OS

There is some inconsistency in the functionality of ES modules across different operating systems. You can read about this inconsistency here: https://github.com/nodejs/node/issues/31710#issuecomment-587434048. It is important to make this functionality consistent across all operating systems.
Currently, there is a schema problem when Node.js tries to import a file on Windows, which leads to the following error: code: 'ERR_UNSUPPORTED_ESM_URL_SCHEME'.
To address this issue, we need to make the functionality of ES modules consistent across all operating systems.

Merged


SREP-2260 monitoring script for node health behind each available ELB

BTW, I think you should better submit the PR for stg branch firstly, and then prod.


Global variable samples test fails

Global variable samples test fails
preconditions
1.the balarina-examples/examples folder should be set as a project running balarina init
2.run the command ballerina test global-variables/
---------------------------------------------------------------------------
    T E S T S
---------------------------------------------------------------------------
---------------------------------------------------------------------------
Running Tests of Package: global-variables
---------------------------------------------------------------------------
Failed to execute the test function [testFunc] of test suite package [global-variables]. Cause: error: ballerina.runtime:CallFailedException, message: call failed
	at global-variables:testFunc(global-variables-test.bal:21)
caused by error, message: assertion Failed!: expected This is a sample text
 but found This is a sample text
	at ballerina.test:assertEquals(assert.bal:55)

Tests run: 1, Passed: 0, Failures: 1, Skipped: 0 - in TestSuite

failed tests:
 testFunc: Failed to execute the test function [testFunc] of test suite package [global-variables]. Cause: error: ballerina.runtime:CallFailedException, message: call failed
	at global-variables:testFunc(global-variables-test.bal:21)
caused by error, message: assertion Failed!: expected This is a sample text
 but found This is a sample text
	at ballerina.test:assertEquals(assert.bal:55)

Fix in https://github.com/ballerina-platform/ballerina-examples/pull/375


dydx v4 setup issue

Hi, thanks for making updates for dydx v4, much appreciated! I'm trying to set it up now but facing some issues, i tried following the instruction for .env file variables setup and noticed you mentioned for dydx v4 only mnemonic(seed) phrase is required, i supposed we need to include wallet address too, no? By the way I noticed dydx v4 has 2 addresses, one is standard ETH address and the other one is dydx chain address, do we need to include any of these? I tried both but getting error message: "Error on getting account data". Thanks for your great help and good work in advance, cheers!

hey I've fixed the issue "Error on getting account data" and pushed a new version
Could you deploy the latest version and try again? it should work now
https://tv-connector.gitbook.io/docs/setup/faq#how-to-deploy-the-latest-version-of-the-tool
also, you don't need to provide any wallet addresses.
It's derived from mnemonic automatically.

hi i tried again, with just mnemonic(seed) phrase for required variables, which i obtained from dydx v4 "export secret phrase", it's basically a series of 24 words with space separated, right? but it doesn't seem to work, i'm still getting error "Error on getting account data".

just tested, get message below:
dYdX v3 Account Ready:false, dYdX v4 Account Ready:false, Perpetual Protocol Account Ready:false, GMX Account Ready:undefined

It works well on my end.
Have you deposited your collateral(USDC)?

brilliant, it worked now! thanks a lot!!


Fix parse obj root

Change Summary
Makes nested root models work properly. In particular, the code raising errors in #1190 doesn't raise errors with this patch. I suspect this may resolve other issues related to custom root models as well.
Also required a new flag to make nested root models generate proper json.
I believe all of the newly added checks for the __custom_root_type__ could be moved to model creation time by making use of some method-wrapping logic in the metaclass.
The fact that the unpack_custom_root argument is passed in the various recursive calls to .dict is the only performance thing that I think is likely hard to eliminate (without breaking the json generation). But it would be possible to move the actual check (and subsequent unwrapping) into a wrapper function so that it is only performed for types that actually have a custom root. I'm not sure if adding another argument itself adds an amount of performance overhead that is worth fretting over. (But I would also understand if you don't want to add the flag at all.)
Related issue number
Addresses #1190
Checklist

[ ] Unit tests for the changes exist
[ ] Tests pass on CI and coverage remains at 100%
[ ] Documentation reflects the changes where applicable
[ ] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)

This seems like a backwards incompatible change to __init__ logic? So I guess, if we make this change, we should wait for v2.
I'm also concerned about the madness of dict(unpack_custom_root=True) not returning a dict, but then again I'm conscious that dict() and json() should have the same (or as similar as possible) interfaces and json() need to return the root object not {"__root__": thing} in v2.
In summary, are we sure there isn't a similar way of fixing the original problem?

I think the main problem is that there are some places in the code where object initialization happens via Model(**data) or similar. This generally makes it difficult to handle issues with a custom root model properly because validation requires a dict.
I'm not saying that a backwards-compatible fix isn't possible, but I think it would help a lot if there was a single model-validation logic path that involved passing a raw object, rather than making use of the __init__ function and requiring a dict as input. It seems like some of these issues might be easier to address, for example, if parse_obj was the main way to create a validated instance, and __init__ delegated to that function, rather than the other way around (as it is now).
But I haven't thought through things like performance implications etc. where the current approach may have marginal benefits for the most common use cases.
At any rate, I agree that this PR is likely unworkable in the near term, so I'm going to close it.


graphhopper.sh windows

hi, i have problems with set up graphhopper on windows.
i've cloned the git repository but when i want to import the map i get a failure
$ ./graphhopper.sh import europe_germany_berlin.pbf
./graphhopper.sh: Zeile 2: $'\r': Kommando nicht gefunden.
./graphhopper.sh: Zeile 25: Syntaxfehler beim unerwarteten Wort $'{\r'' '/graphhopper.sh: Zeile 25: function printUsage {
do you have ideas what i have to do?

Did you follow the windows specific documentation and installed cygwin or similar?

yes, and i did it with cygwin :/

you may have cloned with a wrong line ending convention, causing git to do a conversion while getting the files
check git config -l output

@mprins good idea. Do you know if our settings are wrong? https://github.com/graphhopper/graphhopper/blob/master/.gitattributes
I would prefer if it works out of the box on every OS

I'm not sure, there might also be peculiarities between git clients eg. cygwin has a git client that behaves 'unix' but eg toirtoise or another win native client has windows behaviour.
also none of them share the config  afaik.
Think it's a trial and error thing..

An idea could be to download the GH repository zip from the main page and unzip (normally the OS can handle it better this way).

for a project I have building on the AppVeyor CI I do this:
  - git config --global core.autocrlf true
  - git config --global core.safecrlf true

looking at the .gitattributes i notice that ths .sh extension is not listed, so git will just guestimate the appropriate line ending convention to use, maybe adding a line  *.sh text will just make it behave properly

oh, thanks - fixed!
@domi-nique please drop the checked out repo and try again. If still not working you can also explicitly convert to windows format via the tool unix2dos I think

Closing here, if still not fixed - please reopen!

Maybe we should use editorconfig to fix this properly (?)

Isn't that something that depends on various IDE / plugins support?
From a quick look I see not all features (e.g. end_of_line) are supported completely everywhere.

Isn't that something that depends on various IDE / plugins support?

See download - many IDEs have native support

From a quick look I see not all features (e.g. end_of_line) are supported completely everywhere.

What do you mean? The line ending could be additionally forced this way as it looks that the git-way fails sometimes.

Well the majority works with plugins (Eclipse, NetBeans, etc).
Even then, searching the supported properties sections, not all are there.
Doesn't that mean that essentially we could have missing functionality?

I'm only speaking for the line ending ... but yes, the functionality for code formatting is kind of not existing for this

Still user experience this issue, and I needed to install a unix environment with unix line endings to fix this. Hmmh.

We could have two batch files (sh and bat), i.e. that's how Gradle wrapper [1] [2] in GH Android sample works.
More to support but Windows users will not have to setup a Unix-like environment.

I wouldn't be able to maintain this.

dos2unix graphhopper.sh worked for me.

There is a maintenance-free solution, based on this stackoverflow answer.
Adding this line just after the #! line:
(set -o igncr) 2>/dev/null && set -o igncr; # this comment is required for handling Windows cr/lf

Note that the comment is needed in order to ignore the CR in that line, which is read before the option takes effect.

Created PR #816


belongsToMany query with 'offset' is not working

Given
const { resolver } = require('graphql-sequelize')

const Post = sequelize.define('post')
const Tag = sequelize.define('post')
const PostTag = sequelize.define('post_tag')
Post.Tags = Post.belongsToMany(Tag, {through: PostTag})
Tag.Posts = Tag.belongsToMany(Post, {through: PostTag})

resolver(Tag.Posts)

tag.getPosts({limit: 2, offset: 1})

Got
SELECT "post".* FROM ((
  SELECT "post"."id", post_tags"."postId" AS "post_tags.postId", "post_tags"."tagId" AS "post_tags.tagId" FROM "posts" AS "post"
  INNER JOIN "post_tags" AS "post_tags" ON "post"."id" = "post_tags"."postId" AND "post_tags"."tagId" = 1
  LIMIT 2
)) AS "post";

Clearly, It is missing the OFFSET statement.
Without the resolver(Tag.Posts) patch, it works fine:
  SELECT "post"."id", "post_tag"."postId" AS "post_tag.postId", "post_tag"."tagId" AS "post_tag.tagId" FROM "posts" AS "post"
  INNER JOIN "post_tags" AS "post_tag" ON "post"."id" = "post_tag"."postId" AND "post_tag"."tagId" = 1
  LIMIT 2 OFFSET 1;

Having the same issue. Any progress on making a fix? Or any suggestion where the issue is?

Likely this piece of code: https://github.com/mickhansen/dataloader-sequelize/blob/master/src/index.js#L136

Thanks @mickhansen. This is also affecting belongsTo relationships. Submitted a PR at #55


Check HAXiam apache settings

been getting timeouts and weird connection drops w/ current configuration and appears to be directly apache based as this is in requesting static html files via JS. We ship some sane defaults for apache / php so should review to make sure they actually are sane :)

https://github.com/elmsln/HAXiam
https://www.digitalocean.com/community/tutorials/how-to-configure-apache-http-with-mpm-event-and-php-fpm-on-freebsd-12-0
might help

moved to haxiam


1.3 Release

[x] Setup Issues into Milestone: https://github.com/patronage/bubs-next/milestone/4
[x] Complete dev
[x] Pre-release and test on a few sites
[x] Write Release Notes
[x] Release!

Done!
https://github.com/patronage/bubs-next/releases/tag/v1.3.0


Add a way to run multithreaded lookups in mmdblookup

This is similar to the existing benchmark mode, but runs in several
threads.

Not too certain about the Windows detection here.
Also, perhaps this would be better as a separate program, or be combined more with the existing benchmark code. Possibly it doesn't belong in this repo at all. I did find it useful for finding one race, so I think it or something like it would be good to have somewhere.


Enable bundling of mixin scripts

This PR enables the bundling of the mixin scripts. To do so, it replaces the custom bundler I wrote with https://github.com/Benjamin-Dobell/luabundle. Additionally, since it results in larger file sizes in general, I minified the resulting bundles using https://github.com/mathiasbynens/luamin.
Some things to note:

The majority of the bundling now depends on 3rd-party dependencies, so if there are bugs we'd need to fix them in the upstream repos
The mixins (e.g., things located in src/mixin) are not included unless the mixin library is required (e.g., require("library.mixin"). The exact heuristic is that if the text library.mixin is used anywhere in the file, all the mixins will be bundled.

This means we don't need to statically determine which mixins are used, which is a much more complex issue
It also means that anytime you use a mixin, you must put library.mixin somewhere in the file (usually the require statement is sufficient).

Imported lua files are no longer required to be located in src/library. They can now be located in any subfoler inside the src folder (including folders added in the future).
I cannot entirely test things out locally, so it's possible there will be build errors when this PR is merged

Overall, I think this is a net win.
Allows the merging of #183.
Feel free to merge this PR when approved.

One reason I'm punting on mergin is the merge conflicts with the dist files. I'm wondering if they should even be tracked.

One reason I'm punting on merging is the merge conflicts with the dist files. I'm wondering if they should even be tracked.

Right now they should be tracked.
When someone clicks "Download" on the website, they download the bundled file directly from GitHub. For instance, if someone wants to download the "simplify accidentals" script, they are essentially doing the following:
<a download href="https://raw.githubusercontent.com/finale-lua/lua-scripts/master/dist/accidental_simplify.lua">Download</a>

It's slightly more complex than that to meet more edge cases, but that's the idea.
I'm open to not checking in these bundled files. It just means we'll need to figure out another place to store the bundled scripts.

I figured there might be an issue that they couldn't be on github if not tracked. I see removing them as very low priority.


Add a TRY macro for early-out on types that satisfy Try and report failure (Option, Result, optional, expected..)

Chromium is gaining RETURN_IF_ERROR(e) which is built around base::expected (like std::expected).
STX has TRY_OK and TRY_SOME which are similar and hard coded to its Result and Option types.
Subspace has a sus::ops::Try concept which is used to early terminate operations like try_fold. The concept is satisfied by sus::result::Result and sus::Option. It can be extended to make std::optional and std::expected satisfy sus::ops::Try as well, in the same way that std::vector now satisfies sus::iter::FromIterator.
Providing a TRY() macro that will early out when the Try object reports failure would allow this mechanism to be applied across many different types.
If needed we can add additional hooks into the requirements of the Try concept to provide good error messages.

I'm adding Error and its type-erased Box<ErrorBase> and that means a function returning Result<T, Box<ErrorBase>> can return any Error if it spells it sus::err(sus::into(SomeErrorType())) so the TRY macro should also do the into step.


Cannot change or drill into folders anymore?

The normal expected JupyterLab behavior of changing or drilling into a folder is gone.
Double-clicking expands folders, but there is no way to make a folder and set the current working folder anymore.   We need this functionality back to drill into folder to set the current working folder.

@richlysakowski would you mind providing the versions you are using for JupyterLab and this extension?

I just downloaded it today and ran into this problem. I'm on the latest version of Chrome, Ubuntu 22, and jupyterlab-unfold-0.2.2
Without a way to change directories in the file browser, I can't use this feature, unfortunately.

Same issue like mine, right?
https://github.com/jupyterlab-contrib/jupyterlab-unfold/issues/38


Block Legacy Auth without ADFS?

The article only lists blocking Legacy Authentication if you are using ADFS. How do we block Legacy Authentication if we have moved to Cloud and don't have ADFS anymore?

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

ID: ad5f26dc-2f8f-ef76-3d20-a0a42d2bbada
Version Independent ID: a0e971d1-05f2-2c21-06d4-60c087d45c0a
Content: Set up SharePoint Online and Exchange Online for Azure Active Directory conditional access
Content Source: articles/active-directory/active-directory-conditional-access-no-modern-authentication.md
Service: active-directory
GitHub Login: @MarkusVi
Microsoft Alias: markvi

@NickDorak Thanks for your feedback! We will investigate and update as appropriate.

@NickDorak Please note that there is already feedback provided in O365 user voice regarding it. Please upvote this feedback so that feature team can prioritize the implementation.
Also please check this TechNet article which may help - Client Access Rules in Exchange Online

@NickDorak We will now proceed to close this thread. If there are further questions regarding this matter, please open a new issue and we will gladly continue the discussion.


depth vs. diff

regarding the websocket depth data... (@depth vs @depth levels): is all of the information in the diff stream already in the depth stream? i'm thinking of saving some of this data for research at a later date, is there an advantage to having one or the other? do i need both to "reconstruct" the  depthbook activity in a historical "simulated" live trade scenario? I'm a little unclear as to how they differ. Does the diff stream simply exclude redundant levels?
It's not clear how this works, could someone direct me to a good source? I've sent a message on telegram but no response.

Yes, it is as long as you're using only 20 levels of the orderbook. The advantage of using diff is that you can only store the updates every timestamp and reconstruct the full book whenever you want.

Thanks for your reply. How would you go about setting up the starting an archive, since it is a diff, would the snapshot be the first entry in the archive? i guess my concern is getting the wrong snapshot and then have the diff adding to the wrong starting values.
Also, are there any advantages to collecting the full depth ( i.e. not the diff)?
I realize it would mean storing more data but perhaps it is better? If not, why would it even be available?
Cheers.


fixweek should be able to recognized as function name in function call context

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[x] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[x] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

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...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.51 seconds
load time: 492 seconds
storage size:<PHONE_NUMBER>4 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230109123539_clickbench_pr_76132.html


switch to binance mainnet

@hewigovens https://github.com/TrustWallet/blockatlas/blob/master/coin/slip44.go#L214 you can change this decimal and use this parameter in the code for consistency?

still need to fix the test

The binance test is failed because the Value is decoded as "1"

The binance test is failed because the Value is decoded as "1"

json.Unmarshal converts "0.01" => "1" but "1" => "1" and "10" => "10" converted as expected. At first i thought it because of type mismatched https://github.com/TrustWallet/blockatlas/pull/74/commits/640d5d28d35d1509e9edcad910c8179963fdbb39 but this is something else causing

Will be implemented by #58

@hewigovens Thanks for the fixes, I rebased onto newest master in #75. Everything in this PR is implemented, so feel free to delete the branch

@terorie 👍，I'm still confused why the test was passed locally but failed in CI

@hewigovens The import path was wrong. CI had trust-wallet/blockatlas instead of trustwallet/blockatlas


Update README.md

You need git in the path for autocomplete-plus.

Thanks! Makes me 😔. Will investigate.

Bah, 'You' shouldn't be capitalized.  That's what I get for submitting PRs on a bus in the morning.

Bah, 'You' shouldn't be capitalized

I'll take care of it :+1:

@jonathandturner I asked the autocomplete-plus team. It shouldn't be required

(ref : https://gitter.im/atom-community/autocomplete-plus?at=54d933a8938f0fd216cf1bcb)
@jonathandturner can I get the details of the error you had?

autocomplete-plus doesn't need git in your PATH. apm needs git in your PATH. It's how package installation works.

@joefitzgerald awesome Thanks!

Portable git is bundled with the Windows distribution - on other platforms it is assumed git is in the PATH. Ref:

https://github.com/atom/apm/blob/457b87b683118feaa1295742b62cbfea46dfaa41/src/git.coffee#L47-L50
https://github.com/atom/apm/blob/11f10075e7f060602ddd58e1fb2e0c91e0ed3b01/src/install.coffee#L170
https://github.com/atom/apm/blob/11f10075e7f060602ddd58e1fb2e0c91e0ed3b01/src/install.coffee#L173-L181

@jonathandturner perhaps you tried on a mac?

Macs have git in their path by default, unless they are prehistoric.
$ which -a git
/usr/local/bin/git
/usr/local/bin/git
/usr/bin/git

Said another way, you should not be able to install atom-typescript without git in your path either :smile:.

looking at https://github.com/atom/apm/blob/11f10075e7f060602ddd58e1fb2e0c91e0ed3b01/src/install.coffee#L174-L202 I see they do show an error for windows. So I'll just be safe and make it a requirement :smile:

Just phrase it such that you make clear it's not a requirement of your package but of installing any atom package.


Eliminate race condition in completempupload and support overwrite

Cherry pick of https://github.com/Alluxio/alluxio/pull/16578
What changes are proposed in this pull request?
Make CompleteMultipartUpload call idempotent
Support Overwrite now only for createobject from s3client thru multipart upload
Why are the changes needed?
When use  Spark + hadoop s3a to write to Alluxio, spark will retry if CompleteMultipartUpload took too long, since alluxio doesnt allow overwrite, s3 proxy does a delete and create in an non-atomic way which causes race condition, rendering the retry fail, and as a result failing the entire spark write.
Does this PR introduce any user facing changes?

New timer (with uniformTimer call)using UniformReservoir is introduced in the MetricsSystem
the uniformTimer is added to measure different parts of CompleteMultipartUpload call, as well as common places such as UFS and AlluxioOutstream close call.

alluxio-bot, merge this please


Add layer support to sc.tl.score_genes

... so that we can do crazy things like scoring velocities or scoring "unspliced expression".

@gokceneraslan I've been thinking we should have options like layer and obsm in many more places. I've started trying to implement this in a systemic way with an internal API and some test helpers in: #1173.
What would you think of using that here?

@ivirshup is there an elegant way to get subset of genes with  _get_obs_rep? because in the current code there are things like:
    _adata = adata.raw if use_raw else adata
    _adata_subset = _adata[:, gene_pool]

Not much beyond _get_obs_rep(adata[:, genes], ...) or something like _get_obs_rep(adata.raw.to_adata()[:, genes], ...).


Instructions for inspecting the dump output?

I've got the code running and dumping out screen mirrored frames but I'm not sure how to look at these.
Can I concatenate these into a file viewable with Quicktime or something? Do I need to create a header?

I had missed a socket exception (I was already running AirServer). When I quit AirServer I started to get the dump.h264 file.
Then I was able to use
ffmpeg -i dump.h264 -c copy output.mp4

to take a look at the output.

Exactly!
The dump is saved only when you close the application, to carry out the tests.
You can move this logic wherever you want 🔥


After switch to master channel and upgrade to 1.24.0, some parameters of widgets gone

For some reason I use version 1.20.0 for a while, and this days I upgrade to master channel and this showed.
Like child onPressed, navigate to source code I found the required dose start with @
Is the only way to fix this is to re-download the SDK?

After I tried to clone flutter git this situation still there :(

@required is now required . Please include output of flutter doctor -v
Thanks

You may need to restart your analyzer

@required is now required . Please include output of flutter doctor -v
Thanks

flutter doctor -v here

[✓] Flutter (Channel unknown, 1.20.0, on Microsoft Windows [Version 10.0.19041.572], locale en-US)
• Flutter version 1.20.0 at D:\flutter
• Framework revision 840c9205b3 (3 months ago), 2020-08-04 20:55:12 -0700
• Engine revision c8e3b94853
• Dart version 2.9.0
• Pub download mirror https://pub.flutter-io.cn
• Flutter download mirror https://storage.flutter-io.cn
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
• Android SDK at C:\Users\Flutter.Dev\AppData\Local\Android\Sdk
• Platform android-30, build-tools 30.0.0
• ANDROID_HOME = C:\Users\Flutter.Dev\AppData\Local\Android\Sdk
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
• All Android licenses accepted.
[✓] Chrome - develop for the web
• Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe
[✓] Visual Studio - develop for Windows (Visual Studio Enterprise 2019 16.7.5)
• Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise
• Visual Studio Enterprise 2019 version 16.7.30523.141
• Windows 10 SDK version 10.0.18362.0
[✓] Android Studio (version 4.0)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 48.1.2
• Dart plugin version 193.7361
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
[✓] VS Code (version 1.50.1)
• VS Code at C:\Users\Flutter.Dev\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.15.1
[✓] Connected device (5 available)
• Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 10 (API 29) (emulator)
• Windows (desktop)                  • windows       • windows-x64    • Microsoft Windows [Version 10.0.19041.572]
• Web Server (web)                   • web-server    • web-javascript • Flutter Tools
• Chrome (web)                       • chrome        • web-javascript • Google Chrome 86.0.4240.111
• Edge (web)                         • edge          • web-javascript • Microsoft Edge 86.0.622.48
• No issues found!

You may need to restart your analyzer

I don't know if restart vscode can do that, but if it is, I tried

Hi @ShiinaManatsu
As said above required is now a keyword in Dart, it was introduced along with Null Safety.
This is just the analyzer complaining about it.
Closing this as invalid Flutter issue.


removing k20 and xpressohack

getting rid of old k20 and xpressohack. Please have a safety check.

Tested with 4 nodes linear network. All ping command rearched. firefox Copper plugin work fine.


Add a utility for inspecting go import trees

I wrote this small utility for tracking down go dependency issues, and it seems like it could be useful to other people. There are alternatives (such as godepgraph and goviz) but I couldn't find any that computed paths between packages.
/cc @lavalamp

Nifty-- will look more when I have more time.
Also note that I added import-boss to the main repo, which can be used to prevent unwanted imports :)

Not sure why you decided to close this but I just used this and it is awesome!
Thanks a lot for writing this :)

I decided to put it in a separate repo. Unfortunately it's been stuck in internal OSS process, but I can try to move it along. Glad you found it useful!

How did you use it since github.com/timstclair/godepq doesn't exist?

Downloaded the files in this PR and used them :)
There are 4 files.

@nikhiljindal but https://github.com/kubernetes/contrib/pull/494/files#diff-bdaa96558ba7879c62c97a8ac1a69e29R27 ???

You could put the files in $GOPATH/src/github.com/timstclair/godepq. I'll ping this PR once the official release is done (poked the issue).

@eparis Yes had to change that line to point to my location of godepq/pkg.

FYI, this was released at https://github.com/google/godepq


Attempt to add arm64 support

Description

Fixes # (issue)
Type of change

[ ] 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 not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

[x] bpftime_daemon_tests
[x] bpftime runtime unit tests & old tests

Test Configuration:

Hardware: M1 Macbook Air w/ Ubuntu 23.04 running in QEMU

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Thanks a lot!

@ShawnZhong @Officeyutong By the way, is it possible to make a GitHub Action for testing bpftime on arm64?

@ShawnZhong @Officeyutong By the way, is it possible to make a GitHub Action for testing bpftime on arm64?

Many of features existing rely on x86, e.g user space syscall trace. We need a new syscall trace trampoline on arm64.

@ShawnZhong @Officeyutong By the way, is it possible to make a GitHub Action for testing bpftime on arm64?

Running tests itself is relatively easier

Many of features existing rely on x86, e.g user space syscall trace. We need a new syscall trace trampoline on arm64.

That's fine. At least we can get the uprobe and runtime work.
Is there any other features rely on x86?

@ShawnZhong @Officeyutong By the way, is it possible to make a GitHub Action for testing bpftime on arm64?

It seems that Arm-based hosted runners are still in public beta as of January 2024: https://github.blog/changelog/2023-10-30-accelerate-your-ci-cd-with-arm-based-hosted-runners-in-github-actions/
One option is to use self-hosted runners, which requires some setup (and cost money)

https://docs.github.com/en/actions/hosting-your-own-runners/managing-self-hosted-runners/about-self-hosted-runners
https://actuated.dev/blog/native-arm64-for-github-actions
https://buildjet.com/for-github-actions/blog/hosted-arm-runners-on-github-actions

My least favorable option is to run inside QEMU.

Thanks a lot!

Many of features existing rely on x86, e.g user space syscall trace. We need a new syscall trace trampoline on arm64.

That's fine. At least we can get the uprobe and runtime work. We can get this PR work first, and then consider others(syscall trace, CI, syscall trace should be easier on arm)
Is there any other features rely on x86?

Calling original syscall at prog array implementation was bound to x86, needs updating


/bin/conda and /bin/mamba missing in mambaorg/micromamba:0.14.0

Problem
It seems like /bin/conda and /bin/mamba, which were available in 0.13.1, are no longer present in 0.14.0?
Steps to reproduce
$ docker run --rm -it --platform linux/amd64 mambaorg/micromamba:0.14.0
(base) root@bc1eb1302a18:/# which -a micromamba
/bin/micromamba
(base) root@bc1eb1302a18:/# which -a conda
(base) root@bc1eb1302a18:/# which -a mamba

I tried testing with micromamba 0.14.1, but it seems like that's not available on Docker Hub: https://hub.docker.com/r/mambaorg/micromamba/tags

Hi @jli -- thanks for the interest in micromamba.
Before 0.14.0 there were symlinks from conda and mamba to micromamba within the image. Conda and mamba binaries have never been part of the micromamba image.  The symlinks were originally put in as a convenience, but they were sometimes confusing people (see https://github.com/mamba-org/micromamba-docker/issues/12#issuecomment-808843294 for an example). So I decided to remove the symlinks. Sorry if that broke something for you, but you should just be able to replace conda/mamba with micromamba and get the same functionality.
micromamba 0.14.1 hasn't made it to conda-forge yet, which is where the automated docker build downloads it from.

@wholtz Ah, that all makes sense. Thanks!


Convert database/main/Ag3AsS3/Hulme-e.yml to UTF8. Remove trailing tabs

Hello,
Just some more very minor fixes to satisfy the Python YAML parser:

Looks like tabs are forbidden in the YAML specs: http://www.yaml.org/faq.html
One file had an ISO-8859-15 charset including a degree character. Here converted to proper UTF8.

Thank you!
Thibault

Hi Thibault,
thank you again!
Merged

Thanks!
Thibault


Filter box

I have a question .
Is it possible to fixed width of a filter box in this picture

Hi!
You can do it many ways, but I would recommend to use just CSS without modifying php code.

Ok
I'm really sorry to ask this question
Which file I should write CSS . Did you have some example ?

Write it to main .css file of your project :)
Or you can include it using <style> tag to layout of page where you use grid.

And about alias
'HTML'  => 'Collective\Html\HtmlFacade',
I already add in my app config
'HTML' => Collective\Html\HtmlFacade::class,
But I can't use HTML in controller
HTML:: .....

What i should do next plz help

Try \HTML::

Oh It's works !
Thank you very much Mr.Nayjest >.<

No problem :)


Fix nginx index option url routing

I think we should use the index instead of just assuming /index.html.
@reglim or was there a good reason to not use index?

Yes, I think the reason was that we need to fall back to index.html because routing is handled by react-router. See this post. But if it works, why not.

I tested it and it worked just as well, so I think we can merge. :+1:


Adding auto configuration to bower.json

Fixes #3, updating README with necessary config info and adding auto configuration to bower.json

Thanks, poke me later and I'll merge + release.


[Forecasting] Error thrown when making prediction using generated Web API project

System Information (please complete the following information):

Model Builder Version (available in Manage Extensions dialog): <IP_ADDRESS>0302
Visual Studio Version: VS2022

Describe the bug

On which step of the process did you run into an issue: Consume
Clear description of the problem: When making an HTTP request using the generated Web API project, an error is thrown.

To Reproduce
Steps to reproduce the behavior:

Start generated Web API project
Make an HTTP POST request to predict Web API endpoint with ModelInput data

{
	"ProductSales": 20.0
}

Expected behavior
I would expect a response that looks like a ModelOutput object.
Additional context
Error Message:
 System.ArgumentOutOfRangeException: Must be a row to row mapper (Parameter 'transformer')
         at Microsoft.ML.Runtime.Contracts.CheckParam(IExceptionContext ctx, Boolean f, String paramName, String msg)
         at Microsoft.ML.PredictionEngineBase`2.TransformerChecker(IExceptionContext ectx, ITransformer transformer)
         at Microsoft.ML.PredictionEngineBase`2..ctor(IHostEnvironment env, ITransformer transformer, Boolean ignoreMissingColumns, SchemaDefinition inputSchemaDefinition, SchemaDefinition outputSchemaDefinition, Boolean ownsTransformer)
         at Microsoft.ML.PredictionEngine`2..ctor(IHostEnvironment env, ITransformer transformer, Boolean ignoreMissingColumns, SchemaDefinition inputSchemaDefinition, SchemaDefinition outputSchemaDefinition, Boolean ownsTransformer)
         at Microsoft.ML.PredictionEngineExtensions.CreatePredictionEngine[TSrc,TDst](ITransformer transformer, IHostEnvironment env, Boolean ignoreMissingColumns, SchemaDefinition inputSchemaDefinition, SchemaDefinition outputSchemaDefinition, Boolean ownsTransformer)
         at Microsoft.ML.ModelOperationsCatalog.CreatePredictionEngine[TSrc,TDst](ITransformer transformer, Boolean ignoreMissingColumns, SchemaDefinition inputSchemaDefinition, SchemaDefinition outputSchemaDefinition)
         at Microsoft.Extensions.ML.PredictionEnginePoolPolicy`2.Create()
         at Microsoft.Extensions.ObjectPool.DefaultObjectPool`1.Create()
         at Microsoft.Extensions.ObjectPool.DefaultObjectPool`1.Get()
         at Microsoft.Extensions.ML.PredictionEnginePool`2.GetPredictionEngine(String modelName)
         at Microsoft.Extensions.ML.PredictionEnginePoolExtensions.Predict[TData,TPrediction](PredictionEnginePool`2 predictionEnginePool, String modelName, TData example)
         at Microsoft.Extensions.ML.PredictionEnginePoolExtensions.Predict[TData,TPrediction](PredictionEnginePool`2 predictionEnginePool, TData example)
         at Program.<>c.<<<Main>$>b__0_2>d.MoveNext() in C:\Users\luquinta.REDMOND\source\repos\ReactorContainerAppsLive\TimeSeries_WebApi1\Program.cs:line 34
      --- End of stack trace from previous location ---
         at Microsoft.AspNetCore.Http.RequestDelegateFactory.<ExecuteTask>g__ExecuteAwaited|58_0[T](Task`1 task, HttpContext httpContext)
         at Microsoft.AspNetCore.Http.RequestDelegateFactory.<>c__DisplayClass46_3.<<HandleRequestBodyAndCompileRequestDelegate>b__2>d.MoveNext()
      --- End of stack trace from previous location ---
         at Microsoft.AspNetCore.Routing.EndpointMiddleware.<Invoke>g__AwaitRequestTask|6_0(Endpoint endpoint, Task requestTask, ILogger logger)
         at Swashbuckle.AspNetCore.SwaggerUI.SwaggerUIMiddleware.Invoke(HttpContext httpContext)
         at Swashbuckle.AspNetCore.Swagger.SwaggerMiddleware.Invoke(HttpContext httpContext, ISwaggerProvider swaggerProvider)
         at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)

The reason for this error most likely is due to PredictionEngine and TimeSeriesPredictionEngine work slightly differently.
The generated code might need to change for this specific scenario. Here's a working sample of a MinimalWebAPI using TimeSeriesPredictionEngine.
https://github.com/luisquintanilla/mlnet-minimal-weather. Note that this sample uses a preview version of .NET 6 so the updated version would look closer to the existing template.

@LittleLittleCloud Can you take a look?

Can someone provide a fix for this?

@LittleLittleCloud Is this fixed? Jake thinks it might be fixed.

IIRC, I tried with the latest release and it still wasn't fixed.

@LittleLittleCloud to investigate

@v-Hailishi can you please validate whether this has been fixed in the latest version. Thanks.

@luisquintanilla I have tried on the latest main branch <IP_ADDRESS>6201 for VS2022, the webapi.program.cs has been updated like below and Web API project also can be run successfully.

Perfect. Thanks @v-Hailishi @LittleLittleCloud
Closing this issue.


Remove Travis CI configuration.

Fixes #54.

Codecov Report

Merging #59 (9331496) into develop (146b774) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #59   +/-   ##
========================================
  Coverage    90.08%   90.08%
========================================
  Files           10       10
  Lines          464      464
========================================
  Hits           418      418
  Misses          46       46

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 146b774...9331496. Read the comment docs.


Build failing npm install

The command "npm install -g istanbul mocha" failed and exited with 1 during .
See full log here: https://s3.amazonaws.com/archive.travis-ci.org/jobs/167065453/log.txt

Gladly it is passing at the second try. :-)


Include .json file in the package

User can install this package non-editable.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.林均鹏 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Will reopen this because cant sign CLA


AddressSanitizer failed to deallocate

==22977== ERROR: AddressSanitizer failed to deallocate 0x23000 (143360) bytes at address 0x7fff680d0000
==22977== AddressSanitizer CHECK failed: ../../../../src/libsanitizer/sanitizer_common/sanitizer_posix.cc:81 "(("unable to unmap" && 0)) != (0)" (0x0, 0x0)
==22977== ERROR: AddressSanitizer failed to deallocate 0x80000 (524288) bytes at address 0x7fff67fc7000
==22977== AddressSanitizer CHECK failed: ../../../../src/libsanitizer/sanitizer_common/sanitizer_posix.cc:81 "(("unable to unmap" && 0)) != (0)" (0x0, 0x0)
==22977== ERROR: AddressSanitizer failed to deallocate 0x200000 (2097152) bytes at address 0x7fff67da4000
==22977== AddressSanitizer CHECK failed: ../../../../src/libsanitizer/sanitizer_common/sanitizer_posix.cc:81 "(("unable to unmap" && 0)) != (0)" (0x0, 0x0)
==22977== ERROR: AddressSanitizer failed to deallocate 0x100000 (1048576) bytes at address 0x7fff67c81000
==22977== AddressSanitizer CHECK failed: ../../../../src/libsanitizer/sanitizer_common/sanitizer_posix.cc:81 "(("unable to unmap" && 0)) != (0)" (0x0, 0x0)
==22977== ERROR: AddressSanitizer failed to deallocate 0x4000 (16384) bytes at address 0x7ffff7fdd000
==22977== AddressSanitizer CHECK failed: ../../../../src/libsanitizer/sanitizer_common/sanitizer_posix.cc:81 "(("unable to unmap" && 0)) != (0)" (0x0, 0x0)
==22977== ERROR: AddressSanitizer failed to deallocate 0x23000 (143360) bytes at address 0x7fff66f00000
==22977== AddressSanitizer CHECK failed: ../../../../src/libsanitizer/sanitizer_common/sanitizer_posix.cc:81 "(("unable to unmap" && 0)) != (0)" (0x0, 0x0)

What version are you running?  I am seeing the same errors with 7.1.0.

7.1.0

Also reported via issue #1498.  I think there is a problem with freeing memory if the free list is deallocated (-f).  When I accidentally left the -f flag on after I reinstalled a non-ASAN build, my system crashed due to out of memory within minutes.  Without the -f flag, my production box has been running for days without appreciable memory growth.

Duplicate of #1498.


LAN issue (probably my fault 🙃)

FabricTPAFixTitles is causing a ClassDefNotFound error, presumably because it has the side set to server, and I was testing on a client LAN server. I was also testing on 21w10a just to see if the snapshot broke it, so I am going to test on LAN server again on 1.16.4 and send an update
Crash Report

AFAIK FabricTPA won't be loaded in LAN, so idk how you even got an error /shrug

I edited the fabric.mod.json to have side set to * instead of server

Well that was stupid :facepalm: I just tested it on 1.16.5 and of course it didn't cause any errors. FixTitles is only called if version is ^21w08a. Should have tested on 21w08a ._.

Tested on 21w08a, still got class not found error, so im 99% sure it's just because the library mod is set to be server sided only, going to quickly fix that


Psat not calculated in most robust way from IV curves

As I understand it, IV data is flipped during analysis, so that the first element in an array will be the last datapoint that was taken, and vice-versa (see this comment) https://github.com/simonsobs/sodetlib/blob/c7bc19793ccfd168435885930d819fb58a506e8e/sodetlib/operations/iv.py#L326
In one sense that makes looking at the data confusing, because it does not match the actual time order, but in another sense more intuitive, because the left-to-right direction on most IV plots matches the start-to-finish order of the data arrays, so, fine.
Where it does run into a problem is computing Psat values. Currently, the code looks for the first time R/R_n is above psat_level:
https://github.com/simonsobs/sodetlib/blob/c7bc19793ccfd168435885930d819fb58a506e8e/sodetlib/operations/iv.py#L218-L228
But because IV curves can be ill-behaved in the low voltage, superconducting portion of the IV curve, which in this analysis scheme comes first, the code should instead look for the last time R/R_n is below psat_level. So something like
        cross_idx = np.where(R/R_n < level)[0]

        if len(cross_idx) == 0:
            iva.p_sat[i] = np.nan
            continue

        # Takes cross-index to be the last time Rfrac crosses psat_level
        cross_idx = cross_idx[-1]
        if cross_idx == 0:
            iva.p_sat[i] = np.nan
            continue

I have yet to make and run with this change to make sure this fixes the problem, but thought I'd raise this issue.

I finally got around to addressing this issue. The plot below illustrates the problem, where the p_sat calculation is thrown-off by an instability in the low-R_tes portion of the IV curve. Note that target bias voltages are not affected by this.

Also, the full P_sat distribution looks like the below. Note all the values between 0 pW and 16 pW

I then made the committed changes to the code, switching the logic from the first time R is above the target r_frac to the last time R is below the target r_frac. This fixed the psat determination in the selected detector, and it also fixed the total psat histogram to get rid of those spurious low values.

Tested on a full UHF UFM in a Princeton test cryosat.


Py Deprecation Warning

I am running A1111 in Stability Matrix. Is this something that you are able to fix?
Or should I post this at Stability Matrix ?
Installing requirements for Face Editor
D:\STABLEDIFF\Data\Packages\stable-diffusion-webui\extensions\sd-webui-controlnet\install.py:3: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
import pkg_resources
D:\STABLEDIFF\Data\Packages\stable-diffusion-webui\extensions\sd-webui-faceswaplab\install.py:4: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
Thank You !

This error is not from Face Editor, but from sd-webui-controlnet and sd-webui-faceswaplab, so I think you should contact them 😃

Thank you !


[safety-rules] error when no execution pubkey, but a vote proposal has a signature

I can't think of any malicious scenario that would trigger this, but this is probably a misconfiguration and we want to be alerted as soon as possible that something is going on.
Note that someone could use this to DoS the TCB by adding a signature to this request when the TCB is not configured to handle execution signatures. But there are many other ways to DoS the TCB (for example by sending an initialization request with a fake epoch change proof) so the assumption is that malicious consensus performing a DoS are out of scope.

makes sense : o


Use a Yargs command for wiby test

As per https://github.com/pkgjs/wiby/pull/3#discussion_r446881882

:tada: This issue has been resolved in version 0.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 0.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Switch from geiser to racket-mode

As per the discussion at https://github.com/jaor/geiser/issues/187, it would probably be more "Racket-like" to use racket-mode rather than geiser.

I haven't used Racket in years, and nobody else has complained about this. Rather than migrate, I will simply remove Geiser. The user can add their preferred Scheme mode in local configuration.


MINIFICPP-2009 - CWEL should add resolved attributes with json output…

… as well
Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] If applicable, have you updated the LICENSE file?
[x] If applicable, have you updated the NOTICE file?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI results for build issues and submit an update to your PR as soon as possible.

Currently we have the options XML, Plaintext, Both, JSON for Output Format. The "Both" option is quite ambiguous now that we have 3 different output formats, should we revise this? Either have an "All" option for all 3 output formats, or rename it to "XML and Plaintext'.

Currently we have the options XML, Plaintext, Both, JSON for Output Format. The "Both" option is quite ambiguous now that we have 3 different output formats, should we revise this? Either have an "All" option for all 3 output formats, or rename it to "XML and Plaintext'.

Yeah thats a bit confusing I agree. I think we should address that. (But I dont see an easy fix without breaking compatibility with previous configurations and seems out of the scope of this PR)
I've created a jira for this https://issues.apache.org/jira/browse/MINIFICPP-2042


Market Graph in market watcher in UI is inaccurate

[X ] bug report
[ ] question about the decisions made in the repository
I notice a discrepancy between the price "Market Graph" in the live market watcher (gdax / 1 minute candles) and the close price returned by the running getCandle function in the background.
Trying to understand how the price line is plotted to the chart in order to debug this, any idea how the .svg chart data points are calculated from the candles data?
Anyone noticed the same?

After a deeper analysis of the data, it seems that the chart plots "open" prices, not "close", hence the discrepancy. Was this the intended behavior?


Improve output from Invoke-DBODeployment

Should return more than a simple object returned from DbUp. Should probably be a custom class that would be accepted through a pipeline by future helper functions: send emails, invoke tSQLt tests, PowerBI reports, etc.

resolved in #27


ShopifyAPI::Rest::Base#initialize has a from_hash argument but subclasses override it out

Issue summary
There's a seeminglyl useful from_hash: argument in ShopifyAPI::Rest::Base#initialize, but it's not usable because all subclasses override #initialize.
# base.rb
def initialize(session: nil, from_hash: nil)
   # elided code

  from_hash&.each do |key, value|
    set_property(key, value)
  end
end

# asset.rb
def initialize(session: ShopifyAPI::Context.active_session)
  super(session: session)
  # elided code
end

Expected behavior
Initialize a ShopifyAPI::Rest::* object with #new(from_hash: my_hash)
Actual behavior
ArgumentError: unknown keyword: :from_hash

Steps to reproduce the problem
Specifications

shopify_api version: 13.2.0
Shopify API version used: 2023-10):

Up! this function is still featured quite prominently in the latest version of doc:
https://github.com/Shopify/shopify-api-ruby/blob/6c2a45f8377ac46217f9dd24068667ccd2a4ee5c/docs/usage/rest.md?plain=1#L51


⚠ Samsung has degraded performance

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

HTTP code: 200
Response time: 3101 ms

Resolved: Samsung performance has improved in 7121a4b.


Adding deps to setup.py

currently have to install deps separately. This will install them during setup.py. Begins to enable direct installation via git+https://github.com/microsoft/CDM

Just as FYI example of what happens without this...

hey @dciborow,
I've worked on making this changes to the SDK internally. It should be available on Github on our next release that should happen next week.

@victor-gutemberg how can i move forward with this?
I am pulling the 1.2.0 version and can not install it because of the various dependancy hard pins. I have updated those pins here as well.


fix rpms

ping @calavera this will need to be included in the release, I'm building now but fingers crossed looks like its going to work, we should wait just in case to merge ;)

grosssss
LGTM
(for the curious, this comes from https://github.com/CentOS/sig-cloud-instance-images/issues/24)

Cool, thank you, LGTM too but I'll wait until you're sure this works.

yayyyy it worked! we should have rpms now after the fedora ones finish building

ok this can be merged

Cherry picked into #14264 :raised_hands: