Installing miRge3.0 on Windows - Error: "Failed building wheel for isal"

Hi,
I am trying to install miRge3.0 on Windows 10 following the instructions provided in the documentation.
I am at the step Installing miRge3.0 with PyPi > First install miRge dependencies. After executing the command python3.8 -m pip install --user cutadapt reportlab==3.5.42 biopython==1.78  scikit-learn==0.23.1  hypothesis==5.15.1 pytest==5.4.2  scipy==1.4.1  matplotlib==3.2.1  joblib==0.15.1  pandas==1.0.3 future==0.18.2, at a certain point I get the error Failed building wheel for isal. After executing some other lines, the script exists with the message:

Command "/usr/bin/python3.8 -u -c "import setuptools, tokenize;__file__='/tmp/pip-build-e2j491ir/isal/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /tmp/pip-fzsl_opz-record/install-record.txt --single-version-externally-managed --compile --user --prefix=" failed with error code 1 in /tmp/pip-build-e2j491ir/isal/

I cannot find a solution for how to fix this problem. Could you please help me out?
Thank you!
Marta

I found that executing Ubuntu as administrator solved my issue. I'm leaving the original comment in case it can be useful for other people having my same problem.

@marta-nazzari, Great (I was looking for a solution and got your reply).


Rename pending to skipped, fix “Running XX tests” message.

pending is confusing to people at FB and I agree. Skipped makes more sense. This only updates the user output and continues to use pending in code – to align it with what Jasmine uses internally.
I also removed a reference to a variable that doesn't exist which broke the Running XX tests... feature.

@facebook-github-bot import


V2.4.1

Minor changes before release v2.4.1

Coverage remained the same at 41.393% when pulling 5373a363b84451b6f004921ba1281d92714ad9b6 on v2.4.1 into 7def5bc74a9f24f081a95fb192771df0c062e82b on master.


CameraView iOS

Description of Change
Adds the iOS part (and shared part) of the CameraView
Bugs Fixed

Related to issue #211

API Changes
Added:

public enum CameraOptions enum for selecting front, back, default or external camera
public enum CameraFlashMode enum for enabling, disabling, torch or auto the flash
public enum CameraCaptureOptions enum for setting the camera to photo, video or default

public class CameraMediaSource : MediaSource
{
	public static MediaSource FromStream(Func<Stream> stream) ;
	public static MediaSource FromResource(string resource, Type resolvingType);
	public static MediaSource FromResource(string resource, Assembly sourceAssembly = null);
}

public class StreamMediaSource : MediaSource, IStreamImageSource
{
	protected CancellationTokenSource CancellationTokenSource { get; }
	public virtual Func<CancellationToken, Task<Stream>> Stream { get; set; } // Bindable
	protected override void OnPropertyChanged(string propertyName);
	protected void OnLoadingCompleted(bool cancelled);
	protected void OnLoadingStarted();
	public virtual Task<bool> Cancel();
}

public class MediaCapturedEventArgs : EventArgs
{
         internal MediaCapturedEventArgs();
	public object Data { get; }
	public ImageSource Image { get; }
	public MediaSource Video { get; }
}

public class CameraView : View
{
	public event EventHandler<MediaCapturedEventArgs> MediaCaptured;
	public event EventHandler<string> MediaCaptureFailed;
	public event EventHandler<bool> OnAvailable;

	public bool IsBusy { get; set; } // Bindable
	public bool IsAvailable { get; set; } // Bindable
	public CameraOptions CameraOptions { get; set; } // Bindable
	public bool SavePhotoToFile { get; set; } // Bindable
	public CameraCaptureOptions CaptureOptions { get; set; } // Bindable
	public bool KeepScreenOn { get; set; } // Bindable
	public bool VideoStabilization { get; set; } // Bindable
	public CameraFlashMode FlashMode { get; set; } // Bindable
	public Aspect PreviewAspect { get; set; } // Bindable
	public double Zoom { get; set; } // Bindable
	public double MaxZoom  { get; set; } // Bindable
	public void Shutter();

         internal event EventHandler ShutterClicked;
	internal void RaiseMediaCaptured(MediaCapturedEventArgs args);
	internal void RaiseMediaCaptureFailed(string message);
}

Behavioral Changes
New control, so should have any behavioral changes :)
PR Checklist

[x] Has tests (if omitted, state reason in description)
[x] Has samples (if omitted, state reason in description)
[x] Rebased on top of main at time of PR
[x] Changes adhere to coding standard
[ ] Updated documentation

I think this is good to go also!

Oh I see you had some more questions @pictos I didn't see them yesterday sorry about that. I'm going to merge this so I unblock you and we'll polish this later. Add the bits that you think should be changed about this in a PR so we can discuss it there.

