Segmentation fault with Webpack & node sass

Same issue as https://github.com/webpack/webpack/issues/2371, not sure if this is a 3.5.2 issue or not?

Tested w/ 3.4.2 and doesn't exist

We're having the same issue as @robcalcroft
Although, 3.5.3 (fix: https://github.com/sass/node-sass/commit/ca96aa71d10055812a00bd0eaf3204d11afc411f) has been tagged, but isn't available on npm, yet.

@steadweb It is now
https://www.npmjs.com/package/node-sass

I don't believe the segfault is related to ca96aa7. This typo was never in a released version of Node Sass.

@robcalcroft please confirm your segfault still happens with<EMAIL_ADDRESS>

3.5.3 doesn't fix the segfault issue.

@steadweb can you please create a small github repo that reproduce this issue so I debug it locally?

+1

Same problem, came back to 3.4.2 works fine

Gonna vote for this one. I build by software with docker, and the webpack fails with segfault. I'm trying to go back to 3.4.2.

These comments are not helpful. Saying "I have a problem" does not give me any information I can use to fix this problem.
The only way to progress this issue is to create a small github repo that produces this error so I can debug it and create a patch. Help me help you.

Seems to be issue with libsass https://github.com/sass/libsass/issues/2017
I am using grunt-sass so had to created a fork to roll back to node-sass 3.4.2 https://github.com/liamkeaton/grunt-sass

@liamkeaton Grunt-sass caused my issue also, explicitly defining node-sass as a dep at 4.3.2 worked and meant I didn't need to fork. https://github.com/jadu/pulsar/pull/167#merged-event
@xzyfer I don't have time right now to try track down the specific sass throwing the seg fault but I'll see if I can pinpoint it this evening/tomorrow.

@Stanton awesome, didn't realise that would work!

@Stanton @liamkeaton I don't believe that works if you do a clean npm install. I'm still getting the later version from the package that depends on it.

@travm from my tests earlier today I'm pretty sure it works if you use '3.4.2' and not '^3.4.2'

@travm it has also worked for me from a clean install

@Stanton thanks works perfectly

+1, getting seg faults with latest 3.5.x node-sass. Switched package.json to ~3.4.2 and thing are working again.

Error in 3.5.* version, i changed to the 3.4.2 and worked well,
my problem is that i can't use any Mixin, or Include of SCSS

if it helps at all the root of the 'Backtrace' that gets printed out stems from this in our error logs:
at options.error (PROJECT_PATH/node_modules/node-sass/lib/index.js:274:32)
If I have more time I will try to get a basic example up..

Seeing this via grunt-sass - confirmed to break on osx with node 5.10, centos running jenkins, and debian linux via docker... had to fork and change the node-sass version because grunt-sass is using the ^ for versions...

I'm also seeing this when building materialize-css with webpack.
I was able to track it down to this line of sass: https://github.com/Dogfalo/materialize/blob/v0.97.5/sass/components/_buttons.scss#L169
Removing that line (or downgrading node-sass versions) fixed the issue.

Everbody: without a self contained sample that you can confirm segfaults we cannot do much here. :exclamation: Everything else is not helping us to understand the problem :exclamation: Somebody needs to take the time to strip down his case to the bare minimum to show us the specific problem.

Trouble is @mgreter it isn't as simple as writing a "self contained sample" when really basic examples work as expected. The application that I work on has many other dependencies and the scss compilation isn't small. I understand you guys need testing steps in order to replicate, but to have that to hand and know where the problems lies may take some time.

I'm trying to put together a small example, but not having much luck yet.
I have, however, isolated a single @import in our Sass files which, when removed, results in successful compilation. Early signs point to some combination of nested imports, extends and/or mixins which I'll try and replicate in a clean repo, but I'm not sure when that will be.
The line in question for me: https://github.com/jadu/pulsar/blob/develop/stylesheets/_component.remove-button.scss#L2 I appreciate that this may not be useful information on it's own.

I can only further suggest to also check your code base with official ruby sass and also sassc (or any other binding that links to libsass with the same version). Then we would at least know if the segfault happens in libsass or node-sass binding. I also wonder if your code base produces an error in ruby sass and your seeing a variant of https://github.com/sass/libsass/issues/1962 in combination of https://github.com/sass/node-sass/issues/1457.

@mgreter I can confirm that the issue is probably is either in libsass or node-sass. Official ruby sass works fine, node-sass segfaults.
Command I ran:
$ sass main.scss out.css
$ node-sass main.scss out.css

main.scss
@charset "UTF-8";

.mcss {
    @import 'materialize';
}

Where _materialize.scss is the SCSS entry point for the Materialize framework, which itself is just a bunch of imports.

Here you go, @mgreter, tested this as @prashcr stated, and it segfaults -
package.json (npm start):
{   "name": "node-sass-breakage",   "version": "1.0.0",   "description": "",   "main": "index.js",   "scripts": {     "start": "node-sass sassfile.scss output.css"   },   "author": "",   "license": "ISC",   "dependencies": {     "materialize-css": "0.97.6",     "node-sass": "3.5.3"   } }
sassfile.scss
@charset "UTF-8"; @import 'node_modules/materialize-css/bin/materialize-src/sass/materialize';

Is that really a minimal test case? It pulls in all of the sass of materialize, which is a lot of content for the node-sass developers to find the issue in.
My 2c, but you're the guy who put in the effort to make the minimal test case. So, thanks, too!

Great point, but I tried to put a minimal version with 2 or 3 imports, and it worked fine... seems to be an issue with bigger projects.

At least it's straight forward to setup and the first one to actually post something we can try ourself without too much guessing on our part. I can confirm the segfault and incidentaly it is already fixed on master -> https://github.com/sass/libsass/pull/2022. So this will be fixed in next versions.

I guess node-sass keeps issues open until libsass is updated ...

The segfault with materialize-css has been fixed in v3.6.0. I'll publish it as stable in 48hrs after my weekend. Until then you can install is using the next tag.
npm install node-sass@next

or in your package.json
"node-sass": "next",

<EMAIL_ADDRESS>was released today which fixes the segfault with normalize-css

The below code will still cause<EMAIL_ADDRESS>to encounter this error
Notice the error in line 2
ul.menu {
  width:
  li {
    margin: 5px auto;
    a {
      color: #333;
    }
  }
}

Screenshot:
xieran@xieran-PC MINGW64 /d/Playground/micloud-pc/app/modules/icons (master)
$ cat a.scss
ul.menu {
  width:
  li {
    margin: 5px auto;
    a {
      color: #333;
    }
  }
}

xieran@xieran-PC MINGW64 /d/Playground/micloud-pc/app/modules/icons (master)
$ node-sass a.scss
/c/Users/xieran/AppData/Roaming/npm/node-sass: line 14:  3924 Segmentation fault      node "$basedir/node_modules/node-s
ass/bin/node-sass" "$@"

xieran@xieran-PC MINGW64 /d/Playground/micloud-pc/app/modules/icons (master)
$ node-sass -v
node-sass       3.8.0   (Wrapper)       [JavaScript]
libsass         3.3.6   (Sass Compiler) [C/C++]

xieran@xieran-PC MINGW64 /d/Playground/micloud-pc/app/modules/icons (master)
$

@xieranmaya this seems unrelated to this issue and you may want to open a new ticket for that. Just checked perl-libsass bindings, and [it shows the correct error] 1 without a segfault.

@mgreter so it seems a bug of node-sass or node?

@xleranmaya you can see that the code supplied errors as expected with the latest node-sass on sassmeister.
There appears be an environmental issue. I suggest cleaning your local environment
rm -rf node_modules
npm cache clean

After a fresh npm install, if you're still experiencing issues please provide the output the following commasds
npm -v
node -v
node -p process.versions
node -p process.platform
node -p process.arch
.\node_modules\.bin\node-sass --version
node -p "console.log(require('node-sass').info)"
dir node_modules\node-sass\vendor

I'm getting this message: Segmentation fault
just after sass started.
 Alpine support was only added in<EMAIL_ADDRESS>

Hi, we are still seeing a seg-fault.
os: alpine (node:8-alpine docker image)
node: 8
npm: 5
webpack 3.0.0
node-sass: 4.5.3
It works if we use very simple sass.

@torvitas see #2031 for alpine issues

@torvitas I was having this segmentation fault just now, and I was based on an alpine 3.6 docker image.
I "just" changed my Dockerfile to start FROM alpine:3.7 and the segmentation fault went away.


Fix typos + minor edits.

Thanks for creating such a wonderful module!
This PR addresses the following:

I noticed that there was a typo in README.md: New-Empahsis.
Also, fixed misspelling in function help.
I also believe this is a bug which I fixed.

Sorry for all the whitespace changes 😄
I think I have VS Code set to format documents on save and it trims whitespace at end of line.

Hi @endowdly, did you get a chance to review my PR further?
Thanks.

I had approved this back in July and for some reason didn't merge. I've been caught up in a cross-country move and at this point I cannot remember the reason for the delay (if any). Good PR!

No worries. Thanks for accepting my PR. All the best for your move!
I've moved cross-country twice before and it was not fun 😅


linter not doing anything on MacOS

My linter-pyflakes is doing nothing in Atom.
Configuration:

MacOS X 10.9.5
Atom 0.176.0
linter 0.11.1 (enabled)
linter-pyflakes 0.0.4 (enabled, with pyflakes path = /Library/Frameworks/Python.framework/Versions/2.7/bin/)
Atom command-line tools installed
python.org python 2.7.9
pyflakes 0.8.1 at the path listed above

The console (Vew:Toggle Developer Tools) shows one entry, which looks fine to me:
/Users/rowen/Dropbox/Config/atom/packages/linter-pyflakes/lib/init.coffee:7 activate linter-pyflakes
index.js:54 Window load time: 1439ms
When I edit a python file and intentionally introduce errors I see no evidence of any linting.
Any idea what to do? This has been going on for a long time (with older versions of everything) which is one reason Atom is not my primary editor. I see one ticket on this got fixed, but apparently there are other causes.

As a followup I tried linter-flake8 and it works fine, so it appears to be a bug in linter-pyflakes. I'll re-file this against linter-pyflakes.
I'm not keen on flake8, but with enough errors and warnings suppressed I find the results useful. Here is my suppression list, in case anyone is interested: E112, E113, E121, E122, E123, E124, E125, E126, E127, E128, E129, E131, E133, E201, E202, E203, E211, E221, E222, E225, E226, E227, E228, E231, E241, E251, E261, E262, E265, E271, E272, E301, E302, E303, E502, W292, W293, W391

Closing in favour of AtomLinter/linter-pyflakes#4 - let's keep discussion in one place, please.


Bloxstrap Programs don't start and end Roblox immediately

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

What problem did you encounter?
[?]
I have checked around the Wiki yea there the solution of these problems but i didn't have roblox installed so it might not.
I have tried uninstall the Bloxstrap and launch Roblox instead it work fine, but when i back to used Bloxstrap it not working
Showing this screen then disappeared
Im using .Net latest version too.
the page i read
https://github.com/pizzaboxer/bloxstrap/wiki/Roblox-crashes-or-does-not-launch
https://github.com/pizzaboxer/bloxstrap/wiki/Bloxstrap-doesn't-run-when-opening-it
https://github.com/pizzaboxer/bloxstrap/wiki/Bloxstrap-doesn't-launch-with-Roblox
https://github.com/user-attachments/assets/e0bd01a3-1d9e-4a37-a514-4bebad896bae
Bloxstrap Log
2024-10-04T07:51:41Z [App::OnStartup] Starting Bloxstrap v2.7.0
2024-10-04T07:51:41Z [App::OnStartup] Compiled Wednesday, 10 July 2024 at 1:03:19 AM from commit a7d2a2c3d5af7137804416ba4e5fbdd27bd10f1e (tag/v2.7.0)
2024-10-04T07:51:41Z [App::OnStartup] Loaded from %UserProfile%\AppData\Local\Bloxstrap\Bloxstrap.exe
2024-10-04T07:51:41Z [LaunchSettings::Parse] Parsing launch arguments
2024-10-04T07:51:41Z [JsonManager::Load] Loading from %UserProfile%\AppData\Local\Bloxstrap\Settings.json...
2024-10-04T07:51:41Z [JsonManager::Load] Loaded successfully!
2024-10-04T07:51:41Z [JsonManager::Load] Loading from %UserProfile%\AppData\Local\Bloxstrap\State.json...
2024-10-04T07:51:41Z [JsonManager::Load] Loaded successfully!
2024-10-04T07:51:41Z [JsonManager<Dictionary2>::Load] Loading from %UserProfile%\AppData\Local\Bloxstrap\Modifications\ClientSettings\ClientAppSettings.json... 2024-10-04T07:51:41Z [JsonManager<Dictionary2>::Load] Loaded successfully!
2024-10-04T07:51:41Z [FastFlagManager::SetValue] Changing of 'FFlagHandleAltEnterFullscreenManually' from 'False' to 'False' is pending
2024-10-04T07:51:41Z [ProtocolHandler::ChangeChannel] Ignored channel change from LIVE to zassetcacheinitprod because Roblox is already running
2024-10-04T07:51:41Z [JsonManager::Save] Save request ignored
2024-10-04T07:51:41Z [RobloxDeployment::TestConnection.https://setup.rbxcdn.com] Connecting...
2024-10-04T07:51:41Z [HttpClientLoggingHandler::ProcessRequest] GET https://setup.rbxcdn.com/version
2024-10-04T07:51:41Z [RobloxDeployment::InitializeConnectivity] Testing connectivity...
2024-10-04T07:51:42Z [HttpClientLoggingHandler::ProcessResponse] 200 OK https://setup.rbxcdn.com/version
2024-10-04T07:51:42Z [RobloxDeployment::InitializeConnectivity] Got https://setup.rbxcdn.com as the optimal base URL
2024-10-04T07:51:42Z [Logger::Initialize] Initializing at %UserProfile%\AppData\Local\Bloxstrap\Logs\Bloxstrap_25671004T075142Z.log
2024-10-04T07:51:42Z [Logger::Initialize] Finished initializing!
2024-10-04T07:51:42Z [NotifyIconWrapper::NotifyIconWrapper] Initializing notification area icon
2024-10-04T07:51:42Z [App::OnStartup] Initializing bootstrapper
2024-10-04T07:51:42Z [App::OnStartup] Initializing bootstrapper dialog
2024-10-04T07:51:42Z [Bootstrapper::Run] Running bootstrapper
2024-10-04T07:51:42Z [Bootstrapper::Run] Performing connectivity check...
2024-10-04T07:51:42Z [Bootstrapper::SetStatus] Connecting to Roblox...
2024-10-04T07:51:42Z [RobloxDeployment::GetInfo] Getting deploy info for channel LIVE (extraInformation=False)
2024-10-04T07:51:42Z [HttpClientLoggingHandler::ProcessRequest] GET https://clientsettingscdn.roblox.com/v2/client-version/WindowsPlayer/channel/LIVE
2024-10-04T07:51:42Z [HttpClientLoggingHandler::ProcessResponse] 200 OK https://clientsettingscdn.roblox.com/v2/client-version/WindowsPlayer/channel/LIVE
2024-10-04T07:51:42Z [Bootstrapper::Run] Connectivity check finished
2024-10-04T07:51:42Z [Bootstrapper::CheckForUpdates] Checking for updates...
2024-10-04T07:51:42Z [HttpClientLoggingHandler::ProcessRequest] GET https://api.github.com/repos/pizzaboxer/bloxstrap/releases/latest
2024-10-04T07:51:42Z [HttpClientLoggingHandler::ProcessResponse] 200 OK https://api.github.com/repos/pizzaboxer/bloxstrap/releases/latest
2024-10-04T07:51:42Z [Bootstrapper::CheckForUpdates] No updates found
2024-10-04T07:51:42Z [RobloxDeployment::GetInfo] Getting deploy info for channel LIVE (extraInformation=False)
2024-10-04T07:51:42Z [RobloxDeployment::GetInfo] Deploy information is cached
2024-10-04T07:51:42Z [HttpClientLoggingHandler::ProcessRequest] GET https://setup.rbxcdn.com/version-d2bde6b0a05e4840-rbxPkgManifest.txt
2024-10-04T07:51:42Z [HttpClientLoggingHandler::ProcessResponse] 200 OK https://setup.rbxcdn.com/version-d2bde6b0a05e4840-rbxPkgManifest.txt
2024-10-04T07:51:42Z [JsonManager<Dictionary2>::Save] Saving to %UserProfile%\AppData\Local\Bloxstrap\Modifications\ClientSettings\ClientAppSettings.json... 2024-10-04T07:51:43Z [JsonManager<Dictionary2>::Save] Save complete!
2024-10-04T07:51:43Z [Bootstrapper::ApplyModifications] Roblox is running, aborting mod check
2024-10-04T07:51:43Z [Bootstrapper::CheckInstall] Checking install
2024-10-04T07:51:43Z [JsonManager::Save] Saving to %UserProfile%\AppData\Local\Bloxstrap\Settings.json...
2024-10-04T07:51:43Z [JsonManager::Save] Save complete!
2024-10-04T07:51:43Z [JsonManager::Save] Saving to %UserProfile%\AppData\Local\Bloxstrap\State.json...
2024-10-04T07:51:43Z [JsonManager::Save] Save complete!
2024-10-04T07:51:43Z [Bootstrapper::SetStatus] Starting {product}...
2024-10-04T07:51:43Z [Bootstrapper::StartRoblox] Started Roblox (PID 7728)
2024-10-04T07:51:43Z [Bootstrapper::StartRoblox] Using Discord Rich Presence
2024-10-04T07:51:43Z [DiscordRichPresence::DiscordRichPresence] Established connection with Discord RPC
2024-10-04T07:51:44Z [DiscordRichPresence::DiscordRichPresence] Received ready from user ᲼<PHONE_NUMBER>29988106)
2024-10-04T07:51:46Z [ActivityWatcher::StartWatcher] Opening Roblox log file...
2024-10-04T07:51:46Z [NotifyIconWrapper::InitializeContextMenu] Initializing context menu
2024-10-04T07:51:46Z [ActivityWatcher::StartWatcher] Opened %UserProfile%\AppData\Local\Roblox\logs\0.645.1.6450667_20241004T075144Z_Player_D3DCE_last.log
2024-10-04T07:51:46Z [Bootstrapper::StartRoblox] Waiting for Roblox to close
2024-10-04T07:51:50Z [Bootstrapper::StartRoblox] Roblox has exited
2024-10-04T07:51:50Z [DiscordRichPresence::Dispose] Cleaning up Discord RPC and Presence
2024-10-04T07:51:50Z [App::OnStartup] Bootstrapper task has finished
2024-10-04T07:51:50Z [NotifyIconWrapper::Dispose] Disposing NotifyIcon
2024-10-04T07:51:50Z [MenuContainer::Window_Closed] Context menu container closed
2024-10-04T07:51:50Z [App::OnStartup] Waiting for bootstrapper task to finish
2024-10-04T07:51:50Z [App::StartupFinished] Successfully reached end of main thread. Terminating...
2024-10-04T07:51:50Z [App::Terminate] Terminating with exit code 0 (ERROR_SUCCESS)

Additional to said that : it say roblox still running when i try to uninstall Bloxstrap ,so there might able to push solution.

Roblox produces a log file at %localappdata%\Roblox\logs\0.645.1.6450667_20241004T075144Z_Player_D3DCE_last.log, can you send it to me privately over email please<EMAIL_ADDRESS>

Just to confirm, have you sent it?

Did you read this section of "Roblox crashes or does not launch" at all?

You said you read the wiki page on "Roblox crashes or does not launch". Did you read this section at all?
I'll reopen when you confirm you've read and tried this and the issue still persists.

i have reinstalled the bloxstrap with no fastflag setting on it work properly, but when i attempt just turn on fastflag reminding that every setting is by default no change at all, just try turn that on my roblox doesn't run anymore, then i have to reinstall for it to work again

but when i attempt just turn on fastflag reminding that every setting is by default

Show a screenshot of the flag editor please


WebAC ACL Interface

Would this interface be able to support a method that takes a resource and/or an agent (as URIs) and returns the authorization(s) that mention that resource and/or agent? Alternatively, a method that returned the list of authorizations in the ACL, and the client class could iterate over that list to find the authorizations that were relevant to its interests? If either of these are desired, I would suggest an additional interface/class to represent a single authorization ("authorization" being the WebAC terminology for a single rule relating an agent(Class) to an accessTo(Class) resource with an access mode.

@peichman-umd, Although per the WebAC "spec", agents are represented as URIs. My understanding was that most installations would only be able to support Strings.
How are you defining "authorizations"? I thought this interface represented a single "authorization", i.e. all principals that have these modes on these resources (and resource classes).

@awoods, The way I read the WebAC "spec", an ACL is a graph containing one or more acl:Authorization resources, which all seem to be represented as blank nodes with acl:accessTo, acl:agent, acl:mode, etc. properties. Thus I was assuming that an ACL class would represent that graph.
Maybe this is a case of terminological confusion? If so, I would suggest renaming this interface to something like WebACAuthorization.

@awoods, For the issue of representing agents as strings or URIs, would the new acct: URI scheme (RFC 7565) help at all? It is specifically designed to make no commitment about how the URIs are resolved, it is just for naming accounts on a particular system.  The only issue might be what to put in the host section of the URI.

From: https://wiki.duraspace.org/display/FF/2015-07-22+-+WebAccessControl+Authorization+Delegate+Planning+Meeting
"Probably not using URIs for agents at least at first"

Those acl:Authorizations don't have to be blank nodes, right? They could be something more sane, like hash-URI nodes.

My grasp of some of the W3C lingo is weak. But based on this
"Note that there are properties of an Authorization which allow you to specify who gets access either by giving a relation to a specific user or a different relation to a class of user. The same technique is used for expressing to which resource or class of resources access is being granted."
My assumption is that one Authorization contains:

1 or more triples with a predicate from the set [acl:agent, acl:agentClass]
1 or more triples with a predicate from the set [acl:accessTo, acl:accessToClass]
1 triple with the predicate acl:mode

@whikloj, I had a similar representation in mind. I think we need to take a moment and decide on a clear, clean, consistent perspective on how Authorizations are represented in the repository.

@ajs6f Yes, the Authorizations could definitely be hash-URI nodes.
@whikloj Yes, that fits with my reading of Authorization as well.

Ok, I can change the name to WebACAuthorization.
As for your first question @peichman-umd.
As the acl:mode will be the same for all users and resources referenced by this ACL.
The only check is whether the agent exists in the ACL. Not sure this is the right place for that check.

Is there a reason to even create hash resources? In other words, does an ACL resource need to contain a sub-graph of Authorizations, versus simply having all of the triples @whikloj described attached to the ACL resource itself?

@awoods, I don't think so. Based on my understanding we would have simple triples on the Acl resource where the Acl resource would always be the subject.

@whikloj: The way I have been reading the W3C spec is that the ACL is never actually the subject of any triples. There is an acl:acl property on a protected resource that identifies the location of its ACL, but that is just a URI to be retrieved.

@peichman-umd, are you saying that your understanding is that "protected resources" have a property that points to their ACL?

@awoods That was my understanding of this section of the spec: http://www.w3.org/wiki/WebAccessControl#Modifying_Access_Control_information

@peichman-umd, yes I see what you mean.
But then I can't understand how we handle the accessToClass functions. If the acl is not the subject of the triple that provides acl:accessToClass to rdf:type, what is?

@whikloj, I think it is the Authorization. Note that the ontology does not actually define a class for the ACLs, which suggests that they are not intended to be RDF resources themselves. Instead, I think the ACL is supposed to be used as just a URL where a graph of authorizations can be retrieved.

I have no understanding of nor opinion about this issue, but just from the diction and tone of this conversation, perhaps this is a question best put to whomever is writing or maintaining WebAC?

Also, from the WAC and LDP section:

So even though the acl file is neither an LDPR nor an LDPC it should be editable using the same methods.

This seems a bit confusing: maybe check with the (ex-)LDP WG for some clarification? @azaroth42?

:+1:

:+1:

I agree, what fine work. Someone press the button.

Someone should squash the commits before pressing the button.

I'll give it a try, never done that before.

i.e.

git checkout --track origin pull/4
git log
git rebase -i

"pick" first, "s" others
update comment
git push origin pull/4:master

Assuming you already did a "git pull" with the following ".git/config" update:
[remote "origin"]
        url = https://github.com/fcrepo4-labs/fcrepo-module-auth-webac.git
        fetch = +refs/heads/*:refs/remotes/origin/*
        fetch = +refs/pull/*/head:refs/remotes/origin/pull/*

I hope that works.

I does not look like it. Now I see 7 commits instead of one squashed commit. Was that intentional, @whikloj?

Sorry, I should have asked. I just removed some of the commits. I'll squash it all.


Get digital signature

Hello everyone, I'm trying to get the digital signature on a sign excel file (xlsm extension), I had review all the object returned but I can't find it, Anyone know how can I approach this?
Thanks in advance

Digital signatures are not currently supported.  They are supported in some of our Pro builds and may eventually make their way to the open source, but it will definitely be after v1.0.0


Error in codelist-mapping.xml causing validator problems

There’s a bug with codelists mappings that’s fixed in this PR:
https://github.com/IATI/IATI-Codelists/pull/109
The same bug is repeated here:
https://github.com/IATI/pyIATI/blob/f3c9b54dd3fb6d5e6fc48b3329e6b9a9682d1318/iati/resources/standard/202/codelist-mapping.xml#L60
So iati.default.codelist_mapping() loads that… And that’s used by the validator:
https://github.com/IATI/pyIATI/blob/b454bdae51bf706fcb6e46a7c0a0a4b3b27d8e09/iati/validator.py#L287
So when I run iati.validator.is_valid(dataset, schema) I hit an error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File ".ve/src/pyiati/iati/validator.py", line 693, in is_valid
    return _correct_codelist_values(dataset, schema) and _conforms_with_ruleset(dataset, schema)
  File ".ve/src/pyiati/iati/validator.py", line 500, in _correct_codelist_values
    error_log = _check_codelist_values(dataset, schema)
  File ".ve/src/pyiati/iati/validator.py", line 341, in _check_codelist_values
    error_log.extend(_check_codes(dataset, codelist))
  File ".ve/src/pyiati/iati/validator.py", line 307, in _check_codes
    parents_to_check = dataset.xml_tree.xpath(parent_el_xpath)
  File "src/lxml/etree.pyx", line 2289, in lxml.etree._ElementTree.xpath (src/lxml/etree.c:69770)
  File "src/lxml/xpath.pxi", line 359, in lxml.etree.XPathDocumentEvaluator.__call__ (src/lxml/etree.c:179350)
  File "src/lxml/xpath.pxi", line 227, in lxml.etree._XPathEvaluatorBase._handle_result (src/lxml/etree.c:177714)
lxml.etree.XPathEvalError: Invalid expression

This would likely have been caught with more detailed checks on the SSOT test data.
Slight ramble of my thoughts on some tests that should be added... The SSOT test data is designed for verifying that the Schema permits various types of data. This means that full validation cannot necessarily be undertaken. Many placeholder values are, however, made up of xs - it should be possible to filter error logs of this kind of error, leaving only information that is useful, actionable and relevant.
Do you have a minimal test case Dataset that causes this error? (am assuming a populated default 2.02 Schema)

Do you have a minimal test case Dataset that causes this error? (am assuming a populated default 2.02 Schema)

Yep, exactly – populated default schema and test_data/202/valid_iati.xml does the trick.
from os.path import join
import iati

fn = join('iati', 'resources', 'test_data', '202', 'valid_iati.xml')
with open(fn) as fh:
    xml_str = fh.read()

schema = iati.default.activity_schema()
dataset = iati.Dataset(xml_str)

iati.validator.is_valid(dataset, schema)  # lxml.etree.XPathEvalError: Invalid expression

Looking in test_validator.py, there are tests against Schemas with each specific type of Codelist (complete, incomplete, vocabularies, Language), though nothing against a fully populated Schema - this looks like an oversight when adding populated default Schemas.

The XPath at the point of failure is: //iati-activity/crs-add/crs-add/channel-code[@ext()].
This indicates that the problem is with the assumption that XPaths being checked by a mapping file refer to attributes, rather than text in an element.
This seems like a mis-implementation in the Standard (introduced at 2.02) - Codelists should refer to attribute values, NOT element text.

Github does not allow me to react to that with a 😢  so you will just have to imagine.
Right, yes. You are right. So there are maybe three things to fix, then:

the duplicate crs-add typo, both in the 2.02 SSOT and over here
the issue with the standard that you mention (i.e. channel codes in the text of crs-add/channel-code rather than an attribute)
pyIATI probably needs a workaround for (2) ?

So there are maybe three things to fix

Yep

the duplicate crs-add typo, both in the 2.02 SSOT and over here

pyIATI simply has a copypaste of SSOT content, so at present it's a bug with the SSOT rather than pyIATI - the latter can be fixed once the former is.

the issue with the standard that you mention

I've started the internal process for determining how best to progress with this issue within the constraints of backwards-compatibility and upgrade-schedules.

pyIATI probably needs a workaround for (2) ?

Aye - will look to sort that out now.

and 3) are fixed in https://github.com/IATI/pyIATI/pull/229

In 0.3.0.


esp_https_ota fails with ESP_ERR_INVALID_ARG (IDFGH-1593)

Environment

Development Kit:      None/ESP32D0WDQ5
Kit version: ?
Module or chip used:  ESP32-SOLO-1
IDF version v4.0-dev-1287-gd7e659df2-dirty
Build System:         Make
Compiler version: 1.22.0-80-g6c4433a
Operating System:     macOS
Power Supply:         USB

Problem Description
I'm calling esp_https_ota and it fails with ESP_ERR_INVALID_ARG
This is what I see in the console just before the error:
I (4125) IL_Firmware: HTTP_EVENT_ON_CONNECTED
I (4125) IL_Firmware: HTTP_EVENT_HEADER_SENT
I (4145) esp_https_ota: Starting OTA...
I (4145) esp_https_ota: Writing to partition subtype 16 at offset 0x10000
I (7045) esp_https_ota: Connection closed, all data received
I (7045) IL_Firmware: HTTP_EVENT_DISCONNECTED
I (7045) IL_Firmware: HTTP_EVENT_DISCONNECTED
E (7055) IL_Firmware: Upgrading firmware: Failed: ESP_ERR_INVALID_ARG
And I can't figure out what's causing this error.
I enabled HTTP for OTA in the flash config.
And the url is pointing to the latest build firmware. I assume that i can flash with a version that is not newer?
This is my flash partitions:
Name,   Type, SubType, Offset,  Size
nvs, data, nvs, , 0x4000
otadata, data, ota, , 0x2000
phy_init, data, phy, , 0x1000
ota_0, 0, ota_0, , 0x130000
ota_1, 0, ota_1, , 0x130000
factory,  0,    0,  , 0x130000
And I've executed "make erash_flash"

Is it not too fast to download whole binary:
I (4145) esp_https_ota: Starting OTA...
I (4145) esp_https_ota: Writing to partition subtype 16 at offset 0x10000
I (7045) esp_https_ota: Connection closed, all data received

Also im wondering how is it possible that  your first OTA partition is beginning at 0x10000:
Writing to partition subtype 16 at offset 0x10000
because we have partition table at 0x8000, then according to your partition table you should have 0x8000 + 0x4000 + 0x2000 + 0x1000 = 0x15000

@chegewara don't forget the length of the partition table itself

0x8000 + 0x1000 + 0x4000 + 0x2000 + 0x1000 = 0x10000 ( hex numbers go to 15)

0x8000 + 0x1000 + 0x4000 + 0x2000 + 0x1000 = 0x10000 ( hex numbers go to 15)

You are correct, looks like issue i have with my app exhausted me so much so im having problem with simple math. Sorry for confusion.

Yes, it turned out to be way too fast a download. I got it working now and it seems like it's downloading about 300bytes at a time. taking about 10-15secs.
So for others with the same error, this could be a clue.
My mistake was silly, I used non standard HTTP/HTTPS ports and forgot to switch port when I went back to HTTP for testing as I couldn't get the certificate working.
Thanks for the help.


feat: [BB-140] Feature**  main page header component

Description
Created a Main Page Header Component that aligns with the designer's ideas.
Type of changes
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] 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
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have already been accepted and merged

Further comments

Thanks for the PR!
Deployments, as required, will be available below:

Frontend
Backend

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are deployed in:

Merge Workflow

Returned it back to draft, will create another pr for this #140 ticket which is branched off main. Once Gui merged in his work. Will delete this once I transfer the code from the new branch. Apologies for the bad practice!


• iPhone 12 (mobile)   • 12D8FF8E-2815-436E-9951-B8A6A42E4ACF • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)      • macos                                • darwin-x64     • macOS 11.1 20C69 darwin-x64
    • Chrome (web)         • chrome                               • web-javascript • Google Chrome 87.0.4280.88

• No issues found!

Thank you

Hi @bhanuka96
Thanks for the reproducible code de sample
Preview
iOS	Web

Code Sample
complete code sample
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // Try running your application with "flutter run". You'll see the
        // application has a blue toolbar. Then, without quitting the app, try
        // changing the primarySwatch below to Colors.green and then invoke
        // "hot reload" (press "r" in the console where you ran "flutter run",
        // or simply save your changes to "hot reload" in a Flutter IDE).
        // Notice that the counter didn't reset back to zero; the application
        // is not restarted.
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      // This call to setState tells the Flutter framework that something has
      // changed in this State, which causes it to rerun the build method below
      // so that the display can reflect the updated values. If we changed
      // _counter without calling setState(), then the build method would not be
      // called again, and so nothing would appear to happen.
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text(widget.title),
      ),
      body: Center(
        // Center is a layout widget. It takes a single child and positions it
        // in the middle of the parent.
        child: Column(
          // Column is also a layout widget. It takes a list of children and
          // arranges them vertically. By default, it sizes itself to fit its
          // children horizontally, and tries to be as tall as its parent.
          //
          // Invoke "debug painting" (press "p" in the console, choose the
          // "Toggle Debug Paint" action from the Flutter Inspector in Android
          // Studio, or the "Toggle Debug Paint" command in Visual Studio Code)
          // to see the wireframe for each widget.
          //
          // Column has various properties to control how it sizes itself and
          // how it positions its children. Here we use mainAxisAlignment to
          // center the children vertically; the main axis here is the vertical
          // axis because Columns are vertical (the cross axis would be
          // horizontal).
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headline4,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

Assets

Logs
logs
flutter doctor -v
Thank you

@TahaTesser I think you added the wrong sample code. please check

Hi @bhanuka96
Thanks for the reproducible code de sample
Preview
iOS	Web

Code Sample
complete code sample
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // Try running your application with "flutter run". You'll see the
        // application has a blue toolbar. Then, without quitting the app, try
        // changing the primarySwatch below to Colors.green and then invoke
        // "hot reload" (press "r" in the console where you ran "flutter run",
        // or simply save your changes to "hot reload" in a Flutter IDE).
        // Notice that the counter didn't reset back to zero; the application
        // is not restarted.
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      // This call to setState tells the Flutter framework that something has
      // changed in this State, which causes it to rerun the build method below
      // so that the display can reflect the updated values. If we changed
      // _counter without calling setState(), then the build method would not be
      // called again, and so nothing would appear to happen.
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text(widget.title),
      ),
      body: Center(
        // Center is a layout widget. It takes a single child and positions it
        // in the middle of the parent.
        child: Column(
          // Column is also a layout widget. It takes a list of children and
          // arranges them vertically. By default, it sizes itself to fit its
          // children horizontally, and tries to be as tall as its parent.
          //
          // Invoke "debug painting" (press "p" in the console, choose the
          // "Toggle Debug Paint" action from the Flutter Inspector in Android
          // Studio, or the "Toggle Debug Paint" command in Visual Studio Code)
          // to see the wireframe for each widget.
          //
          // Column has various properties to control how it sizes itself and
          // how it positions its children. Here we use mainAxisAlignment to
          // center the children vertically; the main axis here is the vertical
          // axis because Columns are vertical (the cross axis would be
          // horizontal).
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headline4,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

Assets

Logs
logs
flutter doctor -v
Thank you

@TahaTesser I think you added the wrong sample code. please check

/cc @ditman

/cc @ditman

This is a quite unconventional way of adding a custom marker from an asset, have you tried:
BitmapDescriptor.fromAssetImage?
Take a look at the example, it might provide some inspiration!

This is a quite unconventional way of adding a custom marker from an asset, have you tried:
BitmapDescriptor.fromAssetImage?
Take a look at the example, it might provide some inspiration!

Note that the error that you're mentioning is happening in your own code, here:

at getBytesFromAsset (http://localhost:49473/packages/name/screens/dashboard/logic/logic.dart.lib.js:30:77)

Note that the error that you're mentioning is happening in your own code, here:

at getBytesFromAsset (http://localhost:49473/packages/name/screens/dashboard/logic/logic.dart.lib.js:30:77)

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

Hi, i am experiencing the same issue when running flutter run web --web-renderer html but if i am using flutter run web it's working fine.
So here you go:
Flutter Doctor -v:
Flutter Doctor

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.3, on Linux, locale en_US.UTF-8)
    • Flutter version 2.0.3 at /home/idelub/Development/flutter
    • Framework revision 4d7946a68d (7 days ago), 2021-03-18 17:24:33 -0700
    • Engine revision 3459eb2436
    • Dart version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at /home/idelub/Android/Sdk
    • Platform android-30, build-tools 30.0.1
    • Java binary at: /opt/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /usr/bin/google-chrome-stable

[✓] Android Studio (version 4.0)
    • Android Studio at /opt/android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] IntelliJ IDEA Ultimate Edition (version 2020.2)
    • IntelliJ at /opt/idea-IU-202.6397.94
    • Flutter plugin version 49.0.4
    • Dart plugin version 202.6397.47

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 88.0.4324.150

• No issues found!

Error log

Error: Unhandled error NoSuchMethodError: invalid member on null: 'buffer' occurred in Instance of 'AnnouncementMarkersBloc'.
packages/hero/application/marker/announcement_markers_bloc.dart 200:66  getBytesFromAsset
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 45:50      <fn>
dart-sdk/lib/async/zone.dart 1612:54                                    runUnary
dart-sdk/lib/async/future_impl.dart 152:18                              handleValue
dart-sdk/lib/async/future_impl.dart 704:44                              handleValueCallback
dart-sdk/lib/async/future_impl.dart 733:13                              _propagateToListeners
dart-sdk/lib/async/future_impl.dart 539:5                               [_completeWithValue]
dart-sdk/lib/async/future_impl.dart 577:7                               callback
dart-sdk/lib/async/schedule_microtask.dart 40:11                        _microtaskLoop
dart-sdk/lib/async/schedule_microtask.dart 49:5                         _startMicrotaskLoop
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 166:15     <fn>

    at Object.throw_ [as throw] (http://localhost:46739/dart_sdk.js:5334:11)
    at http://localhost:46739/packages/bloc/src/cubit.dart.lib.js:122:21
    at announcement_markers_bloc.AnnouncementMarkersBloc.new.onError (http://localhost:46739/packages/bloc/src/cubit.dart.lib.js:123:27)
    at announcement_markers_bloc.AnnouncementMarkersBloc.new.onError (http://localhost:46739/packages/bloc/src/cubit.dart.lib.js:261:15)
    at _RootZone.runBinaryGuarded (http://localhost:46739/dart_sdk.js:38832:11)
    at sendError (http://localhost:46739/dart_sdk.js:32448:26)
    at _ControllerSubscription.new.[_sendError] (http://localhost:46739/dart_sdk.js:32465:11)
    at _ControllerSubscription.new.[_addError] (http://localhost:46739/dart_sdk.js:32388:27)
    at _SyncStreamController.new.[_sendError] (http://localhost:46739/dart_sdk.js:35345:39)
    at _SyncStreamController.new.[_sendError] (http://localhost:46739/dart_sdk.js:35455:33)
    at _SyncStreamController.new.[_addError] (http://localhost:46739/dart_sdk.js:35157:27)
    at _RootZone.runBinaryGuarded (http://localhost:46739/dart_sdk.js:38832:11)
    at sendError (http://localhost:46739/dart_sdk.js:32448:26)
    at _ForwardingStreamSubscription.new.[_sendError] (http://localhost:46739/dart_sdk.js:32465:11)
    at _ForwardingStreamSubscription.new.[_addError] (http://localhost:46739/dart_sdk.js:32388:27)
    at _ForwardingStreamSubscription.new.[_addError] (http://localhost:46739/dart_sdk.js:36613:25)
    at _MapStream.new.[_handleError] (http://localhost:46739/dart_sdk.js:36567:24)
    at _ForwardingStreamSubscription.new.[_handleError] (http://localhost:46739/dart_sdk.js:36639:38)
    at _RootZone.runBinaryGuarded (http://localhost:46739/dart_sdk.js:38832:11)
    at sendError (http://localhost:46739/dart_sdk.js:32448:26)
    at _ControllerSubscription.new.[_sendError] (http://localhost:46739/dart_sdk.js:32465:11)
    at async._DelayedError.new.perform (http://localhost:46739/dart_sdk.js:35875:27)
    at _StreamImplEvents.new.handleNext (http://localhost:46739/dart_sdk.js:35957:15)
    at async._AsyncCallbackEntry.new.callback (http://localhost:46739/dart_sdk.js:35709:16)
    at Object._microtaskLoop (http://localhost:46739/dart_sdk.js:39176:13)
    at _startMicrotaskLoop (http://localhost:46739/dart_sdk.js:39182:13)
    at http://localhost:46739/dart_sdk.js:34689:9

Code Responsible for the Error:
CODE
if (getIt<CategoriesInjectable>().categories.isNotEmpty) {
               final Uint8List markerIcon = await getBytesFromAsset(
                   'assets/marker/${announce.icon}.png', 100);
               final sourceIcon = BitmapDescriptor.fromBytes(markerIcon);
               final pinInfo = AnnouncePinInformation(
                  pinPath: "assets/pin/${announce.icon}.png",
                   locationName: Categories.get(announce.type).title,
                   location:
                       LatLng(announce.point.latitude, announce.point.longitude),
                   announce: announce); // AnnouncePinInformation
               _markers.add(
                 Marker(
                   icon: sourceIcon,
                   markerId: MarkerId(announce.id.getOrCrash()),
                   position:
                      LatLng(announce.point.latitude, announce.point.longitude),
                  onTap: kIsWeb
                      ? null
                      : () => state.context
                       .bloc<MarkerSelectorBloc>()
                          .add(MarkerSelectorEvent.markerTapped(pinInfo)),
                ), // Marker
              );
    }

As mentionned earlier, when using the command line:
flutter run web

There is no error but i am using firebase storage, so to render the pictures from the storage i need the flag
--web-renderer html

p.s: Settings the cors with gsutil doesn't fix the error when trying to render the image stored in cloud storage

@arnaudelub

Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


🐛 [epicfreegames] Needs update, throws error

Which addon?
Epic free games node

Addon name :
Addon version : 03-02-2023

I am not 100% how all updates work, but it seems this is set to a version from 3rd Feb.
I've been running into this issue: https://github.com/claabs/epicgames-freegames-node/issues/288
And the maintainer updated the image in hopes of fixing it.

Thanks, just tested it. still not working. Might be the hassio setup that makes it detect that this is automated.
So maybe mark the addon as potentially non working with error: " Login returned error: Incorrect response. Please refresh the page."

Mine works now with latest update. Had to login with captcha but it has the fresh cookie now and all is well.

So maybe mark the addon as potentially non working with error: " Login returned error: Incorrect response. Please refresh the page."

This is done automatically : once an issue is started that contains the addon name in its title, it appears on the readme and anyone installing it would see it


Put momobox to sleep when not being used

The firmware runs continuously when the batteries are connected. Goal is to have the pico go into sleep mode after some amount of idle time, and then awake upon a signal from the hall sensor.
A good starting point with resources is here: https://ghubcoder.github.io/posts/deep-sleeping-the-pico-micropython/
May be better to also read this post about dormant and sleep modes: https://ghubcoder.github.io/posts/awaking-the-pico/
This is micropython based (momobox currently uses circuit python), so may not be fully applicable.
A good thing to start with is to measure the current draw from the batteries during the following operations:

idle time
RFID request
playing a song
song paused

Some measurements made with a DMM:

Condition
Current Draw [mA]

Standby
65.6

Playing
70-110

Paused
68.6

RFID Chip present without magnet
70.6

Magnet present, no RFID
69.5

RFID, no magnet, after magnet had been activated
74

Crashed
45

The 2X batteries I'm using have a capacity of 1500 mAh (3000 total). 3000 mAh / ~70 mA =~ 42 hours
This operating time matches our experience with the momobox so far, although we typically get to about 24-30 hours before it turns off.
It may be promising to reduce power usage by rewriting the firmware in a lower level language that isn't constantly running an interpreter. For comparison, running "blinky" in Rust only takes 22-24 mA


Problems with 3D Scene

Hi every one. In the most recent version of manim (4/Feb/19) when I try run:
python3 -m manim old_projects/sphere_area.py SphereCylinderScene -pl

I have:
Traceback (most recent call last):
  File "/Users/alexlvazquez/Downloads/manim-master/manimlib/extract_scene.py", line 153, in main
    scene = SceneClass(**scene_kwargs)
  File "/Users/alexlvazquez/Downloads/manim-master/manimlib/scene/three_d_scene.py", line 149, in __init__
    if self.camera.get_pixel_width() == PRODUCTION_QUALITY_CAMERA_CONFIG["pixel_width"]:
AttributeError: 'SphereCylinderScene' object has no attribute 'camera'

And if I add in manimlib/scene/three_d_scene, in the line 149:
 self.camera = self.camera_class(**self.camera_config)

Then I have:
/Users/alexvazquez/Downloads/manim-master 2/media/videos/old_projects/sphere_area/480p15/partial_movie_files/SphereCylinderScene/partial_movie_file_list.txt: Invalid data found when processing input

File ready at /Users/alexvazquez/Downloads/manim-master 2/media/videos/old_projects/sphere_area/480p15/SphereCylinderScene.mp4

But the directory is empy.
Thanks for you help, I don't know what to do.

I understand perfectly what you want to tell me. You don't care that your old videos run in all versions of manim because that is very tedious. The reason of my cuestion is because I'm not  expert to understand why. And do not worry, I can perfectly compile the videos in old versions, I just wanted to learn more.
Thanks for answering.


winston 2 was better

convention > configuration 99.999999^ % of the time. i really hate winston 3, logging non strings in console is now hell. i can't get the format the way i want it. sucks
feelsbadman.jpg

Speaking in terminology that winston@3 defined you are asking for a better "default format". The rewrite made adding such a feature incredibly trivial which illustrates the value in that endeavor.
Please be constructive with your feedback in the future. For example: in this case go look at logform and make a suggestion as to features (i.e. what exact formats) you feel would be most important to have in a default format that we could add to winston@3.

The rewrite made adding such a feature incredibly trivial which illustrates the value in that endeavor.

i think this is a blind spot for you because you are a maintainer of the project, and thus you are more familiar with how to do exactly what you want. i wouldn't call this trivial:
logger.add(new winston.transports.Console({
        format: format.combine(
            format.errors({ stack: true }),
            format.splat(),
            format.colorize(),
            format.timestamp(),
            format.printf(({level,message,timestamp,...restProps})=>{
                var retval = `${timestamp} ${level}: `;
                if(message) {
                    retval += message;
                }else{
                    retval += util.inspect(JSON.parse(restProps[Symbol.for('message')]));
                }
                return retval;
            })
        )
    }));

i think more adapt descriptors would be "complicated" and "wasteful", but its what i had to do to match the import aspects of console.log, which i would consider to be:

string interpolating messages
inspecting values (object, array, etc)

In my opinion, there's no issue with convention-over-configurtion approach at all, in any case library API and documentation are the most important.
I came from Java world and found formatters configuration indeed confusing.. :(
You'd better to separate all formatters into two groups:

functions that modifies fields of log entry object (aka TransformableInfo)
functions used to build final string from that object

And to write createLogger api so that one always can add arbitrary number of modifying functions and set only single final formatter (json/printf etc) globally and/or per transport.
How library users should understand why this config will produce "undefined" on each logger.log call?
const logger = createLogger({
    format: format.timestamp(),
    transports: [
        new transports.Console()
    ]
});

PS I'd also add check to transformers to never print "undefined"..

Having a clear delineation between formatters that mutate the incoming object/message, and formatters that simply output a string is very important and a weakness of the current configuration. In winston@2 we had two conventions for this, formatting and rewriting. While these aren't the clearest names, it certainly made it simpler to understand that sometimes I'm adjusting the message I'm going to log and other times I'm actually constructing the output that I will write to the logging facility.
For a given application, I may want to always add a timestamp and redact some fields, but for a given logging facility, I will need different output written.
For example, in .combine(), it doesn't work to put printf() anywhere but the very end because it doesn't output anything useful for a different formatter to consume. That makes is a different class of 'formatter' and probably should be bound to the logger in a different way.


Configure MIDI velocity on/off and white only mode

Add the option to average two white keys when playing a black key in white mode
Pass and distribute unmanipulated MIDI messages from IN to OUT.
refs #73 #172

Will receive a few more changes that depend on #132

Ready for review.

Reviewed on Discord.


OptionButton popup menu opening at incorrect location in combination with Camera2D and a negative x/y coordinate.

Godot version:
3.3 RC9
OS/device including version:
Ubuntu 20.04
Issue description:
I've seen that when we have an OptionButton with a Camera2D as a child, and the position.x or position.y of the button is negative, the popup menu does not open directly underneath the button, but at an offset.
This also happens if we have a sub-scene with an option menu in it, and then add a Camera2D as the child of the instantiated scene.
Steps to reproduce:

Create a new project.
Add a Node2D as root node
Add an OptionButton as a child to the root
Add a Camera2D as a child of the OptionButton
Move the OptionButton to the left so that it's x coordinate is negative (or move it up so that y is negative)
Run the scene, then click on the option button. The popup menu appears in a wrong location.

Minimal reproduction project:
test-option.zip

Out of curiosity, what do you use a Camera2D as a child of an OptionButton for?

I don't. That's the minimal case I got. The main use case, which I mentioned above was that I had a scene with an OptionButton. I had instantiated into the main scene. Where I had a camera. The subscene was to the left of the main screen and I had a camera to switch to it. Then I noticed that the popup menu shows right at the edge of the screen.

I can confirm that this happens. The popup seems to have minimum position of (0,0), but can go as positive as need be.

This might have to do with how Popup is designed: "All popup methods ensure correct placement within the viewport."

This issue is even more visible in master, as Popups are now Windows. We end up with this:


Update DpsCounterPlugin.java

Added NPC's from hard mode theatre

Please don't close and open new prs. Update your has and push to the same branch.


[Feature Request]: Allow a user to pan, zoom and crop before 'printing' a map

Provide a clear and concise description of what you want to happen.
Users want to be able to have more control when they export a map from PRISM using the 'print' feature.
After clicking on the 'print' button, a user gets to the map preview screen. In this screen, I want to able to:

Pan the map
Zoom in/out
Crop the map

It's important to be able to do this in the map preview screen because the main application has other UI elements in view which obscure the map which is then exported.
Panning and zooming should behave the same as in the application (i.e. click / drag to pan, and use mouse / trackpad to zoom).
Cropping will need a new button which should be added to the right side panel:

After clicking the crop button, users should be able to drag from the edges of the canvas to crop to the area of interest similar to how photo editing software works:
https://github.com/WFP-VAM/prism-app/assets/3343536/4d4b0ceb-82cc-4e8b-bef6-5097ecbd8fb6
Is there anything else you can add about the proposal? You might want to link to related issues here, if you haven't already.
No response

hello @wadhwamatic. About the cropping functionality, would you like to change the exported image's width or height? The behavior in video linked above, i believe can be achieved by zooming and panning

Hi @echaidemenos - if possible, it would be good to give the ability to adjust the canvas size too. That way users can adjust the size according to what works best for their area of interest. "Wide" countries like Indonesia would likely be cropped differently than "narrow" countries like Mozambique.


Specify a set of Dask workers to use for a specific process.

Let's say I have a Dask scheduler with 3 Dask workers (each worker is a process) set up.
I have 3 streaming jobs, each needing 1 Dask worker each. If I use the same Dask cluster to share all the 3 streams, the jobs switch between the Dask workers across batches, which I don't want.
Is there a way I can specify the number of workers/the name of workers for each stream, so that each stream gets its own dedicated set of worker(s)?
Creating 3 different schedulers, and attaching one worker to each works well, but I was thinking if there is a more convenient way to achieve sharing a cluster among jobs.
FYI: I am using streamz to write streaming jobs.

I'm not sure about streamz, but tasks can be bound to workers (or sets of workers) by passing a list of workers to methods like Client.submit or Client.compute (docs here).

In principle we should be able to trust that Dask is running tasks in the
best place for them to run.  It takes into account things like the cost of
moving data, and the backlog on every worker.  If it is making bad choices
here then it's probably a good idea to figure out why, rather than
explicitly pinning workers (which is often bad in the long run).  A common
cause to investigate is whether or not the dask.sizeof.sizeof
function returns the right size for your objects.  If that is wrong (too
low) then Dask may move things around when it shouldn't.
On Wed, Apr 15, 2020 at 6:49 AM Jim Crist-Harif<EMAIL_ADDRESS>wrote:

I'm not sure about streamz, but tasks can be bound to workers (or sets of
workers) by passing a list of workers to methods like Client.submit or
Client.compute (docs here
https://distributed.dask.org/en/latest/locality.html#user-control).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/6095#issuecomment-614051877, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACKZTDYHABOIPUF6UZWJTLRMW3NTANCNFSM4MIDSH2Q
.

Also, it's worth noting that most Dask maintainers don't know anything
about streamz.  If you can reduce your situation to a Dask-only problem my
guess is that you'll get better feedback from folks here.
On Wed, Apr 15, 2020 at 7:18 AM Matthew Rocklin<EMAIL_ADDRESS>wrote:

In principle we should be able to trust that Dask is running tasks in the
best place for them to run.  It takes into account things like the cost of
moving data, and the backlog on every worker.  If it is making bad choices
here then it's probably a good idea to figure out why, rather than
explicitly pinning workers (which is often bad in the long run).  A common
cause to investigate is whether or not the dask.sizeof.sizeof
function returns the right size for your objects.  If that is wrong (too
low) then Dask may move things around when it shouldn't.
On Wed, Apr 15, 2020 at 6:49 AM Jim Crist-Harif<EMAIL_ADDRESS>wrote:

I'm not sure about streamz, but tasks can be bound to workers (or sets
of workers) by passing a list of workers to methods like Client.submit
or Client.compute (docs here
https://distributed.dask.org/en/latest/locality.html#user-control).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/6095#issuecomment-614051877, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACKZTDYHABOIPUF6UZWJTLRMW3NTANCNFSM4MIDSH2Q
.

Thanks @jcrist, @mrocklin! I'll try both options — pinning workers using the docs provided above, and allowing Dask to handle the jobs by itself and see how it performs.

I also have a use case where one of the jobs needs a higher worker memory limit like 8GB, while the other two jobs need around 4GB. Maybe that's why I thought it was best to pin workers.

I think the original questions have been answered, so closing this.


Add Version Option

Summary

If merged this pull request will
Support surfactant --version and surfactant -v to print version information.
Fix TypeError: Context.__init__() got an unexpected keyword argument 'log_level'
when running either python -m surfactant --version or python -m surfactant version
to address issue #137

I found this explanation helpful when looking into the unexpected argument issue. I think it had something to do with click not liking arguments getting passed in
if  __name__ == "__main__":
    main(log_level="INFO")

Looks good, thanks!


Git-flow core

To implement a singleton that will make it possible to execute git-flow commands for feature managing.

Everything is done except feature pulling and publishing.

Will move pulling and publishing to one of the next releases.


Cache Informer sets ResourceVersion=0 which causes API list chunking to break

/kind bug
What happened:
Something in the way the cache ListWatch/Informer machinery sets ResourceVersion="0" breaks API chunking when listing pods.
It seems that the API server ignores the limit parameter when ResourceVersion = "0". It's not clear what the actual issue here is.
In contrast, kubectl does not appear to set ResourceVersion for the first GET request.
Is there a semantic difference between ResourceVersion "0" and ""?
How to reproduce it (as minimally and precisely as possible):
Create a cluster with more than one pods, run the following code accessing the cluster via kubectl proxy -v=7 and notice how chunking is not happening:
// client := ...
lw := &cache.ListWatch{
  ListFunc: func(options metav1.ListOptions) (runtime.Object, error) {
    options.Limit = 1
    // Uncommenting the following "fixes" the chunking:
    // if options.ResourceVersion == "0" {
    //  options.ResourceVersion = ""
    // }
    return client.Core().Pods(metav1.NamespaceAll).List(options)
  },
  WatchFunc: func(options metav1.ListOptions) (watch.Interface, error) {
    return client.Core().Pods(metav1.NamespaceAll).Watch(options)
  },
}
_ informer := cache.NewInformer(lw, &apiv1.Pod{}, 0, cache.ResourceEventHandlerFuncs{})
informer.Run(context.Background())

Environment:
client-go Version: 78700dec6369ba22221b72770783300f143df150
Server Version: version.Info{Major:"1", Minor:"9+", GitVersion:"v1.9.2-gke.1", GitCommit:"4ce7af72d8d343ea2f7680348852db641ff573af", GitTreeState:"clean", BuildDate:"2018-01-31T22:30:55Z", GoVersion:"go1.9.2b4", Compiler:"gc", Platform:"linux/amd64"}

This is working as intended as far as the chunking behavior is concerned. The server will not return a continue token that conflicts with a resourceVersion you specify.

Is there a semantic difference between ResourceVersion "0" and ""?

Yes, "" is unspecified and leaves the server free to return a continue token pinned to the current resourceVersion. "0" means "serve the current version held by the API server cache", which could not be counted on to be consistent when making the next chunk request.
@smarterclayton did you know reflectors were setting resourceVersion=0 and therefore unable to use chunking?

@liggitt Thanks for the clarification. In the interim while reflectors aren't doing chunking, is doing if options.ResourceVersion == "0" { options.ResourceVersion = "" } as a workaround (see the commented lines above) a safe operation in the sense that the store will get synced to a good state?

Yes I did know.  When watch cache is on hitting the watch cache is faster than chunking.  When watch cache is off etcd serves chunks (0 means nothing to etcd layer).  Watch cache initial establishment hits etcd with chunking on (or at least it should), which prevents super large reads from etcd from dying.
At some point watch cache chunking probably makes sense but it’s more expensive to implement.

Could you comment on https://github.com/kubernetes/client-go/issues/366#issuecomment-364673743? I can deal with listing being less efficient

If you make your lists avoid sending resourceVersion=0, that shouldn't cause any correctness issues

See also https://github.com/kubernetes/kubernetes/issues/59684


PostgreSQL extension PostGIS is missing

Sail Version: 1.14.1
Laravel Version: 9.1.7
PHP Version: 8.1.4

Description:
It is impossible to use $table->point('position') column type in migrations because, apparently, the "postgres" Docker image that Laravel Sail is using (has referenced in the docker-compose.yml file for the developer) does not include postgis extension.
Steps To Reproduce:
Create migration:
Schema::create('venues', function (Blueprint $table) {
    $table->id();
    $table->point('position');
    $table->timestamps();
});

Run migrations:
./vendor/bin/sail artisan migrate

It generates an error:
SQLSTATE[42704]: Undefined object: 7 ERROR:  type "geography" does not exist at character 72 (SQL: create table "venues" ("id" bigserial primary key not null, "position" geography(point, 4326) not null, "created_at" timestamp(0) without time zone null, "updated_at" timestamp(0) without time zone null))
Also running this does not help:
DB::statement('CREATE EXTENSION IF NOT EXISTS postgis;');

It generates an error:
SQLSTATE[58P01]: Undefined file: 7 ERROR:  could not open extension control file "/usr/share/postgresql/14/extension/postgis.control": No such file or directory (SQL: CREATE EXTENSION IF NOT EXISTS postgis;)
Possible solutions:
While I was writing this issue, I realized that this is not a problem in Laravel Sail's Docker image. The problem is that the docker-compose.yml file that Sail was initially creating references a Docker image postgres:14 which, apparently, does not contain any extensions. I think that in documentation it should be mentioned that there are no extensions included in Docker image postgres and that you should, for example, use Docker image postgis/postgis instead or that you should build a custom Docker image... :shrug:
Would be great if you could use all Laravel's features using Laravel Sail right out of the box... :wink:

You are free to change the docker compose file to anything you need 👍

@taai Use postgis/postgis:14-3.3 image instead of the default that Sail provides.
Reference: https://github.com/postgis/docker-postgis#versions


WIP: fix windows compilation (see #3)

These are just the easy pickings.
Missing/incorrect use statements, and some missing features in Cargo.toml
This still doesn't compile on windows: https://ci.appveyor.com/project/willstott101/if-addrs/builds/35244539

Thanks, I didn't test Windows because I don't own a Windows machine, help appreciated!

I've added Windows CI to this repo.

Looks like it works now, can you give it a try? I'll release a new version later.


Further alternative for "Injecting Prototype Beans..." with java.util.Function

Dear baeldung team
I enjoyed your overview article
http://www.baeldung.com/spring-inject-prototype-bean-into-singleton
I would like to propose to add the very nice solution that Roman Golyshev suggested in this stack overflow answer:
Spring Java Config: how do you create a prototype-scoped @Bean with runtime arguments
beyond what is discussed there about type safety

it allows the mix of singleton-service-references and runtime-arguments.
Using a java.util.BiFunction allows passing in two runtime-arguments

AppConfig
	@Scope(ConfigurableBeanFactory.SCOPE_PROTOTYPE)
	@Bean
	public PrototypeBean prototypeBean(LocalDateTime runtimeArg) {
		return new PrototypeBean(
				serviceRef1(),
				serviceRef2(),
				runtimeArg);
	}

	@Bean
	public Function<LocalDateTime, PrototypeBean> prototypeBeanFactory() {
		return runtimeArg -> prototypeBean(runtimeArg);
	}

Usage

	@Autowired
	private Function<LocalDateTime, PrototypeBean> prototypeBeanFactory;

	// ...
	PrototypeBean bean = prototypeBeanFactory.apply(LocalDateTime.now());

One could add for safety reasons an @Qualifier just in case there are more functions like this in the ApplicationContext
	@Autowired
	@Qualifier("prototypeBeanFactory")
	private Function<LocalDateTime, PrototypeBean> prototypeBeanFactory;

Hey @bartolom Thanks for the suggestion. I've added a new section with this solution in the article.
Cheers.

Looks good. Thank you for the quick turn around.


explode implementation (for v2)

This is the PR that relates to the explosion proposal.
It is currently missing:

compiler tests
new syntax documentation
benchmarks

I can/will create these once the rest of the approach has been validated and performance concerns are addressed.
I opened this so we have a more formal/public way to discuss/review the implementation.
I expect this to be a living, breathing PR for a while while we work through:

What features from the proposal we should keep / discard
How we want to implement said features

closing, as this branch has been merged with the vm_hooks branch


Fixes to integer conversions and their tests

Fix prop integer conversions to smaller types

What does this implement/fix? Explain your changes.
Two issues were found in rdvalue_cast functions:

casting to uint16_t was actually casting to uint8_t
casting to uint8_t was casting from the unsigned value of the union, if the value was typed as signed

Any other comments?
There was two issues in the tests as well, which is why the above issues were not caught. In addition, conversions of all min/max numeric limits of the smaller types i.e. (u)int8/16 are now explicitly tested.

@bp-kelley : please merge if these make sense to you too


Vertical scrollbar appears when notification menu is opened

I would have submitted a pull request, but I don't know SCSS at all.
.NotificationList--open needs something similar to height: calc(100% - 83px); where 83px is the value calculated for top.

Thanks! Could you submit a screenshot illustrating the issue?

@eatonphil Sure:


Update PasswordBroker.php

Add setter and getter for $emailView

Duplicate. Already rejected,

This property is not meant to be dynamically set.

Then please describe how can we change PasswordBroker's $emailView for different parts of site - eg for backend and frontend. As i understand it's not possible becouse it use \Config('auth.password.email') value. And it's impossible change this value for specific part of site.
So please explain how use different views for password reset.

Sorry, please, how can i use diffrent emailView for diffrent part of site (backend reset password and frontend reset password) ?
Thanks

Here https://github.com/laravel/laravel/blob/master/config/auth.php#L62

thanks but i see one line.. how set more? thanks


feat: Merge anomaly detection in StreamDQ to main branch(Closes #3)(PR #8)

Issue #3

Added anomaly detection feature for StreamDQ, implemented 5 strategies: SimpleThreshold, OnlineNormal, IntervalNormal, AbsoluteChange, RelativeChange.
corresponding unit tests are also added

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

Hi Tong,
Thank you! I will take a closer look in the next couple of days. One quick comment, can you include a PR message according to this PR template? https://github.com/stefan-grafberger/mlinspect-private-branches/blob/master/.github/pull_request_template.md
It would be good to have a description of your changes and, if you are fine with that, the rights notice that it is okay for others to use your code.

Also, there seem to be some conflicts, could you rebase/merge the changes from main and resolve the conflicts?

Hi Stefan, sure I will resolve this

Hi Tong,
Thank you! I will take a closer look in the next couple of days. One quick comment, can you include a PR message according to this PR template? https://github.com/stefan-grafberger/mlinspect-private-branches/blob/master/.github/pull_request_template.md
It would be good to have a description of your changes and, if you are fine with that, the rights notice that it is okay for others to use your code.

Ok, I will do that, but this template link returns me a 404

Ah, sorry, here is the correct link: https://github.com/stefan-grafberger/mlinspect/blob/master/.github/pull_request_template.md

Also, when merging this and the other PR in the end, could you try to match the commit message to the naming scheme of the other commit messages in the main branch?

Thanks so much for all your work; almost ready to merge now!
One more request, if that's alright: @sscdotopen suggested we should add copyright statements for all files as a header. This is an example for something like this: GitHub example. Would you mind adding something like this, but with UvA instead of the Apache Software Foundation as the license holder, for all files that you touched (or directly for all code files)?
I guess our header could look like this:
/**
 * Licensed to the University of Amsterdam (UvA) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The UvA licenses this file to You under the Apache License, Version 2.0
 * (the "License"); you may not use this file except in compliance with
 * the License.  You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

/**

Licensed to the University of Amsterdam (UvA) under one or more
contributor license agreements.  See the NOTICE file distributed with
this work for additional information regarding copyright ownership.
The UvA licenses this file to You under the Apache License, Version 2.0
(the "License"); you may not use this file except in compliance with
the License.  You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

Hi Stefan, I'm totally okay with this, copyright is pretty important

I think it is ready to merge now! Thanks so much for all your work! I think you should be able to merge it yourself once I approved it (Squash & Merge). If not, please let me know.
Please try to keep the commit message of the one squash commit in line with the others :)
Once it is merged, could you resolve merge conflicts of your other PR, if there are any?

Sure no problem Stefan!


Update middleware to accept named url patterns for LOGIN_URL

Issue is that LOGIN_URL is supposed to handle named url patterns - https://docs.djangoproject.com/en/1.9/ref/settings/#login-url
Quite by accident, I discovered it doesn't by trying to set the LOGIN_URL to "casauth:sigin" in a Django app that I layer on top of django-cas-ng.
I will supply a pull request.

closed due to in-active.


Respect nixpkgs commit in devbox add by using nix search

Summary
Devbox was not respecting the nixpkgs.commit from devbox.json for some operations
like devbox info and devbox add.
This PR fixes this by changing the PkgInfo function to rely on the nix search
command, which lets us query for the exact package when given a nixpkgs commit.
For now, we strictly fail if the exact match was not found.  In the future, we could
improve the error message to suggest some of the specific alternatives that nix search
can provide.
Alternatives include:

nix-env --json --file https://github.com/nixos/nixpkgs/archive/af9e00071d0971eb292fd5abef334e66eda3cb69.tar.gz  -qa pulumi, but this takes 13 seconds.
Using nix-instantiate, but this doesn't let us later suggest specific alternatives like nix search does.

nix-instantiate --eval --strict --json --expr 'let
  pkgs = import
    (fetchTarball {
      url = "https://github.com/nixos/nixpkgs/archive/af9e00071d0971eb292fd5abef334e66eda3cb69.tar.gz";
    })
    { };
in { inherit (pkgs.pulumi-bin) name pname version system; }'

How was it tested?
❯ devbox info pulumi
Package pulumi not found

and
❯ devbox info pulumi-bin
pulumi-3.37.2-x86_64-darwin

Also, verified that devbox add now works, including inside a devbox shell.
testdata/rust/rust-stable on  savil/nix-search is 📦 v0.1.0 via 🦀 v1.61.0 on ☁  (us-east-2) on ☁<EMAIL_ADDRESS>❯ devbox shell
Installing nix packages. This may take a while... done.
Starting a devbox shell...
project dir is /Users/savil/code/jetpack/devbox/testdata/rust/rust-stable
info: using existing install for 'stable-x86_64-apple-darwin'
info: default toolchain set to 'stable-x86_64-apple-darwin'

  stable-x86_64-apple-darwin unchanged - rustc 1.64.0 (a55dd71d5 2022-09-19)

(devbox)
testdata/rust/rust-stable on  savil/nix-search is 📦 v0.1.0 via 🦀 v1.64.0 on ☁  (us-east-2) on ☁<EMAIL_ADDRESS>❯ devbox add pulumi-bin
Installing nix packages. This may take a while... done.
pulumi-bin is now installed. Run `hash -r` to ensure your shell is updated.
(devbox)
testdata/rust/rust-stable on  savil/nix-search [!] is 📦 v0.1.0 via 🦀 v1.64.0 on ☁  (us-east-2) on ☁<EMAIL_ADDRESS>❯ which pulumi
/usr/local/bin/pulumi
(devbox)
testdata/rust/rust-stable on  savil/nix-search [!] is 📦 v0.1.0 via 🦀 v1.64.0 on ☁  (us-east-2) on ☁<EMAIL_ADDRESS>❯ hash -r
(devbox)
testdata/rust/rust-stable on  savil/nix-search [!] is 📦 v0.1.0 via 🦀 v1.64.0 on ☁  (us-east-2) on ☁<EMAIL_ADDRESS>❯ which pulumi
/Users/savil/code/jetpack/devbox/testdata/rust/rust-stable/.devbox/nix/profile/default/bin/pulumi

Since pulumi doesn't exist in the stable nixpkgs channel, this would previously fail

Current dependencies on/for this PR:

main

PR #314   👈

This comment was auto-generated by Graphite.


Update: Dashboard - Adjust to show transactions per month in the current year.

Per month is better than per week only.

Merged in PR #3.


Make mapping of host LVM directories optional

Alternatively, we could just have one config option that disables the host mapping of linstor and lvm.

Thx


Pool CancellationTokenSource for ActiveHealthCheckMonitor

Is it possible to use ActivityCancellationTokenSource.Rent for ActiveHealthCheckMonitor?
https://github.com/microsoft/reverse-proxy/blob/1855217be702f3a6d721095b36b07f5458ee24a5/src/ReverseProxy/Health/ActiveHealthCheckMonitor.cs#L170

Yes, it should be possible to plug in ActivityCancellationTokenSource.Rent with CT.None as the linked token, but the benefits would be in the noise range.
Another place that may be more impactful is in StreamCopyHttpContent for HTTP/2 and 3, but that would require a different pooled CTS.


Wrap NFT request

Native NFT chainid (Andromeda 1088, Stardust 588, Ethereum 1, Rinkeby 4): 1
Native NFT contract address: 0x503a3039e9ce236e9a12e4008aecbb1fd8b384a3
Deployed wrap NFT contract address: 0x60c61a20e2c49ad60ccab7ba1894209a41e4a55a
Transaction hash of the contract deployment: 0x712088a8084e08658868859487fd674834b8bcc67fa33e56a9f77b6d4075cd38

done

excellent thank you very much!

hey eric
im getting an rpc error when i'm trying to confirm transfer of the nft via the nft bridge.
Uncaught (in promise) TypeError: r.fromNet.rpcUrls is undefined

@uint160xffff Hi, can you try to connect your wallet again? it seems to be a wallet connection error


scss compile, expanded for dev

ouptuts explanded SCSS in dev env
dev env now includes development and local
scss error message supressed in production (only basic message shown)

Hi @justinkruit I pushed the requested changes


Update README and config

Following the feedback of @aslom in #473:

The README was updated to run make ocp-deploy with IMAGE_ORG=netobserv
The configuration of FLP was updated. #418 introduced a new API field in extract_aggregate which required this update.
On the way, updated the operational metric prefix to flp_op_ (added underscore) to avoid metric names like flp_opmetrics_processed

/approve


return back watch.Deleted event to clients when watch object is removed in OjbectFilters

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage

/kind feature
What this PR does / why we need it:
ResponseFilter return watch.Deleted event back to clients when Standalone object is filtered to nil in ObjectFilters.
by the way, this feature will be helpful for all ObjectFilters, not only for nodeportisolation filter.
Which issue(s) this PR fixes:

Fixes #1994
Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

lgtm, but I still have questions for some of the modifications. @rambohe-ch PTAL

@Congrool I have fixed you comments, PTAL

/rerun

/approve
/lgtm


(Fix) Duplicate criteria instead of constructing from scratch

Amazingly, this seems slightly less hacky then before.
We encountered a bug whereby embedded items were not returned in pagination. Turns out this is because when we were constructing the mongoid criteria from scratch (to correctly chain together queries), all parent associations on the criteria were not maintained, including the parent collection for an embedded object.
This PR first duplicates the original criteria, and then only changes the selector (that's all we want to modify anyways). It's cleaner because by calling criteria.dup we don't have to individually build each necessary attribute of the criteria.

The embedded items not returned in pagination is another bug. You should call it out in CHANGELOG, please (via --amend).

@dblock CHANGELOG amended!

Merged.


File tab hover tooltip get's stuck and won't disappear

Description
Occasionally (seemingly randomly), the tooltip that shows the file path on the file tab when you hover on it gets stuck and won't go away until I restart Atom. Sometimes when this happens and I have multiple tabs open, all of the tab's tooltips will remain.
Steps to Reproduce
Not sure what is causing this - seems to be random, and with general use.
Reproduces how often: Once or twice a day.
Versions
Atom: 1.20.1
Electron: 1.6.9
Chrome: 56.0.2924.87
Node: 7.4.0
Additional Information

Thanks for the report!  I've vaguely noticed this as well in the settings view lately but could never reproduce it on purpose.  I'm guessing it doesn't happen enough for you to try and reproduce in safe mode since it happens sporadically.
We'll go ahead and leave this open for a bit and see if anyone else runs into it and if anyone comes up with steps to reproduce.
Can you share what operating system you're running?

oh I just came across https://github.com/atom/tabs/issues/362 which reminds me that this should probably be moved to https://github.com/atom/tabs (that issue was closed as fix so we should open a new issue).
I can re-open this in https://github.com/atom/tabs/issues/362 tomorrow unless you get to it first @mrseanbaines.


Does not work with eslint installed globally

Visual studio code  1.16.1
OS: Arch Linux

@igorvisi this works for me. Are you using yarn to install eslint globally. If so, yarn is currently not supported.
If you use npm can you provide a setup that shows the problem. And does npm config get prefix return the right installation location?

I'm using nvm . It can be the source of problems ?
the resultat of  npm config get prefix is :
/home/visi/.nvm/versions/node/v8.1.2

And ESLint is installed under /home/visi/.nvm/versions/node/v8.1.2

Yes It is .

Hard to say why this is failing then. It works for me using nvm as well.
Can you enable tracing to see where the extension actually tries to find the node modules. "eslint.trace.server": "verbose" and append the trace here.

I only installed globally which normally works


Allow collate_tpms.R to take a list of filenames only

At present, collate_tpms.R takes a paired list of sample names and filenames. Allow users to alternatively provide a list of file names only (This is more like featureCounts from subread).
Column names in the TPMs TSV file output would be these filenames.
Extend rscripts/tests/testthat/test_collate_tpms.R to test this new feature.
Proposed by @ewallace from #309.

Implemented, see PR #407.

Merged update-file-names-411 (ac1ad88 27/08/21) into collate-tpms-filenames-only-35.  collate-tpms-filenames-only-35 pull request #407 to be merged into develop post-merge of update-file-names-411 pull request #422.


Mention other json schema validation implementations

In the project title, you mention this tool as Another implementation of JSON schema for python.
Could you please mention, how it compares to others, and what are your motivations or scope for this one ? i.e. help a programmer choose the best one depending on her particular needs :)
Thanks !
(Great job heh ;))

Thanks :).
I actually don't have much experience with any of the others. There are a few other libraries which claim to be JSON Schema implementations (see http://json-schema.org/implementations.html) -- I haven't used them though so I can just tell you about this one :) -- It use the comprehensive official test suite (https://github.com/json-schema/JSON-Schema-Test-Suite) to test compliance (n.b. I wrote it). The reason I wrote it was that at the time when I started, I tried the other libraries, and they were either lacking in test coverage, or implemented in a way I didn't like. So I wrote this, which has a few things I still don't like, but generally, I think it's fairly solid, and has thankfully had some pretty wide adoption from what I've seen.
Feel free to follow up with any questions, or if you end up using it, lemme know how it goes!


Concatenation Operation not working properly on update of child record

Hi,
I have the RealTime DLRS which concatenates text field of the child on parent record. but if I change a child record, the text rollup doesn't flow to the parent object unless I manually run the Calculate Job again.
Is this the known issue of concatenation operation?
If yes can anyone please suggest a solution for this.

Hard to tell without more details. It would be helpful if you provide a screenshot of the current configuration.

What kind of field is the text field you're concatenating? Does it or another field need to be added to the Relationship Criteria Fields list?
Is it by chance a formula field?

@Shraddha-Ingale let us know if you want to keep discussing. We can for sure re-open this issue. 👍


Resource Leak (ForkJoinPool)

Description
LuckPerms is using a ForkJoinPool in their BukkitSchedulerAdapter (and probably also in other adapters + maybe CaffeineFactory) and doesn't shut it down. This behavior can be observed trough an attached debugger (e.g. eclipse/intellij-debugger) and this has only been observe if the server is running LuckPerms.

Screenshot of Debugger

Reproduction Steps

reload plugin/server

Expected Behaviour

ForkJoinPools should be shutdown at the end of the plugins life cycle

Server Details
3368-Spigot-8965a50-2a2caa7 (MC: 1.18.1)
LuckPerms Version
v5.3.86
Logs and Configs

default config

Extra Details
No response

Does this only occur with a server-wide /reload? This command unfortunately will cause sporadic problems with a lot of things, but if LuckPerms' own reload command causes this as well, it would be good to know. Also going to tag @lucko on this one - hello Luck!

This only occurs when reloading the plugin with /bukkit:reload, /lp reloadconfig works fine as far as I can tell.

It would be wise if you avoid using the bukkits reload command as it can and will break plugins.  This will always cause issues like this in many plugins not just Luckperms. Since this error is only occuring when using the bukkit reload command, I doubt there will be any reason Luck will look into preventing it from happening purely because the bukkit reload command is what is causing the error to happen and not something on luckperms side.
From now on, if you wish to "reload" the config file for luckperms or any other plugin, either use that specific plugins own reload command or properly stop the server then start it again. This allows for plugins to properly do their shutdown processes and allows for the data to be properly saved and prevent corruption.

I get what your saying, this was never an issue for me it's just something I noticed while testing my own plugin (Orebfuscator). I thought it would be helpful to mention this since this could be solve with a simple graceful shutdown of the pool but not knowning if that is something LuckPerms needs I though it best to just create an issue and leave it to debeat.