cleanup, Sarsa, Contraction mapping

remove trailing whitespace
added gitignore to repo
fixed compile instructions and hyperlink
fix a few things that tripped latex
stop interspersing tabs and spaces
added missing + sign in TD learning update equation
added Sarsa learning section
added Contraction Mapping Theorem
commit latest pdf

Thank a lot! I updated my article in order to quote you as a contributor, I hope you appreciate it.

Thank a lot! I updated my article in order to quote you as a contributor, I hope you appreciate it.

mos def, but even more grateful you put the shared the sheet with the world in the first place, thanks!

It is my pleasure. I am also planning to add DDQL and A3C in the future.


secp256k1 minor naming question

This is very minor point but I noticed the names are reversed and I wonder if its a mistake
ECPubKey.TryMultTweak
ECPrivKey.TryTweakMul
I think on the original its
secp256k1_ec_pubkey_tweak_mul
secp256k1_ec_privkey_tweak_mul
so ECPubKey.TryMultTweak should become ECPubKey.TryTweakMul
https://github.com/MetacoSA/NBitcoin/blob/d1331dc2fd9bc7de18f30e9ae3baf2a2c1eb1545/NBitcoin/Secp256k1/ECPrivKey.cs#L854
https://github.com/MetacoSA/NBitcoin/blob/3d00206f4b159e8dc6c2305b960fad50982db4a2/NBitcoin/Secp256k1/ECPubKey.cs#L350

fixing

fixing

I cant stress by the way how awesome it is you implemented secp256k1 in c# netstandard

I cant stress by the way how awesome it is you implemented secp256k1 in c# netstandard


Drag cursor showing with dragging set to false

When creating a map with { dragging: false }, the map still displays -webkit-grab cursor.

Accepted, although low priority.


Update scalatest to 3.1.0-SNAP7

Updates org.scalatest:scalatest from 3.0.6-SNAP6 to 3.1.0-SNAP7.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a nice day!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "org.scalatest", artifactId = "scalatest" }]

let's stick with 3.0.x for now until there's a "real" 3.1.x release (yes we're on a 3.0.x "SNAP" release, but only because that's what exists for Scala 2.13.0-M5)


merge to live

Docs Build status updates of commit fefc1a7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

office-365-management-api/office-365-management-activity-api-schema.md
:white_check_mark:Succeeded
View

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.


Fixing inconsistent apiPrefix in blockchain explorer insight

The default (or fallback) apiPrefix for an insight-based blockchain explorer is currently set to /api, yet, the default of a bitcore node running insight-api is using /insight-api. This pull requests harmonizes the default setups, and avoids issues when setting up an instance.
See also bitcorenode/index.js

thanks!

ACK

upcoming bitcore-5 uses /api prefix by default, so closing this one. Thanks any ways.


added regex support for blacklist

Codecov Report

Merging #45 into master will decrease coverage by 1.19%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master      #45     +/-   ##
=========================================
- Coverage   58.05%   56.85%   -1.2%
=========================================
  Files          11       12      +1
  Lines         658      707     +49
=========================================
+ Hits          382      402     +20
- Misses        249      278     +29
  Partials       27       27

Impacted Files
Coverage Δ

blacklist/blacklist.go
40.81% <100%> (ø)

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 9798ea4...cd259ef. Read the comment docs.


Extract package persistence from update release

This extracts a secondary collaborator that takes the output of
PackageProcessor as input to persist the results.
There may yet be tests in the UpdateRelease spec that are testing
PackagePersister behavior and would be better to move over. The
PackagePersister tests that were moved were only those that were
discovered because the existing UpdateRelease tests were setting mock
expectations on UpdateRelease itself to validate its behavior and thus
were difficult to leave where they were.

As a first appraisal, this seems like a coherent enough object to be worth extracting. The only red flag for me are the couple tests that are mocking/expecting on the subject job in this case. If you can't write the tests without doing that, maybe this isn't the right code to extract.
Also for what it is worth, I like leaving at least some tests in UpdateRelease that are implicitly testing the PackagePersister. It is the only reliable way to actually test your wiring.

I agree with the assessment of the tests that are mocking the subject. Fwiw-those tests aren't new (I just moved them from UpdateRelease). I haven't had a chance to expand test coverage on the new object yet.

Updated tests to avoid mocking the test subject.

Cool, the tests look better now. I have a few more requests for you:

In the package persister specs, can you put tests for create_package and persist in their own separate contexts that are clearly labeled?
Can you make the class methods other than create_package and persist private? I had to do some detective work to figure out they were not used outside of the PackagePersister.
For the public class methods, can you use named arguments? There are too many arguments to just use ordered args. This is especially noticeable in the tests for UpdateRelease that look like expect(Jobs::UpdateRelease::PackagePersister).to have_received(:persist).with......... I'm not sure, but I'd hope you can change those tests to use hash_including if you have named arguments.

made class methods private where possible.
changed to use keyword arguments instead of long argument lists

I couldn't trivially make all class methods private since there are places where they are called from instance methods.


JIRA: EDGOAIPMH-6 - Define routes in the edge-oai-pmh verticle

MainVerticle, OaiPmhHandler, OaiPmhOkapiClientFactory, OaiPmhOkapiClient skeletons
Viachaslau Khandramai<EMAIL_ADDRESS>

SonarQube analysis reported 4 issues

 1 major
 1 minor
 2 info

Watch the comments in this conversation to review them.


No AttributeError on attempt to delete nonexisting Model field (pypy3.7)

Initial Checks

[X] I confirm that I'm using Pydantic V2 installed directly from the main branch, or equivalent

Description
When attempting to delete a model attribute that doesn't exist on pypy3.7 there is no AttributeError raised.
Example Code
from pydantic import BaseModel

class Model(BaseModel):
    some_field: str

m = Model(some_field='value')
assert not hasattr(m, 'other_field')

del m.other_field  # just passes without any error

Python, Pydantic & OS Version
pydantic version: 2.0a4
        pydantic-core version: 0.36.0 release build profile
               python version: 3.7.13 (7e0ae751533460d5f89f3ac48ce366d8642d1db5, Mar 29 2022, 06:03:31) [PyPy 7.3.9 with GCC 10.2.1 20210130 (Red Hat 10.2.1-11)]
                     platform: Linux-6.2.15-300.fc38.x86_64-x86_64-with-fedora-38-Thirty_Eight
     optional deps. installed: ['devtools', 'email-validator', 'typing-extensions']

I'd call that a bug in pypy.
We didn't even used to test with pypy, it's very slow with pydantic-core and I suspect the download numbers are tiny.

They don't even support Python 3.7 in latest releases. I've filed it for historic purposes only. I honestly believe there isn't much we can do here.

Interestingly the bug doesn't reproduce for regular classes and dataclasses.

Good to find and log, let's not waste time on it.


Level 5 headings and Level 6 headings with lowercase letters do not display properly

Is the bug present when using the default Obsidian theme?
Yes
Is the bug present when snippets and plugins are disabled?
Yes
Minimal theme version
7.3.2
Describe the bug

After entering a five-level heading or a six-level heading in a note, the lowercase letters in the title are not lowercase.
Debug info
Sorry, I won't have done this.

This is a feature not a bug. You can change this in Style Settings under font variant

Sorry I didn't fix this, can you be a little more specific.

