Adding more info to the readme file and migrating to CC

Hi @ericazaja
Great work on updating your tutorial, exciting to see the progress! To continue ahead with publishing your tutorial on the CC website, I have made a repository on the CC account for the data and files associated with your tutorial. https://github.com/ourcodingclub/CC-anova Could you please copy over your content from this repository to the one on the CC organisation and then also update the link in your tutorial markdown file (so that the data are downloaded from the CC account)? You had a lovely comprehensive readme file in the repo for your tutorial and think it would be useful to copy some of the content about the image licenses and data into the readme for this repository.
Looks great! More from me soon but excellent progress! I gave you write access to the repo I made for you so hopefully you are able to upload files, but let me know if not!
Have a good day,
Gergana

Hello @gndaskalova ! Hope your travels are going smoothly. Thanks for taking the time to do all that!
I can confirm I have full access to the Coding Club organisation repo and I've now migrated the data there (https://github.com/ourcodingclub/CC-anova). I have changed the link in my tutorial markdown as well. I also added a more informative README.md as you suggested.
I look forward from hearing from you!
Best
Erica

Looks great @ericazaja ! I changed your license for the tutorial just cause we use a creative commons license not the MIT one. The only other thing I spotted is here "Data created by me" - could you add what you mean here? Did you collect the data, or did you generate random numbers?
Thank you,
Gergana

Looks great @ericazaja ! I changed your license for the tutorial just cause we use a creative commons license not the MIT one. The only other thing I spotted is here "Data created by me" - could you add what you mean here? Did you collect the data, or did you generate random numbers?
Thank you,
Gergana

@gndaskalova "Data created by me", as in a wrote those numbers specifically to make sure I'd get good graphs and so that the ANOVA would run smoothly. I know, I basically p-hacked but I did it for teaching purposes! How can I phrase this in the README.md? Something like dummy data generated by Erica Zaja. Could this work?
Erica

That sounds good @ericazaja !


BUILD Failed: missing input file '@mkl//:LICENSE'.

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Centos 7
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): latest
Python version: 2.7
Bazel version (if compiling from source):0.4.5
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce: bazel build --config=mkl  -c opt //tensorflow/tools/pip_package:build_pip_package

I was trying to build Tensorflow. and hitting this error missing input file '@mkl//:LICENSE'.
i did it a week back, no problem faced. everything worked fine with same command line

Meet the same issue, too.

getting following error message during build in Ubuntu,
"missing input file '@androidsdk//:build-tools/25.0.2/aapt'.
Target //tensorflow/examples/android:tensorflow_demo failed to build"

Sorry for the inconvenience, this is probably due to my change to MKL build.
I have an internal change which should fix MKL build once pushed externally.
Monday or Tuesday we should have a push, which should fix the problem.

@Ishi-Apps your issue is quite different, I recommend filing a new issue by filling out the full template.

@gunan we are seeing the MKL issue.

https://github.com/tensorflow/tensorflow/commit/5442825dfd774a1d1f55d4d0e9d2821abb551caf is now pushed out.
@agramesh1 could you sync past that commit and try again to see if the issue is resolved?

@gunan The issues is resolved in the commit. Thanks.


Added Mvg Widget

Display mvg departure times for MVG station.
Category: Feature
Overview
Display The departure times of the next public transportation for Munich.
New Vars (if applicable)

Field
Type
Required
Description

location
string
Required
The name of the location (exact) or the location id, startin with de:09162:

limit
integer
Optional
Limit number of entries, defaults to 10.

defaults.mvg
Screenshot

Code Quality Checklist (Please complete)

[x] All changes are backwards compatible
[x] All lint checks and tests are passing
[x] There are no (new) build warnings or errors
[x] (If a new config option is added) Attribute is outlined in the schema and documented
[ ] Bumps version, if new feature added ?

Yes, you can go ahead.


change around toolbar

get rid of gpa button
add more... button for rotation and add the rest of the rotations.
try and keep highlights

use jmenu right on there.

use jpopupmenu


Add dracula style

Add dracula style, and tested locally.

Quick question, from where does the dracula css rules come from?

Quick question, from where does the dracula css rules come from?

There aren't any strict rules as far as I can tell. I looked at a few implementations of the dracula theme and it looks different on VIM and VSCODE for example.
I am doing my best to apply the CSS rules based n what I see from the other dracula implementations.
Like these:

https://github.com/joaopsramos/dracula-elixir/blob/main/imgs/elixir.png
https://github.com/dracula/obsidian
https://github.com/dracula/visual-studio-code

Actually it looks like there are guidelines over here: https://spec.draculatheme.com/#

What I mean is that I think new styles have to be contributed to the makeup project, and then they will automatically show up here. :)

What I mean is that I think new styles have to be contributed to the makeup project, and then they will automatically show up here. :)

Oh got you, I am working on another PR for that. Will push both just now.

Sorry about the delay. I only saw your message now:

My bad, I forgot to push to master. I have done that now, can you please rebase? Also, please commit your changes to master. I will rebuild the website after merging. :)

Should I push my change to master? Or should I rebase and push the update to this PR?
I think the changes have been merged in now, and the website can be rebuilt.

I will regenerate this once a new makeup version is out. :)


_target=blank links not open

Bug description:

When href contains _target="blank" it's not open in webview
Environment:
System:
    OS: macOS 10.14.6
    CPU: (6) x64 Intel(R) Core(TM) i5-8500B CPU @ 3.00GHz
    Memory: 8.99 GB / 32.00 GB
    Shell: 5.3 - /bin/zsh
  Binaries:
    Node: 10.16.1 - ~/.nvm/versions/node/v10.16.1/bin/node
    Yarn: 1.17.3 - /usr/local/bin/yarn
    npm: 6.9.0 - ~/.nvm/versions/node/v10.16.1/bin/npm
  SDKs:
    iOS SDK:
      Platforms: iOS 13.0, DriverKit 19.0, macOS 10.15, tvOS 13.0, watchOS 6.0
    Android SDK:
      API Levels: 23, 25, 27, 28, 29
      Build Tools: 25.0.2, 25.0.3, 28.0.3, 29.0.1, 29.0.2
      System Images: android-28 | Google APIs Intel x86 Atom, android-29 | Google APIs Intel x86 Atom
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>91312
    Xcode: 11.0/11A419c - /usr/bin/xcodebuild

└─<EMAIL_ADDRESS>

I'm sorry. Seems like onShouldStartLoadWithRequest broke behaviour. Remove it and all is fine. Thank you


change mobile cursor for touch devices

fixes #1778
This PR changes the pointer/cursor that is displayed for devices that support touch.

One thing to consider is that some devices (like iPad) support both touch and mouse interaction simultaneously.

One thing to consider is that some devices (like iPad) support both touch and mouse interaction simultaneously.

Absolutely. The workflow I proposed isn't ideal by any measure and I'm not invested in it. It just felt it has a good cost/benefit ratio, considering this feature isn't critical, and is mostly a vanity feature (in that getting it wrong won't really do much harm).

@kolharsam as per the DMs we had, I couldn't make it work. Lemme know what's the status, or if you need any help. Or maybe I was doing something wrong.

@kolharsam could you please sync with master so we can review this one and decide what to do?


Can the pdfjsLib use random access of File to parse and render PDF document?

The EPUB parser and renderer has a reasonably low memory footprint by random accessing document file which is neat. I wander if this can be adapted to pdf-js which for now reads the whole file into a ArrayBuffer even just for parsing the document metadata.

I've wondered this myself. From https://github.com/mozilla/pdf.js/blob/e1f9fa4ea54246bbfa974ac75720793b9ff9fd12/src/display/api.js#L117, it says it only accepts TypedArray | ArrayBuffer | Array<number> | string. But it does support HTTP range requests if you give it a URL. Perhaps one should ask the PDJ.js devs.

Nice hint. I will try the HTTP range way.

Awesome. It works like a charm.

