[$500] Web - Workspace - Inconsistent tooltip for workspace name in thread and IOU

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

Version Number: v1.4.3-6
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation: @
Action Performed:

Go to Settings > Workspaces > New Workspace.
Change the workspace name if you want
Go to the workspace expense room.
Send a message.
Go to 'Request money.'
Enter the amount and send it.
Go to the workspace setting and delete it.
Click on IOU and hover over the workspace name at the header.
Go back, hover over the message, and click on 'Reply In Thread'
Hover over the workspace name at the header.
Notice inconsistency between workplace name tooltips.

Expected Result:
The workspace name tooltip should be the same for thread header and IOU.
Actual Result:
Inconsistency in the thread workspace name tooltip and IOU workspace name tooltip
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/65b30c91-6743-4514-888f-6b4dd5fea075

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~010c1c53f77916777c
Upwork Job ID:<PHONE_NUMBER>581572608
Last Price Increase: 2023-11-26

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01e5bcae9f03690137

Please re-state the problem that we are trying to solve in this issue.
Web - Workspace - Inconsistent tooltip for workspace name in thread and IOU
What is the root cause of that problem?
The function getPolicyName() returns "Unavailable workspace policy" when the workspace name is not available. This is happening when we delete the workspace.
What changes do you think we should make in order to solve the problem?
If the workspace name (report.policyName or report.oldPolicyName) is not available, the function will return an empty string or hidden. If the workspace name is available, it will display the workspace name accordingly.
or
When a thread is opened after the workspace has been archived, set the oldPolicyName to the name of the workspace before deletion. Ensure this change is reflected both optimistically (client-side) and on the backend.

C+ reviewed 🎀 👀 🎀
I approve the proposal by @chie2727. In my opinion, obtaining the archived workspace name from the parent-report is a cleaner and more normalized approach in this scenario.

Thank you very much.
I will submit a PR as soon as my proposal is accepted on Upwork.

@cubuspl42 Thanks for letting me know. The PR is ready for review: https://github.com/Expensify/App/pull/32325

Hi @cubuspl42, will the PR be merged soon?

@chie2727
😳
I don't know what happened here, I'll comment on the PR.