I found the Style Settings plugin in the Community Plugins Market and solved this problem. Sorry to disturb your work, thank you very much.


Does iterator read from a snapshot?

i.e. does an iterator include writes that happened after the iterator was created?

Should be covered by https://github.com/Level/classic-level#iterator. If not, LMK what's missing in the docs

There is no reference to snapshot in that section.

I missed it.


Random-Image-slider

Describe the project you want to add with tech stack

In this project whenever user clicks on next then a random image will be generated and displayed.Can you assign this task to me?
Expected behavior

Screenshots (optional)

Additional context (optional )

@MohitGupta121 Can you assign this task to me?


Remove duplicate accelerometer gestures example.

It's been basically replaced by the accelerometer calibration example.

Done with https://github.com/damellis/sensors/commit/78412e15ab09c2903ac84d922e9587f7bd49e242.
Change:
renamed:    user_accelerometer_calibration.h -> user_accelerometer_gestures.h


remove useless functions

Signed-off-by: ChengZi<EMAIL_ADDRESS>

Codecov Report

Merging #1208 (76dfcfc) into main (8c7bbe4) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1208   +/-   ##
=======================================
  Coverage   68.41%   68.41%
=======================================
  Files         246      246
  Lines       11818    11812    -6
  Branches     1948     1948
=======================================
- Hits         8085     8081    -4
+ Misses       3227     3225    -2
  Partials      506      506

Impacted Files
Coverage Δ

towhee/models/clip4clip/until_module.py
61.11% <ø> (-0.80%)
:arrow_down:

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 8c7bbe4...76dfcfc. Read the comment docs.

/lgtm
/approve


Helper Class

hi,
I need to pass some DB data to every page (for navbar)
how can I do this??

You can use a Twig global variable, for example:
$twig = new Twig_Environment($loader);
$twig->addGlobal('navbar_data', $data);

thanks!
but Where should I add?

You need to add that where you create the Twig_Environment object, so in this case in the Core/View class. For example, if you want to have the PHP $_SESSION array available inside Twig templates:
$twig->addGlobal('session', $_SESSION);

and if I need some extra functions in the controllers should I add them in Core\Controller?

Whereas you could connect to the database in a view, that totally goes against the idea of separating application code and presentation code in an MVC framework. Normally the view and the model (where you connect to the database) are totally separate. This is covered in detail on the course.
Yes, if you need some extra methods in all controllers, you add them to the Core/Controller class, as all the controllers should extend this class.


podvm: restart agent-protocol-forwarder on failure

RHEL 9 podvm image is encountering an issue where cloudconfig init hasn't completed before agent-protocol-forwarder starts and it fails because it can't find /peerpods/daemon.json. Restart the service on failure.
Fixes: #388
Suggested-by: Pradipta Banerjee<EMAIL_ADDRESS>Signed-off-by: Bandan Das<EMAIL_ADDRESS>

Thank you very much for investigating this issue.
Is it possible to introduce an appropriate dependency to a cloud-init boot stage in the agent-protocol-forwarder service definition, instead of restarting the service?
https://cloudinit.readthedocs.io/en/latest/topics/boot.html
I am not sure which target we should use, and how to write such dependency in the service definition file.
/etc/cloud/cloud.cfg in Ubuntu contains the following settings. This indicates that write-files is called at the init stage.
# The modules that run in the 'init' stage
cloud_init_modules:
 - migrator
 - seed_random
 - bootcmd
 - write-files
 - growpart
 - resizefs
 - disk_setup
 - mounts
 - set_hostname
 - update_hostname
 - update_etc_hosts
 - ca-certs
 - rsyslog
 - users-groups
 - ssh

https://cloudinit.readthedocs.io/en/latest/topics/modules.html#write-files
The attached SVG file is generated by systemd-analyze plot at a Ubuntu pod VM.
Is there any difference between Ubuntu and RHEL?

Thank you very much for investigating this issue.
...
Hmm.. yeah, the restart method is a little ugly.

Thanks for the handy command. I ran it in the rhel podvm as well.

I can't claim to understand systemd well enough but one thing that sticks out is that in the ubuntu plot, cloud-init.service runs a little before agent-protocol-forwarder in the timeline. However, with rhel, it's after.
So, something like this seems to work:
--- a/libvirt/image/files/etc/systemd/system/agent-protocol-forwarder.service
+++ b/libvirt/image/files/etc/systemd/system/agent-protocol-forwarder.service
@@ -1,11 +1,9 @@
[Unit]
Description=Agent Protocol Forwarder
-After=network.target
+After=network.target cloud-final.service
[Service]
ExecStart=/usr/local/bin/agent-protocol-forwarder -kata-agent-namespace /run/netns/podns -kata-agent-socket /run/kata-containers/agent.sock
-Restart=on-failure
-RestartSec=5s
The problem I see is that both cloud-init.target and cloud-final.service have RemainAfterExit=yes which makes it a little scary to use the After clause. But since we have WantedBy=multi-user.target later, it works but I am not sure if multi-user.target exists for all distros.
The only thing I would add to be on the safe side is a DefaultDependencies=no.
Coming to the error with agent-protocol-forwarder.service, it seems to be failing on the path to daemon.json:
Dec 01 01:07:20 podvm-nginx-c9902f85 systemd[1]: Started Agent Protocol Forwarder.
Dec 01 01:07:20 podvm-nginx-c9902f85 agent-protocol-forwarder[744]: /usr/local/bin/agent-protocol-forwarder: failed to open /peerpod/daemon.json: open /peerpod/daemon.json: no such file or directory
Dec 01 01:07:20 podvm-nginx-c9902f85 systemd[1]: agent-protocol-forwarder.service: Main process exited, code=exited, status=1/FAILURE
Thoughts or any other suggestions ?

I am not an export of systemd, but I found the following description in /usr/lib/systemd/system/cloud-init.target.
# cloud-init.target is a  synchronization point when all cloud-init's initial
# system configuration tasks have completed. To order a service after cloud-init
# is done, add the directives as applicable:
#  After=cloud-init.target and Wants=cloud-init.target
[Unit]
Description=Cloud-init target
After=multi-user.target

So, I think we can add the following lines instead of After=network.target cloud-final.service
After=cloud-init.target
Wants=cloud-init.target

The file daemon.json is generated by cloud-init using write-file. agent-protocol-forwarder loads this file, so it needs to be executed after cloud-init completes generating the file.
https://github.com/confidential-containers/cloud-api-adaptor/blob/7acbf62939405c04fa98ac4c4904209bdaf9de19/pkg/adaptor/hypervisor/ibmcloud/service.go#L179-L188

So, I think we can add the following lines instead of After=network.target cloud-final.service
After=cloud-init.target
Wants=cloud-init.target

Unfortunately, this does not work and ends up in an ordering cycle on RHEL.
○ agent-protocol-forwarder.service - Agent Protocol Forwarder
     Loaded: loaded (/etc/systemd/system/agent-protocol-forwarder.service; enabled; vendor preset: disabled)
     Active: inactive (dead)

Dec 02 19:44:07 podvm systemd[1]: multi-user.target: Found ordering cycle on agent-protocol-forwarder.service/start
Dec 02 19:44:07 podvm systemd[1]: multi-user.target: Found dependency on cloud-init.target/start
Dec 02 19:44:07 podvm systemd[1]: multi-user.target: Found dependency on multi-user.target/start
Dec 02 19:44:07 podvm systemd[1]: multi-user.target: Job agent-protocol-forwarder.service/start deleted to break ordering cycle starting with multi-user.target/start

