Solution build fails with zero errors

Build a solution can cause MSBuild to succeed on all projects build still fail. When it fails, it shows zero errors and no additional information about what went wrong.
Repro:
https://github.com/aspnet/JavaScriptServices/tree/d8128ee3dcb687a854ee509d62d255c1eba48723
dotnet restore JavaScriptServices.sln
dotnet build JavaScriptServices.sln
Output
dotnet build
Microsoft (R) Build Engine version 15.1.548.43366
Copyright (C) Microsoft Corporation. All rights reserved.

  Microsoft.AspNetCore.NodeServices -> C:\dev\Universe\JavaScriptServices\src\Microsoft.AspNetCore.NodeServices\bin\Debug\net451\Microsoft.AspNetCore.NodeServices.dll
  Microsoft.AspNetCore.NodeServices -> C:\dev\Universe\JavaScriptServices\src\Microsoft.AspNetCore.NodeServices\bin\Debug\netstandard1.6\Microsoft.AspNetCore.NodeServices.dll
  Microsoft.AspNetCore.SpaServices -> C:\dev\Universe\JavaScriptServices\src\Microsoft.AspNetCore.SpaServices\bin\Debug\net451\Microsoft.AspNetCore.SpaServices.dll
  KnockoutSpa -> C:\dev\Universe\JavaScriptServices\templates\KnockoutSpa\bin\Debug\netcoreapp1.1\KnockoutSpa.dll
  ReactSpa -> C:\dev\Universe\JavaScriptServices\templates\ReactSpa\bin\Debug\netcoreapp1.1\ReactSpa.dll
  AureliaSpa -> C:\dev\Universe\JavaScriptServices\templates\AureliaSpa\bin\Debug\netcoreapp1.1\AureliaSpa.dll
  Angular2Spa -> C:\dev\Universe\JavaScriptServices\templates\Angular2Spa\bin\Debug\netcoreapp1.1\Angular2Spa.dll
  WebApplicationBasic -> C:\dev\Universe\JavaScriptServices\templates\WebApplicationBasic\bin\Debug\netcoreapp1.1\WebApplicationBasic.dll
  ReactReduxSpa -> C:\dev\Universe\JavaScriptServices\templates\ReactReduxSpa\bin\Debug\netcoreapp1.1\ReactReduxSpa.dll
  Microsoft.AspNetCore.NodeServices.Sockets -> C:\dev\Universe\JavaScriptServices\src\Microsoft.AspNetCore.NodeServices.Sockets\bin\Debug\net451\Microsoft.AspNetCore.NodeServices.Sockets.dll
  NodeServicesExamples -> C:\dev\Universe\JavaScriptServices\samples\misc\NodeServicesExamples\bin\Debug\netcoreapp1.1\NodeServicesExamples.dll
  Microsoft.AspNetCore.SpaServices -> C:\dev\Universe\JavaScriptServices\src\Microsoft.AspNetCore.SpaServices\bin\Debug\netstandard1.6\Microsoft.AspNetCore.SpaServices.dll
  Microsoft.AspNetCore.NodeServices.Sockets -> C:\dev\Universe\JavaScriptServices\src\Microsoft.AspNetCore.NodeServices.Sockets\bin\Debug\netstandard1.6\Microsoft.AspNetCore.NodeServices.Sockets.dll
  Microsoft.AspNetCore.AngularServices -> C:\dev\Universe\JavaScriptServices\src\Microsoft.AspNetCore.AngularServices\bin\Debug\net451\Microsoft.AspNetCore.AngularServices.dll
  Microsoft.AspNetCore.ReactServices -> C:\dev\Universe\JavaScriptServices\src\Microsoft.AspNetCore.ReactServices\bin\Debug\net451\Microsoft.AspNetCore.ReactServices.dll
  LatencyTest -> C:\dev\Universe\JavaScriptServices\samples\misc\LatencyTest\bin\Debug\netcoreapp1.1\LatencyTest.dll
  Microsoft.AspNetCore.ReactServices -> C:\dev\Universe\JavaScriptServices\src\Microsoft.AspNetCore.ReactServices\bin\Debug\netstandard1.6\Microsoft.AspNetCore.ReactServices.dll
  Microsoft.AspNetCore.AngularServices -> C:\dev\Universe\JavaScriptServices\src\Microsoft.AspNetCore.AngularServices\bin\Debug\netstandard1.6\Microsoft.AspNetCore.AngularServices.dll
  ReactGrid -> C:\dev\Universe\JavaScriptServices\samples\react\ReactGrid\bin\Debug\netcoreapp1.1\ReactGrid.dll
  MusicStore -> C:\dev\Universe\JavaScriptServices\samples\react\MusicStore\bin\Debug\netcoreapp1.1\MusicStore.dll
  MusicStore -> C:\dev\Universe\JavaScriptServices\samples\angular\MusicStore\bin\Debug\netcoreapp1.1\MusicStore.dll
Controllers\FullPagePrerenderingController.cs(16,13): warning CS0618: 'JavaScriptModuleExport.WebpackConfig' is obsolete: 'Do not use. This feature will be removed. Instead, configure Webpack to build a Node.js-compatible bundle and reference that directly.' [C:\dev\Universe\JavaScriptServices\samples\misc\Webpack\Webpack.csproj]
  Webpack -> C:\dev\Universe\JavaScriptServices\samples\misc\Webpack\bin\Debug\netcoreapp1.1\Webpack.dll

Build FAILED.

Controllers\FullPagePrerenderingController.cs(16,13): warning CS0618: 'JavaScriptModuleExport.WebpackConfig' is obsolete: 'Do not use. This feature will be removed. Instead, configure Webpack to build a Node.js-compatible bundle and reference that directly.' [C:\dev\Universe\JavaScriptServices\samples\misc\Webpack\Webpack.csproj]
    1 Warning(s)
    0 Error(s)

Time Elapsed 00:00:07.24

Returns with error code 1.
The warning does not cause the build to fail. Disabling this CSC warning and recompiling the solution still causes the same zero-error failure.
Increasing verbosity does not show additional information about what failed.
Details
dotnet.exe: 1.0.0-rc4-004911
Does not happen in MSBuild.exe
cc @SteveSandersonMS

BTW we suspect that this is broken because the SLN contains Web Site Projects, and that dotnet build gets confused with it, but of course msbuild.exe has no problem with it.

Glad to have helped find an edge case :)
@Eilon, @natemcmaster The Web Site Projects are an unfortunate throwback, only there because it doesn't seem there's any modern alternative. But TBH we can just remove them from the .sln to unblock ourselves - it's perfectly fine to be using VS Code for all the non-.NET projects.

@SteveSandersonMS if you're OK with that, that's fine with me. An alternative that @natemcmaster suggested was to just build the csproj's by globbing for them, and thus avoid building the SLN until this issue is resolved.

Web Site Projects sounds entirely plausible as a reason. They are treated specially and I bet we didn't have any test coverage on them when porting to Core. Of course, even if we don't support them in .NET Core MSBuild we should at least have an error.

I can confirm it's the Web Sites Projects that are causing the issue. After removing them the solution builds fine.

@rainersigwald yeah at the very least there should be an error stating that there's some unknown entries in the SLN. I don't think I'd expect dotnet build to actually support building Web Site Projects because that's a whole world unto itself. Or, at least, that would have to be a new feature request and treated independently of this issue.

There was test coverage of them in the "trun suites" that MSBuild used to have, but not much in the unit tests. I believe the "trun suite" coverage was not ported to github.

The easiest fix is to just remove the website project from build configuration of solution file.

This is stale and we found a solution for my problem anyways.


update readme file with new namespaces and versions

This is now done. The README could still use some improvement, but the immediate need to address ryanj namespacing and version numbers has been dealt with.


Error: Missing "./macros" specifier in "vue" package