@dukenv0307
Interesting to hear your insight. If you have a look at my PR (https://github.com/Expensify/App/pull/32325/files#diff-577fcc5f3a5e4930916ff310e61c38edfa093792fe3ab55b9a30d2aecd8811dbR507) you can see that my proposal is not to make the thread report rely on the root parent report, but instead to add the root parent report to the search condition for finding the policy name. So in the case that the policy name is not present in the policy object or the report object, it can be obtained from the parent report object instead.

@dukenv0307 Thanks for sharing your findings.
It's unlikely that we change the assignment at this stage.
How many reports does one need to have to trigger paginating?

What do you think?

I'll test this solution on my most report-heavy account (most likely heavier than any real customer). I'll see if I can observe any problems!

I'm good too work together

I don't think the Expensify process has room for such a step.

It's definitely not "normal" or "common", but I'm not saying there was no precedence. Still, we're assuming that the solution being reviewed right can trigger a bug, although I havent't yet reproduced it or seen it reproduced (or maybe produced would be a more suitable term?).

Looks like https://github.com/Expensify/App/pull/32325 was deployed to prod 1/9 and the automation just didn't trigger. I'll update the labels/title accordingly.

Thanks @amyevans, reminder set to pay on the 16th!

Paid and contract ended for @cubuspl42
Offer sent to @chie2727, will pay out once accepted! 🙇

Paid and contract ended!  Thanks all


fix:test

@dead-horse

Coverage remained the same at 76.19% when pulling 1b5e4a40cf7897116a3e33678dbe35441cdde541 on ngot:fix_test into b5c9333da80040c69f714c596c7a78d81fb58b9e on xtemplate:master.

ci 没过，代码风格统一

Coverage remained the same at 76.19% when pulling 2fe86e2ca597e8efdee47f728814271400a1b208 on ngot:fix_test into b5c9333da80040c69f714c596c7a78d81fb58b9e on xtemplate:master.

@dead-horse 可以合吧？

Coverage increased (+3.5%) to 79.73% when pulling e813c9e44c4af9a2fd0bc124719bdcc8541e4067 on ngot:fix_test into b5c9333da80040c69f714c596c7a78d81fb58b9e on xtemplate:master.

@dead-horse 升级了一些依赖还有多个node版本的CI配置

Coverage increased (+3.5%) to 79.73% when pulling ab20609c396f8e0e1e23f0ef7f4fe97305a9b70b on ngot:fix_test into b5c9333da80040c69f714c596c7a78d81fb58b9e on xtemplate:master.


Automatic PR from RestAPI-PR1900

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/1900

Initial PR not merged in current

Codecov Report

Merging #1576 into master will increase coverage by 0.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1576      +/-   ##
==========================================
+ Coverage   53.84%   54.01%   +0.17%
==========================================
  Files        3633     3632       -1
  Lines       90090    89720     -370
==========================================
- Hits        48506    48463      -43
+ Misses      41584    41257     -327

Impacted Files
Coverage Δ

.../virtual_network_gateway_connections_operations.py
10.91% <ø> (+0.96%)
:arrow_up:

...9_01/operations/application_gateways_operations.py
9.05% <ø> (+0.49%)
:arrow_up:

...etwork/v2017_09_01/operations/usages_operations.py
93.75% <ø> (ø)
:arrow_up:

...17_09_01/operations/network_watchers_operations.py
6.69% <ø> (+0.07%)
:arrow_up:

...01/operations/express_route_circuits_operations.py
67.24% <ø> (+6.62%)
:arrow_up:

...1/operations/network_security_groups_operations.py
95.93% <ø> (+16.6%)
:arrow_up:

.../azure/mgmt/network/v2017_09_01/models/__init__.py
100% <ø> (ø)
:arrow_up:

...01/operations/local_network_gateways_operations.py
16% <ø> (+2.61%)
:arrow_up:

.../v2017_09_01/operations/route_tables_operations.py
95.93% <ø> (+16.6%)
:arrow_up:

..._09_01/operations/network_interfaces_operations.py
52.04% <ø> (+11.03%)
:arrow_up:

... and 14 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c9ecc1...bd08345. Read the comment docs.


Customer feedback - issues with KBUGCHECK_REASON_CALLBACK_ROUTINE

Type of issue
Other (describe below)
Feedback
Hi Barry, thank you for sharing this document—it's very helpful.
I am currently working on a driver for a device that monitors system state primarily by using the KBUGCHECK_REASON_CALLBACK_ROUTINE callback to report when the system issues a bug check and to provide a crash dump for analysis. However, I encountered unexpected behavior when attempting to write crash dump data to devices other than the disk by implementing the KbCallbackDumpIo routine.
According to the documentation, when the system issues a bug check, it will call the registered KBUGCHECK_REASON_CALLBACK_ROUTINE multiple times with a KBUGCHECK_DUMP_IO structure in this case. The data inside should eventually form a complete dump file. In my implementation, each time the system calls the bug check function, it transfers the buffer in the input parameter ReasonSpecificData (which, in this situation, should be a KBUGCHECK_DUMP_IO struct type) to the device. However, after the system completes the entire bug check process, only mini dumps are in a valid format (containing correct information), while core dumps and full dumps, although appearing to have correct file sizes, are actually invalid files that cannot be used.
Two details may be relevant:

When I print out the addresses of the buffers (referring to three parts of the dump file), only a small part of the address for the body type buffer appears valid. Starting from a certain buffer, all addresses for the body type buffer become 0x00.
The data read from address 0x00 is consistently 0xff, indicating that no useful and correct information is being written to the dump file.

I also used binary comparison software to compare the dump file I wrote to the device with the corresponding dump file on the hard drive generated by the system. Aside from a small amount of matching header and secondary information, the rest of the information was incorrect.
Could there be an issue with the usage of KBUGCHECK_REASON_CALLBACK_ROUTINE, or are there system limitations during bug check that are not mentioned in the documentation?
Page URL
https://learn.microsoft.com/en-us/windows-hardware/drivers/ddi/wdm/nc-wdm-kbugcheck_reason_callback_routine
Content source URL
https://github.com/MicrosoftDocs/windows-driver-docs-ddi/blob/staging/wdk-ddi-src/content/wdm/nc-wdm-kbugcheck_reason_callback_routine.md
Author
@barrygolden

@zephyr-fun, thanks for reaching out to the Windows Driver Developer Documentation team.
Unfortunately, we are a small team and do not have the resources to provide technical support to individual driver developers.
You may have success posting your question in the following forums, where other driver developers and Microsoft engineers can be found.
Microsoft Q&A: Windows Hardware development
OSR Developer Community


Recent profiles

Added the recent profile generations section just below the hero section. I kindly request a review of my code and appreciate any further suggestions for improvement.
(GSSOC'24)

https://github.com/ashutosh-rath02/git-re/assets/107497296/f62f9e0b-8bc9-4fe4-99c8-aeac78eb6740
Here is the video of the section!

@RoopsagarK Please remove the changes in all-contributorsrc . I will do those changes. Can you please contact me over discord? Have a few things to discuss😊. My username .vasudev

I cannot see the component in the preview. I also have a few more suggestions please reach out to me on discord.

@ashutosh-rath02. I've sent you a message on Discord. Could you please check it

LGTM.🚀

@all-contributors please add @RoopsagarK  for code


ModuleNotFoundError: No module named 'llama'

I am trying to export the llama weight to the directory of llama2.c but once I have initialize the script I got error
Traceback (most recent call last):   File "/Users/username/ai/llama2.c/export_meta_llama_bin.py", line 11, in <module>     from llama import Llama ModuleNotFoundError: No module named 'llama'

do you have the latest version of the code?

I updated the code and it worked. Thanks


cmd/cgo: pass explicit -O0 to compiler

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.10.3 linux/amd64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOOS="linux"
What did you do?
On Solus Linux
git clone https://github.com/golang/go
cd go/src/
git checkout go1.10.3
./all.bash

What did you expect to see?
All tests passing
What did you see instead?
# _/home/wolf/src/go/misc/cgo/test
cgo-dwarf-inference:2:8: error: enumerator value for '__cgo_enum__0' is not an integer constant
cgo-dwarf-inference:4:2: error: initializer element is not constant
cgo-dwarf-inference:4:2: note: (near initialization for '__cgodebug_ints[0]')
FAIL	_/home/wolf/src/go/misc/cgo/test [build failed]
2018/07/19 16:57:27 Failed: exit status 2

The reason can be found in this patch
Quoting from that

When the go compiler uses cgo, removes all
C compilation optimization flags,
See commit: b5d18b50ac591d41.
This is because optimization flags
breaks c code build process, see:
https://github.com/golang/go/issues/14669
cgo creates invalid C code that works
with -O0 using gcc. The code it generates is valid C++
just not valid C.
In Clear Linux gcc compiler uses at least
-O1 optimization flag (if not optimization is specified), this cause an
issue when cgo is used. The go compiler removes any optimization flag
(even -O0 that should work
correctly) letting gcc use -O1 as default.

Since Solus and Clear Linux patch Go in their distribution, normal go users don't run into this problem, but anyone on those distros who tries to compile go from source runs into this problem.
I don't see any disadvantage to upstreaming this patch.  Although go shouldn't be expected to play nicely with any old arbitrarily patched gcc, being explicit about the -O0 requirement instead of implicitly assuming gcc's default seems better all around.  Making it easier for Solus and Clear Linux users to build (and contribute to) go makes this a win/win IMO.
Tagging the original author of this patch @jcvenegas in case he wants to submit this and himself and get due credit.

Because of copyright reasons I don't particularly want to look at a patch that has not gone through the review process.  If the original author wants to send in the patch, that would be great.
Otherwise, I take the problem to be that on some Linux distros the compiler defaults to -O1, but the code generated by cgo only works correctly with -O0.  Therefore, we should explicitly pass -O0.  For #14669 we explicitly removed -O options, but for these distros we need to explicitly add -O0.  That seems fine.

Yes, you've summarized the problem perfectly.  Hopefully @jcvenegas will chime in.

Do we know which distros this happens on? Is there a rule to check by?

@oryanmoshe We don't need to check, we can just always pass -O0.

Do we want to wait for the original author to submit a CL or can I do it? @ianlancetaylor

@oryanmoshe We haven't seen any response from @jcvenegas, so, sure, go for it (for 1.12).  Thanks.

I'm on it

@oryanmoshe  @ianlancetaylor , sorry I miss this issue with a bunch of github notifications. Thanks for add the patch will make easy to work with some distros.


matriarch@: js: can't find file createGains.js

get this error, when loading matriarch@:
js: can't find file createGains.js

and btw, matrix@ can do stereo as well, just pushed a minor bug there..

that file won't effect the device functionality (it was just automation for creating all the knobs are coordinates).  it will be removed when I push the latest matriarch (to-do for me!) thanks klaus


isMetadataEndpointAccessible is not reliable

isMetadataEndpointAccessible attempts to check we can actually reach the GCE server.
We use this along with metada.OnGCE
Logic:
OnGCE: first one wins of

"curl <IP_ADDRESS>" has Metadata-Flavor==Google
dig metadata.google.internal contains <IP_ADDRESS>

We add isMetadataEndpointAccessible which does a simple curl of metadata.google.internal and sees it is reachable. This doesn't seem reliable

The actual requests use <IP_ADDRESS>, so reachability to that is all the is relevant
DNS lookup of  metadata.google.internal is unsafe, see https://github.com/istio/istio/issues/42449 and https://github.com/googleapis/google-cloud-go/issues/7165

cc @rinormaloku @stewartbutler


Invalid Cookie Cause Errors

Cookies are decrypted based on whether or not their value starts with bkmk|. It is possible to abuse this if a cookie is set by some malicious entity. In the event that a cookie starts with bkmk| but it's value is not able to be decrypted, Gatekeeper will error. We should add in some kind of error handling as well as logging to fix this.

Added additional logging to failure to decrypt cookies. get_cookie should be modified to return None if a cookie does not exist. If a cookie does exist but cannot be decrypted then we should return False. In the event that False is returned then we should log the incident and then clear the invalid cookie.


Consider archiving tools/libs

It's always been a hassle to handle libncurses5 used in the GDB build of the ARM(R) embedded toolchains.
Consider archiving (and publishing in a release) a consistent set of:

gcc-arm-none-eabi (with GDB)
libtinfo5
libncursesw5

See also Ubuntu pool ncurses, as used in some CI runs of mine at the time of writing this issue.

Release v1.0 has been created. Now use this release in its own repo's CI to build/run QEMU.

See also #15


Raises warning instead of exception on empty Kafka message

Skips attempting to deserialize an empty Kafka message and raises a suppressible warning instead. This is useful for cases such as consuming CDC row updates from kSQL where keyed messages with a null value indicate no change to the row.

hey, I'm going to close this out for now since it sounds like we should let the application do this. Please feel free to re-open if you think we should do it here or something else has changed!


foggy Cityscapes preparation

Could you show us the details about foggy Cityscapes preparation? Is that same as the GoogleDrive mentioned in the link (https://github.com/tiancity-NJU/da-faster-rcnn-PyTorch)?

I got it.

Hey @kadebudongtan , I'm getting "Warning: NaN or Inf found in input tensor." and loss becomes NaN after one to two epochs while training from Cityscapes to Foggy Cityscapes. Did you experience this

Hey, @benedictflorance, I also meet the question of Nan, Have you solved this problem?


Install Liberica 11 on Solaris systems (for jenkins agent)

Ref: https://github.com/adoptium/infrastructure/issues/2763
Fixes: https://github.com/adoptium/infrastructure/issues/2915
Prereq to next jenkins LTS upgrade as part of #2108

Checklist

[x] commit message has one of the standard prefixes
[ ] playbook changes run through VPC or QPC (if you have access)

Manyh VagrantPlaybookCheck runs later it's looking good: https://ci.adoptium.net/job/VagrantPlaybookCheck/1625/

VPC job above has a failure in SystemMetrics but that is a break in the codebase and not related to this PR - seen on AIX too: https://github.com/adoptium/temurin-build/issues/3261#issuecomment-1443358274 so that does not block this PR.


Remove support for Fedora 36

mssql-server package does not support Fedora at all but it works on Fedora <36. Once Microsoft adds mssql-server package for RHEL 9 it should work on Fedora 36 too.

[citest]


How to import and export the templates

How to import and export the templates

For now there is no such feature

Is the updated template.xml stored somewhere? I searched in all IntelliJ folders known to me and could not find anything related.


conform Unsafe(Mutable)RawBufferPointer conform to _HasContiguousBytes

(cherry picked from commit f87ffd20772f6aa7226901037beb55858fbda736)
backport of #21611 to swift-5.0-branch

@swift-ci please test


Do not skip closing parentheses if syntax error is present.

monaco-editor version: 3.1.0
**Browser: Chrome **
OS: Windows 10
Steps or JS usage snippet reproducing the issue:
Write javascript;
var funk = "some stack";
var str = ("some string" + func && (" some other string" + func))

Press ( in front of func &&
var func = "some stack";
var str = ("some string" + (func && (" some other string" + func))

Move to end of second set of parentheses (" some other string" + func)
Press ).
Commit your code, watch as validation doesn't catch it until your CI board goes red with a runtime syntax error.

Fixed via https://github.com/microsoft/vscode/issues/37315


[RocketMQ Binder] Support anonymous consumer group when it is empty

Which Component
RocketMQ Binder
Is your feature request related to a problem? Please describe.
Now RocketMQ Binder will check consumer group property, it will throw RuntimeException when group is empty.
Describe the solution you'd like
When consumer group is empty, using anonymous consumer group like rmq.UUID.

anonymous consumer group means the offset should be last.
related https://github.com/spring-cloud-incubator/spring-cloud-alibaba/issues/488.


ISPN-10855 Add missing factory() implementations in query tests

https://issues.jboss.org/browse/ISPN-10855

Checkstyle violations, @danberindei . This did not run.

Updated to ignore the factory() method inherited from a super class instead of failing the test.
The original intention was to fail the test, but before #7498 the simulated test failure did not work and it was effectively ignoring the factory() method. So this is reverting to the pre- #7498 behaviour, only making it explicit.

I guess master is currently broken right? I just rebased my PR and started getting tons of
org.infinispan.query.blackbox.ClusteredCacheWithInfinispanDirectoryTest.factory() override is missing, inherited factory() creates instances of org.infinispan.query.blackbox.ClusteredCacheTest

@gustavonalle yes, this PR fixes the errors you're seeing.


Operation flags on the ConnectorInstance

From<EMAIL_ADDRESS>on September 08, 2010 15:45:30
It should be possible to specify which operation could be executed on a particular connector.
Operations to consider are:

asynchronous create
asynchronous update
asynchronous delete
synchronous create
synchronous update
synchronous delete
search
resolve
synchronization on demand
automatic synchronization

PropagationManager must evaluate flags before act.
Original issue: http://code.google.com/p/syncope/issues/detail?id=43

From chicchiricco on September 09, 2010 02:26:56
Status: Started

From chicchiricco on September 10, 2010 06:12:57
This issue was closed by revision r524 .
Status: Fixed


grammar improvements in article: Manage Azure Maps Power BI visual within your organization

Per prior arrangement, if this PR needs approval, or if the commits need to be pulled into the private repo, only @eriklindeman should be notified. Thanks.

#sign-off

#sign-off

#sign-off


Picamera2-WebUI on RPI 4 with buster lite. KeyError: 'Num'

Describe the bug
8 September 2024
Running picamera2-WebUI app.py causes a KeyError: 'Num'
To Reproduce
Steps to reproduce the behavior:

Clean install of RPI Buster lite  on rpi 4 (headless device withe a Pi HQ Camera Installed);
SSH on to the device;
Update and upgrade using 'sudo apt-get update && sudo apt-get upgrade';
Clone the picanera2-WebUi Software using 'git clone https://github.com/monkeymademe/picamera2-WebUI.git;
cd into 'picamera2-WebUI' folder;
Run the picamera2-WebUI python application with python3 app.py;
See The Error

Expected behavior
I Don't know it is the first time I have tried to run it. I expect it not to have an error
Screenshots
The output of the display.
/picamera2-WebUI $ python3 app.py
[0:27:24.384876287] [7635]  INFO Camera camera_manager.cpp:297 libcamera v0.0.5+83-bde9b04f
[0:27:24.414889205] [7636]  INFO RPI vc4.cpp:437 Registered camera /base/soc/i2c0mux/i2c@1/imx477@1a to Unicam device /dev/media3 and ISP device /dev/media0
[0:27:24.414972000] [7636]  INFO RPI pipeline_base.cpp:1101 Using configuration file '/usr/share/libcamera/pipeline/rpi/vc4/rpi_apps.yaml'
~/picamera2-WebUI/static/camera_config
Detected Cameras:
[{'Model': 'imx477', 'Location': 2, 'Rotation': 0, 'Id': '/base/soc/i2c0mux/i2c@1/imx477@1a'}]
Camera Info:
{'Model': 'imx477', 'Location': 2, 'Rotation': 0, 'Id': '/base/soc/i2c0mux/i2c@1/imx477@1a'}
Traceback (most recent call last):
File "~/picamera2-WebUI/app.py", line 506, in
camera_num = camera_info['Num']
KeyError: 'Num'
end of output.

Please refer to the readme requirements section

Raspberry Pi OS / Debian
Please be aware that due to dependencies on newer versions of Picamera2 (see below) and Libcamera this project only works on Raspberry Pi OS Bookworm (or newer). Issues have been reported with older versions (e.g. Bullseye) not functioning due to libcamera no longer being updated on older versions of the Raspberry Pi OS. The recommendation, even on older Pi's, is to use Bookworm (or newer).


Implement near cache statistics and add tests

Implemented statistics for near cache.
Added tests for near cache implementation.
Did some code fixes based on test results.

Test PASSed.


Error Handling in File Reading

Hi, The current implementation does not handle errors related to file access, such as if the file doesn't exist, if it's locked, or if it's unreadable. This can lead to unhandled exceptions, crashing the program.

This is a good insight, however my scope for release 0.1 did not include that type of error handling, I will consider it for the next release.

This is a good insight, however my scope for release 0.1 did not include that type of error handling, I will consider it for the next release.


Change data source

I'm using this typeahead library: http://twitter.github.io/typeahead.js/
I managed to create a sample same as they have on their page with having local values.
$( document ).ready(function() {
     var globalResults = ["Alabama","Alaska","Arizona","Arkansas","California","Colorado","Connecticut","Delaware","Florida","Georgia","Hawaii","Idaho","Illinois","Indiana","Iowa","Kansas","Kentucky","Louisiana","Maine","Maryland","Massachusetts","Michigan","Minnesota","Mississippi","Missouri","Montana","Nebraska","Nevada","New Hampshire","New Jersey","New Mexico","New York","North Dakota","North Carolina","Ohio","Oklahoma","Oregon","Pennsylvania","Rhode Island","South Carolina","South Dakota","Tennessee","Texas","Utah","Vermont","Virginia","Washington","West Virginia","Wisconsin","Wyoming"];

     var substringMatcher = function(strs) {
       return function findMatches(q, cb) {
         var matches, substringRegex;

         // an array that will be populated with substring matches
         matches = [];

         // regex used to determine if a string contains the substring `q`
         substrRegex = new RegExp(q, 'i');

         // iterate through the pool of strings and for any string that
         // contains the substring `q`, add it to the `matches` array
         $.each(strs, function(i, str) {
           if (substrRegex.test(str)) {
             matches.push(str);
           }
         });

         cb(matches);
       };
     };

     $('#search-input').typeahead({
       hint: true,
       highlight: true,
       minLength: 1
     },
     {
       name: 'globalResults',
       source: substringMatcher(globalResults)
     });
   });

The problem is when I update the globalResults, the typeahead still shows the same old results Alambama, Alaska...
I'm trying to update them this way, but it doesn't work:
    globalResults = results;
    var autocomplete = $('#search-input').data('typeahead');
    autocomplete.source = globalResults;

Results are retrieved from a server in a string array. I debugged it and it contained the new data and it did copy them into globalResultes and updated it. But somehow this line does nothing:
    autocomplete.source = globalResults;
Where might be the problem for which the source for typeahead is not updating?
My html looks like this:
    <span style="position: relative; display: inline-block;" class="twitter-typeahead">
                <input type="text" spellcheck="true" class="form-control typeahead tt-input" placeholder="Search for science, search for data ..." id="search-input" autocomplete="off"></input>
    </span>

The only way I was able to update the datasource used is by using BloodhoundJS with TypeaheadJS.
I initialize a local source:
var globalResults = ["Alabama", "Alaska", "Arizona", "Arkansas", "California", "Colorado", "Connecticut", "Delaware", "Florida", "Georgia", "Hawaii", "Idaho", "Illinois", "Indiana", "Iowa", "Kansas", "Kentucky", "Louisiana", "Maine", "Maryland", "Massachusetts", "Michigan", "Minnesota", "Mississippi", "Missouri", "Montana", "Nebraska", "Nevada", "New Hampshire", "New Jersey", "New Mexico", "New York", "North Dakota", "North Carolina", "Ohio", "Oklahoma", "Oregon", "Pennsylvania", "Rhode Island", "South Carolina", "South Dakota", "Tennessee", "Texas", "Utah", "Vermont", "Virginia", "Washington", "West Virginia", "Wisconsin", "Wyoming"];
and an alternative local source:
var otherResults = ['one', 'two', 'three', 'four', 'five', 'six', 'seven', 'eight', 'nine', 'ten'];

I create a bloodhound instance and set it to a local data source:
    var states = new Bloodhound({
      datumTokenizer: Bloodhound.tokenizers.whitespace,
      queryTokenizer: Bloodhound.tokenizers.whitespace,
      local: globalResults
    });

I create a method to switch between the sources.  In this case I use a button and on click, make the change:
    $(document).on('click', '#changeSource', function() {
    	console.log('change the data');
      states.clear();
      states.local = otherResults;
      states.initialize(true);
    });

This will work for remote data as well.
Here is a fiddle demonstration:
https://jsfiddle.net/whipdancer/umc5bjy8/


feat: add useSuspenseAccessor

Proposed change
Since the logic is quite different when we want to use Suspense, I add a new API to achieve the goal.
This hook doesn't support streaming rendering out of box when using nextjs, we need to create another package to leverage it with nextjs.
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Testing
[ ] Refactor
[ ] Chore (tool changes, configuration changes, and changes to things that do not actually go into production at all)

Implementation
Related Issue
#182

There is a problem, we need to pass serverStateKey to accessor.revalidate such that is can update the correct state in server.

There is a problem, we need to pass serverStateKey to accessor.revalidate such that is can update the correct state in server.

fixed by 4e875e9


[Feature] API Routes for companies

Umbrella issue for all companies routes
We should have routes for creating, listing and updating companies entities.

[ ] #31
[ ] #32
[ ] #33
[ ] #34
[ ] #35
[ ] #36

Hello!
first of all, the companies route now is ready for review :tada:!
but i have a question, does we need to use prepared statement?
speaking in security, the prepared statements is a good feature to be added.


move furniture, and then pca

just making some directory structure and templates for my plane flight

yay!


Add unordered notes table for e.g. Python notes

Python notes have a book TOC right now which is numbered even though the notes don't have a meaningful ordering.

obsolete, ordered notes were removed here https://github.com/gdmen/garymenezes.com/pull/45


force all upgradeTest and integrationTest tasks to depend on installDist

Might fix issues we've seen on support/1.14 with classes not in classpath during execution. We'll run a few test iterations and see how this goes...

I ran the jobs Naba pointed out (integrationTest and upgradeTest) many times against this PR, and all passed. Not sure if I ran enough times to have confidence that this is, in fact, a fix for the sporadic class not found error.


Make sure everything is fine in Debug builds

Description of Change
Make sure everything is fine in Debug builds.

Probably an oversight of mine. If we shape a buffer or mutate it in some way we should call GuessSegmentProperties before we do that. Sry for that.

That should be done by the caller, right?

Yes. GuessSegmentProperties does not guarantee everything is configured right but will work most of the time. It can be left out if you know what you have to configure.
For example if you have a sequence of codepoints that represents RTL text GuessSegmentProperties should set the buffer's direction to RTL. You can set it explicitly or implicitly by GuessSegmentProperties. I hope this helps.
I will add additional tests that cover "real world" scenarios. Current tests mostly just make sure that the call to some function works. I didn't think too much about the data that I use.

Ah, I see. So you will have a check - just like in the native code - to throw an exception if the direction has not been set?

We probably should mimic the assertions if we want this to be more user friendly

Make that a separate PR as this one is just for build infrastructure.

Will add exceptions shortly

Moving the discussion to this issue (https://github.com/mono/SkiaSharp/issues/894) until we have a PR.


[feat]: links to example code from docs

Feature description
Maybe I'm misremembering, but there used to be links to the code behind the examples. For example, on https://ui.shadcn.com/examples/dashboard there used to be a link to https://github.com/shadcn-ui/ui/tree/main/apps/www/app/(app)/examples/dashboard
Is there any reason this was removed?
Affected component/components
No response
Additional Context
Additional details here...
Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues and PRs

Thanks for the link i was looking for the tasks components

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Open link in new window - How to?

I want to open a link to a new dashboard into a new window. When I use the following method it opens the new dashboard in the same window and "back" does not work or does not go back to previous dashboard.  So how can I open another dashboard into another window on click? (I.e. mimic the way as data links works in Grafana default visualisations)
echartsInstance.on('click', (params) => {
locationService.replace('/d/PQeH9VFVk/gauge?orgId=1');
});

@asimonok Any suggestions?

@SC-66 to open a new window you can use window.open('/d/PQeH9VFVk/gauge?orgId=1'). Location Service is only for working with the current window.


Inference

The goal of this PR is to implement the inference step and describe how to carry it in the documentation.

[x] fix potential bugs, due to API changes for instance
[ ] describe how to run this step in the documentation
[ ] describe how to load the inference output data from the notebook test_global_control

It would make sense to maybe rename the "testing" directory into "inference", as otherwise people might expect to find software tests in there.

@arthurBarthe , in the Jupyter Notebook there are a lot of sections now labelled "IGNORE THIS". Can we delete these, or what are they used for and why should they be ignored?

It would make sense to maybe rename the "testing" directory into "inference", as otherwise people might expect to find software tests in there.

I am doing this renaming


SwaggerUI upgrade to 3x

fixes #518

Coverage remained the same at 4.92% when pulling ebe5cdf7c3d9a3caf8da7bed18113e36b4cef0d0 on swagger_ui_3x into b27d380c7243a95c3d3b3c3b2a8d6692ebeb4656 on master.


Cluster Manager and Cluster Dashboard show different age for same cluster

Internal reference: SURE-3800
Issue description:
The Cluster Dashboard and Cluster Manager are not showing the same age for the cluster. For example, Cluster Dashboard shows cluster was created 561 days ago, whilst Cluster Manager shows the same cluster as 56 days old.
Business impact:
Confusion as to when the cluster was created.
Troubleshooting steps:
Checked that environment was upgraded from Rancher v2.5.9 to 2.6.1 56 days ago, so likely something with the upgrade reset the cluster age for the cluster management.
Repro steps:

Install Rancher v2.5.x.
Deploy a downstream cluster
Wait a few days
Upgrade to Rancher v2.6.x
Wait a few more days
Check the downstream cluster in cluster management and cluster dashboard

Actual behavior:
Age of cluster does not match between cluster management and cluster dashboard after rancher upgrade.
Expected behavior:
Age of cluster should match

@richard-cox this looks like a BE issue to me:

AGE header defined here: https://github.com/aalves08/dashboard/blob/master/shell/config/product/manager.js#L137-L168 uses LiveDate formatter
LiveDate formatter doesn't change any values coming from the BE... Only does the math to compute to the date from "now" https://github.com/aalves08/dashboard/blob/master/shell/components/formatter/LiveDate.vue

@aalves08 The cluster resources will differ a lot between old (norman cluster) and new (home page mgmt clusters vs cluster management provisioning clusters) ui's. We need to tie down which resources show as different. From that we should get a better understanding of the scenario where the date could differ a lot. We'll then probably need to discuss with BE

@gaktive had some help from BE (slack thread here):

There is a new object (clusters.provisioning.cattle.io) in 2.6 that wasn't in 2.5. The Cluster Management page displays this new object. Since that object was created when they updated to 2.6, that would explain the age.

Since the 56 days count is exactly the count from the day the client upgraded, I believe this might just be the root cause for the issue. If you check SURE-3800, the Cluster Management is exactly where the 56 days are displayed.
On the FE side we don't modify the data (age) from my investigation. (check here)
Not sure what should be the next steps here @gaktive . Will wait for further indications.
FYI: @richard-cox

This one needs discussing with the team. When the steve mgmt cluster / norman cluster 's radically differs from the steve provisioning cluster we could

show a custom age component covering both times (create x, upgrade y)?
just show the original resource age and ignore the prov cluster's ago?

@aalves08 could you please provide a test template for this issue? I noted that in the issue description that several days are needed before upgrading? We're hoping to get further context so that I can properly validate this issue better and if, by chance, days are not needed, I can test this issue sooner.
Thanks in advance!

Hi @markusewalker . I was able to "mock" this for development purposes by hacking the code, then working on the fix... Unfortunately I don't see any other way via the UI that doesn't involve waiting a  couple of days after installing Rancher v2.5.x (so that you can check the Age column with something like 2 days)
Some time can be "shaven" because you don't need to wait a few days after upgrading to 2.6.X (you can already check the outcome).
Also SURE-3800 provides more context into this, be sure to check that.

Verified on v2.6-head https://github.com/rancher/rancher/commit/c8d01c681cdaf9ed8f4d5e0f86c18681848d0734.
ENVIRONMENT DETAILS

Rancher - Docker Install
Rancher version

Verified version: v2.5.14 -> v2.6-head https://github.com/rancher/rancher/commit/c8d01c681cdaf9ed8f4d5e0f86c18681848d0734

TEST RESULT
PASS
VERIFICATION STEPS

Setup Rancher v2.6.5 and navigate to the UI in a browser.
Created a standard user and logged into Rancher as that standard user.
Created a downstream RKE2 node driver cluster.
Once it successfully provisioned, waited for 7 hours before returning to the cluster.
After the time has elapsed, upgraded Rancher to v2.6-head https://github.com/rancher/rancher/commit/c8d01c681cdaf9ed8f4d5e0f86c18681848d0734.
In the UI, navigated to Cluster Manager and noted the time of the cluster:

Navigated to Cluster Dashboard and noted the time matches what I saw above:


Rm extraneous carriage returns

Fixes https://github.com/meraymond2/idris-ide-client/issues/20.
On Windows, newlines in IDE messages are \r\n, but the length header assumes single-character newline chars, so the logic to chop up the message buffer into s-exps gets messed up.
I finally got around to setting up an env on my Windows partition, and now the tests are passing.

Note to self: I forgot the changelog entry.


Allow receiver to check how many producers their are

It would be helpful if there were a way to see the actual reference count rather than just if there are any producers.

Out of curiosity, do you have an example of where you would find this feature useful?

Honestly, I think I was probably solving the problem the wrong way around. But I knew in my system that there would be a cycle (the receiver struct had a reference to the sender), so I couldn't rely on the typical closing behavior when all senders are dropped. I wanted to check for 1 producer and then perform my 'exit' actions.

Closing the issue. If you still need this feature, let me know.


Brew install issue on Linux

Helllo,
brew install pvolok/mprocs/mprocs is working fine in MacOS, but on Linux, it seems that the downloaded binary is missing the x permission:
ls -l /home/linuxbrew/.linuxbrew/Cellar/mprocs/0.3.0/bin

.r--r--r-- 492k farzadmf 29 May 17:16  mprocs

And, if I manually make it executable, I think it's somehow the MacOS executable because I get this error:
zsh: exec format error: mprocs

I see that installing on Linux through brew is not mentioned in the README, but usually there isn't a difference between Mac and Linux when they support installation through brew

Indeed I only added brew distribution for macos only. I didn't know if anyone would install mprocs via homebrew on linux. Tomorrow I will release a new version and will add linux version on homebrew.

Great, glad to hear, thank you!

It seems that homebrew formulae are source based and that's probably why they are by default supported on linux. I will have to investigate more how rust apps are distributed on homebrew. I'd prefer to avoid the rust dependency.

I'd prefer to avoid the rust dependency

Makes sense.
As an example, this seems to be bat's formula; thought it could be useful

@farzadmf Sorry it took so long. Mprocs is now in the core homebrew repo and is available on Linux as well.
brew untap pvolok/mprocs
brew update
brew install mprocs

Wow, AMAZING 🚀🚀🚀
Thank you so much for following up with this, much appreciated (I think I was the only one requesting this 😛)


Fix default avatar in offline mode

Details
#5974 (comment)
App is not loading the user avatar on offline mode. A blank space is displayed.

Fixed Issues
#5974

Tests

QA Steps

Force close the app
Turn on airplane mode
Open the app

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots

Web

Mobile Web

Desktop

https://user-images.githubusercontent.com/92646072/140522671-9e4fe200-a417-4818-87aa-7ab270dc2521.mov
iOS

https://user-images.githubusercontent.com/92646072/140338430-b073f51a-bbdb-4dde-a88c-b84f10c11cc4.mp4
Android

https://user-images.githubusercontent.com/92646072/140528946-62e0d6f4-49f5-4cba-8be5-7ee8d9e06f03.mp4
I have read the CLA Document and I hereby sign the CLA

After reviewing this again, I think we might need to go back to the drawing board here. I think this will also affect the avatar for workspaces when offline as well.

I have read the CLA Document and I hereby sign the CLA

I have read the CLA Document and I hereby sign the CLA

@RGutu You have to sign your commits for them to be merged.

@RGutu You have to sign your commits for them to be merged.

Yes, we understood that from your previous comment in other PR. We will do sign in future the commits. Thanks.

I'm going to close this PR and pay you for your time.
In general, this is not the right direction to be going for this problem, and the issue has changed since this was created.


Change started_at/finished_at to emit bigquery timestamp format.

An example timestamp looks like 2021-12-09T06:49:04

BigQuery documentation for the spec of the format


Class, method, and constructor declarations

Codecov Report
Patch coverage: 47.61% and project coverage change: +0.23% :tada:

Comparison is base (51c76eb) 76.87% compared to head (3e627d6) 77.10%.
Report is 8 commits behind head on develop/1.10.0.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                 Coverage Diff                  @@
##             develop/1.10.0     #307      +/-   ##
====================================================
+ Coverage             76.87%   77.10%   +0.23%
- Complexity             8347     8535     +188
====================================================
  Files                   251      251
  Lines                 26948    27522     +574
  Branches               6789     7072     +283
====================================================
+ Hits                  20715    21221     +506
- Misses                 4162     4200      +38
- Partials               2071     2101      +30

Files Changed
Coverage Δ

...brains/java/decompiler/main/DecompilerContext.java
91.37% <0.00%> (-3.27%)
:arrow_down:

...va/decompiler/main/collectors/ImportCollector.java
82.85% <52.63%> (-5.70%)
:arrow_down:

... and 9 files with indirect coverage changes

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


Where do we pass in the device token of the user to whom we are sending the push?

I am using toFCM function, but in that there is no where mentioned where to specify the registration_id or device_token to send push.
public function toFcm($notifiable)
{
    $message = new FcmMessage();
    $message->content([
        'title'        => 'Test Push',
        'body'         => 'Sample push notification',
    ])->priority(FcmMessage::PRIORITY_HIGH);

    return $message;
}

Place this code in the model
/**
 * Route notifications for the FCM channel.
 *
 * @param  \Illuminate\Notifications\Notification  $notification
 * @return string
 */
public function routeNotificationForFcm($notification)
{
    return $this->device_token;
}

you need to make use of laravel notification feature call on model
php artisan make:notification PushNotification
and Add the method public function toFcm($notifiable) to your notification, and return an instance of FcmMessage:
then use it
$user->notify(new PushNotification($parm));


KeyError: 'OW' while using --insert-molecule

Hi,
I am encountered KeyError: 'OW' while using --insert-molecule while using --insert-molecule TIP4P water. The .cif file is Okey for convert by lammps_interface.
Thanks!
Zhongming
[zeng@localhost lammps_interface]$ python lammps_interface.py HfBTB-MOL-Ni0.cif -ff UFF4MOF
Molecules found in the framework, separating.
WARNING: Molecule 1 with atoms (Ni, O, H, H, O, H, H) will be using the UFF4MOF force field as no  value was set for molecules. To prevent this warning set --molecule-ff=[some force field] on the command line.
Files created! -> /home/zeng/lammps_interface

[zeng@localhost lammps_interface]$ python lammps_interface.py HfBTB-MOL-Ni0.cif -ff UFF4MOF --insert-molecule TIP4P_Water
Molecules found in the framework, separating.
WARNING: Molecule 1 with atoms (Ni, O, H, H, O, H, H) will be using the UFF4MOF force field as no  value was set for molecules. To prevent this warning set --molecule-ff=[some force field] on the command line.
Traceback (most recent call last):
  File "lammps_interface.py", line 21, in <module>
    sim.write_lammps_files()
  File "/home/zeng/lammps_interface/lammps_interface/lammps_main.py", line 742, in write_lammps_files
    self.molecule_template(self.options.insert_molecule)
  File "/home/zeng/lammps_interface/lammps_interface/lammps_main.py", line 527, in molecule_template
    cutoff=self.options.cutoff)
  File "/home/zeng/lammps_interface/lammps_interface/ForceFields.py", line 3133, in __init__
    self.compute_force_field_terms()
  File "/home/zeng/lammps_interface/lammps_interface/ForceFields.py", line 53, in compute_force_field_terms
    self.compute_atomic_pair_terms()
  File "/home/zeng/lammps_interface/lammps_interface/ForceFields.py", line 61, in compute_atomic_pair_terms
    self.pair_terms(n, data, self.cutoff)
  File "/home/zeng/lammps_interface/lammps_interface/ForceFields.py", line 3138, in pair_terms
    data['pair_potential'].eps = UFF4MOF_DATA[data['force_field_type']][3]
KeyError: 'OW'

Hi Zhongming,
Sorry for the delay. The reason you are getting an error is because it is searching for the TIP4P force field types in the UFF4MOF forcefield. To fix this you have to run the command:
python lammps_interface.py -ff UFF4MOF --insert-molecule TIP4P_Water --molecule-ff TIP4P HfBTB-MOL-Ni0.cif
so just add the --molecule-ff TIP4P and it should work properly. Seems a little redundant, but the intention was that you could insert a general particle like 'methane' and then apply whatever force field you wanted to it with the --molecule-ff command.
Let me know if you still experience problems.
Pete


Added homepages to a handful of mods that did not specify one

Added homepages to a handful of mods that did not specify one (usually using KSP Forums as a source), and tweaked metadata for EditorExtensions to use its forum post as its homepage rather than its github repository (which is now moved to 'repository')

So.. I was looking at the Jenkins build log, and it looks like the build failed because some of the netkans I updated (x64 builds of ATM) are only compatible with KSP==0.90.  This probably hasn't come up before, because typically if a mod is updated it's being made compatible with the current KSP release -- and thus wouldn't trip up Jenkins.
That said, it's entirely possible to reach a position where mod authors develop and update mods for an older game version by design.  This already happens now in Minecraft, which is currently at version 1.8.4 but has a mod community that largely targets 1.7.10.  Thus, there's probably sufficient impetus for a fix.

I assume these mods show up without homepages in the CKAN client, aka lack this information in their corresponding .ckan files in CKAN-meta? If so I can merge this based on my visual confirmation of everything looking fine.

That is correct. I found them by scrolling through the list in the client
and only updated the ones that listed Homepage: n/a
The one exception is the mod that listed its github repository as its
homepage (directly in the netkan), which I changed to be its repository and
added the forum page as the homepage. (My reasoning is that somebody
wanting to know more about a mod will get more out of the forum thread than
they will out of a repository, and the repository link is still available
to them within the client.)
On May 9, 2015 2:23 PM, "Willhelm Rendahl"<EMAIL_ADDRESS>wrote:

I assume these mods show up without homepages in the CKAN client, aka lack
this information in their corresponding .ckan files in CKAN-meta? If so I
can merge this based on my visual confirmation of everything looking fine.
—
Reply to this email directly or view it on GitHub
https://github.com/KSP-CKAN/NetKAN/pull/1184#issuecomment-100550969.

Perfect! Merging this based on visual inspection and it seems Jenkins didn't have anything to say about formatting atleast.


Error detected while processing .../tender-lightline.vim

This is my first time installing a theme for vim, and that might be my problem.  But, when I try to install it, I get the error below?  any thoughts on how to fix it?
It doesn't matter if I do it on the terminal or GUI.
Error detected while processing /home/irving/.vim/plugged/tender/plugin/tender-lightline.vim:
line   23:
E117: Unknown function: lightline#colorscheme#flatten
E15: Invalid expression: lightline#colorscheme#flatten(s:p)

Same's happening for me.

You must install https://github.com/itchyny/lightline.vim

Sorry guys, I just forgot that not everybody is using lightline. Give me 5 minutes please

@gustavofc now you must add a line to your vim config in order to enable the lightline theme:
let g:tender_lightline = 1

I hope this work for you, thanks for pointing it

@gustavofc  remember to add this line in order to enable the lightline theme:
let g:tender_lightline = 1

Thank you!

it works now!!!  thank you!
now, according to this site -> http://vimcolors.com/436/tender/dark, it should work for both gui and terminal.  I only got it to work on my gui.  Not sure, if this needs to be updated.

@blacknred0 true color only works with gvim or neovim with set termguicolors. Vim on terminal should show the closest xterm colors to hex ones.
Are you getting any error or it is just different color tones?

@jacoborus I get the error below and on vim terminal, it just stayed with whatever my terminal colors are (so no color tones).
E518: Unknown option: termguicolors
Press ENTER or type command to continue
but, it seems that gvim doesn't have termguicolors in my compile version, not sure why.  it could be an error on my end.

set termguicolors is for neovim 0.1.5dev or newer.
What happens when you type: :colors tender?

When I run :colors tender the terminal still looks the same. :/

wow, this is really weird, I need more clues. Does your terminal support 256 colors? What terminal are you using? Did you try any other color scheme succesfully? vim version?
This could be helpful: https://gist.github.com/XVilka/8346728

aha!!! thanks for the link/inspiration.  Apparently I found through this link -> http://askubuntu.com/questions/67/how-do-i-enable-full-color-support-in-vim that I would have to override vim's autodetection and enable 256 colors for GNOME Terminal.
So, after adding this piece of code below, it looks beautiful as expected!
if $COLORTERM == 'gnome-terminal'
  set t_Co=256
endif

Thanks for your help
PS: I've tested this theme -> https://github.com/lifepillar/vim-wwdc16-theme and it works just fine, when I do colorscheme wwdc16 on my ~/.vimrc without having the need to set 256 for terminal color.
We can say this is officially closed.  Cheers!

great! but I'm pretty sure you are not having the right colors, take a look to my screenshots in the readme file to see the differences.
Is your vim instance running over tmux?

you are right, I didn't noticed that.  weird :/ ... no, I am running straight from GNOME Terminal 2.31.3 on RHEL 6.7 (Santiago).

You need to know if 256 color support is enabled in Gnome Terminal, try echo $TERM in your terminal please, result has to be term-256color.
Otherwise you can set this with
export TERM=xterm-256color

you can also add that line to your .bashrc

so, you were right, when I invoke that command, here is what I got
$ echo $TERM
xterm

now, I went into the terminal and did the following
$ export TERM=xterm-256color
$ echo $TERM
xterm-256color
$ tput colors
256

I even went to the extend of setting set t_Co=256 on ~/.vimrc and these are the results (still same).

Not sure how much you want to debug this... or what else you would like me to do.

The last screenshot you posted looks ok (parens ans statements in light blue and strings in light brown).
This is how 256 color terminal should print tender color scheme:

BTW: it's 2016, I strongly recommend you to try neovim and gnome 3 :)


WrenAI API endpoint as UserPromptText-to-SQL_Query converter tool

Also does  WrenAI have API endpoint?
I mean can I via API send the text with user prompt and get the converted SQL query from WrenAI?
Basically serving as UserPromptText-to-SQL_Query converter tool.

Also does WrenAI have API endpoint? I mean can I via API send the text with user prompt and get the converted SQL query from WrenAI?
Basically serving as UserPromptText-to-SQL_Query converter tool.

Hi @qdrddr, we have an API you can call directly, but we haven't published our API documentation yet. Someone asked a similar question a few days ago. @wwwy3y3 provided some guides on how to trace the API work in the Discord channel:
https://discord.com/channels/1227143286951514152/1238353430985179208/1240650085474369617
Feel free to check it out and give it a try. I think we may have formal documentation for it soon.

Is something like websocket supported for this or we currently have to check the endpoint if the task is finished? @goldmedal

Allow me to pass this question along to @wwwy3y3 and @onlyjackfrost, as they are responsible for API development. They can provide a clearer answer.

Currently, web socket is not supported, you have to keep pooling to check the status of askingTask.
The result of an askingTask usually consists of an array of elements, each containing a SQL statement. The array typically has a length of 3, but it can be anywhere from 1 to 4.

Can I add feature request to support sockets? @wwwy3y3 & @onlyjackfrost
Or maybe I just can convert this task into a feature request?

@qdrddr feel free to open another ticket discussing the support of web socket 😁

Let's move this discussion to WrenAI.


⚠ GitLab Server has degraded performance

In 7b4dce1, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 10988 ms

Resolved: GitLab Server performance has improved in d6764ff.


Fix/a11y suggest toggle item 13 7

should this reach master branch as well?

should this reach master branch as well?

is in on master https://github.com/UiPath/angular-components/commit/e218b4d2c2705644645f8017c44597e9bf66c5e0


Directional focus isn't working in latest build

Whether using skhd keyboard shortcuts (alt - h) or typing the command myself (yabai -m window --focus west) does not change focus.
▲ ~ sw_vers
ProductName:	Mac OS X
ProductVersion:	10.14.5
BuildVersion:	18F132

 ▲ ~ yabai --version
yabai-v1.0.3

 ▲ ~ skhd --version
skhd version 0.3.4

verbose output of yabai shows it's getting the message, but fails to produce action
EVENT_HANDLER_DAEMON_MESSAGE: msg 'window --focus east'
EVENT_HANDLER_DAEMON_MESSAGE: msg 'window --focus west'
EVENT_HANDLER_DAEMON_MESSAGE: msg 'window --focus south'
EVENT_HANDLER_DAEMON_MESSAGE: msg 'window --focus north'

other commands, i.e. toggling zoom-parent works as expected:
EVENT_HANDLER_DAEMON_MESSAGE: msg 'window --toggle zoom-parent'
EVENT_HANDLER_WINDOW_RESIZED: Alacritty 1636
EVENT_HANDLER_DAEMON_MESSAGE: msg 'window --toggle zoom-parent'
EVENT_HANDLER_WINDOW_RESIZED: Alacritty 1636

There has not been any changes to focusing since one of the very first commits.
Make sure you have SIP disabled and that the scripting addition is propery installed. Check the wiki for instructions.
Sent with GitHawk


(feature) display vacancy contact email

show the given contact email on the vacancy show page

adds a spec to check presence of email address
adds the email address to the vacancy show page sidebar
adds a sass utility class to wrap long email addresses so they don’t break the layout
adds hint text to the form views

 Contact email is captured but not displayed on vacancy page


Fatal error: Uncaught Error: Class 'Roots\Sage\Assets\JsonManifest' not found in

Hello Guys,
Whenever I upload my WordPress theme to my web hosting server I get this error message about a Fatal Error.
I just finish my theme on my local computer on WordPress know trying to test out on a web hosting server and also using a WordPress builder theme called sage by roots.io
Here the repo: https://github.com/brandonpowell/pixelcity/blob/master/src/helpers.php
Fatal error: Uncaught Error: Class 'Roots\Sage\Assets\JsonManifest' not found in /Users/brandonpowell/sites/valet/demo/web/app/themes/pixelcity/src/helpers.php:27 Stack trace: #0 /Users/brandonpowell/sites/valet/demo/web/app/themes/pixelcity/src/setup.php(71): App\asset_path('styles/main.css') #1 /Users/brandonpowell/sites/valet/demo/web/wp/wp-includes/plugin.php(524): App\{closure}('') #2 /Users/brandonpowell/sites/valet/demo/web/wp/wp-settings.php(395): do_action('after_setup_the...') #3 /Users/brandonpowell/sites/valet/demo/web/wp-config.php(9): require_once('/Users/brandonp...') #4 /Users/brandonpowell/sites/valet/demo/web/wp/wp-load.php(44): require_once('/Users/brandonp...') #5 /Users/brandonpowell/sites/valet/demo/web/wp/wp-admin/admin.php(31): require_once('/Users/brandonp...') #6 /Users/brandonpowell/sites/valet/demo/web/wp/wp-admin/themes.php(10): require_once('/Users/brandonp...') #7 /Users/brandonpowell/.composer/vendor/laravel/valet/server.php(106): require('/Users/brandonp...') #8 {main} thrown in /Users/brandonpowell/sites/valet/demo/web/app/themes/pixelcity/src/helpers.php on line 27

hello @brandonpowell
you created this issue a little while after you already made a thread on discourse, which was the appropriate place to ask your question
please don't use github issues for personal support request (which is noted in our contributing guidelines as well as the issue template that you cleared out when creating this issue)


Update regular expression in callout.ts

Refine the regular expression used for parsing panel markdown syntax in callout.ts. The updated regex improves the matching of callout types and handles optional collapse types more accurately. This change should enhance the accuracy of parsing and transforming markdown to Confluence-specific syntax.
Fixes #441

Happy to merge once you have signed the CLA

As per https://github.com/markdown-confluence/markdown-confluence/pull/443#issuecomment-1664753213 merging :)


bug: Getting "Invalid session properties" when requesting signature from trust wallet on mobile sometimes.

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
On requesting signature from trust wallet using wagmi on mobile, it throws "Invalid session properties" error.
It happened with me 3-4 times repeatedly and after that it starts working perfectly fine.
Expected Behavior
On requesting signature from trust wallet app, it should give me the desired signature hash and that too in the first try only.
Steps To Reproduce

Go to https://wagmi.sh/examples/sign-message link on mobile.
Try to connect your trust wallet app with wagmi.
Then try signing a message.
Trust wallet will show sign message popup, which you have to accept.
On redirecting back to wagmi link, you will see an error saying: "Invalid session properties. Version: viem 1.0.0". Something along these lines.

Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
If I get this issue again, will attach the screenshot for sure.
Edit: Here is the screenshot of error.

Unfortunately this is an issue with Trust Wallet. Best to try and get in touch with them to resolve the issue.


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit 570d739:
:clock10: Pending: waiting for processors (22 builds ahead of you)
:warning: Docs Build is busy, currently there are 22 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 570d739:
:clock10: Full build: the average full build time is 7 min(s) 44 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 570d739:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Raspberry Pi arm64 error

https://www.nuget.org/packages/Avalonia.Skia.Linux.Natives/<IP_ADDRESS>
does't work with Raspberry Pi arm64
Unhandled Exception: System.DllNotFoundException: Unable to load shared library 'libSkiaSharp' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: liblibSkiaSharp: cannot open shared object file: No such file or directory    at SkiaSharp.SkiaApi.gr_glinterface_assemble_gl_interface(IntPtr ctx, IntPtr get)    at SkiaSharp.GRGlInterface.AssembleGlInterface(Object context, GRGlGetProcDelegate get)    at SkiaSharp.GRGlInterface.AssembleGlInterface(GRGlGetProcDelegate get)    at Avalonia.Skia.PlatformRenderInterface..ctor()    at Avalonia.Skia.SkiaPlatform.Initialize()    at Avalonia.Controls.AppBuilderBase1.Setup()
at Avalonia.Controls.AppBuilderBase1.Start(AppMainDelegate main, String[] args)    at UdsUI.Program.Main(String[] args) in /home/curtisy/Projects/Avalonia/UdsUI/Program.cs:line 12

I ran into this today as well. I worked around it by building v<IP_ADDRESS> for ARM64 on the Pi itself, and dropping the resulting libSkiaSharp.so file directly into the output directory of my Avalonia app (so it's in the same dir as SkiaSharp.dll). My app ran fine on the Pi after that. I've attached the SO here in case anyone wants to try using it until official support is added.
libSkiaSharp.zip

Did you use the bootstrapper provided by SkiaSharp to build or did you manually run the build? Ideally, we would cross-compile it but haven't figured out to do that.

easier solution right now would be to use JellyFin's SkiaSharp lib for ARM
https://www.nuget.org/packages/Jellyfin.SkiaSharp.NativeAssets.LinuxArm/

@Gillibald we need to figure out how to x-compile these libs for our own SkiaSharp package...

Did you use the bootstrapper provided by SkiaSharp to build or did you manually run the build? Ideally, we would cross-compile it but haven't figured out to do that.

It was a fairly manual process - I'll write a gist about it and link it here when it's ready. In a nutshell, I mostly followed these steps but had to build gn from source (since the included binary is for x64), had to use the distro's version of ninja since depot_tools doesn't know what to do with aarch64, and had to make a few other tweaks (like this one).

Okay, here's my writeup of the process I used. Hope it helps!

SkiaSharp now has built-in support for arm and arm64


Should prevent the word groups field from accepting @ symbols

Users are still trying to include the @ symbol in the word group names. Should just ignore it or remove it.

@aidanfeay is this still a thing?


ARROW-8132: [C++] Fix S3FileSystem tests on Windows

The new Minio version is less robust when deleting directories.

@pitrou appveyor is failing

Ok, I'm skipping some assertions on Windows.

AppVeyor build green both on my fork: https://ci.appveyor.com/project/pitrou/arrow/builds/31501196 and on the ASF account: https://ci.appveyor.com/project/ApacheSoftwareFoundation/arrow/builds/31501193


Selection in sub-grid selects in main grid as well

In the sub grid demo: http://w2ui.com/web/demos/#!grid/grid-7

Click on the first "+" to show the sub grid
Select multiple rows in the sub grid via single-click-dragging

Result: In addition to the rows in the sub grid, the same number of selected rows is being selected in the main grid below.

Yes, I think the bug. I will be refactoring subgrids completely, will address it then.

Subgrids are refactored. All changes in current master


Dont overwrite entry.server in Vite preset

Don't replace the entry.server file, only give the user a warning instead. By replacing the entry.server files it breaks integrations with tools like i18next where you want to set up the <I18nextProvider /> in the entry file.

Hi there. Thanks for bringing this up and showing the use case of when a modified entry.server file would be necessary.
It's kind of a tricky situation because the Remix templates now include a default entry.server file implementation, so there's no good indicator as to whether the app actually needs that file (because it has made modifications to it) vs. when they don't need it since it's just the default that was included with the template (in which case, Vercel should supply its version).

I get that this is tricky, I see that the documentation now mentions that you should delete your entry.server/client.
I would say that it's quite common to modify both entry.server and entry.client in Remix. Isn't it better to document what's needed in a custom entry.server/client? If your not going to allow this in vercelPreset I'm out of luck and can't deploy to Vercel?

Ya, I hear you. We're going to make sure your use case is supported. We're evaluating a couple options. Will report back soon.

Follow along in #89.


star field doesn't work with WEBGL in p5js

Hi!
I was toying with the star field example here,
and I wanted to add some planets, etc., so I turned on WEBGL.
However, as soon as I enable WEBGL,
the stars disappear, and I get:
"7790: Uncaught TypeError: undefined is not a function"
I tried turning a few things on and off to fix it, but no luck.
I'm using 0.6.1 on a MacBook with El Capitan (and I'm using the p5.js editor).
[Not sure if I should be posting this here or on the p5 page,
apologies if I chose the wrong one!]

To be clear,
I did adjust the shapes to exist in a 3D plane.
I also tried swapping out the p5.js Star constructor with a version based on NOC Cosmos,
among other things.

var stars = [];
var speed;
var depth;
var factor;

function setup() {
  createCanvas(600, 600, WEBGL);
  depth = 5000;
  factor = 1;
  speed = 5;

  for (var i = 0; i < 800; i++) {
    stars[i] = new Star();
  }
}

function draw() {
  background(0);
  ambientLight(255);
  for (var i = 0; i < stars.length; i++) {
    stars[i].update();
    stars[i].show();
  }
}

var pos;
var originalVel;
var vel;

function Star() {
  pos = createVector(random(-5*width, 5*width), random(-5*height, 5*height), random(-depth, depth));
  originalVel = createVector(0, 0, random(1, 5));
  vel = copy(originalVel, vel);

  this.update = function() {
    vel.z = originalVel.z*factor;
    pos.add(vel);
    // Recycle the star when it passes far away
    if (pos.z > depth)
      pos.z = -depth;
    }
  }

  this.show = function() {
    stroke(255, 127);
    strokeWeight(2.5);
    point(pos.x, pos.y, pos.z);
    beginShape();
    for (var i = 0; i < 5; i++) {
      vertex(pos.x-vel.x*i, pos.y-vel.y*i, pos.z-vel.z*i);
    }
    endShape();
  }

This should be attempted again now that I've done the WebGL tutorials!


fix_img_size

In the config file, fix_img_size is set to True by default. What is the purpose (advantages) of fixing image size?

The benefit of fixing the image size is that the inference time is lower for consecutive images, probably because of memory allocation.


WIP: Machine learning module

Starting to work on the machine learning module for event reconstruction, draft PR to get early feedback.
This will probably get pretty large, so I will try to pick small parts out for individual PRs to make it easier to review.
This will include code  and ideas taken from / adapted from aict-tools and protopipe mva

I know it's only a draft, but I put a few minor comments above.  Overall it's nice and easy to use so far!

One feature that I think is very useful is the ability to make features on-the fly that are composed of existing features.  That allows us to also apply transformations to existing features (log10(energy), etc).  Do you have an idea of how you will implement that? In protopipe, @HealthyPear used Pandas's DataFrame.eval() to generate new columns on the fly, but that also required making single-row DataFrames in the event-wise predict step, which is maybe a bit awkward.
 dataframe.eval(f"{feature_name} = {feature_expression}", inplace=True)

Do you have an idea of how you will implement that?

Yes, this is easy using numexpr, same as pandas eval

I'm trying to rebase this branch, which yielded the following uncaught problems:

len(events) < n_cross_val: we could skip cross validation or fail gracefully
no signal or background events (but the other), sklearn will not fit a model due to not being able to calculate ROC AUC with only 1 class in y_true

len(events) < n_cross_val: we could skip cross validation or fail gracefully

I guess we should just error out here, this is never a situation we'd actually want to be in.

@kosack I restructured the ml module to simplify the structure. The separate low-level classes only dealing with sklearn are now gone and the hirarchy is:
ctapipe.reco.Reconstructor
   ctapipe.ml.SKLearnReconstructor
     ctapipe..ml.SKLearnRegressionReconstructor
       ctapipe.ml.EnergyRegressor   # giving it the full name was too long for my taste, python has introspection
     ctapipe.ml.SKLearnClassificationReconstructor
        ctapipe.ml.ParticleIdClassifier

@kosack @LukasBeiske @LukasNickel @nbiederbeck Please have a look again.

@kosack I restructured the ml module to simplify the structure. The separate low-level classes only dealing with sklearn are now gone and the hirarchy is:
ctapipe.reco.Reconstructor
   ctapipe.ml.SKLearnReconstructor
     ctapipe..ml.SKLearnRegressionReconstructor
       ctapipe.ml.EnergyRegressor   # giving it the full name was too long for my taste, python has introspection
     ctapipe.ml.SKLearnClassificationReconstructor
        ctapipe.ml.ParticleIdClassifier

@kosack @LukasBeiske @LukasNickel @nbiederbeck Please have a look again.

Looks good. Removing the low-level classes makes everything easier to follow imo. I will change the disp component to be a subclass of ctapipe.reco.Reconstructor, as requested.

@kosack For now I introduced a GeometryReconstructor in between, as I couldn't find a nice solution to read the model yet in a general manner.
I will have a look again.

I ran this on the prod5b La Palma Alpha Layput using the config here:
https://github.com/maxnoe/test_ctapipe_ml/
Find some performance metrics here:
https://github.com/maxnoe/test_ctapipe_ml/blob/main/performance.ipynb
Looks good I'd say

I hope I addressed all review comments. Please have a look again.

@kosack I think I addressed all comments.


Create table containing junction and marker id correlation

The table with contain the following columns:

junction Geometry
Osm tags
long (of a point inside of the geometry)
lat (of a point inside of the geometry)
marker id (to correlate between markers and involved)

Currently running the query...

Query result is located in:
https://drive.google.com/drive/folders/115H5yPyDjnCWHesmgLezHGm4WIyQMyM8?usp=sharing
Import research.backup file to postgres using the GUI tools.
(open gui tool ,pgAdmin III, right click on database anyway select restore and choose research.backup)


⚠ 한경주요지수-코스피200 has degraded performance

In 1607ae5, 한경주요지수-코스피200 (https://markets.hankyung.com/indices/kospi200) experienced degraded performance:

HTTP code: 200
Response time: 5934 ms

Resolved: 신규상장 performance has improved in 301c447.


Update progress bar to rich

Bringing the progressbar into the modern era.
angr pr https://github.com/angr/angr/pull/3885
angr-doc pr https://github.com/angr/angr-doc/pull/462

@twizmwazin Thoughts on this? I don't quite like adding progressbar as a dependency when its only use is for a single test-case

If it's just a single test case I'd vote to remove it

Going to remove the dependency entirely.


🐛 : Delete Other Pictures Bug

Describe the bug
When attempting to delete other pictures, a bug occurs.
To Reproduce
Steps to reproduce the behavior:

Go to the profile section.
Click on "Photos."
Select a photo other than the profile picture.
Click on the delete button.
Observe the error.

Expected behavior
The selected photo should be deleted without any errors.
Screenshots
https://github.com/langxapp/langx/assets/3280475/21db7a11-9ef7-4454-be0b-ebf3d55a0c0a
Desktop:

OS: [e.g. Windows 10]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 97.0.4692.99]

Smartphone:

Device: [e.g. iPhone 12]
OS: [e.g. iOS 15.4]
Browser: [e.g. Safari]
Version: [e.g. 15.4.1]

Additional context
N/A
If you want to speed up the process of this issue, please send 🚀 as a reaction.

within 24h, it will be fixed and new version released. @haticealtunn


Crime Experience

This isn't an issue, just wanting to confirm whether my understanding of crime experience is correct?
So with the default formula: ((WILL*0.8)/2.5)+(LEVEL/4)
Assuming that you have 100 will and are level 1, this would be:
((100*0.8)/2.5)+(1/4) = 32.25
So does this mean that the crime success rate is 32.25%?
Are you able to elaborate on how you came up with this particular formula? Thanks.

It's just a starter/placeholder formula to give you an idea of what you can use. As your level/will increases, you'll see that formula's chance for success increases. As you see here, you can input LEVEL, EXP, WILL, IQ.
In all actuality, you could probably make a very complex formula such as
(((IQ*LEVEL)/EXP)/WILL) or, your own random formula. You should be able to, in theory, also just enter in a flat number if you want everyone to only have a specific chance of success.


feat(ssr): render portals

It's probably not the best possible solution, so I'm waiting for comments for improvements.

@yyx990803 should they be then received by server rendering code and moved where required, right?

I was testing this locally and realized it's better to pass the ssr context via the app context instead. Handles concurrency and doesn't require nested passing.
The previous passing implementation would also require the compiler to generate different code (in order for renderComponent calls used in ssr render functions to use the correct context)


How to get base address of process?

I have a game that doesnt have any dll modules so i cannot use those, instead i need to take the base address of the game and add my offsets to it. How may i do this?

Hi!
Which way are you trying to do, external or internal? It depends, but i think it'll work if u pass executable name like "foo.exe" instead of dll name.
if it doesnt work, let me know!

I am currently using external, and I just tried passing the executable name instead of a dll name and it errors with a PartialCopy error.

May I take a look at a minimal reproducible example of the bug?
If its because of your code or target memory, i can point it out, or if it looks my code's fault, I'll fix it so:)

    let lol: Process = match Process::from_process_name("League of Legends.exe") {
        Ok(p) => p,
        Err(why) => {
            panic!("{}", why)
        }
    };

    let module_info = lol.get_module_base("League of Legends.exe");

    println!("{:#?}", module_info);
    ```

Will print

Err(
ReadMemoryFailed(
ErrorPartialCopy,
),
)

Does it print at the line of panic!("{}", why) ??

thank you !!

plz be informed that it may take a while cuz im swamped this week.

Thats completely fine! im in no rush just experimenting with gamehacking.

Thanks.
This is off topic but cpp is way easier than rust when u manipulate memory. Rust complain about types every time u compile...
I just leaving this code as a reference for later and backed to cpp haha.

So I tested it with random 64bit process I own now and it just worked.
use toy_arms::external::process::Process;

fn main() {
    let process = Process::from_process_name("WAIUA.exe").unwrap();
    println!("process handle: {:?}", process.handle);
    let module_info = process.get_module_info("WAIUA.exe").unwrap();
    println!("{}", module_info);
    while true {}
}

# output
process handle: 0xac
module_name: WAIUA.exe
module_size: 290816
module_base_address:<PHONE_NUMBER>76736  <-- 0x7ff6d5870000 in hex which is correct as shown below image
module_handle: 0x7ff6d5870000
module_path: C:\Users\pseuxide\AppData\Local\Programs\WAIUA\WAIUA.exe

probably League of Legends has a sort of memory protection technique. or maybe hooking VirtualProtect or something. I'll debug with League of Legends tomorrow.
Are you not getting baned? Careful if u use this kind of library on Anti cheat-equipped game, it might get u baned unless u reverse engineer the integrity checker and hack it beforehands.

So I tested it with random 64bit process I own now and it just worked.
use toy_arms::external::process::Process;

fn main() {
    let process = Process::from_process_name("WAIUA.exe").unwrap();
    println!("process handle: {:?}", process.handle);
    let module_info = process.get_module_info("WAIUA.exe").unwrap();
    println!("{}", module_info);
    while true {}
}

# output
process handle: 0xac
module_name: WAIUA.exe
module_size: 290816
module_base_address:<PHONE_NUMBER>76736  <-- 0x7ff6d5870000 in hex which is correct as shown below image
module_handle: 0x7ff6d5870000
module_path: C:\Users\pseuxide\AppData\Local\Programs\WAIUA\WAIUA.exe

probably League of Legends has a sort of memory protection technique. or maybe hooking VirtualProtect or something. I'll debug with League of Legends tomorrow. Are you not getting baned? Careful if u use this kind of library on Anti cheat-equipped game, it might get u baned unless u reverse engineer the integrity checker and hack it beforehands.

They dont care if I do passive memory reading, which is all im doing for now.

I just examined it and indeed, error get caused in the memory reading part. It's been long time since i peeked at this code, but rust's code is horribly hard to read. omg

i find rust to be easier to read than C++, but thats because ive used it more. It is probably the same thing with you just opposite

So, i found the possible problem that might be causing this issue, and confirmed it by testing things in C++ haha. Anyway now I have an idea to fix it. I'll fix it in this few days ~~


Add favicon

The current boilerplate includes no favicon.  I think a couple of our internal apps did alternate implementations.  I'll dig them up and paste here and we can discuss.

Option 1
Add this links section to .enutrinorc.use[0][1].html:
          links:[
            {
              href: "https://static.veritone.com/assets/favicon/apple-touch-icon.png?v=lkgpRBRLYl",
              rel: "apple-touch-icon",
              sizes: "180x180"
            },
            {
              href: "https://static.veritone.com/assets/favicon/favicon-32x32.png?v=lkgpRBRLYl",
              rel: "icon",
              sizes: "32x32",
              type: "image/png"
            },
            {
              href: "https://static.veritone.com/assets/favicon/favicon-16x16.png?v=lkgpRBRLYl",
              rel: "icon",
              sizes: "16x16",
              type: "image/png"
            },
            {
              href: "https://static.veritone.com/assets/favicon/safari-pinned-tab.svg?v=lkgpRBRLYl",
              rel: "mask-icon",
              color: "#597cb1"
            },
            {
              href: "https://static.veritone.com/assets/favicon/favicon.ico",
              rel: "shortcut icon",
              type: "image/x-icon"
            }
          ]

Option 2
yarn install react-favicon and
<Favicon url="https://static.veritone.com/assets/favicon/favicon-32x32.png" />

Option 3
Just add a favicon.ico to project root

@olslash thoughts?

I don't know a whole lot about this, but given that the .ico format is container for many sizes, would ie. option 1 need all those entries? or can it just be a single ICO?
either way, I'd go with option 1 because we're already generating all the basic markup for index.html from there... nice to have all that in one place

Option 1's my top choice too. From what I know about option 1, they should be different pngs usually.  They're used for things like phone app icons or dock icons or whatever as well as the favicon.


Pretty print JSON output

Currently, the printed JSON output is raw json output that is not human readable. Pretty print the JSON to make it more easily readable

Closed by #23


Custom CSS page isn't working

I have $wgSiteCSS = 'bootstrap-mediawiki/custom.css'; in LocalSettings, but my changes added there never seem to show up. Did I miss a step with this?

I'm having an issue loading anything via "custom.css".
$wgSiteCSS = 'custom.css';
Doesn't seem to do anything

Same issue here. It is not loading custom.css in either way. Can you please help to fix it?


Disable automatic use of System Units

Make it possible to enable the automatic use of system units, but disable it by default.

Fixed with #19


apacheHttpdPackages.mod_tile: Update and use cmake to build mod_tile.so and renderd

Description of changes
the openstreetmap mod_tile repo also contains the sourcecode of the renderd daemon, but the derivation did not build that so far.
While inspecting that, I have seem that the project uses cmake nowadays, so I changed the derivation, which also makes it a bit simpler.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 221357 run on x86_64-linux 1

1 package built:

apacheHttpdPackages.mod_tile (apacheHttpdPackages_2_4.mod_tile)

@trofi @GabrielDougherty thank you for the actionable suggestions. They're incorporated and the glib.out include path thing is explained with a comment in the code now. Plz have another look.

@oforg build apacheHttpdPackages.mod_tile

@trofi done

Meanwhile i managed to get an apache httpd + mod_tile + renderd running and producing map tiles. So this thing seems to work well.

@trofi are you going to merge or should i CC more people?


[FEATURE] Allow to add an optional NotFound handler

router.NotFound = func(_ *Params) error {
// make whatever
return nil
}
Execute must run NotFound handler on ErrNotFound

closed with #2


Add option to not throw on unknown enum values

We would instead use the default value specified in thrift (if the field has one), or null for optional/unrequired fields.  We'd still have to throw for required fields without a default, but I'm OK with that - the design of enums in thrift isn't forward-compatible as is, and required fields doubly so.
See #84 for the conversation sparking this feature.

We also ran into the same issue. For now, we've forked the codegen to do this. If you want, I can send a PR for this.
I'm not sure if there should be an option for this, or using the default (or null) should be the only supported way.

If I'm totally honest, I have to say that I do not fully agree with this feature - at least, not as formulated, because it irretrievably loses data.  In protobuf, for example, generated parsers skip over enum values that are unrecognized, BUT those unrecognized values are maintained in the associated "unknown fields", and when re-serialized those messages will have the unrecognized enum value intact.
Thrift has no such concept, so if the server (in the android case) sends an enum value the client doesn't recognize, there's no way to get that value back.  This is on top of the fact that, by going with a default, the client is unintelligently overriding whatever the server is trying to say.
I'd be happy to take a PR for this though, provided that its changes are behind a compiler switch of some sort.
In the linked thread, we talked about adding a command-line flag that, when enabled, would cause generated parsers, on encountering an unrecognized enum value, to:
a) fall back on the field's default value, if specified in thrift
b) fall back on null if the field is optional,
c) throw as it currently does for required fields.


wait for deletion to complete (aws-pke) - long running activities

Q
A

Bug fix?
no

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
fixes #X, partially #Y, mentioned in #Z

License
Apache 2.0

What's in this PR?
On cluster deletions (aws - pke) waiting for the deletion to complete is factored out into a dedicated activity. The activity sends heartbeats to the cadence server while waits for the aws lib specific blocking call.
Why?
The deletion workflow gets more robust with this approach; the possibility of failures due to timeouts is reduced.
Additional context
This is a WIP it is to be extended to other blocking calls too
Checklist

[X] Implementation tested (with at least one cloud provider)
[X] Error handling code meets the guideline
[X] Logging code meets the guideline (TODO)

relates to #2504


How to make sure that a month is changed in JTAppleCalendar?

I am using the latest branch.
pod 'JTAppleCalendar', :git => 'https://github.com/patchthecode/JTAppleCalendar.git'

I am aware that there's a delegate method which will call whenever we make some scroll. But this is not enough to tell that a month is changed. So how to know this using in-built API?

func calendar(_ calendar: JTAppleCalendarView, didScrollToDateSegmentWith visibleDates: DateSegmentInfo) {

}

Right now, I am using this delegate method but it will always invokes even if I slightly scrolls. I will try with that new delegate today and let you know.

func calendar(_ calendar: JTAppleCalendarView, willScrollToDateSegmentWith visibleDates: DateSegmentInfo) {}
For me this delegate is there, but it's not getting called. Anything to set?

Ok, so now it's calling, I wasn't on the master branch. Now, I am on a master branch and I have this delegate (in your code) and it gets called every time, but this isn't the solution to my question. Because, what I am looking for is a delegate should only get called if a month is changed.
    func calendar(_ calendar: JTAppleCalendarView, willScrollToDateSegmentWith visibleDates: DateSegmentInfo) {}

    func calendar(_ calendar: JTAppleCalendarView, didScrollToDateSegmentWith visibleDates: DateSegmentInfo) {}

These guys called every time. Even if the month isn't changed.

Ok. So now i understand want you want.
You just want to know when a month has changed.
The thing is, this is know know by the developer (you) not the library. The reason why I cannot know when a month has changed is because you devs can configure this lib in too many ways.
Here is a simple case

With a calendar like this, multiple months are displayed on the screen at the same time.
With a setup like this, there is no way for me to tell what the developer considers the month on the screen.
So the way i handle it is: Whenever a scroll happens, in those delegate functions above, you get a bunch of inDates, monthDates and outDates. You can run a check of the the dates to see if the month has changed. You could run a check to see if the dates contain some date in the middle of the next month. This is the beast way i currently have of determining if the month has changed. (unless a developer can suggest me a better way).

maybe if you tell me what youre trying to do, then i can suggest something?

I want to call the API whenever the month is change. Right now, even if it's not change it will call the API. I am performing few other tasks when the month will get change. That, all logic is written in this delegate,
func calendar(_ calendar: JTAppleCalendarView, didScrollToDateSegmentWith visibleDates: DateSegmentInfo) {

}

But this will call even if I change the height of the calendar or just slide scroll.

Hi @vishnude , as @patchthecode explained currently because of there can be multiple way that one can create a calendar for their apps it's not possible for him to give us a way to check whether it's scrolled or not. So for my case, I have find an alternative which will make sure whether we have a new date? If it's a new date, I will be do the things which I want. Like wise. Hope this may help, however, I am believing that if @patchthecode will provide a solution to us it will be more help than patching our code to accomplish particular tasks.

finally got around to reviewing this.

I am believing that if @patchthecode will provide a solution to us it will be more help than patching our code to accomplish particular tasks.

I do not believe this is patching. Me developing a solution for one developer, will not work for another developer. Because there are many different ways that a developer can define a "Month", then if i developer a solution that a "month" is changed, then i would only be developing a solution for one developer and not the other.
The best way to determine if month has changed (i think) would be to monitor distance scrolled.
Since you know the size of your month might be X, then you know that if the contentOffset of the calendar changed by more than X, it will be a new month.
I assume you have your calendar's scrolling mode to be either
.stopAtEachFrame
.stopAtEachSection

Therefore, in the following function put this.
func calendar(_ calendar: JTAppleCalendarView, didScrollToDateSegmentWith visibleDates: DateSegmentInfo) {
  // if my calendar's content offset is > than X {
  //   run my API's because the month is changed
  // } else {
  //   do nothing
  // }
}

I think this is the best solution.
Will close this issue now. But let me know your thoughts.


Fix crash that happens when Firebase API key gets revoked

Fixes https://github.com/fullstackreact/react-native-firestack/issues/318
This has been really hard to reproduce and track down: When an API key gets revoked the stored auth token is not valid any more, causing this error. error localizedFailureReason doesn't contain anything in that case, so it's nil, causing Objective-C to crash as you can't have it as a value for a dictionary.
It's probably save to have the default text here, but feel free to tweak it and modify the PR directly as necessary.
Let me know if you have any other questions. I'm using firestack for my side project wwdc.family :+1:

@KrauseFx this has long since been fixed on my v3 fork, I re-wrote auth from scratch and it has better error handling and matching web SDK error codes. Have actually re-written all the modules to do the same as well as adding remote config, perf-mon, analytics, crash etc.
Fork: https://github.com/Salakar/react-native-firestack

@Salakar glad to hear. When do you think that will be part of master?

@KrauseFx We moved the fork to a standalone library as it was too hard to maintain not owning the repository: https://github.com/invertase/react-native-firebase
Includes loads more features too.