From the plot above, it looks like the ordering of cloud-init.target wrt multi-user.target is not the same for Ubuntu and RHEL. I suggest we disable implicit dependencies by specifying DefaultDependencies=no as I did in this code update but I am open to other ideas. Thanks again for the review.


Pause Button even if app is not focused

Would you like a way to make it where even if you somehow had another app on top you can still pause for example?
Because I have a library that can be used that has different key events.  So even if somebody is for example in notepad and want to pause without using the mouse, you can.  The nuget package is this one.
https://github.com/musictopia2/WindowsKeyboardCapture
This has a nuget package as well.  I had somebody many years ago do that function for me because I have a media player where I want to be able to pause a song even if the program is minimized (or if another app is full screen) like a game.

I originally pictured MediaDock to be floating on the desktop or on top of other windows but this could still add a lot of value.
Global hotkeys could be a convenient feature for users without media keys built into their keyboard and users who don't want MediaDock on top of other windows. I'll keep the issue open until I figure out whether to add this to the planned features.
Thanks for the suggestion and nuget package to potentially use!

@musictopia2 I've added this to the roadmap. Thanks!


Adding layers (based on keras) supporting multiple outputs

This draft PR is tracking the implementation of keras layers in Java; will update with a README once the API is complete, and mark for review when it's ready.

@dhruvrajan , can you please provide some feedbacks for the status of this PR? Do you still intend to make it official?

I might be interested in picking this up if there is interest in this

Hi @KartikChugh , there is certainly interest in this and happy if you could help!
We did not hear from @dhruvrajan for awhile so I don't know if he was still planning to complete his work. But @JimClarke5 is another member of this community who is working actively in the new Keras layer, we should probably sync up with him. Jim, did you already start to work on the layers? How would you like Kartik to help?

Thanks @karllessard
@KartikChugh, I welcome the offer to help.
I am in the early stages of looking at layers. My game plan is to build Input, Dense, and Flatten, and this will drive flushing out the base Layer class, as well as Model and Sequential. I am working in my own repository and can push what I currently have locally to it, JimClarke5/tensorflow-java-keras. You can start looking at this repository to get an understanding on how I am approaching the Keras port. The plan is to then move these classes over to the main repository. I have already created the first PR on the Optimizers to check into the main repository and as I expected this is creating a lot of discussion on changes to the other Java modules.

I would also be interested in helping. I have limited time and I'm a deep learning beginner. Perhaps I could go find the Python Keras tests and start transliterating them to Java, using whatever classes and interfaces @JimClarke5 and others have built out, while scaffolding others that don't exist yet?
I imagine our goal would be to transliterate Python Keras pretty closely? Like if I looked at a Python Keras program and I'm a Java expert, I should be able to imagine the idiomatic Java way to code those things and that's the API I can expect? And then I should expect the exact same calls on TensorFlow from the Java version as from the Python version?

Transliterating directly from Python code might not be that straightforward in all cases, as the TF Python client used to exist way before Keras was added to the project so it has a lot of high-level (and deprecated?) APIs that wraps up the raw operators and can bring some complexities that might not be required for Keras usage. Since in TF Java we basically only deal with raw operators (with a few exceptions), we need to identify what is really required by Keras out of these wrappers and discard the rest, at least for now... but other than that, yes, that's pretty much it!
I'm not taking part of the Keras Java library so much so far but if I can help you to synchronize your work, by creating a branch in the repo or speeding up some PRs (yes sorry for that @JimClarke5 ;) ), just let me know and I will be pleased to assist. Also note that the second-monthly SIG meeting will be pretty much be dedicated for Keras Java development in the future, that can also help with the whole collaboration process.

I haven’t looked at multiple inputs/outputs closely, but have designed my layer base class to accept them, based on the python Layer class. Dense and Flatten only accept 1 input/output. I would be glad to see the USE cases though.

As far as test cases,  I am copying test cases from python when it makes sense. Some python test cases use numpy for comparison, which is not concise when porting directly to Java.  Some tests don’t exist, so I have to create them from scratch.
I find that I first port the Python Keras  code over, then immediately do the test case, that in turn results in fixing the original port of the Java Keras code.
Some python ism’s do not port easily, like accessing tensor values using overloaded operators.

I've heard from Dhruv that the JS code is much better to port over

Just a quick mention that anyone interested in participating to the development of Keras API for Java is invited to join our monthly meeting about this topic, next session is Friday August 28th at 9am PST and you can find the agenda and links here. Thanks!

@karllessard, @Craigacp : In my work analyzing Callbacks, I  have discovered a lot of dependencies on Model and Layers, and TensorBoard assumes a Keras model is exported in JSON Keras format from the TensorBoard callback.
As a result, I am starting initial work on a version of Models and Layers. I will probably do it in phases with the first phase including framework.layers.InputLayer, framework.layers.Dense, framework.layers.Flatten, as well as the initial framework.models.Model and framework.models.Sequential classes. Do we still want this PR, #65? Or, just create a new one when I get my first working Phase 1 version done ? My currentModel/Layersbranch depends on Metrics (Phase 1 and 2) and the new #174 TTypes.
My initial approach is to mimic Keras, but maybe we should have some discussion on this.

Hey @JimClarke5 , sounds good to me, I suggest we start a new PR and close this one.


fatal error with GuzzleHttp

reported at https://wordpress.org/support/topic/error-guzzlehttpexceptionrequestexceptiongeterrors/

Hello, i found this error message after authorized Google Cloud Storage.
The authorization is success, but my entire WordPress installation showing this error message:
Fatal error: Call to undefined method GuzzleHttp\Exception\RequestException::getErrors() in D:\root\home\dev078\wp-content\plugins\wp-stateless\lib\classes\class-bootstrap.php on line 934
I running in development environment that currently working in my local computer using addressing my domain into local IP (<IP_ADDRESS>).
Is this plugins require that website need to be online?
I also has meet the requirement of GuzzleHttp below:
http://docs.guzzlephp.org/en/stable/overview.html#requirements
Thank you for your help

Thank you, now this problem fixed for GuzzleHttp by using the hotfix.


SyntaxError: unexpected token

doesn't work on my Linux ?
> lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 17.04
Release:        17.04
Codename:       zesty

> node -v
v4.7.2

> lspe
/usr/local/lib/node_modules/ls-psvr-encoder/node_modules/string-kit/lib/unicode.js:49
        return String.fromCodePoint( ... array ) ;
                                     ^^^

