unison: 2.53.0 -> 2.53.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/bcpierce00/unison/releases.
meta.description for unison is: Bidirectional file synchronizer
meta.homepage for unison is: https://www.cis.upenn.edu/~bcpierce/unison/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.53.1 with grep in /nix/store/mbhkr6jgj31ybvm32gj9v2dacv0nx8gi-unison-2.53.1
found 2.53.1 in filename of file in /nix/store/mbhkr6jgj31ybvm32gj9v2dacv0nx8gi-unison-2.53.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
unison
usync

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mbhkr6jgj31ybvm32gj9v2dacv0nx8gi-unison-2.53.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A unison https://github.com/r-ryantm/nixpkgs/archive/2d87db4b612fef808b5623c9996716e5ec8de2d9.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/mbhkr6jgj31ybvm32gj9v2dacv0nx8gi-unison-2.53.1
ls -la /nix/store/mbhkr6jgj31ybvm32gj9v2dacv0nx8gi-unison-2.53.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

unison
usync

Maintainer pings
cc @viric for testing.

This release should be skipped in favor of 2.53.2:

This release is identical to 2.53.1, except that the included version string is correct.


Handling Large Offsets for read on 32-bit Systems May Exceed usize Capacity

A potential issue in the system where i64 is used for file offsets. On 32-bit architectures, usize is limited to 2^31-1, which means that offsets near the upper limit of i64 could exceed usize capacities. This can lead to runtime errors or panics if not properly handled.

@Xuanwo
Just changing u64 in api.rs for all offset to usize followed by changes in all implementations should be fine, right?
Can I take it up?
Thank you!

Just changing u64 in api.rs for all offset to usize followed by changes in all implementations should be fine, right?

I don't have plan to change it yet. Please note that u64 is fine on 32 bit systems is ok.

I think if the underlying client uses usize then it could be an issue if the offset provided is pretty large.

Your understanding is correct. However, the proper solution is to have the underlying client accept u64 instead. Changing u64 to usize is incorrect.
That said, there's no immediate need to address this. We can revisit the issue if we encounter it with users on 32-bit systems.


APDS-455: Download results of ECL query

This pull requests extends the functionality of the concepts endpoint to support text/csv responses, where ECL queries are already supported.
https://jira.ihtsdotools.org/browse/APDS-455

Thank you!


Support primitive action types

This commit adds the ability for a Microcosm to process string actions. In these cases, a simple identity function is used.
This falls perfectly in line with the use case for lifecycle actions, and so they have also been transitioned into simple string keys.
This also fits the use case for a lot of simple actions, such as using a store to maintain a global menu state:
// UI actions
const openMenu = 'OPEN_MENU'

const UIStore = {
  open(state) {
    return { ...state, open: true }
  },
  register() {
    return { [openMenu]: this.open }
  }
}

app.push(openMenu)

Nice -- seems like a solid change.  :+1:

:+1:


Option to use system ANTs / N4 installation

Rolf suggested in the MIRTK chat to enable use of an already existing N4 installation instead of building ITK from scratch. This should be easy to facilitate with a change of the CMakeLists.txt and would shorten build time.

Just looked at the CMakeLists.txt, for what it's worth you can at least use an existing ITK installation to build the N4 binary included in Draw-EM. But of course I see the issue of having a duplicate N4 binary in the MIRTK installation PATH. I think the N4 binary build and installed as part of Draw-EM should probably have a different name, such as DrawEM-N4 to not shadow an existing ANTs installation. [besides the option to skip build/installation as part of Draw-EM altogether as suggested]

Just looked at the CMakeLists.txt, for what it's worth you can at least use an existing ITK installation to build the N4 binary included in Draw-EM. But of course I see the issue of having a duplicate N4 binary in the MIRTK installation PATH. I think the N4 binary build and installed as part of Draw-EM should probably have a different name, such as DrawEM-N4 to not shadow an existing ANTs installation. [besides the option to skip build/installation as part of Draw-EM altogether as suggested]

Thanks Andreas. The background to this is that I have suggested inclusion of DrawEM in the NixOS package of MIRTK. Heads up to @lheckemann who maintains it.

Thanks Andreas. The background to this is that I have suggested inclusion of DrawEM in the NixOS package of MIRTK. Heads up to @lheckemann who maintains it.

I see, thanks for this additional info. Sounds good to me.
Regarding N4, I think ANTs only contains a program called N4BiasFieldCorrection as part of the Examples folder. Is this one installed as part of the Nix package ants? Otherwise, I think N4 (as most of ANTs) is probably included in ITK 4 now. I wonder if it would be best for Draw-EM to use a custom Python script to execute the N4BiasFieldCorrectionFilter via SimpleITK instead. How would you handle such Python package dependency in Nix?
More generally, I wonder about a other challenges for creating a distribution package for Draw-EM at the moment:

How to handle data dependencies such as brain atlases (usually expected to be located in source tree).
Shell scripts may need changes as these refer to configuration files in the source tree. Also, neither these scripts nor the configuration files are being copied to installation at the moment. If these should be included in the distribution package, it would probably be necessary if the names of these Draw-EM commands would be encapsulated in some namespace, e.g., invocation via drawem <script> or mirtk drawem <script> (luckily the latter wouldn't clash with already existing mirtk draw-em binary whose source is in the tools/ folder).

I see, thanks for this additional info. Sounds good to me.
Regarding N4, I think ANTs only contains a program called N4BiasFieldCorrection as part of the Examples folder. Is this one installed as part of the Nix package ants? Otherwise, I think N4 (as most of ANTs) is probably included in ITK 4 now. I wonder if it would be best for Draw-EM to use a custom Python script to execute the N4BiasFieldCorrectionFilter via SimpleITK instead. How would you handle such Python package dependency in Nix?
More generally, I wonder about a other challenges for creating a distribution package for Draw-EM at the moment:

How to handle data dependencies such as brain atlases (usually expected to be located in source tree).
Shell scripts may need changes as these refer to configuration files in the source tree. Also, neither these scripts nor the configuration files are being copied to installation at the moment. If these should be included in the distribution package, it would probably be necessary if the names of these Draw-EM commands would be encapsulated in some namespace, e.g., invocation via drawem <script> or mirtk drawem <script> (luckily the latter wouldn't clash with already existing mirtk draw-em binary whose source is in the tools/ folder).

On an additional note, MIRTK extension packages such as Draw-EM can be build out of MIRTK source tree. Meaning, one can first build and install MIRTK with (MODULE_DrawEM=OFF, the default), and then separately clone and build this DrawEM repository. The FindMIRTK.cmake module included should find the previously installed MIRTK. This would allow for distributions to create separate distribution packages for these, which I would recommend (e.g., mirtk, mirtk-drawem).

On an additional note, MIRTK extension packages such as Draw-EM can be build out of MIRTK source tree. Meaning, one can first build and install MIRTK with (MODULE_DrawEM=OFF, the default), and then separately clone and build this DrawEM repository. The FindMIRTK.cmake module included should find the previously installed MIRTK. This would allow for distributions to create separate distribution packages for these, which I would recommend (e.g., mirtk, mirtk-drawem).

Regarding N4: how is DrawEM's N4 materially different from ANTs's? Does it even need to exist?
Data dependencies: yes, the data dependencies issue is the next one I ran into — I think the ideal solution here in principle would be for cmake to allow "substituting" ExternalProject sources. That way, no additional work would be necessary on the DrawEM side (and this problem would be solved for all uses of ExternalProject).
Out-of-source-tree build: I didn't realise this! That sounds like a much better solution than bundling everything in with mirtk itself.

Regarding N4: how is DrawEM's N4 materially different from ANTs's? Does it even need to exist?
Data dependencies: yes, the data dependencies issue is the next one I ran into — I think the ideal solution here in principle would be for cmake to allow "substituting" ExternalProject sources. That way, no additional work would be necessary on the DrawEM side (and this problem would be solved for all uses of ExternalProject).
Out-of-source-tree build: I didn't realise this! That sounds like a much better solution than bundling everything in with mirtk itself.


Support runtime checks of intersection types

Pull Request Description
Fixes #6165 by removing support for \ and providing support for &. As a result of that we can now request multiple type classes for a single value.
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] All code follows the
Scala,
Java,
All code has been tested:

[x] Unit tests have been written where possible.
[x] Remove support for \
[x] Check benchmarks for slowdowns & another.
[x] Check for duplicates

equality loses symmetry and reflexivity!

With https://github.com/enso-org/enso/pull/7796/files#r1324206886 the equality has reflexivity back.


[DOC] Add Examples to classifier docstrings that currently lack examples

Describe the issue linked to the documentation

Some of the classifiers don't include and examples section in their docstring. We should update the docstrings to include the examples section.

[x] ColumnEnsembleClassifier
[x] ElasticEnsemble #1584
[x] ProximityForest #1584
[x] ProximityTree #1584
[x] ProximityStump #1584
[x] TimeseriesForest #1583
[x] RandomIntervalSpectralForest #1583
[x] SupervisedTimeseriesForest #1583
[x] ShapeletTransformClassifier (example imports from non-public module and probably shouldn't)

Suggest a potential alternative/fix

Add an examples section to each of these classifiers. Ideally this would be possible by adapting the examples section of other classifiers, but I'm not 100% sure whether the same dataset will be good at illustrating all of them or not since this is not my area of expertise.

@TonyBagnall and @MatthewMiddlehurst classification isn't my area of expertise, so I don't know if the same dataset would be good for each classifier.
The current classifier examples all appear to either use dataset from load_unit_test or load_italy_power_demand. Could you comment as to which dataset would seem like a good option for each classifier?

We use load_unit_test for speed as they are all run in the testing process. We will get to them all eventually in the refactor.

As Matt says, unit test is there to reduce testing speed and is probably best for examples. Good first issue maybe?

But also the ones you list in particular are not stable and scheduled for revision or deprecation

For STC you can replace RotationForest with a RandomForest if it is an issue. Can people using the public release not import anything from contrib?

@TonyBagnall and @MatthewMiddlehurst I just wanted to collect the ones that needed the examples added so we had a record. Agree this is probably a good first issue.
@MatthewMiddlehurst I’m not sure on the contrib part. But the import with the leading “_” usually denotes from a non-public sub module.

Maybe we should refactor contrib as _contrib?

Do we want it to be private?
There is no structure or init files in contrib i'm pretty sure, its the sktime wild west. I don't really think rotation forest should even be there tbh, but its not a time series classifier so it has no other home.

maybe the rotation forest should go in the STC classifier itself? We do that for other specific classifiers, and I dont think the rotation forest implementation is ready to go into scikit-learn...

all these now have examples, rotation forest is now public


make a calculator with c++

Description
Description
Domain
CPP
Type of Contribution
No response
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

/assign

/assign


Account name not updating after renaming

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

Action Performed:

Open new two accounts
Send a money request from account A to account B
Observe account B on LHN which is an email
Go to account B and rename it to anything
Open account A and observe the name of account B on LHN

Expected Result:
Name should update on LHN after renaming account B
Actual Result:
Name does not update on LHN after renaming
NOTE : This bug occurs if only account A sends money request to account B before starting any report or posting any report action.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

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

Version Number: 1.3.22-0
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
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/cf22976c-e99d-438a-8faf-05c31dcbcffa
https://github.com/Expensify/App/assets/43996225/0da4a490-4d10-4aef-be79-1cb8030b6175
Expensify/Expensify Issue URL:
Issue reported by: @nathan-mulugeta
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1685439541539559
View all open jobs on GitHub

This looks like the issue in backend

@s-alves10 can you clarify what you mean by that? Is there another issue related to this?

@s-alves10 can you clarify what you mean by that? Is there another issue related to this?

This issue is related to syncing by push events from server.
By my observation in the above scenario, A doesn't receive any events when B renames.
This is the issue of backend, I think.

This is valid, but I don't think it's something we care enough about to fix.
The next time the user opens the app, the names should be updated. So it's not necessary to update the name immediately.


Gamified Constitutional Literacy Platform Using Appwrite

"Developed an interactive, gamified platform to simplify the learning of the Indian Constitution. Utilized Appwrite for authentication, real-time data, and backend management to deliver a scalable and engaging user experience."

@Sidd-17052004 the project repo does not feature any Appwrite product being used, which is mandatory for the hackathon. Please submit a project that fulfills the rules of the hackathon and create a new PR.


odo dev on podman: Add support for devfile volume

What type of PR is this:
/kind feature
What does this PR do / why we need it:
Which issue(s) this PR fixes:
Fixes #6330
PR acceptance criteria:

[x] Unit test

[ ] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

Wondering if we shouldn't generate EmptyDir volumes when the Ephemeral preference is true, to be consistent with the cluster mode? I just saw that Podman could support EmptyDir volumes by creating anonymous volumes (containers/podman#15473). (But if you prefer, we can handle this in a separate issue/PR as well).

Effectively, podman can handle emptyDir and PVC, and they are instantiated practically the same way: both are podman volumes, and emptyDir ones will be removed when pod is removed, as opposed to the PVC will be removed by odo at the same time the pod is removed.
I didn't want to add complexity for this small difference. But we can handle this in a separate PR is necessary

/override ci/prow/v4.11-integration-e2e
Pass on IBM Cloud


Define EnhancedTextarea hintText as string

Hello! Is it possible to make hintText node instead of string? It's very useful for internationalisation using components (like react-intl).
https://github.com/callemall/material-ui/blob/v0.18.0/src/TextField/EnhancedTextarea.js#L37

A related issue is thatTextField passes prop hintText to EnhancedTextArea. The former defines its type as PropTypes.node while the later expects PropTypes.string, resulting in the warning

Warning: Failed prop type: Invalid prop hintText of type object supplied to EnhancedTextarea, expected string.

Hello! Is it possible to make hintText node instead of string?

Yes! Do you want to submit a PR 😄 ?


DSA 1.2.20 Aho-Corasick Algorithm

Description
Give an introduction and explain the working of Aho-Corasick algorithm.
Task

[ ] Documentation
[ ] Audio
[ ] Video

Note

While requesting the issue to be assigned, please mention your NAME & BATCH NUMBER.
Kindly refer to Issue Guidelines.
In case you see that the issue has already been assigned, try to request for a different task within the same issue.
Changes should be made inside the DSA/directory & DSA branch.
Name of the file should be same as the issue.
This issue is only for 'GWOC' contributors of 'DSA' Domain.

Vanshika Sable batch-1- documentation

@pjdurden  assign me this issue

Assign this to me pls,
Name: Mohd Uvaish Siddiqui
Batch: 7
Task: Documentation

@aishapervin @0xUvaish Assigned to you

@Subhradeep10 i also had to do the documentation part,so plz clarify whether i am assigned the documentation part or not.Thankyou

@aishapervin You have to do only the documentation part

@goelesha Pls assign this issue to @aishapervin @0xUvaish

@mehabhalodiya This issue is opened by Prajjwal Sir, not Esha Mam

@0xUvaish Are you sure because no one is currently assigned to this issue, that's why I requested for it. If possible please check with the mentors/supervisors.

@pjdurden I would like to work on this issue.


Added dpStatus configuration for Wofea garage door

Wofea uses dp 101 to indicate door status.
Examples
Closed
{"1":false,"7":0,"101":false}
Open
{"1":true,"7":0,"101":true}
Note dp 1 is still used for state changes.

Hi longzheng,
how does your config looks ike? Since the new version my wofea garage door crashes homebridge.
Mine Loks like:
{
                    "dpAction": 1,
                    "dpStatus": 2,
                    "flipState": false,
                    "model": "WiFi Switch Garage Door Controller",
                    "type": "GarageDoor",
                    "id": "xxx",
                    "name": "Garagentor Wofea",
                    "manufacturer": "",
                    "key": "xxx",
                    "ip": "xxx"
                },

Thanks a lot.
Mark

@MarkM500 your homebridge is crashing because the dpStatus is incorrect and the plugin's garage code has a bug which can't handle the invalid state. I submitted a fix for here but hasn't been merged https://github.com/iRayanKhan/homebridge-tuya/pull/201/files
Anyway your dpStatus should be 101 for the Wofea garage door which will make it both work and not crash.
        {
            "platform": "TuyaLan",
            "name": "TuyaLan",
            "devices": [
                {
                    "name": "Garage door",
                    "type": "GarageDoor",
                    "manufacturer": "Wofea",
                    "model": "Smart Wifi Garage Door",
                    "id": "...",
                    "key": "...",
                    "dpStatus": 101
                }
            ]
        }

Thanks a lot. Tested it and it is Ok now.
Mark

Am 24.02.2021 um 23:54 schrieb Long Zheng<EMAIL_ADDRESS>{
"platform": "TuyaLan",
"name": "TuyaLan",
"devices": [
{
"name": "Garage door",
"type": "GarageDoor",
"manufacturer": "Wofea",
"model": "Smart Wifi Garage Door",
"id": "...",
"key": "...",
"dpStatus": 101
}
]
}

Thank you!


Add Support for hhvm

Please add support for hhvm and fallback to php-fpm and vice versa

Addition of PHP-FPM support would be awesome!

Hello @ARKKYN!
I've added support for multiple HHVM sockets.
@ob-jon fallbacking to PHP is another challenge 😉

Thanks but it still needs fallback for hhvm  when it crashes to fallback to php-fpm

Thanks for the example @ARKKYN!
I've deployed the support of the backup PHP server.


Add German translation

This PR is related to #33. Added German translation. Suggestions from German native speaker are welcome! :)

⭐  the repository please 😉


Master server client

Added proper server handling of sockets and threads
Standardize cli running
Fix GUI to retrieve streams properly
Fix Client to close sockets properly each time
Fix GUI X Button to close properly
Pipe through logging data through the GUI into the control widget
New parameters added to launch client in cli or gui mode

Functional, accepted by other team members. Fairly responsive, may need additional bug fixes in future but overall a good working model for POC.


fix(v2): properly link top-level github pages sites in deploy command

Motivation
Found this issue while trying to make a site, so I fixed it
Have you read the [Contributing Guidelines on pull requests]
Yes
Test Plan
I tested it by creating a docusaurus site and I verified that the correct URLs are logged.
Related PRs
This was in #2556 but @yangshun reverted it

Sorry @yangshun, fixed now


fix: TTL generator exe path when building for windows

Fixes failing build when using using make to build for windows, due to path to utils/lv2_ttl_generator.exe being relative, but the generate-ttl.sh script changes working directory.
make -C dpf/utils/lv2-ttl-generator WINDOWS=true
make[1]: Entering directory '/home/chris/work/ykchorus/dpf/utils/lv2-ttl-generator'
i686-w64-mingw32-gcc lv2_ttl_generator.c -Wall -Wextra -pipe -MD -MP -fno-gnu-unique -posix -D__STDC_FORMAT_MACROS=1 -D__USE_MINGW_ANSI_STDIO=1 -mstackrealign -DNDEBUG -O3 -ffast-math -fdata-sections -ffunction-sections -mtune=generic -msse -msse2 -mfpmath=sse -fvisibility=hidden -std=gnu99 -DPTW32_STATIC_LIB -Werror -o ../lv2_ttl_generator.exe -fdata-sections -ffunction-sections -Wl,-O1,--as-needed,--gc-sections -Wl,--strip-all -static -Wl,--no-undefined -static -static-libgcc -static-libstdc++ -static
touch ../lv2_ttl_generator
make[1]: Leaving directory '/home/chris/work/ykchorus/dpf/utils/lv2-ttl-generator'
dpf/utils/generate-ttl.sh: line 38: dpf/utils/lv2_ttl_generator.exe: No such file or directory
make: *** [Makefile:125: gen] Error 127

I think this bug was introduced in 648452a, because only the code path for macOS and Posix was adapted to the changes.

Makes sense, thanks!


Running scripts/deploy from different machines causes multiple services to be created

If I try and run scripts/deploy from my laptop where the nykrio source is checked out in a hunter directory the script will create a second set of services to be created on the production machine which will eventually fail because they can't bind to the required ports.
It seems like the source directory name has some impact on the image names created during docker compose.

Out of date.


Word Annex clause headers must be bolded

Currently:

All Annex headers should be bolded just like normal headers.

"A.1.1." is bolded but "A.1." is not.

Annex subheadings are in the special style h2Annex in Word; need to ensure that style is still in all the stylesheets.

Bizarreness of Word CSS: I told it h2Annex uses Heading 2 Char, but it still had to be told explicitly that the style is boldface; the corresponding h2 definition does not need the explicit notation. Fixing all h2Annex instances.


docs: update flowsettings.py location

Description
Commit 654501e01c17f61f75326dcdf38002f3b5786a77 moved the location of the flowsettings.py file. This PR adjusts sources to changes.
Type of change

[ ] New features (non-breaking change).
[x] Bug fix (non-breaking change).
[ ] Breaking change (fix or feature that would cause existing functionality not to work as expected).

Checklist

[x] I have performed a self-review of my code.
[ ] I have added thorough tests if it is a core feature.
[ ] There is a reference to the original bug report and related work.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] The feature is well documented.

Adding this fix to #282.


If an await operation on SerialTestConnection.SendCommand fails with an error, the test script stops

The test script should just log that an error occurred and then continue with the next test.
This is because the IsFaulted property on CommandData is checked and then ReportError is called which stops the execution completely. LogError or similar should probably be called instead in the places where CommandData.IsFaulted is checked.

No change needed. In the script the following code can be used if error handling is needed.
procedure void ProofOfConcept()
{
    var result = connection.SendCommand("about");
    while (!result.IsCompleted)
    {
        delay(10ms);
    }

    if (result.IsFaulted)
    {
        // Try again
    }
}


FeatureIndex class

interval forest (dictionary of interval trees)
sequence regions, both inferred and explicit (if any)
functions to consume features, directives, and files

[ ] Split out consume feature/seqreg functionality, expose to API
[ ] add to API documentation


Fixes #10: Setting up Tailwind CSS

[x] Setting up Tailwind CSS
[x] Change the UI according to the Tailwind CSS : Assigned to @AlphaGaurav7

@HridoyHazard Please put hacktoberfest2022 / hacktoberfest-accepted labels in this PR.

@HridoyHazard  Kindly Change the label to Hacktoberfest-accepted, Currently it is hactoberfest-accepted


The order of code in the server might be wrong

The original code is
reticulate::virtualenv_create(envname = virtualenv_dir, python = python_path)
reticulate::virtualenv_install(virtualenv_dir, packages = PYTHON_DEPENDENCIES, ignore_installed=TRUE)
But maybe the following is right:
Create virtual env and install dependencies
reticulate::virtualenv_install(virtualenv_dir, packages = PYTHON_DEPENDENCIES, ignore_installed=TRUE)
reticulate::virtualenv_create(envname = virtualenv_dir, python = python_path)
reticulate::use_virtualenv(virtualenv_dir, required = T)

I've been working recently on following the tutorial for deployment of an app on shiny.io. It runs fine locally, but I have the familiar RETICULATE_PYTHON issue others seem to have had.
I just switched the order as you suggested and the project successfully deployed to shiny.io.

I switched the order and I am still getting that same issue.

hi there, same here and I have been trying for the last 4 days.


Fix bug in InstanceOperationsImpl.getServer()

There was a bug in InstanceOperationsImpl.getServer() where getScanServer() was being used for the TABLET_SERVER case.
https://github.com/apache/accumulo/blob/e45b3b400732fddb9b07f8319462e9191d1dcb82/core/src/main/java/org/apache/accumulo/core/clientImpl/InstanceOperationsImpl.java#L504-L521

Marked as draft for now as I look into usage of this method as well as potential test improvements.

I would assume that InstanceOperationsIT.testGetServers would have failed....

I would assume that InstanceOperationsIT.testGetServers would have failed....

Nevermind, that test does not test that code. That code is for contacting a specific server. Should probably add a test for that in InstanceOperationsIT.


attach extra data into request of req.pipe(request.(...))

How can I use req.pipe(request.(...)) and attach some extra data into request? if I put it by form,error occurs with info 'write after end'.

var form;
req.pipe(request.post(URL,{form:form},...));

like this,but this word will occur error.
form is the extra data I want to attach to original req

Just create a new request in this case request.post(URL,{form:form},...).pipe(res). Also there is no need to open up a second issue for the same problem. Use the other one.


Dirty Mutation Testing

The mutated code remains (some times) in the final code. This causes errors in the next runs.
And mutation testing framework produces many new methods in the mutated class when the class has a Trait.

TODO: exclude trait methods:
AClass methods select: [:method | method traitSource isNil]


doc intro is empty

our intro page is empty: https://sklearn-evaluation.ploomber.io/en/latest/intro.html
we should briefly describe the features in the library (possibly with some short examples) and add links to our quick starts

Reopening as we still need to add some value propositions into it.


Make HMR configurable

I propose that HMR be made optional/configurable when running the devServer, since not everyone is a fan of HMR.
In much the same way that you have a cache flag for enabling/disabling FuseBox's cache, hmr could enable/disable HMR when using the devServer.
As per the discussion on Gitter, FuseBox's cache is always enabled when hmr: true (since it is required by HMR).
The config might look like this:
FuseBox.init({
  hmr: false,
  cache: true,
  homeDir: 'src',
  outFile: 'bundle.js'
}).devServer('>index.js')

Having cache set to true would allow FuseBox to rebundle files rapidly in memory, but not update/inject the changes over the websocket.
Perhaps a log could still be printed in the console prompting the user that files have been updated and to refresh the page to see the changes (like webpack does)?
Conversely another valid configuration might be as follows:
FuseBox.init({
  hmr: true,
  cache: false, // This is overridden when using the devServer with hmr: true
  homeDir: 'src',
  outFile: 'bundle.js'
}).bundle('>index.js')

I'm not sure if there is much/any benefit to the above when simply calling bundle? Beyond faster bundle times, is there any other benefits to disabling the cache?

Thanks!

hmr and bundle cancel eath other
hmr false can be introduced. I agree

Perhaps a log could still be printed in the console prompting the user that files have been updated and to refresh the page to see the changes (like webpack does)?

Can be done now using the loader plugin api : http://fuse-box.org/#loader-plugins

:rose:

@basarat could you fix the typings? Travis does not like something.

:rose:

45 mins. Till I get to a computer 🌹 Ref for me https://travis-ci.org/fuse-box/fuse-box/builds/203604036

https://github.com/fuse-box/fuse-box/pull/255 :rose:

https://github.com/fuse-box/fuse-box/commit/c3d4cf3f5b29d309aec2f3d23ab10f4a1b0ea68c


Improvements to the helm chart

With this PR I'd like to fix the hurdles I've run into.
These are:

one bug: the source: postgresURI  line was merged the the comment above, because the if statement strips newlines at both ends.
the memory limits are too low, causing Clair to be scheduled on hosts that doesn't have enough memory available - and eventually OOM the container or machine.
Switch to the stable release, as the git version doesn't work out of the box.
Explicitly mention that postgresql.enabled can be set to false, because the requirements.yaml allows that.

Do you also have plans to move this chart to the official helm chart repositories?

Thanks for looking into this. I've updated my pull request to omit that image change!
I do think it might be useful to mention this in the docs, so people can choose to install the stable version instead. I didn't get the master version running last week.


Improve naming and better typespec

there is also one small minor optimization using bit shift to check if a child number is hardened or not

rebased


chrony plugin stopped WebUI

Dear OPNsense team,
This is my first issue I report here. Please let me know if something is wrong.
I already have some discussion at https://forum.opnsense.org/index.php?topic=23492.msg111698#msg111698
I want to test the chrony plugin.
My current version info is:
OPNsense 21.1.6-amd64
FreeBSD 12.1-RELEASE-p16-HBSD
OpenSSL 1.1.1k 25 Mar 2021
I freshly installed 21.1.5 from OPNsense-21.1-OpenSSL-serial-amd64.img on a APU (specification) an did an update to 21.1.6. The only other plugin installed is os-dyndns (uninstalled it first to get rid of the "misconfigured" message). The dyndns plugin was uninstalled before the final tests are executed which are covered by the logs.
The matter is, that the WebUI simply do not respond to any clicks outside the chrony "pane".
https://user-images.githubusercontent.com/13496906/121790544-c7cac680-cbe0-11eb-993b-2cc242e81dd3.mp4
The above video give you an visual impression:

I viewed the dashboard (about 3 seconds)
I navigate to the chrony pane in the menu (until 6.166)
I change to several pane within the chrony general pane (until 8.916)
I clicked on the OPNsense-Logo to switch to the dashboard (round about 10.0)
after this the WebUI hang, the activity indication of Firefox seems to show activities, the dashboard will "never" appear.

Hopefully this description explains the issue better.
I installed the chrome browser in a virtual machine running windows 10. I was able to reproduce the issue. Furthermore I was not able to connect to my OPNsense from an other pc.
BUT after a long time (I would guess more than 10 minutes, I did not measure because some shopping took my focus) I can connect to the WebUI again. Based on this discovery I did further tests.
I established three connections to the WebUI of my OPNsense:

from chrome browser in my VM
from Firefox browser in my VM
from Firefox browser on my real PC

I used the following procedure to do some measurements:

Navigate to some pane for within each browser
in chrome click though the chrony plugin, test if dashboard is reachable
if dashboard is answering, continue with step 2
take the time until browsers display correct WebUI pane

Please find attached the logs. Here are the timestamps which are relevant for the tests:

at 19:00 I restarted OPNsense to get a working test environment
around 20:40 all three browser hung for 30 up to 60 seconds
at 20:47 all browser hung for about 4.5 up to 5 minutes
from 20:56 the 3rd connection hung for about 60 seconds, the 2nd connection hung for 90 seconds and the 1st connection hung for 140 seconds
after  these tests I rebooted OPNsense to get rid of the delays during navigation.

20210612-audit.log
20210612-configd.log
20210612-lighttpd.log
20210612-system.log

Does this also happen when you enable the chrony daemon?

Hi Michael,
I did not apply an changes to the NTP settings.
I only changed the settings of chrony as displayed in the attached screen shot and pressed SAVE:

The WebUI hang after pressing SAVE. After resetting the OPNsense  the chrony plugin does not display any change on its General pane.
Kind Regards
Thomas

It hangs because, while you browse that configuration page, there are some XHRs requesting the status of chronyd, even when not running, every 5 seconds.
https://github.com/opnsense/plugins/blob/f7bc50c/net/chrony/src/opnsense/mvc/app/views/OPNsense/Chrony/general.volt#L98-L101
The API calls are hanging if chronyd is not running because the chronyc client has a longer than 5 seconds overall timeout.
https://github.com/opnsense/plugins/blob/e89c921/net/chrony/src/opnsense/mvc/app/controllers/OPNsense/Chrony/Api/ServiceController.php
In the meanwhile more calls are queued via setInterval. If you close the page it should unblock itself after a while, depending how many calls are waiting.
If you enable the service before the 5 seconds interval that should prevent the issue, then you can make the required configuration changes.
What I would do:

call the chronyc commands with timeout, for example timeout 0.1 /usr/local/bin/chronyc tracking
or maybe better chronyc -m 'timeout 100' 'retries 0' tracking
https://github.com/opnsense/plugins/blob/70497698e08900c7d6926402a19b36168a4d3572/net/chrony/src/opnsense/service/conf/actions.d/actions_chrony.conf
this already should mitigate the problem
```
[chronysources]
command:/usr/local/bin/chronyc -m 'timeout 100' 'retries 0' sources
parameters:
type:script_output
message:show chrony sources

[chronysourcestats]
command:/usr/local/bin/chronyc -m 'timeout 100' 'retries 0' sourcestats
parameters:
type:script_output
message:show chrony sourcestats

[chronytracking]
command:/usr/local/bin/chronyc -m 'timeout 100' 'retries 0' tracking
parameters:
type:script_output
message:show chrony tracking

[chronyauthdata]
command:/usr/local/bin/chronyc -N -m 'timeout 100' 'retries 0' authdata
parameters:
type:script_output
message:show chrony authdata
```

find a way to not query the various statistics if the daemon is not running

Thx for digging into this, totally forgot about this one.


Handle Undefined Object in isInternalEntity

Problem
When adding a non-entity in a relation property, the following error occurs:
  TypeError: Cannot use 'in' operator to search for '__type' in null

This also breaks dropping a db.
Expected Outcome
The code should not throw because of a null object, but error or warn the user about him using a normal object instead of an entity for a relation.

Hy, @JoseRFelix. Thanks for raising this.
This is a valid concern and I agree that the library should warn/error when attempting to assign a plain compatible object as the relational property. This looks like a partial follow-up of #93.

Have just stumbled upon this issue. Do I understand correctly that the following thing is not possible at the moment:
const db = factory({
  project: {
    id: primaryKey(String),
    name: String,
    owner: oneOf('user'),
  },
  user: {
    username: primaryKey(String),
    name: String,
  },
})

[{ username: 'username1', name: 'Jack' }, { username: 'username2', name: 'John' }].forEach(user => {
  db.user.create(user)
})

[{ id: '1', name: 'Project 1', owner: 'username1' }, { id: '2', name: 'Project 2', owner: 'username2' }].forEach(project => {
  const owner = db.user.findFirst({ where: { username: { equals: project.owner } } })
  db.user.project({ ...project, owner })
})

Interesting, that when I run this code i get this TypeError: Cannot use 'in' operator to search for '__type' in null not every time, but once in a while.

Hm... seems like even you do it like in docs
const db = factory({
  project: {
    id: primaryKey(String),
    name: String,
    owner: oneOf('user'),
  },
  user: {
    username: primaryKey(String),
    name: String,
  },
})

const user = db.user.create({ username: 'username1', name: 'Jack' })
const project = db.project.create( { id: '1', name: 'Project 1', owner: user })

You can still get this error once in a while

@smashercosmo, if I'm not mistaken, assigning plain objects as relational values isn't supported at the moment. It doesn't error, but won't be treated as a node reference, as opposed to when you assign the result of db[model].create(). This is raised in https://github.com/mswjs/data/issues/92 as well, where updating a relational property with a plain object doesn't update the references. I didn't have time to get to this, so I'd appreciate any thoughts/brainstorming on how to make this happen and will support the effort as much as I can given the time constraints.

Ok, but that's another issue, I guess. Currently there is an error raised occasionally even if you assign a result of db[model].create(). Seems like there is some racing condition going on. Does db[model].create() happens synchronously? Well, probably I'll just try to provide minimal reproduction.

Ok, I figured out what my issue was about. You get this error if you try to assign null to any of the model's fields. Don't know, if it's a bug or by design. But can't think of any reason why null can't be a valid value.

I've opened a pull request that'd ensure the value passed to isInternalEntity is always a defined object (#117). Passing anything else is unexpected and shouldn't happen. Since the values of the models are defined by the end-user, the function may still get an arbitrary value that violates its call signature.
@smashercosmo, using null as the model property value is not correct. All model properties are nullable and don't have any value by default. There's no reason to use explicit null. Instead, you should use a getter function that either resolves to an initial value ({ name: () => 'John' }), or indicates the data type that can be inferred { name: String }.

@kettanaito I'm not sure that model properties are nullable. At least typescript doesn't know about this.
const db = factory({
  entity: {
    id: primaryKey(datatype.uuid),
    name: String,
  }
})

const instance = db.project.create({ name: null }) // <- TS2322: Type 'null' is not assignable to type 'Partial   >> | undefined'.

@smashercosmo, I think at the moment you can set undefined instead of null. I'm open to discussion to support Vaule | null instead.


Chromatogram resolver, Local minimum resolver: Issue with Peak duration range

My operating system: Windows 11 Pro, 64 bits
My MZmine version: 3

Hi, I am getting started with MZmine3 to develop nontarget screening strategies to process LC-HRMS data (converted to mzXML files).
I have found a strict filtering of the number of features when I reduce the Peak duration range in the Local minimum resolver from 0.5 to 0.3 min. The chromatographic peak width of my features are around 0.2-0.25 min, so in terms of workflow optimization, I checked the reduction of the number of features by applying different parameters. For example, I checked the presence of an IS (m/z 245.1518 at 10.12 min). In the preview option of the Local minimum resolver, I observed that this chromatographic peak which has 0.25 min of peak width is not detected when the Peak duration range is set from 0 - 0.3 min, while it is detected when this parameter is set from 0 - 0.5 min. Please, find below the details of my workflow:
Mass detection
Retention time: autorange
MS level: 1
Polarity: Any
Spectrum type: Any
Feature detection - LC-MS - Chromatogram builder
Min group size in # of scans: 5
Group intensity threshold: 1.0E5
Min highest intensity: 5.0E5
Scan to scan accuracy 0.02 m/z or 10 ppm
Smoothing
Algorithm: Savitzky Golay
Retention time smoothing: 5
Chromatogram resolver, Local minimum resolver
Dimension: Retention time
Chromatographic threshold: 90%
Minimum search range RT: 0.075
Minimum relative height: 0%
Minimum absolute height: 1.0E6
Min ratio of peak/top edge: 2
Peak duration range (min): 0 - 0.3
Min of # data points: 5
Please, find attached a screenshot of the irregular behavior using both parameters.
Peak duration range: 0 - 0.3 min

Peak duration range: 0 - 0.5 min

Thank you for your time and help.

Just an additional comment: the version of MZmine I am using is 3.1.0-beta
Thanks in advanced!

My guess is that the detected peak is tailing and the taling is still recognised as a part of the peak. Therefore it extends for more than the expected range.
You can try a different resolver, but that's the expected behaviour for the local minimum resolver.


Ben_NOAA_changes

Added a README.txt for the output zipfile. Not sure whether the code to copy it into the zip directory will actually work.
Added AGAGE data availability/usage statements as attributes to the output files, as per NOAA ObsPack
Added a json converting standard_name attributes of the mf and mf_repeatability variables to accepted standard_names under CF conventions
NOT YET added a test for CF Compliance (Gareth and I to discuss)

I would make the ZIPFILE_README.txt file a markdown (.md) file and remove all the hashes on the left side as they serve no purpose.
The naming of the file could also be more informative, e.g., agage-data_README.md or something similar. (zipfle isn't obvious)

Also, is it worth putting a licence in the README? And/or should it appear as an attribute?
CC BY 4.0 (or maybe even CC BY-SA 4.0) I would imagine

Made and pushed the changes you mentioned @lukewestern - should be ready to go now I think

It says that there are merge conflicts in test_formatting.py then need to be fixed @qq23840.
These will need to be fixed before merge.

Merge conflicts fixed - was just a silly formatting thing. Also committed the changes in your previous comments.

Thanks for doing this Ben. Please take a look at my latest fix to test_formatting, and note that I don't think the standard_names table was actually being read before?
The CF test still fails for me, because it doesn't seem to like the repeatability variable as having a standard name. Should we prevent these variables from having a standard name (i.e., only have the "mf" variable have a standard name)?

@qq23840 or @lukewestern , are either of you available to review the latest changes I've made? See commit messages

I can take a look now


zhimi.airpurifier.mb4

Hi can you supoprt this purifier???
Sensors, filter_filter_life_level,environmen_pm2_5_density, filter_filter_used_time are shown but as not available
And on/off is shown but not avaliable too with error:
4 11:04:04 WARNING (MainThread) [custom_components.xiaomi_miot_raw] New state (None) of purificador doesn't match expected values: True/False 2021-02-14 11:04:13 ERROR (MainThread) [custom_components.xiaomi_miot_raw] Got exception while fetching purificador 's state: {'code': -9999, 'message': 'user ack timeout'}
And Led control and fan power are not shown

I find there is a “-9999” error code, and it means you have to configure
this device to fetch data via cloud.
asiersan<EMAIL_ADDRESS>周日18:05写道：

Hi can you supoprt this purifier???
Sensors, filter_filter_life_level,environmen_pm2_5_density,
filter_filter_used_time are shown but as not available
And on/off is shown but not avaliable too with error:
4 11:04:04 WARNING (MainThread) [custom_components.xiaomi_miot_raw] New
state (None) of purificador doesn't match expected values: True/False
2021-02-14 11:04:13 ERROR (MainThread) [custom_components.xiaomi_miot_raw]
Got exception while fetching purificador 's state: {'code': -9999,
'message': 'user ack timeout'}
And Led control and fan power are not shown
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ha0y/xiaomi_miot_raw/issues/35, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACO4PHK4QTPKIMMCULLKAP3S66N7ZANCNFSM4XTCLMUA
.

--
发自移动版 Gmail

The code -9999 means that your device uses miio protocol instead of MIoT. To use the device with this integration, you have to log in your Xiaomi account, so it can be controlled via Xiaomi cloud server, which requires a stable Internet access. Currently this integration only supports China server but other servers will be supported soon.

The server option is available now. Go to integration page and change option for you xiaomi account to use other servers. Have a try!

The server option is available now. Go to integration page and change option for you xiaomi account to use other servers. Have a try!

Os working now


added way to add custom callbacks

Summary
Added the ability to specify a custom callback object for observability logging in the LLM provider.
Details

Added a new parameter callback_obj to the __init__ method of the LLMProvider class
The callback_obj parameter is used to set the observability logging callbacks for success and failure events
The default value for callback_obj is set to 'supabase'
The _setup_observability method was updated to use the callback_obj value instead of hardcoding 'supabase'
This change allows developers to provide a custom callback object for observability logging, enabling more flexibility and customization in the LLM provider

✨ Generated with love by Kaizen ❤

Original Description
### Summary
Added the ability to specify a custom callback object for observability logging in the LLM provider.
Details

Added a new parameter callback_obj to the __init__ method of the LLMProvider class
The callback_obj parameter is used to set the observability logging callbacks for success and failure events
The default value for callback_obj is set to 'supabase'
The _setup_observability method was updated to use the callback_obj value instead of hardcoding 'supabase'
This change allows developers to provide a custom callback object for observability logging, enabling more flexibility and customization in the LLM provider

✨ Generated with love by Kaizen ❤

Original Description
None

!review

Good Example of bad PR.


log/slog: handler.Enabled should be called with context.Background() instead of nil ctx

What version of Go are you using (go version)?

$ go version
go version go1.21rc2 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What did you do?
Look at the source code: https://cs.opensource.google/go/go/+/refs/tags/go1.21rc2:src/log/slog/logger.go;l=209-210
What did you expect to see?
This if must be executed before calling l.Enabled(ctx): https://cs.opensource.google/go/go/+/refs/tags/go1.21rc2:src/log/slog/logger.go;l=222-224
What did you see instead?
Current implementation violates current doc. https://pkg.go.dev/log/slog#Handler says:
	// If called from a Logger method, the first argument is the context
	// passed to that method, or context.Background() if nil was passed
	// or the method does not take a context.
...
	Enabled(context.Context, Level) bool

Oops, sorry, looks like this is already implemented in l.Enabled().


[Tracking Issue] Documentation Completion

Milestone

[ ] Blitz course to SparseTIR (due Nov 17)
[ ] System Overview (for developers) (due Nov 20)
[ ] Tutorial: SpMM operator and optimization (due Nov 23)
[ ] Tutorial: SDDMM operator and optimization (due Nov 26)
[ ] Tutorial: Block Sparse operator and optimization (due Nov 29)
[ ] Tutorial: RGMS operator and optimization (due Dec 2)
[ ] Tutorial: Integration with existing Machine Learning Frameworks. (due Dec 10)
[ ] Blog: Why another sparse compiler abstraction (due Dec 31)

Hello, is there any update on documentation? It seems that the current documentation has not been updated for over a year.


Bug Report #69 C:/HaxeToolkit/haxe/lib/polymod/git/polymod/format/XMLMerge.hx:189: characters 30-43 : Warning : This typedef is deprecated in favor of haxe.xml.Access

C:/HaxeToolkit/haxe/lib/polymod/git/polymod/format/XMLMerge.hx:189: characters 30-43 : Warning : This typedef is deprecated in favor of haxe.xml.Access
Help how do i do fix this

im pretty sure thats normal because its something that i always see when i compile, unless i did something wrong
also funni number

this is just a warming, the real issue is probably above thaf

this warning stops me from compiiiiiiiiiiliiiiiiiiiiiiiiiing

this warning stops me from compiiiiiiiiiiliiiiiiiiiiiiiiiing

did you install haxe 4.1.5

oh, i installed 4.2.4 to compile psych ;-;

is cuz im making a mod for both

Ok, If You Have Problems, Just Leave Kade Engine, And Move To Psych Engine,
It's Easier Than .
On 15 Jan 2022 12:25 a.m., "JorgeMiguel22" @.***> wrote:

is cuz im making a mod for both
—
Reply to this email directly, view it on GitHub
https://github.com/KadeDev/Kade-Engine/issues/1732#issuecomment-1013538975,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUOLFIQCJMIAFFGVRMKZDA3UWCWE3ANCNFSM5CEKVLOA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Installation - Basic GitHub Checkout fix

To add nodenv path to $PATH on Ubuntu/Debian/SUSE bashrc needs to
updated, not bash_profile. For CentOS/REHL, bashrc or bash_profile
will work.

Tested usage of .bashrc on CentOS, Ubuntu, Debian, OpenSUSE. bash_profile would only work on CentOS.

conversation at #66

I may make new one with regards to resolving issue 66 while using the login shell. It's not a huge issue as a workaround is easy but it might create a smoother install.


Build new GATK inputs for hg38

Sadly the liftover files from here do not work and others have run into the same issues I did.
After perusing some other options I'm going to try and follow this page and this makefile

Thanks to @melissacline , I no longer have to do this, yay!

Files are in process of upload to s3://cgl-pipeline-inputs/variant_grch38.


Recent regression on master branch causes syntax errors

Boot a pier using current master (4c2ac929 and its pill), and attempt to apply this patch to dojo.hoon:
--- fakezod/home/app/dojo.hoon	2017-02-26 20:20:37.746006999 -0800
+++ arvo/app/dojo.hoon	2017-02-26 20:16:38.950006999 -0800
@@ -279,12 +279,10 @@
     ++  dy-amok  +>(poy ~)                              ::  terminate
     ++  dy-ford                                         ::  send work to ford
       |=  {way/wire kas/silk:^ford}
       ^+  +>+>
       ?>  ?=($~ pux)
-      ::  adding this line breaks compile on master.
-      ~&  %starting-dojo
       (he-card(poy `+>+<(pux `way)) %exec way our.hid `[he-beak kas])
     ::
     ++  dy-eyre                                         ::  send work to eyre
       |=  {way/wire usr/(unit knot) req/hiss:^eyre}
       ^+  +>+>

I consistently get the following syntax error:
syntax error: 285 25
ford: fade /~zod/home/3/app/dojo
ford: core /~zod/home/~2017.2.27..05.16.14..28ac/app/dojo

To confirm, line 285 is the line with the ~& after the patch.
If you go back to an older point on the master branch and boot a pier from it (4288aab089 and its pill), the patch above compiles and puts a debugging line on each generator call. So this is a recent regression.
(Discovered this when trying to rebuild a ship with the doccords patch when applied to current master, which throw likewise impossible syntax errors.)

Can you give a vere revision and the OS, platform, etc. to aid in my reproduction attempts?

I've reproduced this on OS X, Terminal.app, bash 3:
urbit/arvo#4c2ac92947d4e4e6d499fbfad5295ce2be458178
urbit/vere#cee04c794bb596484e475a560895c516441bda31
I've seen it on other commits as well, but can't verify them right now ...

Thanks! By the way, do you ever experience urbit/urbit#817?

No, I haven't reproduced that one.

I can verify that I can reproduce using the commits @joemfb posted, though on Ubuntu 16.04 instead of OSX.

Thanks! More to come shortly.

This is a text diff problem - while it's being tracked down, a workaround is deleting, committing, recreating, and committing again.


php

chmod 0600
https://help.krystal.uk/web-hosting/what-file-and-directory-permissions-should-i-use-for-my-web-files


machine_configuration: Add registered trademark symbol for Intel QAT

Signed-off-by: Hersh Pathak<EMAIL_ADDRESS>

looks good to me.


Tiller in a bad state / unable to decode

I think I got my Tiller in a bad state. This might be hard to repro, so I'll just describe what I did:
I did a dry run install of my release.
I tried to install a release with my own release name. An error occurred:
failed: [unable to decode "": [pos 121]: json: expect char '"' but got char 'n', unable to decode "": [pos 121]: json: expect char '"' but got char 'n', unable to decode "": [pos 121]: json: expect char '"' but got char 'n']

Not sure what that means.
I run helm ls and no releases exist.
I try to install again and it errors, saying the release already exists.
I delete the Tiller pod to clear all releases from memory.
Do a fresh helm install and things are good.
I've encountered this 2-3 times now while working on various charts. Once I see a "unable to decode ..." error, I'm not sure how to recover without deleting the Tiller pod then re-installing all my charts.
It seems to happen when something in my K8S manifest is incorrect and causes an error when getting applied, but I think "unable to decode" is masking the problem, so it's hard to figure out.
Related to https://github.com/kubernetes/helm/issues/1035
helm v2.0.0-alpha.3

I think the current version of helm ls is not showing releases whose state is FAILED. There's an issue to add --all and --deleted flags (#973) that would help this case a lot.

FWIW, the error failed: [unable to decode "": [pos 121]: json: expect char '"' but got char 'n', unable to decode "": [pos 121]: json: expect char '"' but got char 'n', unable to decode "": [pos 121]: json: expect char '"' but got char 'n']  indicates that a quotation mark was expected somewhere in the YAML. We're working on improving some of this output.

Thanks @technosophos. It seems that once I accidentally introduce a yaml error during a helm upgrade, it's stuck in that state. After fixing the yaml by reverting to a known good state, the error remains the same. Do I need to delete the release?

Can't delete the release either. Tiller logs when I try:
2016-08-25 17:42:59.885674 I | Using reaper for deleting foo
2016-08-25 17:42:59.903093 I | uninstall: Failed deletion of "foo-staging": yaml: line 56: did not find expected ',' or ']'

Where is this being stored and how do I reset the state?

Oh. Hmmm... that is a nasty problem. I'll CC @michelleN on this. I think we just missed a verification step in upgrade.
In other news, helm list --failed and helm list --deleted are now in master.

Great! Except I can't helm list either because:
helm list
2016/08/25 16:27:10 transport: http2Client.notifyError got notified that the client transport was broken EOF.
Error: transport is closing

I got stuck in what I believe to be the same problem:
# helm ls --all
crazy-armadill
dapper-maltese
eating-oyster
enervated-gupp
full-hummingbi
loopy-bumblebe
morbid-deer
operatic-salam
queenly-skunk
ringed-platypu
whimsical-slot
# for r in $(helm ls --all) ; do helm delete $r ; done
Error: [unable to decode "": [pos 484]: json: expect char '"' but got char '{', unable to decode "": [pos 1144]: json: expect char '"' but got char '{', unable to decode "": [pos 384]: json: expect char '"' but got char '{']
Error: [unable to decode "": [pos 1144]: json: expect char '"' but got char '{', unable to decode "": [pos 384]: json: expect char '"' but got char '{', unable to decode "": [pos 484]: json: expect char '"' but got char '{']
Error: [unable to decode "": [pos 480]: json: expect char '"' but got char '{', unable to decode "": [pos 380]: json: expect char '"' but got char '{', unable to decode "": [pos 1140]: json: expect char '"' but got char '{']
Error: [unable to decode "": [pos 1144]: json: expect char '"' but got char '{', unable to decode "": [pos 484]: json: expect char '"' but got char '{', unable to decode "": [pos 384]: json: expect char '"' but got char '{']
Error: [unable to decode "": [pos 384]: json: expect char '"' but got char '{', unable to decode "": [pos 484]: json: expect char '"' but got char '{', unable to decode "": [pos 1144]: json: expect char '"' but got char '{']
Error: [unable to decode "": [pos 1159]: json: expect char '"' but got char '{', unable to decode "": [pos 499]: json: expect char '"' but got char '{', unable to decode "": [pos 384]: json: expect char '"' but got char '{']
Error: [unable to decode "": [pos 372]: json: expect char '"' but got char '{', unable to decode "": [pos 1132]: json: expect char '"' but got char '{', unable to decode "": [pos 472]: json: expect char '"' but got char '{']
Error: [unable to decode "": [pos 484]: json: expect char '"' but got char '{', unable to decode "": [pos 384]: json: expect char '"' but got char '{', unable to decode "": [pos 1144]: json: expect char '"' but got char '{']
Error: [unable to decode "": [pos 380]: json: expect char '"' but got char '{', unable to decode "": [pos 480]: json: expect char '"' but got char '{', unable to decode "": [pos 1140]: json: expect char '"' but got char '{']
Error: [unable to decode "": [pos 1159]: json: expect char '"' but got char '{', unable to decode "": [pos 384]: json: expect char '"' but got char '{', unable to decode "": [pos 499]: json: expect char '"' but got char '{']
Error: [unable to decode "": [pos 384]: json: expect char '"' but got char '{', unable to decode "": [pos 1159]: json: expect char '"' but got char '{', unable to decode "": [pos 499]: json: expect char '"' but got char '{']

Installing new releases also fails. It seems I cannot do anything anymore. Even kubectl --namespace=kube-system delete deployments/tiller-deploy && helm init does not rescue me, as the broken releases will reappear after a few seconds.

The problem appears to be that helm install uploads everything in the release's folder and then chokes on it.
I looked at the configmaps (with random names, not hinting at tiller) it created, specifically at the release data. It contains the end results from template substitution in templates/*.yaml, but also all files in the release directory (including the not-substituted templates, binary and JSON files). The substituted templates do not contain any JSON anymore (I base64 escaped everything, to exclude them as a possible cause), so I conclude that tiller/helm has a problem with the other files that happen to be lying around.

I believe this is now solved. At this point, failed YAML parses should not prevent a deletion event (though you will still get the YAML error as output).


feat: use the EVM keystore to sign attestations and transactions in orchestrator and relayer

…rchestrator and relayer

Overview
Closes https://github.com/celestiaorg/orchestrator-relayer/issues/267

Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

@evan-forbes 100%, you're right. We could refactor a lot. However, my plan right now is to have the functionality working so that we can start testing and writing docs. Then, i will refactor and make it better.
I will create issues for refactroing.


[ISSUE #9837] remove meter metric in jvm metrics handler

Fix #9837

[ ] Add a unit test to verify that the fix works.
[ ] Explain briefly why the bug exists and how to fix it.

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[ ] Update the CHANGES log.

Please follow pull request template to update changes.md.


Fixing compile issues for indysdk->libvcx ...

Signed-off-by: Navid Mohaghegh<EMAIL_ADDRESS>

@WadeBarnes thank you ...

@navidmo, Have you had any luck/time looking into the email registration issue?


undefined symbol: glNamedBufferData when running test code

Hi, when I run the tests/square_test.py. It said,
undefined symbol: glNamedBufferData when running test code

the result of $ ldd dirt/librasterise.so  is as below.

There might be something wrong with my GL or EGL library, but I don't know hot to solve it, could you give me some advice?

I think I got a similar error. The building went fine but got the follwoing error while making
aslab@aslab-Serval-WS:~/experiments/dirt/build$ make
[ 14%] Building CXX object CMakeFiles/rasterise.dir/rasterise_egl.cpp.o
/home/aslab/experiments/dirt/csrc/rasterise_egl.cpp: In member function ‘void RegisteredBuffer::ensure_size(const size_t&)’:
/home/aslab/experiments/dirt/csrc/rasterise_egl.cpp:81:76: error: ‘glNamedBufferData’ was not declared in this scope
         glNamedBufferData(gl_index, required_size, nullptr, GL_DYNAMIC_COPY);
                                                                            ^
make[2]: *** [CMakeFiles/rasterise.dir/rasterise_egl.cpp.o] Error 1
make[1]: *** [CMakeFiles/rasterise.dir/all] Error 2
make: *** [all] Error 2

Any leads for fixing this error ?

Fixed it by downloading GL/gl.h and GL/glext.h files,
Thanks

anaconda
tensorflow-gpu=1.13.1
python=2.7.17
cuda=10.0    cudnn=7.6.4    RTX2080Ti
I have not nvidia-*** foler in /usr/lib/
I install OpenGL by:   sudo apt install freeglut3-dev
and how can I install OpenGL correctly?

@wkailiu  You need the version of OpenGL that comes with the nvidia driver -- not the one shown from freeglut/mesa. I don't know where the nvidia one is installed on your system; it should have shown up in that find command. Have you tried reinstalling the nvidia driver?


Enable english captions for embedded player (test recording) by default

Description:
It make sense to adjust default player settings if possible and enable by default english captions to see them in both views (collapsed and expanded).
Now it is required to enable them manually by going to player settings (Captions -> ON)
Attachments:

I can enable it on UI using

cancelled request


Need help setting up local development environment

Heya,
I came across this package and it got me very excited. Amazing work you did!
I tired forking the repo and contributing some fixes upstream (I hit some issues with the ligature table), but I can't set up a local dev environment:
git clone https://github.com/siefkenj/unified-latex
cd unified-latex
npm install

Errors out after a while:
npm ERR! code ENOENT
npm ERR! syscall chmod
npm ERR! path<EMAIL_ADDRESS>npm ERR! errno -2
npm ERR! enoent ENOENT: no such file or directory, chmod<EMAIL_ADDRESS>npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent

There is no node_modules folder in the root project folder, which I guess is why chmod is failing? I'm not sure if there's anything else I need to do to set up a local dev environment.
Thanks for any help!

Hmm. Very strange. I do not get that issue on my computer, but I also have @unified-latex/unified-latex-cli installed globally.
You can try what the CI does and run
npm ci

That should be the exact same thing as what happens here: https://github.com/siefkenj/unified-latex/actions/runs/5318277900/job/14391170634


Correct chef export usage message

chef export's usage is incorrect. It states a client.rb will be in created in the root of the export and includes -c client.rb in the example. export actually creates a .chef/config.rb and can be invoked with the simpler chef-client -z.
The message after the export command completes is correct. This PR corrects it here too.
Obvious fix.

+1

:+1:


Added support of the enhanced notification format

The Apple Push Notification Service has an "enhanced notification format".
When this format is used, the gateway server responses with a detailed error instead of just ignoring the request.
This commit adds support for this feature, which includes raising exceptions depended on the server response.

I think this was obsoleted by https://github.com/djacobs/PyAPNs/pull/71 and can be closed?


fix: 🐛 allow non-agents to pre-approve assets

Description
correct authorization logic that was preventing non asset agents from pre-approving assets
Breaking Changes
None
JIRA Link
None
Checklist

[ ] Updated the Readme.md (if required) ?

:tada: This PR is included in version 24.4.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


v1.0.6 include not all changes from master

@ilcato subj!

@10der, do you want me to publish your last PR ? Does it fix the problem?

@10der published the current master version with your last PR. Should fix the error.

Fixed


wallet test coverage

Signed-off-by: sklump<EMAIL_ADDRESS>

Codecov Report

Merging #453 into master will increase coverage by 1.13%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #453      +/-   ##
==========================================
+ Coverage   89.47%   90.60%   +1.13%
==========================================
  Files         248      248
  Lines       12213    12216       +3
==========================================
+ Hits        10927    11068     +141
+ Misses       1286     1148     -138


Java 7 Support?

I have a big project I'd like to use your PhantomJS wrapper for that is currently on Java version 7. The maven artifact you distributed seems to be compiled using Java 8 bytecode, hence it doesn't work on a Java 7 JRE.
I noticed the code uses a few Java 8 language features. I forked your repository and created a version that works with Java 7. Would you be willing to create a distributed version for Java 7?
Also, I forked off of the latest version 2 release. The logging change was causing a dependency conflict on my project (I didn't dig too deep into why), but we could potentially create two pull requests to support version 2 and 3 if that works.  The effort is pretty minimal.
Thanks!

I would disadvise to create a specific Java 7 build version. Java 7 is EOL. The correct way, in my opinion, would be for you to upgrade your project to Java 8. Having a personal fork for the time being until you made the upgrade seems legit.
What excatly is you dependency conflict with the logging?

I like those java 8 language features-is it possible to build a binary compatible with java 7 without changing any of the code?
I'm also curious about the logging dependency conflicts-you can run mvn dependency:tree -Dverbose to get a view of the conflicts

Basically no. You can use Java 7 code on a JVM 8, but not the other way round. Using Java 8 features makes it necessary to target 1.8 which means that any lower JVM is out of scope.


initial implementation of work orders

Work Orders:

I per closed brief
1 supplier selected per Work Order from the list of applicants
extra data stored in a json field

I refactored as per your comments

Just needs the new migration, I think.


Recording conda environments

Hi,
May I ask if it is possible to record information of the conda environment (e.g., name) in which the corresponding experiment was run?
Thank you!

Hi,
I think the only way to get the conda environment name is via the environment variable CONDA_PREFIX. You can then either add CONDA_PREFIX to sacred.SETTINGS.HOST_INFO_CAPTURED_ENV which will add this environment variable to the "host_info" of your run. Or you can write a custom host info gatherer, like this:
from sacred import Experiment
from sacred.observers import FileStorageObserver
from sacred.host_info import host_info_gatherer
from pathlib import Path

@host_info_gatherer(name='conda_env')
def conda_env():
    import os
    return Path(os.environ.get('CONDA_PREFIX', '')).parts[-1]

ex = Experiment(additional_host_info=[conda_env,])
ex.observers.append(FileStorageObserver('.'))

@ex.automain
def main():
    pass

The latter one has the advantage that you can modify the environment variable however you like (e.g., extract only the environment name as in the example). This will add a key "conda_env" to the "host_info" of your run.

I just found out that there are other ways to find the current conda environment (https://stackoverflow.com/questions/36539623/how-do-i-find-the-name-of-the-conda-environment-in-which-my-code-is-running), but all of them can be integrated into a custom host info gatherer.

Hi,
Thank you for your detailed reply!


Ensure the Capital Expenditure field logic within the EC sheets is correct

Verify that the Labor field is gray/ungrayed correctly based on the Capital Expenditure info. For example, on EC2, column AM should not gray out if the response to capital expenditure is "Yes."

Clarify with partners what "budget" refers to. Here's what I think...For EC 2 & 3, it is actually Total Expected Capital Expenses. For EC 5 it is the Adopted Budget (as there is no CapEx for EC5).

Confirmed with the partners that for EC 1 - 2, the "budget" refers to the Total Capital Expenses.

The bug in EC2 (column AM conditional formatting) is fixed and verified EC1 and EC3 are correct (they are). I have added this to v20220830.


completly wrong

how people write docs here?
aoslotly wron, no gpo tab since 2012
no windows defender firewall in Network Connections

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

ID: 68262173-057b-e0ec-03a4-54045e7a8bb8
Version Independent ID: a0bb0dbb-e9e1-0e0c-cbac-0f6718c59ece
Content: Open the Group Policy Management Console to Windows Defender Firewall with Advanced Security (Windows 10)
Content Source: windows/security/identity-protection/windows-firewall/open-the-group-policy-management-console-to-windows-firewall.md
Service: unspecified
GitHub Login: @brianlic-msft
Microsoft Alias: justinha

@alouko -- I will take a look at this issue and provide clearer details in the content.

@alouko Thanks for the feedback. I corrected how to open the GPO. I confirmed location for Windows Defender Firewall under Network Connections.


Update README.md

if I use npx gedcom file.ged instead of npx parse-gedcom file.ged per the suggestion of this pull request, instead of the syntax error I get in issue #47, I get a successful run, but nothing actually happens.  No output to the terminal, no new file created.


[docs] typeBeat.mli: Formatting; typos

This PR polishes some of the docstrings for the typeBeat.mli.
Questions
I have a question about the pp_parameter docstring: does it read right, now?

Yes, I did a mistake when I write the document, nice catch 👍 thanks !


qiskit_machine_learning - ModuleNotFoundError

I tried to run the code on the page in Azure Quantum workspace, and get the following error.
ModuleNotFoundError: No module named 'qiskit_machine_learning'
I also installed !pip install qiskit-machine-learning

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

ID: eb056c67-eed8-4e5c-1d83-470909d85f74
Version Independent ID: eb056c67-eed8-4e5c-1d83-470909d85f74
Content: Adapting Qiskit samples - Azure Quantum
Content Source: articles/how-to-adapt-qiskit.md
Service: azure-quantum
Sub-service: core
GitHub Login: @SoniaLopezBravo
Microsoft Alias: sonialopez

Hi @karthikeyanVK - Thank you for your feedback. I tried to reproduced you error but the sample works fine for me. I assume you followed the steps in Understanding quantum machine learning.
Did you  install qiskit in the notebook? Add a cell prior intalling quantum machine learning and copy:
!pip install qiskit
Let me know if the error persists.


[e2e issue]  Decrypt error related to new device creation

I find another issue related to e2e and new device creation
B has device device_1
In the past B and C has joined same rooms but they have no more room in common now
-> So for C, list of device  of B is [device_1]
B starts application on a new device called device_2
-> C is not notified of this new device because no more common room with B
A invites B and C in room XX
B is the first to accept the invitation from device_2
B checks all room members who has joined the room XX to notify a a new device
B only sends a message new device to A because A is the only one who has already joined the room.
(request matrix.whitenoise.ch/_matrix/client/unstable/sendToDevice/m.new_device)
C accepts the invitation
C checks all room members who has joined the room XX and from now will check new device announcement from A and B but not before
So for C, list of device  of B is still [device_1]
So when C sends a message in room XX, it cannot be decrypt from device_2.
If my analysis is correct, is it an already known issue?

This issue is related to tracking devices for other matrix users
To avoid missing some devices, a first solution is to ask again all devices of all rooms members when we joins a new room by using POST /_matrix/client/unstable/keys/query but it is quite bandwith consuming
Can we imagine a light version of this request to only get current list fo device_ID for users?
Client will call this request when it joins  a room, check which device_id is missing or has been deleted and ask complete information only for needed device?
We can also imagine to associate a track value for list of device of each user? This track value will be init to 0 and will be incremented each time a device is added or removed?
example:
1st device created : device_tracker[user XX] =  0
2nd device created :device_tracker[user XX] =  1
1st device deleted : device_tracker[user XX] =  2
3rd device created : device_tracker[user XX] =  3
This device_tracker would be returned in room state -> for each room member
So that when a client joins a room, he could easily check if its image of device list is obsolet or not for each room member.

Other remark: if a device cannot decrypt a message because of unknown inbound session, it could be useful to create a new device request to ask to device emitter the next megolm key associated to a megolm session.
So that we miss one message but not the other ones.
What do you think of these proposals?

The issue is similar to https://github.com/vector-im/riot-web/issues/2305.
We have a test testLeftAndJoinedBob that almost checks this scenario.

tracking this as https://github.com/vector-im/riot-web/issues/4983 on the riot-web side


Daily limit

Hi,
Just to check on it does this bot have limit or will my premium account will be banned or suspend if I use this discord and if yes chances of getting this problem. Thanks for your effort it been easy to use when I need answer of course if I where use it I do not want it lead to account problem.

I believe i saw something in the code that mentioned a cooldown but im not sure

Hi,
Just to check on it does this bot have limit or will my premium account will be banned or suspend if I use this discord and if yes chances of getting this problem. Thanks for your effort it been easy to use when I need answer of course if I where use it I do not want it lead to account problem.

There are some cooldowns in place to prevent Chegg's bot detection, which I've tested to prevent the Bot detection page/Captcha. This code can be seen in the parsing library as well as the normal bot code. You can remove this code, however the bot may not be as reliable due to Chegg detecting it.
I'm not sure whether or not it would get your Chegg account nor your Discord account suspended/banned. Use the bot at your own discretion and read the disclaimer.


OpenVPN Instances: Problem with Redirect Gateway multiple flags

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
I use a typical OpenVPN Setup for connecting different users to the company network while they are working from home. With enabled tunnel, the users are able to access ressources inside the companies network and also browse the internet over our companies internet-connection.
I tried to migrate to the new Instances-Configuration. For that, I re-created my currect Configuration from "Server" as Instance. I switched off the old server configuration and enabled the new instances-config. I encountered some problems while doing this, especially with Redirect Gateway options:

The Redirect Gateway configuration allows to select multiple flags (for example "standard" and "bypass dns") at the same time. But when I select more than one flag from this list, the server won't start and instead shows this error:
/usr/local/opnsense/scripts/openvpn/ovpn_service_control.php: The command '/usr/local/sbin/openvpn --config '/var/etc/openvpn/instance-dd9472e0-2f64-46eb-9613-4f6a14fbcd35.conf'' returned exit code '1', the output was 'Options error: PUSH OPTION FAILED (illegal comma (',') in string): 'redirect-gateway def1,bypass-dns' Use --help for more information.'
According to the official OpenVPN documentation, multiple "redirect-gateway" flags are possible, but they need to be separated by space and not by comma.
https://openvpn.net/community-resources/reference-manual-for-openvpn-2-5/

EDIT: I found a problem on my side, need to investigate. Right now, I don't think that this following second part of the bug exists.
2. When only one flag is selected, the server starts. My client is able to connect, but the routing is not right.
If I set Redirect Gateway to "standard", my client at home can reach my company network, but not the internet.
If I set it to "block local" it is the same, I can reach IPs in my companys network, but no public IP like <IP_ADDRESS>.
If I don't select anything for Redirect Gateway, my client with enabled VPN can't reach the company network, but can access the internet directly without using the tunnel.

When I switch back to my old server config (there is only a checkbox for Redirect Gateway, it's not possible to configure anything further like with the new instances setup), redirecting the gateway works as usual and my client can access the company network and the internet all over the tunnel.

To Reproduce

Create a new OpenVPN Server with Instances instead of Server menu.

2a. Select multiple flags for Redirect Gateway
3a. Try and fail to start server, watch for error in Logs
EDIT: I found a problem on my side, need to investigate. Right now, I don't think that this following second part of the bug exists.
2b. Try to use only "standard" flag for redirecting gateway
3b. Try to use the internet over active tunnel connection

Expected behavior
I expected that selecting multiple flags for redirecting gateway works and does not result in an error and server start failure.
Environment
OPNsense 23.7.1_3-amd64
FreeBSD 13.2-RELEASE-p2
OpenSSL 1.1.1v 1 Aug 2023

Thank you for this quick fix!!


Vite dev server closing unexpectedly

Verify canary release

[X] I verified that the issue exists in the latest Turborepo canary release.

Link to code that reproduces this issue
npx create-turbo@latest -e with-vite
What package manager are you using / does the bug impact?
pnpm
What operating system are you using?
Mac
Which canary version will you have in your reproduction?
1.12.0
Describe the Bug
When running turbo run dev the Vite dev server starts up correctly but instantly finishes the task and closes itself unexpectedly.
Expected Behavior
The Vite dev server should keep running until closed manually.
To Reproduce

Clone the vite example: npx create-turbo@latest -e with-vite
Run pnpm dev (turbo run dev with the selected pkg manager)

Additional context
No response

+1

+1

+1

+1

Version 1.12.1 fixed it almost, but having same issue when using --filter, like turbo run dev --filter xyz

ah, sorry for not reading the release log.. I just noticed the PR was merged and a new version was out.
At least version 1.12.2 works as a charm ❤

I'm on 1.12.2 here and it still happens...

I'm on 1.12.2 here and it still happens...

Same

@chris-olszewski can we please get any help on this? 1.12.2 still has the issue, and as you can see I'm not the only one...

Did you add persistent: true to the task in turbo.json?


Reaction Avatars should update on "remote" changes

If you're on an issue/PR and there are reactions to comments the avatars will disappear if someone else reacts. There should be a Mutation Observer or something to track those elements and re-add the avatars on an update.
It's actually hiding the avatars, so something is happening.

Rather than a mutation observer, I think we can just leverage the fact that GitHub's JS emits events on change.
$(document).on('socket:message', (e, data) => {
	// data will be an object of the format:
	// {timestamp:<PHONE_NUMBER>, reason: "reaction #1209345 updated"}
});

We'd know to go run our updates on reactions anytime the reason prop mentions a reaction has been updated

We can just run them all or we can run just the one comment that was updated, but that latter option is not currently supported.

I'll work on it this week :)

Would $(document).on('socket:message') actually work? I got no events when testing it for #132, even though I saw GH's own code triggering the event.

@bfred-it  I also tried $(document).on('socket:message') but it didn't work :(

I thiiiink this happens because jQuery doesn't trigger real DOM events but it just short-circuits within its own event Map. So unless we use GH's own jQuery (we can't) we probably won't be able to listen to them.
Pinging jQuery experts around the world. It'd be nice to plug into GH's own events rather than rely on MutationObservers

I'm reopening this issue hoping for a proper resolution. My fix doesn't work all the time and especially doesn't work on ajax-loaded comments + their new reactions.


Seed Collide in Different String

PoC

Hi @soranoo, just because I'm too lazy to create a new issue, I'm commenting on the old bug here.
I think it's very rare that this happens, but is there any way to be sure it doesn't overlap anymore?

Package version 2.2.6
I just hope this patch won't change the class name again, because it seems like you'll be editing the recovery-random package.

from https://github.com/soranoo/next-css-obfuscator/issues/18#issuecomment-1940525029


[usecase] Add ContextMenu test for lite wrt usecase

Enable bootstrap framework for usecase-litewrt-android-tests
Add ContextMenu test for lite wrt usecase

Impacted tests(approved): new 1, update 1, delete 0
Unit test platform: Crosswalk Project for Android 16.45.410.0
Unit test result summary: pass 2, fail 0, block 0
BUG=https://crosswalk-project.org/jira/browse/XWALK-4973

LGTM


Text Attribute from LTTextLineHorizontal (LTTextBox)

I can't access the text attribute of an LTTextLineHorizontal object. I can get the text via get_text(), but I can't write into the text attribute.
E.g., I've written all LTTextBox objects  (obj) of a document into a list. I now want to go over these objects and clean them. It's possible to access all attributes with e.g. obj.y1, but not the text. I can get the text with obj.get_text() but not access it. When I call obj._objs[0]I get the LTTextLineHorizontal object, but still don't know how to access the text attribute and work on it.

I can't access the text attribute of an LTTextLineHorizontal object. I can get the text via get_text(), but I can't write into the text attribute.

A LTTextLineHorizontal object is a hierarchical grouping of objects that contain text. The LTTextLineHorizontal object itself does not contain any text. The LTTextLineHorizontal.get_text() method only concattenates the text from its children. That is why you cannot write to any text attribute.

E.g., I've written all LTTextBox objects (obj) of a document into a list. I now want to go over these objects and clean them. It's possible to access all attributes with e.g. obj.y1, but not the text. I can get the text with obj.get_text() but not access it. When I call obj._objs[0]I get the LTTextLineHorizontal object, but still don't know how to access the text attribute and work on it.

If you want to work with the text, I would recommend extracting the text from the LT*objects and store the result in a list or dict structure. That way you can manipulate the result without having to write the values to anyLT*` object.


Default Angular Template creates duplicate html and body tags

Create a new ASP.NET Core 2.0 Web App with the default Angular template. Build and run. View the source. Note the html, head, and body tags located within the  tag. The console also enumerates the errors. This is obviously unintended, and I assume is an artifact of the pre-renderer.

I'm not sure why this is still a problem even though it was supposedly fixed:
https://github.com/angular/universal/issues/347
I ended up putting the following in boot.server.ts for now:
                    resolve({
                        html: state.renderToString()
                            .replace('', '')
                            .replace('', '')
                            .replace(/<\s*app.*?>/,'')
                            .replace('</app>', '')
                    });

@ltctech I don't see anywhere in that issue that they say they fixed it. AFAIK it remains an issue.
@MarkPieszak has been working with the Angular team on hopefully improving the APIs around server-side rendering so we can get the distinct outputs (HTML, title, styles, etc.) separately instead of them all being mashed together in a big HTML document string. If Angular does implement that then we'll certainly make use of it, but failing that @ltctech's workaround is as good as it gets.

@Anyone found a solution to this problem?


Patching firmware of existing ASV

Hi, I have an ASV and I'm trying to jailbreak it so I can set the maxPS value to 5 while keeping the minPS at 3.
Would I be able to achieve this by following the standard firmware patching instructions?
(By patching the official firmware of my ASV device)
Will it still function the same way, but with more freedom to set maxPS?
For more context, the ASV has solved my CA events but I've noticed my heart rate goes up whenever the PS ramps up
Thanks :)

Were you ever successful in doing this?
I attempted to patch the 305 ASV firmware with the patches from:
https://github.com/Asmageddon/airbreak-plus/blob/master/patch-airsense#L149
https://github.com/Asmageddon/airbreak-plus/blob/master/patch-airsense#L139
https://github.com/Asmageddon/airbreak-plus/blob/master/patch-airsense#L40
(with some minor address fix ups) But had no luck. Do you have 40x ASV firmware to test with?

I managed to patch the 5cm difference limit for asv-37043-SX567-0305.bin firmware, even though it didn't seem to allow having a difference less than 5 on first install. After messing with some other firmware it eventually worked ok - I think it required the memory erase that it sometimes does when switching firmware.
base64 xdelta patch follows:
JVhEWjAwNCUAAAAIABgAFwAAAAAAAAAAAAAAAAAAAABhc3YtMzcwNDMtU1g1NjctMDMwNS5iaW5h
c3YtcHMtbm8tbGltaXQtMzB4LmJpbh+LCAAAAAAAAAM74HaHlYHBCEEwMDAwAkkuBk9GNgYWhi8M
wgwngGKYcgxAOQ0UOQYGAHtlTJhOAAAAH4sIAAAAAAAAA2NgaGBmYGDSaLv99Pe/Caf/bWv7Zvsp
WY+7ocGBkYlHoyCxJDlDISWxJFHz52EVr9OXlhzZL+B9carm60A/RkaJxOIyXWNzAxNj3eAIUzNz
XQNjA1O9pMw8Pwmen5d/cs2JsZgVlipxzBRoGgODMCPDB0agXYyfGLfNBTJYGdfOZxQH0SfnMz6P
ZWCwZXz7l7Ed6BxGxq8NTN2MIEZjE1P3NWkQq/eGrDaI3nlDdipY6vxN2Y3LlACW9/WKwQAAAAAA
AIolWERaMDA0JQ==


Fix output was not set if fetching from upstream resulted in error

Was this causing an issue? I remember all my tests going fine when I didn't explicitly the output to false. Probably a good fix, just want to understand your use case.

If the output value is empty, then both check output == 'true' and output == 'false' are false in subsequent steps or jobs
Without the fix

With the fix

https://github.com/Fubuchi/clr-boot-manager/blob/846892eee9/.github/workflows/main.yml#L58

Sweet, thanks for sharing. I'll merge this. 👍
I'm waiting on another issue fix to be confirmed right now. I think I'll put them together in the same release. Shouldn't be too long.

Pre-released to v3.3!