@jfversluis don't worry, my last questions are minimal. It's ok to discuss that later (:

There are still some bugs I noticed...

if you put the app in background and then resume the app, the camera is frozen because the Camera got deactivated.
Once you leave the page and use a BindingContext for the properties, all properties are reset and the whole RetrieveCameraDevice will run, which can slow down the closing process of the page closing.

Thanks for already testing this @acuntex ! I love the early feedback.
Would you mind maybe opening separate issues for that with as much details as you have so we can track them? Thanks!

Oh it's a pleasure, love it :-)
Wasn't sure if this repo is accepting bug reports, but I'll open them.


See home screen without being logged

Hello,
Is it possible to configure the network to not require signup before using it?
If no, could you point me a way to do that? I would like to help developing this option.
Cheers,
Carlos

Hi Carlos,
You have to find the particular view you want people to be able to view without login, and delete the @login_required if it is a function based view and the LoginRequiredMixin if it is class based views. I don't think this will cause any immediate problems, but it might in the future, just as a warning.
Hope this helps,
Anand

It worked, thank you!

That was how I did it, maybe it would help other people.
https://github.com/EncontrosDigitais/bootcamp/commit/1504d11d121693ff5abf0e421ec8ccbaea4d7b46
It would be wonderful have this optional in the .env file.


TS->JS schema transpilation does not work

The schema compiler (capnpc-ts) currently has all the code necessary to transpile the emitted .ts file into .js using the TypeScript compiler API directly, but I have failed to get it to actually work without choking on the capnp-ts import.
The compiler configuration is here: https://github.com/jdiaz5513/capnp-ts/blob/b5cb174a990c5fb45288289bceaf17362c19b778/packages/capnpc-ts/src/constants.ts#L54-L66
The transpile step is commented out here: https://github.com/jdiaz5513/capnp-ts/blob/b5cb174a990c5fb45288289bceaf17362c19b778/packages/capnpc-ts/src/index.ts#L61-L62
I tried using the baseUrl and paths options to no avail; but it's entirely possible I missed something.
Ideally, the TypeScript compiler should load capnpc-ts's copy of capnp-ts from node_modules, and not depend on capnp-ts being available from the path it's compiling from... or something.

Just familiarising myself with things. I'm thinking that being able to emit js should maybe not be the responsibility of capnpc-ts. As the schema compiler for TS its job is done when you have a set of valid TS modules that you can use directly in your TS code. This means when serving TS users we don't need to know what their TS->JS compilation configuration needs are, they handle that downstream.
Obviously it's pretty low hanging fruit to then also release a capnpc-js tool that uses capnpc-ts internally, and we should do that. It would work by running capnpc-ts to an intermediary dir, then set up some stock tsc configuration and run the typescript compiler to produce js (basically as you were trying to do here). If you'd like I could start working on that layer to build out a direct to JS version, although as a customer I have more use for the TS than the JS capnpc. So I might have a better feel for other features, but if this is a good priority I'm keen to help.
Let me know if that seems off architecturally, still familiarising myself after all.

No, that's totally fair. I just know a lot of folks are going to want a pure JS schema compiler one way or another, but it can wait for now.
Having it be a separate package is probably best (capnpc-js?).

@efokschaner I know it sounds boring, but I could use some help with test coverage on the schema compiler - I'm currently working on coverage for the main library.
It's a decent way to get familiar with that part of the code; just beware that the TypeScript compiler API is barely documented so... good luck!

Adding tests sounds fine to me. And I've actually got a few dozen hours under my belt of interfacing with the tsc lib. That said, should tests fuss with using the tsc lib, or should we just invoke tsc through the command line?
Regarding capnpc-js there is a project in that space already: https://github.com/capnp-js . I don't think this should stop us releasing a potentially better version based on capnp-ts, but it's worth taking a look and maybe even reaching out to the author to create a plan to avoid creating ambiguity for users, whatever that might look like.
One interesting thing to note is that developing capnpc-js and adding tests for that would indirectly provide test coverage of capnpc-ts. I'm a big fan of writing really end to end tests when starting out, i feel it gets the most coverage for your investment. So maybe we could start out that way?

I like that plan. Look forward to seeing your work!
I'll reach out to the capnp-js maintainer(s) later and see how we can work
this all together. Given the utter lack of activity from that end I don't
exactly expect friction.
On Sat, Jun 10, 2017, 7:12 PM efokschaner<EMAIL_ADDRESS>wrote:

Adding tests sounds fine to me. And I've actually got a few dozen hours
under my belt of interfacing with the tsc lib. That said, should tests fuss
with using the tsc lib, or should we just invoke tsc through the command
line?
Regarding capnpc-js there is a project in that space already:
https://github.com/capnp-js . I don't think this should stop us releasing
a potentially better version based on capnp-ts, but it's worth taking a
look and maybe even reaching out to the author to create a plan to avoid
creating ambiguity for users, whatever that might look like.
One interesting thing to note is that developing capnpc-js and adding
tests for that would indirectly provide test coverage of capnpc-ts. I'm a
big fan of writing really end to end tests when starting out, i feel it
gets the most coverage for your investment. So maybe we could start out
that way?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jdiaz5513/capnp-ts/issues/5#issuecomment-307595551,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACn_Z0oDPNyftRrEKE6YnW6_9SRGdExiks5sCyLugaJpZM4N00KN
.

Btw I recommend using the tsc lib since it'll be more likely to be portable
than anything involving a shell (I'm thinking of Windows, specifically).
On Sat, Jun 10, 2017, 9:09 PM Julián Díaz<EMAIL_ADDRESS>wrote:

I like that plan. Look forward to seeing your work!
I'll reach out to the capnp-js maintainer(s) later and see how we can work
this all together. Given the utter lack of activity from that end I don't
exactly expect friction.
On Sat, Jun 10, 2017, 7:12 PM efokschaner<EMAIL_ADDRESS>wrote:

Adding tests sounds fine to me. And I've actually got a few dozen hours
under my belt of interfacing with the tsc lib. That said, should tests fuss
with using the tsc lib, or should we just invoke tsc through the command
line?
Regarding capnpc-js there is a project in that space already:
https://github.com/capnp-js . I don't think this should stop us
releasing a potentially better version based on capnp-ts, but it's worth
taking a look and maybe even reaching out to the author to create a plan to
avoid creating ambiguity for users, whatever that might look like.
One interesting thing to note is that developing capnpc-js and adding
tests for that would indirectly provide test coverage of capnpc-ts. I'm
a big fan of writing really end to end tests when starting out, i feel it
gets the most coverage for your investment. So maybe we could start out
that way?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jdiaz5513/capnp-ts/issues/5#issuecomment-307595551,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACn_Z0oDPNyftRrEKE6YnW6_9SRGdExiks5sCyLugaJpZM4N00KN
.

Hey, it's been a while but I just wanted to let you know I have work in progress on capnpc-js and associated tests. It's actually running into failures in the schema deserialisation, so first bug found maybe? =D I hope to share soon.

Cool! I wonder if we found the same bug.


Add app label to gateways

This PR adds app labels to the gateways via helm.
It is entirely focused on improving the UX for various monitoring solutions (including the servicegraph), as labels["app"] is used as an expression in various places. This moves these values from being "unknown" to properly reflecting their purpose.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: douglas-reid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [douglas-reid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: andraxylia, douglas-reid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [andraxylia,douglas-reid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Device permissions for the browser are going GA

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/concepts/device-capabilities/browser-device-permissions.md
87
6
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

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

File
Status
Preview URL
Details

msteams-platform/concepts/device-capabilities/browser-device-permissions.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

LGTM 👍

Already updated:


NavigationBar shows different style on Android and IOS

I found navigationBar on android present different style compare to ios.(the title did not display on the middle of sceen)
I think the current android style of navigationBar's Title is more likely to design for BreadcrumbNavigationBar, so i removed marginLeft and changed alignItems to center.
I started a PR(https://github.com/facebook/react-native/pull/11318) but get some problem with circleci, i must modify the circle.yml to pass the test and i don't know whether should i modify it.

That's not a bug, it's just following the design specs, they have the different height too.
You can read the source code, there are different styles for iOS and Android platforms, but you can customize a NavigationBar to use a unified style

This should be the default behaviour @Zhu-Arthur, both platforms have different HIG.

Thanks for @nihgwu @alvaromb , i just thought the title should be showed on the middle of the screen.
It seems like i misunderstand the default design.
Anyway, thanks a lot for you two, again.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with Canny! Check out its new home: https://react-native.canny.io/feature-requests/p/navigationbar-shows-different-style-on-android-and-ios


Add an option to override the installPath (use directory other than /tmp or tmpdir)?

This may be useful on slow network connections, where it may take over 20 seconds to download after restarting a computer and clearing the system temporary directory.
Something like the following approach in index.js may help when multiple developers or CI jobs use the same application code on different OSes
 var Config = {
-    installPath: path.join(os.tmpdir(), 'dynamodb-local'),
+    process.env.DYNAMODB_LOCAL_INSTALL_PATH || installPath: path.join(os.tmpdir(), 'dynamodb-local'),
     downloadUrl: 'https://s3-us-west-2.amazonaws.com/dynamodb-local/dynamodb_local_latest.tar.gz'
 };

(I didn't see this suggested elsewhere in PRs/issues after a quick check)

https://github.com/rynop/dynamodb-local/blob/master/examples/simple.js never mind, it was in the bottom of the README


Update supported Python versions

Acc. to GitHub actions config the software is tested on 3.7 - 3.10

Thank you for the PR!


SCSS stylelint task should use the stylelint-config-wordpress SCSS config

Follow up to #60 #62

Thanks! I'd done a quick run, and saw that @mixin, for instance, was coming up as an unknown media type, but I hadn't had chance to investigate further.


v2

Hi @rahul69x ,
When opening an issue, please try to provide all or some of the information indicated in the template.
Closing this issue, since no information has been provided.
Thank you.


support mac os

FW lite will require an icu binary to run, we already have those for windows and linux thanks to this package.
To get the binary we can copy what paranext did here:
https://github.com/paranext/paranext-core/pull/942
During build the install ICU via MacPorts
      - name: Install MacPorts
        if: ${{ matrix.os == 'macos-latest' }}
        uses: melusina-org/setup-macports@v1

      - name: Install icu4c on macOS
        if: ${{ matrix.os == 'macos-latest' }}
        run: |
          sudo port -v install icu

then they configure msbuild to copy the binary into the ouput:
  <!--
    This assumes that icu4c libs are installed under "/opt/local/lib" on macOS. This is the default
    location for libs that MacPorts installs. If we need to start bundling icu4c libs from
    somewhere other than "/opt/local/lib", this path and some other paths in the build scripts will
    need to be updated. We have to bundle the icu4c libs somehow unless macOS starts to include them
    by default.
    -->
  <ItemGroup>
    <Content Include="/opt/local/lib/libicu*.??.dylib" Condition="$([MSBuild]::IsOsPlatform('macOS'))">
      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
    </Content>
  </ItemGroup>

we could do the same thing, we could probably just build all versions on mac and it'll work just fine, we're already building all versions on Linux and that works fine.

[ ] #1006

FWLite works on Mac(ARM) now!

This is so cool!  Awesome job!


Save recipe unlocks

From that_one_ITGUY:

Maybe add a command to save unlocked alternate recipes. To save space on your host you might be able to use reactions in its own channel. Like each recipe is its own message with a check or x reaction and then the user would check one when they unlock it and it could use that as the default recipe for crafting requests (unless specifically requesting some other alternative). Not sure how much programming it would take but it would be a nice feature.

This could either be in DMs with individual users or in a specifically named channel


docs: Lowercase Fargate compute type annotation value

Description
Change the custom annotation on coredns pods to fargate instead of Fargate.
Motivation and Context
I just set up a cluster using Fargate for the kube-system namespace, and the additional configuration for coredns as expressed by the example subject of this PR failed, because the node taint is against an annotation with value fargate.

This PR is included in version 20.2.2 :tada:


Implement MNAUTH and allow unlimited inbound MN connections

This implements MNAUTH, which allows us to verify that an incoming connection is indeed from another MN. This in turn allows us to exclude inbound MN connections from limit checks and at the same time protect these connections from eviction.
This is required to ensure that intra-quorum connections stay active as long as they are needed and never get evicted due to MNs being spammed with inbound connections.
The reason we can exclude authenticated MN connections from eviction is because these are naturally limited. MNAUTH will drop any previous connections that were authenticated with the same MN, which means that each MN can cause only one incoming authenticated (non-evictable) connection.
MNAUTH is also sent/verified for outgoing connections, this is however not of interest when it comes to the inbound connection eviction and DoS (or network partition) protection. It is however useful to avoid unnecessarily connecting to a MN that already connected to us when ensuring intra-quorum connections.

Rebased on develop to fix merge conflicts. Needs re-ACK


labels and output

how to make labels without that redrawn lanes ? or how to get those redrawn lanes
can we change the final output to be a convolution and not a deconv and what should the output be

This project uses a fully convolutional neural network, so the only possible output, as labeled, is an identification of the lane within the lane lines. It is not set up to only do the lines.

Closing out since no further comments.


[FEATURE] 简洁阐述功能 / Concise description of the feature

How do I add a knowledge base with my fine-tuned model？

fastchat.model.apply_lora.apply_lora

By using this method to merge Lora weight files into model files, the merged model file path needs to have a 'pert' identifier
https://github.com/lm-sys/FastChat/pull/1905#issuecomment-1627801216


Line endings

This did not work on my windows machine due to different line endings,
I changed this
to var infoData = data[1].split(/[\n\r]+/);
I think that should cover all cases but have not tested

Thanks! I'll open a PR for this.

Patched with PR #14


Allow for call-out style customizations in settings

Allow for customization of the Call-out style in the settings (e.g. abstract, info, note, success, question, warning, failure, danger, bug, example, quote, custom, none)

Done. As of v0.2 this is now a customization that can be set in the "Settings" screen.


Add additional Azure DevOps (TFBuild) properties

There's a few missing properties on the TFBuild classes for Azure DevOps Pipeline build variables, documented in https://docs.microsoft.com/en-us/azure/devops/pipelines/build/variables?view=azure-devops&tabs=yaml
Working on a PR to add these in

I just noticed that @gitfool has done a couple of these recently (eg. Reason)

I was more focused on laying the foundation for identifying a pull request, across multiple providers, rather than capturing every last detail.

Probably good timing then that I come after your commit and can add on to your work, given you've looked at the other providers as well.
👍

fixed by  #2492


PCMark UI error

Hello,
When running WA from Lisa, PCMark crashes at every iteration with the following error:
run.log
I'm running WA 3.0.0 on a Pixel 2 device with Android 8.1.
Any suggestions on how to fix this ? Thank you in advance.

Hello,
Would you be able to tell me the version of PCMark you are using and could you also send me the captured .png and .uix file (which should be located in the __failed/ directory)?
Thanks

I am using PCMark 2.0. Below is the zip with the two files.
pcmark-capture.zip
Thank you!

Thanks for providing them, it looks like the device is still determine whether the tests are installed or not at this particular stage.
If you launch the app manually could you tell me what the "Work 2.0 Initializing" text changes to once it's finished loading?

It remains there, and the orange install button does not appear even when I launch the app manually.

Ah I see. So it seems like it is a problem with the app itself, could you try uninstalling it completely and reinstalling to see if that allows it to load correctly?

I did that just now, plus rebooting the device, and it still doesn't load correctly

Hmm.. Could you confirm the exact version of the apk you are using? For reference the one I am testing with is v2.0.3716.

I have the same apk, 2.0.3716

The device is connected to the network, I think it is also visible in the png capture from before

Does the network have internet access? / Do other apps on the device work correctly?

Yes, other apps behave as expected. I just reverted back to PCMark v1.3.3083 and it is stuck in the same state, however this one displays a device unrecognizable message on the 'my device' tab

Is there anything running on the device that could be interfering with the app / its internet connectivity in the background?

No, nothing is interfering with the app. The only thing could be that other tests are run with airplane mode enabled, but for PCMark I enabled back the network access.

Could you try leaving airplane mode off, checking the web browser works correctly etc and then launching the app manually to see if that could be the cause of the problem?

I attempted that. Other apps work correctly (browser, youtube etc.), but PCMark keeps hanging in all scenarios. I also flashed a new kernel and the result is still the same

Hmm.. I'm sorry I have not managed to recreate that issues on any of our devices here so I'm not really sure what to suggest really apart from maybe try contacting the developers of PCMark to see if they have any further ideas?

I'll continue to investigate, thank you very much for your time

Assuming no longer relevant.


Fix output column in transforms

Before submitting (must do checklist)

[ ] Did you read the contribution guide?
[ ] Did you update the docs? We use Numpy format for all the methods and classes.
[ ] Did you write any new necessary tests?
[ ] Did you update the CHANGELOG?

Proposed Changes

Closing issues

Codecov Report

Merging #1152 (1386015) into master (9f0f663) will decrease coverage by 18.11%.
The diff coverage is n/a.

:exclamation: Current head 1386015 differs from pull request most recent head 922ffd7. Consider uploading reports for the commit 922ffd7 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##           master    #1152       +/-   ##
===========================================
- Coverage   86.98%   68.88%   -18.11%
===========================================
  Files         177      177
  Lines        9746     9742        -4
===========================================
- Hits         8478     6711     -1767
- Misses       1268     3031     +1763

Impacted Files
Coverage Δ

etna/commands/__main__.py
0.00% <0.00%> (-87.50%)
:arrow_down:

etna/datasets/hierarchical_structure.py
22.01% <0.00%> (-75.23%)
:arrow_down:

etna/libs/pmdarima_utils/arima.py
28.84% <0.00%> (-71.16%)
:arrow_down:

etna/pipeline/assembling_pipelines.py
26.31% <0.00%> (-71.06%)
:arrow_down:

etna/pipeline/mixins.py
32.03% <0.00%> (-67.00%)
:arrow_down:

etna/analysis/outliers/hist_outliers.py
10.82% <0.00%> (-66.25%)
:arrow_down:

etna/reconciliation/bottom_up.py
35.29% <0.00%> (-64.71%)
:arrow_down:

etna/auto/optuna/config_sampler.py
32.25% <0.00%> (-62.91%)
:arrow_down:

etna/pipeline/hierarchical_pipeline.py
37.31% <0.00%> (-61.20%)
:arrow_down:

etna/models/deadline_ma.py
35.18% <0.00%> (-59.26%)
:arrow_down:

... and 59 more

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


WIP: build: initial linux x86_64 support using nix

it basically builds without error. Both the shared graal library "LibScimacs" and the Rust library "scimacs". It can't be loaded for a reason I don't yet understand.
Module could not be opened: "/home/benny/Code/Playground/scimacs/scimacs.so", "/home/benny/Code/Playground/scimacs/scimacs.so: undefined symbol: graal_create_isolate"

Full results:
$ emacs --script test.el
Testing scimacs...
Loadable module found: /home/benny/Code/Playground/scimacs/scimacs.so

Error: module-open-failed ("/home/benny/Code/Playground/scimacs/scimacs.so" "/home/benny/Code/Playground/scimacs/scimacs.so: undefined symbol: graal_create_isolate")
  mapbacktrace(#f(compiled-function (evald func args flags) #<bytecode 0x10aa1fbde7aef0ea>))
  debug-early-backtrace()
  debug-early(error (module-open-failed "/home/benny/Code/Playground/scimacs/scimacs.so" "/home/benny/Code/Playground/scimacs/scimacs.so: undefined symbol: graal_create_isolate"))
  require(scimacs)
  (progn (message (format "Loadable module found: %s" location)) (require 'scimacs) (message "Symbols loaded from scimacs:") (mapatoms #'(lambda (symbol) (let ((symbol-name (symbol-name symbol))) (if (string-prefix-p "scimacs-" symbol-name) (message (format "  %s" symbol-name)))))) (message "Evaluating forms with sci:") (mapcar #'(lambda (form) (message (format "  %s ; => %s" form (scimacs-eval-sci form)))) '("[1 2 3]" "(+ 1 1)" "(mapv inc (range 5))" "(reduce + (range 5))")))
  (if location (progn (message (format "Loadable module found: %s" location)) (require 'scimacs) (message "Symbols loaded from scimacs:") (mapatoms #'(lambda (symbol) (let ((symbol-name (symbol-name symbol))) (if (string-prefix-p "scimacs-" symbol-name) (message (format "  %s" symbol-name)))))) (message "Evaluating forms with sci:") (mapcar #'(lambda (form) (message (format "  %s ; => %s" form (scimacs-eval-sci form)))) '("[1 2 3]" "(+ 1 1)" "(mapv inc (range 5))" "(reduce + (range 5))"))) (message "Could not find loadable module!"))
  (let* ((location (and t (locate-library "scimacs")))) (if location (progn (message (format "Loadable module found: %s" location)) (require 'scimacs) (message "Symbols loaded from scimacs:") (mapatoms #'(lambda (symbol) (let ((symbol-name (symbol-name symbol))) (if (string-prefix-p "scimacs-" symbol-name) (message (format "  %s" symbol-name)))))) (message "Evaluating forms with sci:") (mapcar #'(lambda (form) (message (format "  %s ; => %s" form (scimacs-eval-sci form)))) '("[1 2 3]" "(+ 1 1)" "(mapv inc (range 5))" "(reduce + (range 5))"))) (message "Could not find loadable module!")))
  #<subr F616e6f6e796d6f75732d6c616d626461_anonymous_lambda_99>(#<buffer  *load*> "/home/benny/Code/Playground/scimacs/test.el")
  load-with-code-conversion("/home/benny/Code/Playground/scimacs/test.el" "/home/benny/Code/Playground/scimacs/test.el" nil t #<subr F616e6f6e796d6f75732d6c616d626461_anonymous_lambda_99>)
  command-line--load-script("/home/benny/Code/Playground/scimacs/test.el")
  command-line-1(("-scriptload" "test.el"))
  command-line()
  normal-top-level()
Module could not be opened: "/home/benny/Code/Playground/scimacs/scimacs.so", "/home/benny/Code/Playground/scimacs/scimacs.so: undefined symbol: graal_create_isolate"

Just to share my thoughts on where it's stuck at the moment:
The rust shared library is not re-exporting the symbols from the graalvm shared library on Linux.
I've played around with it a bit and then found this and it doesn't look good for linux:
https://github.com/rust-lang/rfcs/issues/2771
The workarounds with lto and incremental all didn't help. I haven't spent the time yet to see if a wrapping export is fruitful.

Reason why I think it's related to rust and still be target specific is for example this kind of comment: https://github.com/rust-lang/rust/issues/50007#issuecomment-437174617
Yesterday I saw several such comments but my history is long and I don't have a full text search. :-)

@bennyandresen I'm trying to make a CI config for Github actions in this branch:
https://github.com/jackrusher/scimacs/actions/runs/4490409982/jobs/7897440985
It fails with a maybe similar error?
@jackrusher is going to try a C-wrapper as well, maybe it will help also with this problem.

@bennyandresen I'm trying to make a CI config for Github actions in this branch:
https://github.com/jackrusher/scimacs/actions/runs/4490409982/jobs/7897440985
It fails with a maybe similar error?

That error is related to the case sensitivity in linux in the build.rs file. With a lower case "s" it should continue past that stage.


Move and remove old versions from other repos

https://github.com/OpenEnergyPlatform/tutorial/tree/develop/data/datapackage/oep-scenario-data-datapackage

@Ludee What do we want to make available in the oedatamodel repo. besides the ERM. I can imagine that on the one hand the collected provision of meta data, .csv and erm is practical for the user but unfortunately this disturbs our structure because elements are packed together that do not fall directly under the same term.

the metadata is part of the datapackage and should be included.
I would like to have an example and/or an template as well.

@Ludee I added the data package files. Please review the current status. The metadata files are not yet filled, because I had no good idea for example content. If you approve I will merge the release branch.
Current state: https://github.com/OpenEnergyPlatform/oedatamodel/tree/release/oemetadata-v1.0.0/oedatamodel/latest/v100


Plugin throws an exception and fails if connection to MusicBrainz fails

Occasionally, my ISP assigns me a shared IP that's banned by MusicBrainz/ListenBrainz, causing listen submission to fail. This failure is expected.
However, it appears that the retry policy doesn't actually work in that case. Additionally, it doesn't print the raw JSON request (anymore). I think this was introduced when recording attributes started to be resolved through MusicBrainz directly.

Logs
Feb 21 00:37:12 iridium docker[303242]: Unhandled exception. [23:37:12] [FTL] [36] Main: Unhandled Exception
Feb 21 00:37:12 iridium docker[303242]: System.Threading.Tasks.TaskCanceledException: The request was canceled due to the configured HttpClient.Timeout of 100 seconds elapsing.
Feb 21 00:37:12 iridium docker[303242]:  ---> System.TimeoutException: A task was canceled.
Feb 21 00:37:12 iridium docker[303242]:  ---> System.Threading.Tasks.TaskCanceledException: A task was canceled.
Feb 21 00:37:12 iridium docker[303242]:    at System.Threading.Tasks.TaskCompletionSourceWithCancellation`1.WaitWithCancellationAsync(CancellationToken cancellationToken)
Feb 21 00:37:12 iridium docker[303242]:    at System.Net.Http.HttpConnectionPool.GetHttp11ConnectionAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
Feb 21 00:37:12 iridium docker[303242]:    at System.Net.Http.HttpConnectionPool.SendWithVersionDetectionAndRetryAsync(HttpRequestMessage request, Boolean async, Boolean doRequestAuth, CancellationToken cancella>
Feb 21 00:37:12 iridium docker[303242]:    at System.Net.Http.DiagnosticsHandler.SendAsyncCore(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
Feb 21 00:37:12 iridium docker[303242]:    at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
Feb 21 00:37:12 iridium docker[303242]:    at Microsoft.Extensions.Http.Logging.LoggingHttpMessageHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
Feb 21 00:37:12 iridium docker[303242]:    at Microsoft.Extensions.Http.Logging.LoggingScopeHttpMessageHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
Feb 21 00:37:12 iridium docker[303242]:    at System.Net.Http.HttpClient.<SendAsync>g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts>
Feb 21 00:37:12 iridium docker[303242]:    --- End of inner exception stack trace ---
Feb 21 00:37:12 iridium docker[303242]:    --- End of inner exception stack trace ---
Feb 21 00:37:12 iridium docker[303242]:    at System.Net.Http.HttpClient.HandleFailure(Exception e, Boolean telemetryStarted, HttpResponseMessage response, CancellationTokenSource cts, CancellationToken cancella>
Feb 21 00:37:12 iridium docker[303242]:    at System.Net.Http.HttpClient.<SendAsync>g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts>
Feb 21 00:37:12 iridium docker[303242]:    at Jellyfin.Plugin.Listenbrainz.Clients.BaseHttpClient.SendRequest(HttpRequestMessage request)
Feb 21 00:37:12 iridium docker[303242]:    at Jellyfin.Plugin.Listenbrainz.Clients.BaseMusicbrainzClient.DoRequest[TResponse](HttpRequestMessage requestMessage)
Feb 21 00:37:12 iridium docker[303242]:    at Jellyfin.Plugin.Listenbrainz.Clients.BaseMusicbrainzClient.Get[TRequest,TResponse](TRequest request)
Feb 21 00:37:12 iridium docker[303242]:    at Jellyfin.Plugin.Listenbrainz.Clients.MusicbrainzClient.GetRecordingData(String trackId)
Feb 21 00:37:12 iridium docker[303242]:    at Jellyfin.Plugin.Listenbrainz.Clients.ListenbrainzClient.SubmitListen(LbUser user, User jfUser, SubmitListenRequest request)
Feb 21 00:37:12 iridium docker[303242]:    at System.Threading.Tasks.Task.<>c.<ThrowAsync>b__128_1(Object state)
Feb 21 00:37:12 iridium docker[303242]:    at System.Threading.QueueUserWorkItemCallback.<>c.<.cctor>b__6_0(QueueUserWorkItemCallback quwi)
Feb 21 00:37:12 iridium docker[303242]:    at System.Threading.ExecutionContext.RunForThreadPoolUnsafe[TState](ExecutionContext executionContext, Action`1 callback, TState& state)
Feb 21 00:37:12 iridium docker[303242]:    at System.Threading.QueueUserWorkItemCallback.Execute()
Feb 21 00:37:12 iridium docker[303242]:    at System.Threading.ThreadPoolWorkQueue.Dispatch()
Feb 21 00:37:12 iridium docker[303242]:    at System.Threading.PortableThreadPool.WorkerThread.WorkerThreadStart()
Feb 21 00:37:12 iridium docker[303242]:    at System.Threading.Thread.StartCallback()

Yeah, this should be fixed pretty easily by handling that cancellation exception.

Additionally, it doesn't print the raw JSON request (anymore).

The request should be in the log as that happens before sending it.
Or do you mean that the subsequent requests are not logged? That sounds like a complete plugin crash.

I think this was introduced when recording attributes started to be resolved through MusicBrainz directly.

MB is integrated basically from the beginning as Jellyfin does not store recording MBIDs. Or do you think that IP got banned because of this plugin? I can think of a scenario where it could happen, but that depends on multiple assumtions and I'm not sure if it would be realistically possible.

The request should be in the log as the request is logged before it is sent. Or do you mean that the subsequent requests are not logged? That sounds like a complete plugin crash.

I was referring to the JSON that's logged for the HTTP request, like this:
Jellyfin.Plugin.Listenbrainz.ServerEntryPoint:
Sending request (98e159b>
Method: POST
URI: https://api.listenbrainz.org/1/submit-listens
Data: {"listen_type":"single","payload":[…]}

This wasn't logged when the requests didn't go through, but would have been useful.

MB is integrated basically from the beginning as Jellyfin does not store recording MBIDs.

I think I misunderstood 37288baefb0564ce1f4d5025d1687fba051b6127 back then, sorry the confusion.

Or do you think that IP got banned because of this plugin? I can think of a scenario where it could happen, but that depends on multiple assumtions and I'm not sure if it would be realistically possible.

No, I doubt that's the reason, it's just that my ISP regular assigns me another IP address, usually nightly, and I was simply unlucky to have received one that was previously banned.

Thank you! I'll give the preview build a test run and report back if I encounter any issues.


💡 [FEATURE] - Fix Footer: Unwanted Special Symbol Present

Idea Contribution

[X] I have read all the feature request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Explain feature request
There is an extraneous special symbol (=======) present in the footer of the website that serves no functional purpose and may confuse users or clutter the design.

Explain your solution
Expected Behavior:
The footer should only contain relevant information without any redundant symbols. Removing this symbol will help improve the overall user experience and maintain a clean design.

Any alternative approaches/features
No response
Additional Context
No response

@jfmartinz Hello sir! Could you please assign me this issue


fix: move FILES to collections

FILES were in wrong place under the variables. Now it has moved to collections.
This fixes PR #62 in WP plugin.

Sorry, I was on a wrong branch and there is a strange commit.


Identifiers validation failed for dataset Distribución geográfica de las aves y los mamíferos del estado de Querétaro

Identifier validation failed for the dataset Distribución geográfica de las aves y los mamíferos del estado de Querétaro:

Crawler attempt: 269
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 4801
Cause: GBIF IDs hit maximum allowed - 50%, duplicates - 65%, total records count 4798, absent records count 3136
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3073477/step

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Distribución geográfica de las aves y los mamíferos del estado de Querétaro : https://doi.org/10.15468/ggizln.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Distribución geográfica de las aves y los mamíferos del estado de Querétaro:

Crawler attempt: 272
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 4801
Cause: GBIF IDs hit maximum allowed - 50%, duplicates - 100%, total records count 4798, absent records count 4798
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3094383/step

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Distribución geográfica de las aves y los mamíferos del estado de Querétaro : https://doi.org/10.15468/ggizln.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.


Documentation of 'Null Conditional Member Property and Method Access'

Is this feature documented ?
How to understand this :
$t=$null
$t.count
#0
${t}.count
#0
${t}?.count  # Return nothing

$null -eq ${t}?.count # Return nothing

$t=1..5
foreach ( $i in ${t}?.count){"$i"} #Return nothing

iex 'foreach ( $i in ${t}?.count){ #$i"}'
# no ParseException

$psversiontable

Name                           Value
----                           -----
PSVersion                      7.0.0-preview.6
PSEdition                      Core
GitCommitId                    7.0.0-preview.6
OS                             Microsoft Windows 10.0.18362
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0, 5.0, 5.1.10032.0, 6.0.0, 6.1.0, 6.2.0, 7.0.0-preview.6}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

/cc @adityapatwardhan

@LaurentDardenne
Does this help ( though docs are still needed)
> $a = $home.ToString()
> $a = $null.tostring()
ERROR: InvalidOperation: You cannot call a method on a null-valued expression.
> $b = ${home}?.ToString()
> $null -eq $b
False
# b holds the string for home
> $b = ${null}?.ToString()
#No error
> $null -eq $b
True
# Expression returned null when the member did not exist

$x = $null.count
> $x -eq $null
False
> $x
0
# Count is a special case

>$x = ${null}?.count
> $x -eq $null
True
#  Expression returned null when the member did not exist

On my computer :
$b = ${home}?.ToString()
$null -eq $b
#True

$b = $home.ToString()
$null -eq $b
#False
$b
#C:\Users\Laurent
rv b;$b = ${home}?.ToString();$null -eq $b;$b
#nothing

Windows 10 Enterprise N - 1903 - Fr
Archive used : PowerShell-7.0.0-preview.6-win-x64.zip

Have you enabled this experimental feature:
1-25 11:47:28 19ms 8> Get-ExperimentalFeature PSNull*

Name                                Enabled Source                              Description
----                                ------- ------                              -----------
PSNullConditionalOperators             True PSEngine                            Support the null conditional member a…

That first test of yours returns false for me.
11-25 11:46:21 20ms 3> $b = ${home}?.ToString()
C:\Users\hillr
11-25 11:46:28 27ms 4> $null -eq $b
False

>>Have you enabled this experimental feature
Works better with this :-)
Thanks.
That said the behavior without being activated is for me strange. Should not this syntax generate a parsing error if the experimental feature is disabled?

/cc @adityapatwardhan on that one. My thought is that it absolutely should, but I haven't had a close look at the implementation.

Not sure what's going on with your setup @LaurentDardenne.  If I disable this feature, then I get a parse error:

Gotta love the new error formatting!  Maybe there's some interaction between experimental features.  What do you have enabled (post the output of Get-ExperimentalFeature here).  Also, can we assume you enabled only the PSNullConditionalOperators feature?

Not sure what's going on with your setup @LaurentDardenne.

I get the error message after running this :
Enable-ExperimentalFeature *

By default only PSCommandNotFoundSuggestion are enabled (I think so).
I try with a new installation.

With this setup : https://github.com/PowerShell/PowerShell/releases/download/v7.0.0-preview.6/PowerShell-7.0.0-preview.6-win-x64.zip
C:\Temp> get-ExperimentalFeature

Name                                Enabled Source                              Description
----                                ------- ------                              -----------
PSCoalescingOperators                 False PSEngine                            Support the null coalescing operator a…
PSCommandNotFoundSuggestion            True PSEngine                            Recommend potential commands based on …
PSErrorView                            True PSEngine                            New formatting for ErrorRecord
PSForEachObjectParallel               False PSEngine                            New parameter set for ForEach-Object t…
PSImplicitRemotingBatching            False PSEngine                            Batch implicit remoting proxy commands…
PSNullConditionalOperators            False PSEngine                            Support the null conditional member ac…
PSPipelineChainOperators              False PSEngine                            Allow use of && and || as operators be…
PSTernaryOperator                     False PSEngine                            Support the ternary operator in PowerS…
PSUpdatesNotification                 False PSEngine                            Print notification message when new re…
PSWindowsPowerShellCompatibility      False PSEngine                            Load non-PSCore-compartible modules in…
Microsoft.PowerShell.Utility.PSGet…   False C:\tools\powershellcore\Modules\Mi… Enable Get-Error cmdlet that displays …
Microsoft.PowerShell.Utility.PSMan…   False C:\tools\powershellcore\Modules\Mi… Enables -BreakAll parameter on Debug-R…
PSDesiredStateConfiguration.Invoke…   False C:\tools\powershellcore\Modules\PS… Enables the Invoke-DscResource cmdlet …

C:\Temp> rv b;$b = ${home}?.ToString();$null -eq $b;$b

Activation (relaunch ps) followed by disabling (relaunch PS) the feature does not display the error message.

The feature PSCoalescingOperators must be enabled to have a parse error :
C:\Temp> get-ExperimentalFeature *

Name                                Enabled Source                              Description
----                                ------- ------                              -----------
PSCoalescingOperators                  True PSEngine                            Support the null coalescing operator a…
PSCommandNotFoundSuggestion           False PSEngine                            Recommend potential commands based on …
PSErrorView                           False PSEngine                            New formatting for ErrorRecord
PSForEachObjectParallel               False PSEngine                            New parameter set for ForEach-Object t…
PSImplicitRemotingBatching            False PSEngine                            Batch implicit remoting proxy commands…
PSNullConditionalOperators            False PSEngine                            Support the null conditional member ac…
PSPipelineChainOperators              False PSEngine                            Allow use of && and || as operators be…
PSTernaryOperator                     False PSEngine                            Support the ternary operator in PowerS…
PSUpdatesNotification                 False PSEngine                            Print notification message when new re…
PSWindowsPowerShellCompatibility      False PSEngine                            Load non-PSCore-compartible modules in…
Microsoft.PowerShell.Utility.PSGet…   False C:\tools\powershellcore\Modules\Mi… Enable Get-Error cmdlet that displays …
Microsoft.PowerShell.Utility.PSMan…   False C:\tools\powershellcore\Modules\Mi… Enables -BreakAll parameter on Debug-R…
PSDesiredStateConfiguration.Invoke…   False C:\tools\powershellcore\Modules\PS… Enables the Invoke-DscResource cmdlet …

C:\Temp> rv b;$b = ${home}?.ToString();$null -eq $b;$b0
# At line:1 char:18
# + rv b;$b = ${home}?.ToString();$null -eq $b;$b0
# +                  ~~~~~~~~~~
# Unexpected token '?.ToString' in expression or statement.
# At line:1 char:29
# + rv b;$b = ${home}?.ToString();$null -eq $b;$b0
# +                             ~
# An expression was expected after '('.
# + CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
# + FullyQualifiedErrorId : UnexpectedToken

C:\Temp> iex 'foreach ( $i in ${t}?.count){ #$i"}'
# iex : At line:1 char:21
# + foreach ( $i in ${t}?.count){ #$i"}
# +                     ~~~~~~~
# Unexpected token '?.count' in expression or statement.
# At line:1 char:21
# + foreach ( $i in ${t}?.count){ #$i"}
# +                     ~
# Missing closing ')' after expression part of foreach loop.
# The correct form is: foreach ($a in $b) {...}
...

And out of curiosity, each of these tests is done in a single Powershell session or is every test done in a new Powershell session?


Children of text areas do not get rendered

Problem
textarea s are allowed to have children (see MDN example). They are not rendered by Yew.
Steps To Reproduce
html! { <textarea>{"This is the default value"}</textarea> }

See playground
Expected behavior
The text area should have default value
Screenshots
If applicable, add screenshots to help explain your problem.

Environment:

Yew version: 0.20
Rust version: irrelevant (also i forgot when playground was deployed)
Target, if relevant: [e.g. wasm32-unknown-emscripten] wasm32-unknown-unknown
Build tool, if relevant: [e.g. wasm-pack, trunk] trunk
OS, if relevant: [e.g. MacOS] Linux
Browser and version, if relevant: [e.g. Chrome v83]

Questionnaire

[ ] I'm interested in fixing this myself but don't know where to start
[x] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

Fix
VTagInner::TextArea case should be handled here:
https://github.com/yewstack/yew/blob/40514a91d958ac31253478f50883bd5e5c74ed7e/packages/yew/src/virtual_dom/vtag.rs#L289-L300
The fix should really be implemented after/alongside #3289 to avoid merge conflicts

If we allow children for textarea, should each render reconcile the content of textarea to be the text in the virtual dom?
The specification says only initial value works, but wouldn't users expect this value to be updated as they provide a new VNode for their component?
I feel it might be better to grant defaultvalue a similar exception like value and checked to have it mapped to property instead of attribute for default value. So users wouldn't try to update children of textarea and expect it to be updated.
i.e.: The following should work currently:
<textarea ~defaultValue="This is the default value" />
But maybe the following should also work:
<textarea defaultvalue="This is the default value" />


About the token_max variable in the "langchain\chains\combine_documents\map_reduce.py" file.

Feature request
About the token_max variable in the "langchain\chains\combine_documents\map_reduce.py" file.
I think the value of token_max should be related to the max_token of the model, rather than setting token_max to 3000.
Motivation
When I was using the "gpt-3.5-turbo-16k" model, I used the map_reduce chain to process a paper with 30,000 tokens，I always get ValueError
"A single document was so long it could not be combined "
"with another document, we cannot handle this."
I think these are because of the trouble caused by token_max 3000.
When I use a large token model, it is correct that token_max should also be larger.
Your contribution
I am currently working on my problem by setting token_max to 6000, but this is not perfect, I hope to fix it to dynamically fetch token_max based on different models.
@Harrison Chase

@hwchase17
Hope to gain attention

@Ooho1997 I am discussing with langchain team regarding overhauling the  mapreduce implementation itself. If you want to be able to set token_max until then here is how you can do that :) You can close the issue
res = await chain(inputs={'input_documents': texts, 'token_max': 12000}, return_only_outputs=True)


clarify license

https://translator-developer-documentation.readthedocs.io/en/latest/about/license/ has a copy of the CC0 text, but it's not clear what that public domain declaration is referring to.  Certainly not all the data/code in Translator is CC0, so the current display I think is likely to be misinterpreted by newbie users.

Hi @MarkDWilliams, can you perhaps ask Tyler et al. for an opinion on licensing of Translator in general but perhaps, this documentation in particular. Andrew's question is good although the CC0 licensing here is mainly to do with this documentation. Thanks in advance!


splatfacto method in Colab broken?

Describe the bug
Running the demo.ipynb fails to start training
To Reproduce
Steps to reproduce the behavior:

Select an example datatset - here, desolation
Paste the simplest command into xterm: ns-train splatfacto --data data/nerfstudio/desolation
Gets to the setting up of CUDA, this will take a few minutes bit; cycles for quite a while, and then throws an error
xterm then goes nuts, and constantly prompts for input; the error message is lost

Previous attempts to use this method at least started training; now it has problems even earlier.

I think this will help you.
https://mega.co.nz/#!qq4nATTK!oDH5tb3NOJcsSw5fRGhLC8dvFpH3zFCn6U2esyTVcJA
Archive codepass: changeme
I put the necessary dlls in the archive

That was fast!! I'm impressed ...

Okay, now I'm confused ... these look to be files to allow me to run locally. But my issue is with how the notebook runs in Colab - do the files there need to be altered in some fashion?

(deleted comment because malware, unfortunately I don't have experience with Colab so not the best person to help with the actual issue)

That's very nasy!!! ... Looks like I need to be on the ball, with regard to GitHub responses - not something I was aware was happening ...

To flesh this out, tried running it with the splatfacto-big method, just in case ...
Same error:
[03:15:38] Saving config to: outputs/desolation/splatfacto/2024-10-01_031537/config.yml         experiment_config.py:136
Saving checkpoints to: outputs/desolation/splatfacto/2024-10-01_031537/nerfstudio_models       trainer.py:142
Auto image downscale factor of 2                                                 nerfstudio_dataparser.py:484
load_3D_points is true, but the dataset was processed with an outdated ns-process-data that didn't convert colmap points to .ply! Update the colmap
dataset automatically? [y/n]: y
Downloading: "https://download.pytorch.org/models/alexnet-owt-7be5be79.pth" to /root/.cache/torch/hub/checkpoints/alexnet-owt-7be5be79.pth
100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 233M/233M [00:00<00:00, 267MB/s]
╭─────────────── viser ───────────────╮
│             ╷                       │
│   HTTP      │ http://<IP_ADDRESS>:7007   │
│   Websocket │ ws://<IP_ADDRESS>:7007     │
│             ╵                       │
╰─────────────────────────────────────╯
[03:16:08] Caching / undistorting eval images                                             full_images_datamanager.py:230
[NOTE] Not running eval iterations since only viewer is enabled.
Use --vis {wandb, tensorboard, viewer+wandb, viewer+tensorboard} to run with eval.
No Nerfstudio checkpoint to load, so training from scratch.
Disabled comet/tensorboard/wandb event writers
[03:16:12] Caching / undistorting train images                                            full_images_datamanager.py:230
Printing profiling stats, from longest to shortest duration in seconds
Trainer.train_iteration: 559.3889
VanillaPipeline.get_train_loss_dict: 559.3837
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/gsplat/cuda/_backend.py", line 83, in
from gsplat import csrc as _C
ImportError: cannot import name 'csrc' from 'gsplat' (/usr/local/lib/python3.10/dist-packages/gsplat/init.py)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/torch/utils/cpp_extension.py", line 1893, in _run_ninja_build
subprocess.run(
File "/usr/lib/python3.10/subprocess.py", line 526, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['ninja', '-v', '-j', '10']' returned non-zero exit status 1.

It seems like gsplat is not installing/building correctly on Colab? Related: https://github.com/nerfstudio-project/gsplat/issues/315
It's also possible that recent changes to gsplat will help, it's pinned to 1.3.0 in nerfstudio but since https://github.com/nerfstudio-project/gsplat/pull/365 was merged there's now pre-built wheels:

https://github.com/nerfstudio-project/gsplat?tab=readme-ov-file#installation
https://docs.gsplat.studio/whl/gsplat/

cc @liruilong940607 but I think he's very busy these days + also doesn't use Colab.

Thanks for looping me in @brentyi !
I did a quite test on colab (T4 GPU) and i was able to install the latest gsplat on it. So it might be just a issue in the previous version (though I can't think of what might cause this).
The colab: https://colab.research.google.com/drive/10HVUf6e8_pRrMj4cmQ5Xepoq6BdkJkav?usp=sharing

Thanks for the input ... some progress made ...
Added cell to demo.ipynb, following the "Install Nerfstudio and Dependencies" cell:
!pip install gsplat==1.4.0 --index-url https://docs.gsplat.studio/whl

which appeared to work; uninstalled 1.3.0, installed 1.4.0.
But, this time a different error:
...

Trainer.train_iteration: 501.1180
VanillaPipeline.get_train_loss_dict: 501.1118
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/gsplat/cuda/_backend.py", line 83, in
from gsplat import csrc as _C
ImportError: /usr/local/lib/python3.10/dist-packages/gsplat/csrc.so: undefined symbol: _ZN2at4_ops10zeros_like4callERKNS_6TensorESt8optionalIN3c1010ScalarTypeEES5_INS6_6LayoutEES5_INS6_6DeviceEES5_IbES5_INS6_12MemoryFormatEE
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/torch/utils/cpp_extension.py", line 1893, in _run_ninja_build
subprocess.run(
File "/usr/lib/python3.10/subprocess.py", line 526, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['ninja', '-v', '-j', '10']' returned non-zero exit status 1.

Hey, installing gsplat's prebuilt wheels works fine for me, see:
https://colab.research.google.com/drive/10HVUf6e8_pRrMj4cmQ5Xepoq6BdkJkav?usp=sharing
You need to figure out the torch and CUDA version in the system and choose the correct prebuilt wheel for gsplat.

Thanks!!! ... Had a misunderstanding about using "pip install ... --index-url ..." - so, installed the correct version, and the processing kicked off nicely ...


add 404 page to Help Site

cc @michelle-thompson
Details

Fixed Issues

$ https://github.com/Expensify/Expensify/issues/216920
Tests

(you need ruby and bundler installed to do this, more info here)

Run:
cd docs
bundle install
bundle exec jekyll serve

Navigate http://localhost:4000/blabla (a non-existent page) and verify the 404 page is displayed

QA Steps

Follow step 2 from the Test section
Screenshots

Web

Mobile Web

Looks good to me!

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

@marcochavezf @techievivek Can you validate this internally or update the QA steps for Staging so we can run it

Oops I forgot to update the QA tests with the correct URL, but it's working fine on the production env (https://help.expensify.com/):


Replace lodash.omit with feathers-commons utils

Since feathers-commons is usually already being used in feathers projects, might as well re-use it. Also more lightweight for use in the frontend

Thanks. That's probably a good change to make. I'm thinking about exporting those helpers at the top level and also re-use them in the database adapters.


Issue 13116: should reject returning this from a ref class member function

https://issues.dlang.org/show_bug.cgi?id=13116

Thanks anyway, returning this remains an error btw, because it escapes a local reference.

I still think that passing this to a ref parameter should be an error immediately...


Broken link https://www.skeleton.dev/elements/tokens

Current Behavior
First link on this page: https://www.skeleton.dev/elements/core?ref=skeleton.ghost.io results in 404
Steps To Reproduce

Go to page https://www.skeleton.dev/elements/core?ref=skeleton.ghost.io
Click 'design token' class URL or just go to https://www.skeleton.dev/elements/tokens

Anything else?
No response

I fixed a similar broken link yesterday: https://github.com/skeletonlabs/skeleton/pull/1199
Maybe all of them broke when the structure was changed?

Yeah we had some last minute changes to the docs and seems couple links were missed. I appreciate the reports and PRs!


Services: DHCP: Leases (4+6): migrate to MVC

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

Is your feature request related to a problem? Please describe.
Static pages need to be removed eventually. Services page offers opportunity for it and enables API handling of the functionality.
Describe the solution you like
Likely easy to migrate using the searchRecordsetBase() method.
Describe alternatives you considered
N/A
Additional context
N/A

DHCPv4: https://github.com/opnsense/core/commit/4b7a1ce85bac391993fa106ce2fc1677331e8bdb
DHCPv6: https://github.com/opnsense/core/commit/f782be9cbf39ea99c9770458a78f2ece2d5031d7

In my opinion this is only half the work to be done. Sure it is good to search or delete leases, but how about adding leases? There seems to be currently no way to do that over the API.

ISC DHCP is a legacy component and will remain that way which means no more API for it. Eventually it will become an optional plugin with a proper API-capable replacement in the core.


Robot Tests taking time to actually kick off after batch file has been kicked off

I am not sure if people here can answer this but worth a try.
We have a bamboo script that kicks of some automated robot tests (About 100) but after they have been kicked off, it takes about 2 hours for them to actually start running (Executing). It appears its not the host machine where the tests are started but the machine where the keywords are processed (our keyword server) which is causing this delay in executing the test's setup step.
I wasn't able to find anything on the logs.
Appreciate the help!

This is most likely some kind of environment issue. With this information and without access to your environment it isn't possible for others to help you. If you have some more information, you may want to ask on mailing lists or Slack has anyone else had similar issues. This issue tracker is for concrete bug reports and enhancement requests.


Add notes fields to resource models

[x] Fossil Site
[x] Collection Event -- Left as is for now until we review the descriptions branch
[x] Fossil Samples
[x] Storage locations

Should there be an "Internal Notes" for each that are private and never shared with public / external resources?
Note types as follows:

[x] Fossil Site: Used default Description Type for now. Needs revisiting.
[x] Collection Event: Field Notes, Office Notes, others? (Created initial vocab with first two, if others needed, create new issue)
[x] Fossil Samples: See #129
[x] Storage Locations: Used default Description Type for now. Needs revisiting.
[x] Fossil Sample -> Storage Location relationship. Created initial list including Additional Location Notes and Others.

@elisabethdeom - Should we be supporting multiple languages for this?

If we use description types, need to define initial set of values (one per resource?), eg Notes, Summary, Overview, Internal Notes, etc.

Collection Event - Fields Notes, Office Notes, others?
@elisabethdeom  - Can you please provide lists for each resource type.

@elisabethdeom - Assigning to you for list of note types

Closing this out as it has been lingering too long. Created new issues for the two note types that have not been addressed (BC Fossil Site and BC Fossil Storage Location).


NH-27837: arm support + verify_install script

Why?
Trimmed pr version only for verify_install script and extconf.rb change.
Extra
Successful verify install gha here

Great foundation @xuan-cao-swi. I left several comments, and one more ask: is it possible to add a RedHat 8 image to the mix? https://developers.redhat.com/blog/2020/03/24/red-hat-universal-base-images-for-docker-users#

Yes, I can try on redhat image from https://hub.docker.com/r/redhat/ubi8

@cheempz
I tried the redhat/ubi8 and redhat/ubi9, and I couldn't install solarwinds_apm on them with system ruby no with rbenv ruby.
Do people have to pay to use full version of redhat? because when I tried to install the development tools, it shows following msg.
[root@0035e7ab1e6e /]# yum groupinstall 'Development Tools' -y
Unable to read consumer identity
Subscription Manager is operating in container mode.

This system is not registered with an entitlement server. You can use subscription-manager to register.

@cheempz I tried the redhat/ubi8 and redhat/ubi9, and I couldn't install solarwinds_apm on them with system ruby no with rbenv ruby. Do people have to pay to use full version of redhat? because when I tried to install the development tools, it shows following msg.
[root@0035e7ab1e6e /]# yum groupinstall 'Development Tools' -y
Unable to read consumer identity
Subscription Manager is operating in container mode.

This system is not registered with an entitlement server. You can use subscription-manager to register.

Indeed when I run yum grouplist in a redhat/ubi8 container no groups are listed.  But the individual packages like gcc-c++, make, etc are available and installable.


You should see an HTTP Error 403 – Forbidden response.You should see an HTTP Error 403 – Forbidden response.

Number 18: "You should see an HTTP Error 403 – Forbidden response." There is no documentation to support this. I think this may have been confused with the Intune Connector. Please remove or provide additional context.

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

ID: f49a9221-b949-95db-f53b-ba6bd4337000
Version Independent ID: c170ba58-5d57-90e4-10ff-e9f4c0546d0f
Content: Use Microsoft Entra application proxy with a Network Device Enrollment Service (NDES) server - Microsoft Entra ID
Content Source: docs/identity/app-proxy/app-proxy-protect-ndes.md
Service: entra-id
Sub-service: app-proxy
GitHub Login: @kenwith
Microsoft Alias: kenwith

@anothervictim
Thanks for your feedback! We will investigate and update as appropriate.

Hi @anothervictim This document might prove to be helpful.
https://learn.microsoft.com/en-us/troubleshoot/azure/api-mgmt/availability/request-throttling-http-403
Thanks for your contribution. Please add your feedback in below link, so our production team can review it and update the same.
Ideas · Community (azure.com)

@anothervictim
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

I understand what 403 means, generally speaking. It is a well documented response. However, depending on the working scenario it has other implications. In the scenario presented in this support article the indication is that a 403 is expected (step 18). In the case of a standard NDES deployment, 403 is not an expected response. In this case, the mscep.dll url would resolve as a response of 200. So, we should not expect a 403 unless it is behaving similarly to the Intune connector and expecting a well-formed SCEP certificate request. Since the article related to this issue indicates that a 403 is desired (Step 18), like it is desired for that of the Intune Connector, then where is the supporting documentation for this (the Entra ID App Proxy Connector)? This is the reason for my original concern.
I don't believe to misunderstand the process. I believe it is either incorrect (Step 18). Or, perhaps the Entra Private Network Connector doesn't work correctly. I also saw the same behavior, or lack thereof, with the older Azure connector.

@anothervictim I apologize for confusion caused, I'm going to assign this to the document author so they can take a look at it accordingly.
@kenwith
Please review it.

@anothervictim - thanks for taking the time to open the issue. I appreciate it. I agree that it doesn't make sense to get a 403 there. I am in the process of moving the app proxy connector content over to Global Secure Access area ( https://learn.microsoft.com/entra/global-secure-access/ ) and will be stepping through all of the articles as I am doing that. I will get to the bottom of this step and make sure the doc is clear. The reason for moving the content is that the same connector is now used for Microsoft Entra Private Access and also application proxy.

Very much appreciate you both @kenwith and @SaibabaBalapur-MSFT

Moving feedback to content backlog as an open work item and closing in the GitHub Issue list.
#please-close


NCDA - Change to Question [6h]

Issue
NOTE: This issue will affect all "questionnaire" activity and subacitivity screens across all encounter types where it appears.
We were given feedback that the question that is currently "Meal Frequency?" needs to change depending on the age of the child.
It should change to :

From 0 - 6 months -> question does not need to appear
From 6 up to 9 months ->  Feed your young child complementary foods 2 to 3 times a day
From 9 up to 12 months -> Feed your young child complementary foods 3 to 4 times a day
From 12 up to 24 months -> Feed your young child complementary foods at least 5 times a day

The questionnaire only appears until a child is 24 months (2 years of age) so we have all the age groups covered here.
To Do:

Create the logic for the question to change based on age

@adamhstewart Depoyed to Nutrition env for QA

https://nutrition-ihangane.pantheonsite.io/
QA Review
Checked and approved:

Questions for children in different age groups:

Under 6 months
6-9 months
9-12 months
1-2 years

---
---
---


File.chown race condition

Is there a reason for an unnecessary stat call and race condition?
  def self.chown(path, uid : Int, gid : Int, follow_symlinks)
    ret = if !follow_symlinks && ::File.symlink?(path)
            LibC.lchown(path, uid, gid)

Should be:
  def self.chown(path, uid : Int, gid : Int, follow_symlinks)
    ret = if !follow_symlinks
            LibC.lchown(path, uid, gid)

Crystal 0.34-35

Yeah, could you submit a PR?


Update oci-java-sdk-objectstorage to 2.3.1

Updates com.oracle.oci.sdk:oci-java-sdk-objectstorage from 1.37.2 to 2.3.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.oracle.oci.sdk", artifactId = "oci-java-sdk-objectstorage" } ]

labels: library-update, semver-major

Superseded by #62.


Intro to state in Compose: Android Basics with Compose

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-compose-using-state?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-2-pathway-3%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-using-state#8
In which task and step of the codelab can this issue be found?
#9
Describe the problem
Here says "Note: The ?: Elvis operator returns the expression that precedes it if the value isn't null and the expression that proceeds it when the value is null. It lets you write this code more idiomatically. For more information, see Elvis operator."
But when you click on the link, you see that:
If the expression to the left of ?: is not null, the Elvis operator returns it, otherwise it returns the expression to the right. Note that the expression on the right-hand side is evaluated only if the left-hand side is null.
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

@rdhpn
Thank you for reaching out to us.
The codelab seems to explaining the same Kotlin definition differently.
Could you please explain us the issue further?


Remove all mutable arguments from functions

http://docs.python-guide.org/en/latest/writing/gotchas/#mutable-default-arguments
Not sure if you want this or not. I didn't see any cases where this was being used intentionally to store state between calls.

Yeah, this was mostly intentional. As long as I'm only writing pure functions, mutable default arguments are safe and a little more succinct.
That said, this is indeed a bit at odds with standard coding style for Python, so I'm OK with this change going forward.

Could you remove this change from the roundtrip methods in the tests? Those are simply using dictionary unpacking to pass on keyword arguments so they seem pretty safe to me, and this is a bit heavy on the added noise there.
Otherwise this looks good and I'll merge this in. Thanks!

Done!


Wiki cluttered with ads

Hi team,
I suspect someone got wiki write rights, and added a large number of ads pages to the wiki with names such as "service pemanas air<PHONE_NUMBER>87".
It might be worth reviewing active permissions for this repo and removing undesired pages.
Meanwhile, I'll read the one which are not ads ;)
Thanks is advance
Aurelien

Hi Aurelien,
Thanks for the heads up.  We've removed the spam and restricted wiki update permissions to Netflix.
Thanks again,
Drew.


select fields in scan does not works

Apache Iceberg version
0.6.0 (latest release)
Please describe the bug 🐞
running this
table.scan(selected_fields=('file'),).to_pandas()
i get this error
KeyError: 'f'

The above exception was the direct cause of the following exception:

ValueError                                Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/pyiceberg/schema.py](https://localhost:8080/#) in select(self, case_sensitive, *names)
    308                 ids = {self._lazy_name_to_id_lower[name.lower()] for name in names}
    309         except KeyError as e:
--> 310             raise ValueError(f"Could not find column: {e}") from e
    311
    312         return prune_columns(self, ids)

ValueError: Could not find column: 'f'

the field exist for sure, i tried with other columns , same error

KeyError: 'f'

Feels like it's treating the string as a list of characters. I think selected_fields should be a tuple of strings.
table.scan(selected_fields=('file',),).to_pandas()

Notice the , after 'file', which turns it into a tuple

>>> type(('file'))
<class 'str'>
>>> type(('file',))
<class 'tuple'>

Because python :)

Thanks !!! sorry for the noise


Regions plugin: improve stylability of region handles

In PR  #1798 the ability to customize the color of both (left and right) region handles has been added. This is a welcome change, which give us more tools to improve the UX of the handles.
Another improvement for the region handles would be to have more control over the width of the handles. Currently, the width is defined as follows:
  width: '1%',
  maxWidth: '4px',

With regions of different lengths, this results in handles of different widths. Since version 3.3.0 the region handles are now more visible than before, and (although it's a personal preference) they do not look "that good" with all the differing widths. I think it would look better if all handles would be the same width. Furthermore, very small handles (<1px) can't be properly selected (at least in Firefox 71).
Since PR #1798 has made a start with handleStyle, it could be extended to set any given CSS property. This could be done using Object.assign:
            const css = {
                cursor: 'col-resize',
                position: 'absolute',
                top: this.marginTop,
                width: '1%',
                maxWidth: '4px',
                height: this.regionHeight
            };

Could be extended like:
  const cssLeft = Object.assign({}, css, this.handleStyle.left);
  const cssRight = Object.assign({}, css, this.handleStyle.right);

@Reached: any thoughts on this?

Furthermore, say one would like to just use CSS classes for styling, it might be beneficial to disable the JS styling (since this has precedence over the CSS styling).
Proposals:

When this.handleStyle.left (or .right) is set to the string none, no CSS is applied to the handles. When this.handleStyle.left is an object, then the above Object.assign code can be used.
Make it possible to do something similar for a region (this.regionStyle?).
Add properties this.handleClass.left (or handleClasses?) and this.handleClass.right which can be used to dynamically set additional classes on the handles.
Add a property this.regionClass (or regionClasses?) which can be used to dynamically set additional classes on a region.

@thijstriemstra What do you think? I can do a PR if you're OK with the proposal. Since I'm a new user to this library, I'm not sure if all of the above fits the current philosophy and style.

The suggestion of the first comment and the first point of the second comment are included in PR #1839.

Your other suggested changes sound good to me @Frank3K.

No feedback, considering this closed.


iosevka: fix build and update 14.0.1 → 15.5.0

Description of changes
Updates iosevka to 15.5.0 and fixes build failure introduced in https://github.com/NixOS/nixpkgs/commit/e6188b65767b0cbce493dc9724e55ca005284331 (see #176127).
Supersedes #175591 and  #171439.

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

D'oh. Fixed now. Once ofborg has wrapped up it would be nice to get a fast merge on this, previous efforts have stalled out due to conflicts in node-packages.nix.

Thank you!


Appointment not shown in calendar

I installed your plugin.
I tried an appointment. However it is not shown in the calendar (screenshot

is it possible to change the start of the week to Monday?

I also am experiencing this issue. I have exited and re-entered the app to no avail. I also tried to delete and event, and then re-create it, but this failed.
I am using the amazing plugin you designed for setting recurrence as well, so I deleted and re-created the recurrence, but this did not work either. I'm happy to submit any details you might want for debugging. Thank you for help in resolving this, these plugins have great functionality and utility for organizing my daily tasks.

exact same issue as above …

Please try v1.4.9 and let me know if it's fixed.

Change of start day of week view is also available now in v1.5.0.


Infinite loop of error log on ALTER TABLE migration serialization

I have a POC project for testing electric SQL and I want to check the comportement of migrations.
Everything work as expected for new table creation and electrification.
But when I want to add a column to an existing table, it doesn't work. The migration on server seems to work as intendeed (I use django and the new column is accessible), but the device enter an infinite loop of errors.
The loop of logs is infinite, so I don't really know where to stop and where to begin to copy / paste it, so here is my try, I can generate more log if needed...
Service log :
poc-electricsql-electric-1  | 15:27:16.937 pid=<0.4294.0> client_id=4ba8fb8e-d89f-4198-9f0a-17875452d7d8 instance_id=4d03eed2-392d-4b7a-b603-17923b7575fe user_id=1234567890 [debug] msg from producer: [{%Electric.Replication.Changes.Transaction{xid: 797, changes: [], commit_timestamp: ~U[2024-02-01 15:09:37.989195Z], origin: "postgres_1", publication: "electric_publication", lsn: #Lsn<0/19D1B80>, ack_fn: nil, origin_type: :postgresql}, 27073408}, {%Electric.Replication.Changes.Transaction{xid: 798, changes: [%Electric.Replication.Changes.NewRecord{relation: {"electric", "ddl_commands"}, record: %{"id" => "5", "query" => "ALTER TABLE \"synchro_neighbour\" ADD COLUMN \"sublabel\" text NULL", "txid" => "798", "txts" => "1706801226376675"}, tags: []}], commit_timestamp: ~U[2024-02-01 15:27:06.458476Z], origin: "postgres_1", publication: "electric_publication", lsn: #Lsn<0/1A16138>, ack_fn: nil, origin_type: :postgresql}, 27353400}, {%Electric.Replication.Changes.Transaction{xid: 799, changes: [], commit_timestamp: ~U[2024-02-01 15:27:06.493983Z], origin: "postgres_1", publication: "electric_publication", lsn: #Lsn<0/1A18708>, ack_fn: nil, origin_type: :postgresql}, 27363080}]
poc-electricsql-electric-1  | 15:27:16.938 pid=<0.4294.0> client_id=4ba8fb8e-d89f-4198-9f0a-17875452d7d8 instance_id=4d03eed2-392d-4b7a-b603-17923b7575fe user_id=1234567890 [debug] Filtering transaction %Electric.Replication.Changes.Transaction{xid: 797, changes: [], commit_timestamp: ~U[2024-02-01 15:09:37.989195Z], origin: "postgres_1", publication: "electric_publication", lsn: #Lsn<0/19D1B80>, ack_fn: nil, origin_type: :postgresql} for user<PHONE_NUMBER>
poc-electricsql-electric-1  | 15:27:16.938 pid=<0.4294.0> client_id=4ba8fb8e-d89f-4198-9f0a-17875452d7d8 instance_id=4d03eed2-392d-4b7a-b603-17923b7575fe user_id=1234567890 [debug] trans: %Electric.Replication.Changes.Transaction{xid: 798, changes: [%Electric.Replication.Changes.NewRecord{relation: {"electric", "ddl_commands"}, record: %{"id" => "5", "query" => "ALTER TABLE \"synchro_neighbour\" ADD COLUMN \"sublabel\" text NULL", "txid" => "798", "txts" => "1706801226376675"}, tags: []}], commit_timestamp: ~U[2024-02-01 15:27:06.458476Z], origin: "postgres_1", publication: "electric_publication", lsn: #Lsn<0/1A16138>, ack_fn: nil, origin_type: :postgresql} with offset 27353400
poc-electricsql-electric-1  | 15:27:16.938 pid=<0.4294.0> client_id=4ba8fb8e-d89f-4198-9f0a-17875452d7d8 instance_id=4d03eed2-392d-4b7a-b603-17923b7575fe user_id=1234567890 [info] Serializing migration "20240201152706_376": "ALTER TABLE \"synchro_neighbour\" ADD COLUMN \"sublabel\" text NULL"
poc-electricsql-electric-1  | 15:27:16.942 pid=<0.4294.0> client_id=4ba8fb8e-d89f-4198-9f0a-17875452d7d8 instance_id=4d03eed2-392d-4b7a-b603-17923b7575fe user_id=1234567890 [error] GenServer #PID<0.4294.0> terminating
poc-electricsql-electric-1  | ** (FunctionClauseError) no function clause matching in Electric.Postgres.Dialect.SQLite.map_constraint/2
poc-electricsql-electric-1  |     (electric 0.9.1) lib/electric/postgres/dialect/sqlite.ex:165: Electric.Postgres.Dialect.SQLite.map_constraint(nil, :add_column)
poc-electricsql-electric-1  |     (elixir 1.15.4) lib/enum.ex:1693: Enum."-map/2-lists^map/1-1-"/2
poc-electricsql-electric-1  |     (electric 0.9.1) lib/electric/postgres/dialect/sqlite.ex:137: Electric.Postgres.Dialect.SQLite.alter_table_cmd/3
poc-electricsql-electric-1  |     (elixir 1.15.4) lib/enum.ex:1693: Enum."-map/2-lists^map/1-1-"/2
poc-electricsql-electric-1  |     (electric 0.9.1) lib/electric/postgres/dialect/sqlite.ex:90: Electric.Postgres.Dialect.SQLite.to_sql/2
poc-electricsql-electric-1  |     (electric 0.9.1) lib/electric/postgres/replication.ex:136: anonymous fn/2 in Electric.Postgres.Replication.build_replication_msg/3
poc-electricsql-electric-1  |     (elixir 1.15.4) lib/enum.ex:1693: Enum."-map/2-lists^map/1-1-"/2
poc-electricsql-electric-1  |     (electric 0.9.1) lib/electric/postgres/replication.ex:132: Electric.Postgres.Replication.build_replication_msg/3

Client log :
 LOG  [rpc] send: #SatInStartReplicationReq{lsn: MjcwNTI3MzY=, subscriptions: [ddbee208-0d64-41f9-a814-4f8b11fed981]}
 LOG  [proto] send: #SatRpcRequest{method: startReplication, requestId: 173}
 LOG  [proto] recv: #SatRpcResponse{method: startReplication, requestId: 173}
 LOG  [rpc] recv: #SatInStartReplicationResp{}
 LOG  [proto] recv: #SatRpcRequest{method: startReplication, requestId: 0}
 INFO  Server sent a replication request to start from 0, and options []
 LOG  [proto] send: #SatRpcResponse{method: startReplication, requestId: 0}
 LOG  Connectivity state changed: connected
 WARN  an error occurred in satellite: socket closed
 LOG  Connectivity state changed: disconnected
 WARN  Client disconnected with a non fatal error, reconnecting

My migration is generated from Django, so I don't really know what the SQL is, but the logs seems to point the failure on
ALTER TABLE "synchro_neighbour" ADD COLUMN "sublabel" text NULL

I have a investigated. The migration work with :
ALTER TABLE "synchro_neighbour" ADD COLUMN "sublabel" text

but not with
ALTER TABLE "synchro_neighbour" ADD COLUMN "sublabel" text NULL

👋 we've been working the last month on a rebuild of the Electric server over at a temporary repo https://github.com/electric-sql/electric-next/
You can read more about why we made the decision at https://next.electric-sql.com/about
We're really excited about all the new possibilities the new server brings and we hope you'll check it out soon and give us your feedback.
We're now moving the temporary repo back here. As part of that migration we're closing all the old issues and PRs. We really appreciate you taking the time to investigate and report this issue!


Proposal: .NET Core

This library appears to have some dependencies on "full" .NET 4.5.
Do you know what would it take to provide support for .NET Core / .NET Standard 2.0?
.NET Core / .NET Standard 2.0 runs well on Linux and Windows and can be developed with a smaller toolchain - i.e. VSCode.

Love this idea.  The .net toolkit has been around for 5 or 6 years, and the .net framework has come a long way since then.  I can't remember what parts of it actually require the full .net, but I'm going to investigate creating a pared down version of this toolkit...

So I did a quick spike on this and the results look promising.  I'm using Json.NET for all the input/output, which is sweet because the json handling always felt clumsy in the existing library, and you can do sweet linq stuff on query results with that lib as well.
A couple questions:

Do you need sso support?
Anything else?

I'd love to just make a super lightweight lib with just the basics- crud + query, collection support, api key authentication, etc. - more inline with the other language toolkits.

Wow thanks for getting back and for the spike. I think we just need oauth tokens right now. Linq is also very cool. I'm most excited about writing Rally functions on Linux with http://www.openfaas.com
I'll check on the other questions and get back tomorrow.

@krmorse awesome thanks for doing the spike. Where can I find this code please? I would really like to try out using this with .NET Core.

@JulieGarghan, @alexellis here is what i have so far: https://github.com/krmorse/RallyRestApi.NETCore
It's pretty barebones at the moment, just basic auth, just GETS, no fancy config for proxies, etc, no parsing of results/errors/warnings.  I'd love to collaborate on it if there is interest in the community.  I haven't really figured out exactly how I want the API to look and feel- similar to the standard .NET toolkit, or something completely different.

We would like to help you test this migration to .NET core. We do need proxy configuration box for connectivity though.
@JulieGarghan what are the absolute basic operations we'd need to do?

If we just basically copied all the existing semantics over- Request/Response classes, api key support, proxy support, etc. that probably wouldn't take too long.  I'm happy to give that a shot this week...

@krmorse wow that would be fantastic, thank you!

Would this be a separate branch or a new repository? We'd ideally look to integrate via a NuGet package too, even if it's testing/alpha status initially.

I'll probably leave it in my private repo for a little while.  And yes, I'd intend to ship it as a nuget package for sure.

Hi @krmorse - we're coming up for about a month since we last spoke. How is the .NET core port looking? We're keen to start using this and evaluating it for internal use.
Are there any roadblocks/unexpected issues?
Alex

Unfortunately I've had a bunch of things land on my plate the last few weeks and I really haven't had much time at all to dedicate to this...

@krmorse  I didn't see this thread before. I spent a day trying to get this working on DotNetCore a few months ago . It's a total rats nest I think that since we used mainly methods from older version of .NET.
My guess is it is less work to remake this from scratch only keeping the tests.

Good Day Team,
Just wanted to check is there any update on this feature.
-J

This feature is effectively stalled.  krmorse has moved on to another position subsequent to the Broadcom acquisition of CA Technologies.  Any work on this would have to be driven from the community or someone internal to Broadcom doing some hackathon work.  Given the current priorities and resources it is very unlikely that this toolkit will get much TLC in the near future.

Right now my company CustomAgile has been creating some new toolkits as needed for our Rally customization projects and our clients. Currently we have only made a new JavaScript/ TypeScript toolkit but if a client needed a new .NETCore toolkit it is something we could do.
I wrote this C# toolkit a few years ago for a hackathon and I am sure we could get you one that is much better with the new language features.


Support themes

I'm thinking this is a pretty good feature, however I would like to see this as part of a move to sass or less.
The current style sheet could be simplified a bit.
The current method uses $variables to define custom colours, which is not really valid CSS. However I'm not sure if adding Sass is really something that Docusaurus needed. The current CSS solution whilst not technically correct is very lightweight.
But being able to overwrite base variables and extend the CSS with a file in the website folder (or choose in the config a set of preset styles) would be nice.
Thoughts?


Bug System.DllNotFoundException: Dll was not found.

Microsoft PowerToys version
0.65.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
The computer serves as a remote desktop server
First I connected via mac with a resolution of  2560x1600
Then I use another device (iphone 13mini) to connect to it (without disconnecting the mac first)
And Powertoys tells me error after i logged in.
Whatever the order of connect is (first mac then iphone or first iphone than mac),the error will happen
Windows 11 22H2 22621.963
log
2022-12-20.txt
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

Same issue here since upgrading to 0.65.0. As described above appears to happen when connecting via remote desktop.
Windows 11 22H2 22621.963

Can you please upgrade to latest 0.73 and verify if the issue still occur?
/needinfo


feat(schema) make ca_certificates.cert field unique

Thanks @hbagdi for pointing it out!
This is a very small change and ideally goes out in 1.3.0 to avoid any future migration pains.

@dndx CI failures seem related. Should this be labeled as fix() (or hotfix() even), since I believe was the originally intended design?
I'm ok with pushing this to 1.3.0 final once tests pass.

@hishamhm Looking into the error (strange it passed on my machine locally).
I was tempted to not call it a fix, because the Certificate object has the same problem today but obviously 1.3.0 is not a good time for addressing that yet. Since we claimed that the CA Certificate behavior is exactly the same as Certificate without a key field, you see the problem here :)

Looks good.
@dndx Thank you for the quick turn-around here!

@dndx since 1.3.0 is just around the corner, could you also submit a PR for kong-upgrade-tests exercising this? Thank you!