SyntaxError: Unexpected token ...
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:373:25)
    at Object.Module._extensions..js (module.js:416:10)
    at Module.load (module.js:343:32)
    at Function.Module._load (module.js:300:12)
    at Module.require (module.js:353:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (/usr/local/lib/node_modules/ls-psvr-encoder/node_modules/string-kit/lib/string.js:54:13)
    at Module._compile (module.js:409:26)
    at Object.Module._extensions..js (module.js:416:10)

I believe your version of Node is too old. Can you upgrade to the current Node LTS version and try again?

I'm on Ubuntu 17.04, using nodejs from the repo. You should give some
instructions about the minimum version required:
https://nodejs.org/en/download/package-manager/
On Fri, Dec 29, 2017 at 6:58 PM Andrew Grathwohl<EMAIL_ADDRESS>wrote:

I believe your version of Node is too old. Can you upgrade to the current
Node LTS version and try again?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/littlstar/ls-psvr-encoder/issues/5#issuecomment-354478992,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB12xfmwyHxDRsfZMIbCgDQ00S8eDgtyks5tFSgpgaJpZM4RO5Ls
.

Using nodejs from the above repo just killed my nodejs install. npm rebuild doesn't solve it. I give up.
➜  lspe -i 1440P_6000K_142452572.mp4 -d 180 -t sbs
module.js:664
  return process.dlopen(module, path._makeLong(filename));
                 ^

Error: The module '/usr/local/lib/node_modules/ls-psvr-encoder/node_modules/get-cursor-position/build/Release/pos.node'
was compiled against a different Node.js version using
NODE_MODULE_VERSION 46. This version of Node.js requires
NODE_MODULE_VERSION 57. Please try re-compiling or re-installing
the module (for instance, using `npm rebuild` or `npm install`).
    at Object.Module._extensions..node (module.js:664:18)
    at Module.load (module.js:554:32)
    at tryModuleLoad (module.js:497:12)
    at Function.Module._load (module.js:489:3)
    at Module.require (module.js:579:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/usr/local/lib/node_modules/ls-psvr-encoder/node_modules/get-cursor-position/index.js:1:74)
    at Module._compile (module.js:635:30)
    at Object.Module._extensions..js (module.js:646:10)
    at Module.load (module.js:554:32)

You'll just need to start over again to fix that problem.
First, do:
npm -r littlstar/ls-psvr-encoder -g
Then do:
npm install littlstar/ls-psvr-encoder -g
That should fix it.


Python: Setup Typing

We should setup typing so that mypy can detect them

Here's how I did it for python-ksuid:
https://github.com/svix/python-ksuid/commit/9277a3b8e435fa408f417503762b9000ac1d618c
https://github.com/svix/python-ksuid/commit/148285eaf2bc4b84f8a1ac99ccae6d7c96b253a6

This is a duplicate of #52, though I'll close that one instead of this one as this one already has extra information. Please search existing issues before opening new ones.


Handle missing log directory

When the log directory doesn't exist the application fails to start. Manually creating a folder named log in the application's root directory fixes the problem. It should be created automatically on install.

Creation of 'logs' directory is forced on startup in 460c1bd to handle NLog not creating it automatically.


Fix versioning bug in release tooling

When the release tooling iterates through all of the defined Pods, it uses the range(of:) String API to find the first occurrence of the pod and then bumps the versioning. Because the API only finds the first occurrence, each pod will only get versioned bumped once. Because the FirebaseAnalytics substring now occurs in two places (after PR #7900 added 'FirebaseAnalytics/WithoutAdIDSupport'):
https://github.com/firebase/firebase-ios-sdk/blob/acf106a33d50e893f85a8563a19557b3df6110d1/Firebase.podspec#L37
and
https://github.com/firebase/firebase-ios-sdk/blob/acf106a33d50e893f85a8563a19557b3df6110d1/Firebase.podspec#L72
it was only bumped once (the first occurrence).
A new String API has been added to collect all of the ranges of the occurrences of a given substring.
Fixes #8179

Changes LGTM - were you able to validate this works locally and doesn't have any unintended consequences?

I feel we have enough logic here to start adding tests to it. Test for this piece of logic could be giving different podspecs as inputs and validate expected outputs. Fortunately SPM provides pretty good support for tests.


add styles.cursorDefault to mini context menu.

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/22310
PROPOSAL: https://github.com/Expensify/App/issues/22310#issuecomment-1622492862
Tests

Open any report.
Send any message.
Hover on message, Observe the action menu pops up.
Move the mouse cursor to the space between two action menus and verify the cursor is a default and not a pointer.

[x] Verify that no errors appear in the JS console

Offline tests

N/A.
QA Steps

Same as Tests step.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/41129870/10677c2b-2db5-4549-ab91-d21fda7dd3d0

Mobile Web - Chrome
https://github.com/Expensify/App/assets/41129870/aec19b71-c290-42df-947e-b96d4690feeb

Mobile Web - Safari
https://github.com/Expensify/App/assets/41129870/ea63504b-0945-42b6-8ec9-ff21e1a90278

Desktop
https://github.com/Expensify/App/assets/41129870/7689afd4-9eee-4ab2-ae04-9e6eb5fa66c0

iOS
https://github.com/Expensify/App/assets/41129870/2abfcb87-126f-4778-875b-ab8fbcbb23f0

Android
https://github.com/Expensify/App/assets/41129870/194845d1-5f98-4005-92ed-07fd9f8a4dd3

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/Li357 in version: 1.3.40-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Julesssss in version: 1.3.40-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/Li357 in version: 1.3.42-0 🚀

platform
result

🤖 android 🤖
cancelled 🔪

🖥 desktop 🖥
cancelled 🔪

🍎 iOS 🍎
cancelled 🔪

🕸 web 🕸
cancelled 🔪

🚀 Deployed to production by https://github.com/jasperhuangg in version: 1.3.42-26 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


added test for MPCTensors

added tests for AST
fixes #3

Hey @aanurraj - I noticed that you've stored the tests under tests/tensor. Wouldn't it be a good idea to put these under tests/sympc/tensor instead? So as to follow the structure of sympc. Also, that is how is being done in Syft, so it might make more sense?

Hey @aanurraj - I noticed that you've stored the tests under tests/tensor. Wouldn't it be a good idea to put these under tests/sympc/tensor instead? So as to follow the structure of sympc. Also, that is how is being done in Syft, so it might make more sense?

done :)

Looks good - but checking the tests coverage:
src/sympc/tensor/mpc_tensor.py            152     22     60      9    82%   74->77, 77-81, 111->112, 112-118, 120->121, 121, 140-149, 159->160, 160, 188->189, 189, 211->212, 212, 283, 291->292, 292, 301->308, 322->327, 327

It seems that we need to cover more lines.
Could you add test cases/update the ones already added in this PR such that we get 100% coverage (if there is any problem reaching 100% you can ping me on slack) :D

Looks good - but checking the tests coverage:
src/sympc/tensor/mpc_tensor.py            152     22     60      9    82%   74->77, 77-81, 111->112, 112-118, 120->121, 121, 140-149, 159->160, 160, 188->189, 189, 211->212, 212, 283, 291->292, 292, 301->308, 322->327, 327

It seems that we need to cover more lines.
Could you add test cases/update the ones already added in this PR such that we get 100% coverage (if there is any problem reaching 100% you can ping me on slack) :D


[master] ConcurrentModificationException from MetaDataRepository _registered

https://issues.apache.org/jira/browse/OPENJPA-2767
Signed-off-by: Will Dazey<EMAIL_ADDRESS>

Also LGTM


Question: create spectogram from audio file with Microcharts?

Hello,
I would like to know if there is a way to create a spectogram related to the audio record?
I would like to create a spectogram like this.
Regards,

that is not possible with this library at this time.


Deprecate PySOA

Since PySOA is a public repo, we need management approval in order to change the status to deprecated.

What do we believe next steps for this are?

LGTM :)

LGTM


remove output directory from file list for digest

Added a filter function to remove output files created by chart-verifier from the list of files used to calculate the sha256 digest
Closes #316

