Refactor api

Sorry for the large PR. Here are what it inlcudes:

Move oauth, fields, expansions to their dedicated py files
Create UserExpansions and TweetExpansions classes
Adopt the new fields and expansions model in all endpoints

In my view, the code is much clear now and it will be easier to add new endpoints for the future.

This looks awesome! I will review it when I get a chance. FYI, I am just going to need some time and probably won't have it until the weekend.

This looks awesome! I will review it when I get a chance. FYI, I am just going to need some time and probably won't have it until the weekend.

Take your time. I won't add any new functions/classes any time soon. Just need to work on the documentation a little big.

Take your time. I won't add any new functions/classes any time soon. Just need to work on the documentation a little big.

Just to want to check in and make sure that this is still on your plate @mr-devs . I'm hoping to bump the pip version after we merge this PR and do some advertisement on social media. But no rush.

Just to want to check in and make sure that this is still on your plate @mr-devs . I'm hoping to bump the pip version after we merge this PR and do some advertisement on social media. But no rush.

I hear you. Yes, this is on my to-do list. I was planning on getting to it Saturday. Sorry, the vaccines stuff has really really snowballed beyond what I intended to get involved with and has set me back on a bunch of other stuff that I am not trying to catch up on.

I hear you. Yes, this is on my to-do list. I was planning on getting to it Saturday. Sorry, the vaccines stuff has really really snowballed beyond what I intended to get involved with and has set me back on a bunch of other stuff that I am not trying to catch up on.

Don't worry and no rush.

Don't worry and no rush.


Next.js

@the-ethan-hunt I would like to work on it.

Assigned to @sourabh1031

@the-ethan-hunt  i mean to say this --> https://nextjs.org/learn/

By the way thanks @the-ethan-hunt  i came to know one more new thing from you


Create Posts “20220918-execute-os-command-in-springboot”

Automatically generated by Netlify CMS

👷 Deploy Preview for monetaboot processing.

Name
Link

🔨 Latest commit
de744ef06c33e13ff5861713e3242422e08202cb

🔍 Latest deploy log
https://app.netlify.com/sites/monetaboot/deploys/6327588eee5c4200095f480d


Deployed but cannot run workflow and tests webhook from DO app.

Hi John,
I came across your video you posted a week ago, and tried deploying n8n on digital ocean. While it is deployed, i came across an issue that I cant test webhooks and run request on the workflow. Upon seeing the log, I am seeing error message like this.
[n8n-app-do] [2022-05-17 19:40:07] ERROR RESPONSE
[n8n-app-do] [2022-05-17 19:40:07] ResponseError: Execution not found
[n8n-app-do] [2022-05-17 19:40:07]     at /workspace/node_modules/n8n/dist/src/Server.js:1510:23
[n8n-app-do] [2022-05-17 19:40:07]     at processTicksAndRejections (node:internal/process/task_queues:96:5)
[n8n-app-do] [2022-05-17 19:40:07]     at async /workspace/node_modules/n8n/dist/src/ResponseHelper.js:91:26 {
[n8n-app-do] [2022-05-17 19:40:07]   httpStatusCode: 404
[n8n-app-do] [2022-05-17 19:40:07] }
[n8n-app-do] [2022-05-17 19:40:07] Attempt to read execution was blocked due to insufficient permissions

Do you know what could be the issue ? I am guessing its due to the insufficient permissions
Thanks in advance

Hey, can we jump on a Zoom call so I can troubleshoot the issue?
https://calendly.com/funnelpress/1-on-1-meeting

With pleasure. I will create an invitation shortly. If you dont mind, I am inviting a friend of mine to the call who is also interested on the topic.
Best,

No problem, Lets get it figured out!


"Person is typing..." message appear to be off-center in that bottom space

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Login to app with account A
From account B on another device/browser, start typing

Expected Result:
"Person is typing..." message should be centered in the bottom space and vertically aligned
Actual Result:
"Person is typing..." message appear to be off-center in that bottom space
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:

Where is this issue occurring?

Web
Desktop App

Version Number: 1.1.63-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @rafecolton
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1652905715417509
View all open jobs on GitHub

Proposal
We have to remove the styling on line 59
https://github.com/Expensify/App/blob/cc912ffac883d59f1ea1ca7a4173ed565bdf3f61/src/pages/home/report/ReportTypingIndicator.js#L55-L61

Maybe a regression from https://github.com/Expensify/App/pull/8691 cc: @luacmartins

Yea, I think so. Here's the fix. Thanks for the ping @parasharrajat!


blink.py doesn't work on Pico W

The onboard LED is not tied to GPIO 25 on the new Pico W so the example code in blink.py doesn't work.
Replacing the initialisation with:
led = Pin("LED", Pin.OUT)
..works using an alias and is portable (for recent firmwares) between Pico and Pico W

See section 3.4 :  https://datasheets.raspberrypi.com/picow/connecting-to-the-internet-with-pico-w.pdf
led = machine.Pin("LED", machine.Pin.OUT)


Add SHA1, SHA256 package hashes

It is relatively common to distribute archives with a majority of common hash types. Let's include sha1 and sha256 in addition to the existing algorithms.

Added sha1 package hash
Added sha256 package hash
Perform streamed hashing of packages to improve performance with larger archives

Codecov Report

Merging #11 (a02bc5f) into main (bf0350e) will decrease coverage by 0.09%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
- Coverage   71.08%   70.98%   -0.10%
==========================================
  Files          22       22
  Lines        3154     3157       +3
==========================================
- Hits         2242     2241       -1
- Misses        912      916       +4

Impacted Files
Coverage Δ

autobuild/autobuild_tool_edit.py
67.45% <ø> (ø)

autobuild/hash_algorithms.py
81.25% <75.00%> (-3.37%)
:arrow_down:

autobuild/autobuild_tool_package.py
67.09% <85.71%> (-0.15%)
:arrow_down:

autobuild/common.py
73.30% <94.11%> (-1.01%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


An issue where map is not displayed properly on other surfaces

Because the map object uses the game's core as a reference for width and height, it creates display problems for surfaces larger than the game's core. This causes odd scaling behaviour, where the map's display will not be scaled, but the dimensions of the tiles will be (so at 3x scale, a collision tile that was at (32,32) will collide at (96,96) appropriately). As far as I can see, the map object is the only object that does not behave appropriately to scaling.

I'm working on a Wii U game at the moment that leverages enchantJS and specifically the Map, as it provides the best performance on the system for my needs. However, this issue is impacting my release and I'd like to work on fixing it, but could use some direction before digging through the Map code.
Essentially, when the scene is scaled, I'd like the Map to scale accordingly, like Sprites, Labels, and other enchant objects behavior (which seems to be the nature of the reported issue above). Does anyone have a good starting point to jump in here?

So, last night I dug into this and made modifications to the redraw function to respect the scale value of either the map or its parentNode (with the map's scale value taking precedence). Based on my personal use case, it's working as intended now; however, I have no idea if my solution is appropriate to commit as I'm not really sure if the approach aligns with the rest of the project, nor do I know if it behaves appropriately in all use cases.
Any suggestions on next steps? I'd love to help out any fellow enchant users who are experiencing the same problem, but I definitely need the guidance of the official team before I do anything.


Python wrapper for computing stripes?

Hello!
Is it possible to use potpouri3d for computing stripes (using computeStripePattern)?
Thank you!
cheers,
roberto

There is no binding for this currently, I'd be happy to take a PR for one!


Backport of https://github.com/knative-sandbox/control-protocol/commi…

What this PR does / why we need it:

Fixes an issue with upgrade-tests https://github.com/openshift-knative/serverless-operator/pull/2175
Backports https://github.com/knative-sandbox/control-protocol/commit/06411c45a53ad168e53e3b6ba079661bc042c019

Does this PR needs for other branches:
No, the fix is already in 1.11.
Does this PR (patch) needs to update/drop in the future?:
It is only for 1.10 in 1.10 branch.

/assign @skonto

/lgtm
/approve


its so over

over....

Sorry but this is not something I consent to being done with my voice recordings, in the future you should ask voice actors if this is something they're okay with being done with their work. You could always request me to record lines in the Mercenary voice!


Update App Broker to Spring Boot 3.0

Would there be a need to update the Spring Cloud Open Service broker as well?

@fombico yes, it will require an updated release.

App Broker 2.0.0-SNAPSHOT builds with Boot 3 are being produced now, but we might want to keep this open until we have a first GA release build.


Script to copy settings from VSC

Copy settings from VSCode to Melty:
bash ./scripts/copy_settings.sh

Thanks for sharing this! I really appreciate the time and effort you put into creating such informative and engaging content. Jack Depp It's always a pleasure to read and learn something new. Keep up the great work and looking forward to your next post!


[Broker] Make PersistentTopicsBase#internalExamineMessage async

Master Issue: #14013
Motivation
See #14013
Modifications

Make PersistentTopicsBase#internalExamineMessage async

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Documentation

[x] no-need-doc

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

These changes of codes have been moved to https://github.com/apache/pulsar/pull/16226. Close this one.


try codeql

This seems to technically work now albeit reporting 0 problems -- but that is good! :) Let's merge and see how it fares.


WFLY-4556 The provided Lucene modules should also include lucene-querypa...

...rser
As suggested by @alesj :
https://issues.jboss.org/browse/WFLY-4556

Linux Build 6358 is now running using a merge of 19487c18caf68454fa4a53fac897b4603a5419a8

Linux with security manager Build 1286 is now running using a merge of 19487c18caf68454fa4a53fac897b4603a5419a8

Windows Build 1438 is now running using a merge of 19487c18caf68454fa4a53fac897b4603a5419a8

Linux with security manager Build 1286 outcome was SUCCESS using a merge of 19487c18caf68454fa4a53fac897b4603a5419a8
Summary: Tests passed: 824, ignored: 356 Build time: 0:05:00

Windows Build 1438 outcome was SUCCESS using a merge of 19487c18caf68454fa4a53fac897b4603a5419a8
Summary: Tests passed: 2857, ignored: 412 Build time: 0:47:27

Linux Build 6358 outcome was FAILURE using a merge of 19487c18caf68454fa4a53fac897b4603a5419a8
Summary: Tests failed: 1 (1 new), passed: 2855, ignored: 412, muted: 1 Build time: 0:49:59
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.jca.datasource.remove.EjbAppDsRemoveTestCase.testDatasourceRemove: <no details avaliable>

@bstansberry Approved.

retest this please

Windows Build 1440 is now running using a merge of 19487c18caf68454fa4a53fac897b4603a5419a8

Linux with security manager Build 1288 is now running using a merge of 19487c18caf68454fa4a53fac897b4603a5419a8

Linux Build 6360 is now running using a merge of 19487c18caf68454fa4a53fac897b4603a5419a8

Linux with security manager Build 1288 outcome was SUCCESS using a merge of 19487c18caf68454fa4a53fac897b4603a5419a8
Summary: Tests passed: 824, ignored: 356 Build time: 0:05:08

Linux Build 6360 outcome was SUCCESS using a merge of 19487c18caf68454fa4a53fac897b4603a5419a8
Summary: Tests passed: 2856, ignored: 412, muted: 1 Build time: 0:46:39

Windows Build 1440 outcome was SUCCESS using a merge of 19487c18caf68454fa4a53fac897b4603a5419a8
Summary: Tests passed: 2857, ignored: 412 Build time: 0:47:34


New OSM-Bright is slow in rendering

I've pulled the new version and imported a new OSM set with the new imposm mapping and now it is incredibly slow in rendering. I've also updated TileMill (0.9.0) so I'm not sure if it is tilemill being slow, or the new queries of OSM-Bright.
Anyone noticed a performance decrease with the advanced tunnel/intersection rendering?

yes i noticed that too ... rendering time is more than 2 x longer as with the older version.

I've experienced it as well and I don't think it's necessarily attributed to the newer versions of osm-bright. Most likely, it's probably these issues:
https://github.com/mapnik/mapnik/issues/1027
https://github.com/mapnik/mapnik/issues/1028
https://github.com/mapnik/mapnik/issues/1055
Also see: http://support.mapbox.com/discussions/tilemill/655-slow-rendering-speed


Purview - "Next steps" links pointing to each other

The 'Next steps' link at the bottom of the Quickstart page links to the Catalog Permissions page. 👍
However, after advancing to the Catalog Permissions page, the current page's 'Next steps' seems to link back to where we just came from - the Quickstart page.

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

ID: 378c23c2-bdbe-47c1-0ea9-805c61670fbd
Version Independent ID: fa0832ca-6396-1fc8-d0b1-f091f9a1ece2
Content: Catalog Permissions (preview) - Azure Purview
Content Source: articles/purview/catalog-permissions.md
Service: purview
Sub-service: purview-data-catalog
GitHub Login: @yarong
Microsoft Alias: yarong

@helizabethwhite - Thanks for sharing valuable feedback. We are looking into it and will update you shortly.

@helizabethwhite - Thanks for sharing valuable feedback. We are looking into it and will update you shortly.

@helizabethwhite - Yes, I can see that we are forming a circular loop in these documents. As per my understanding, creating an purview account can be a additional information in the catalog permissions document and next steps can be to derive data insights from the purview. I have addressed this in the PR - which needs review from the content author. Once approved, you will be able to see the changes. Thanks for your valuable contribution!

@helizabethwhite - Yes, I can see that we are forming a circular loop in these documents. As per my understanding, creating an purview account can be a additional information in the catalog permissions document and next steps can be to derive data insights from the purview. I have addressed this in the PR - which needs review from the content author. Once approved, you will be able to see the changes. Thanks for your valuable contribution!

@yarong - Can you please look at the PR and approve if appropriate.

@yarong - Can you please look at the PR and approve if appropriate.

@HarithaMaddi-MSFT FYI, i left a small spelling comment on the PR you linked! 🙂

@HarithaMaddi-MSFT FYI, i left a small spelling comment on the PR you linked! 🙂

@helizabethwhite - Thanks for the review. As suggested by author, raised PR  in private repo and merged the changes. I will proceed with closing this issue now, please let us know for any further queries.

@helizabethwhite - Thanks for the review. As suggested by author, raised PR  in private repo and merged the changes. I will proceed with closing this issue now, please let us know for any further queries.


docker erizo.js  No such file

**error
cp: cannot stat '/opt/licode/erizo_controller/erizoClient/dist/erizo.js': No such file or directory

please provide more info, do you still see this error with the latest builds?


Fastnlight: Watch bestgamingsite.online

Fastnlight requests the watch of the watch_keyword bestgamingsite\.online. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

bestgamingsite\.online has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Take advantage of mmtune-speedup in 0.10.1

@cjo20 can you review this please when you get a chance?

Actually, it looks like the latest release of mmeowlink is 11.0, which will need a change in the egrep code

the mmtune speed up is nice, and working for me, it does go much faster so :+1:

Fixed the version check; merging.


cant compile minimal example on OS X 10.11 & Debian 8

pwd
~/velo

git clone https://github.com/arangodb/velocypack

cd velocypack/build
cmake -DCMAKE_BUILD_TYPE=Release -DBuildExamples=ON -DBuildTests=ON -DBuildTools=ON .. && make -j8  DESTDIR=~/velo/ install

ls
main.cpp	usr		velocypack

ls usr/local/*
usr/local/bin:
json-to-vpack	vpack-to-json

usr/local/include:
velocypack

usr/local/lib:
libvelocypack.a

g++ -std=c++11 -I ~/velo/usr/local/include/ -L ~/velo/usr/local/lib/  main.cpp -l velocypack -o test
In file included from main.cpp:1:
In file included from ~/velo/usr/local/include/velocypack/vpack.h:31:
In file included from ~/velo/usr/local/include/velocypack/AttributeTranslator.h:37:
~/velo/usr/local/include/velocypack/Slice.h:154:12: error: use of undeclared identifier 'VELOCYPACK_HASH'
    return VELOCYPACK_HASH(start(), checkOverflow(byteSize()), seed);
           ^
~/velo/usr/local/include/velocypack/Slice.h:165:12: error: use of undeclared identifier 'VELOCYPACK_HASH'
    return VELOCYPACK_HASH(start(), static_cast<size_t>(stringSliceLength()), seed);
           ^
2 errors generated.

# same output for Debian
g++ -std=c++11 -I ~/usr/local/include/ -L ~/usr/local/lib/  main.cpp -l velocypack -o test
In file included from /root/usr/local/include/velocypack/AttributeTranslator.h:37:0,
                 from /root/usr/local/include/velocypack/vpack.h:31,
                 from main.cpp:4:
/root/usr/local/include/velocypack/Slice.h: In member function ‘uint64_t arangodb::velocypack::Slice::hash(uint64_t) const’:
/root/usr/local/include/velocypack/Slice.h:154:68: error: ‘VELOCYPACK_HASH’ was not declared in this scope
     return VELOCYPACK_HASH(start(), checkOverflow(byteSize()), seed);
                                                                    ^
/root/usr/local/include/velocypack/Slice.h: In member function ‘uint64_t arangodb::velocypack::Slice::hashString(uint64_t) const’:
/root/usr/local/include/velocypack/Slice.h:165:83: error: ‘VELOCYPACK_HASH’ was not declared in this scope
     return VELOCYPACK_HASH(start(), static_cast<size_t>(stringSliceLength()), seed);

In Slice.h
#ifdef VELOCYPACK_XXHASH
// forward for XXH64 function declared elsewhere
extern "C" unsigned long long XXH64(void const*, size_t, unsigned long long);

#define VELOCYPACK_HASH(mem, size, seed) XXH64(mem, size, seed)
#endif

#ifdef VELOCYPACK_FASTHASH
// forward for fasthash64 function declared elsewhere
uint64_t fasthash64(void const*, size_t, uint64_t);

#define VELOCYPACK_HASH(mem, size, seed) fasthash64(mem, size, seed)
#endif

VELOCYPACK_XXHASH is defined when the lib is compiled (via cmake)
so add #define VELOCYPACK_XXHASH 1 as first line to main.cpp solved the problem.


Incomplete Response in HTTP1.1

UPDATE: i printed the self._buffer_end value and in the failed cases it is always = 65536
I assumed that this value should be  <=65535.
Should the new_buffer() be called before this happens?
OLD MESSAGE:
Hi Lucas,
I had a problem with HTTP1.1 library. (After making 90 serial 'GET' method calls)
The Wireshark pcap trace shows the full response but the BUFFER over here shows incomplete response.
This self._sock.buffer and the recv_into in self._sock.fill() is provided by socket library, Right?
Any tips on how to debug this?

Problem Identified:
Overview:

In the (common to http1 and http2) buffsocket.py file a new buffer
(space allocation) is created if the current one is filled before it is
used by the other files

In the case of the other some files, the data is read
before the buffer in buffsocket.py is full, so this case is never
encountered.

Problem:

In other file cases, the buffsocket.py buffer after the second last file has
only 71 bytes left, BUT as the condition [1] is:
"Only if 0 space remaining in the buffer create a new one (allocate more
space)"
The 22nd segment (size>71 Bytes) starts to download the buffer is full
only 71 Bytes of the response of next file are stored.
This stored value is given to the parser which sees incomplete
response and says  Response=None [2] which fails in here [3], and causes
error and TCP RST message to be sent.

Soultion:
I have set this [4] code:
     if self._remaining_capacity<=11935:

self.new_buffer()
and it seems to work.
Help:
Does it sound reasonable the keep the above comparison value a constant?
Thanks,
Raj
Links:
[1]
https://github.com/Lukasa/hyper/blob/75f25f73c17a16cb1f2ca7d46f390b0dfe4925dc/hyper/common/bufsocket.py#L165
[2]
https://github.com/deshmukhrajvardhan/hyper/blob/ecf17b6bc143d3af75757bca43b1794000a96ae2/hyper/http11/parser.py#L75
[3]
https://github.com/deshmukhrajvardhan/hyper/blob/ecf17b6bc143d3af75757bca43b1794000a96ae2/hyper/http11/connection.py#L291
[4]
https://github.com/Lukasa/hyper/blob/75f25f73c17a16cb1f2ca7d46f390b0dfe4925dc/hyper/common/bufsocket.py#L164


Expose /api/settings/ endpoint in the admin pages

This will be an endpoint restricted to superusers, which shows the settings of the current instance.
No action needed until we actually implement it in fractal-server.
Available in fractal-server 2.3.3

Is displaying the records with *** meaningful?

Is displaying the records with *** meaningful?

It's useful to know they exist (because it may remind the admin that maybe they got one wrong..), but displaying them in this kind of table creates a lot of unnecessary noise.
I would remove those fields from the table, and maybe add a sentence on top of the table (e.g. "obfuscated settings are not shown"). If we want to go even further, we can have a switch "hide/show obfuscated fields", but only if it takes very little effort.

I've added a switch


Option to backup data

Both the database itself and all the data in the folder need to be able to be backed up easily.
For extra credit we could optionally let the user sync their data to an online service, such as Dropbox or SpiderOak.
This issue is also opened on the server side:
https://github.com/OpenBazaar/OpenBazaar-Server/issues/186

We've decided not to do this.

This seems like an odd decision.

Sorry this is not clear enough. We don't plan to support it for version 1.1.x. We will be supporting backups for the next major version!


Default word's list

Is default word's list empty?
Or maybe there is some option in your library to see the all list of words which will be filtered?

The default word list is not empty. The default word list is just a default profanity filter. If you would like to see the words in the default list, you could look through the default list file in the source code (https://github.com/MrDogeBro/content_filter/blob/master/content_filter/data/filter.json).

@MrDogeBro , oh, I see now!
Thank you for the answering! Amazing tool :)

@hontarenkoYana no problem. Glad to help.


adding binary to int conversion

Describe the PR
adding binary to int conversion
Relation issue
https://github.com/swaggo/swag/issues/1466
Additional context
All the context is in the issue

Please also add unit tests for this change.

@ldrazic, Any updates?

@ubogdan I got stuck on the unit tests and left it there

A same PR as #1593.


refactor(website): create a dedicated page for import organizer

Summary
Add a dedicated page for the import organizer.
This makes it easier to discover this feature.
This also prepare the addition of future tools under Analyzer.
I also fixed the heading level of the page that highlights differences between Prettier and the Biome formatter.
Test Plan
Manually tested.

Suggestion: rename Imports Organizer to Import Sorting. The majority of users out there use this wording.


P2: Date of Birth range warnings

Add warning on Date of birth field when the data in in the future or more then 120 years in the past

won't do


some files not showing up on the explorer pane

Issue Type: Bug
1- Open folder with code
2 - Navigate to subfolder where the file lives
3 - File doesn't appear (whereas on the terminal does)

VS Code version: Code 1.38.1 (b37e54c98e1a74ba89e03073e5a3761284e3ffb0, 2019-09-11T13:35:15.005Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5 CPU         750  @ 2.67GHz (4 x 2132)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: unavailable_offprotected_video_decode: unavailable_offrasterization: unavailable_offskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
16.00GB (8.93GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (6)

Extension
Author (truncated)
Version

vscode-npm-script
eg2
0.3.9

vscode-pull-request-github
Git
0.11.1

vscode-docker
ms-
0.8.1

python
ms-
2019.9.34911

remote-wsl
ms-
0.39.5

debugger-for-chrome
msj
4.11.7

What's the value of your "files.exclude" setting?

Was the file created since you opened VSCode or was it already there?
Is it reproducible when you restart VSCode?
Can you check if there's a exclusion pattern set in files.exclude

@GitHubFilipe Can you please answer @aeschli questsions and can you reproduce with VS Code insiders https://code.visualstudio.com/insiders/

Was the file created since you opened VSCode or was it already there?
Is it reproducible when you restart VSCode?
Can you check if there's a exclusion pattern set in files.exclude

@aeschli It was indeed due to the entry src/**/*.scss.ts in the files.exclude setting.
Thank you!


Refactor SimpleEvolutionaryAlgorithm

The class has too many dependencies, which should be grouped together in fewer objects.

first part finished


Add "See More" to the projects Section

Adding a "See-More" to the projects section would be a great idea.
You could then prioritize your projects keeping the best ones on the top while the projects little lower in importance can be loaded when clicking see-more.

Good idea! I'm going to try and work this into one of the next releases as an optional feature for those who want to list lots of projects.

Added in release 1.2.0, thanks for feedback!


As a guest and host, I want to type in all ingredients I have in my fridge/storage room, in order to let others know what I can contribute for the meal.

Notes

Every guest and the host have to add minimum one ingredient, she/he wants to contribute
Each item is listed on its own
The user can finally submit her/his contributions

Development Tasks
Backend:

[x] sopra-fs23-group-03/sopra-fs23-group-03-server#104
[x] sopra-fs23-group-03/sopra-fs23-group-03-server#105
[x] #197
[x] #210

Frontend:

[x] sopra-fs23-group-03/sopra-fs23-group-03-client#57
[x] sopra-fs23-group-03/sopra-fs23-group-03-client#58
[x] sopra-fs23-group-03/sopra-fs23-group-03-client#59
[x] sopra-fs23-group-03/sopra-fs23-group-03-client#60
[x] sopra-fs23-group-03/sopra-fs23-group-03-client#61
[x] sopra-fs23-group-03/sopra-fs23-group-03-client#62
[x] sopra-fs23-group-03/sopra-fs23-group-03-client#63

Acceptance Criteria

[ ] A user has to add one ingredient minimum to be able to click the "continue" button
[ ] All contributions are listed separately
[ ] There is a button to submit the final contributions

Add task "Display the overall allergies of the group if any" for frontend


Invoke main dependencies in order

To make sure the dependencies will be executed in order, the pipe '|' sign is required before the list start.
Important while executing in multi-process (-j) build.

Thanks!


Add GT resources file locator

We can use GtResourceReference to find a GToolkit resource file, e.g., a document or a figure. On other hand, it is more convenient to hide the GtResourceReference lookup behind FileLocator. For example to find the GToolkit logo, we want to use:
FileLocator gtResource / 'feenkcom/gtoolkit/doc/gtoolkit.png'


.htaccess deploy request advice

Hello to everyone,
I'm a newbie in capistrano world but I tried to understand it :)
I have a question, I have manage to deploy my project that I stored on my git repo without problems.
After some configuration with "linked_files" and "linked_dirs" all works very well.
But today I noticed that .htaccess file in root folder of my project (that is present updated in my git repo) not deploy correctly (deploy always an old version of that (maybe first deploy that I made)).

Thank you in advance

Have you added .htaccess to :linked_files? If so, Capistrano will delete whatever .htaccess file is in your git repo and replace it with a symlink to the version that is in the shared directory. Maybe that is what is happening?

No, I added others files and folders to linked_dirs and linked_files but not .htaccess.
In fact based on what I made would have .htaccess updated ( like others files and folders of my projects  ) with that on git repo but not in this case.
.htaccess is old respect of what I have on my git repo.

Hmm, I am stumped then. As you probably know, when Capistrano creates a release it makes a copy of the latest files in git (the master branch, by default) into a releases/<TIMESTAMP> directory. Unless you have custom tasks that modify the contents of directory, every file within that release should match what was in git.
Subsequently, Capistrano modifies the current symlink to point to that release.
Possibly, if your server is failing to detect the update to the current symlink, it could be serving files out of an old release. I would recommend SSH-ing into the server and checking the contents of the latest releases/<TIMESTAMP> directory and the destination of the current link to verify what is happening.

Infact I am stumped too.
Current folder point correct releases/ folder with project but in the latest releases folder there is old (or default) htaccess and not my htaccess that is in git repo.
I try to redeploy various times but everytime all files are correct with that in my git repo but htaccess not.
So it's strange behaviour and infact I thought that is my fault somewhere.
Maybe are there problems with hidden files? I'ts only my guess...
I don't really know where to bang my head
Thank you in advance for your answer.

seems that capistrano overwrite htaccess with default one

One option to try would be to delete the current symlink and all of the releases/* folders, and then do a new deploy. If the wrong htaccess file is still there, delete the repo/ folder and redeploy again.

One option to try would be to delete the current symlink and all of the releases/* folders, and then do a new deploy. If the wrong htaccess file is still there, delete the repo/ folder and redeploy again.

Nothing.. I tried all of that but problem persists and seems very strange

I tried various solution but nothing.
Maybe are there a problems with hidden files? perhaps capistrano have difficult to match hidden files? I don't know is only my suppose because this behaviour is very strange ans seems impossible that heppens only in my env.
Thank you in advance for your answer

Ok, i resolved the issue.
it's not a capistrano problem but a command during magento install with capistrano (composer install) that overwrite my htaccess from my repo.
Thank you for your time


@typefox/monaco-editor-react does not work with React StrictMode enabled

Hi, I was looking to adopt your React wrapper for Monaco to use the language client services. An issue I ran into was, whenever <StrictMode> was enabled in React, the editor would fail to load properly and initialize the model. If you remove <StrictMode>, it works as expected.
I've reproduced the issue in a codesandbox for your review/testing..

Hi @zzullick thanks for reporting. We need to fix. It was already brought up some time ago, but know I have an idea what could be the root cause. I will work on a fix in the upcoming days.


Fix pixel picker for square pixels

Why was this done in the first place?
Did (x,y) refer to the pixel edges and not the centers?

Why was this done in the first place?
Did (x,y) refer to the pixel edges and not the centers?

No, for the old Rectangle, (x, y) refered to the lower left corner. See changes here: #1819


Unable to ping clients from OpenVPN Server

I installed this OpenVPN on my server using this. My VPN is working properly without any issues but I wanted to interconnect the hosts which are connected to my VPN.
Host A : <IP_ADDRESS>
Host B : <IP_ADDRESS>
VPN Server S : <IP_ADDRESS>
When I ping B to S it works.
When I ping from A to S it works.
But I can't ping from S to A and S to B.
And because of that I can't ping from A to B. But both A and B are able to access Internet without  any issues.
While breaking my head with the problem I decided to capture traffic on my vpn server using tcpdump.
I used the
tcpdump -i tun0 | grep ICMP

to capture traffic.
# Now in Host A, if I do
ping <IP_ADDRESS>
# then the packet capture on vpn server is this.

12:43:41.492078 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 6757, seq 1, length 64
12:43:42.496108 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 6757, seq 2, length 64
12:43:42.496134 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 6757, seq 2, length 64
12:43:43.496314 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 6757, seq 3, length 64
12:43:43.496346 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 6757, seq 3, length 64
12:43:44.495899 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 6757, seq 4, length 64
12:43:44.495922 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 6757, seq 4, length 64
12:43:45.500215 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 6757, seq 5, length 64
12:43:45.500236 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 6757, seq 5, length 64

# This shows that ping is working properly.

# Now when I ping from VPN Server S to Host A using command
ping <IP_ADDRESS> # in server S

the packet capture running captures the following
22:15:16.867718 IP 139.59.5.xx > <IP_ADDRESS>: ICMP echo request, id 2886, seq 10, length 64
22:15:17.867828 IP 139.59.5.xx > <IP_ADDRESS>: ICMP echo request, id 2886, seq 11, length 64
22:15:18.870699 IP 139.59.5.xx > <IP_ADDRESS>: ICMP echo request, id 2886, seq 12, length 64
22:15:19.869963 IP 139.59.5.xx > <IP_ADDRESS>: ICMP echo request, id 2886, seq 13, length 64

# and host A is not responding to these ICMP packets. What could be the problem?

#when I change
server <IP_ADDRESS> <IP_ADDRESS>
#to
server <IP_ADDRESS> <IP_ADDRESS>
#then the problem gets solved, A can ping B, B can ping A and everyone can ping everyone, everyone is happy.. but Internet stops working on A and B.

#Please help. I have broken my head over this problem for very long time now.

The "road warrior" setup was not intended for this, internal networking is intentionally not working. Still, this can be easily changed.
If you take a look at the script, you'll see the following rule being used:
iptables -t nat -A POSTROUTING -s <IP_ADDRESS>/24 -j SNAT --to $IP
If instead you add an exception for the VPN subnet, internal networking will work:
iptables -t nat -A POSTROUTING -s <IP_ADDRESS>/24 ! -d <IP_ADDRESS>/24 -j SNAT --to $IP
I hope this helps.
PS: I always get notified of new issues opened here. I don't appreciate you spamming my personal email and demanding a "quick response".

Thanks a ton. I tested it and it's working perfectly fine.
I completely missed out the email notification part. Sorry for the spam again.

From: Nyr<EMAIL_ADDRESS>Sent: Sunday, September 4, 2016 10:34 PM
To: Nyr/openvpn-install
Cc: Vikas Yadav (IMT2013060); Author
Subject: Re: [Nyr/openvpn-install] Unable to ping clients from OpenVPN Server (#190)
The "road warrior" setup was not intended for this, internal networking is intentionally not working. Still, this can be easily changed.
If you take a look at the script, you'll see the following rule being used:
iptables -t nat -A POSTROUTING -s <IP_ADDRESS>/24 -j SNAT --to $IP
If instead you add an exception for the VPN subnet, internal networking will work:
iptables -t nat -A POSTROUTING -s <IP_ADDRESS>/24 ! -d <IP_ADDRESS>/24 -j SNAT --to $IP
I hope this helps.
PS: I always get notified of new issues opened here. I don't appreciate you spamming my personal email and demanding a "quick response".

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Nyr/openvpn-install/issues/190#issuecomment-244614857, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGGiY-TufEJitRxX3aCMspLbSrDKy-spks5qmvoegaJpZM4J0k4p.

Hi @Nyr thank you very much for this repo and for your reply here. I have a question for you please and I hope this is the right place to ask it.
I now have everything working in terms of my vpn clients can ping the server and the server can ping the client.
I have another machine on the server side.  So lets say:
S: Server local IP: <IP_ADDRESS>
O: Other machine: <IP_ADDRESS>
O and S can ping each other.
S can ping A and B
How can I get O to ping A and B please?
I added a route on O to <IP_ADDRESS> and set the gateway to S.
But S can not ping A.
Btw A can ping S.
Pleasse let me know if you like me to raise another ticket.
Best,
A

I hope this is the right place to ask it

It is not, I would suggest at least a new issue because this question is unrelated, but the issue tracker is not really a support forum.

S can ping A and B
But S can not ping A.
Btw A can ping S.

I don't understand that, it is contradictory. I don't even understand where are A and B, are they VPN clients I guess?
Anyway, you have two options:

Connect everyone needing access to the VPN server
Set up routes on the VPN server so clients can connect to hosts in the network which is available to the server but not the client. You need to push those routes to the clients. Check the OpenVPN forums if you need specific instructions for that.

Hi @Nyr

I hope this is the right place to ask it

It is not, I would suggest at least a new issue because this question is unrelated, but the issue tracker is not really a support forum.

Thank you very much for your reply :)
So fair point and fair point.
I was just intending on extending the question above and the scenario discussed. Obviousely I was not clear enough :)
The idea is A,B and S are the servers per this issue.
And the Ovpn has been set up per your repo and has the changes above to allow A and B to talk added inside the clint side network (VPN subnet internal networking)

S can ping A and B
But S can not ping A.
Btw A can ping S.

I don't understand that, it is contradictory. I don't even understand where are A and B, are they VPN clients I guess?

A,B and S are per the issue ticket.
O is on S side Lan.
I should have said A and B can ping O currently.

Anyway, you have two options:
Connect everyone needing access to the VPN server
Set up routes on the VPN server so clients can connect to hosts in the network which is available to the server but not the client. You need to push those routes to the clients. Check the OpenVPN forums if you need specific instructions for that.

A,B and S can ping birecetionaly, once I "add an exception for the VPN subnet"
A and B and S can ping O
S and O can ping birecitionally.
O can't ping A nor B
O and S are on the same LAN.
If you are able to help here without too much hassle then I shall move this onto an issue.
Otherwise thanks for your response and I shall try the forums.
Best.

@aliradd

The changes described in this issue to enable internal networking are no longer relevant. Internal networking was enabled some time ago in the script.
This is what you want: https://openvpn.net/index.php/open-source/documentation/howto.html#scope

Hi,
Thank you very much for your reply. Just ping me if you think this needs a different issue and I would be happy to create one.
All the best,
A.

Best Regards,
Ibrahim Almalki

On Feb 21, 2018, at 2:05 AM, aliradd<EMAIL_ADDRESS>wrote:
Hi,
Thank you very much for your reply. Just ping me if you think this needs a different issue and I would be happy to create one.
All the best,
A.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Dear @Nyr Time passed and the script changed. How can I fix it now so that devices on the network can communicate with each other?
 Thank you! Happy New Year.


Data aggregation not functioning

@ggdhines I requested a new aggregation export from Whales as Individuals two days ago, then again yesterday, with no apparent result. When I download data, result is an aggregation from January.
Thx ~ Ted

hi Ted, is your project id 11? I don't know how to easily get the a project name from an id but I know that an aggregation run for project 11 caused an error a couple of days ago. That error has been fixed. I didn't receive an error yesterday but I there was stuff going on with rectangle aggregation that only got fixed today. Apologies - need to get a better notification system set up.
That said, new builds aren't working right now - for some reason config files aren't being copied over correctly from AWS (amazon web services). I notified the admin person at Zooniverse but that might not be get fixed until Monday. Will keep you updated.

Hi Greg. Thanks, yes, project 11. So as far as getting the aggregation data, sometime Monday or so?

Hopeful that the config file issue can be quickly fixed - in which case definitely Monday.

quick q: Is the config file issue affecting all projects? Or just projects with rectangle aggregations?

@aliburchard was unfortunately affecting everything but is now fixed. @tedcheese give it a go. I'm heading out for the head but will check tonight or tomorrow for any issues.

... requested again a five hours ago with no result... how long should it take?

@ggdhines Any update on this?

Hi @tedcheese - things are looking good but there are some clusters that aren't being included in the final output. They seem to be missing some data fields (those fields get calculated for every cluster so trying to understand why these clusters are different). Once that's done, I'll run the code and email you the results.


CTT carrier not parsing correctly

From my test (maybe others need to confirm), if you use lower case on CTT tracking, for example Rh949499356pt it will not find the shipping
But if you use UPPERCASE like RH949499356PT it does find.
So to parse the input to uppercase at least with CTT can probably be a solution.

Hey @diogodh, yeah lowercase letters usage is not common in tracking ids. Even if you use that in the CTT website, they don't allow it. Some retailer sent you the id in lowercase?
Right now our regex expressions that assign ids to carriers only take in consideration the uppercase letters (valid id). If we uppercase all ids by default, it might break some carriers that use lowercase letters (can't tell by memory one that uses it).
Maybe we can add a warning message below the input when the user is user lowercase letters, it's probably an invalid id.

Hello @carlosflorencio in ctt website, I can use both lower or upper case, either using website input,  short url like https://ctt.pt/t/XX123456789XX or full url like https://appserver.ctt.pt/CustomerArea/PublicArea_Detail?IsFromPublicArea=true&ObjectCodeInput=XX123456789XX&SearchInput=XX123456789XX
On the input in my case, it was a typo I did while writing the tracking code, cause my browser on mobile uses uppercase on the first letter as default.

Okay, I think I'll set the id regex expressions to be case insensitive. I'll let you once it's deployed. Thanks for the feedback 💪

Should work fine now with lowercase ids.


Support for HTTP Basic Auth?

This is likely entirely my fault for missing it, but — I looked in the Kiosk app, and Artsy seems to encode the API keys into the the URL itself, but the API I’m working against encodes the keys using HTTP Basic Auth. Is that a supported feature of Moya, or is that something that I should drop down into Alamofire for (or even open a PR for)?
Thanks.

(I should add — a lengthy answer isn’t necessary! just a yes or no would suffice.)

tl;dr Moya doesn't currently support this.
Hmm. Interesting question. Alamofire seems to handle this with NSURLCredential. Is this how you've done HTTP auth in the past?
Maya doesn't currently have support for this, but using the sharedSession NSURLSession, so you should be able to set things there and have all HTTP requests use those credentials. Obviously that's not a great fix, though.
I guess what this would really need is some kind of (optional) way to specify which NSURLSession to use: the default, or one configured for your endpoints alone.
Others are likely more familiar with the network library underneath Alamofire. Anyone have suggestions on how to implement this?

Unless I misunderstand, HTTP Basic Auth is implemented by Alamofire as just a chained method after .request.
Here’s my naive suggestion: pass the username/password to MoyaProvider#sendRequest as an optional parameter of type NSURLCredential, and if it’s not nil, call .authenticate after the Alamofire.Manager.sharedInstance.request(request) call.
Granted, that makes the method messy, hence my referring to it as “naive”.

That makes sense. But it also adds a lot of complexity around individual API calls. Developers now need to remember which endpoints need auth and which don't, and that need is spread across their entire codebase.
I think we need one of two things:

an optional part of the MoyaTarget protocol that specifies per-target credentials
an optional closure in the MoyaProvider initializer that gets last-moment chance to specify credentials based on the target. If it returns credentials, they're added to the request in the provider.

I like the second option better because most users don't need HTTP basic auth and it keeps our protocol leaner. What do you think?

Good point, I didn’t think through how my API would break a lot of applications in production. I like your suggestion of the the optional closure in the MoyaProvider initializer.
Would you like me to open a pull request in the next day or so?

That would be lovely! Please let me know if you'd like to bounce ideas around or anything :smile:

👍

So the API I’m writing against is going to be deprecating their basic auth, and because of that, I don’t need to add that to Moya. Since nobody else seemed to expressed a need for it, I think it's best if the issue is closed.

Cool, thanks for the update @davidbarsky. Please let us know if there's anything else you'd like to see added to Moya!

Thanks for your help!


Clean up materialised tables that Splink creates when we're not using linker.py (profiling)

This PR ensures that we delete tables created by the profiling code so the database doesn't get littered with lots of temp tables.
This does mean they won't exist in the cache, meaning if for some reason want to call profile_columns twice, the second one call won't be able to use the cache.   But I can't see when that'd matter.

example

import logging

import duckdb

import splink.comparison_library as cl
from splink import (
    DuckDBAPI,
    SettingsCreator,
    block_on,
    splink_datasets,
)
from splink.blocking_rule_library import block_on
from splink.datasets import splink_datasets
from splink.profile_data import profile_columns

settings = SettingsCreator(
    link_type="dedupe_only",
    comparisons=[
        cl.ExactMatch("first_name"),
        cl.ExactMatch("surname"),
        cl.ExactMatch("dob"),
        cl.ExactMatch("city"),
        cl.ExactMatch("email"),
    ],
    blocking_rules_to_generate_predictions=[block_on("first_name")],
    retain_matching_columns=True,
    retain_intermediate_calculation_columns=True,
)

df = splink_datasets.fake_1000
df2 = splink_datasets.fake_1000.copy()
con = duckdb.connect()

con.sql("CREATE TABLE my_table AS SELECT * FROM df")

db_api = DuckDBAPI(connection=con)
db_api.register_multiple_tables(["my_table", df2])
con.execute("SELECT name FROM sqlite_master WHERE type='table'").df()
# db_api.debug_mode = True
# logging.basicConfig(
#     format="%(message)s",
# )
# splink_logger = logging.getLogger("splink")
# logging.getLogger("splink").setLevel(1)

profile_columns(
    ["my_table", df2], column_expressions=["first_name", "surname"], db_api=db_api
)

con.execute("SELECT name FROM sqlite_master WHERE type='table'").df()

@ADBond apologies!  Tests passing now


Added option to disable hooks on job. I need this, since the class I'…

…m delaying also have a 'before' method

Coverage increased (+0.02%) to 91.143% when pulling caa82a745f6afc6de34041d93c730cd4b0c77b4b on holgersindbaek:master into e3772d4f0c8470d0fcba00c86ca3bc4f5e876830 on collectiveidea:master.

Coverage increased (+0.02%) to 91.143% when pulling caa82a745f6afc6de34041d93c730cd4b0c77b4b on holgersindbaek:master into e3772d4f0c8470d0fcba00c86ca3bc4f5e876830 on collectiveidea:master.

I think there might be a better, less all or nothing, approach but this is an interesting issue. We might also need to rely on a less collision prone implementation in future versions but that would need to be phased in to avoid breaking existing apps.


Successful JSON API request with filter resolving to empty record set

I am using the JSON API source that is retrieving data from my API using customized filtering (that is connected to a MySQL database in conjunction with a Loopback API server).
    store.query((q) =>
      q.findRecords("planet")
      .sort('name')
      .filter({
        attribute: "where",
        value: {
          name: {
            like: "%ven%"
          }
        }
      })
    )
      .then((planets) => {
        console.log(planets); // This is an empty array
      })
      .catch((err) => {
        console.log(err); // No errors
      });

It's weird because the resolved promise returns an empty array... despite the HTTP request working exactly as expected (see below cURL request copied from Chrome devtools):
curl 'http://localhost:8080/api/planets?filter%5Bwhere%5D%5Bname%5D%5Blike%5D=%25ven%25&sort=name' -H 'Accept: application/vnd.api+json'

Which returns the following data set:
{
  links: {
    self: "http://localhost:8080/api/planets?filter%5Bwhere%5D%5Bname%5D%5Blike%5D=%25ven%25&sort=name"
  },
  data: [
    {
      "type": "planets",
      "id": "13ff0af2-07b7-11e6-af98-09e369f0ea4e",
      "attributes": {
        "name": "venus"
      }
    }
  ]
}

Note: I know I could get the proper set with a simple { attribute: "name", value: "venus" }, but my real use case is a more complex SQL query to return a set based on the presence of a string within a JSON array, thus the above is just an example.

I'm actually getting this with normal attribute filters. Does the store attempt to do a local in-memory filter after receiving the remote request? Is there any way to force using the results from the remote request as-is?

Does the store attempt to do a local in-memory filter after receiving the remote request?

Yes. If you query the store, the results will come directly from the store. In other words, store.query will never return results that are not in the store.
It's the responsibility of your coordination strategies to ensure that data from another source is pushed into the store as a result of a query, prior to the application of the query on the store. For instance, you can configure a beforeQuery request strategy such that your query is applied to your JSONAPISource, and then a sync strategy to move all data from your JSONAPISource into your store. If these are both blocking strategies, then the results will be moved into the store before the query is applied to the store.

Is there any way to force using the results from the remote request as-is?

We can consider implementing Queryable on the JSONAPISource in addition to Pullable so that you could query the JSONAPISource directly and access the results as an array of records rather than an array of operations.

@dgeb Yes, that's what I have (the strategies for sync and query straight from the guide). The issue is that the api I'm calling takes multiple filters in  comma separated form, not square bracket arrays i.e. filter[a]=x,y,z not filter[a][0]=x&filter[a][1]=y...
I realized the json api source wasn't queryable when I tried using it directly. Is there any way I can help get this working?

I am having the same issue as @tatemz described above and I think implementing Queryable to the JSONAPISource is one solution.
@dgeb, if you see that as something, which fits the the ideology of orbit. I would be happy to help and possibly open a PR of that.

FWIW I've noticed that datatypes must match when applying filters.
For example, if I have the following schema:
const schema = new Schema({
  models: {
    foo: {
      keys: { remoteId: {} },
      attributes: {
        bar: { type: 'number' },
      },
    },
  }
};

and try to apply a filter like so, where the local value bar is a string.
let foos = await store.query(q =>
  q.findRecords('foo').filter({
    attribute: 'bar',
    value: bar // <--- this is a string
  })
);

It will return 0 results. However as long as the datatype matches, I'm good:
let foos = await store.query(q =>
  q.findRecords('foo').filter({
    attribute: 'bar',
    value: parseInt(bar) // <--- make the datatype match what is defined in the schema
  })
);

I think this issue should be solved with the new hints feature https://github.com/orbitjs/orbitjs-site/issues/23

Yes, @tchak is right. Hints have been supported since v0.16 and are described in the guides here: https://orbitjs.com/v0.16/guide/coordination.html#Using-hints


TypeScript Support

hey @DominusKelvin
I could pickup on type support for the SDK.

Nice idea but you'd have to look at treblle-utils and how Types are exported

yeah. I just went through it this afternoon and would reference it for sure. I'm also doing a bit of light reading on declaration files.


Display latest solution by default

In classroom, when displaying the solutions given by a student for a given exercise, the first one displayed is the first one that was submitted. I would prefer to display first the last one, so that the tutor can immediately spot the current status.

I edited the problem description, which was a bit awkwardly redacted.


last_seen and "…the configured sensor for room temperature is unavailable"

Version of the custom_component

Version: 6.2.9 (latest)
Configuration

My VTherm attributes are the following:
hvac_modes:
  - "off"
  - heat
min_temp: 14
max_temp: 26
target_temp_step: 0.5
preset_modes:
  - none
  - eco
  - comfort
  - boost
current_temperature: 22.9
temperature: 19.5
preset_mode: comfort
is_on: false
hvac_action: null
hvac_mode: "off"
type: null
is_controlled_by_central_mode: true
last_central_mode: Auto
frost_temp: 0
eco_temp: 19.5
boost_temp: 23
comfort_temp: 19.5
frost_away_temp: 0
eco_away_temp: 0
boost_away_temp: 0
comfort_away_temp: 0
power_temp: 13
target_temperature_step: 0.5
ext_current_temperature: 7.9594858575244665
ac_mode: false
current_power: null
current_power_max: null
saved_preset_mode: comfort
saved_target_temp: 19.5
saved_hvac_mode: null
motion_sensor_entity_id: null
motion_state: null
power_sensor_entity_id: null
max_power_sensor_entity_id: null
overpowering_state: null
presence_sensor_entity_id: null
presence_state: null
window_state: "off"
window_auto_state: "off"
window_bypass_state: false
window_sensor_entity_id: null
window_delay_sec: 120
window_auto_enabled: false
window_auto_open_threshold: null
window_auto_close_threshold: null
window_auto_max_duration: null
window_action: window_turn_off
security_delay_min: 60
security_min_on_percent: 0.5
security_default_on_percent: 0.1
last_temperature_datetime: "2024-09-14T10:40:09.080166+02:00"
last_ext_temperature_datetime: "2024-09-14T10:52:00.102638+02:00"
security_state: false
minimal_activation_delay_sec: 10
device_power: 1
mean_cycle_power: null
total_energy: 0
last_update_datetime: "2024-09-14T10:52:00.112424+02:00"
timezone: Europe/Vienna
temperature_unit: °C
is_device_active: false
ema_temp: 22.85
is_used_by_central_boiler: false
is_over_climate: true
start_hvac_action_date: null
underlying_climate_0: climate.wohnzimmer_eve_trv_thermostat
underlying_climate_1: null
underlying_climate_2: null
underlying_climate_3: null
is_regulated: true
regulated_target_temperature: 19.5
auto_regulation_mode: auto_regulation_slow
regulation_accumulated_error: -576
auto_fan_mode: auto_fan_none
current_auto_fan_mode: auto_fan_none
auto_activated_fan_mode: null
auto_deactivated_fan_mode: null
auto_regulation_use_device_temp: true
friendly_name: Wohnzimmer Heizung
supported_features: 401

Describe the bug
https://github.com/jmcollin78/versatile_thermostat/issues/398 added a last_seen for room temperature sensors. My Aqara temperature devices do not offer such a value. It might be available as an attribute somewhere, but I cannot enter that into the config. See:

My logs are full of messages like: WARNING (MainThread) [custom_components.versatile_thermostat.pi_algorithm] Temporarily skipping the self-regulation algorithm while the configured sensor for room temperature is unavailable , so it seems I won't have a self-regulation until this is fixed or I can find an alternative solution. Thanks!

Hello @maia ,
Not sure to have understand. If you don't have last-seen attribute on your temperature device, I suppose it is not configured in your VTherm. So why are you talking about the last-seen ?
The error message is not relative to last-seen but relative to the temperature entity itself.
With Z2M, you have to enable the last-seen in the can check if the last-seen is visible:

Do you have this ?
if yes, I think you have to add the last-seen entity by clicking on the "+3 entities not shown" in the first screen.

I look into the code, and the error is thrown when the room_temp is not available (nothing to do with last-seen):
if room_temp is None:
            _LOGGER.warning(
                "Temporarily skipping the self-regulation algorithm while the configured sensor for room temperature is unavailable"
            )

Thanks, so the room temp error is probably just startup-related.
As for last seen room temperature datetime, as you are asking for the entity in the "Main" config screen, this suggests that this isn't an optional entity but required. I know that there's no "*" next to it, but still it is very confusing. It's lacking an explanation what it is used for, what happens if you don't provide it, where to search for it. Especially as not all devices expose this entity and even with Z2M you manually have to enable it (which might not even be possible in ZHA).
I guess many/most/all devices have the datetime for the last measurement of the internal thermometer, but they might only be exposed as attribute and not as entity. See below for my Eve Thermo (Matter).

As you can see, to the right is last changed, last updated, I imagine this is the necessary information?

Hello @maia ,
Yes this experimental feature have not been documented yet. You have more explanation here: https://github.com/jmcollin78/versatile_thermostat/issues/398
The objective is to avoid the safety state which happens when the temperature doesn't change for a long period but the thermometer is alive. Last_seen is a Zigbee feature activable that can be useful for this case.
If you don't have one, don't worry, you are just potentially subject to the safety mode while your thermometer is alive.
And yes, it is not mandatory.

As noted in https://github.com/jmcollin78/versatile_thermostat/discussions/497#discussioncomment-10669390 : You should only show this configuration if vtherm cannot automatically identify the value itself.
For example all my Matter TRV devices return a UTC datetime for: {{ states.climate.living_room_eve_trv_thermostat.last_updated }}. I cannot enter this attribute in the vtherm config, but it exists and vtherm should use it automatically.

I have to correct myself: you're not interested in the last room temperature of the TRV internal sensor but from the external room sensor, right? I just noticed that my Aqara temperature sensors (Zigbee) don't report for long time spans when the temperature doesn't change above an internal threshold. In one room the temperature measured was very constant between 20,8°C and 21,2°C and the Aqara sensor only reports about once per hour. This might trigger the safety feature.
I wonder if you could prevent early trigger of the safety feature by checking the internal TRV temperature sensor too. Because if the external room sensor doesn't report anything this might just mean that the temperature is currently very constant.

but from the external room sensor, right?

You should configure the last seen date of the thermometer that controls the VTherm.

the Aqara sensor only reports about once per hour. This might trigger the safety feature.

Exactly the case.

Because if the external room sensor doesn't report anything this might just mean that the temperature is currently very constant.

And how can I use this in your opinion ?

I think the safety feature is more of a problem than a feature when using over_climate. When users do not manually enable last seem temperature entities and add these, they will trigger the safety feature frequently, simply because some thermometers do not report if the temperature doesn't change. I think:

the safety feature should be an option and not a fixed feature
the safety feature should be disabled by default for over_climate mode
set the "Safety delay" in "Advanced - Central configuration" to 120 or 180 as default
add an option for safety temperature in "Advanced - Central configuration", it currently is only available for "Power percent"

Hello @maia, thank you for this report. I understand your point but:

you can always disable safety feature by setting a high number in delay,
the temperature is critical information for auto-regulation algorithm and can lead to wrong target temp specially if you use strong auto-regulation. Then this can lead to over heating a room.

I will check for point 3 or 4, they are more relevant.

Hello @maia ,
After some thoughts, the only relevant point is the 4.:

what can I propose when an over_climate is in safety. And I don't have any good proposal: setting a defaut setpoint is not accurate (winter ? summer ? AC mode ? Heat only ?). The only thing to do is to turn off the device when that happens or let the device do its live with the last regulated target given by Vtherm event if it is no more relevant.

All this options are already possible without any changes, so the only think I can do is to explain that the default power in safety do not apply to over_climate.
This is done in README now.


feat: Allow overriding application name in manifest.json

Generate manifest.json dynamically, using MANIFEST_NAME and MANIFEST_SHORT_NAME from the environment.
Pull Request Checklist

[X] Description: Briefly describe the changes in this pull request.
[X] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[ ] Documentation: Have you updated relevant documentation?
[X] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?

Description
Allow overriding the application name in manifest.json for installing Open WebUI as a PWA.

Changelog Entry
Changed

The name of the application in manifest.json can now be configured with the environment variables MANIFEST_NAME and MANIFEST_SHORT_NAME, which is used when installing Open WebUI as a PWA.

Incidentally, the dimensions of the icon in the manifest are incorrect and weird, I'm not sure if that was intentional but I left it at the same value it was in the static manifest.json previously, 844x884.
favicon.png: PNG image data, 500 x 500, 8-bit/color RGBA, non-interlaced

LGTM, Thanks!


dx | 1459 unit test cases (oclif v4)

updated all test cases according to oclif v4

I will look into the failing check

@cs-raj , can you pls review this PR and approve it ?


upload-sarif action seems to always use GITHUB_WORKSPACE Git information to compute commit_oid

Uploading SARIF files using the upload-sarif@v1 action can fail with:
Processing sarif files: ["scan.sarif"]
Uploading results
Error: commit not found
RequestError [HttpError]: commit not found

The problem here seems to be that the commit_oid parameter in the uploaded JSON is from an entirely different repository: the repository that's the "main" checkout (the one with empty path parameter to actions/checkout@v2), even with checkout_path specified.

Full log output
 Processing sarif files: ["jenkins-security-scan.sarif"]
 Uploading results
 Error: commit not found
 RequestError [HttpError]: commit not found
     at /home/runner/work/_actions/github/codeql-action/v1/node_modules/@octokit/request/dist-node/index.js:66:23
     at processTicksAndRejections (internal/process/task_queues.js:93:5)
     at async Job.doExecute (/home/runner/work/_actions/github/codeql-action/v1/node_modules/bottleneck/light.js:405:18) {
   name: 'HttpError',
   status: 404,
   headers: {
     'access-control-allow-origin': '*',
     'access-control-expose-headers': 'ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset',
     connection: 'close',
     'content-encoding': 'gzip',
     'content-security-policy': "default-src 'none'",
     'content-type': 'application/json; charset=utf-8',
     date: 'Fri, 25 Feb 2022 10:29:15 GMT',
     'referrer-policy': 'origin-when-cross-origin, strict-origin-when-cross-origin',
     server: 'GitHub.com',
     'strict-transport-security': 'max-age=31536000; includeSubdomains; preload',
     'transfer-encoding': 'chunked',
     vary: 'Accept-Encoding, Accept, X-Requested-With',
     'x-content-type-options': 'nosniff',
     'x-frame-options': 'deny',
     'x-github-media-type': 'github.v3; format=json',
     'x-github-request-id': '0403:3D83:83138D:1507C61:6218AF7B',
     'x-ratelimit-limit': '1000',
     'x-ratelimit-remaining': '995',
     'x-ratelimit-reset': '1645785350',
     'x-ratelimit-resource': 'code_scanning_upload',
     'x-ratelimit-used': '5',
     'x-xss-protection': '0'
   },
   request: {
     method: 'PUT',
     url: 'https://api.github.com/repos/daniel-beck-org/sample-plugin/code-scanning/analysis',
     headers: {
       accept: 'application/vnd.github.v3+json',
       'user-agent': 'CodeQL-Action/1.1.3 octokit-core.js/3.1.2 Node.js/12.13.1 (linux; x64)',
       authorization: 'token [REDACTED]',
       'content-type': 'application/json; charset=utf-8'
     },
     body: '{"commit_oid":"92e0b0945a6334eeeb4c65a78c2be5a7767e3cc9","ref":"refs/heads/main","analysis_key":".github/workflows/jss.yaml:scan","analysis_name":"Jenkins Security Scan","sarif":"...","workflow_run_id":1898119060,"checkout_uri":"file:///home/runner/work/sample-plugin/sample-plugin","environment":"null","started_at":"2022-02-25T10:29:14.509Z","tool_names":["Jenkins Security Scan"]}',
     request: { agent: [Agent], hook: [Function: bound bound register] }
   },
   documentation_url: 'https://docs.github.com/rest'
 }

If no repository is checked out at this location (i.e. every actions/checkout@v2 has a path), then the error is the same as in #944 (but at least the upload still happens, at least for a non-PR upload).

Have you tried setting the ref: and sha: properties?
https://github.com/github/codeql-action/blob/21c48e7a01c38ddfa489aa2d1f22829f40325298/upload-sarif/action.yml#L16-L21

I have. Made no difference.

but at least the upload still happens, at least for a non-PR upload

I only did fairly superficial checks before trying to work around the problem, but IIUC, PRs also fail to upload results, since the PR merge ref cannot be checked out.

If I understand correctly, the workflow is trying to upload some SARIF to a different repository than the one currently runing the workflow?
The codeql-action was not designed to operate in this manner. If this is what you are trying to do, you will need to avoid using the codeql-action at all and use the codeql CLI directly to perform the upload. essentially, you will need to treat the upload as coming from a third party CI system.
There is a command, codeql github upload-results that you can invoke to do the upload.
It will be a bit more finicky since you will need to specify the correct SHA, head, and base refs, and repository. But I think it should work.
Also, you will need to make sure that the PAT used to upload will have enough privileges to upload security results to the appropriate repository. In order to do this, you will need to create a new repository secret for a PAT you have generated that will have upload access to this repo.

If I understand correctly, the workflow is trying to upload some SARIF to a different repository than the one currently runing the workflow?

No, it's the correct destination repository, but it specifies a wrong SHA for the commit_oid parameter of the undocumented API it uploads to, resulting in the server rejecting the upload (AFAICT, undocumented!).
Basically, while a checkout_path parameter exists, that is ignored when determining various parameters of the /code-scanning/analysis API. This is non-fatal in cases like #944 (when there is no checkout to the top-level directory), but if there is a wrong repo there, or we're building a PR, then the git commands will fail and the upload aborts.
Since my use case includes scan results in PRs from forks, this action is the only (documented) solution. I don't know whether I'd violate the ToS of GitHub by just calling this API directly, so I've since completely restructured my workflow to make it work by checking out the component being scanned without a path argument to actions/checkout@v2. This is still a bug IMO: It is both unexpected (why else would there be checkout_path if it's ignored for Git operations?) and undocumented.

OK. Thanks for explaining. That does sound like a bug. The checkout_path is not being used to determine the commit_oid. It shouldn't be difficult to address this.
That being said, my previous comment would still be a viable workaround until there is a fix. You can still call the codeql github upload-results directly.

You can still call the codeql github upload-results directly.

I'm not that familiar with the GitHub actions security model, but AFAIUI even PR authors can edit workflows and therefore access any secrets made available to them, which is why GITHUB_TOKEN has fewer permissions when run from PRs. The GitHub CLI docs specifically point out that a token with security_events: write permission is needed.
If I wanted that I could have had it with my curl-based approach I used originally, but that seemed unsafe. Apparently /code-scanning/analysis doesn't need this permission, but it's undocumented and I didn't want to risk a ToS violation -- so here I am, with no real options other than this action 😉


Store key values as RLP encoded data

Description
It is possible that key values are not stored as bytes rather can be recursive RLP lists.
Currently this ENR library assumes that key values are stored as bytes and then interpreted. As the specific encoding is key-specific, we should generalise how keys are stored in the RAW ENR.
We currently decode the RLP of an ENR assuming it is always a list of bytes. The more general solution is to iterator through the RLP, firstly collecting the default required types such a sig and seq_no. Once we start iterating through the key/value pairs, the keys MUST be encoded as bytes but the values may be RLP lists. We should store RLP encoded bytes of the value in the BTree rather than the decoded bytes.
Then regardless of known key/value encodings an application can read a key and perform their own custom RLP decoding on the RLP encoded bytes stored as the value.
This strategy will involve updating the current known keys, such as ip, udp, tcp etc to decode the RLP from the BTree first before converting to the correct type.
Once updated, this should resolve #3 and this library should be able to decode the geth's DNS ENR entries (one of these should be added as a test).

Closed in #6


add maui-preview in nuget.config

Our goal is to no longer have a NuGet.config file in these samples. Can you explain a bit more on what you’re trying to do here? We can maybe do something on the develop branch for this, but not main.

if we do not have this, the project cannot pass the build. ---around 10 Red Errors

@infofromca can you share a build log? What are the errors?
If you do dotnet build -bl, it will create a msbuild.binlog file you should be able to zip and share here. Thanks!

the other guy got the same as mine.
fix https://github.com/dotnet/maui/issues/1114

to avoid it, I think it is better to add  nuget lib  inside the project

to avoid it, I think it is better to add nuget lib inside the project

Nope, we need to understand what is actually missing, so it can be pushed to NuGet.org.

Things changed a lot in Preview 6. .NET MAUI is now a "workload" and not a NuGet package anymore.
Can you test and see if you're running into this issue still? Thanks!


Remove unused plugin dependencies

As pointed out by @HenningSE, some of the unused plugin dependencies in the WFSim->fuse transition went under the radar in the review of #72. Before the next release, make sure to clean up the code of these dependencies.
Related to this: let's also clean up the deprecated options in the config *json files.

I have this autogenerated table of all the configs currently needed in fuse, if anyone is interested.


Ft login page 161578378

What does this PR do?
Built the login pages for the users
Description of Task to be completed?
All works finished
Any background context you want to provide?
User login forms
What are the relevant pivotal tracker stories?
#161578378

You could leave a screenshot of the UI its easier without me having to clone your repo.

Add screenshots and describe your topics well

add screenshots please

Okay guys thanks will add the screenshots


--password-prompt asks for password too early in Bolt execution

Describe the Bug
When using the --password-prompt flag to prompt for the users transport password, Bolt seemingly prompts for this before doing any other initialization steps. This often means the user will enter their password, and Bolt may exit unsuccessfully because a plan/task is not found, or parameters are incorrect/missing, or there's parsing errors in plans.
Expected Behavior
Bolt should only prompt for a user's password after initial validation is done.
Steps to Reproduce
Steps to reproduce the behavior:

Run a bolt plan/task with the --password-prompt flag.

Environment

Version: 3.0.0
Platform: RHEL 8

I agree this is a better experience but will require a significant refactor to how the --password-prompt option is implemented. If you frequently run bolt with password prompt you can workaround this behavior by creating the following inventory.yaml file in your project.
---
config:
  ssh:
    password:
      _plugin: prompt
      message: "Please enter your password:"

Thanks @adreyer, sad to hear it's a tough one to fix but the workaround achieves exactly the behavior I was after. Cheers for the tip!


Prolem with local running

Good morning
I tried to run the application using my own collection with my own set of documents, however, when launching the command npm start It seems every thing is going good but I got this error message
Unhandled rejection TypeError: Cannot read property 'length' of undefined
at C:\Users\user\Desktop\Projet FFA\watson-discovery-ui\lib\utils.js:133:41

Hello @SBSsbs
While this code pattern tries to be somewhat generic, it does require the underlying dataset to have specific fields. At a minimum, it requires: text, id, date, and title.
That error indicates that a text field does not exist in the data. Please look at one of the json files found in the data/airbnb directory for an example of the data used to feed the default discovery collection.


add filter to connectQuery

make connection target optional by filter pls

Done https://github.com/igorkamyshev/farfetched/pull/425 and will be released in 0.12


feature: 云区域列表增加可用 Proxy 数量信息 (#1083)

Codecov Report

Merging #1108 (6d2f8ad) into V2.2.23-rc (3873344) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@              Coverage Diff               @@
##           V2.2.23-rc    #1108      +/-   ##
==============================================
+ Coverage       73.21%   73.23%   +0.01%
==============================================
  Files             322      322
  Lines           22978    22983       +5
==============================================
+ Hits            16823    16831       +8
+ Misses           6155     6152       -3

Impacted Files
Coverage Δ

apps/node_man/handlers/cloud.py
93.04% <100.00%> (+0.31%)
:arrow_up:

apps/node_man/handlers/job.py
84.84% <0.00%> (-0.31%)
:arrow_down:

...nents/collections/agent_new/choose_access_point.py
95.60% <0.00%> (+0.48%)
:arrow_up:

apps/backend/subscription/steps/plugin.py
55.86% <0.00%> (+0.65%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


remove some of the python2 compat and update syntax

is there a reason not to simply add the pyupgrade hook?

https://github.com/python-poetry/poetry/pull/4758 adds the hook for poetry, so i imagine we should do the same here

python-poetry/poetry#4758 adds the hook for poetry, so i imagine we should do the same here

The intention is to copy the automation from poetry once I've got it dialled in to everyone's satisfaction. Still, a PR adding pyupgrade would not be unwelcome -- I just plan to replace the .pre-commit-config.yaml in this repo in a day or two.


Update setting-up-the-meeting-migration-service-mms.md

Missing a word there.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@tonysmit PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


remove yt channel info

@ChunkyProgrammer any suggestions on making it smaller?
https://github.com/FreeTubeApp/FreeTube-Docs/pull/84#discussion_r1085441445

Maybe just say "Local API"

isn't it weird to mention Local API as functionality for the Local API?

We can probably remove the "functionality" part and remove videojs-vtt-thumbnails-freetube (youtube.js is the only module that should be displayed for local api)


docs(readme): Update to list add-prop

Proposed behaviour
A link to the add-prop codemod in the list.
Current behaviour
No link exists at present
Checklist

[x] Release notes (Conventional Commits)
~- [ ] Unit tests~
[x] Readme updated

Additional context
This was missed from the recent PR to add the add-prop codemod
Testing instructions
View the README.md file in Github and click the link. If it works it's tested.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Incorrect highlighting of some Sass’ @-directives

Code examples are based on Sass directives documentation.

Reference code for testing
// @import
@import "foo.scss";

// @media
@media (orientation: landscape) {}

// @extend
.seriousError {
  @extend .error;
}

// @at-root
.parent {
  @at-root .child {}
}

// @debug
@debug 10em + 12em;

// @warn
@mixin adjust-location($x, $y) {
  @if unitless($x) {
    @warn "Assuming #{$x} to be in pixels";
  }
}

// @error
@mixin adjust-location($x, $y) {
  @if unitless($x) {
    @error "$x may not be unitless, was #{$x}.";
  }
}

// @if
p {
  @if 1 + 1 == 2 {}
}

// @for
@for $i from 1 through 3 {}

// @each (list items)
@each $animal in puma, sea-slug, egret, salamander {}

// @while
@while 6 > 0 {}

// @mixin
@mixin large-text {}

// @include
.page-title {
  @include large-text;
}

// @function and @return
@function grid-width($n) {
  @return $n * 40px + ($n - 1) * 10px;
}

Above code reference includes all tested code, below screenshots include only problematic code.

I think it's mostly ok now.  Also, dashes in unquoted strings are a headache. I'm sure I can figure it out, but I'm waiting for the right inspiration to come to me 😎

Looks good on master now.


you forgor

wtf whydid you put anchor tags in a meta tag attribute

this still wont fix it

because tk_game_description is for the meta tag and the p tag

and the anchor tag has double quotes that are unescaped inside double quotes

ok but where is the tk_game_description here

config.csv

what

what

what does it have to do with the index.html file

its being generated

ok wait l.et me take a look

lemme fix it

you should be able to do a better job 🥶


Probook 4440s - No Audio Device, Webcam inoperative & BT dicey Monterey

Hi Chris...thanks for the huge amount of work you performed in getting Monterey 12.6.1 working on Probooks. I've got it installed and working. I've updated to the latest OC & Kexts using OC tools thinking it was the problem but have a few issues and I was hoping you could assist. First, I'm a total newb at this. I have no audio and there's no audio device found in the system report. BT works but has difficulty connecting to mice - it's hit or miss. Also, the webcam turns on but no video shows up on screen in facetime or photo booth. Attached is my config.plist and installer log. Thanks in advance for your help.
Installer Log 6-Dec-2022.txt
config.zip

Why Updated OC and kext
Revert to the working EFI
All is good in DSDT patching
Compilation complete. 0 Errors, 5 Warnings, 49 Remarks, 1759 Optimizations
Dec  6 12:22:46 Steves-MacBook-Pro package_script_service[1263]: ./postinstall: Done

Hi Chris… Thanks for the response. I updated the OC and text because the original EFI was not working and had the issues identified here. The problems still remained after the update. I’ve wiped the drive and cleared the NVRAM now twice with the same and results. What am I missing?
On Wednesday, December 7, 2022, 6:08 AM, chris1111 @.***> wrote:
Why Updated OC and kext
Revert to the working EFI
All is good in DSDT patching
Compilation complete. 0 Errors, 5 Warnings, 49 Remarks, 1759 Optimizations
Dec  6 12:22:46 Steves-MacBook-Pro package_script_service[1263]: ./postinstall: Done
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

For Audio try
alcid=13

OK…thank you. Audio now working. Any suggestions for the webcam (recognized but no video) and Bluetooth (recognized but inconsistent)?

On Dec 7, 2022, at 6:38 AM, chris1111 @.***> wrote:
For Audio try
alcid=13
—
Reply to this email directly, view it on GitHub https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1340842301, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57SZ7RR65GKSWPCL4DTWMBZKHANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you authored the thread.

For the Web Cam and other App you need change the SIP
Try  00080000
Make sure Clear NVRAM openCore

Thanks again for the help. Can you say more about how and where to change the SIP?

On Dec 7, 2022, at 10:32 AM, chris1111 @.***> wrote:
For the Web Cam and other App you need change the SIP
Try 00080000
Make sure Clear NVRAM openCore
—
Reply to this email directly, view it on GitHub https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1341141234, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57SAZXZ3FLG5PNUQG53WMCUZTANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you authored the thread.

config.plist under NVRAM/Add/7C436110-AB2A-4BBB-A880-FE41995C9F82
Change the value under <           >

OK…did that and cleared NVRAM at reboot however no change in webcam…still no video

On Dec 7, 2022, at 10:57 AM, chris1111 @.***> wrote:
config.plist under NVRAM/Add/7C436110-AB2A-4BBB-A880-FE41995C9F82
Change the value under < >
https://user-images.githubusercontent.com/6248794/206227588-b4254ede-cc7a-4ef8-803a-a022d9b62be3.png
—
Reply to this email directly, view it on GitHub https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1341175583, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57QZBUOZLRKYRAKBC7DWMCXVTANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you authored the thread.

Type in terminal
csrutil status
Post picture of the input

Last login: Wed Dec  7 11:15:54 on ttys000
@.*** ~ % csrutil status
System Integrity Protection status: enabled.
@.*** ~ %

On Dec 7, 2022, at 10:57 AM, chris1111 @.***> wrote:
config.plist under NVRAM/Add/7C436110-AB2A-4BBB-A880-FE41995C9F82
Change the value under < >
https://user-images.githubusercontent.com/6248794/206227588-b4254ede-cc7a-4ef8-803a-a022d9b62be3.png
—
Reply to this email directly, view it on GitHub https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1341175583, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57QZBUOZLRKYRAKBC7DWMCXVTANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you authored the thread.

this is not a picture anyway its not good

Just donated $50 CAD. Thanks for your help so far.
…
On Dec 7, 2022, at 10:57 AM, chris1111 @.***> wrote: config.plist under NVRAM/Add/7C436110-AB2A-4BBB-A880-FE41995C9F82 Change the value under < > https://user-images.githubusercontent.com/6248794/206227588-b4254ede-cc7a-4ef8-803a-a022d9b62be3.png — Reply to this email directly, view it on GitHub <#159 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57QZBUOZLRKYRAKBC7DWMCXVTANCNFSM6AAAAAASWAMH2Y. You are receiving this because you authored the thread.

Thanks for donation and support

send me  system report about this mac in ZIP file also config.plist

FYI…this Probook is a 2-SSD drive dual boot with Windows 10

On Dec 7, 2022, at 11:56 AM, Steve Mezey @.***> wrote:

On Dec 7, 2022, at 11:45 AM, Steve Mezey @.***> wrote:

Press F10 at OpenCore boot screen
Chek In EFI folder you have a screen shot of boot screen
Post Picture here

Hi Chris…just checking back in with you. Do you need anything else from me?

On Dec 7, 2022, at 12:09 PM, chris1111 @.***> wrote:
Press F10 at OpenCore boot screen
Chek In EFI folder you have a screen shot of boot screen
Post Picture here
—
Reply to this email directly, view it on GitHub https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1341292323, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57RI52FZAUI7HHQJXS3WMDADBANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you authored the thread.

Hi Chris…just checking back in with you. Do you need anything else from me?

On Dec 7, 2022, at 12:09 PM, chris1111 @.***> wrote:
Press F10 at OpenCore boot screen
Chek In EFI folder you have a screen shot of boot screen
Post Picture here
—
Reply to this email directly, view it on GitHub https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1341292323, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57RI52FZAUI7HHQJXS3WMDADBANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you authored the thread.

Yea
Send me System report on About this mac in ZIP file also config.plist

Hi Chris…just making sure you received the system report & config

On Dec 8, 2022, at 10:22 AM, chris1111 @.***> wrote:
Hi Chris…just checking back in with you. Do you need anything else from me?
On Dec 7, 2022, at 12:09 PM, chris1111 @.***> wrote:
Press F10 at OpenCore boot screen
Chek In EFI folder you have a screen shot of boot screen
Post Picture here
—
Reply to this email directly, view it on GitHub #159 (comment) https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1341292323, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57RI52FZAUI7HHQJXS3WMDADBANCNFSM6AAAAAASWAMH2Y https://github.com/notifications/unsubscribe-auth/A4TK57RI52FZAUI7HHQJXS3WMDADBANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you authored the thread.
Yea
Send me System report on About this mac in ZIP file also config.plist
—
Reply to this email directly, view it on GitHub https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1342895341, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57XDCNWBDSD7DJHFKIDWMH4LLANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you authored the thread.

I sent them in email at 10:31am in response to your request. Did you receive them?

On Dec 8, 2022, at 10:22 AM, chris1111 @.***> wrote:
Hi Chris…just checking back in with you. Do you need anything else from me?
On Dec 7, 2022, at 12:09 PM, chris1111 @.***> wrote:
Press F10 at OpenCore boot screen
Chek In EFI folder you have a screen shot of boot screen
Post Picture here
—
Reply to this email directly, view it on GitHub #159 (comment) https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1341292323, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57RI52FZAUI7HHQJXS3WMDADBANCNFSM6AAAAAASWAMH2Y https://github.com/notifications/unsubscribe-auth/A4TK57RI52FZAUI7HHQJXS3WMDADBANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you authored the thread.
Yea
Send me System report on About this mac in ZIP file also config.plist
—
Reply to this email directly, view it on GitHub https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1342895341, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57XDCNWBDSD7DJHFKIDWMH4LLANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you authored the thread.

Attached to the post

On Dec 8, 2022, at 3:00 PM, chris1111 @.***> wrote:
How can I recive if your not attach the file to the post 🤣
—
Reply to this email directly, view it on GitHub https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1343282745, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57V73RH2EUEX34XSNKLWMI47RANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you authored the thread.

MacBook Pro.zip
config.plist.zip

Everything is recognise, all looks perfect
Whats the problem persist now

Open FaceTime or Photo Booth, camera recognized, camera light turns on but no video just black screen. Try to add a mouse via Bluetooth it shows up, click connect and it never connects (spinner spins). Tried multiple mice.

On Dec 8, 2022, at 3:30 PM, chris1111 @.***> wrote:
Everything is recognise, all looks perfect
Whats the problem persist now
—
Reply to this email directly, view it on GitHub https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1343316560, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57TKYOFYRNLKCH6QY6TWMJAO3ANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you commented.

What is the exact model of Wifi Card you have? Combo Wifi Bluetooth?

Yes combo…attached images from Windows 10 device manager

On Dec 8, 2022, at 3:40 PM, chris1111 @.***> wrote:
What is the exact model of Wifi Card you have? Combo Wifi Bluetooth?
—
Reply to this email directly, view it on GitHub https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1343326536, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57WBKWSLQ5MYSUZVFWTWMJBSBANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you commented.

Image?

Ok for fixing Bluetooth issue Disabled Bluethoot in Bios
Use this adapter
Kinivo-BTD-400-Bluetooth-4-0

For Camera FaceTime you need to connect to Apple (Sign to Apple)

I have been signed into Apple all this time. Camera not working. Wi-Fi/BT device is a Dell Wireless 1550 made by Broadcom. Not in system report sent? Broadcom BCM43xx 1.0 (<IP_ADDRESS> AirPortDriverBrcmNIC-1710.4)

On Dec 8, 2022, at 3:59 PM, chris1111 @.***> wrote:
Ok for fixing Bluetooth issue Disabled Bluethoot in Bios
Use this adapter
Kinivo-BTD-400-Bluetooth-4-0 https://www.amazon.ca/Kinivo-BTD-400-Bluetooth-4-0-USB/dp/B007Q45EF4/ref=sr_1_3?gclid=CjwKCAiAs8acBhA1EiwAgRFdw7bPnudMUbbcSu97gN5beZOGl_WLLNadmAU-sRftk0Lko78HsACcCRoCDXsQAvD_BwE&hvadid=596009754395&hvdev=c&hvlocphy=9000468&hvnetw=g&hvqmt=e&hvrand=3115618508989228577&hvtargid=kwd-299111161166&hydadcr=1503_10289859&keywords=kinivo+bluetooth&qid=1670532862&s=electronics&sr=1-3
https://user-images.githubusercontent.com/6248794/206565865-e70e1eb4-eff5-4526-88c1-2d39bb44b115.jpg
For Camera FaceTime you need to connect to Apple (Sign to Apple)
—
Reply to this email directly, view it on GitHub https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1343344117, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57Q3JNDQ5JB4AULCEDLWMJDZNANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you commented.

Worked in El Capitan
http://en.techinfodepot.shoutwiki.com/wiki/Dell_Wireless_1550_(DW1550) http://en.techinfodepot.shoutwiki.com/wiki/Dell_Wireless_1550_(DW1550)

On Dec 8, 2022, at 4:07 PM, Steve Mezey @.***> wrote:
I have been signed into Apple all this time. Camera not working. Wi-Fi/BT device is a Dell Wireless 1550 made by Broadcom. Not in system report sent? Broadcom BCM43xx 1.0 (<IP_ADDRESS> AirPortDriverBrcmNIC-1710.4)

On Dec 8, 2022, at 3:59 PM, chris1111 @.*** @.***>> wrote:
Ok for fixing Bluetooth issue Disabled Bluethoot in Bios
Use this adapter
Kinivo-BTD-400-Bluetooth-4-0 https://www.amazon.ca/Kinivo-BTD-400-Bluetooth-4-0-USB/dp/B007Q45EF4/ref=sr_1_3?gclid=CjwKCAiAs8acBhA1EiwAgRFdw7bPnudMUbbcSu97gN5beZOGl_WLLNadmAU-sRftk0Lko78HsACcCRoCDXsQAvD_BwE&hvadid=596009754395&hvdev=c&hvlocphy=9000468&hvnetw=g&hvqmt=e&hvrand=3115618508989228577&hvtargid=kwd-299111161166&hydadcr=1503_10289859&keywords=kinivo+bluetooth&qid=1670532862&s=electronics&sr=1-3
https://user-images.githubusercontent.com/6248794/206565865-e70e1eb4-eff5-4526-88c1-2d39bb44b115.jpg
For Camera FaceTime you need to connect to Apple (Sign to Apple)
—
Reply to this email directly, view it on GitHub https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC/issues/159#issuecomment-1343344117, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4TK57Q3JNDQ5JB4AULCEDLWMJDZNANCNFSM6AAAAAASWAMH2Y.
You are receiving this because you commented.

Yea but everythings change in macOS 11, 12, 13
For the Camera its really strange, I can not testing on macOS 12.6.1 with HD 4000 because I no longer have my HP Probook 6570B Intel HD 4000
You can try Big Sur 11 then not need using Patch HD 400O  HD 4000 is native

It is a BCM4352 chipset; was supported in El Capitan with DSDT/Brcm4360 patch. Does that help?

Wifi from this card as no probleme with AirportBrcmFixup.kext the probleme is Bluethoot I have the same issue with my G50 G1 with Bluethoot, that why I told you for fixing Bluetooth issue Disabled Bluethoot in Bios
Use this adapter
Kinivo-BTD-400-Bluetooth-4-0

OK…thanks for your help. I can live without the Bluetooth…just wish there was a fix for the camera.
Steve Mezey
On Thursday, December 8, 2022, 5:52 PM, chris1111 @.***> wrote:
It is a BCM4352 chipset; was supported in El Capitan with DSDT/Brcm4360 patch. Does that help?
Wifi from this card as no probleme with AirportBrcmFixup.kext the probleme is Bluethoot I have the same issue with my G50 G1 with Bluethoot, that why I told you for fixing Bluetooth issue Disabled Bluethoot in Bios
Use this adapter
Kinivo-BTD-400-Bluetooth-4-0
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


[.macos] Hide battery percentage (com.apple.menuextra.battery user defaults)

In reply to this commit, it is still possible to hide battery percentage [at least on OS X 10.9.5 Mavericks]. However, display remaining battery time has been removed by Apple.
defaults write com.apple.menuextra.battery ShowPercent -bool false # Hide battery percentage.

This is working on my machine running 10.12.5 (Sierra).
defaults write com.apple.menuextra.battery ShowPercent YES # Show battery percentage

The correct values to be set on High Sierra are YES and NO:
defaults write com.apple.menuextra.battery ShowPercent -string "YES"
defaults write com.apple.menuextra.battery ShowPercent -string "NO"
I'm not sure since which macOS version, though.

what @kvpb posted does not work with OS X 10.11.6 El Capitan, at least.

Works with macOS 10.15.7, remember to killall SystemUIServer for it to take effect.

In macOS Monterey, we should use the following commands to show the battery percentage:
defaults write ~/Library/Preferences/ByHost/com.apple.controlcenter.plist BatteryShowPercentage -bool true

defaults write ~/Library/Preferences/ByHost/com.apple.controlcenter.plist BatteryShowPercentage -bool false

It is not necessary to reset any module/application to see the changes.

You can just use -curentHost
defaults -currentHost write com.apple.controlcenter.plist BatteryShowPercentage -bool true