Let pdf.js handle the raw url is a pain. But pdfjsLib also has a PDFDataRangeTransport which can be adapted with file slicing to random access the document content. And according to this comment, the default viewer of pdf.js uses this approach.
@@ -111,9 +111,15 @@ const makeTOCItem = item => ({
 })

 export const makePDF = async file => {
-    const data = new Uint8Array(await file.arrayBuffer())
+    const initialData = await file.slice(0, 4096).arrayBuffer()
+    const transport = new pdfjsLib.PDFDataRangeTransport(file.size, initialData)
+    transport.requestDataRange = (begin, end) => {
+        file.slice(begin, end).arrayBuffer().then(chunk => {
+            transport.onDataRange(begin, chunk)
+        })
+    }
     const pdf = await pdfjsLib.getDocument({
-        data,
+        range: transport,
         cMapUrl: pdfjsPath('cmaps/'),
         standardFontDataUrl: pdfjsPath('standard_fonts/'),
         isEvalSupported: false,

In this way well structured PDF document can be loaded progressively aka. lazy loading only the data for pages that is about to be rendered. If this approach is sound and reasonable I can send a PR for this.


Write iteratively to batch_objects.jsonl for recovery and catch openai.APITimeoutError

I'm seeing a bunch of the timeouts ^  as we try to upload all the batches:
[36m(_Completions pid=125780, ip=<IP_ADDRESS>)[0m INFO:openai._base_client:Retrying request to /files in 0.839617 seconds
[36m(_Completions pid=125780, ip=<IP_ADDRESS>)[0m INFO:openai._base_client:Retrying request to /files in 0.884488 seconds
[36m(_Completions pid=125780, ip=<IP_ADDRESS>)[0m INFO:openai._base_client:Retrying request to /files in 0.821620 seconds
This is what killed the job
openai.APITimeoutError: Request timed out.
raise APITimeoutError(request=request) from err
File "/tmp/ray/session_2024-11-14_19-11-03_110436_1/runtime_resources/pip/8f9a6c08a6f7b36cef5b248cf848c00d3b8e4aef/virtualenv/lib/python3.10/site-packages/openai/_base_client.py", line 1591, in _request
return await self._retry_request(
File "/tmp/ray/session_2024-11-14_19-11-03_110436_1/runtime_resources/pip/8f9a6c08a6f7b36cef5b248cf848c00d3b8e4aef/virtualenv/lib/python3.10/site-packages/openai/_base_client.py", line 1581, in _request
return await self._retry_request(
File "/tmp/ray/session_2024-11-14_19-11-03_110436_1/runtime_resources/pip/8f9a6c08a6f7b36cef5b248cf848c00d3b8e4aef/virtualenv/lib/python3.10/site-packages/openai/_base_client.py", line 1581, in _request
return await self._request(
File "/tmp/ray/session_2024-11-14_19-11-03_110436_1/runtime_resources/pip/8f9a6c08a6f7b36cef5b248cf848c00d3b8e4aef/virtualenv/lib/python3.10/site-packages/openai/_base_client.py", line 1533, in request
return await self.request(cast_to, opts, stream=stream, stream_cls=stream_cls)
File "/tmp/ray/session_2024-11-14_19-11-03_110436_1/runtime_resources/pip/8f9a6c08a6f7b36cef5b248cf848c00d3b8e4aef/virtualenv/lib/python3.10/site-packages/openai/_base_client.py", line 1839, in post
return await self._post(
File "/tmp/ray/session_2024-11-14_19-11-03_110436_1/runtime_resources/pip/8f9a6c08a6f7b36cef5b248cf848c00d3b8e4aef/virtualenv/lib/python3.10/site-packages/openai/resources/files.py", line 443, in create
batch_file_upload = await async_client.files.create(
There are 100+ batches successfully submitted and in the dashboard…. I wonder if we can recover and just use that.
When this happens,  no batch_objects.jsonl. In the cache.
This should be written during the batch creation. So if some of the batch submission fails then we still have partial and can use that

Also check the logic here....
now if we are iteratively writing batch_objects.jsonl, just checking that it exists doesn't mean we can skip batch submission (we need to instead resume by looking at the metadata of which batches have been submitted and which ones haven't)


Ease consumption by means of .tool-versions

This allows using a .tool-versions file as per asdf constraints.
The README.md changes should make the content of the pull request explicit.

Edit
This is working as expected as can be seen for example repo (link to .tool-versions):

here: success / versions installed as per .tool-versions,
here: error / version-type =/= strict
and here: error / colliding inputs

Closes #137.

Moving to Review-ready.
@kevinlang

From what you wrote, I have a feeling it doesn't hurt, but none of you are inclined to accept it, is that it, @starbelly, @wojtekmach? I guess if that's the case we need a better discussion workflow. 😄 There was an issue around this, and the idea wasn't reject by anyone. I later wrote I'd start to implement it, and I was apparently encouraged, but I might have over-read it, or under-reading what you wrote here.
I don't mind closing this pull request and moving forward, but I would like more concrete opinions on whether we keep it or drop it. 👍

I think having .tools-versions checked into the project is mainly beneficial for private applications, as @starbelly noted. I don't think disliking the default ASDF behavior or respecting .tools-versions is a sufficient reason for excluding this feature. If one does not like that ASDF behavior, the solution is to configure their ASDF to not respect repository-local .tools-versions file. This can be done as outlined in this comment

Sorry I wasn't clear. I personally wouldn't add it for the reason mentioned. But I'm not opposed to adding it. Does that make sense?

Sorry I wasn't clear. I personally wouldn't add it for the reason mentioned. But I'm not opposed to adding it. Does that make sense?

It does, thanks :)

And I know there might be this tendency to say "but it's optional", etc., but options have to be maintained too, so I do understand the rationale behind your rationale. (and also that we might run into issues regarding proper parsing of the tools file).

I just wanted to say great job on this @paulo-ferraz-oliveira , there's a lot of teams that will get a lot of mileage out of it! ❤


Update utils.lua

Does this help? Adding optional parent input to show_notes_dialog().

Can you please summarize the issue this is intended to solve?

I can summarize it. The issue is that when you launch a modal window from another window, that window needs to be specified as the parent. This is of particular importance on Windows. On Windows, if you don't properly specify the parent, the modal window ends up modal with respect to the Finale window but not modal with respect to the window that launched it. That is, you will be able to access and interact with the window that launched it unless you specify the launching window as parent.

@rpatters1 Particularly on Win, but the param can be passed on both platforms with no ill effect?
I'd like to suggest changing the order of params to the function--a breaking change to a function no one is using yet. 😄 It sounds like parent will be used for most calls, and the other three will be used less often (defaults should generally be fine). So I think parent should be moved to the head of the param list, so that callers don't need to pass three nils every time.

It actually has benefit on macOS as well: it centers the new window on the parent if there is one.
I'm okay with a breaking change in the order of parameters.  @cv-on-hub will need to add corrections in the PR in every script where it is called.

Good suggestions. I've checked /src/ and no scripts currently use show_notes_dialog so changing parameter order looks benign. I've also added  :gsub("%.%.%.", "") to the default caption.


bug fix: llama kv cache part

I'm wondering if the part of using kv cache is wrong because of wrong indexing.

Do you experience strange behavior?

@keisuke-niimi-insightedge-jp Thank you!


compiletest: no need to call rustc here

This also means we do not need the RUSTC_TEST_SUITE env var any more.

@bors r+

:pushpin: Commit 791ec8fef740ecbf3ad63ae1c5a3001e1ec45598 has been approved by RalfJung

:hourglass: Testing commit 791ec8fef740ecbf3ad63ae1c5a3001e1ec45598 with merge a85f0611535ca76c79703457f333d4a2361cfca9...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing a85f0611535ca76c79703457f333d4a2361cfca9 to master...


Add game Call Of Duty

Description
Adds game Call Of Duty
Fixes:  #1941

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I have made this from my own
[x] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@Sulagna-Dutta-Roy I have updated the directory name. Please merge this

Closing this one as I received new PR @arunimaChintu


Latency measurement incorrect

I have the gaming running locally and when I open the developer mode (the ctrl+shift+j in chrome) I see a message regarding latency showing up regularly. This measure seems very wrong, as it shows values like  "Latency:<PHONE_NUMBER>819ms".

Same here on (Firefox) Nightly…

The ping-pong functions can't be named "ping" and "pong" in socket io because those namespaces are already used somewhere else in socket io. Renaming the sockets to "drip" and "drop" for example should fix it.


Fix cross-attention

Proposed changes
With the current code, ln2 is a no-op. Its output should be passed to the cross-attention layer
Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[ ] I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have updated the necessary documentation (if needed)

@jbochi Would you like to add your name to our contributors section? We just added it

What an honor to be the first contributor on the list. :D
BTW, I'm super impressed by MLX. Amazing work!


0 4 58

Description
Just bumping js-slang to 0.4.58.
Fixes a bug in Non-Det, and restricts === in Source §1-2 to deal only with numbers and strings.
Fixes # (issue)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Code quality improvements

How to Test
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist
Please delete options that are not relevant.

[x] I have tested this code
[ ] I have updated the documentation

Pull Request Test Coverage Report for Build 6882

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

Totals

Change from base Build 6824:
0.0%

Covered Lines:
2345

Relevant Lines:
6305

💛  - Coveralls


GD-292: SimulateKeyPressed for SceneRunner not working

The used GdUnit4 version:
https://github.com/MikeSchulze/gdUnit4/releases/tag/v4.2.0
The used Godot version:
Godot Engine v4.2.stable.mono.official.46dc27791
OS including version:
Vulkan API 1.3.242 - Forward+ - Using Vulkan Device #0: NVIDIA - NVIDIA GeForce GTX 1650
Windows 10 - 10.0.19045 Build 19045
Describe the bug
First I show manual Space, which creates 2 nodes and passes.
Then I use the SimulateKeyPressed on the SceneRunner to automate the key press, which does not create a second node. Instead I see (no focus set):
ActivateTimeFactor: Engine.TimeScale=1, Engine.PhysicsTicksPerSecond=60
set time factor: 1
set physics iterations_per_second: 60
    no focus set
    process event <Node2D#81486939523> (PlayerTests) <- Space
    no focus set
    process event <Node2D#81486939523> (PlayerTests) <- Space

Steps to Reproduce

Minimal reproduction project:
N/A since project is currently private

I believe you are correct, my issue still occurred when I moved the action handling logic to _Input.
After more debugging, I figured out that it was due to not having nodes in a given group - thus skipping the "split" call.
Ultimately this was caused by a deferred call to add the player node to a group in _Ready().
More details:
// scene script
public override void _Input(InputEvent @event)
{
	base._Input(@event);
	if (Input.IsActionJustReleased("Split"))
	{
		List<Player> players = GetTree().GetNodesInGroup(Level.instance.GroupName).Cast<Player>().ToList();

                // this was being skipped since players list was empty
		foreach (Player player in players)
		{
			player.Split();
		}
	}
}

// player script
public override void _Ready()
{
    base._Ready();

-    // this waits to add the player node to the group at the end of _Process frame
-    CallDeferred(Node.MethodName.AddToGroup, GroupName);
+  AddToGroup(GroupName);

    ...
}

I appreciate your help on this issue as I am new to Godot & C# in general.

Nice that we were able to narrow down the problem. Since the simulate function works as expected, I would now close the ticket as resolved.
Have fun with GdUnit4, I am always grateful for hints or suggestions for improvement


Upgraded KiCad to v4.0.0

Just noticed PR #15629. Looking into that.

fc057fd should address the suggested outcome of the conversation of #15629

Closing in favor of https://github.com/caskroom/homebrew-cask/pull/15692 (more changes were needed)


sama5: Disable CONFIG_NETUTILS_NETLIB_GENERICURLPARSER for a config

There is no clear reason why this option is enabled here.
This is a preparation for
https://github.com/apache/incubator-nuttx-apps/pull/278
Summary
Impact
Testing

LGTM, it seem that github become unstable and cacnell "Build / macOS (sim)" accidently.


chore(Java): add cyclically rotate on arrays

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] New algorithm
[ ] Optimization in previous algorithms
[ ] Code style update
[ ] Refactor
[ ] Documentation
[ ] Other, please describe:

Briefly describe the changes in this PR

Other information:

Hi @pratik-a,
Any update?

Hi @pratik-a,
Any update?

will update in docs

Hi @pratik-a,
Any update?

will update in docs

Ok, waiting for your update

Hi @pratik-a,
This pull request will be closed, any updates please re-open this pull request.


tagging fails

when i try to add a tag to an bookmark this does not work. in the log i get:
ElasticsearchIllegalArgumentException[failed to execute script]; nested: ScriptException[scripts of type [inline], operation [update] and lang [groovy] are disabled]; , called from sub Search::Elasticsearch::Role::Client::Direct::ANON at /home/mila/Projects/Software/OpenShift/bkmrx/bkmrx.com/script/../lib/BkxMojo/Ajax.pm line 480. With vars: {'status_code' => 400,'body' => {'status' => 400,'error' => 'ElasticsearchIllegalArgumentException[failed to execute script]; nested: ScriptException[scripts of type [inline], operation [update] and lang [groovy] are disabled]; '},'request' => {'method' => 'POST','mime_type' => 'application/json','body' => {'script' => 'if (!ctx._source.tags.contains(tag)) {ctx._source.tags += tag} else {ctx.op = "none"}','params' => {'tag' => 'dddddd'}},'qs' => {'parent' => '96e7ce197da22d57c8c308c14a548c4d11005019'},'serialize' => 'std','path' => '/bkmrx/bookmark/072d0c6b0dd9cb636615355030790a0e220fa1c2/_update','ignore' => []}}
adding a description results in:
DocumentMissingException[[bkmrx][4] [bookmark][072d0c6b0dd9cb636615355030790a0e220fa1c2]: document missing], called from sub Search::Elasticsearch::Role::Client::Direct::ANON at /home/mila/Projects/Software/OpenShift/bkmrx/bkmrx.com/script/../lib/BkxMojo/Crud.pm line 350. With vars: {'body' => {'status' => 404,'error' => 'DocumentMissingException[[bkmrx][4] [bookmark][072d0c6b0dd9cb636615355030790a0e220fa1c2]: document missing]'},'status_code' => 404,'request' => {'ignore' => [],'serialize' => 'std','path' => '/bkmrx/bookmark/072d0c6b0dd9cb636615355030790a0e220fa1c2/_update','qs' => {'parent' => '072d0c6b0dd9cb636615355030790a0e220fa1c2'},'mime_type' => 'application/json','body' => {'script' => 'ctx._source.user_description = page_desc','params' => {'page_desc' => 'sssssssss'}},'method' => 'POST'}}

Hmm which version of ElasticSearch are you running? This function uses scripting features that were disabled in v1.3 I think and I haven't had time to update the functions
Have clarified install instructions on version required.

ok, thanks. i used 1.7.1.

You can read more about the issue here: https://www.elastic.co/guide/en/elasticsearch/reference/current/modules-scripting.html
It is possible to switch scripting back on I believe even in later versions


Help with a version to use with Prestashop 1.7+

Hi i have more or less a week trying to get Docker to work with Nginx Php-fpm an mariadb locally to use it with Prestashop 1.7
when i found your repo, it just took me half night to run it.
the problem is that to make it work, i had to chance many conf files and i think that i disable redis
i have no idea how to check it or if i mess up something, so can you help me checking?.
Prestashop 1.7 only runs with PHP 7.2 max, so i took you Docker php-fpm file and did this to build a new image:
`FROM php:7.2-fpm-alpine3.9
ENV VAR_PREFIX=/var/run
ENV LOG_PREFIX=/var/log/php-fpm
ENV TEMP_PREFIX=/tmp
ENV CACHE_PREFIX=/var/cache
RUN set -x
&& apk add --no-cache --virtual .build-deps
wget
linux-headers
curl
unzip
&& echo '@community http://dl-cdn.alpinelinux.org/alpine/v3.9/community' >> /etc/apk/repositories
&& apk add --no-cache --update
php7-dev@community
php7-bcmath@community
php7-dom@community
php7-common@community
php7-ctype@community
php7-cli@community
php7-curl@community
php7-fileinfo@community
php7-fpm@community
php7-gettext@community
php7-gd@community
php7-iconv@community
php7-json@community
php7-mbstring@community
php7-mcrypt@community
php7-mysqli@community
php7-mysqlnd@community
php7-opcache@community
php7-odbc@community
php7-pdo@community
php7-pdo_mysql@community
php7-pdo_pgsql@community
php7-pdo_sqlite@community
php7-phar@community
php7-posix@community
php7-redis@community
php7-session@community
php7-simplexml@community
php7-soap@community
php7-tokenizer@community
php7-xml@community
php7-xmlreader@community
php7-xmlwriter@community
php7-simplexml@community
php7-zip@community
php7-zlib@community
php7-intl
mysql-client
curl
monit
bash
xz
openssl
icu-libs
ca-certificates
libxml2-dev
tar
&& mkdir -p /var/run
&& mkdir -p ${LOG_PREFIX}
&& rm -rf /tmp/*
&& rm -rf /var/cache/apk/*
&& touch ${LOG_PREFIX}/access.log
&& touch ${LOG_PREFIX}/error.log
&& ln -sf /dev/stdout ${LOG_PREFIX}/access.log
&& ln -sf /dev/stderr ${LOG_PREFIX}/error.log
COPY conf/monit/ /etc/monit.d/
COPY docker-entrypoint.sh /docker-entrypoint.sh
COPY check_wwwdata.sh /usr/bin/check_wwwdata
COPY check_folder.sh /usr/bin/check_folder
EXPOSE 9000
RUN chmod +x /docker-entrypoint.sh /usr/bin/check_wwwdata /usr/bin/check_folder
ENTRYPOINT ["/docker-entrypoint.sh"]
CMD ["php-fpm7", "-g", "/var/run/php-fpm.pid"]`
then i had to change the nginx secure.conf nginx.conf location.conf as the folders are different to WP, here is the PS official Nginx config https://github.com/PrestaShop/PrestaShop/blob/develop/docs/server_config/nginx.conf.dist
I think i mess up on the default.vhost as i had to change the location / to this
location / {      error_page                   502 =200 @failed;      try_files $uri $uri/ /index.php?$args;             if (!-e $request_filename) {                 rewrite ^/.*$ /index.php last;             }   } include                         /etc/nginx/conf.d/location.conf;   include                         /etc/nginx/redis.d/location.conf;
taking out the proxy (if i live it, the admin panel does not work)
also i did change the location .php in location.conf as when i was getting into the web page it was downloading the php file instead of ruing it.
`location ~ [^/].php(/|$)
{
try_files                    $fastcgi_script_name /index.php$uri&$args @php-fpm;
fastcgi_index                index.php;
include                      fastcgi_params;
fastcgi_pass                 php-fpm;
include                      /etc/nginx/fastcgi.d/fastcgi.conf;
include                      /etc/nginx/redis.d/cache.conf;
}
location ~*  .(jpg|jpeg|png|gif|ico|css|js|woff(?:2)?)$ {
expires 365d;
}
location ~* .(css|js)$ {
expires 7d;
}
    location = /favicon.ico {
        auth_basic off;
        allow all;
        log_not_found off;
        access_log off;
    }

    location = /robots.txt {
        auth_basic off;
        allow all;
        log_not_found off;
        access_log off;
    }

    # Images
    rewrite ^/([0-9])(-[_a-zA-Z0-9-]*)?(-[0-9]+)?/.+.jpg$ /img/p/$1/$1$2$3.jpg last;
    rewrite ^/([0-9])([0-9])(-[_a-zA-Z0-9-]*)?(-[0-9]+)?/.+.jpg$ /img/p/$1/$2/$1$2$3$4.jpg last;
    rewrite ^/([0-9])([0-9])([0-9])(-[_a-zA-Z0-9-]*)?(-[0-9]+)?/.+.jpg$ /img/p/$1/$2/$3/$1$2$3$4$5.jpg last;
    rewrite ^/([0-9])([0-9])([0-9])([0-9])(-[_a-zA-Z0-9-]*)?(-[0-9]+)?/.+.jpg$ /img/p/$1/$2/$3/$4/$1$2$3$4$5$6.jpg last;
    rewrite ^/([0-9])([0-9])([0-9])([0-9])([0-9])(-[_a-zA-Z0-9-]*)?(-[0-9]+)?/.+.jpg$ /img/p/$1/$2/$3/$4/$5/$1$2$3$4$5$6$7.jpg last;
    rewrite ^/([0-9])([0-9])([0-9])([0-9])([0-9])([0-9])(-[_a-zA-Z0-9-]*)?(-[0-9]+)?/.+.jpg$ /img/p/$1/$2/$3/$4/$5/$6/$1$2$3$4$5$6$7$8.jpg last;
    rewrite ^/([0-9])([0-9])([0-9])([0-9])([0-9])([0-9])([0-9])(-[_a-zA-Z0-9-]*)?(-[0-9]+)?/.+.jpg$ /img/p/$1/$2/$3/$4/$5/$6/$7/$1$2$3$4$5$6$7$8$9.jpg last;
    rewrite ^/([0-9])([0-9])([0-9])([0-9])([0-9])([0-9])([0-9])([0-9])(-[_a-zA-Z0-9-]*)?(-[0-9]+)?/.+.jpg$ /img/p/$1/$2/$3/$4/$5/$6/$7/$8/$1$2$3$4$5$6$7$8$9$10.jpg last;
    rewrite ^/c/([0-9]+)(-[.*_a-zA-Z0-9-]*)(-[0-9]+)?/.+.jpg$ /img/c/$1$2$3.jpg last;
    rewrite ^/c/([a-zA-Z_-]+)(-[0-9]+)?/.+.jpg$ /img/c/$1$2.jpg last;
    rewrite ^/[a-zA-Z][a-zA-Z]/index.php(.*)$ /index.php$1;
    try_files $uri $uri/ /index.php?$args;

    # AlphaImageLoader for IE and fancybox
    rewrite ^images_ie/?([^/]+)\.(jpe?g|png|gif)$ js/jquery/plugins/fancybox/images/$1.$2 last;

    # Web service API
    rewrite ^/api/?(.*)$ /webservice/dispatcher.php?url=$1 last;

`
so at the end i am not sure of what i have done, if you can help me it will be fantastic!, thank you

Hi @msaustral , a bit late to reply. Hope you figured this out.  It would take a fair amount of time to troubleshoot your work. This falls outside of this project given we do this for free and you (I assume) are getting paid for your work ; )
If you wanted to contact an engineer to dig into this, we do have hourly rates.

Hi thank you for your reply, we have no problem to pay for if the price is right, have a good day.


Resolve dependencies to the databases automatically

@hasegeli Your changes are currently causing failures in the grant_spec acceptance test's on lines 98 and 110. Please fix this before it can be merged.

Could you please retrigger the tests?

@hasegeli I've closed and opened the PR. The test's should show fine now.

Hello, we have been using your submodle for a while and we are currently having an issue
when I run puppet parser validate on server.pp I get the following error message
Error: Could not parse for environment production: Syntax error at 'Hash'; expected ')' at /home/mayres/git/puppetlabs-postgresql/manifests/server.pp:59
Can you please point me in the correct direction to fix ?

@hasegeli I am know closing this as you had previously mentioned in another PR that you had finished working with Puppet. I would like to once again state how sorry I am to hear that, and hope to one day hear from you again in the future.


Package user reviews

Description
This is more of an enhancement request.
It would be nice to have a review system of packages, that way you can see the quality of the package other than just the number of times it was downloaded

Thanks for the suggestion!  I was almost positive I've seen this request before but couldn't find an issue - can you re-open this in https://github.com/atom/atom.io if there's no existing issue already so we have it in the right place?  I thinkkkkk that might be a better place for this request.

okay


[vcpkg-scripts][boost] Improve generate-ports.ps1

Change homepage URLs to point to documentation instead of GH.
Allow encoding alternative propagation of supports/platform so that changes don't need to be selected manually.

Allow to suppress the generated dependency platform expression when a dependency is non-optional.
(boost-parameter requires boost-python, and the dependency transitively determines the supported platforms of the dependent port.)
For port boost, generate platform expression from transitive supports limitations.
(boost-parameter platform expression must account for boost-python.)
This fixes port boost for uwp. And so it can now be directly referenced from vcpkg-ci-boost which is already enforced to  pass in ci.baseline.txt. :tada:

Use the same tarball download area and naming as vcpkg install.

This PR doesn't include the updates to boost-container (#38806) and bost-interprocess (#38815).

FTR I do have a variant of boost-generate.ps1 that parses CMakelists.txt for use Boost:: targets instead of headers and sources for use of  #include. With all tarballs downloaded, generation run time is down from 180 s to 6 s. However, the list of dependencies is significantly different.

Addition of dependencies due to use of unneeded CMake targets (cf. #38806)
Addition of dependencies due to being unable to detect conditions. (However, the current parser isn't aware of conditions either.)
Removal of dependencies which I assume to be still available as transitive dependencies.

FTR I do have a variant of boost-generate.ps1 that parses CMakelists.txt for use of Boost:: targets instead of parsing headers and sources for use of #include. With all tarballs downloaded, generation run time is down from 180 s to 6 s. However, the list of dependencies is significantly different.

That should've been done when build was changed to cmake. A better solution would be to ask Boost to list dependencies in meta/libraries.json.

That should've been done when build was changed to cmake. A better solution would be to ask Boost to list dependencies in meta/libraries.json.

Again, the key problem is to attach the right guards on conditional dependencies.

Add reviewed tag for further review by the vcpkg team.

WAH why is the script flashing terminals in my face now.


Crash when trying to load worlds that have been opened to LAN

I don't know if you guys are already aware, but the latest release (0.86) causes a crash when loading a world that has previously been opened to LAN in a manner that resembles crashes caused by client-side only mods being run on a server.
Is the mod intended to be SMP/LAN incompatible? or am I simply doing something wrong to cause this?
below is a pastebin link with the crash log:
http://pastebin.com/GUpTDW5P

Hmmm, looks interesting. Will try to fix it myself if @BigBastard doesn't get here first.
I'm running it on a dedicated server and it works fine, so it's probably something to do with open to LAN. Wil come back later with moar infoz.

Can you post the full log though?

do you mean the FML-client log?
http://pastebin.com/vYRZfvt5

If I start a new world and open it to LAN it doesn't crash when loading. It's only existing worlds. So I may have misdiagnosed this. It may be caused by items/blocks/entities from previous mods being placed in the world.
The crash log and FML-log are both from running the game with only this mod installed, although as the FML log shows the save was run previously with other mods on.

The crash report doesn't show anything from Valkyrien Warfare.

But it only happens with it installed, also still happens with only it installed.
I can load the saves normally if I uninstall Valkyrian Warfare.
I thought given those facts I can assume it's caused by this mod.

sorry accidentally clicked close and comment earlier

I use the same forge as is on the github,  as well as openjdk 8

Well a temporary solution that for some reason worked is removing them mod, loading into the world, executing /kill @e then installing the mod and the world loads normally.

Issues seems to be fixed in latest release, please tell us if issue persists in this version. https://jenkins.ironhide.de/job/Valkyrien-Warfare-Revamped/lastBuild/artifact/build/libs/ValkyrienWarfareRevamped-1.10.2-0.8.6d-23.jar

Ok, now it has to be fixed. I'm closing for now, lan worlds I tested were fine


Environment copied before being mutated

This line initializes the Environment, when not initialized by the caller:
https://github.com/compose-spec/compose-go/blob/8585af8ec50b47489c1e7304dde3a7577aa658e4/loader/loader.go#L269
But the environment gets copied (implicitly) earlier on this line:
https://github.com/compose-spec/compose-go/blob/8585af8ec50b47489c1e7304dde3a7577aa658e4/loader/loader.go#L249
So, when Environment is nil, the set of COMPOSE_PROJECT_NAME in the map is not visible by the call to LookupValue.
A potential fix is to change func (cd ConfigDetails) LookupEnv(…) to a pointer receiver to avoid the copy.

I don't get it. we capture LookupEnv func, so actually the configDetails receiver, then when executed this function will access receiver field Environment, even this one was set after.
Please provide an illustration example (either code or docker compose usage) to demonstrate a bug


bootstrap.less not compiled by less.js

I get "b.value is undefined" in firefox and "Cannot call method 'charAt' of undefined" in chrome.
Am I doing something wrong?

Me too.
I find fixed the in chrome. modify the sprites.less file:(add double-quote)
background-image: url("@iconSpritePath");
background-image: url("@iconWhiteSpritePath");
But I still don't know what's the problem with firefox!!

Thanks a lot, it works for me in both chrome and firefox now, right?

That fixes the error that LESS churns out but it stops any sprites from being displayed

I'm a newcomer to less.js, but I was able to solve this issue by changing the lines in sprites.less to:
background-image: url("@{iconSpritePath}");
background-image: url("@{iconWhiteSpritePath}");

Yup, resolved in 2.0.2-wip.

Would you please merge master branch with this fix?


Include openmetrics integrations in http validation

What does this PR do?
This PR makes sure that integrations that are not directly using the http wrapper but inherit the OpenMetricsBaseCheck are also validated. Also makes the validation a little bit faster by only checking python files and fixes message warning to incorporate openmetrics errors.
Motivation
I noticed that some openmetrics integrations that were missing the openmetrics template did not fail validation because it was only looking for explicit self.http calls.
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

🔥

/azp run


Unable to select anything after screenshotting react-flow pane

Steps to reproduce

On Mac, press cmd + shift + 4 to switch to screenshot cursor
Click and drag to screenshot roughly the entire pane of react-flow board
Click on any node, any edge, zooming, moving around in the pane, etc

Actual Behaviour
Anything inside the pane is unselectable
Expected Behaviour
Can select node, edge, zoom, move around, etc
Bug demo

However, it can be fixed when I click the shift button.

I am not sure if this answers your question but in order for a node to be selected via "multi-select", the selection box needs to encapsulate the entire node.  In the animated gif you provided, roughly 4th select in, the selection box encapsulated the entire "message_1" node which successfully selected the node.

duplicate of #824


[FP]: Maven package 'airac' detected as Keybase desktop app because of developer org URL in pom file

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:keybase:keybase:1.0.0:::::::*
CVE
CVE-2019-7249
ODC Integration
{"label"=>"CLI"}
ODC Version
7.3.0
Description
Hello,
The Maven package called 'airac' with the following coordinates:
<groupId>com.ko-sys.av</groupId>
<artifactId>airac</artifactId>
<version>1.0.0</version>
is incorrectly detected as cpe:2.3:a:keybase:keybase:1.0.0:::::::* ('Keybase' desktop application for secure messaging, https://keybase.io/), apparently because of the following evidences:

Type
Source
Name
Value
Confidence

Vendor
pom
developer org URL
https://keybase.io/wjkohnen/
Medium

Product
pom
developer org URL
https://keybase.io/wjkohnen/
Low

which leads to the following CVEs being incorrectly detected: CVE-2019-7249, CVE-2021-34422, CVE-2018-18629, CVE-2021-34426, CVE-2021-23827, CVE-2022-22779.
Thank you in advance.
Regards,

approved


Why are there discrepancies between the documentation claim and actual benchmarks?

Please check https://flageval.baai.ac.cn/#/taskIntro?t=en_qa

This is just misspelling of benchmark datasets... @ifkite @fggggg7142 @xuanricheng @philokey Any update?

Closed since the latest documentation fix this mismatch:


Add initial functions

build/2
create/2
create_pair/2
create_list/2
assoc/2

What do you think about making separate test files for build, create, etc?


Use Steam Web API in watchlog.py instead of crawling steamdb

As a recommendation, you should translate steam app id's by using steam's own web API rather than requesting steamdb.info and crawling that page for the title of the depot. It's a cleaner approach, requires fewer overall requests and puts less strain on steamdb.
You can get a list of all steam games from this open steam API endpoint:
https://api.steampowered.com/ISteamApps/GetAppList/v2/
It has a much larger output (3.7MB) than steamdb (150KB), but you'll only have to download the applist once :)

Hi muffeeee, unfortunately this wouldn't do all of what's required - we need a way to translate the depot ID to the app ID.
Open to ideas of how to fetch the app ID based on the depot in a reliable and guaranteed way. Whilst it's generally true that for depots 551 to 559 it will be appid 550 for instance, the same isn't true for other games with more depots.

Closing this for now as the suggestion isn't workable as far as we know.
Please reopen if this proves to be incorrect.


Easily switch between Settings UI and settings.json by reusing the shortcut

When hitting CMD+,, the settings UI opens. It'd be neat/handy if upon hitting CMD+, again, it'd swap out the settings UI for the settings.json file.

You can set up a separate keybinding to open the json file (I use alt+cmd+,), or you could even set up a cmd+, keybinding to do that using a when clause to only activate when the settings UI is open. I don't think I want to do this out of the box by default though.


support Cloud Events as first-class event type

Should we support Cloud Events (https://cloudevents.io/) as a first-class event type since Event Grid supports them (https://azure.microsoft.com/en-us/roadmap/azure-event-grid-support-cncf-cloud-events/)?
@asw101 thanks for the suggestion!

Makes sense to me! The work to do here is pretty thin.

Define structs which unmarshal these nicely. This should likely be automatically done for us by extending the model definitions in the specs repository. I'd be hesitant to write my own until I knew the Event Grid team wasn't going to write Swagger for them.
Extend the "wellKnownEvents" to include them.


chore: eslint - fix unused vars warnings, add precommit lint - part 1

Which problem is this PR solving?

This PR removes warnings regarding  unused vars (https://github.com/open-telemetry/opentelemetry-js/issues/1093)

Short description of the changes

Stub classes methods arguments were prefixed with "_". Unused vars - deleted. Now this rule will throw error

Also, precommit lint hook introduced

Thank you for starting this process. All the warnings have been driving me crazy 😜


Inherited Process Template named SCRUMUA didn't get exported to my other VSTS Account. Can you please do let me know what am i doing wrong.

Hi ,

I installed the NPM through the “node-v8.11.3-x64”  from your link and it installed successfully.
Then I create the PAT from both the Source VSTS account and from the Target VSTS account.
Then I manually created the Configuration.json file.
The Configuration.json file is in the attachments for your review, I changed the PATs from it for
security reason.
Then I ran the Node.Exe file.
Command Prompt open and there I ran the command
process-migrator [--mode=<migrate(default)/import/export> [--config=]

Which I ran on my machine like this:-
process [export [--config=C:\Users\uabbasi\Documents\configuration.json]
But the TFS Process Template named SCRUMUA didn’t get exported to Target VSTS account.
I think the command i am running is incorrect so can you please let me know how to correct this command And if something else is missing or incorrect then can you please do let me know that.
If possible can you please share any video or stepbystep guide regarding how to use this process migrator which shall by very helpful for us.
I have attached the node.exe command prompt image and also the configuration.json file in this ticket.

configuration.zip

Hi ,
Can you please help me, as no reply has been received yet on my issue posted on 18th July.
Thanks,
Umair Abbasi.

Don't run node.exe first. When you install packages through NPM it adds executable commandline files in your path. Just open a regular command prompt and run the command

Hi ,
Thanks for your reply.  Can you please check my Configuration.json File that is present in the attachments above and please can you do let me know that exact command that i shall run, the command which i ran above in the image is it correct ? please reply.
Thanks,
Umair Abbasi.

Sorry, I haven't actually used this tool, my comment was just based on general Node/NPM conventions and seeing your screenshot.
PS: Email attachments don't work for GitHub issues, you can only add files on the web.

I'm not a Microsoft employee, I'm just giving you some general advice based on reading your issue
PS: You've posted you token that you might want to revoke now https://github.com/Microsoft/process-migrator#-configuration-file-has-pat-right-protect-it-

Hi ,
So can you please guide me to whom shall i contact in this regard.
And this token is incorrect and incomplete purposely and i shall also delete it from my vsts account too.
Thanks,
Umair Abbasi.


Make image resposive

Hey! How can I make image from editor responsive?

This repo is just a wrapper for CKEditor 4.x
Maybe you can solve your problem with CKEditor pluins: http://ckeditor.com/addons/plugins/all


remove Labels from ControllerFields

This commit removes Labels field from ControllerFields struct
in pkg/types.go.
We do not require Labels because ControllerFields already has
(meta)v1.ObjectMeta struct embedded into it which contains the
Labels field.

it looks like this is already done in https://github.com/kedgeproject/kedge/pull/355

Closing since redundant, already fixed in #355


[Question] Are multiple API Calls handled in parallel?

(Probably) a very simple question but I didn't find an answer to it:
Does TheHives API answer multiple requests in parallel? I am trying to automate an observable creation and I dont want to run into a race condition (TheHive not noticing that the same observable is simultaneously beein created by another API Call)

So I started submitting multiple url observables (some identical) in paralell. If they are added to the case successfully I store the obsId if I get a conflicterror (obs already exists in case) I get all case observables via the api, look for the identical one and get it's obsId.
This is where I noticed that sometimes creating an observable is answered with a conflicterror but the case observables list does not yet contain the observable, when I request this list right after the conflicterror.

Hello @Passimist I'm not sure to understand the use case.
The API calls are not particularly queued

Hello @nadouani
thanks for the reply. I am very confused myself by the issues I faced in the last days. I hope this is helpful. If you want me opene seperate issues for each of the following topics I can do that instead.
What I noticed is in steps:
1: I create an observable via thehive4py
2a: I try to create the same observable again and get a conflict error
2b: I get all case observables of this case and the observable is not part of the list of returned observables (tho it should already be part of this case since I got the conflict error telling me the observable already exists)
Step 1 and 2 run in parallel while 2a is done before 2b.
Another issue I noticed while adding a lot of file observables this way is that sometimes when I create a file observable the logs show a NoSuchFileException like in https://github.com/TheHive-Project/TheHive/issues/1089 and https://github.com/TheHive-Project/TheHive/issues/1098
But while in those two issues the observable has not been created or was not downloadable I now noticed a file observable beeing created but its SHA256 beeing that of an empty file (which is not what I uploaded) and all analyzers analyzing an emtpy file. (This might be dangerous I hope I have the time and skill to reproduce this in a few days)
The Error in TheHive/application.log:

2020-11-09 13:31:06,213 [ERROR] from akka.actor.RepointableActorRef in application-akka.actor.default-dispatcher-3 - Error during preStart in [FileSource(/tmp/playtemp6353343187215886743/multipartBody9054269108493492983asTemporaryFile, 51200)]: /tmp/playtemp6353343187215886743/multipartBody9054269108493492983asTemporaryFile
java.nio.file.NoSuchFileException: /tmp/playtemp6353343187215886743/multipartBody9054269108493492983asTemporaryFile
at akka.stream.impl.io.FileSource$$anon$2.preStart(IOSources.scala:74)
at akka.stream.impl.fusing.GraphInterpreter.init(GraphInterpreter.scala:291)
at akka.stream.impl.fusing.GraphInterpreterShell.init(ActorGraphInterpreter.scala:557)
at akka.stream.impl.fusing.ActorGraphInterpreter.tryInit(ActorGraphInterpreter.scala:664)
at akka.stream.impl.fusing.ActorGraphInterpreter.preStart(ActorGraphInterpreter.scala:712)
at akka.actor.Actor.aroundPreStart(Actor.scala:528)
at akka.actor.Actor.aroundPreStart$(Actor.scala:528)
at akka.stream.impl.fusing.ActorGraphInterpreter.aroundPreStart(ActorGraphInterpreter.scala:655)
at akka.actor.ActorCell.create(ActorCell.scala:656)
at akka.actor.ActorCell.invokeAll$1(ActorCell.scala:527)
at akka.actor.ActorCell.systemInvoke(ActorCell.scala:549)
at akka.dispatch.Mailbox.processAllSystemMessages(Mailbox.scala:283)
at akka.dispatch.Mailbox.run(Mailbox.scala:224)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)

I then installed TheHive 4 to test if I face this issues in that version too.
The NoSuchFileException has not occurred so far. But I noticed something else that confuses me:

I create an URL observable.
I try to create the same one again and get a Create Error because the observable already exists
I run any analyzer on the created observable
I try again to create the same observable and it suddenly succeeds creating this identical observable after I ran an analyzer on the first one.
Is this intended?

Now this post is a huge mess... sorry. I hope you can still help me.

After some more testing I think I figured out whats causing the last issue. (The one about beeing able to create identical observables).
If I use thehive4py I am able to not fill the "message" attribute of the observable. If the observable does not have a message I am able to create another identical observable after waiting for a few seconds. Running an analyzer does not influence this (it just made me wait the few seconds.)


Unprivileged Users Not Supported?

I think since the merge of the dev branch it is no longer possible to use PowerZure with users who do not have an active subscription. During initialization PowerZure throws the error Get-AzRoleAssignment : Object reference not set to an instance of an object.
I think, most of the users "owned" during an assessment won't have any active subscriptions. Wouldn't it be better, if you could also query Azure/Azure AD with those users (or at least the parts of Azure, to which the users possibly have some access to) or am I missing something here?

there is something strange which at this point I am not sure is related to this issue you have raised or to the one you created a fix for.
When I tried your patched version for the first time I got an error about "an obect cannot be empty", and I noticed it happened because SubscriptionName TenantId fields were empty (after using connect-azaccount)
I am not sure why those fields are not populated sometimes, I've deleted the 3 json files  AzureRmContext AzurePSDataCollectionProfile AzureRmContextSettings, run connect-azaccount, fields are populated and all is good.
And I cannot reproduce the error.
But yes, it looks like that if you do not have a subscription (or powershell cannot retrieve it) you run into issues (basically you cannot use PowerZure)

Fixed in https://github.com/hausec/PowerZure/commit/f5ade6117ac7d0348a199d2c35b5f1b5a710cb2b


LIMIT OFFSET in delete()

How to enable SQLITE_ENABLE_UPDATE_DELETE_LIMIT, because this one
let messages = Table("messages")
.filter(Expression<String>("chatId") == chat[Expression<String>("id")])
.order(Expression<Date>("createdAt").desc)
.limit(Int.max, offset: index > 29 ? 1 : 30)

try db.run(messages.delete())

removes ALL rows from messages

@jberkel since #722 is now merged, should this be closed?

let's leave it open for now, might take a bit before the release branch gets merged into master

Released as part of 0.11.4


increase  does not increase beyond 100

pulse does not have 100 as the limit. I can do "set-volume 120", and it works. So, increase should check the max volume level, and allow the increase, instead of maxing it out at 100.

This is working as designed -- the limit of 100 via "increase" is meant as a safety measure. If you know that you want to increase the volume above 100, then you can override by using set-volume.

Perhaps a --max-volume flag could be added to change this behavior.

My understanding, from using GUI like pavucontrol, is that these devices have max limit anyway - like 150. So, it anyway gets clamped at say, 150.
The problem I am facing is I have mapped my i3 keybindings to increase and decrese by 10%. And then it gets clamped at 100. So, I have to manually type the command "set-volume" to increase the volume.

There has long been a max-volume flag:
$ ponymix increase 150
100
$ ponymix increase 150 --max-volume 150
150

But warning, likely will clip your speakers like crazy. If pulseaudio isn't getting loud enough at 100, something else is wrong - this is just a workaround.

Ah good job me, I already added this 4 years ago. So then this is just a dupe of https://github.com/falconindy/ponymix/issues/19.


Install Istio with istio-operator

Q
A

Bug fix?
no

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets

License
Apache 2.0

What's in this PR?

Istio was previously installed in an old-fashioned way, using Istio's Helm charts. Instead in this PR, we have a InstallIstioOperator posthook which installs istio-operator first, then we have the InstallServiceMesh posthook which creates an Istio CR which triggers the istio-operator to install Istio.
Additional context

InstallIstioOperator has higher priority than InstallServiceMesh and the istio-operator is created with wait: True so that the operator is ready by the time the Istio CR is created.
The same three options can be configured for now as before: mtls, autoSidecarInjectNamespaces and bypassEgressTraffic.
The API needs to be called a little bit differently with this solution, InstallIstioOperator has to be specified (without values) as a posthook like this:

  "postHooks": {
    "InstallIstioOperator": {},
    "InstallServiceMesh": {
      "mtls": true,
      "autoSidecarInjectNamespaces": [
          "default"
      ],
      "bypassEgressTraffic": true
    }
  }

Checklist

[x] Implementation tested (on GKE)
[x] Error handling code meets the guideline
[x] Logging code meets the guideline (TODO)
[x] OpenAPI and Postman files updated, client regenerated (make generate-client) (if needed)
[x] Related Helm chart(s) updated (if needed)

Can't InstallIstioOperator accept itself the same options? It's a bit weird that you need both InstallIstioOperator and InstallServiceMesh. I would deprecate InstallServiceMesh eventually or just remove it.

Can't InstallIstioOperator accept itself the same options? It's a bit weird that you need both InstallIstioOperator and InstallServiceMesh. I would deprecate InstallServiceMesh eventually or just remove it.

Reduced to one posthook. The posthook can be called the same way as before.


Alternative yarn approach

Currently we have two issues with yarn:

--modules-folder doesn't work (yarnpkg/yarn#3792)
npm run prepublish is run when installing, we don't need that

While issue (1) might be fixed in future, issue (2) may be related to (1) but may be fixed when (1) is fixed or maybe not.
So alternative approach would be to extract *dependencies portion of packages json into /usr/lib/node_modules/package.json and copy yarn.lock, the do yarn install there. Then copy yarn.lock  back.
Any obvious downsides of that? I think it may be problematic if anything other than our supported subset of package.json is used for installation. (or should be blacklist items there rather than whitelist?)
/cc @playpauseandstop, @anti-social, @alexmost

Am OK with solution, only comment to have /usr/lib/node_modules configurable in vagga.yaml as !YarnDependencies option or environment var.


[5.4] Add isSoftDeleting() method

As requested in https://github.com/laravel/framework/pull/17189#issuecomment-272025780

I guess I don't totally understand. Why would isSoftDeleting return true if I'm not actively deleting the model. I guess I don't see the use case for this?

I'm suggesting a method to check in a safe way if a model soft deletes. Currently you'd have to do something like method_exists($this, 'bootSoftDeletes') or check trait implementation using class_uses(). Since there's already a safeguard method for forceDelete(), I thought it could be beneficial to also allow checking of existence of the SoftDeletes trait as well. I actually encountered the code case mentioned below quite some times.
Guess it'll probably need a rename to softDeletes()?
trait SomeTrait {

    // This trait inherently doesn't know if its implementing model uses SoftDeletes in advance.
    protected function bootSomeTrait()
    {
        if (method_exists(static::class, 'bootSoftDeletes')) {
            static::restored(function() { // });
        }
    }
}

vs
trait SomeTrait {

    // This trait inherently doesn't know if its implementing model uses SoftDeletes in advance.
    protected function bootSomeTrait()
    {
        if (static::softDeletes()) {
            static::restored(function() { // });
        }
    }
}

Oops, sorry for closing accidentally.

Probably needs to go to 5.5.

@Propaganistas Would suggest that you call the method isSoftDeletable(). isSoftDeleting sounds like you are checking if the model is currently being deleted.

No plans on this right now.

@Shkeats I don't tend to agree. You would at least need class_uses_recursive() to accommodate to all use cases. And in that regard the following is more performant:
return method_exists(static::class, 'bootSoftDeletes');

Anyhow, this PR is closed.


PETSc usage in MFEM

Hello,
I ran into an issue when using PETSc in MFEM. I have a good initial guess for the solution to the linear system, so I would like to start GMRES from a nonzero initial guess. I was experimenting with the hdg_poissonp example (hdg branch), where the solver is set up as
PetscLinearSolver *petsc_solver;
petsc_solver = new PetscLinearSolver(MPI_COMM_WORLD, "solver_");

I tried the following two-line long PETSc input file
-solver_ksp_type minres
-solver_ksp_initial_guess_nonzero true

but the second line seems to be ignored. The number of iterations is the same even if it is set to "false".
Any help would be appreciated.
Best,
Tamas

Does Lambda include your initial guess satisfying boundary conditions?

The ParGridFunction lambda satisfies the BC. The projection lambda.ProjectCoefficientSkeleton(ucoeff); should take care of that. I just pushed my experimental version, where I have a few added lines
HypreParVector *Lambda =  new HypreParVector(M_space);
*Lambda = 0.0;
Lambda->Add(1.0, lambda);

With these lines, Lambda should be fine.

If I set the problem up such that u = 1, so the projection contained in Lambda is the exact solution, I would expect 0 iterations, and it does more than that. (Again, the same number independently of the second line).

Got it. That seems suspicious. @stefanozampini do you have a gut feeling about this?

@thorvath12 Yep, the option was always superseeded by the value of iterative_mode. https://github.com/mfem/mfem/pull/3309 should fix your problem. let me know

Discussion continued in #3309.


Other block disappear

Adding this plugin other blocks disappears

Are you able to reproduce it with the latest versions (grapesjs core and the plugin)?


ACP port support

Hi,
I have a simple question regarding your driver: does it support data transfer through the ACP port?
I tried with a zynq ultrascale+ device and I get the following log:
[  126.148496] xilinx-vdma a0000000.dma: Channel ffffffc87b1c0e18 has errors 200, cdr 704c0000 tdr 704c0180
[  127.148486] xilinx-vdma a0000000.dma: Cannot start channel ffffffc87b1c0c18: 10009
[  137.196167] axidma: axidma_dma.c: axidma_start_transfer: 301: DMA receive transaction timed out.
Failed to perform the AXI DMA read-write transfer: Timer expired
DMA read write transaction failed.
I also tried with the HPC (high performance coherent) port and it works nicely.

I didn't try the ACP port, the HPC port worked fine in non-coherent mode without any mods to the driver. Small change required to make it work cache coherent.

I tried again this morning and by enabling kernel logs at the highest level.
For a scatter gather hw, I get the previous logs and without scatter gather, I get:
[  190.360801] xilinx-vdma a0000000.dma: Channel ffffffc87b191418 has errors 20, cdr 0 tdr 0
I'm using your axidma_transfer user space example program and, in this last hw configuration, I get an output file correctly sized but filled with "@@@@@".

Here, you can find the logs for both the benchmark application & the dmesg:
axidma_benchmark.log
dmesg.log


Documentation 🐸

From #1

[ ] The usage of the protocol is well documented
[ ] The README gives enough background
[ ]  The protocol is clearly motivated
[ ] The README discusses closely related work

       .--._.--.
      ( O     O )
      /   . .   \\
     .\`._______.'.
    /(    \\_/    )\\
  _/  \\  \\   /  /  \\_

.~   `  \  \ /  /  '   ~.
{    -.


The replaceAll function is a new function in javascript, somehow obsidian must be using an older version https://www.designcise.com/web/tutorial/how-to-fix-replaceall-is-not-a-function-javascript-error. No idea why its different for you :/
A simple fix would be to replace all calls to replaceAll

A simple fix would be to replace all calls to replaceAll
That worked out for now.
I hope that this is issue only came up because I'm running the Insider Build and that it won't be a problem for other users once the next version of Obsidian appears on the official channel.


wakatime-cli not installed (Windows fix)

I use this plugin for Git Windows and the change for path in commit #15  breaks that support!
if (( ! $+commands[wakatime] )); then  <--- This works
if ! wakatime_loc="$(type -p "$wakatime_bin")"; then <--- This does not work

Maybe some type of else check using the old logic might work.
It seems that the type -p command does not actually work on windows.
So if you get questions about this in the future that might be the issue.

Thanks! @hiq-davidholmstedt can you please send a pr with windows-friendly solution?
I don't have any windows machines, so I cannot test it.


Shell surface view interfaces

Instead of using events for the shell surface requests and notifications to compositors, it might be a little cleaner to explicitly define interfaces for these things.
I don't think we can unify them all with a single view interface (like Weston does) because they are just different enough for this to cause problems. I also don't think it would be appropriate to have interfaces for communication from the compositor to the surface. The idea of a "desktop_surface" abstraction is not appealing to me because it removes some fine grain control over the behavior of surfaces and adds a lot of weird edge cases when the shell interfaces don't line up. This is likely to cause even more problems in the future because shells will come and go and our assumption about what a shell surface does is likely to change.
For instance, the xwayland view interface might look something like this:
struct wlr_xwayland_view_interface {
    void (*configure)(struct wlr_xwayland_surface *surface, int x, int y, int width, int height);
    void (*request_resize)(struct wlr_xwayland_surface *surface, uint32_t edges);
    void (*set_title)(struct wlr_xwayland_surface *surface, char *title);
    // and so on...
};

The compositor defines an implementation of this interface and then sets it on the shell surface when it is notified of its creation. Instead of emitting events, the shell surface calls the methods of the compositor implementation if they are present. The compositor responds to these calls in the same way it responds in the event handlers presently.
Advantages:

Interfaces are easier to document
Clearer what the compositor's responsibility is when implementing their views
Return values from interface methods, possibly for denying requests like maximization or state configuration
Get rid of all the event types for these things resulting in less types and cleaner code

Disadvantages:

Only one handler is allowed for view event

What do you guys think?

Only one handler is allowed for view event

I think this is a pretty substantial disadvantage. Many places within wlroots use listeners internally to communicate across the system.

I think this is a pretty substantial disadvantage. Many places within wlroots use listeners internally to communicate across the system.

The system was purposefully designed so the shells are leaves in the dependency tree. No shell-specific code should ever be in any other part of wlroots. They are written as if they are separate libraries and I think it should stay that way.
Anything in the shell that may legitimately need a signal should not be in this interface For instance, the destroy signal.

Well, many components of wlroots aside from the shells are meant to talk to each other. For consistency we should design them similarly unless it's well justified.

It's just a minor style thing so not a big deal if you don't like it. Let's wait to see if it is needed #343.
However, please don't write code deep in wlroots that responds to X11 protocol requests.


Added feature of showing enchantability in tooltip

Hey!
Great idea! Thanks for the PR, I'll merge it in as a new feature in 1.18 😃


SceneForm 1.14

/azp run

This looks good to me - are we waiting on something to get it in? @moljac / @redth?

/azp run

/azp run


Added nav-item potato on tools page

Added missing nav-item to potato page from the tools page

Nice! :)


mAP for IR?

Hi,
is the matchzoo mean average precision really the "real" mAP" for multiple queries? It looks for my like the calculation of just the average precision.
My problem is that I want to evaluate on multiple queries. For each query I have just 1 or two "right" documents out of lets say 500. So I have to have for each query to compare with each of the 500 docs. So I have a text match with one doc and the rest of the docs should be 0.
So this blows up the test data,when you have multiple test queries and need to augment every one with all the rest documents.
Is this the only way to evaluate my IR text Matching since I need to compare each query with all documents in the pool, right?
I would know howto calculate the mAP in numpy, but not to that in tensorflow? I seems there is just the average precision metric but not the mAP.
I  also don't know how to use it because training in tensorflow is really different than testing it in the above way like I want.
Any ideas or help?

Right, you cannot simply use model.evaluate, but you may use the trained model to predict the score for each query-doc pair you have for test, and build your re-ranked list for each query.
Then you may write a MAP metric function by yourself (There are a lot of implementation to copy on github), or just output the ranked list to use TREC evaluation tools.


[BUG] SyntaxError: Expected ',' or '}' after property value when string field has a integer-like source

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to source code editor
Paste

- from:
    uri: slack:test
    parameters:
      maxResults: 1
    steps: []

Click Sync code
--> nothing happens in Canvas

Expected behavior
A clear and concise description of what you expected to happen.
Logs
in console log:
SyntaxError: Expected ',' or '}' after property value in JSON at position 8351
overrideMethod @ VM510 react_devtools_backend.js:2655

The request which seems to be incorrect is:
http://kaoto-kaoto.apps.smaug.na.operate-first.cloud/api/v1/integrations

with payload:
- from:
    uri: slack:test
    parameters:
      maxResults: 1
    steps: []

returns response like:
{
	"steps": [
		{
			"name": "slack",
			"type": "START",
			"id": "slack-consumer",
			"kind": "Camel-Connector",
			"icon": "data:image/svg+xml;base64,XXXXXXX\n",
			"title": "Slack",
			"description": "Send and receive messages Error processing deployment: java.lang.Integer cannot be cast to java.lang.String (through reference chain: io.kaoto.backend.api.resource.v1.model.Integration["steps"]->java.util.ImmutableCollections$ListN[0]->io.kaoto.backend.model.step.Step["parameters"]->java.util.LinkedList[8]->io.kaoto.backend.model.parameter.StringParameter["value"])

it is coming from example from Red Hatter Hugo Guerrero https://github.com/hguerrero/serverless-examples/blob/4233bbc1168eecdc70f2f967ba80adc67b4b070a/knative-slack-demo/camel-source/camel-slack-source.yaml#L24

maxResults is a string: https://camel.apache.org/components/3.20.x/slack-component.html#_endpoint_query_option_maxResults
If we use
- from:
    uri: slack:test
    parameters:
      maxResults: '1'
    steps: []

(note the quotes) it works.
Anyway, it shouldn't explode.

in yaml, strings can be written with or without quotes. I have not found in the specifications exceptions for string that would be only numbers.

Yes, I don't know why it is not understanding it. It is weird.


Print all open ports

Requirement - what kind of business use case are you trying to solve?
Every binary should print all open ports on startup. This makes it easy to find the port and connect. The log message should contain info about the port - e.g. what it is used for.

For example collector does not say anything about opening 14267 for jaeger agent.
SPAN_STORAGE_TYPE=elasticsearch go run ./cmd/collector/main.go                                                                                 4:04
{"level":"info","ts":1537365891.8120868,"caller":"healthcheck/handler.go:99","msg":"Health Check server started","http-port":14269,"status":"unavailable"}
{"level":"info","ts":1537365894.9461088,"caller":"static/strategy_store.go:77","msg":"No sampling strategies provided, using defaults"}
{"level":"info","ts":1537365894.9463255,"caller":"collector/main.go:141","msg":"Registering metrics handler with HTTP server","route":"/metrics"}
{"level":"info","ts":1537365894.9463742,"caller":"collector/main.go:149","msg":"Starting HTTP server","http-port":14268}
{"level":"info","ts":1537365894.946397,"caller":"healthcheck/handler.go:133","msg":"Health Check state change","status":"ready"}

+1


docs: add PostgreSQL read and query samples for SpannerIO

Adds PostgreSQL-specific samples for read and query operations using SpannerIO:

Cloud Spanner GoogleSQL column names use camel case.
Cloud Spanner PostgreSQL column names all lowercase with underscores to separate words. This because PostgreSQL automatically folds identifiers to lowercase.

All Pipeline variables that were named p are renamed to pipeline for better readability in the documentation that imports the code samples here: https://cloud.google.com/spanner/docs/dataflow-connector#read-without
Fixes b/232995390

@Shabirmean Would you have time to review this PR? These samples are required to update the docs at https://cloud.google.com/spanner/docs/dataflow-connector


Error in get_props() with lighttpd

Observations
When using lighttpd webdav module as a webdav server.
When using get_props on a resource (directory or file).
The following exception is raised:
    223 def get_response_for_path(self, hostname: str, path: str) -> Response:
    224     """Provides response for the resource with the specific href/path.
    225
    226     Args:
   (...)
    231             resource.
    232     """
--> 233     return self.responses[join_url_path(hostname, path)]

KeyError: 'path/to/my/file'

This is because propfind returns an empty content (the XML is valid, but with no data).
In[0]: client.propfind("./").content
Out[0]: '<?xml version="1.0" encoding="utf-8"?>\n<D:multistatus xmlns:D="DAV:" xmlns:ns0="urn:uuid:c2f41010-65b3-11d1-a29f-00aa00c14882/">\n</D:multistatus>\n'

Solution
After several tests I found out that propfind works well if the correct Depth header.

Depth: 1 as used by ls is listing files correctly.
Depth: 0 allows to get properties of a single filepath. Example below:

In[0]: client.propfind("path/to/file", headers={"Depth": "0"}).content
Out[0]: '<?xml version="1.0" encoding="utf-8"?>\n<D:multistatus xmlns:D="DAV:" xmlns:ns0="urn:uuid:c2f41010-65b3-11d1-a29f-00aa00c14882/">\n<D:response>\n<D:href>http://webdav-url/path/to/file</D:href>\n<D:propstat>\n<D:prop>\n<D:creationdate ns0:dt="dateTime.tz">2024-07-11T09:33:29Z</D:creationdate><D:getcontentlanguage>en</D:getcontentlanguage><D:getcontentlength>508</D:getcontentlength><D:getlastmodified ns0:dt="dateTime.rfc1123">Thu, 11 Jul 2024 09:33:29 GMT</D:getlastmodified></D:prop>\n<D:status>HTTP/1.1 200 OK</D:status>\n</D:propstat>\n</D:response>\n</D:multistatus>\n'

As get_props is meant to read properties of a single filepath, it seems relevant to add Depth: 0 to the headers in this method.
Here is a fix which is working for me:
    def get_props(
        self,
        path: str,
        name: Optional[str] = None,
        namespace: Optional[str] = None,
        data: Optional[str] = None,
    ) -> "DAVProperties":
        """Returns properties of a resource by doing a propfind request.

        Can also selectively request the properties by passing name or data.
        """
        data = data or prepare_propfind_request_data(name, namespace)
        headers = {"Content-Type": "application/xml"} if data else {}
        headers["Depth"] = "0"  # <-- This line enforce the Depth header
        result = self.propfind(path, headers=headers, data=data)
        response = result.get_response_for_path(self.base_url.path, path)
        return response.properties

But I am not sure of the all the consequences, maybe I’m missing the big picture. I did not test this with another webdav server like Nextcloud.
What do you think?

The WebDAV RFC was published June 2007 (which is 17 years ago)
https://www.rfc-editor.org/rfc/rfc4918#section-9.1

A client MUST submit a Depth header with a value of "0", "1", or
"infinity" with a PROPFIND request.  Servers MUST support "0" and "1"
depth requests on WebDAV-compliant resources and SHOULD support
"infinity" requests.  In practice, support for infinite-depth
requests MAY be disabled, due to the performance and security
concerns associated with this behavior.  Servers SHOULD treat a
request without a Depth header as if a "Depth: infinity" header was
included.

get_props() should most definitely be updated to send an appropriate Depth header.

Note: not all servers support Depth: infinity.
lighttpd supports Depth: infinity, but disables PROPFIND Depth: infinity by default, unless explicitly configured in lighttpd.conf: webdav.opts += ("propfind-depth-infinity" => "enable")
Therefore, if you send Depth: infinity, you should be prepared to fall back to multiple PROPFIND requests with Depth: 1


SlideNext instead SlidePrev on click button Next or preview

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 4.4.1

Platform/Target and Browser Versions: Windows,CHROME 69.0.3497.92

Live Link or JSFiddle/Codepen or website with isssue:
http://idangero.us/swiper/demos/

Actual Behavior
On click right (left) arrow sometimes you go to previous (next) slide, not to next. Same problem on all projects only with swiper

This is still an issue, I think it is related on how many items you are displaying/pages

I still encountered it too. Does anyone have a solution to resolve it?


0001-0500/0022. Where to use target?

0001-0500/0022. Generate Parentheses.swift
target unused.

Yes, you're right.
In the new corrected version:

Removed the target parameter.


Hook result data shouldn't have sequelize info

When using hooks on sequelize services hook.result has a lot of extra properties on it then just the data. This makes modifying the result data confusing.

This is the expected result of using an ORM.  That data can be really useful in other scenarios, so it's generally better to use Sequelize's methods to convert the model instance to an object.  See here for ways of doing this:
http://stackoverflow.com/questions/21961818/sequelize-convert-entity-to-plain-object

It's probably worth noting that you would do this in an after hook.

Hm, I can see what you're saying. I was just thinking it's inconsistent with the regular service hooks. In some cases you're doing hook.result.myProp = 'myProp'; in others you're doing hook.result.dataValues.myProp = 'myProp';, or even worse for an array (I think it's something like) hook.result.dataValues[0].dataValues.myProp = 'myprop'.
At some point those dataValues are converted to the format that you would normally expect. What if it happened earlier in the stack? Right before the after hooks. If we really want that ORM information for other purposes then why not put it in another property on the hook?

@RickEyre for now we leave that up to you because it's not one size fits all. The Sequelize models that get returned have some model properties that come in handy in certain instances, like @marshallswain said. If we are to pluck them from the model and put them as other attributes on hook.result would remove the context, especially when dealing with multiple models that are returned, as is the case with  find queries or batch creates, patches, and removes.
With that said, we have bundled hooks with some of the modules and have proposed in #18 and #19 to have similar behavior to the feathers-mongoose adapter, so that you have a hook that will call toJSON for you.

@RickEyre I just generated a new app and took a closer look. I get what you are saying but my previous statement still stands. I think #18 and #19 should solve the problem quite nicely, if you want your results converted to regular objects right away, as opposed to having sequelize models returned.
I'm going to close this as we can track progress in those 2 issues.

Sounds good! Thanks for pointing me in the direction of those issues.

@RickEyre if you want to implement the hook yourself in the mean time, it's pretty much exactly the same as this one: https://github.com/feathersjs/feathers-mongoose/blob/master/src/hooks.js. PR's are even better! :smile:

@ekryski Yeah, I was thinking of taking a crack at it :smile: I'll see if I can find some time in the next few days. Doesn't look that difficult!

@RickEyre @ekryski would using the sequelize model instance's built in toJSON function do the job? Working well for me, though haven't tested thoroughly.

Sequelize (unfortunately) wraps results by default in DAO's. You can disable them everywhere with {raw: true}.
About to go through my project and do that everywhere, in fact.
Not that feathers-sequelize should necessarily be opinionated about it, but there are noted performance benefits for disabling DAO's, particularly with complex queries, or queries that return large data sets.

@petermikitsh @bedeoverend we have been talking about doing a breaking change to both feathers-sequelize and feathers-mongoose so that services are initialized with {raw: true} and {lean: true} respectively. @daffl saw some pretty massive performance improvements with using lean vs. without.

I support that effort. Note that setting {raw: true} won't turn off DAO's for all Sequelize API calls. See my writeup at https://github.com/sequelize/sequelize/issues/6501 for more details regarding the Sequelize Model.create api. It's a bit of a gotcha. 😕


Uanble to launch pinokio on windows

"I ran the EXE file and tried to install Pinocchio, but all I see is a blank, white screen."
Pinocchio - 2.1.14
Windows - Windows 11

same here
after installation the process ends without even showing any windows.
tried to launch the app same issue

the installation dir must be english

Same issue here. Windows 11 23H2, Version 26100.

Please use Pinocchio version 2.0.4 to fix the issue . Mine is working fine

Same problem:
Ereignisname: APPCRASH
Antwort: Nicht verf�gbar
CAB-Datei-ID: 0

Problemsignatur:
P1: Pinokio.exe
P2: <IP_ADDRESS>
P3: 64dcda3e
P4: StackHash_1b57
P5: 10.0.26100.1301
P6: ede24b56
P7: c0000374
P8: PCH_FF_FROM_ntdll+0x000000000015F5D4
P9:
P10:

I would suggest follow below steps .
Uninstall previous version: Remove any existing software from your Windows system, specifically the previous installation of [Pinokio].
Download and install Pinokio version 2.0.4: Download the attached updated software package
Run Pinokio once again: Once the installation is finished, run the Pinokio application.
Wait for initialization: Allow the Pinokio to initialize and start up properly, which may take 10-15 minutes.

Done, Pinokio starts, but crashes after a few seconds.

Yeah, i have the same issue. Im annoyed.


clang_tidy:torch_dipu/csrc_dipu/profiler/CorrelationIDManager.cpp

torch_dipu/csrc_dipu/profiler/CorrelationIDManager.cpp
torch_dipu/csrc_dipu/profiler/CorrelationIDManager.h

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Using 'type' as key in the scheme attributes will crash PHP

HI,
using 'type' as key in the scheme attributes will crash PHP.
public function getAttributes($resource)
{
    /* @var ProductGroupAttributeSetAttributeDTO $resource */
    return [
        'type'         => 'ddd',
    ];
}

Took me forever to find, but it turns out using 'type' is not possible. It will crash php fpm resulting in an internal server error. I don't think it is forbidden by the spec...
PHP 7.1.4-1+deb.sury.org~xenial+1 (cli) (built: Apr 11 2017 22:12:32) ( NTS )
Copyright (c) 1997-2017 The PHP Group
Zend Engine v3.1.0, Copyright (c) 1998-2017 Zend Technologies
    with Zend OPcache v7.1.4-1+deb.sury.org~xenial+1, Copyright (c) 1999-2017, by Zend Technologies

Fields for a resource object MUST share a common namespace with each other and with type and id. In other words, a resource can not have an attribute and relationship with the same name, nor can it have an attribute or relationship named type or id.

http://jsonapi.org/format/#document-resource-object-fields

Ok Thanks.
But why does it crash and not throw an exception?

That is strange, it throws an Exception here:
https://github.com/neomerx/json-api/blob/master/src/Document/Presenters/ElementPresenter.php#L299-L304
Not sure why that would be causing a crash.

I'll see if it gets there

Ok my mistake - it does throw an exception... Thanks


Add tutorial to Video Viewer

Added and modified all the files to let the tutorial start and be localized

Nice.
Few remarks:

Could you add a popup on the video item to explain to the user that he should click to launch the video?
Your localization don't work because you didn't initialize the language with the Sugarizer language (see here)
You could add [fr], [es] and [pt] in the locale.ini, just use the same value than then one in [en].

Very cool.
Could you also add a popup on the input field to explain that it allow you to filter by name?

Thank you!
Of course, I'll add this popup in a moment.

Could you adapt media queries to avoid to override the search field when the window is resized.


[kn-plugin-workflow] Minify the openAPI spec files to trim components/schemas

This issue is a continuation of the work on minifying OpenAPI specifications files.
The goal is to remove all unused components and schemas from the OpenAPI spec file.

https://github.com/apache/incubator-kie-tools/issues/2601


typo

"To transfer ownership of a resource, ou can either:"

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

ID: 407d2781-1fe1-38a6-dc18-37d4e37d651c
Version Independent ID: 919a2206-13f9-4f51-2e4f-45fe062d7d43
Content: How to use authoring and runtime keys - LUIS - Azure Cognitive Services
Content Source: articles/cognitive-services/LUIS/luis-how-to-azure-subscription.md
Service: cognitive-services
Sub-service: language-understanding
GitHub Login: @diberry
Microsoft Alias: diberry

@dargilco Thanks for the feedback. We will update you shortly.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Hsm key rotation

FYI - @DerekGn

@djfr FYI
@DerekGn few tweaks+test layer but the core is as before...


🐞 [Bug]: Deployment is stuck at validating Nodes on the Dashboard, and the old deployment lists are not visible.

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?
Mat is unable to connect to a VM on Node ID 687 due to the deployed list is not visible on his dashboard which he deployed 6 months back. Also, check the specific issue for this showing logs..  This issue follows a previous issue where deployments were also not displayed.
He attempted to deploy a new VM on Node ID 687, but the process is stuck on node validation.
A testing video is available for reference from our side where In contrast Scott , Sherwin, Shang, and I successfully deployed on Node ID 687 without any issues, and the process went smoothly.
Also, here are the errors on the client side on his browser console while trying to deploy on his Node : 687

We tested the same between Node 687 & Node 1, you can see the difference in errors

Steps To Reproduce

Open Dashboard

After you login click on Side menu, and click on Deploy and then choose Virtual machines

Choose any one of the VM

Select Nodes manually and then choose the Node 687

Then click on deploy.

which network/s did you face the problem on?
Main
version
NA
Twin ID/s
NA
Node ID/s
687
Farm ID/s
209
Contract ID/s
NA
Relevant screenshots/screen records
Here's the screen record on Client side while deploying VM on the Node 687 : https://drive.google.com/file/d/1rcWeU59E6SXyKpFc88SdB9Fmnd1om31u/view?usp=sharing
Relevant log output
NA

Update from the farmer: https://t.me/threefoldtesting/15012
He is now unable to deploy a VM on any node from his location: Poland.

Update from Farmer on Crisp: The user deleted the VM he could get access to and created a new one but is still having issues getting access, and this happens only when he tries to load nodes from Poland (his country) all other countries work well.

The user reports that they are able to create deployments after creating a new account.
I suspect that the issue may be with the RMB public key set on tfchain for the original account.

I suspect that the issue may be with the RMB public key set on tfchain for the original account.

sound yes. but the dashboard auto-updates the public key once login if it's incorrect. so what is the twin ID of the old account?

Right, I would have expected the public key to get updated too, especially since the user logged in again using incognito tabs and also tried multiple browsers.
The twin id is 737.

could this cache issue as explained here https://github.com/threefoldtech/rmb-rs/issues/200#issuecomment-2435549736


doc: first cut at accessiblity section

Signed-off-by: Michael Dawson<EMAIL_ADDRESS>

There's room to talk about making software libraries more accessible to their consumers as well, considering that neurodiversity issues are also included in a11y concerns.
For example, JavaScript library authors should include TypeScript declaration files with good inline documentation (JSDoc blocks). These are very helpful at reducing cognitive load: it's way easier to just read the API docs in your editor than go code spelunking on github.com, and for users with ADHD to pick one example of many, opening another browser tab may come with a significant cognitive load cost.


[sw, tests] Create directory for Mask ROM unit/sanity tests

This change simply creates a directory with an empty meson.build
file. Mask ROM unittests should reside in this directory.

Any reason for this to be its own PR? To me it seems more natural to include this with the first round of unit/sanity tests rather than on it's own, as this PR alone is effectively a no-op.

Any reason for this to be its own PR? To me it seems more natural to include this with the first round of unit/sanity tests rather than on it's own, as this PR alone is effectively a no-op.

Hey, well, the short answer is - no specific reason. I am about to start messing around with proof of concept implementation for several different changes. I don't have any strong feeling about this, happy to drop it (already discussed it with @alphan ). I think this change on its own also makes sense, but really I don't think it deserves discussing too much. So I am happy to merge it or drop it.

Please see #3451 .


Snippet placeholder redesign

Overview
Continued a little of the refactoring from replacing the snippet placeholder, so that the snippet placeholder component is now used inside a snippet component. This means that I could re-use the insert logic from the snippet buttons and if we wanted in the future we could include buttons in the same style as for inserted snippets. Since I was not including buttons for now, I was able to use @elpoelma 's redesign implementation work almost untouched.
For consistency, I move the snippet list ids information to RDFa for inserted snippet nodes.
connected issues and PRs:
Builds on https://github.com/lblod/ember-rdfa-editor-lblod-plugins/pull/495
Jira ticket: https://binnenland.atlassian.net/browse/GN-5117
Setup
N/A
How to test/reproduce
Insert, replace, delete snippets and snippet placeholders with different configurations and make sure everything works as expected. The expected interactivity is different depending on which sample page:

Besluit: It's only possible to insert snippets using the in-node insert button. placeholders cannot be inserted
Reg statement: The in-node insert snippet button is hidden, but the sidebar button remains.
Snippet edit: The in-node snippet insert button is shown, the sidebar button is not there. Placeholders can be inserted.

Challenges/uncertainties
N/A
Checks PR readiness

[ ] UI: works on smaller screen sizes
[ ] UI: feedback for any loading/error states
[x] Check if dummy app is correctly updated
[x] Check cancel/go-back flows
[x] changelog
[x] npm lint
[x] no new deprecations

@elpoelma are you looking at these comments or should I?

@elpoelma are you looking at these comments or should I?

You can do it if you want :) I am currently working on an editor PR which fixes the specificity css styling regression which should remove the need of the !important modifier.


hook fails to load in sails 0.11

error: A hook (`userconfig`) failed to load!
error: Error: Invalid module:[object Object],[object Object],[object Object]

Looks like this was an issue with adding array-based fixtures under config/. Fixed in hashware/api


Add soft deletable forms

References https://bluesquare.atlassian.net/browse/IA-452?atlOrigin=eyJpIjoiYTY4ODQ3NmQ5NjQyNGMwMWFmYTBkODVlYjZjZjIyMTUiLCJwIjoiaiJ9

Closing this branch, since there was a bad merge by mistake replaced by https://github.com/BLSQ/iaso/pull/107