Thanks @dcurran90 we'll consider this for the next release


Button: hierarchies and types are not correctly mapped to antd type and variant

The description of the bug or the rationale of your proposal
The  hierarchy and type of the button type should have a well-defined mapping to the corresponding antd props, to have a consistent behavior with our design system.
Such mapping is currently implemented via simple boolean cheks, and also the variant prop of antd is not used, which I think should address some issues like the outlined type explained below.
A thorough map of our enums, implemented for example with a switch case statement, should be a more solid approach to map our props to antd.
A snippet of code for replicating the issue or showing the proposal usage if applicable
For example a button with hierarchy='primary' and type='outlined  should look like the following, with a white background:

However, the current mapping passes the ghost=true prop to antd, which also makes the background transparent, which is undesired:

The hierarchy Danger has the same issue.
The expected result for your bug
Each combination between hierarchy and type should result in the correct button layout and colors.
Your environment
Not relevant for this issue.

Button styling is currently customized in the design system, however with v5.21.0 antd introduced button variants that could help us remove customizations and use their own implementation instead

Button styling is currently customized in the design system, however with v5.21.0 antd introduced button variants that could help us remove customizations and use their own implementation instead

Yes I saw the variant in the docs, I see from our package.json that we are already on the v5.21.1, so yes I agree with you, we could use variants to customize the button for our needs


windows: uv_close destroys stdio handles on Windows

The equivalent unix code explicitly checks for stdio fds before closing them.
/cc @bnoordhuis : this makes it not possible to use stdio inside workers in windows.

I propose implementing something like this and calling it instead of calling CloseHandle directly:
void uv__close_win_handle(HANDLE handle) {
  if (handle->handle != GetStdHandle(STD_INPUT_HANDLE) &&
      handle->handle != GetStdHandle(STD_OUTPUT_HANDLE) &&
      handle->handle != GetStdHandle(STD_ERROR_HANDLE)) {
    CloseHandle(handle);
  }
}

I can start working on the PR myself to do the work if you acknowledge this is ok

I'm not the right person to ask.  Handing off to @piscisaureus.

Just for my understanding, is the problem that uv- win closes stdio handles when it shouldn't? Or do we just need these checks as a means to catch bugs?

The checks are so that stdio handles are never closed both to match unix and allow them to be used globally in a process.  CloseHandle is called from like 40 different places that dont check for stdio, hence the indirection

@petkaantonov I'm cool with it, but since closing handles is not too uncommon maybe it'd make sense to cache the GetStdHandle() return values somewhere instead of calling them every time.
Another thing is that streams that were opened from a crt fd should really be closed with close() and not CloseHandle(), otherwise the fd allocation is leaked. Libuv doesn't really do that at the moment.

The values can change so the result cannot be cached, but I really doubt they have any performance effect.

Closing since this https://github.com/libuv/libuv/pull/396 landed.


Parsed output is not really parsed

It is the same as in Raw view, just green colored.
Shouldn't it be more readable?

Hi,
Thank you for issue report. It means that the app didn't recognised the response as a JSON response. This means that there were no proper Content-Type header set for the response. I can guess it's something like text/html and this is what the app is trying to highlight (markup source). If the server returns correct content type then you'll have full support for syntax highlighting.
I'm inclined to close this issue as WAI.

Will test it tomorrow and provide response.

Thank you.

Sorry for being late!
You were right. If content-type is correct, it is beautifully parsed!
So a well known, big company does not send correct headers... :D

...and thank you for the app!

No problem.
Every kind of company can make a mistake ;)
And thanks for using it :)


new operator proposal: \ (inverted division)

proposal
I'd like to propose a new arithmetic operation \ (inverse division), i.e. b\a = a/b
cf. It's supported in Julia.

x \ y | inverse divide | equivalent to y / x