Describe the bug
I was going a little crazy trying to investigate the following error:
error when starting dev server:
Error: Missing "./macros" specifier in "vue" package
    at e (file:///Users/araminfar/Workspace/dozzle/node_modules/.pnpm/vite@5.3.3_@types+node@20.14.10/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:46568:25)
    at n (file:///Users/araminfar/Workspace/dozzle/node_modules/.pnpm/vite@5.3.3_@types+node@20.14.10/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:46568:627)
    at o (file:///Users/araminfar/Workspace/dozzle/node_modules/.pnpm/vite@5.3.3_@types+node@20.14.10/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:46568:1297)
    at resolveExportsOrImports (file:///Users/araminfar/Workspace/dozzle/node_modules/.pnpm/vite@5.3.3_@types+node@20.14.10/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:47189:18)
    at resolveDeepImport (file:///Users/araminfar/Workspace/dozzle/node_modules/.pnpm/vite@5.3.3_@types+node@20.14.10/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:47212:25)
    at tryNodeResolve (file:///Users/araminfar/Workspace/dozzle/node_modules/.pnpm/vite@5.3.3_@types+node@20.14.10/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:46977:16)
    at ResolveIdContext.resolveId (file:///Users/araminfar/Workspace/dozzle/node_modules/.pnpm/vite@5.3.3_@types+node@20.14.10/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:46727:19)
    at PluginContainer.resolveId (file:///Users/araminfar/Workspace/dozzle/node_modules/.pnpm/vite@5.3.3_@types+node@20.14.10/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:49506:17)
    at async file:///Users/araminfar/Workspace/dozzle/node_modules/.pnpm/vite@5.3.3_@types+node@20.14.10/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:66561:15
    at async file:///Users/araminfar/Workspace/dozzle/node_modules/.pnpm/vite@5.3.3_@types+node@20.14.10/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:50411:14
 ELIFECYCLE  Command failed with exit code 1.

After downgrading from 0.18 --> 0.17.6, it started working again.
I am trying to go through the changelog to see what could have caused this.
Reproduction
Still trying to figure out what is different with my project that is causing this
System Info
System:
    OS: macOS 14.5
    CPU: (12) arm64 Apple M3 Pro
    Memory: 94.98 MB / 18.00 GB
    Shell: 3.7.1 - /opt/homebrew/bin/fish
  Binaries:
    Node: 22.4.1 - ~/.local/state/fnm_multishells/36228_1721056518990/bin/node
    npm: 10.8.1 - ~/.local/state/fnm_multishells/36228_1721056518990/bin/npm
    pnpm: 9.5.0 - ~/.local/state/fnm_multishells/36228_1721056518990/bin/pnpm
    bun: 1.1.18 - /opt/homebrew/bin/bun
  Browsers:
    Chrome: 126.0.6478.127
    Safari: 17.5

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Removing vue/macros, from AutoImport seems to fix the problem. Not sure if this is to be expected?

I guess you can remove vue/macros as it should no longer needed. Otherwise you can set viteOptimizeDeps to false


Bug report: readtheDocs missing figures

Bug description
<In the readthedocs documentation the figures are missing. Precisely in section 4.3.2 of the documentation which describes Input Files of the Aerodyn.>
To Reproduce

Steps to reproduce the behavior:

Compile with '...'
Run '...' case with '...' settings
Open '...' output
See the error

Expected behavior

Screenshots, if applicable

OpenFAST Version

**************************************************************************************************
 OpenFAST

 Copyright (C)  National Renewable Energy Laboratory
 Copyright (C)  Envision Energy USA LTD

 This program is licensed under Apache License Version 2.0 and comes with ABSOLUTELY NO WARRANTY.
 See the "LICENSE" file distributed with this software for details.
 **************************************************************************************************

 OpenFAST-v2.0.0
 Compile Info:
  - Architecture: 64 bit
  - Precision: double
  - Date: Nov 27 2018
  - Time: 17:19:38
 Execution Info:
  - Date: 11/29/2018
  - Time: 10:52:28-0700

System Information (please complete the following information):

OS: <e.g. Ubuntu 14.04 or macOS 10.12>
Compiler: <e.g. GFortran 4.4>
Compiler settings: <e.g. CMake flags or other settings>

Additional context

@abksr Do you have an ad blocker enabled in your browser? If so, disable it and reload the page. The AeroDyn figures are sometimes blocked because they're prepended with "ad_".

@andrew-platt and @rafmudaf, it was the adBlock causing the figures to not show up. Also, ignore the fast version information, it was left from the template. I was talking about the online documentation in readthedocs.

Great to hear that worked. I'm keeping this open since it is kind of a bug - or at least something that should be handled.


Styling just 1 control button

Hi,
I created a custom button and I would like to apply some CSS just on it. When clicking the element in the browser to find out its class, I realise that all the buttons have the same class leaflet-buttons-control-button. Is there a way to select/access one single button to apply some CSS?
Here my button:
mymap.pm.Toolbar.copyDrawControl('Rectangle', {
                  name: 'RectangleCopy',
                  block: 'custom',
                  onClick: () => {
                  //Whatever here
                  }
                });

Many thanks.

You can add a className:
mymap.pm.Toolbar.copyDrawControl('Rectangle', {
                  name: 'RectangleCopy',
                  block: 'custom',
                  className: 'myCustomCSSClass',
                  onClick: () => {
                  //Whatever here
                  }
                });

Thanks Falke-Design, but that class just seems to be modifying the icon inside the button not the button itself. I would like to make the button bigger.
Look the screenshot below (I changed the colour to beige with the new class)

Many thanks

Hi,
I do not need this capability anymore.
Many thanks.


API calls - issue

Hi Alon,
Thank you for the project!
Any chance to update tami4edge for Home Assistant?
➜  ~ docker pull 0xalon/tami4edgecloudtokenextractor:latest
latest: Pulling from 0xalon/tami4edgecloudtokenextractor
Digest: sha256:1d0e17f6141216430e3c8ac3c73c8bb711b0cd78e8a993f6644a20b34903c90a
Status: Image is up to date for 0xalon/tami4edgecloudtokenextractor:latest
docker.io/0xalon/tami4edgecloudtokenextractor:latest
➜  ~ docker run -it 0xalon/tami4edgecloudtokenextractor:latest
/usr/local/lib/python3.8/site-packages/pydub/utils.py:170: RuntimeWarning: Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work
warn("Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work", RuntimeWarning)
What's your phone number to sign up? <+972XXXXXXXXX>: xxxxxxxxx
OTP: XXXX
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/requests/models.py", line 971, in json
return complexjson.loads(self.text, **kwargs)
File "/usr/local/lib/python3.8/json/init.py", line 357, in loads
return _default_decoder.decode(s)
File "/usr/local/lib/python3.8/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/local/lib/python3.8/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "main.py", line 37, in
response = submit_otp(phone_number, otp)
File "main.py", line 23, in submit_otp
response = requests.post(
File "/usr/local/lib/python3.8/site-packages/requests/models.py", line 975, in json
raise RequestsJSONDecodeError(e.msg, e.doc, e.pos)
requests.exceptions.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Same here, any workaround

any news on the issue?

any news on the issue?

You can use this repository to get the token:
https://github.com/Guy293/Tami4EdgeAPI


How to perform sliding window evaluation?

Hi,
Apologies if this was already stated somewhere, but may I know how we could perform the sliding window evaluation as described in the paper Appendix B and Table 7? The current example in README seems to support only non-overlapping evaluation.
Thanks!

It seems like we can just change the context-window argument to S? For example, we can use the following command to reproduce the test split result in Table 7:
l=3072; fairseq-eval-lm data-bin/wikitext-103/     --path wt103/checkpoint_best.pt  --sample-break-mode none --gen-subset test   --max-sentences 1 --model-overrides "{'max_tokens':$l, 'tokens_per_sample':$l, 'max_target_positions':$l}"  --tokens-per-sample $l --max-tokens $l  --max-target-positions $l  --context-window 512

Oh wow I apologize, we really didn't mention in the README how to do sliding window eval.
Yes you just need to set the context-window arg, but you should set it to L-1. So in your example it would have to be 3071. All examples in the paper use a sliding window that slides by 1 token every time, and to get that, you have to set context-window to be L-1.
Thanks, and please tell me if it doesn't work!


error Directory not Found

got this error message
[Error  : Unity Log] DirectoryNotFoundException: Could not find a part of the path "C:\koikatsu_model\Logging.log".
Stack trace:
System.IO.File.Delete (System.String path)
PmxExporter..cctor ()
Rethrow as TypeInitializationException: An exception was thrown by the type initializer for PmxExporter
then the plugin cant work, how to fix this ?

I fixed it by maunally create the file there. (ie. Create an empty text file and rename it to the exact same name.)

Issue has been fixed with the new release


USER STORY/ACCEPTANCE CRITERIA

##USER STORY
Using Bootstrap, I want to recreate my portfolio site with the following items using bootstrap components:
A Navigation bar
A navigation menu at the top.
Include links that are applicable to my portfolio.
Links should navigate to the appropriate sections
A hero section
A jumbotron featuring my picture, name, and any other information.
A work section displaying my work in grid.
Using bootstrap cards for each project.
The description should give a brief overview of the work.
A skills section
Listing out the skills I expect to learn from the bootcamp.
An about/contact section.
An "About Me" section in the same row.
A footer section.
All hyperlinks should have a hover effect.
All buttons should display a box shadow upon hover.
Bootstrap solution should minimize the use of media queries.
Deploy new Bootstrap-powered portfolio to GitHub Pages.
##Acceptance Criteria
Satisfies all of the above.
Application deployed at live URL.
Application loads with no errors.
Application GitHub URL submitted.
The GitHub repository contains application code.
The application resembles the functionality of their previous portfolio.
The repository has a unique name.
The repository follows best practices for file structure and naming conventions.
The repository follows best practices for class/ID naming conventions, indentation, quality comments, etc.
The repository contains multiple descriptive commit messages.
The repository contains quality readme with description, screenshot, and link to the deployed application.

A Navigation bar created using the format of last task adding menu to the top and links navigate to the appropriate sections.
A hero section using jumbotron featuring my icon, name, and any other information.
Work section displaying my work in grid using bootstrap cards for each project with the description giving a brief overview of the work.
A skills section created listing out the skills I have and expect to learn from the bootcamp adding an about/contact section and an "About Me" section in the same row.

Didnt use media queries in my manual CSS
Deployed new Bootstrap-powered portfolio to GitHub Page.
The repository has a unique name.
The repository follows best practices for file structure and naming conventions.
The repository follows best practices for class/ID naming conventions, indentation.
The repository contains multiple descriptive commit messages.
The repository contains quality readme with description, screenshot, and link to the deployed application.


Possible fix for issue #10

See issue #10

Looks like you are getting this ready for the first stable release single handedly :) I'll be giving a workshop next week and the next weekend I'll spend with an old friend but after that I'll prepare the initial release on PyPi. Maybe until then @ultrabug also released py3status 3.20 which will contain the entry point based discovery.

Nah I just had the 'luck' of having a really hard time getting it running at all!!
But fun to be able to contribute in the end. Quite frustrating at EuroPython to not even get the thing running...


Global and organization registries

Adds way to add server wide and organization wide registries

[x] Database and store changes
[ ] Server API changes
[ ] Organization settings UI
[ ] Server settings UI

close  #1448

Codecov Report
Patch coverage: 20.58% and project coverage change: -0.10 :warning:

Comparison is base (0970f35) 39.19% compared to head (c99d72b) 39.10%.

: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
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1672      +/-   ##
==========================================
- Coverage   39.19%   39.10%   -0.10%
==========================================
  Files         167      167
  Lines       10759    10785      +26
==========================================
  Hits         4217     4217
- Misses       6274     6299      +25
- Partials      268      269       +1

Impacted Files
Coverage Δ

server/model/registry.go
0.00% <0.00%> (ø)

server/pipeline/items.go
0.00% <0.00%> (ø)

server/store/datastore/registry.go
52.27% <25.00%> (-47.73%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Should be ready for review now

lint failure not related to my changes
server/store/datastore/migration/common.go:198:64: `alterColumnNull` - `null` always receives `false` (unparam)
func alterColumnNull(sess *xorm.Session, table, column string, null bool) error {


Update graphql to 5.1.0

Will there be an update for graphql version 5.1.0?
Thanks you for this awesome plugin!

check latest version https://pub.dev/packages/graphql_flutter_bloc


Unsubscribe method?

I see rustis lacks the unsubscribe method. Yes there is close, but it is not remotely close to unsubscribing a pubsub from a specific queue.
So I went ahead and integrated it myself for now. I thought I'd help u out and share the code.
I see that my code is not correctly highlighted, but for some reason I am not doing it correct, that's what I could come up with.
pub_sub_stream.rs
`
pub async fn unsubscribe<C, CC>(&mut self, channels: CC) -> Result<()>
where
C: SingleArg + Send,
CC: SingleArgCollection,
{
let channels = CommandArgs::default().arg(channels).build();
    self.client
        .unsubscribe_from_pub_sub_sender(&channels, &self.sender)
        .await?;

    let mut existing_channels = CommandArgs::default();
    std::mem::swap(&mut existing_channels, &mut self.channels);
    self.channels = existing_channels.arg(channels).build();

    Ok(())
}

`
client.rs
`
pub(crate) async fn unsubscribe_from_pub_sub_sender(
&self,
channels: &CommandArgs,
pub_sub_sender: &PubSubSender,
) -> Result<()> {
let (result_sender, result_receiver): (ResultSender, ResultReceiver) = oneshot::channel();
    let pub_sub_senders = channels
        .into_iter()
        .map(|c| (c.to_vec(), pub_sub_sender.clone()))
        .collect::<Vec<_>>();

    let message = Message::pub_sub(
        cmd("UNSUBSCRIBE").arg(channels.clone()),
        result_sender,
        pub_sub_senders,
    );

    self.send_message(message)?;

    result_receiver.await??.to::<()>()
}

`

unblock also need impl for blocking client when stop

unblock also need impl for blocking client when stop

Could you elaborate on this?

@Demy076 Thank you!

@Demy076 to create a code block in markdown, you need to use the tag ``` at the begining and at the end of your code block

added in https://github.com/dahomey-technologies/rustis/commit/546fd29fe0a1d3977d8cec0fcc6ee56550060b21


Can I edit the JSON?

I'd sometimes prefer to make quick changes by just hand-editign the JSON. I can edit the JSON in the editor but the changes don't seem to "stick".

Editor is highly vulnerable to bugs due to json data dependency
I don't recommend it.
If you do, you will need to save the modification and reload the editor webview.

Alright, thanks. Shame though, this was my primary use-case.


Add more answers related to isolation status - SK

Sorry I just rebased on master since a couple of new languages were added. But thanks, I can just get the texts from here :)

Ah thanks for fixing


Cannot use this package in a static library - Missing required module '_AtomicsShims' error.

In static library swift file, on attempt to import Atomics I'm getting Missing required module '_AtomicsShims' error.
This is structure of project in a nutshell:

StaticLibA // has Atomics static SPM library as dependency (but NOT as "Link Binary with Libraries" phase)

foo.swift   // needs atomic stuff, so does import Atomics which fails with the above error

FrameworkB   // links with StaticLibA

has some swift files which depend on StaticLibA types

ExecutableC // links with StaticLibA, FrameworkB & Atomics

uses both the static lib and dynamic framework

The very same import Atomics works well and without errors when it's in a final executable swift files, so problem is specific for static library target only.
Note also that StaticLibA deliberately is NOT linked with Atomics - if I did so then step # 1 succeeds but then step # 3 fails with "ld: 465 duplicate symbols for architecture arm64". I believe it's b/c the very same symbols of Atomic lib comes to final executable from both StaticLibA and FrameworkB.
Could you please advice what could be wrong or missing? OR, is it just not supported yet configuration? Thanks!

Thanks for the report! Unfortunately the reproduction steps don't contain enough information to debug this problem -- could you please provide a sample project that exhibits this issue?

Yup, I'll prepare the sample soon. Thanks!

👋  Any news regarding this issue? I encountered the same  problem when adding some tests to my iOS app.
To reproduce the issue:

Create an iOS app with unit tests in Xcode
Add the swift-atomics package to your main target
Try to execute the tests on your M1 Macbook

test.zip

Hello,
I have the same issue as @gaetanzanella mentioned above.
The difference is that my project has the grpc-swift package dependency, which in turn uses swift-atomics.
Xcode 14.0.1
macOS 12.6.1
M1 Pro Macbook Pro

I'm also getting this error. Running a Multiplatform app and when I go to test it I'm getting this error "_atomicsshims" on every imported module in the test. The swift_atomics package is included on my target because of another package I'm using.

I'm also getting this error when trying to archive a Swift package (from the command line using xcodebuild archive) to convert my package into a framework.
The binary just looks to be absent after the first build step and it fails when trying to ld Swift Atomics. Sorry this is all pretty new to me - don't know what I don't know - happy to add any information that might be helpful here.

Hello,
I have the same issue as @gaetanzanella mentioned above.
Xcode 13.4.1
macOS 12.5.1
M1 Pro Macbook Pro

Hello,
I have the same issue too, but different Xcode version
Xcode 13.4.1
macOS 12.6.1
M1 Pro Macbook Pro

Hi🎉
I fix it㊗
I imported it into the UITest bundle and was able to build it.
Added Atomic to Link Binary With Libraries.
Please see the attached photo.

Quick update: @gaetanzanella's sample project has been very helpful in reproducing this. (Thank you!)
Unfortunately it's a tooling issue, so the package won't be able to fix it on its own.
A proper fix is in the pipeline, but it will need time. @keisukeYamagishi's or @imthath-m's suggestions are currently the best workarounds that I am aware of.

Getting rid of the C module with something along the lines of https://github.com/apple/swift-atomics/pull/74 would likely also get rid of these issues. However, this will require some Swift compiler & stdlib work, so it will take time.

In my case, async-http-client was a dependency to a package I'm using, OpenAIKit, which is dependent upon swift-atomics (as @imthath-m mentioned) and adding that to my UI and unit test build phases as @gaetanzanella did allowed my tests to build.


relative library URLs

Is it possible to have a library with a relative URL? So that in cases where the project and libraries' origins are on the same server, then they can move together to a new server / new base URL, without breaking references or history.
If possible, is this documented? If not, can you add this feature?

@screamerbg,

[1] Mbed Online Compiler and [2] Mbed Studio. Also the 3 tools will have to support the old behavior AND the new behavior.

What's the 3rd tool?

The proposed domain/host inheritance from the parent won't work either as some repository may move from one host to another (e.g. github/ARMmbed/ may move to bitbucket), but that doesn't mean that Mbed OS Partner drivers are moving to bitbucket either

It would be incumbent on the user to only use the relative option when the repos would indeed be expected to move or be copied/mirrored together, if/when that happens. I don't think your counter-examples fit that use case.

Perhaps a config option that allows you to rewrite the URLs would be much more handy and also backwards compatible, where the URL rewriting will take care of the src<->dst URL translation.

I don't follow.

Thank you for raising this issue. Please note we have updated our policies and
now only defects should be raised directly in GitHub. Going forward questions and
enhancements will be considered in our forums, https://forums.mbed.com/ . If this
issue is still relevant please re-raise it there.
This GitHub issue will now be closed.


Design v0 API

The v0 API includes:

Request: Querying by bundle identifiers
Response: Delivery of multiple expression matrix data formats based on de facto standards

Acceptance Criteria:

[x] Complete review and finalize REST API v0 specification
[x] Update the proof-of-concept implementation to unblock prototyping for Matrix service integration with Data Browser

https://github.com/HumanCellAtlas/matrix-service/pull/28


Roll up FWA watersheds by stream order

Roll up FWA watersheds by stream order as an option

library(stringr)
library(dplyr)
library(bcdata)
library(sf)

# OPTION 1: GET WATERSHEDS FROM BC DATA ####

  ws <- bcdc_query_geodata("freshwater-atlas-watersheds") %>%
    filter(WATERSHED_GROUP_CODE == "BOWR") %>%
    collect()

  ws2 <- ws %>% select(WATERSHED_ORDER,FWA_WATERSHED_CODE) %>%
    mutate(iFWA_1 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,1],
           iFWA_2 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,2],
           iFWA_3 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,3],
           iFWA_4 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,4],
           iFWA_5 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,5],
           iFWA_6 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,6],
           iFWA_7 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,7],
           iFWA_8 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,8],
           iFWA_9 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,9],
           iFWA_10 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,10],
           iFWA_11 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,11],
           iFWA_12 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,12],
           iFWA_13 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,13],
           iFWA_14 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,14],
           iFWA_15 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,15],
           iFWA_16 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,16],
           iFWA_17 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,17],
           iFWA_18 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,18],
           iFWA_19 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,19],
           iFWA_20 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,20],
           iFWA_21 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,21])

  ws_all <- bind_rows(lapply(paste0("iFWA_",1:21), function(i=paste0("iFWA_",1:21)[1]){
    print(i)
    ws3 <- ws2 %>% rename(iFWA = i)
    ws3 %>% pull(iFWA) %>% unique() %>% length()
    ws3 %>% filter(as.numeric(iFWA) > 0) %>% group_by(iFWA) %>% summarize(WATERSHED_ORDER = max(WATERSHED_ORDER))
    }))

  st_write(ws_all, "C:/Users/bevin/Desktop/BOWR.gpkg")

# OPTION 2: GET WATERSHEDS FROM ####

  l <- st_layers("F:/FWA_WATERSHEDS_POLY/FWA_WATERSHEDS_POLY.gdb")[1] %>% as.data.frame() %>%
                filter(!name %in% c("_max_watershed_feature_id",
                                    "_fwa_watersheds_poly_updates_BACKUP",
                                    "_fwa_watersheds_poly_updates"))
  lapply(l[,1], function(id = "PARS"){

    if(!file.exists(paste0("C:/Users/bevin/Desktop/",id,".gpkg"))){
    ws <- st_read("F:/FWA_WATERSHEDS_POLY/FWA_WATERSHEDS_POLY.gdb", layer = id)

    ws2 <- ws %>% select(WATERSHED_ORDER,FWA_WATERSHED_CODE) %>%
      mutate(iFWA_1 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,1],
             iFWA_2 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,2],
             iFWA_3 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,3],
             iFWA_4 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,4],
             iFWA_5 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,5],
             iFWA_6 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,6],
             iFWA_7 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,7],
             iFWA_8 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,8],
             iFWA_9 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,9],
             iFWA_10 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,10],
             iFWA_11 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,11],
             iFWA_12 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,12],
             iFWA_13 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,13],
             iFWA_14 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,14],
             iFWA_15 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,15],
             iFWA_16 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,16],
             iFWA_17 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,17],
             iFWA_18 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,18],
             iFWA_19 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,19],
             iFWA_20 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,20],
             iFWA_21 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,21])

    ws_all <- bind_rows(lapply(paste0("iFWA_",1:21), function(i=paste0("iFWA_",1:21)[1]){
      print(i)
      ws2 %>% dplyr::rename(iFWA = i )%>% filter(as.numeric(iFWA) > 0) %>% group_by(iFWA) %>% summarize(WATERSHED_ORDER = max(WATERSHED_ORDER, na.rm = T))
      }))

    ws_all <- ws_all %>%
      st_make_valid() %>%
      mutate(id = "PARS", area_m = round(as.numeric(st_area(.)), 0)) %>%
      arrange(-WATERSHED_ORDER)

    st_write(ws_all, paste0("C:/Users/bevin/Desktop/",id,".gpkg"))}
    })

Related to #2

I didn't look super closely at this code but I've noticed rolling up
FWA_WATERSHED_CODE upstream runs into problems in some areas where two line
rivers occur.
Stuart lake has same FWA_WATERSHED_CODE as Stuart river, so if you attempt
to roll up codes from the outlet of the lake, it will include the entire
Stuart river, and therefore all tribes to the river below the lake.
On Sat, Jan 13, 2024, 11:19 a.m. Alexandre Bevington <
@.***> wrote:

Related to #2 https://github.com/bcgov/watershedBC/issues/2
—
Reply to this email directly, view it on GitHub
https://github.com/bcgov/watershedBC/issues/18#issuecomment-1890744782,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMVJ7EZAU3UVVUDLGPZFRPTYOLM55AVCNFSM6AAAAABBZQSRR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQG42DINZYGI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Good point!
I also noticed the way I showed above only computes by WATERSHED_GROUP_CODE... so the Fraser looks like this:

I'm testing a way to combine all of the WATERSHED_GROUP_CODE outputs such that I can then dissolve the entire Fraser into a single polygon, and so forth. I'm not certain this will fix the Stuart Lake issue, but I think it will. Seeing as the Stuart Lake watersheds will have the initial Fraser code of 100, then the Nechako, then Stuart River, etc. Standby!

This works OK, it builds nested watersheds for each stream segment by stream order for the freshwater atlas. Good for nested basins, bad for point of interest basins.

l <- st_layers("FWA_WATERSHEDS_POLY.gdb")[1] %>% as.data.frame() %>%
  filter(!name %in% c("_max_watershed_feature_id",
                      "_fwa_watersheds_poly_updates_BACKUP",
                      "_fwa_watersheds_poly_updates"))

ws_d <- lapply(l[,1], function(id = "PARS"){

  if(!file.exists(paste0(id,".gpkg"))){

    ws <- st_read("FWA_WATERSHEDS_POLY.gdb", layer = id)

    ws2 <- ws %>% select(WATERSHED_ORDER,FWA_WATERSHED_CODE) %>%
      mutate(iFWA_1 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,1],
             iFWA_2 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,2],
             iFWA_3 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,3],
             iFWA_4 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,4],
             iFWA_5 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,5],
             iFWA_6 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,6],
             iFWA_7 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,7],
             iFWA_8 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,8],
             iFWA_9 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,9],
             iFWA_10 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,10],
             iFWA_11 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,11],
             iFWA_12 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,12],
             iFWA_13 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,13],
             iFWA_14 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,14],
             iFWA_15 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,15],
             iFWA_16 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,16],
             iFWA_17 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,17],
             iFWA_18 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,18],
             iFWA_19 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,19],
             iFWA_20 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,20],
             iFWA_21 = str_split(FWA_WATERSHED_CODE, "-", simplify = T)[,21])

    ws_all <- bind_rows(lapply(paste0("iFWA_",1:21), function(i=paste0("iFWA_",1:21)[1]){
      print(i)
      ws2 %>% dplyr::rename(iFWA = i )%>% filter(as.numeric(iFWA) > 0) %>% group_by(iFWA) %>% summarize(WATERSHED_ORDER = max(WATERSHED_ORDER, na.rm = T)) %>% mutate(class = i)
    }))

    ws_all <- ws_all %>%
      st_make_valid() %>%
      mutate(id = id, area_m = round(as.numeric(st_area(.)), 0)) %>%
      arrange(-WATERSHED_ORDER)

    st_write(ws_all, paste0(id,".gpkg"))
  }
  ws_all
  })

l <- list.files(full.names = T, pattern = "*.gpkg$")

fwa_list <- paste0("iFWA_",1:21)

t <- lapply(fwa_list, function(fwa="iFWA_1"){

  print(fwa)

  all <- bind_rows(lapply(l, function(i=l[1]){

    print(paste(fwa, i))

    name <- st_layers(i)[[1]]

    st_read(i, query = paste0("SELECT * FROM ",name," WHERE class = '",fwa,"'"), quiet = T)

  }))

  all <- all %>% group_by(iFWA) %>% summarize(WATERSHED_ORDER= as.character(max(WATERSHED_ORDER)))
  st_write(all, paste0(fwa,".gpkg"))
  all
  })

tt <- bind_rows(t)
st_write(tt, "iFWA.gpkg")

I found an error in the script I'd emailed, adding the corrected here for your reference. Also realized we were talking about two different methods! This code fails at lake outlets with two line rivers because the lake will have the same FWA_WATERSHED_CODE as its draining river.. It also fails if you want to map to a specific point along a two line river.
I think this works for small (non-two line river) watersheds.. but looking at LOCAL_WATERSHED_CODE doesn't present an obvious solution to these issues either.
All this is to say - Using the FWA Watersheds dataset alone will NOT allow you to reliably map watersheds to a point, but maybe a logic statement could determine whether the point is along a simple stream segment - then apply this method.
Be forewarned the query is slow for large watersheds.
Map watershed to pour point by rolling up FWA_WATERSHED_CODE
library(tidyverse)
library(bcmaps)
library(bcdata)
make a pour point (this is outlet of Stuart Lake)
pp <- sf::st_sf(st_sfc(st_point(c(-124.270991, 54.417258))), crs =4326) %>%
st_transform(src = 4326, crs = 3005)
find the FWA_WATERSHED_CODE corresponding to pour point location
fwa_watershed_code <- bcdata::bcdc_query_geodata("https://catalogue.data.gov.bc.ca/dataset/3ee497c4-57d7-47f8-b030-2e0c03f8462a") %>%
filter(INTERSECTS(pp)) %>%
dplyr::select(FWA_WATERSHED_CODE) %>%
collect()
code <- str_remove_all(fwa_watershed_code$FWA_WATERSHED_CODE, "-000000")
query all watersheds within the named watershed, then filter to only codes upstream of pp
cql <- paste0("FWA_WATERSHED_CODE like '%", code, "%'")
fwa_watershed_queried <- bcdata::bcdc_query_geodata("https://catalogue.data.gov.bc.ca/dataset/3ee497c4-57d7-47f8-b030-2e0c03f8462a")  %>%
filter(CQL(cql)) %>%
collect()
fwa_watershed.m <- st_transform(fwa_watershed_queried, src = 3005, crs = 4326) %>%
as_Spatial()
pp.m <- st_transform(pp, src = 3005, crs = 4326) %>%
as_Spatial()

Just a follow up - Looks like from the FWA manual that its worth exploring the "DOWNSTREAM_ROUTE_MEASURE" to solve this double line river issue I'm having

Nice, thanks for the code! Agreed. The FWA watersheds aren't great for point of interest watershed mapping because of the inability to delineate watersheds smaller than the polygons provided in the FWA watersheds layer.
But I think we are trying to do different things. I'm not trying to do POI watersheds. I'm trying to roll up the FWA by watershed order such that I can pre-compute whole watersheds by stream order using the FWA.  So if you click on a 3rd order stream that has no name, it will show you the roll up of that 3rd order stream with the upstream polygons.
So for your Stuart River example below Stuart Lake... if you select polygons that intersect Stuart River, then you'd get Stuart River, Nechako River and Fraser River:

I think I'm ready to add the "roll up by FWA watershed order" dataset to the app and I'll close the issue once the app is updated.
For your question of using the FWA for POI watersheds... maybe make a new issue? I think the only way is to do a hybrid raster-vector watershed delineation. So use your method to map the watershed to the closest watershed polygon, then use a raster based watershed delineation script in only the lowest watershed polygon to resolve the sub-basin watershed.... not sure that is well explained. In any case, I think it is a separate question.

This makes sense - feel free to close the issue!  I'm nervous that the raster method will lead to errors related to snapping points.. but surely your suggestion on querying closest FWA watershed would minimize that risk.. I'm giving one final crack at doing this solely with FWA queries, otherwise I"ll open an issue on your tool!


DOI for white paper?

White paper(s) can be found here: http://help.brain-map.org/display/observatory/Documentation
But it would be nice to have a persistent link for those

Note that the brain map website keeps changing pretty radically, so this links of mine from even a month ago no longer route to the same place...

Saskia sent what they could over email; will talk to others about getting DOI but no guarantee


Fixed #35665 -- Fixed a crash when passing empty order_by to Window.

Trac ticket number
ticket-35665
Branch description
This also caused un-ordered sliced prefetches to crash as they rely on Window.
Regression in e16d0c176e9b89628cdec5e58c418378c4a2436a that made OrderByList piggy-back ExpressionList without porting the empty handling that the latter provided.
Supporting explicit empty ordering on Window functions and slicing is arguably a foot-gun design due to how backends will return undeterministic results but this is a problem that requires a larger discussion.
Refs ticket-35064.
Thanks @awbacker for the report.

Thank you @charettes ❤ this looks great
@awbacker can you confirm this works for you?

The change works great for me, in both situations where it occured.  Thanks!

Explicit empty order_by()
Model lacking ordering = ["field"]


getUserByKey

Hey there,
I've implemented a getUserByKey function, added documentation, and created a test (which needs a valid userKey to be added to pass, but which otherwise should). I've never submitted a pull request before, but it seems like I need permission to create the branch and then submit the request -- or let me know if there's some other way you'd rather I submitted.
Thanks for building this, it's been super helpful!

I no longer have access to the proposed changed code, so closing this ticket.


Release PR#3 created workflow for new releases

@ mention of reviewers
@ObadaS
A brief description of the purpose of the changes contained in this PR.
Created a workflow and json file for release information.
Issues this PR resolves

#1582 -> Point 4

A checklist for hand testing

[ ] We need to test if with a new release publish, this workflow is triggered

Checklist

[ ] Code review by me
[ ] Hand tested by me
[ ] I'm proud of my work
[ ] Code review by reviewer
[ ] Hand tested by reviewer
[x] CircleCi tests are passing
[ ] Ready to merge

Looks good on my end, we only need to test it now

I see in the code that you are using >, what do you think about using >> to have a local file in the repo with all the changes ?
The code doesn't specify a branch, does it push the develop or master branch ? I think we want it to push on the master branch, otherwise we would need to do a separate PR to bring the updated file into master after we already merged into master (since we only publish the Release notes once we have successfully merged into master)

The CircleCI test fails with the same error I have on my local machine when trying to deploy the current develop branch. Seems like there is a problem with the Python version used, which could be an easy fix.

Looks good on my end, we only need to test it now

I see in the code that you are using >, what do you think about using >> to have a local file in the repo with all the changes ?

> will rewrite the file while >> will append. I think we want to rewrite the file

The code doesn't specify a branch, does it push the develop or master branch ? I think we want it to push on the master branch, otherwise we would need to do a separate PR to bring the updated file into master after we already merged into master (since we only publish the Release notes once we have successfully merged into master)

I have updated the workflow to push changes to develop branch and automatically create a PR. We should test it on github if this works or not because I have never worked on workflows. I chose develop branch so that everything is consistent and there is no mismatch between develop and master

Here, version.json looks like this:
{
    "tag_name": "1.1.0",
    "release_name": "Second release",
    "published_at": "November 09, 2023",
    "body": ""
}

However, on the PR#2 (#1592) that'll be merged to develop, we have:

```json
{
  "tag_name": "v1.11.0",
  "release_name": "Release 1.11.0",
  "published_at": "2024-09-16",
  "body": "",
  "release_url": "https://github.com/codalab/codabench/releases/tag/v1.11.0"
}

That looks problematic

json updated in the latest commit

So should we merge this to develop? @ihsaan-ullah @ObadaS

So should we merge this to develop? @ihsaan-ullah @ObadaS

@ObadaS do you know when the workflow will be triggered? if this is in develop or master?

It might not get triggered because the workflow file does not exist yet, and there is nothing that was published after the merge happens.
But it should trigger automatically, after the merge, when I publish the 1.12 release


[10.x] Allow Mail Facade alwaysTo method to accept arrays of email addresses and names

Follow-up of #47625.
This Pull Request allows the alwaysTo method on the Mail facade to accept an array of email addresses and, optionally, names.
For example, the following can be supplied to the alwaysTo method:
Existing Functionality - 1 email address and 1 name:
<EMAIL_ADDRESS>'Taylor Otwell')
New Functionality - an array of email addresses:
<EMAIL_ADDRESS><EMAIL_ADDRESS>New Functionality - an associative array of 'email-address' => 'name' pairs:
<EMAIL_ADDRESS>=> 'Taylor Otwell'<EMAIL_ADDRESS>=> 'Nuno Maduro'])

@kylekanderson draft PRs aren't reviewed. Are you finished with this?


Update is_dyanmic check for new accpetable value after snowflake bundle change 2024_03

resolves #1016
docs dbt-labs/docs.getdbt.com/#

Problem
https://docs.snowflake.com/en/release-notes/bcr-bundles/2024_03_bundle updates the is_dynamic field from YES to Y which we query to see if a table is a dynamic table for not.
this currently fails due to looking for different values from when the is_dynamic field was first introduced via https://github.com/dbt-labs/dbt-snowflake/pull/937

Solution
updating to check for both values for users pre and post change and updating same similar call in test

ISSUE:
we had to rebase this branch due to local repo getting corrupted leading to excessive amounts of extra commits that didn't mean or point to anything.
Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

This PR is going to need to be rebased or something since it's way off main.

Closing as currently thoughts are that the real issue is around changes to show terse kind options for the bundle in question.


issue with content type of output file

Hi,
the output file resulting from the combination of several yaml files has an ANSI encoding, how can I keep utf-8 as the original yaml files?

@knoxg

Hi @gcornacchia.
I can look at adding an encoding parameter but in the meantime you could try setting the default encoding on the commandline via
-Dfile.encoding=UTF-8

( from https://stackoverflow.com/questions/1006276/what-is-the-default-encoding-of-the-jvm ). That page also suggests setting the LC_ALL environment variable but I have tried either of these suggestions so not sure if either will work.

hi @knoxg, checkout my PR https://github.com/randomnoun/yaml-combine-maven-plugin/pull/4 and tell me what do you think

Looks great, thanks.
Will merge that in & cut a new release

Released v2,0,8 , should appear in maven central in a few hours


As a user, I would like to be able to click on the dots in the time line chart and see the result list for that year in a pop-up window

@lilimelgar commented on Tue May 23 2017
The timeline chart of the Search tools is currently static, since its points are not clickable. Users request that it is possible to click on the points and see the number of results (hits) per date/year, and get the result list by clicking on the point in a pop-up window. This window could offer a button asking the user if s(he) wants to update the query to that year only, then refreshing the date range.
See for example AVResearcher: http://avresearcher.clariah.beeldengeluid.nl/#):
-When the user clicks on the point for 2003:

-The user gets a result list in a pop-up window, as in the graphic below:

The idea is also to add a button to that window asking the user if s(he) wants to refine the query to update the date filtering to the year that the user has clicked on, or just close the window and go back to the same query.
Relates to issue #82.
Also requested: click on bars in the histogram (old #59)

@roelandordelman commented on Fri Jan 19 2018
I put this in the backlog as we need to revise the comparative search recipe/tool first before starting to code on things.

@lilimelgar commented on Fri Jan 19 2018
@roelandordelman This is not a function proper of the Comparative search tool only, but of the single search/visualization of search results.

@lilimelgar Include this as part of issue #12. Ask @jblom to move to labo components repo.

@lilimelgar test redirect


e2e long yellow path

Can you test the live toggling and make sure it works? And then let's merge after I enabled live toggling in the long planner.

Should we try doing hue based on acceleration plan for e2e_long?

Yellow at start seems good to me

I think this looks okay, tried to increase the saturation/lightness of the yellow a bit


Update Syntax Highlighting

Context
Since the release of the extension Cadence has gained some more features.
Make sure the grammar is up-to-date.
Definition of Done

[ ] Improve the syntax highlighting for Cadence by updating the grammar

[ ] Pragmas

Why did this get closed?

@DylanTinianov This would be a relevant improvement for current ongoing work. I would say adding pragma syntax highlighting as well as documentation around pragma would be nice.


Improved BindingInfo validation for xWebsite

Improved BindingInfo validation for xWebsite: the HostName property is now required for use with Server Name Indication (i.e., when the SslFlags property is set to 1 or 3).
The Test-TargetResource should return the following error if the HostName binding property is either empty or is not specified:

Trimmed trailing spaces.

Can I suggest connecting your GitHub account up to Reviewable.io for this repo? It makes reviewing way easier.

LGTM - thanks for all the work on this @SNikalaichyk

Hello! There are also some pending changes by @nzspambot in #132 . Can we postpone restyling of this module to avoid merge conflicts until the next time?

@SNikalaichyk - Ok cool. Sounds like a good idea.

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable

So this looks good to me, am I merging #135 or does it really matter?

So I should probably do this one first then and let you (@nzspambot) deal with the smaller merge then? :grin:


fix(RELEASE-1187): error when Dockerfile has a different filename

We assumed that the Dockerfile in the *.dockerfile oci artifact would always be called Dockerfile. Wrong assumption. We created STONEBLD-2822 for the build team to always use the same name (instead of just copying whatever the user has). But for now let's read the filename from the metadata and use that.

So we always fetch the Dockerfile with the .dockerfile pull spec, but it can end up in the directory as any name?

Yes, the pull spec is always *.dockerfile.

Would it not be easier to just check $DOCKERFILE_DIR to see if a file is there? If so, that's the dockerfile

Yes, but I thought this would be more robust - what if somehow there were two files and not just one? (But then you might also have two layers in the metadata and I pick the first one, so probably similar to just getting the first file, I know...) If you think I should just do that, I can change it.

So we always fetch the Dockerfile with the .dockerfile pull spec, but it can end up in the directory as any name?

Yes, the pull spec is always *.dockerfile.

Would it not be easier to just check $DOCKERFILE_DIR to see if a file is there? If so, that's the dockerfile

Yes, but I thought this would be more robust - what if somehow there were two files and not just one? (But then you might also have two layers in the metadata and I pick the first one, so probably similar to just getting the first file, I know...) If you think I should just do that, I can change it.

It is would be less oras calls which is kind of nice. I'll leave it up to you. I do think we should echo something and maybe even fail if there is more than one file. Anything to alert us that they are pushing two things to the .dockerfile pullspec now and we need to adjust on our side

very elegant! nice

So we always fetch the Dockerfile with the .dockerfile pull spec, but it can end up in the directory as any name?

Yes, the pull spec is always *.dockerfile.

Would it not be easier to just check $DOCKERFILE_DIR to see if a file is there? If so, that's the dockerfile

Yes, but I thought this would be more robust - what if somehow there were two files and not just one? (But then you might also have two layers in the metadata and I pick the first one, so probably similar to just getting the first file, I know...) If you think I should just do that, I can change it.

It is would be less oras calls which is kind of nice. I'll leave it up to you. I do think we should echo something and maybe even fail if there is more than one file. Anything to alert us that they are pushing two things to the .dockerfile pullspec now and we need to adjust on our side

My goal is to keep the Jira open and we can discuss later and maybe remove this logic once it's fixed in build service (and there is always just Dockerfile). So I would prefer to keep this as is - it was supposed to be a quick fix. If I add a check for multiple files, then at that point I could remove the logic to figure out the filename... which is already implemented. But if you prefer that, I can do it (but likely not today, so they won't get the fix today).

Approving, but can you link this PR in the related jira or whatever (not sure if it is RELEASE-1187 or a followup one) so we know what code we are talking about when we refine it?

Thanks. Yeah, it's RELEASE-1187 . And I added a note in the description about this above.


Can I get some help please?

I don't've sufficient tech knowledge to figure out how to run & use this, but I'm really interested and want to be able to.
I installed the 'periodic-table-cli-main.zip' file and have extracted it, but I can't figure out what to do next to use it...
I'm a windows user. I hope I can get to use this, it isn't too complicated, right?
Please forgive me if I'm not supposed to ask for help here, I didn't know where else to go...

Nvm! I managed to run it by myself, hurray to me! (and thanks to co-pilot for telling me to install Node.js)


New remade version

CockyGrabber is outdated and many Paths that led to the Databases have changed with time so I went and remade it: link

You can make a PR if you want

Actually I was TheBackdoor. I got banned cuz I found an GitHub API Exploit (spam creating Issues/PR/...) and tried it on an private repo but it seems like github didn't like that.
It looked like CockyGrabber won't be updated anymore by urself so I decided to rebrand it.

What a great idea ! I'm contacting Bytixo and tell him to transfer u the repos, u'll push everything yourself ok ?

Is this satire? I can't tell

Is this satire? I can't tell

check your mails

Ah, I see ok thx. Imma update the Repo then

@Stanley-GF ta encore twitter ou pas


Added jwgmeligmeyling GitHub actions to upload Maven reports to PR

Added jwgmeligmeyling's actions to the workflow.
The actions seems obsolete as they are not maintained since 2020 and depends on Node.js 16. But they still works if the projects declare the right permissions.
The README.md has been updated accordingly to declare the permissions required to properly run the workflow.

Also, in the PR, don't forget to mention the GitHub issues you're fixing (#7, #8, #9, #10)


Prevent gaps in tall delimiters

Fixes #1975 by typesetting tall delimiters with a small (0.005em) overlap where glyphs meet.
As evidence, I offer two screenshots. The first one is before this PR:

The second screenshot is after this PR:

The value of 0.005em is entirely a guess on my part and may need later adjustment.

@kevinbarabash This PR does several vertical alignment calculations and the screenshotter tests declare several errors. It appears to me that all of them are sub-pixel issues except the Array test, in which the diff shows one thin green line.
I'm willing to declare this PR as good to go, but I'll leave the screenshotter tests as is for now. If you agree that the PR is okay, I'll replace the screenshots with new ones.

I went ahead and updated the screenshots.

Codecov Report

Merging #1986 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1986      +/-   ##
==========================================
+ Coverage   93.36%   93.38%   +0.01%
==========================================
  Files          79       79
  Lines        4931     4942      +11
  Branches      860      860
==========================================
+ Hits         4604     4615      +11
  Misses        287      287
  Partials       40       40

Flag
Coverage Δ

#screenshotter
89.15% <100%> (+0.02%)
:arrow_up:

#test
86.58% <50%> (-0.1%)
:arrow_down:

Impacted Files
Coverage Δ

src/delimiter.js
87.31% <100%> (+0.52%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #1986 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1986      +/-   ##
==========================================
+ Coverage   93.36%   93.38%   +0.01%
==========================================
  Files          79       79
  Lines        4931     4942      +11
  Branches      860      860
==========================================
+ Hits         4604     4615      +11
  Misses        287      287
  Partials       40       40

Flag
Coverage Δ

#screenshotter
89.15% <100%> (+0.02%)
:arrow_up:

#test
86.58% <50%> (-0.1%)
:arrow_down:

Impacted Files
Coverage Δ

src/delimiter.js
87.31% <100%> (+0.52%)
:arrow_up:

Continue to review full report at Codecov.

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


ngCookies not always updating

Hi all,
i try to update some cookie data in my Angular Application. This works more or less, but sometimes it behaves very strange:
For example i want to update the "Tabs" which are saved in the cookies when a user sent a search request (i want to save the search parameters into the tabs, so you can leave a tab and once you return back to it, you can reload your search results). In my angular Controller there is a function which handles the search, and once it finished its job, the following method is called to update cookie data:
`updateSearch: function (id, search) {

        var count = 0;

        var index = false;

        angular.forEach($rootScope.tabs, function (tab_) {

            if (id === tab_.id)

                index = count;

            count++;

        });

        var tab = $rootScope.tabs[index];

        tab.search = search;

        $rootScope.tabs[index] = tab;

        console.log(tab);

        $cookies.putObject("wawiTabs", {

            tabs: $rootScope.tabs

        });
    },`

But this does not update the tabs. When i inspect the cookies, the new value is not yet added. the $rootScope.tabs are updated correctly, and the way i try to update those cookies with this logic, it works in other application parts.
The bug i am describing here also appears when i try to save a pagination number. When i click on a page number inside of the tab, everything works fine. But when i leave the active tab and return to it again, it still saves the old value. It seems like there is a kind of cache which prevents updating cookie data sometimes...can you help me here?

Hi, your issue reads like a support question. This github repo is only for bug reports and feature requests. Please try one of the listed Support Channels.


How can We USE A FUNCTION INSIDE YOUR CODE??

How can we add a function inside your code. IT's totally unclear

What do you mean? The https://github.com/progschj/ThreadPool/blob/master/example.cpp shows hot to use the thread pool.

This should do it i think:
int foo(float x) { return x; }
//...
future result = pool.enqueue(foo, 3.14f);

yes Prog I coded it before. For a class I should do the following:
IF i have a function bar in a class
int foo::bar(float x) { return x; }
//...
future result = pool.enqueue(&foo::bar, this, 3.14f);


[Snyk] Upgrade web3 from 1.8.0 to 1.10.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade web3 from 1.8.0 to 1.10.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 8 versions ahead of your current version.
The recommended version was released 2 months ago, on 2023-05-10.

Release notes

Package name: web3

1.10.0 - 2023-05-10Fixed

Improved the error propagation in web3-providers-http package to effectively propagate useful error infomation about failed HTTP connections (#5955)
Fixed "Uncaught TypeError" calling a contract function that revert using MetaMask (#4454) and related "n.data.substring is not a function", that is raised when there is a revert and web.eth.handleRevert = true (#6000)

Changed

transaction.type is now formatted to a hex string before being send to provider (#5979)
When sending a transaction, if transaction.type === '0x1' && transaction.accessList === undefined, then transaction.accessList is set to [] (#5979)
Removed an unnecessary chainId parameter from toChecksumAddress() function types (#5888)

Added

Added support for getPastEvents method to filter allEvents and specific event (#6015)

Security

Updated dependencies (#6044)

1.10.0-rc.0 - 2023-05-02Fixed

Improved the error propagation in web3-providers-http package to effectively propagate useful error infomation about failed HTTP connections (#5955)
Fixed "Uncaught TypeError" calling a contract function that revert using MetaMask (#4454) and related "n.data.substring is not a function", that is raised when there is a revert and web.eth.handleRevert = true (#6000)

Changed

transaction.type is now formatted to a hex string before being send to provider (#5979)
When sending a transaction, if transaction.type === '0x1' && transaction.accessList === undefined, then transaction.accessList is set to [] (#5979)
Removed an unnecessary chainId parameter from toChecksumAddress() function types (#5888)

Added

Added support for getPastEvents method to filter allEvents and specific event (#6015)

Security

Updated dependencies (#6044)

1.9.0 - 2023-03-20Fixed

Fixed skipped ws-ganache tests (#5759)
Fixed "provider started to reconnect error" in web3-provider-ws (#5820)
Fixed Error: Number can only safely store up to 53 bits (#5845)
Fixed types for packages which have default exports but not declared default export in .d.ts (#5866)
Fixed Transaction type by adding missing properties (#5856)

Changed

Add optional hexFormat param to getTransaction and getBlock that accepts the value 'hex' (#5845)
utils.toNumber and utils.hexToNumber can now return the large unsafe numbers as BigInt, if true was passed to a new optional parameter called bigIntOnOverflow (#5845)
Updated @ types/bn.js dependency to 5.1.1 in web3, web3-core and web3-eth-contract as reason mentioned in #5640 (#5885)
Add description to error for failed connection on websocket (#5884)

Security

Updated dependencies (#5885)

1.9.0-rc.0 - 2023-03-07Fixed

Fixed skipped ws-ganache tests (#5759)
Fixed "provider started to reconnect error" in web3-provider-ws (#5820)
Fixed Error: Number can only safely store up to 53 bits (#5845)
Fixed types for packages which have default exports but not declared default export in .d.ts (#5866)
Fixed Transaction type by adding missing properties (#5856)

Changed

Add optional hexFormat param to getTransaction and getBlock that accepts the value 'hex' (#5845)
utils.toNumber and utils.hexToNumber can now return the large unsafe numbers as BigInt, if true was passed to a new optional parameter called bigIntOnOverflow (#5845)
Updated @ types/bn.js dependency to 5.1.1 in web3, web3-core and web3-eth-contract as reason mentioned in #5640 (#5885)
Add description to error for failed connection on websocket (#5884)

Security

Updated dependencies (#5885)

1.8.2 - 2023-01-30Changed

Updated Webpack 4 to Webpack 5, more details at (#5629)
crypto-browserify module is now used only in webpack builds for polyfilling browsers (#5629)
Updated ethereumjs-util to 7.1.5 (#5629)
Updated lerna 4 to version 6 (#5680)
Bump utils 0.12.0 to 0.12.5 (#5691)

Fixed

Fixed types for web3.utils._jsonInterfaceMethodToString (#5550)
Fixed Next.js builds failing on Node.js v16, Abortcontroller added if it doesn't exist globally (#5601)
Builds fixed by updating all typescript versions to 4.1 (#5675)

Removed

clean-webpack-plugin has been removed from dev-dependencies (#5629)

Added

https-browserify, process, stream-browserify, stream-http, crypto-browserify added to dev-dependencies for polyfilling (#5629)
Add readable-stream to dev-dependancies for webpack (#5629)

Security

npm audit fix for libraries update (#5726)

1.8.2-rc.0 - 2023-01-11 Read more

1.8.1 - 2022-11-10

1.8.1-rc.0 - 2022-10-28

1.8.0 - 2022-09-14

    from web3 GitHub release notes

Commit messages

Package name: web3

f3846d1 Build commit for 1.10.0
2c74586 v1.10.0
3ed053f build output and changelog update for v1.10.0-rc.0
c7bdd99 v1.10.0-rc.0
13a2edc Remove the unnecessary chainId parameter (#5888) (#6057)
7b3ce91 1x update  (#6044)
195cd3c Filter option doesn't work in getPastEvents (#6015)
48958ee Nicos99/revert call (#6009)
6ce085b Fix error: "n.data.substring is not a function" (#6000)
4e5afa1 Format `transaction.type` to hex. Add empty `accessList` is `tx.type === '0x1'` (#5979)
9238e10 Bump webpack from 5.75.0 to 5.76.0 (#5925)
2097f8d Improve logging of connection errors (#5955)
b4d1272 Create CODEOWNERS (#5952)
11bb4d4 github conf for stale issues and PRs (#5893)
46d414b Release/1.9.0 (#5895)
5508f94 Add empty job to unstuck actions in 1.x (#5932)
2b3fb3a Nikos/5835/websocket provider keeps important error message back (#5884)
ef23642 dependencies updates (#5885)
8d369a9 Nikos/5821/transaction type fix (#5856)
afa2943 fix types default export (#5852) (#5866)
e4b25bf Add optional param `hex` to `getTransaction` and `getBlock`. (#5845)
8621030 handled "provider started to reconnect error" (#5820)
5009bdd Update web3-eth-accounts.rst (#5810)
5807398 updating ganache failing test (#5779)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
Patch coverage has no change and project coverage change: -0.86% :warning:

Comparison is base (c8b532b) 79.10% compared to head (1e3c903) 78.24%.

:exclamation: Current head 1e3c903 differs from pull request most recent head 8118c19. Consider uploading reports for the commit 8118c19 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1560      +/-   ##
==========================================
- Coverage   79.10%   78.24%   -0.86%
==========================================
  Files          19       32      +13
  Lines        2077     2625     +548
  Branches      442      544     +102
==========================================
+ Hits         1643     2054     +411
- Misses        303      405     +102
- Partials      131      166      +35

see 14 files with indirect coverage changes

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


⚠ Bofrost.be has degraded performance

In fa1a741, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 874 ms

Resolved: Bofrost.be performance has improved in 00e4256.


ZF3 Support, BC break

Also a little bit of PSR2 loving ;)

@Saeven could you please remove CS related changes? They're entirely subjective and I don't see reason to include them in this PR.

Any chance of getting ZF3 compat merged?

@exptom I guess I'll cherry pick the relevant commits.

I have opened PR #3 with just the ZF3 fixes and not all the extra CS changes if you are able to merge and tag please :-)

superseded by #3


[scaffolder] add support to CATALOG_FILTER_EXISTS

Hey, I just made a Pull Request!

This PR modifies the EntityPicker to allow catalog filters that has one specific key without filtering any value on that key
Example:
We can filter entities that has github.com/team-slug annotation using this catalog API filter: /catalog/entities?filter=metadata.annotations.github.com/team-slug bringing all the entities with that annotation with any team-slug value
with this PR the same query can be done using the true value:
catalogFilter:
    metadata.annotations.github.com/team-slug: true

RFC:

Until this PR, non string values were ignored, with this we convert true-> CATALOG_FILTER_EXISTS and false -> 'false' but not sure on this last thing, maybe is better to keep ignoring the false values.
We decided to keep this logic inside EntityPicker to not "break" the current catalog client behaviour.
All the keys inside the catalog filter should be flat or will be ignored (as object are not string values)

catalogFilter:
    spec:    # this filter is currently ignored
       type: team

we didn't changed any of this behaviour but maybe should be considered as valid filter and flat those before calling the Catalog API??
Posible Alternatives:
1. Modify the catalog client
Modify the catalog client to allow this type of queries without using a symbol
simplifies the logic but missing the whole point of using the CATALOG_FILTER_EXISTS symbol
2. use null value instead of true
this can be also valid and we get a null value on that key
catalogFilter:
   metadata.annotations.github.com/team-slug:

the same logic would be the same but reading the yaml file look more like a type that a feature for me
3. using a special key
we can add some special name to considering this cases:
catalogFilter:
   has: metadata.annotations.github.com/team-slug

but will differ with the catalog API logic
4. using a special value
using something like:
catalogFilter:
   metadata.annotations.github.com/team-slug: $$exist$$

but not sure if worth it
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Seems I don't have push permissions to push the rebase, let me know if you prefer to close this and add a new PR from fork instead

moved to https://github.com/backstage/backstage/pull/16779


Support AWS S3 Simple Authentication (Access/Secret Key)

What went wrong?
When creating as new table with qbeast-spark on an S3 bucket configured with Access Key/Secret Key credentials, Spark inaccurately indicates that the table already exists.
How to reproduce?
1. Code that triggered the bug, or steps to reproduce:
Configure spark to use S3 with simple credentials & Qbeast:
spark.packages io.delta:delta-spark_2.12:3.2.0,org.apache.hadoop:hadoop-aws:3.3.1,io.qbeast:qbeast-spark_2.12:0.6.0

spark.sql.extensions io.qbeast.spark.internal.QbeastSparkSessionExtension
spark.sql.catalog.spark_catalog io.qbeast.spark.internal.sources.catalog.QbeastCatalog

spark.hadoop.fs.s3a.access.key			<ACCESS KEY>
spark.hadoop.fs.s3a.secret.key			<SECRET KEY>
spark.hadoop.fs.s3.impl				org.apache.hadoop.fs.s3a.S3AFileSystem
spark.hadoop.hive.metastore.warehouse.dir	s3a://<BUCKET NAME>/

Attempt to create the bucket:
spark-sql (default)> CREATE TABLE delta.`s3a://spieg-qbeast/qbeast-table` (id INT, name VARCHAR(255)) USING qbeast;
[TABLE_OR_VIEW_ALREADY_EXISTS] Cannot create table or view `delta`.`s3a://spieg-qbeast/qbeast-table` because it already exists.
Choose a different name, drop or replace the existing object, or add the IF NOT EXISTS clause to tolerate pre-existing objects.

2. Branch and commit id:
0.6.0
3. Spark version:
3.5.1
4. Hadoop version:
3.3.4
5. How are you running Spark?
Local computer. Reproduced in Qbeast cloud
6. Stack trace:
[TABLE_OR_VIEW_ALREADY_EXISTS] Cannot create table or view `delta`.`s3a://spieg-qbeast/qbeast-table` because it already exists.
Choose a different name, drop or replace the existing object, or add the IF NOT EXISTS clause to tolerate pre-existing objects.

Thanks, @spiegela. It sounds like the table is already registered in the catalog you're using. Out of curiosity, not that I particularly think it is going to make a difference, have you tried using qbeast-spark 0.7.0?

Hi Aaron!
The proper way to create that table is to issue the following command:
CREATE TABLE  my_table (id INT, name STRING)
USING qbeast
LOCATION 's3a://spieg-qbeast/qbeast-table'
OPTIONS( 'columnsToIndex'='id,name')

I had to change the type to STRING and add the LOCATION and OPTIONS settings.
As far as I know, the syntax delta.s3://somewhere is used to delta table from the storage without registering it as a table in the catalog. For instance, you can run this:

SELECT * FROM delta.`s3a://spieg-qbeast/qbeast-table`

However, this syntax only works if you are using the DeltaCatalog, while for some reason, we have to investigate why it doesn't work with our catalog (also see syntax qbeast.s3/ doesn't work). I'll open an issue about this.

I've created issue #412 to take care of the lack of support of delta.` path` and qbeast.`path` syntax in QbeastCatalog.

Thanks @cugni. Now that I figured out, I had an obsolete jar in my include path, the USING qbeast LOCATION ... syntax is working fine. I'm GTG.


PHP functions prefixed with a slash are being ignored

I thought this issue was directly related to wp-cli and so I posted the original message there: https://github.com/wp-cli/i18n-command/issues/344
Basically when prefixing a PHP i18n with a slash it gets ignored.
e.g.
<?php
// wp i18n make-pot . --domain=bar
\esc_html__('foo', 'bar'); // ignored
esc_html__('baz', 'bar'); // processed

This is in Gettext 4, right?
I'm not working on maintaining this version (not enough time) but accept PR. If you want to work on that, I guess this can be fixed somewhere here: https://github.com/php-gettext/Gettext/blob/4.x/src/Utils/PhpFunctionsScanner.php#L117

Fixed and released in v4.8.8


Need to support spaces in file names in python wrapper

This seems to be fixed? Works in Windows with a couple of tests.

I thought the issue was on Macs?
On Tue, Feb 2, 2016 at 2:42 PM, GabeAl<EMAIL_ADDRESS>wrote:

This seems to be fixed? Works in Windows with a couple of tests.
—
Reply to this email directly or view it on GitHub
https://github.com/GabeAl/NINJA-OPS/issues/2#issuecomment-178810448.

Check out the pull request #1

FYI I fixed the white space issue last week.
Dan
--
Dan Knights, PhD
Assistant Professor
BioTechnology Institute, Computer Science and Engineering
University of Minnesota
http://www.knightslab.org
cell:<PHONE_NUMBER> (dials backwards)
office: MEMORIZE-PI<PHONE_NUMBER>)
lab: 612-5-MICROB<PHONE_NUMBER>)
On Tue, Feb 2, 2016 at 3:13 PM, Benjamin Hillmann<EMAIL_ADDRESS>wrote:

Check out the pull request #1 https://github.com/GabeAl/NINJA-OPS/pull/1
—
Reply to this email directly or view it on GitHub
https://github.com/GabeAl/NINJA-OPS/issues/2#issuecomment-178823187.

Nice, looks good!
On Tue, Feb 2, 2016 at 3:15 PM, danknights<EMAIL_ADDRESS>wrote:

FYI I fixed the white space issue last week.
Dan
--
Dan Knights, PhD
Assistant Professor
BioTechnology Institute, Computer Science and Engineering
University of Minnesota
http://www.knightslab.org
cell:<PHONE_NUMBER> (dials backwards)
office: MEMORIZE-PI<PHONE_NUMBER>)
lab: 612-5-MICROB<PHONE_NUMBER>)
On Tue, Feb 2, 2016 at 3:13 PM, Benjamin Hillmann <
<EMAIL_ADDRESS>wrote:

Check out the pull request #1 <
https://github.com/GabeAl/NINJA-OPS/pull/1>
—
Reply to this email directly or view it on GitHub
https://github.com/GabeAl/NINJA-OPS/issues/2#issuecomment-178823187.

—
Reply to this email directly or view it on GitHub
https://github.com/GabeAl/NINJA-OPS/issues/2#issuecomment-178824816.


[Bug]: Error when using data source snowflake_users

Terraform CLI Version
1.6.6
Terraform Provider Version
0.92.0
Terraform Configuration
data "snowflake_users" "all_users" {
  pattern = "%MATSMART%"
}

Category
category:data_source
Object type(s)
data_source:users
Expected Behavior
I expect the data source to return a list of users, since the there are users matching the pattern.
Actual Behavior
null is returned.
Steps to Reproduce
Snowflake version: 8.23.5

Create a user with the name<EMAIL_ADDRESS>Use the resource as specified above, it should return the user you just created

How much impact is this issue causing?
Medium
Logs
https://gist.github.com/calleo/9b37c61b37eb138c16518bbc19c50d86
Additional Information
Looks like a problem with parsing the output of SHOW USERS. I have tried executing the exact same SQL statement, with the same user and role, and can see that it actually returns users.
My initial thought was that the disabled column contained null values, but I can also see true/false.
Would you like to implement a fix?

[ ] Yeah, I'll take it 😎

Hey @calleo . Thanks for reaching out to us.
The user datasource will be redesigned as part of https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/ROADMAP.md#preparing-essential-ga-objects-for-the-provider-v1. I will link this issue to https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/v1-preparations/ESSENTIAL_GA_OBJECTS.MD. As a workaround, you can use https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/resources/unsafe_execute with query parameter.

I am having the same issue at the moment, this used to work up to one month ago from the date, I guess there were some changes on the API in Snowflake side

Closing the issue due to inactivity. Please create a new one if the issue persists in the newest version of the provider.


NNCP: call to_dict() if not self.res

Short description:
More details:
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for reviewer:
Bug:

/verified


IndexerHandler

I was getting There is no such indexer handler: none with Magento 2.4.3 during setup:install. This ought to fix it.

Thanks for your contribution, I'll create a new version 0.1.0 with dependency to magento/framework v103.0.3 to keep retro compatibility with previous versions.


Functions to perform forward pass on the test set, create a submission and score it.

The output of this function should be graphs embedded in a markdown file as describerd in the readme.md
A lot of this likely exists, you'd have to look into https://github.com/orcasound/aifororcas-livesystem/blob/main/ModelEvaluation/readme.md  and https://github.com/orcasound/aifororcas-livesystem/blob/main/ModelEvaluation/score.py
This "scoring" procedure will be used to create the body of the PR that we will eventually check in.
You'll like need to check in with Aayush for the forward inference part of this.

@prakruti I would like to take this issue.


Added SceneNet RGB-D

Hi, @danielnjoo, this qiuwch/synthetic-computer-vision is a fork from unrealcv/synthetic-computer-vision. Please submit your pull request to the original repo.
For this SceneNet paper, it has been included in the 2016 list.
Thanks.


[Enhancement] Better install.ps1 network checks

Hello again!
I've noticed a few PMAT students (particularly our analyst friends in countries other than the US) report install.ps1 failing network checks. The script currently doesn't give a lot of info about how the network checks failed. This PR seeks to provide more info so an analyst can troubleshoot more effectively.
All checks are still encapsulated so the -noChecks bypass will skip them:

Running without a custom config and all checks succeed:

Example of the ping check failing:

Erroring out on the GET request to GitHub:

When running install.ps1 while pulling a remote custom config file, the check will run for raw.githubusercontent.com:

Network checks are now encapsulated in a simple function, so if we want to add more of them (choco site, etc) it's as simple as:
    Test-WebConnection "google.com"
    Test-WebConnection "github.com"
    # Test-WebConnection "[any other URL we need]"

Apologies for the wacky rebase, my formatter went crazy on the rest of the file but I've restored it to its previous state! We should be good now.


Add progress/by-audible-id/:audibleId route and modify token middleware to allow Authorization: Bearer

Based on issue 637 this adds:

A route to allow adding progress to an item via just its audibleId (progress/by-audible-id/:audibleId). This will allow a hook integration with AudioBookShelf so that as content is played in that system, it can automatically update MediaTracker without having to know the itemId in MediaTracker.

A method to the MediaItemRepository to find an item by its audibleId (findByAudibleId)

A change to the token middleware so that the token can be passed in one of three ways (checked in this order):

in the header, via Authorization: Bearer   as is commonly used in JWT
in the header, via Access-Token:  as is done in the current implementation
as a query parameter ?token= as is done in the current implementation

Hope these changes can help add even more scrobbling to MediaTracker and extend its functionality for Audiobooks!

@marcousa Could you test it with the changes I have made

@bonukai Thank you! I can confirm everything works with the updates you made.
It's nice now that if the audibleId is not already in the local items it'll search for it via external sources.
One comment on that though, if its not in the local DB and it finds it externally, it's not adding it to the local so you can't see the progress until you take the extra step of searching for that item (and then adding it to one of the local lists).
Would it be possible to have it auto-add to in-progress if you just added progress to it via that API endpoint?
BTW, thanks for making the most comprehensive media tracking tool with scrobbling!!

Would it be possible to have it auto-add to in-progress if you just added progress to it via that API endpoint?

This already happens, but "Audiobooks" and "Progress" pages show only items on your watchlist, and those items weren't automatically added to watchlist.
Now I have amended this, after updating progress, mediaItem will be added to your watchlist.

@marcousa Does AudioBookShelf requires audibleId to be present in the path? Could we use instead an existing endpoint: /api/progress/by-external-id, with payload:
{
  "progress": "0.1",
  "mediaType": "audiobook",
  "id": {
    "audibleId": "B016N9U1XW"
  }
}

@bonukai I think Audiobookshelf could work with the existing endpoint, but the current /api/progress/by-external-id does not accept audibleId as an option.
I didn't want to mess too much with the internal logic of existing methods, so I had just added the direct api/progress/by-audible-id route

Thanks @bonukai! I can confirm it works with the new endpoint!

I created a socket.io based auto-updater for Audiobookshelf to feed MediaTracker.
GitHub Repo
It’s setup as a separate docker image right now but if you like it you can incorporate it into MediaTracker directly to allow for scrobbling.
as long as the user adds the ASIN to their Audiobookshelf item (can be done by matching an item to the metadata provider) then the auto-update will work.


Straw-man outline of ID obfuscation.

We use our IDs as a breadcrumb trail to allow us find objects easily without needing to maintain reverse mapping tables. However, this ID structure is not part of the protocol, and should not be depended on (or used at all) by clients as we might change this at any time. Also, other implementers should not receive the impression that this is the official or required structure for GA4GH IDs.
Therefore, I think we should obfuscate our ID values, so that the underlying structure at least isn't obvious. This should not be seen as a security measure, as it's not intended to make anything secret.
This is an initial straw man proposal where we base64 encode the IDs. For example, we have
an ID like dataset1:1kg-phase3 becoming ZGF0YXNldDE6MWtnLXBoYXNlMw==. Base64 seems like a good option as it's cheap and easily reversible, but I'm sure there are other options.
This breaks lots of tests, as we use this structure quite a lot in our own test cases, so I haven't bothered updating these until we have a consensus on whether this a good idea or not.

I guess I'm -0 on this.  I don't really see the need.

@diekhans, any thoughts on this? I'm worried that people will start writing conformance tests around our ID format, and this seems like a good way to ensure that clients don't start writing unportable code based on incorrect assumptions.

Seems like a great stop-gap measure until GA4GH gets somewhere
with identification.  We seem to have the mentality of
implementation by imitation rather than specification, so I
think this could making switching to GUIDs less of a shock.
Jerome Kelleher<EMAIL_ADDRESS>writes:

@diekhans, any thoughts on this? I'm worried that people will start writing
conformance tests around our ID format, and this seems like a good way to
ensure that clients don't start writing unportable code based on incorrect
assumptions.
—
Reply to this email directly or view it on GitHub.*

Closed in favour of #568.


Fixed bug when relative time should be applied automatically

#617
The bug happens when the automatic relative time is meant to be applied e.g.
secondary_info: "{last_changed}"
The blow was working fine:
secondary_info: "{last_changed|reltime()}"

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.7%) to 88.481%

Totals

Change from base Build<PHONE_NUMBER>:
0.7%

Covered Lines:
387

Relevant Lines:
422

💛  - Coveralls


possibility of overlapping the elements

Hello,
I would like to know if there is a way to overlay the elements?
Please only submit bugs to this tracker. If you have a feature request, please implement it. This is free software and the author is busy with other projects.
Please use the CodeSandbox Template to demonstrate your bug. It is much easier for us to help you if you do.

does anyone have an answer?

@Remy-J Are you finding for allowOverlap (example), which will be included in the next release?

@sehilyi  @STRML  it works thank you


Get rid of unused parameter in TestEventListenerBase::assertionEnded()

Just don't name this parameter to avoid warnings about it being unused.
No real changes.

:+1:

Nice and simple. :+1:

Thanks for catching that.
I couldn't merge the PR as it conflicts with the single_include header (in general please don't update the header in single_include as it is generated by splicing all the other headers together).
So I've just reapplied the same fix - will make it in to the develop branch soon,.


Remove .Web IVTs and refactor. Fixes #12713

This cleans up the layering a lot:

Removes the IVTs, obviously
Eliminates the whole concept of RendererRegistry from server-side Blazor, and moves it to become a WebAssembly-specific thing (because in server-side, there's always exactly one renderer per circuit, so we can already access it directly). This eliminates another [AsyncLocal] - there aren't many of them left!
Eliminates the weird indirection when dispatching events. Previously, CircuitHost would already know which renderer instance to use, but instead of calling it directly, called via a [JSInvokable] method on RendererRegistry. Now it just invokes directly.
Refactors how incoming event data is represented and parsed. It was pretty quirky before, with a static method that mutated an object you passed to it. Now it's a thing you construct (WebEventData) and nothing visible from outside gets mutated.
Removes the hardcoded guard clause from CircuitHost that would throw if you tried to invoke an event through JS interop. There's no longer any [JSInvokable] method for this in server-side Blazor, so the check is redundant.

Eliminates the whole concept of RendererRegistry from server-side Blazor, and moves it to become a WebAssembly-specific thing (because in server-side, there's always exactly one renderer per circuit, so we can already access it directly). This eliminates another [AsyncLocal] - there aren't many of them left!

YES!!
I was eyeing to do this in the following days. I'm also doing simplifications to the client side code to reflect this.

Do you think we could get rid of the JSRuntimeBase async local too?

I think that's a great thing to aim for. I agree it's separate from this PR though.


go release notes for the 2022-10 release

This is an automated pull request to gather the go release notes for the 2022-10 release. See Producing Release Notes for details of the processing.

armnetwork v2.0.1 will be in the 2022-11 notes


Compile fail on openSUSE Tumbleweed

Describe the bug
During command make build I get following error (2):
 nwg-dock git:(main) ✗ make build go build -o bin/nwg-dock . \# github.com/gotk3/gotk3/glib cgo-gcc-prolog: In function ‘_cgo_200740c87a77_Cfunc_g_binding_get_source’: cgo-gcc-prolog:71:2: warning: ‘g_binding_get_source’ is deprecated: Use 'g_binding_dup_source' instead [-Wdeprecated-declarations] In file included from /usr/include/glib-2.0/glib-object.h:24,                  from /usr/include/glib-2.0/gio/gioenums.h:30,                  from /usr/include/glib-2.0/gio/giotypes.h:30,                  from /usr/include/glib-2.0/gio/gio.h:28,                  from ../go/pkg/mod/github.com/gotk3/gotk3@v0.6.1/glib/gbinding.go:3: /usr/include/glib-2.0/gobject/gbinding.h:115:23: note: declared here   115 | GObject *             g_binding_get_source          (GBinding *binding);       |                       ^~~~~~~~~~~~~~~~~~~~ cgo-gcc-prolog: In function ‘_cgo_200740c87a77_Cfunc_g_binding_get_target’: cgo-gcc-prolog:107:2: warning: ‘g_binding_get_target’ is deprecated: Use 'g_binding_dup_target' instead [-Wdeprecated-declarations] /usr/include/glib-2.0/gobject/gbinding.h:119:23: note: declared here   119 | GObject *             g_binding_get_target          (GBinding *binding);       |                       ^~~~~~~~~~~~~~~~~~~~ \# github.com/nwg-piotr/nwg-dock ./tools.go:69:12: cannot use window.Change (variable of type sway.WindowEventChange) as type string in struct literal ./tools.go:87:33: cannot use eventHandler (variable of type swayEventHandler) as type sway.EventHandler in argument to sway.Subscribe: 	swayEventHandler does not implement sway.EventHandler (missing BarStateUpdate method) ./tools.go:117:33: cannot use eventHandler (variable of type swayEventHandler) as type sway.EventHandler in argument to sway.Subscribe: 	swayEventHandler does not implement sway.EventHandler (missing BarStateUpdate method) make: *** [Makefile:11: build] Error 2
To Reproduce
Steps to reproduce the behavior:

Clone repository
cd into repository folder
run command make get
run command make build
See the error

Expected behavior
I expect succesful build to proceed to running command sudo make install
Screenshots

Desktop (please complete the following information):

Linux distribution :  openSUSE Tumbleweed x86_64
go version : go version go1.19.3 linux/amd64
gtk-layer-shell version:  0.8.0-1.1

A temporary solution:
In go.mod, change the go version to 1.16, then go mod tidy.
On my machine at home it did build, but now at the office I'm getting into trouble, too. Will take a closer look when I'm back home.

Anyway, it needs to be fixed, together with updating the go-sway version. But I'm addicted to IDEs, and have none at the office. :D

Could you, please, try and build from the update-go-sway branch?

Excellent. I'll merge and release later tonight. Thanks for confirmation.


Logging fails if the caller does not supply all additional fields

If I define an additional field A, I can no longer issue a call like:
$logger->log($msg);

It will fail with SQLSTATE[HY093]: Invalid parameter number
I would have to always pass an extra array on all logging calls, like this:
$logger->log($msg, ['A'=>'something']);

Not all logging calls in an application can be made to always supply all the extra fields, sometimes that is not even under the control of the application developer (a third-party library may be making it's own logging calls, for instance).

You can solve this by adding the following code at line 130 of MySQLHandler.php:
$contentArray = $contentArray + array_combine (
    $this->additionalFields, array_fill (0, count ($this->additionalFields), null)
  );

Having the same issue when I define no extra columns:
Invalid parameter number

If I have only the default columns: channel, level, message, time it will do an array_merge on line 120 and add $record['context'] which results in the invalid parameter number.

Is this project still alive?

It is alive, yes. Well... at least I am :)  Going to look at this tomorrow, sorry for the very long delay

Could you please test latest master commit whether it fixes the problem?

Thank you, @waza-ari