ref. https://en.wikipedia.org/wiki/Division_(mathematics)#Notation
motivation
It would be easy to write an expression like ans\3 instead of 3/ans. This will reduce the number of key types.
technical concern
If an escape character is needed in the future, we may consider ` (backtick) instead to avoid conflict.
cf. Powershell use backtick for escaping.

Thanks for the suggestion. I considered it, but decided not to assign functions to "" because it is easily confused with "/".
Instead, I would like to consider: while the evaluated value is displayed, if the cursor is moved to the beginning of the line by pressing the Home key, replace the evaluated value with "ans".

This choice sounds reasonable to me. Thank you for your consideration!
For now, let me leave this issue open, and please feel free to close it when you open an issue for the alternative.


Cucknorris plugin outputs no fortunes found at first use

Hello,
I just installed the plugin and got the error when running chuck
fortune: /home/adrien/.oh-my-zsh/plugins/chucknorris/fortunes: No fortune files in directory. fortune:/home/adrien/.oh-my-zsh/plugins/chucknorris/fortunes not a fortune file or directory No fortunes found
To solve the problem I ran zsh /home/adrien/.oh-my-zsh/plugins/chucknorris/chucknorris.plugin.zsh

What did you do to install the plugin? Normally it generates that fortune data file as part of its initialization when you start a new zsh session and Oh My Zsh initialization runs.

I installed oh-my-zsh on archlinux.
I added the plugin to the conf.
It told me I didn't have the fortune command so I installed "community/fortune-mod".
After that I had the following error
fortune: /home/adrien/.oh-my-zsh/plugins/chucknorris/fortunes: No fortune files in directory. fortune:/home/adrien/.oh-my-zsh/plugins/chucknorris/fortunes not a fortune file or directory No fortunes found

That makes sense. In the current implementation, the plugin only gets a chance to generate the fortune file at shell startup time. Restarting your shell session like that will fix it.
The chuck command could also be modified to check for the fortune file and generate or update it each time it's called, instead of at shell startup time, to fix this.


Sunsynk settings for when using the Aux port as an output

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Currently I can add a options to change the Aux port from Generator to Aux (excess solar) output but I cant then change the associated parameters
Expected behaviour
Pl add the additional features to access the other Aux port settings>
-Solar Power,
-Aux Load OFF Batt and
-Aux Load ON Batt
Possible Solutions
No response
Mode
full
Context / Reason
When adjusting the settings from Gen input to Aux output (Excess solar) I find it necessary to often change the settings based on various external factors

Hi. I'm not sure I understand this request. Are you looking for the ability to be able to toggle these settings from the card?
If so can you share the sensor/switch that you have that lets you do this?

Firstly thank you for a phenomenal flow card.
Pl cancel the request as this is not a power flow card related but more the
modbus integration which allows setting features
J
On Wed, 27 Dec 2023 at 15:18, slipx06 @.***> wrote:

Hi. I'm not sure I understand this request. Are you looking for the
ability to be able to toggle these settings from the card?
If so can you share the sensor/switch that you have that lets you do this?
—
Reply to this email directly, view it on GitHub
https://github.com/slipx06/sunsynk-power-flow-card/issues/196#issuecomment-1870300045,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIACBA4DSBYJRWIBPNJ2JBTYLQN2FAVCNFSM6AAAAABBELHPA2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZQGMYDAMBUGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Set up default settings

I'm seeing this message:
:: <-  elixir window/showMessage: {'type': 3, 'message': 'Did not receive workspace/didChangeConfiguration notification after 5 seconds. Using default settings.'}

If you set up settings in the "settings" entry of LSP-elixir.sublime-settings, they will be sent with workspace/didChangeConfiguration. It will allow users to customize them, too.

Done. Thanks for the report. Had some trouble finding what the settings exist. I'll bring it up with elixir-ls people.

Usually what language server maintainers do is to make a list of the settings in their README like this:
"elixirLS.dialyzerEnabled": true,
"elixirLS.dialyzerWarnOpts": [],
"elixirLS.dialyzerFormat": "dialyxir_long"

You've turned this path-notation for keys into python dicts, but note that for ST4 you can take these path-notation keys and literally paste them into the "settings" dict.

OK, makes sense to use the dot notation. Is this documented anywhere?
I've chosen the dicts since that's what all other projects seem to use.

@rwols Wait, will the dot notation work in ST3?

The dot notation won't work for ST3 :)

Sigh. :ok_hand:  I'll revert it for now.

Also yeah, I'll update the docs about this. There are already examples of this in https://github.com/sublimelsp/LSP/blob/st4000-exploration/docs/index.md (see dockerfile, julia, python). But "settings" should have its own section with some clear explanation and examples.

Thanks for the help, I'll read through that.
It's taking me a long time to get up to speed with sublime package dev. It's not that well documented for 3, let alone 4. That's what you get when you work with closed source I guess.


IMAP-Lo Science Direct Event Decompression

Change Summary
Overview
This PR adds a Science Direct Event data class for IMAP-Lo L0 to L1A processing. This data class includes methods to decompress the direct event data.
New Dependencies
None
New Files

compression_tables.py

decompression tables for science direct events
decompression tables for science counts

lol0.py

general Lo L0 data class to be inherited by all data product classes
mainly used for parsing the packet data and setting attributes

science_direct_events.py

data class for handling science direct event L0 data during L1A processing
includes methods for data decompression

Deleted Files
None
Updated Files
None
Testing

Added test_science_direct_events.py with unit tests for science direct event decompression. These unit tests should test most of the code added, but because I don't have compressed DE data, some workarounds were used to fully test the functionality. These tests will need to be updated when data becomes available. Currently, the tests will only work if the code in init() is commented out because there is no data to input. Not sure of a better way to handle this, but the tests will fail in the pipeline checks.

NOTE: I did not add any tests for the class lol0.py class that will be inherited by all the L0 data classes. I will be unable to test this until my next PR when I add classes for packets that I have available or when I get DE data.

Also, you probably want to add some kind of lo.rst file for documentation in this PR

Would everyone be ok with me adding pass at the beginning of each unit test since I needed to comment out the class's init for the unit tests to run since I don't have data currently? Once I get a Sci-De packet this should all run normally. Or would it be worth trying to fake space-packet-parser output?

Would everyone be ok with me adding pass at the beginning of each unit test since I needed to comment out the class's init for the unit tests to run since I don't have data currently? Once I get a Sci-De packet this should all run normally. Or would it be worth trying to fake space-packet-parser output?

You should add a pytest.xfail() or pytest.skip() (either to individual tests, or to the module) instead. But, yes that is fine with me to get this in.


Refactor: configmanager Package's Readability Enhancement

Signed-off-by: Daniel Hu<EMAIL_ADDRESS>Pre-Checklist
Note: please complete ALL items in the following checklist.

[x] I have read through the CONTRIBUTING.md documentation.
[x] My code has the necessary comments and documentation (if needed).
[ ] I have added relevant tests

Description

configmanager package's readability enhancement
Related Issues

N/A
New Behavior (screenshots if needed)

N/A

we can put LoadCofig in pluginengine, Then all main logic func can be starts with uppercase. I think export func which only used in module is not a good choice

I'll take a deeper think at this afternoon.


Make a small tutorial or navigation legend

added a window at the beginning of cube window, dismissed by any key, timer starts after window is dismissed
request feedback


feat: add kinesis schemas

Current dependencies on/for this PR:

11-29-feat_add_lambda_schemas

PR #12   👈

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: j7Bu8pCEUHqFlKED3zbX


Display the file names which caused error in compilation

Describe the feature
When the files in the inventory is not structured properly, kapitan compile outputs the message
Invalid inventory structure followed by an error message. The error message, however, doesn't contain the actual path or name of the wrongly structured inventory file. Displaying the name of the file will help save a lot of time searching for it.
To Reproduce
Steps to reproduce the behavior:

Open a .yml file in the inventory
make some changes in the parameters.kapitan.compile section.
E.g input_paths key expects an array, provide a single value instead

parameters:
  kapitan:

    compile:

      - output_path: .
        input_type: jinja2
        input_paths:
          - templates/docs

      - output_path: manifests/html
        input_type: jinja2
        input_paths: templates/index.html

run kapitan compile
see error

Output
Invalid inventory structure

Error: 'templates/index.html' is not of type 'array'
On instance:
"templates/index.html"

Expected behavior
Error message should contain the name or path to the file.

#430 is a similar issue.

Closing as duplicates exist and it's better to track them all in one place. :)


Debug wasmi error for alloc

What
#968
Why
[TODO: Why this change is being made. Include any context required to understand the why.]
Known limitations
[TODO or N/A]

Closing in favor of https://github.com/stellar/rs-soroban-env/pull/972 and https://github.com/stellar/rs-soroban-sdk/pull/1045


Bad path in example

Wrong path in "Issue: Tamper protection is reported as disabled" section
$ sudo grep -F '\[{tamperProtection}\]: Feature state:' /Library/Logs/Microsoftmdatpmicrosoft_defender_core.log \| tail -n 1
should probably be
$ sudo grep -F '\[{tamperProtection}\]: Feature state:' /Library/Logs/Microsoft/mdatp/microsoft_defender_core.log | tail -n 1

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

ID: b466be31-a444-e4f0-f56e-1b514776df63
Version Independent ID: b466be31-a444-e4f0-f56e-1b514776df63
Content: Protect macOS security settings with tamper protection
Content Source: microsoft-365/security/defender-endpoint/tamperprotection-macos.md
Product: m365-security
Technology: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@mjcaparas Please verify PR #8668  and merge if correct.

@RAJU2529 Thank you very much for the contribution and updating the documentation with PR. @pensivepaddle Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Updates to allow submitters to use multiple ParallelCluster clusters …

…at the same time
Update the slurm config to add the ClusterName to the path.
Enable multiple clusters to be used by submitters
Change the mount point to include the cluster name so it is unique between clusters. Updates scripts and config paths to include the cluster name in the paths. Add a symbolic link so the head and compute nodes have access to the same path as the submitter.
Resolves #139
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Going to do a fresh deployment, but looks good.

Found one more issue. Need to update /etc/resolv.conf to add {{ClusterName}}.pcluster to the DNS search path or else srun fails with the following message:
srun: error: get_addr_info: getaddrinfo() failed: Name or service not known
srun: error: slurm_set_addr: Unable to resolve "od-r6i-large-dy-od-r6i-large-cr1-2"
srun: error: fwd_tree_thread: can't find address for host od-r6i-large-dy-od-r6i-large-cr1-2, check slurm.conf
srun: error: Task launch for StepId=12.0 failed on node od-r6i-large-dy-od-r6i-large-cr1-2: Can't find an address, check slurm.conf
srun: error: Application launch failed: Can't find an address, check slurm.conf

Found one more issue. Need to update /etc/resolv.conf to add {{ClusterName}}.pcluster to the DNS search path or else srun fails with the following message:
srun: error: get_addr_info: getaddrinfo() failed: Name or service not known
srun: error: slurm_set_addr: Unable to resolve "od-r6i-large-dy-od-r6i-large-cr1-2"
srun: error: fwd_tree_thread: can't find address for host od-r6i-large-dy-od-r6i-large-cr1-2, check slurm.conf
srun: error: Task launch for StepId=12.0 failed on node od-r6i-large-dy-od-r6i-large-cr1-2: Can't find an address, check slurm.conf
srun: error: Application launch failed: Can't find an address, check slurm.conf

The actual way I solved this is by setting the LOCALDOMAIN environment variable in the modulefile.
The problem with putting it in /etc/resolv.conf like on the head node is that it then globally affects the submission server.
The environment variable overrides the search path in /etc/resolv.conf when the modulefile is loaded.
This prevents DNS resolution from being for the wrong cluster.


fix fix remove unused remote files

Proposed changes
The order of these two variables is reversed.
auto [cooldown_replica_id, cooldown_term] = t->cooldown_conf();
ref:
term and cooldown_replica_id
struct CooldownConf {
int64_t term = -1;
int64_t cooldown_replica_id = -1;
};

run buildall


Requesting to put the 'Cookie Preferences' icon in the right side.

I would like to request the team to allow me to put the 'Cookie Preferences' icon on the right side so that the icon doesn't get overlapped with the index contents.
Issue

Solution

If you think it will be useful to other users I would like to work on this. Thanks.

Hi @cjyabraham
Is this request acceptable ?

Hmm, I see what you're saying, however, the downside is that it will now be at a different location from all other CNCF sites. I think personally I'd prefer to keep common UI elements like this consistent across all sites.

I don't think it will look good if it's more transparent and words can show through. It'll look a bit cluttered and messy. Is this really a problem? If words are behind the cookie, can't you just scroll up to see them?


Small fix if called from an already elevated prompt

command_to_run escaping seems not to be suitable to be passed to exec.  Found when testing the other PR. I guess rarely someone calls sudo from an already elevated prompt...

Yay, thanks!


comments in properties file translate to the JSON structure

need to ignore lines that start with # in properties files

Thanks!!!


Add error filter for tx validator errors

Quick fix for the tx.fhir.org validation errors that we have seen slip through, as in #523 and #524. Now any message containing [Error from tx.fhir.org/r4: will be ignored.
Testing notes
Testing this is tricky since this error is flaky. I used https://mockoon.com/ to create a mock service that always returned a bad error, with the attached response. validate.txt  I got this response from validator.fhir.org using a DocumentReference with a bad mime type "junk/pdf". Note I replaced the sessionID field at the bottom with a mockoon template helper so the sessionID isn't consistent (the validator session startup job broke when every instance got back the same session ID)

A bit of an aside but Mockoon's implementation for creating mock services is somewhat interesting to me as it something we sometimes need to do for client testing.  Might be worth looking at for ideas in that area.


Work with multiple selects

This is a great package for Atom thank you, I would love it to be able to handle multiple selects, so if you select multiple strings and then call this package you can then surround them in the desired tag.
At the moment it wraps all selections in a  tag and then when you type the tag you want it only updates the last one.

Will check this option.
Thanks for suggestion.


vue.config.js doesn't have a "target" parameter

The example completely falls down when looking at the vue.config.js portion of the documentation. Not only isn't there a "target" parameter in the default generate vue.config.js, you don't show us what it should look like at all.

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

ID: c924f1c1-823e-3e29-bf09-405fc4c6bbb8
Version Independent ID: 585c2426-5f9b-8499-8eda-0e56bd64c93c
Content: Create an ASP.NET Core app with Vue - Visual Studio (Windows)
Content Source: docs/javascript/tutorial-asp-net-core-with-vue.md
Product: visual-studio-windows
Technology: vs-javascript
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@thobbsinteractive -- Tim, thank you for your feedback.  Please consider these resources:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@Mikejo5000 -- Mike, please look into this issue.

@thobbsinteractive I tested this on 17.3.0 with both the JS and TS Standalone Vue template, and I see the target: 'https://localhost:5001', section in vue.config.js for both projects. What version of VS are you using?  Since I can't repro the issue, please use the Report a Problem tool to report the issue. When you do this, your report becomes part of the work items that are listed in the in the Visual Studio Developer Community, which allows you to engage directly with the engineering team while they address and fix the problem.

I will close this and we can re-open if there is something more for me to look at here. Thanks!


ARROW-11289: [Rust][DataFusion] Implement GROUP BY support for Dictionary Encoded columns

I am throwing up a draft PR just to give people a heads up that I am working on this feature
This PR adds support for GROUP BY with for columns of Dictionary type.
This definitely will conflict with Dandan's implementation of vectorized hashes / group by in https://github.com/apache/arrow/pull/9213 and https://github.com/apache/arrow/pull/9116. I plan to rework this PR once his are merged.
The code basically just follows the pattern (aka is mostly copy/paste) from the take kernel: https://github.com/apache/arrow/blob/master/rust/arrow/src/compute/kernels/cast.rs#L294
I chose the "correct first, optimzie later" approach here -- there are many ways to make this code faster, especially when grouping on string types,
It feels like a lot of copy/paste and I don't feel great about the coverage of all the types. I am contemplating some more interesting / full coverage as well as  if there is some way to reuse the recurring pattern of switch and dispatch for a dictionary types.

Codecov Report

Merging #9233 (a995b6c) into master (1393188) will decrease coverage by 0.01%.
The diff coverage is 60.41%.

@@            Coverage Diff             @@
##           master    #9233      +/-   ##
==========================================
- Coverage   81.61%   81.59%   -0.02%
==========================================
  Files         215      215
  Lines       51867    51980     +113
==========================================
+ Hits        42329    42412      +83
- Misses       9538     9568      +30

Impacted Files
Coverage Δ

...ust/datafusion/src/physical_plan/hash_aggregate.rs
82.92% <49.12%> (-1.87%)
:arrow_down:

rust/datafusion/src/test/mod.rs
85.51% <53.33%> (-4.33%)
:arrow_down:

rust/datafusion/src/physical_plan/hash_join.rs
82.85% <57.37%> (-1.86%)
:arrow_down:

rust/datafusion/src/execution/context.rs
89.92% <100.00%> (+0.35%)
:arrow_up:

rust/parquet/src/arrow/schema.rs
91.66% <0.00%> (+0.16%)
:arrow_up:

rust/arrow/src/array/transform/fixed_binary.rs
84.21% <0.00%> (+5.26%)
:arrow_up:

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 eaa7b7a...a995b6c. Read the comment docs.

Codecov Report

Merging #9233 (a995b6c) into master (1393188) will decrease coverage by 0.01%.
The diff coverage is 60.41%.

@@            Coverage Diff             @@
##           master    #9233      +/-   ##
==========================================
- Coverage   81.61%   81.59%   -0.02%
==========================================
  Files         215      215
  Lines       51867    51980     +113
==========================================
+ Hits        42329    42412      +83
- Misses       9538     9568      +30

Impacted Files
Coverage Δ

...ust/datafusion/src/physical_plan/hash_aggregate.rs
82.92% <49.12%> (-1.87%)
:arrow_down:

rust/datafusion/src/test/mod.rs
85.51% <53.33%> (-4.33%)
:arrow_down:

rust/datafusion/src/physical_plan/hash_join.rs
82.85% <57.37%> (-1.86%)
:arrow_down:

rust/datafusion/src/execution/context.rs
89.92% <100.00%> (+0.35%)
:arrow_up:

rust/parquet/src/arrow/schema.rs
91.66% <0.00%> (+0.16%)
:arrow_up:

rust/arrow/src/array/transform/fixed_binary.rs
84.21% <0.00%> (+5.26%)
:arrow_up:

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 eaa7b7a...a995b6c. Read the comment docs.

If we are able to describe in the partitioning information that the partition is hashed by some column that is a dictionary, doesn't that allow us to perform very fast hashing (based on the dictionary indexes)?

If we are able to describe in the partitioning information that the partition is hashed by some column that is a dictionary, doesn't that allow us to perform very fast hashing (based on the dictionary indexes)?

If we are able to describe in the partitioning information that the partition is hashed by some column that is a dictionary, doesn't that allow us to perform very fast hashing (based on the dictionary indexes)?

@jorgecarleitao  yes I think that would be a great optimization, or possibly skipping hashing entirely and build the aggregate table entirely on the dictionary indexes -- I suspect this would work well in the common case, but we would have to handle the case where the dictionary itself is not the same across all record batches (and thus indexes in one record batch may not correspond to the same value in another)

If we are able to describe in the partitioning information that the partition is hashed by some column that is a dictionary, doesn't that allow us to perform very fast hashing (based on the dictionary indexes)?

@jorgecarleitao  yes I think that would be a great optimization, or possibly skipping hashing entirely and build the aggregate table entirely on the dictionary indexes -- I suspect this would work well in the common case, but we would have to handle the case where the dictionary itself is not the same across all record batches (and thus indexes in one record batch may not correspond to the same value in another)

This one is now ready for review. cc @andygrove @seddonm1  @jhorstmann and @Dandandan

@alamb looks good to me 👍

Thanks for the review @seddonm1 . @jorgecarleitao  or @andygrove  or @Dandandan  -- any concern of merging this PR?

I filed https://issues.apache.org/jira/browse/ARROW-11635 to track follow on performance work


Missing files *.conf.windows

Missing files ca.conf.windows, client.conf.windows, server.conf.windows . They are use in build-certificates-win64.bat . Only ca.conf, client.conf and server.conf are present

Did you try to build daedalus yourself or used the installer?


Database is missing from repo

The SDF database is missing from the repo, so when you run the Umbraco site, it breaks because it can't connect to the database.

did anyone get a copy of the sdf?


Translate the text

Thanks for the dataset.
I want to generate a motion in Korean.
so, I want to translate the text and make it Korean, is that okay?

Hi,
It should be okay to interpret the text. But it should only serve for
academic purposes.
On Tue, 17 Oct 2023 at 02:10, ChangeNext @.***> wrote:

Thanks for the dataset.
I want to create a motion in Korean.
I want to interpret the text and make it Korean, is that okay?
—
Reply to this email directly, view it on GitHub
https://github.com/EricGuo5513/HumanML3D/issues/99, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKRYNB2KQE3AXEFW77KMBSTX7Y4Q3AVCNFSM6AAAAAA6DM6E52VHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2DMOBRGU4DGNA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you for your response.
I will only use it academically!
Would it be okay to write a paper for a Korean conference?
I will never use it commercially.

Hi, sorry for the late reply. It should be fine.


Write a Python program to convert a string to an integer (without using built-in functions)

Description
Write a Python program to convert a string to an integer (without using built-in functions)
Input  : "123"
Output : 123

How to contribute

Save the solution in program/convert-a-string-to-an-integer-(without-using-built-in-functions).py file
Add convert-a-string-to-an-integer-(without-using-built-in-functions).py file in convert-a-string-to-an-integer-(without-using-built-in-functions) folder

!assign


Not able to get URL of .indd file

I am storing .indd file in cfs:gridfs and i am not gettng url of that file on meteor galaxy but i am able to get url on localhost. This issue only occur with .indd file if i was store .pdf / .doc / .jpg then it will work perfectly on both side localhost and meteor galaxy.
Please provide any way to get url of .indd file on meteor galaxy.

Did you find a solution?


Add new auxiliary setting for "files.autoGuessEncoding"

When I set the "files.autoGuessEncoding": true, sometimes it works wrong (recognize the GBK as Windows1252).

Add new auxiliary setting for "files.autoGuessEncoding", provides an encoding list to scope it.

https://github.com/Microsoft/vscode/issues/30857


Android - Distance - Location access not asking every time

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:  <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: n
Issue reported by: Applause - Internal Team
Action Performed:
Pre-condition: location on, app location access set to ask everytime

Launch app
Tap fab -- request money -- Distance
Note location access permission asked
Select only this time
Complete the distance request
Kill the app
Open the app again
Tap fab -- request money -- Distance
Note map loaded and no location access permission asked

Expected Result:
When location access is set ever time, after completing a distance request if user kills app and open again to make distance request,permission for location access must be asked.
Actual Result:
When location access is set ever time, after completing a distance request if user kills app and open again to make distance request,permission for location access is not asked.
Workaround:
n/a
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/20fe600b-527b-4865-b672-15de3196cfae
View all open jobs on GitHub

We think that this bug might be related to #wave-collect RELEASE 1

Working on reproducing...

I reproduced:
https://github.com/Expensify/App/assets/6833644/0b4b15f1-3816-4c15-9826-3ae2697872dd
It seems like we are saving your previously recorded location since selecting "Your location" as an option uses the location from the previous request without re-prompting for location permission.

Alright I also reproduced on prod v1.4.53-2 so this is not a blocker

Note map loaded and no location access permission asked

But if you try to select a point with current location, you will see you still have the permission, also go to settings and check permission, you will see you still have the permission, but if you wait about 15 min then open the app again, you will see the permission is revoked.
I think this the behavior in android, but unfortunately the documentation is very unclear as to exactly when these permissions get revoked.

I reproduced this but I think we are just not really giving the device time to revoke the permissions. It asks again if you switch to another app between opening New Expensify, but if you don't, it doesn't. I think this is totally fine.
https://drive.google.com/file/d/13vQA67TI3zHID1sTbnG3HmEIwueBQzsc/view?usp=sharing


python312Packages.aiomisc: 17.5.6 -> 17.5.15

Changelog: https://github.com/aiokitchen/aiomisc/blob/master/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

24 packages built:

python311Packages.aio-pika
python311Packages.aio-pika.dist
python311Packages.aiofile
python311Packages.aiofile.dist
python311Packages.aiomisc
python311Packages.aiomisc-pytest
python311Packages.aiomisc-pytest.dist
python311Packages.aiomisc.dist
python311Packages.aiormq
python311Packages.aiormq.dist
python311Packages.caio
python311Packages.caio.dist
python312Packages.aio-pika
python312Packages.aio-pika.dist
python312Packages.aiofile
python312Packages.aiofile.dist
python312Packages.aiomisc
python312Packages.aiomisc-pytest
python312Packages.aiomisc-pytest.dist
python312Packages.aiomisc.dist
python312Packages.aiormq
python312Packages.aiormq.dist
python312Packages.caio
python312Packages.caio.dist

@ofborg eval