Put all generated code in the build/ directory.

That way, the toplevel source is not clobbered with build
artifacts and it is possible to make a hermetic build.
Issues #484
Signed-off-by: Henner Zeller<EMAIL_ADDRESS>

This build cleanup is also a preparation to work on #484 and #483

The bulk of the preparation work for this change here was in #423

Looks like the msys2 build is failing due to some unrelated reason: compiler runs out of memory. Maybe because it needs to keep track of slightly longer pathes ?
Anyway, addressing the Serializer_save.cpp to be less intense on the compiler is a pull request for another time.

I'm getting this error pretty consistently after merging this:
[ 59%] Building CXX object third_party/UHDM/CMakeFiles/uhdm.dir/src/UHDM.capnp.c++.o
c++: error: /home/alain/Surelog/build/third_party/UHDM/src/UHDM.capnp.c++: No such file or directory
c++: fatal error: no input files
compilation terminated.
The only way to get rid of it is to remove the UHDM dir and recheckout

You're not supposed to put this in Surelog yet, I'll do that in a separate PR. This will not work yet the way Surelog accesses these.

Arrg, don't roll back, I have more to come.

Please resubmit (With a note - DO NOT MERGE IN SURELOG YET) :-)


IonicDB lose connection

Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
When you connect to no internet WIFI connection , and after that return to an Internet wifi connection. db.connect(); doesn´t work.
Expected behavior:
I expected reconnect to Ionicdb to get data from it
Steps to reproduce:
Inside the app, in a page with a list filled with a query to IonicDB.
Go to Wifi SETTINGS and choose a wifi without Internet connection
Wait 1-2 minutes
Inside the app try to call the process to fill the list. It will never work

Hello! Thanks for opening an issue with us! As this seems like more of a support question i will urge that you ask this question on our forum or on our slack channel. Thanks for using Ionic!


Add options to dynamically enable/disable widgets

Add a declarative config where you can enable/disable widgets.
This config should perhaps look something like this.
topbar_left_1="active_apps"

topbar_center_1="time"

topbar_right_1="battery"
topbar_right_2="bluetooth"
topbar_right_3="bluetooth"
topbar_right_4="wifi"
topbar_right_5="packages"
topbar_right_6="music"
topbar_right_7="info"
topbar_right_8="recoder"
topbar_right_9="search"

tagbar_1="tags"
tagbar_2="directories"
tagbar_3="trash"
tagbar_4="layouts"

notification_1="user"
notification_2="social"
notification_3="weather"
notification_4="covid"
notification_5="calculator"

This way people could easily disable widgets they are never going to use.
And who knows someday we might implement a GUI for the configurations ;)

This functionality is already present


Pass value to children on render

If a value is passed into the input mask it is not passed on to children on render, only on change. So you can't initialize with a value.

Sorry, didn't mean to actually open this yet, need to write tests.


Extension not working

ocamlmerlin error: File "_none_", line 1:
Error: I/O error: ../../bisect.exclude: No such file or directory

pwd
/home/pc/workarea/software/myproject

which ocamlmerlin
/home/pc/.opam/4.06.1/bin/ocamlmerlin

What exactly am I supposed to do about this? What's a bisect.exclude?

I get this exact error with the vscode-reasonml plugin. Any solutions? I can provide more info for reproducing the error.


icon nowhere to be found in maker-wix typedocs documentation

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
6.0.0-beta.69
Electron version
latest
Operating system
all
Last known working Electron Forge version
No response
Expected behavior
title
Actual behavior
n/a
Steps to reproduce
try and find icon in electron-wix-msi documentation
Additional information
No response

Resolved in beta.70: https://js.electronforge.io/interfaces/_electron_forge_maker_wix.MakerWixConfig.html


Update README.md

Point to the right sbt command to launch server

Indeed, however I will need to change it a bit again after because, it's actually entering the sbt console then (missing atm) enter run in it
thanks!


HELM-175: Use hasOwnProperty instead of _.has to improve performance

JIRA: https://issues.opennms.org/browse/HELM-175
With these changes processing 5000 alarms with 100 event parameters each
goes from taking 67631ms to 799ms (85x faster).

From > a minute to < a second, that’s pretty good! :)

LGTM.
In general we should probably be trying harder to avoid using lodash (which is huge and hard to tree-shake) in favor of using newer builtin JavaScript features.  Babel already contains performant polyfills if we need to be more backwards-compatible.


Maintain project on minecraft.curseforge.com as well

It would be a nice piece of infrastructure that would your users to stay up to date with the latest Big Reactors using the Curse Client. It has such things now, you can create your own modpacks and auto-update them. It would also reduce usage of outdated BigReactors versions.

Probably not going to do this. I've got big-reactors.com nicely automated, and I honestly CBA (a) checking to see if CurseForge has a compatible API and (b) integrating with it.

It's actually pretty easy to automate. ForgeGradle has a built-in task for uploading artifacts to Curse.
see: https://github.com/MinecraftForge/ForgeGradle/wiki/CurseForge-Publishing

@erogenousbeef Would you mind opening this again because something new was offered by @octarine-noise?

I'll take a look, at least.

Any progress?
Would really like to add Big Reactor to my own private profile


Cannot find module 'anime'

When requiring animejs using the require syntax available in the npm documentation (https://www.npmjs.com/package/animejs), I get the following error:

Cannot find module 'anime'

That is when using:
require('anime')

This has been updated on npm


Read mirror auth credentials from the target namespace

We don't know if we want this yet but for sake of not losing it I am opening a PR
Instead of having a central place where we configure the mirror registry we allow now users to configure only the mirror registry URL and when importing images we attempt to read secrets for the configured registry from within the target namespace (target namespace is where the are going to store the Image object).

/hold awaiting for the EP approval

Replaced by https://github.com/shipwright-io/image/pull/21


File Search Result Order Is Unintuitive

When searching for a file with CMD + P I would expect results with exact filenames to be more relevant.

Duplicate of https://github.com/atom/fuzzy-finder/issues/21.


chore(main): release 1.1.0

:robot: I have created a release beep boop
1.1.0 (2024-03-22)
Features

testing Conventional Commit and please-release (3822281)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/CoreySpohn/lod_unit/releases/tag/v1.1.0 :sunflower:


[Bug]: Action redesign: Focus shifts to another tab when renaming a JSObject

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When attempting to rename a JSObject, the selected JSObject does not enter edit mode. Instead, the focus shifts to a different tab, usually the first open tab.
Steps To Reproduce

Have multiple JS Objects open.
Now try to rename the last open tab.

Expected: The name of the JSObject in the currently active tab should become editable.
Public Sample App
No response
Environment
Production
Severity
Medium (Frustrating UX)
Issue video log
https://jam.dev/c/eb44179a-8e43-4f9a-b44b-ab19ff1c5334
Version
Cloud

Verified to be working as expected in release: https://jam.dev/c/a493b1b8-c2ba-4a33-ad87-b431f4664d2d


Unit Collision

Hey there!
Maybe take a look into the Combat Collision by now into comparision with the siege-footage from a few years ago.
Discussed here: https://www.reddit.com/r/MB2Bannerlord/comments/fvfe0c/tavern_maiden_be_thicc_amarite/

Did you post the wrong link?

Apologies!
https://www.reddit.com/r/MB2Bannerlord/comments/fvawli/the_big_problem_with_combat_unit_collision_and/
If it's just shrunken hitboxes to let taleworlds boss's lil brat swim faster through friendly troops to make moshingmoshing, could be fixed rather quickly, but I'm obviously not at all of an expert.

I definitely noticed this as well, the difference is jarring compared to the footage we were shown before release. This would be a must have fix for me if it is possible!

Happy that I'm not the only one bothered here; this reminds me a lot of Rome II, where unti spacing mods alreadt went to considerable length regarding pacing and survivability.
The more I studied the footages along MB;B2 development cycle, the more I can place it happening somewhere in early 2019 / late 2018, not that it helps in any way.
That said, careful benchmarking test on custom combat show an exisiting hitbox despite obvious cliping; looks like a vertical cylinder that would roughly be the circumference of the unit "waist belt", so there's much denser "blobbing" but stilL unit spacing. might be an investigation route to follow.
Also, there seems to be NO mass / friction at all at the moment, it look like AI is "pathing away" to avoid collision with an obstacle (obstacle being player or the unit having the most movement speed); so this might be a bit more complex to look for.

Just to update the thread, someone made a mod with regard to this issue:
https://www.nexusmods.com/mountandblade2bannerlord/mods/254
this does impact unit collision and unit speed, and therefore increase "mass"/"intertia" effect; but still lack some global 'friction' and 'pathing out' solution.


UX | Select and edit Adaptive Cards, to optimize and enhance bot responses

Managed LG | UX LG card builder
3. Composer enables me to select and edit Adaptive Cards, to optimize and enhance bot responses and answers for channels that support this (Teams, Webchat etc). (P1)

It will be great if we could embed adaptive card designer into Composer


Implement FtpDataType in DownloadFile Method

Hello,
Would you consider implement a DownloadFile method with a "FtpDataType" as a parameter ?
Like this for example : DownloadFile(string localPath, string remotePath, FtpDataType type, bool overwrite = true);
Let me Explain : the method .OpenRead(...) is deprecated and we should use .DownloadFile(..) but, at the moment, we can't download files with 0 lenght with the method .DownloadFile(..).
Thank you for your response and your work,
Regards,

Continue using OpenRead since I have removed the Obsolete tag (will take effect next build). I will try to add the arg you requested.

@Misaow Thanks for you post, I'm having the same issue here. Did you find a way to download files with 0 length using OpenRead() ???

What do you need to happen when a file is zero-length? Would you like 0 bytes to be written to the stream? Currently I'm throwing an exception for 0 length files.

@hgupta9 : When a file is zero lenght (in my case some files are 0 lenght but contains text) i need a way to download these files. I opened this issue because the OpenRead Method is gonna be deprecated and then we'll have no way to download these files.
I put an example below to show you how i download these files at the moment.
@DJ-Cur  :  This is an example with DownloadFile method client1DownloadFile(client1.OpenRead(item.FullName, FtpDataType.Binary), outputPath);
This way you can download 0 lenght.

@hgupta9 : I'm having the same issue when downloading a textbased file (.txt, xml, etc) I get the same error telling me to try running the application using SetDataType() and FtpDataType.Binary. That can only work using OpenRead(), but for some reason when I use OpenRead() the stream I get has a length of 0 (But I know the file contains data). However I can still read this stream using StreamReader() but I'm not able to use system.IO.FileStream because a requirement is stream.length which is 0.
(Maybe relevant info: The server I'm downloading from supports binary transfer mode only)
Code:
Stream Stream = client.OpenRead(string path);
FileStream fileStream = System.IO.File.Create(fileFullPath, (int)stream.Length); //(stream.length is 0, exception location)
   // Fill the bytes[] array with the stream data
    byte[] bytesInStream = new byte[stream.Length];
    stream.Read(bytesInStream, 0, (int)bytesInStream.Length);

    // Use FileStream object to write to the specified file
    fileStream.Write(bytesInStream, 0, bytesInStream.Length);

Maybe I'm doing something wrong, feel free to let me know.
@Misaow: I'm not sure that would work, assuming u meant FtpClient.DownloadFile() method, because the syntax is:
public bool DownloadFile(Stream outStream, string remotePath);
public bool DownloadFile(out byte[] outBytes, string remotePath);
public bool DownloadFile(string localPath, string remotePath, bool overwrite = true);
The outputPath will not work at remotePath

@DJ-Cur : Oh my Bad ! I'm sorry for my mistake. I just checked my project and in my case it work with the UploadFile method.
I didn't find a way to solve this actually.

@hgupta9 : I edited my previous comments

Your error is something else (you just want to download a 0 length file) and you are asking for something else (FtpDataType). I don't think FtpDataType is required. I will just check and let you know if a 0 length file can be downloaded.

@DJ-Cur - Thanks for reporting this error. I'll work on it today.

I've released a version which supports uploading/downloaded of zero-length files. Can you see if that works for you?
https://www.nuget.org/packages/FluentFTP/16.2.2

@hgupta9 Thanks for the fast reply and new release. I've tried the new solution but unfortunately its downloading the files but they do not contain data. In other words they are all empty text files even though I'm sure the files contain data(text).
The only way I got it to work is with this work around (which I'm not proud of haha)
Stream DownloadStream = client.OpenRead("RemotePath", FtpDataType.Binary);
StreamReader sr = new StreamReader(DownloadStream);
string file = sr.ReadToEnd();
File.WriteAllText("LocalPath", file);
For some reason this works fine, the System.IO.StreamReader seams to read these 0 Length files with no problem. But its not the best way to do it so I'm still looking for a better solution for this problem.

@DJ-Cur
I really don't understand your use case. Are you trying to download a file which has data and its downloading as zero length with DownloadFile()?? If so, then I think the error is that the server is not reporting the length of the file. Therefore my binary based reading does not work. Your code is not checking the length, you are simply reading until a NULL is encountered, which would work in some cases. Maybe I can add an option in DownloadFile() saying readUptoNull or readZeroLenFiles or something? What could be a simple name for this option, and how would it work? Only for certain file extensions? What is a sensible config? Because if I do this for all files, it could hang up in ReadToEnd method.
@Misaow
Can you check the new version and see if it works for your use-case?

@hgupta9 Exactly you are correct in stating that the server is not reporting the length of the file. (see the FTPTrace below) in this test I've tried it with an .exe file. When I try the download with DownloadFile FluentFTP requests SIZE  . the server then returns with: 502 Command  not implemented.
TYPE I
200 Type set to I.
SIZE curgasone.exe
502 Command  not implemented.
PORT 10,10,10,36,209,203
200 Port command successful.
RETR curgasone.exe
150 Opening BINARY mode data connection.
550 RETR: Transfer failed.
When I download with Core FTP LE the trace looks like this:
TYPE I
200 Type set to I.
PORT 10,10,10,99,43,183
200 Port command successful.
STOR curgasone.exe
150 STOR: Command accepted!
226 Transfer Complete
I think ReadToEnd will only work with text based files (In my case it will work but I'm not sure I can download anything else)
Thanks for your fast reply, I hope this info. helps

@DJ-Cur - I don't understand the Core FTP logs - the STOR command is used to save a file to the server, not to download it. (ref 1, ref 2) The RETR command should been in the logs, which is used to download a file. Can you download a file with Core FTP and send those logs?

Yes you are correct, here it is
TYPE I
200 Type set to I.
PORT 10,10,10,99,44,5
200 Port command successful.
RETR z_TestFile5.xlsx
150 Opening BINARY mode data connection.
226 Transfer complete.
z_TestFile5.xlsx - 6179 bytes transferred in 1 seconds
Transfer time: 00:00:01

According to the FTP docs, your code is correct!

After sending the RETR request and reading a mark, the client reads data from the data connection until EOF (end of file), or a TCP error (for example, TCP reset or network unreachable), or timeout. The client saves the data, remembers whether there was a FIN, and closes the data connection.

So the "read upto EOF" is basically a what ReadToEnd() does. I think I'll add an edge case handling for it. If the length of the file is reporting 0 by the server, then we will use this method.

@hgupta9 : Works for me, Well done !

I've tried to implement a fix for you @DJ-Cur but before I publish it I need to confirm if everything is fine. Can you check this DLL and see if it works for your strange server?
https://we.tl/C3j9D3cWzq

@hgupta9 Great Work! Works good with my strange server :) 👍

Fantastic! This fix will be available in the next build I push to nuget.


feat(security): build spire server and agent from the source in Dockfile

Closes: #3943
Signed-off-by: Jim Wang<EMAIL_ADDRESS>

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x ] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x ] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[ ] I have fully tested (add details below) this the new feature or bug fix (if not, why?) tested on amd64 build, Bryon can help testing on arm64
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

New Dependency Instructions (If applicable)

Build has docker lint warnings.


When staging a directory, ignored files are also staged.

The behavior when staging a directory entry from the status page differs from that of git add <directory>, which recursively stages the contents, but does not stage ignored files.
When staging a directory in GitSavvy's status page, ignored files get staged too. They correctly disappear from the list when unstaged.
Note that git add will also respect nested .gitignore files, scoped to their containing directory.

At present, GitSavvy uses the -f flag when staging files.  Trying to remember why this was originally.  I'll go ahead and change this in master - if nothing unexpected comes from that change, it'll be included in the next release.

Seems the only use of -f is being able to add ignored files. I can see cases where this would be useful, but I remember being able to stage the file simply invoking "Git: add" from the command palette in SublimeGit, which worked well considering how rarely I had needed to do that. So that may still warrant the -f flag, I think?
Anyway, thanks a lot for the fix!

Worked perfectly, thanks!


[BLAS:hipSYCL] Fix build issue

Description
As reported in #434 CUDA backend is broken when used with hipSYCL. This PR enables it using LLVM-17 and latest branch from AdaptiveCPP (new name for hipSYCL).
Fixes # (GitHub issue)
#434
Checklist
All Submissions

[X] Do all unit tests pass locally? Please see attached files:
hipsycl_cuda.log
dpcpp_intel.log

[X] Have you formatted the code using clang-format?

Changes look good to me, thanks for them.
Should we (eventually, maybe in a different PR) change the name of hipsycl in our files to reflect the new name of the project?

We should. We need to update CI system as well. I would wait until the new name becomes more permanent though. (it was called open SYCL first and then changed to adaptiveCPP)


👌 IMPROVE: add use client directive to support Next.js apps

I've made an improvement to add "use client" directive on the top of ScrollSpy component.
This help Next.js users to use and import this component directly from a Server Component.
Rollup by default remove any directive added, so we can add a plugin called rollup-plugin-preserve-directives as described here https://github.com/rollup/rollup/issues/4699.
I've updated the demo-app with the new version. I made some tests on the demo-app without any issues.
Let me know if there is something I missed in the update!

Validate by me


Unable to get completions

Hi. I'm using vim-dadbod and this plugin (through coc-db), and I'm not able to get completion candidates through the plugin. Dadbod works fine for executing queries. How can I debug this?
I'm setting the variable b:db after the buffer loads using dadbod, not through an autocmd. Could that be the reason?
Thanks in advance.

Problem was that cache wasn't refreshed after you set the variable. Do an update and give it another test, should be working fine now.


SOLR-17257: Avoid instantiating NodeStateProvider several times when computing replica placement

https://issues.apache.org/jira/browse/SOLR-17257

Description
When using either the Minimize Cores or the Affinity placement strategy with Solr 9, positioning a new replica becomes very inefficient past a certain collection count.
At Salesforce, we are operating clusters packed with tens of thousands of collection, and each replica placement operation takes several seconds to complete at that scale.
This pull request aims at bringing a first simple change to improve the performance of replica placement when operating with a large number of collections. It will eventually be followed by additional changes as we are progressing towards solutions for addressing such scalability issues.
Solution
When fetching the cluster's metrics and information needed to compute the optimal position of the replica to be positioned, we are calling a seemingly harmless cloudManager.getNodeStateProvider() method 2 * number_of_nodes times in AttributeFetcherImpl.java#L144-L209.
However, this method instantiates a SolrClientNodeStateProvider instance every time it is called, which performs an expensive operation within its constructor to build a nodeVsCollectionVsShardVsReplicaInfo data structure, filled with information about all existing collections and replicas.
  protected final Map<String, Map<String, Map<String, List<Replica>>>>
      nodeVsCollectionVsShardVsReplicaInfo = new HashMap<>();

  public SolrClientNodeStateProvider(CloudLegacySolrClient solrClient) {
    this.solrClient = solrClient;
    try {
      readReplicaDetails();
    } catch (IOException e) {
      throw new SolrException(SolrException.ErrorCode.SERVER_ERROR, e);
    }
  }

  protected void readReplicaDetails() throws IOException {
    ClusterStateProvider clusterStateProvider = getClusterStateProvider();
    ClusterState clusterState = clusterStateProvider.getClusterState();
    if (clusterState == null) { // zkStateReader still initializing
      return;
    }
    Map<String, ClusterState.CollectionRef> all =
        clusterStateProvider.getClusterState().getCollectionStates();
    all.forEach(
        (collName, ref) -> {
          DocCollection coll = ref.get();
          if (coll == null) return;
          coll.forEachReplica(
              (shard, replica) -> {
                Map<String, Map<String, List<Replica>>> nodeData =
                    nodeVsCollectionVsShardVsReplicaInfo.computeIfAbsent(
                        replica.getNodeName(), k -> new HashMap<>());
                Map<String, List<Replica>> collData =
                    nodeData.computeIfAbsent(collName, k -> new HashMap<>());
                List<Replica> replicas = collData.computeIfAbsent(shard, k -> new ArrayList<>());
                replicas.add((Replica) replica.clone());
              });
        });
  }

This operation has been observed to take several seconds, even on clusters with a very moderate amount of small collections (~1000).
The simple change proposed in this pull request will reduce the number of times this expensive operation is performed for each replica placement operation from 2 * number_of_nodes to only once.
While the underlying algorithm will in all likelihood require further optimizations to provide acceptable execution times on large clusters with tens of thousands collections, we will already observe a noticeable improvement with this first fix.
Tests
Changes have been validated as part of pre-production tests at Salesforce.
Checklist
Please review the following and check all that apply:

[X] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[X] I have created a Jira issue and added the issue ID to my pull request title.
[X] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[X] I have developed this patch against the main branch.
[X] I have run ./gradlew check.
[ ] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

Nice!


added 'public' to many methods so I can extend the Fetcher class outside of the Haneke module

@szehnder Thanks for catching this! I'm not keen on making public most of these, but the Fetcher class should definitely be. If you change this I can merge this PR. Otherwise I'll close and add the public myself later.

Awesome, yeah I was moving fast.  I sincerely appreciate your careful response and I'll make the modifications you've pointed out.  Shouldn't take me but a few mins.


feat: implement credentialsets per spec section 802

This provides a minimal implementation of the credentialset format.
Depends on #23

Needs rebasing, but not sure why github hasn't realised.

@glyn - most likely, GitHub didn't realise it needs a rebase because there's a branch protection setting to only allow up-to-date branches to be merged, which I just enabled.

Is there something left that I need to do on this?


docs(metarepos): add specification to management RPCs

What this PR does
This PR adds specifications to "proto/mrpb/management.proto". This helps us
establish agreements for the contracts of the metadata repository's management
RPCs.
Which issue(s) this PR resolves
Update: #658

Current dependencies on/for this PR:

main

PR #687   👈

This stack of pull requests is managed by Graphite.


Kill / remove kernel option

From jupyter notebook extension, I have seen that it is possible to change kernel. I was wondering if it is possible to kill and remove unused kernel instead of just restarting them? This would be useful when multiple kernel are created and unused in the same session.

Thanks for the suggestion. Kernels should be killed when:

You close VS code
You close the notebook/interactive window that had the kernel selected.

Do you need another way to kill a kernel? Maybe while VS code is still open?

I am actually using a remote server for Jupyter Notebook.
So when I close VsCode, kernels are usually still runing which is great when I let something run for hours.
But because of this, sometime I have a lot of kernels that I don't use but are still possible to chose from the list of kernel. It would be great to have an option to remove them if possible!

Duplicate of https://github.com/microsoft/vscode-jupyter/issues/1379


Fix line length to pass style tests

Codecov Report

Merging #91 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #91   +/-   ##
=======================================
  Coverage   97.23%   97.23%
=======================================
  Files          27       27
  Lines         723      723
=======================================
  Hits          703      703
  Misses         20       20

Impacted Files
Coverage Δ

...ions_ui/controllers/reporting_units_controllers.py
100% <100%> (ø)
: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 618aebd...f5d69ba. Read the comment docs.


Trigger zoom and pan events on the map not just layers

We do trigger pan and zoom on the map object.  mymap.geoOn('geo_pan', function (evt) { ... })  works as expected.  src/event.js claims we don't trigger zoom events on the map itself, but this is untrue (we do trigger them).


Fix: MTB project debugger

Update the paths in Cypress MTB project .launch files to have the Debugger work.

Description

Checklist:

[x] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Problem has been fixed in another PR. Close this PR.


NSG rules clarification

Traffic from the node CIDR to the node CIDR on all ports and protocols
Traffic from the node CIDR to the pod CIDR on all ports and protocols (required for service traffic routing)
Traffic from the pod CIDR to the pod CIDR on all ports and protocols (required for pod to pod and pod to service traffic, including DNS)
The documentation indicates that the following NSG rules are necessary for Azure CNI Overlay. However, it's not entirely clear whether both inbound and outbound rules are required for each of these rules. Could the document be updated to provide clarification on this matter?

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

ID: 64f0ba71-573f-b860-1c2c-802a62cae120
Version Independent ID: 28990db8-fce7-c356-da6f-c052a9c201ee
Content: Configure Azure CNI Overlay networking in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/azure-cni-overlay.md
Service: azure-kubernetes-service
Sub-service: aks-networking
GitHub Login: @asudbring
Microsoft Alias: allensu

@asudbring
Can you please check and add your comments on this doc update request as applicable.

Adding a note here - we actually had a situation where we saw that having the rules configured only inbound meant that pod to pod traffic across nodes did not work, so we needed to mirror the rules on the NSG both inbound and outbound and then the issue was fully resolved.

#please-reassign:asudbring

Hello, I also think we need to whitelist access from podCIDR to serviceCIDR as well. In my aks cluster, here is a network profile.
        "networkProfile": {
            "networkPlugin": "azure",
            "networkPluginMode": "overlay",
            "networkPolicy": "azure",
            "networkDataplane": "azure",
            "loadBalancerSku": "Standard",
            "loadBalancerProfile": {
                "managedOutboundIPs": {
                    "count": 1
                },
                "effectiveOutboundIPs": [
                    {
                        "id": "/subscriptions/7e8acd11-c0c8-4434-8456-4708d04dbe33/resourceGroups/MC_JP-rg-JPE-DEV-KIT-shingotoda_JP-aks-JPE-DEV-KIT-poc-shingo-1_japaneast/providers/Microsoft.Network/publicIPAddresses/fee76508-fbea-45a9-936b-f28bffebd8df"
                    }
                ],
                "backendPoolType": "nodeIPConfiguration"
            },
            "podCidr": "<IP_ADDRESS>/16",
            "serviceCidr": "<IP_ADDRESS>/16",
            "dnsServiceIP": "<IP_ADDRESS>",
            "outboundType": "loadBalancer",
            "podCidrs": [
                "<IP_ADDRESS>/16"
            ],
            "serviceCidrs": [
                "<IP_ADDRESS>/16"
            ],
            "ipFamilies": [
                "IPv4"
            ]
        },

In the above example, where pod has <IP_ADDRESS> and DNS server is <IP_ADDRESS>, then DNS query from pod cannot  reach to the DNS server unless there is NSG rule that allows traffic from pod CIDR to service CIDR.

Thank you for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner, and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary.
#please-close


05/10/2018 18:39 : Create Pull Request for branch master -> prTesting1_2018051018

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

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


Clean up SWHS example - Unique Chunks

By "clean up" I mean removing as much duplication as we can, creating new combinators, etc. It will be an ongoing task over the summer, but we should get the first pass done soon.

Specific goal of this pass: remove all duplication of concepts so that each comes from a single unique source.

Duplication of concepts have been removed as they can be found in Unitals. We might be able to close this issue.

I'm still seeing a bit of duplication, not necessarily of chunks, but things like phase_change_material = dcc "pcm" (nounPhraseSP "Phase Change Material") ... and phsChgMtrl  = commonINP "phsChgMtrl" (pn' "Phase Change Material") "PCM" are both using different noun phrases for the same term.

How would I fix that, because they're two different types, so when called upon, they may need to be a certain type right?

There's a couple of different things going on here. First off, nounPhraseSP "Phase Change Material" and pn' "Phase Change Material" are saying the same thing (yes, with different types), except the first states that it's a self-plural noun phrase, whereas the second states it's a proper noun with a plural created by adding 's'.
These should be unified into one NounPhrase and used as appropriate.
phase_change_material should be able to be constructed from phsChgMtrl by using the dcc smart constructor as it has it there, the main difference being that it should look more like:
phase_change_material = dcc "pcm" phsChgMtrl ...

I'm getting an error:
Couldn't match expected type ‘NP’ with actual type ‘CINP’

Try using (^. term) on phsChgMtrl

So it's fine to have both now since the one is sort of built of the other?

Yes, since everything is coming from one source and we're essentially just "tacking on" more (necessary) information. This might change after we look over and re-inventory the current data structures we're using, but for now it's not an issue.

Alright, so I got it to work, so we're good on that specific example. I'll clean up a bit more with the progName and swhsFull things.

All instances of term definition duplication cleared as of commit da09cf60b79d63f0e824cc18f06050d52ad5d683.
NOTE: The method of doing so feels like a hack but functionally works.


Stop KDE from producing negative numbers

This closes https://github.com/quantified-uncertainty/squiggle/issues/2591

I'm going to merge this, as I really need to start using it.


cant change spawn config

hey guys. i want to change wither witches spawn rate. its 5 and i want to make it 20, but when i do it, in core xml, it automaticly changes itself to 5 again. is there a way to prevent that?

You need to make the changes in the user XML file
On 27 Feb 2015 20:49, "dmtryzhkv"<EMAIL_ADDRESS>wrote:

hey guys. i want to change wither witches spawn rate. its 5 and i want to
make it 20, but when i do it, in core xml, it automaticly changes itself to
5 again. is there a way to prevent that?
—
Reply to this email directly or view it on GitHub
https://github.com/CrazyPants/EnderZoo/issues/64.

can you lead me what change should i do in that file?

I will update the user file with some examples


Try to improve ccache usage Android

I think ccache does not work well the way we are using it now. The render test app takes over an hour to build.
I hope this improves it.

I see no explicit reason why this would help. I suspect the main build is just not using ccache correctly. Although now I see that some cache was built. Maybe we retry the pipeline and see if it's much faster now.

Maybe we retry the pipeline and see if it's much faster now.

That's the plan :)

It's 15 minutes quicker the second, but could be a fluke.

2h 25m 34s vs 75 minutes.
So I think it does something.


ion-slides use zoom="true" report errors

Resources:
Before submitting an issue, please consult our troubleshooting guide (http://ionicframework.com/docs/troubleshooting/) and developer resources (http://ionicframework.com/docs/developer-resources/)
Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
<ion-slides zoom="true">  use zoom report errors.
The swiper official website says that if zoom is enabled, you need to add class= "swiper-zoom-container". Sorry, my English is weak
Expected behavior:

Steps to reproduce:

Related code:
insert any relevant code here

Other information:
[]
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
cli packages: (C:\Users\Administrator\AppData\Roaming\npm\node_modules)

    @ionic/cli-utils  : 1.10.2
    ionic (Ionic CLI) : 3.10.3

local packages:

    @ionic/app-scripts : 2.1.3
    Ionic Framework    : ionic-angular 3.6.1

System:

    Node : v6.11.0
    npm  : 3.10.10
    OS   : Windows 7

Hello! Thank you for opening an issue with us! Would you be able to provide a sample application via GitHub that demonstrates the issue you are having? Thanks for using Ionic!

Same problem here, if you are try to enable zoom on ion-slides components and have another element of img, svg, ion-img or canvas, swiper-gesture crash. But if you have one of this elements, the zoom not working.
Try next code, and you can check pinch zoom-in or zoom-out:
<ion-slides zoom>
    <ion-slide>

    </ion-slide>
  </ion-slides>
`

Can you fix? Thanks.

Hi @kensodemann
It is very easy to reproduce this issue. Install the latest version of ionic:
npm install -g ionic@latest
Then
ionic start slideTest blank
In home.html add this code:
 <ion-slides zoom="true">
    <ion-slide>

Slide 1

    </ion-slide>
    <ion-slide>

Slide 2

    </ion-slide>
    <ion-slide>

Slide 3

    </ion-slide>
  </ion-slides>

Ionic serve
If you try to zoom in your device you will get the error.
In desktop you have to make double click inside content slide.
Tested using Chrome for Android and Desktop.

pablo@pablo-Inspiron-7520:~/projects/slideTest$ ionic info

cli packages: (/usr/lib/node_modules)

    @ionic/cli-utils  : 1.10.2
    ionic (Ionic CLI) : 3.10.3

local packages:

    @ionic/app-scripts : 2.1.4
    Ionic Framework    : ionic-angular 3.6.1

System:

    Node : v7.10.0
    npm  : 3.10.10
    OS   : Linux 4.4

Thank you. I have used your code to duplicate this issue.

@pablo-gonzalez - your sample code has zoom true but no images. The swiper does not support that. Do you have a use case for not having images on these pages? (not that zooming would work, but just wanted to check)
@gabfiocchi - if you add a zoom container per the swiper docs, then your example works just fine.
    <ion-slide>

    </ion-slide>

If you implemented safe guards when trying to make zoom without image it seems good to me.

That is all experimental code. I suppose you could fork that branch if you want, but I wouldn't. I doubt it will get merged given the amount of rework being done in this area in v4, which will use the swiper more directly.
Plus, the problem isn't really the lack of safeguards. Everything works fine with the existing code if you follow the swiper API docs, and one could argue that at least if you do things wrong it crashes where-as with my code in place it would just fail to zoom.
You can do an image and text in each slide and it will still work just fine. As an example
    <ion-slide>

      <ion-label>I am a test app</ion-label>
    </ion-slide>

So just use the current version (3.6.1) and make sure you have an image and a wrapper, and you will be fine.
Updating the documentation to include that you need to have an image on each slide and that each slide should be contained with a swiper-zoom-container is a good idea.

What happened to me is that I just put the previous code taken from Ionic documentation for testing and zoom="true" and once in the desktop browser I got the error doing double click on the slide content.
Updating the documentation would be great to avoid bad using of the component. I was not aware that zoom only worked with images and I needed to add a container and a class as additional steps.
Thank you for your support.
Best regards,
Pablo


Use public registry alias

Before: https://gallery.ecr.aws/v6m6o3k4/elm
After: https://gallery.ecr.aws/acilearning/elm

[ ] I manually tested the code locally to make sure that it works.
[ ] If there is documentation, I made sure it's accurate and up to date.
[ ] If possible, I wrote automated tests for the new behavior.

This is similar to https://github.com/acilearning/docker-haskell/pull/11.


[WIP] Add in-repo-engine blueprint

I wanted to pitch in by adding some blueprints to help people get started using this addon.
Currently, this just uses the in-repo-addon blueprint and adds a addon/engine.js file. Other features I think this should include:

[x] option to generate a routable or route-less engine

[x] routable engines should include addon/routes.js
[x] routable engines should modify router.js
[x] routeless engines should include templates/application.hbs

@trentmwillis seems like a good start - thanks!
I agree that routable vs route-less would be good options. The former could create routes.js and the latter could create templates/application.hbs.

@dgeb do you know of the best way to accomplish that? As far as my understanding of blueprints goes, there isn't really a way to specify optional files. My thought is to just generate both and then remove the unused one in the afterInstall hook.

You can implement filePaths and have separate sets of files for each, based on an option.
See https://github.com/ember-cli/ember-cli/pull/4837 and https://github.com/offirgolan/ember-cp-validations/pull/8 for some background.

@rwjblue awesome! Exactly what is needed. Will update this later today.

@rwjblue @dgeb I think this is ready for actual review now. There's a good bit of relying on behavior from other blueprints which I am unsure if this is the best approach or not. There's also a TODO that I'll probably open an issue for on Ember-CLI, but otherwise it works as expected.

That TODO is exactly what https://github.com/ember-cli/ember-cli/issues/5481 is about.  I'd love help fixing that upstream (I believe I left a comment on potential direction in that issue).

This LGTM, I'd like to address https://github.com/dgeb/ember-engines/issues/64 in general but that can happen after we land this if you'd prefer.
@dgeb - Any objections?

Thanks @trentmwillis!


Change metadata to see exponent operation instead of multiplication

Suppose it's typo, that is written 2*x instead of 2^x.


Remove type parameters from Service and use HttpService and RpcService?

As @anuraaga said, some static newDecorator method's return type is
Function<lots and lots of stuff, with even more stuff>.
Perhaps, we can simplify that by removing the type parameters which represent the types of Request and Response and use HttpService and RpcService.

Hello, I'm working on this :-)

Please make sure to propose your change with examples before starting to make real changes since this is non-trivial.

@trustin Sure, I will keep in mind.

After talking with @trustin @ikhoon, I found that this issue needs more discussion.
So, I stop working on this issue. Thank you.

Fixed via #2254


Fix race condition in registry/session

LGTM (janky will be happy!)

LGTM

do we need this in the bump branch

i assume yes


Deprecation warning on Rails 6

It seems this gem is using some code that is deprecated in Rails 6. I get this when I ran an rspec with this gem loaded:
DEPRECATION WARNING: Initialization autoloaded the constants ActionText::ContentHelper and ActionText::TagHelper.

Being able to do this is deprecated. Autoloading during initialization is going
to be an error condition in future versions of Rails.

Reloading does not reboot the application, and therefore code executed during
initialization does not run again. So, if you reload ActionText::ContentHelper, for example,
the expected changes won't be reflected in that stale Module object.

These autoloaded constants have been unloaded.

Please, check the "Autoloading and Reloading Constants" guide for solutions.

See https://github.com/rails/rails/issues/36546 for more.

Hello @heelhook,
I successfully started and synced a Rails 6 project with the last gem version and I was not able to get this deprecation warning.
Also, I'm able to execute the test suite of this gem on Rails 6.0 without any warning.
Someone posted this comment yesterday, maybe it would fix your rspec issue? https://github.com/rails/rails/issues/36546#issuecomment-663930482
It seems however that this line could create some issues, we'll investigate, thank you!
https://github.com/translation/rails/blob/93a663c0afbf2fe24b82b3ee80f454afb719d486/lib/translation_io/railtie.rb#L11

The deprecation warnings should be fixed thanks to this: https://github.com/translation/rails/commit/fdae89e72c62c516b403c00a905e80a79347c2ea
I bumped the gem version to 1.22, could you please execute bundle update translation and check that the warnings are correctly fixed?
Thank you for your feedback!

Thanks! That took care of it 🚀

🍾
Thank you for your help!


Implement simple setting workflow

Implement a simple setting of parameter values workflow (e.g., submission of a file with quantum efficiency values by a user).
The workflow includes:

submission of data and metadata by a user through an interface: simplified first approach is that user submits the metadata in yaml format and the data in ecsv format.
user metadata and data is validated against a reference schema validation
add workflow related metadata
apply necessary data validation and transformation checks
output data and metadata to be submitted to MC model database

This workflow and example metadata schema can be found here

Update on the implementation, which needs a lot more refinement. @orelgueta - please comment.
General approach:

Implementation in gammasim-tools is a generic setting_from_external workflow (and not specific for quantum efficiency tables) -  renamed the application to ./applications/set_modelparameter_from_external.py

The configuration specific to the QE table setting is encoding in a configuration file (example here)

This configuration can be extended by e.g., listing what kind of plotting should be done for a quick check during the setting

The user input consist of metadata (this is what the user would e.g., type into a web interface) and the data tables.

User input metadata is validated against a reference schema: validate_schema

User input data file is validated in the following:

check that all required axes exist
check that axes units are of same type and convert to unit given in config file (e.g., wavelength can be given in Angstrom, but will be converted to nm)
check axes ranges to inside the allowed range and cover at least the required range
validate_data

Complete the metadata with all the required fields as listed here

this is where the code is most incomplete; need to revisit the metadata discussion to make this consistent.

Write two files which should be ready for the submission to the database: a metadata file and a data file

Unit tests are certainly incomplete.

Random questions:

is there a way to retrieve in the code the gammasim-tools software version? Use same approach as ctapipe
What file name should the output model name get? I think at this point we almost always used the same name as received from the telescope teams (e.g., qe_R12992-100-05c.dat). Should that be a user input?
Not sure at all about the python file name. Should the file name be the class name? At this point it is not.

Generally it all sounds very good.

renamed the application to ./applications/set_modelparameter_from_external.py

Shouldn't there be an underscore between model and parameter?

is there a way to retrieve in the code the gammasim-tools software version? Use same approach as ctapipe

As far as I know, we need to implement the conda/pip installation for that to work. I don't think it can be done without it.

What file name should the output model name get? I think at this point we almost always used the same name as received from the telescope teams (e.g., qe_R12992-100-05c.dat). Should that be a user input?

Let's make it a user input, with the original name used as default. We should also add a check that the name does not already exist in the DB. I can add a function to the DB handler API for it if you think it's a good idea.

Not sure at all about the python file name. Should the file name be the class name? At this point it is not.

The python file name is not exactly the class name. The class names are written in PascalCase while the file names are in snake_case. The actual name should be similar, but doesn't have to be the same. Would be good to keep the name as informative as possible.

Last update is the implementation of the PRODUCT:ASSOCIATION field:

a list of telescopes for which the simulation model file can be used for
PRODUCT:ASSOCIATION:ID is derived using the functionality in names.py

The two missing items before issuing a pull request now is to improve the data validation and expand the examples of data transformations:

remove duplicates
introduce sorting order (e.g., that wavelengths are always increasing)

Anything else we want to include into this first version? @orelgueta

No, I think that will already be enough.
The only thing we might want to do already is check if the file name is already in the DB. We can wait with it for the second version, just like with the telescope IDs, but I don't mind implementing it now if you wish.

I would prefer to have anything with data base in a second version.

OK!


Add option to disallow trailing data

Tar files end with two zero blocks where a header would be expected. When we encounter this, the tar reader automatically closes itself and cancels the underlying stream subscription.
As we found out in https://github.com/dart-lang/pub/pull/2935, this can cause issues for users expecting the reader to always read the stream fully. There could be an option to asset that no further data is received at the end of a tar stream. When that option is enabled, we could wait for further data or the end of the stream and throw an error in the first case.

@isoos In the pub server, you're only reading tar streams from memory or a file, right? The reader used to always cancel the underlying stream subscription when encountering an end-of-archive marker. Cancelling response streams had unintended side-effects in the http client, I'm worried something similar might happen when cancelling the request stream in the server. If you fully buffered the stream before there's no concern here.
I've released version 0.3.1 of the tar package which adds a parameter to not cancel the subscription but throw when there's unexpected data after a marker indicating the end of an archive.

Yeah, pub-dev is reading from file (in prod) or from memory (in tests). Thanks for the update, I'll add a test for that. If I understand this properly, this will throw if I append an extra byte to any valid package archive. Am I right with that?

If I understand this properly, this will throw if I append an extra byte to any valid package archive. Am I right with that?

Yes, exactly. Note that throwing will still cancel the stream subscription:
https://github.com/simolus3/tar/blob/fbe1b9465ace95a525023260c5bc57722f659e6e/test/reader_test.dart#L134-L144
If you're only reading from files or from memory, it's not necessary to set the disallowTrailingData flag, since there's no harm when cancelling a stream (the file descriptor will be closed either way). For http clients it resets the TCP connection which was a real issue.
I also haven't checked if there are any existing packages on pub.dev that might break with disallowTrailingData since this is kind of a pedantic requirement for tar files. @jonasfj Suggested that such check could make sense for new packages though, if that's the only place where tar files are decoded.

@simolus3 could you please help me out what the problem may be? Is it that we have a tar archive with bad content, or maybe some edge case in the code?
import 'dart:io';

import 'package:http/http.dart';
import 'package:tar/tar.dart';

Future<void> main() async {
  final rs = await get(Uri.parse(
      'https://pub.dartlang.org/packages/pana/versions/0.12.19.tar.gz'));
  final uncompressed = gzip.decode(rs.bodyBytes);
  // works
  print('default...');
  await _read(TarReader(Stream.fromIterable([uncompressed])));

  // fails
  print('disallowTrailingData...');
  await _read(TarReader(
    Stream.fromIterable([uncompressed]),
    disallowTrailingData: true,
  ));
}

Future<void> _read(TarReader reader) async {
  while (await reader.moveNext()) {
    print('- ${reader.current.name}');
  }
}

It looks like the tar file has a lot more than two empty blocks at the end
cat reference/pub/pana-0.12.19.tar.gz | gzip -d | tail -c 6500 | hexdump -C

00000000  20 20 20 20 65 78 70 65  63 74 28 6e 6f 6e 41 73  |    expect(nonAs|
00000010  63 69 69 52 75 6e 65 52  61 74 69 6f 28 27 20 20  |ciiRuneRatio('  |
00000020  5c 74 5c 6e 5c 72 27 29  2c 20 30 2e 30 29 3b 0a  |\t\n\r'), 0.0);.|
00000030  20 20 20 20 7d 29 3b 0a  0a 20 20 20 20 74 65 73  |    });..    tes|
00000040  74 28 27 61 73 63 69 69  20 74 65 78 74 27 2c 20  |t('ascii text', |
00000050  28 29 20 7b 0a 20 20 20  20 20 20 65 78 70 65 63  |() {.      expec|
00000060  74 28 6e 6f 6e 41 73 63  69 69 52 75 6e 65 52 61  |t(nonAsciiRuneRa|
00000070  74 69 6f 28 27 61 27 29  2c 20 30 2e 30 29 3b 0a  |tio('a'), 0.0);.|
00000080  20 20 20 20 20 20 65 78  70 65 63 74 28 6e 6f 6e  |      expect(non|
00000090  41 73 63 69 69 52 75 6e  65 52 61 74 69 6f 28 27  |AsciiRuneRatio('|
000000a0  61 20 62 20 63 27 29 2c  20 30 2e 30 29 3b 0a 20  |a b c'), 0.0);. |
000000b0  20 20 20 7d 29 3b 0a 0a  20 20 20 20 74 65 73 74  |   });..    test|
000000c0  28 27 6e 6f 6e 2d 61 73  63 69 69 20 74 65 78 74  |('non-ascii text|
000000d0  27 2c 20 28 29 20 7b 0a  20 20 20 20 20 20 65 78  |', () {.      ex|
000000e0  70 65 63 74 28 6e 6f 6e  41 73 63 69 69 52 75 6e  |pect(nonAsciiRun|
000000f0  65 52 61 74 69 6f 28 27  e5 b0 81 e8 a3 85 68 74  |eRatio('......ht|
00000100  74 70 e4 b8 9a e5 8a a1  e6 8e a5 e5 8f a3 27 29  |tp............')|
00000110  2c 20 30 2e 36 29 3b 0a  20 20 20 20 7d 29 3b 0a  |, 0.6);.    });.|
00000120  20 20 7d 29 3b 0a 7d 0a  00 00 00 00 00 00 00 00  |  });.}.........|
00000130  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00  |................|
*
00001960  00 00 00 00                                       |....|

I think we can allow more than two zero blocks at the end, I'll change the implementation.

Thanks again for the repro. I've fixed this in 1aa9e4d4ae7aa6f3e6640db17018f75f8d87fbea and subsequently published version 0.3.2 with the fix.

@simolus3 Fantastic, thank you. Filed https://github.com/dart-lang/pub-dev/pull/4719 as the first step to check the extra bytes in pub.dev.


ARROW-2614: Remove 'group: deprecated' in Travis

Was this doing anything? This must have been some kind of copy-pasta foul

This pinned us to an older Travis image. I think we had some problems months ago with a newly released image. These problems are already fixed since a while but we forgot to remove the pinning.


Create toolchain.md

@rotycenh

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/cloud-adoption/governance/security-management/toolchain.md
:white_check_mark:Succeeded
View

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


Complete the Scraping Notebook

Steps:

Go to 'scraping notebook.ipynb'
Complete unfinished code and Save it as "Scrape_Data.csv"

Hey! I would like to work on this issue. Kindly assign it to me. Thank you.

Great to see you here @Sanjana-Srinidhi. I've assigned the issue to you. Good Luck.

Please assign the issue to me.


Added support for running Jenkins with https

Hello @emmetog,
I'd like to propose adding support for https, using the modifications in my fork. Please let me know what you think.
Regards,
Graham.

Hi @grahamdaley, thanks for this PR!
Here's an alternative: what if we use a reverse proxy like jwilder/nginx-proxy or traefik to handle the SSL and then forward the request to jenkins over HTTP? I prefer this way because it gives a clear separation of concerns: the protocol and other "transport related" stuff is handled by the proxy. Breaking down the responsibilities like this also keeps each part lean, this way jenkins doesn't need to know about HTTPS and other projects like nginx-proxy and traefik are incredibly powerful and flexible when dealing with this stuff since that's what they're built for.
What do you think about adding a section to the README explaining how to set up HTTPS by adding a reverse proxy, something like this:

HTTPS
If you want to enable HTTPS on jenkins we recommend that you use a reverse proxy like jwilder/nginx-proxy or traefik and configure it as the HTTPS endpoint instead of configuring jenkins itself with HTTPS. This gives you more flexibility and better separation of concerns. See the documentation in those projects for more details on how to deploy the proxies.

I would be interested in a PR that does not globally publish the ports on the jenkins container though, since if we are using a reverse proxy we don't need (or want) Jenkins receiving requests on a different port and bypassing the proxy. We could have a jenkins_publish_ports: true/false flag or something similar.
What do you think?

Hi @emmetog, thanks for the comments. I can see where you're coming from.
In my particular usage scenario (which may be a bit different to most users), I am aiming to script the setup of a Jenkins instance on a Google Cloud Engine VM, which will be exposed more or less unrestricted, to the Internet. We have a small number of team members who would be accessing it, from various locations around the world.
My original plan was to terminate the incoming SSL connections on a Google Cloud load balancer, but it seems the current Python API is not really able to do this just yet. I suppose I could just go ahead and pull an nginx container into the same instance and use that to terminate SSL too – it wouldn't be a big deal. It just seemed a bit pointless considering that Jenkins already had this built in, and it wasn't difficult to expose it through a small modification to your Ansible role.
Having thought about it a bit further though, I can see some benefits of going with your approach of having a reverse proxy to handle SSL. For example, Jenkins does not seem to expose any parameters related to SSL other than the certificate and key file paths. This would make it difficult to change the encryption algorithms used, which is necessary whenever one of them eventually becomes compromised. So we'd be depending on the Jenkins container maintainers to do that for us.
So yes, OK... you've talked me into it!


Update default splitting behavior to get fewer, larger, splits

Before this change, the portable Java splitting of BoundedSource was capped at 64k. This matches the Python behavior, which has some fairly reasonable default split sizes that seem to work well in practice.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

CC @bhisevishal

R: @robertwb

Run Java PreCommit

Run Java PreCommit

Run Java_GCP_IO_Direct PreCommit

GCP IO is perma-red due to firestore - looks like it went red without code changes so probably our project got messed up

Hmm this appears to be a common flake.


Add 'internal' flag for NetworkImpl

The current Network definition has the following fields:
private final String name = UUID.randomUUID().toString();
private Boolean enableIpv6;
private String driver;
@Singular private Set<Consumer<CreateNetworkCmd>> createNetworkCmdModifiers;
private String id;
private final AtomicBoolean initialized = new AtomicBoolean();

Could an additional field be added: private Boolean internal; which has its corresponding value effect the final CreateNetworkCmd which is used in the create() method:
private String create() {
       // ..... snipped

       // add the following...
       if(internal != null) {
            createNetworkCmd.withInternal(internal);
       }

       // ... snipped
 }

This will allow us to easily create internal networks.

Given the presence of createNetworkCmdModifiers and that the internal flag is rarely used, I'd say we should not overcomplicate our public API. @rnorth @kiview WDYT?

I'll close this issue because you are right - it is very easy to use createNetworkCmdModifiers like so to achieve the same effect:
private Network generateInternalNetwork() {
    Consumer<CreateNetworkCmd> cmdModifier = (createNetworkCmd) -> {
        createNetworkCmd.withInternal(true);
    };

    return Network.builder()
            .createNetworkCmdModifier(cmdModifier)
            .build();
}


Render a "generated file" warning header

I've been looking through the Template Resources documents, and I don't see this feature.  Is it possible (easy) to add a warning header on the resulting rendered file, to tell others that any changes to the file will be overwritten by confd?
Here's the general idea I had:
/etc/confd/conf.d/haproxy.toml
[template]
src="haproxy.cfg.tmpl"
header="haproxy.cfg.header"
  ...

And /etc/confd/templates/haproxy.cfg.header could be a plain text with with something like:
##############################################################
###   WARNING:  Do not edit this file, changes will be
###      overwritten by confd!
###      Make changes to /etc/confd/templates/haproxy.cfg.tmpl
##############################################################

Thank you!

@calh You can do this inside the template. Checkout the docs on how to add a timestamp to the the comment as well:
https://github.com/kelseyhightower/confd/blob/master/docs/templates.md#datetime


Add Support for Horizontal Bar Chart in fl_chart (Enhancement #113)

Description
This pull request implements a simple solution for enhancing the fl_chart library to support horizontal bar charts.

Issue #113: Enhancement to support horizontal bar chart.

Changes Made

Added functionality to create horizontal bar charts.
Updated examples to demonstrate the new horizontal bar chart implementation.
Ensured that the chart adapts well to different data sets and configurations.

Related Issues

Closes #113

I just want to send a little thank you for horizontal bar charts - so cool :)

Hi @MrRoy121, thanks for contributing this!
We merged an update last week with #1793 that made some changes to bar_chart.dart. Can you please resolve the issues in that file?
Also, there seem to be some formatting changes that are unrelated to this change. It would make reviewing easier if those were made in a separate refactoring PR. Can you revert them in this PR?

Hi @MrRoy121
Thanks for your contribution, but please make sure to read our contribution guidelines.
You have to update the documentation, you need to write some tests.
So, I had planned to implement the horizontal bar chart feature in these days. So you can just ignore it and I will take care of this feature.
Cheers!


Customizable port for daemon mode

If you want this, drop a comment below.

This would be very helpful to customize both the internal and external ports of the program instead of only the external in a scenario such as multi-containers and docker-compose.

v3.4.0


ci: add dependabot gh action

Description
We are using some gh actions & npm dependencies in this project. Usually, we can upgrade all gh actions & dependencies manually but if we add this gh action it will create PR if there are any updates needed. We don't need to worry about it. Just we have to review it & merge it.
I have added a demo screenshot of my repository.
Screenshots

🥦 Browser
Other
Checklist ✅

[X] I checked and didn't find similar issue
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (blank for no)

Suppose Depandabot suggests updating versions, what if the version then instantly breaks production?
I'm not sure TBH, can you help me out ?

Suppose Depandabot suggests updating versions, what if the version then instantly breaks production?
I'm not sure TBH, can you help me out ?

If Depandabot suggests updating a dependency to a new version and the update causes production to break immediately then you can rollback to the previous version


How can I utilize the weights of pre-trained PixArt-α to initialize the parameters of the spatial Transformer block in the Latte T2V model?

How can I utilize the weights of pre-trained PixArt-α to initialize the parameters of the spatial Transformer block in the Latte T2V model.

You can use the weights of pre-trained PixArt-α to initialize this part while making the other part random initialization.

You can use the weights of pre-trained PixArt-α to initialize this part while making the other part random initialization.

Thanks for your reply. I will try it.


a question on async: true (not really an issue)

App.Post = DS.Model.extend({
  comments: DS.hasMany('comment', { async: true })
});

if I left the {async: true} out

App.Post = DS.Model.extend({
  comments: DS.hasMany('comment')
});

how will the adapter behave? It looks like it will load all comments while fetching the post "at the same time", correct?

As you found out, async: false requires the server to sideload the related records into the main payload. This is not something that firebase is capable of doing, thus, we only support async: true or embedded: true


Question security question

I love you git
Its possible to add a boolean for se security question
Thansk :))

Sorry, your request is out of the scope of the library. The project is only meant to be used for account authentication, not for account management.


Update logging and finalize v1.1

Now uses "timewsync" logger instead of root
All prints are going to logger now
Added basic debug messages
Bumped version to 1.1

Thanks! Also, could you rebase your commits so that the Merge remote-tracking branch 'origin/main' into development commits are not included? That would make the commit history look a bit nicer.

That was a good idea! Initially I didn't know you how to do it, but now it looks way better!


Change Oculus to use official ES ruby lib

The rubberband ES library which Oculus uses is now deprecated, change Oculus to use officially supported ES ruby library

Moreover elasticsearch 1.0 renamed a vital API entry point "_cluster/nodes" to just "_nodes" which required me to patch rubberband installation on the spot (the newest rubberband version still uses "_cluster/nodes").


Game crashes on AddComponent method.

What is the game you are using with IL2CPP Resolver?
SCP:SL
Details:
I am trying to add UnityEngine.Light to the Game Object named "Default Camera", but the game crashes on line m_pObject->AddComponent(ptr);.
Also, if I comment out m_pObject->AddComponent(ptr); line, auto m_pObject = Unity::GameObject::Find("Default Camera"); seems to drop my fps from 200+ to 20(if I iterate over gameobjects manually fps doesn't decrease)
Screenshots/Video:
No response
Minimal, Complete and Verifiable Example code:
void ourmain() {
    Unity::il2cppClass* cptr = IL2CPP::Class::Find("UnityEngine.Light");
    Unity::il2cppObject* ptr = nullptr;
    if (cptr) {
       ptr = Unity::Object::New(cptr);
    }
    auto m_pObject = Unity::GameObject::Find("Default Camera");
    if (m_pObject && ptr && cptr) {
        m_pObject->AddComponent(ptr);
    }
}

In main:
IL2CPP::Initialize();
IL2CPP::Callback::Initialize();
IL2CPP::Callback::OnLateUpdate::Add(ourmain);

Thank you for your answer! Could you please clarify what you mean by "You're supposed to pass il2cppObject as type"?

Thank you for your answer! Could you please clarify what you mean by "You're supposed to pass il2cppObject as type"?

AddComponent takes il2cppObject pointer and the argument is called "SystemType" so you will need to use this to get the type:
https://github.com/sneakyevil/IL2CPP_Resolver/blob/9cb0706360c46516a4fd8c6578d1d603ca90b90a/API/Class.hpp#L50

Thank you, I rewrited it, but the game freezes in 5-10 seconds after I add component even if I do it only one time.
void ourmain() {
    static int counter = 0;
    std::string m_sObjectSubstring = "Default Camera";
    Unity::il2cppArray<Unity::CGameObject*>* m_pObjects = Unity::Object::FindObjectsOfType<Unity::CGameObject>(UNITY_GAMEOBJECT_CLASS);
    for (uintptr_t u = 0U; m_pObjects->m_uMaxLength > u; ++u) {
        Unity::CGameObject* m_pObject = m_pObjects->At(u);
        if (!m_pObject) continue;
        auto ptr = m_pObject->GetName();
        if (ptr) {
            std::string m_sObjectName = ptr->ToString();
            if (m_sObjectName.find(m_sObjectSubstring) != std::string::npos) {
                if (counter == 0) {
                    counter++;
                    Unity::il2cppClass* cptr = IL2CPP::Class::Find("UnityEngine.Light");
                    m_pObject->AddComponent(IL2CPP::Class::GetSystemType(cptr));
                    //auto pt = m_pObject->GetComponent("UnityEngine.Light");
                    //pt->SetPropertyValue("intensity", 10.f);
                }
            }

        }
    }
}


docker-compose.yml example

As it took me quite a while to figure out the permission stuff how to run systemd_exporter with the docker image (https://hub.docker.com/r/prometheuscommunity/systemd-exporter), I would just like to document my solution here.
Typical errors were:
caller=systemd.go:247 level=error msg="error collecting metrics" err="couldn't get dbus connection: read unix @->/run/systemd/private: read: connection reset by peer"

caller=systemd.go:247 level=error msg="error collecting metrics" err="couldn't get dbus connection: write unix @->/run/systemd/private: write: broken pipe"

caller=collector.go:169 level=error msg="collector failed" name=systemd err="couldn't get dbus connection: dial unix /var/run/dbus/system_bus_socket: connect: no such file or directory"

couldn't get dbus connection: write unix @: sendmsg: broken pipe

version: '3.2'
services:
  node-exporter:
    container_name: systemd-exporter
    image: prometheuscommunity/systemd-exporter:v0.5.0
    command:
      - "--log.level=debug"
      - "--path.procfs=/host/proc"
      - "--web.listen-address=<IP_ADDRESS>:9558"
      - "--systemd.collector.private"  # Use systemd socket instead of DBus
    network_mode: "host"  # Does not need port specification therefore

    # We need either "privileged: true" (DBus connection) or "user: root" (systemd socket connection). Pick your poison.
    #privileged: true  # Needed to connect to DBus.
    user: "root"  # Needed to connect to systemd socket.

    pid: "host"
    volumes:
      - type: "bind"
        source: "/proc"
        target: "/host/proc"
        read_only: true
      - type: "bind"
        source: "/run/systemd"
        target: "/run/systemd"
        read_only: true
      - type: "bind"
        source: "/var/run"
        target: "/var/run"
        read_only: true
      - type: "bind"
        source: "/sys/fs/cgroup"
        target: "/sys/fs/cgroup"
        read_only: true
    restart: unless-stopped

I'm not sure though if really all that binds are needed or could be at least a bit more specific (e.g. only /var/run/systemd/private).

This has been immensely helpful, thank you so much! Would be great if the maintainers looked this over for correctness, and added something to the examples...


Game crashes upon loading level.

Hi there, our game crashes upon trying to load a level. It doesn't crash when the plugin is disabled. I'm using the marketplace build. The plugin worked fine until recently. This was on UE 4.27.
The log is as follows:
"LogPlayLevel: Error: [2022.01.24-02.41.07:457][727]LogThreadingWindows: Error: Runnable thread RHIThread crashed.
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: begin: stack for UAT
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: === Critical error: ===
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error:
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: Fatal error!
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error:
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: Unhandled Exception: EXCEPTION_ACCESS_VIOLATION reading address 0xffffffffffffffff
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error:
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: [Callstack] 0x00007ff7e4d7f15c Mirem.exe!FD3D12CommandContext::SetRenderTargets() []
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: [Callstack] 0x00007ff7e4dffab4 Mirem.exe!FD3D12CommandContextBase::RHIBeginDrawingViewport() []
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: [Callstack] 0x00007ff7e66e32f7 Mirem.exe!FRHICommand<FRHICommandBeginDrawingViewport,FRHICommandBeginDrawingViewportString1871>::ExecuteAndDestruct() []
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: [Callstack] 0x00007ff7e66e6193 Mirem.exe!FRHICommandListExecutor::ExecuteInner_DoExecute() []
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: [Callstack] 0x00007ff7e66d627f Mirem.exe!FExecuteRHIThreadTask::DoTask() []
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: [Callstack] 0x00007ff7e66e8164 Mirem.exe!TGraphTask::ExecuteTask() []
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: [Callstack] 0x00007ff7e3a9515b Mirem.exe!FNamedTaskThread::ProcessTasksNamedThread() []
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: [Callstack] 0x00007ff7e3a955be Mirem.exe!FNamedTaskThread::ProcessTasksUntilQuit() []
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: [Callstack] 0x00007ff7e67a69ad Mirem.exe!FRHIThread::Run() []
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: [Callstack] 0x00007ff7e3fb298f Mirem.exe!FRunnableThreadWin::Run() []
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: [Callstack] 0x00007ff7e3fa8ea0 Mirem.exe!FRunnableThreadWin::GuardedRun() []
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: [Callstack] 0x00007ff9758a7034 KERNEL32.DLL!UnknownFunction []
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: [Callstack] 0x00007ff9765c2651 ntdll.dll!UnknownFunction []
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error:
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: Crash in runnable thread RHIThread
LogPlayLevel: Error: [2022.01.24-02.41.07:458][727]LogWindows: Error: end: stack for UAT
LogPlayLevel: Error: ERROR: Client exited with error code: 3"

UPDATE: Tried the github branch and I get the same problem. It works fine when loading the main menu but when trying to load any other level it crashes with Unreal stating "fatal error."

I don't think this relates to the Async Loading Screen plugin. Try to disable the plugin and Launch again to see if there is any crash. Also, delete Binaries/Interrmeridate/Saved folders then try again.

I don't think this relates to the Async Loading Screen plugin. Try to disable the plugin and Launch again to see if there is any crash. Also, delete Binaries/Interrmeridate/Saved folders then try again.

I already tried that. The game loads correctly without the plugin. It is 100% a problem with the plugin.

UPDATE: Just to be thorough, I did delete the Binaries, Intermediate and Saved folders and I still get the same problem.

Can you run this plugin in a new project on your PC to see if this bug happens again? Sorry, I can't help you if I can't reproduce this issue on my side. As I've researched, it may relate to unsupported renderer API. Try to change to Vulkan, DirectX11, OpenGL.

I had the same problem


ora.v4.a: error adding symbols: Archive has no index; run ranlib to add one

error :
C:\Users\amira\Documents\GoProjects\TestDBAccess\src\C>gcc -shared -pthread -o testdb.dll dummy.c testdb.a ora.v4.a num.a date.a  -lWinMM -lntdll -lWS2_32
ora.v4.a: error adding symbols: Archive has no index; run ranlib to add one
collect2.exe: error: ld returned 1 exit status
I'm using CGO buildmodes for the purposes of generating a C program that integrates our GO libraries that in turn leverage  database/sql.
Is this a matter where there are some directives in the oci8.pc file that could be included that would generate the needed index ? btw, running ranlib had no effect.
Not a C guy, new to GO , thank you for your help.

Neither a C guy...
AFAIK Oracle does not provide static lib -  I don't know whether this is a
problem, just a fact.
gopkg.in/goracle.v2 uses ODPI-C, includes that, and that uses dl to load
the Oracle client dynamically at runtime.
Maybe that'd help.
amiracam<EMAIL_ADDRESS>ezt írta (időpont: 2017. szept. 14.,
Cs, 19:05):

error :
C:\Users\amira\Documents\GoProjects\TestDBAccess\src\C>gcc -shared
-pthread -o testdb.dll dummy.c testdb.a ora.v4.a num.a date.a -lWinMM
-lntdll -lWS2_32
ora.v4.a: error adding symbols: Archive has no index; run ranlib to add one
collect2.exe: error: ld returned 1 exit status
I'm using CGO buildmodes for the purposes of generating a C program that
integrates our GO libraries that in turn leverage database/sql.
Is this a matter where there are some directives in the oci8.pc file that
could be included that would generate the needed index ? btw, running
ranlib had no effect.
Not a C guy, new to GO , thank you for your help.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rana/ora/issues/220, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPoSpdwYNCQ8RLhBofXZ8JJ0nGoFo8Eks5siVzBgaJpZM4PX5V2
.

and so besides that , what are the differences between that driver and the rana/ora ? thanks

goracle.v2 is a pure database/sql driver - as with Go 1.9, we no longer need the driver-specific extensions as in ora.v4 (Ses).
The other difference is that goracle.v2 uses ODPI-C, an easier-to-use wrapper for OCI. A side effect is that you no longer need Instant Client for development, only for running, as ODPI loads the libclntsh.so at runtime.

and I can confirm that it works in Windows well. To elaborate we NOT just "using" goracle in a normal GO app context i.e. what we are doing is porting our ProC code to use GO based DB connectivity by using CGO so that we can from our C code call the CGO/database/sql libs instead of Oracle's embedded SQL (ProC) thus allowing us to have DB independence


Update fs2-core to 3.0.1

Updates co.fs2:fs2-core from 2.5.4 to 3.0.1.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

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

labels: library-update, semver-major

old PR


Add sonarcloud

Add sonarcloud to CI

sonarcloud automatic analysis is already enabled, no need for the changes in this PR


Duplicate explicit target name on links with same name

Hi,
(Bit of a novice in Sphinx, not completely sure the issue is in sphinx_mdinclude)
Description
I'm adding a README.md with the following contents to my Sphinx documentation:
# Title
Example 1 occurs ...
See example [here](https://github.com/amyreese/sphinx-mdinclude/blob/main/requirements-dev.txt).

Example 2 occurs ...
See example [here](https://github.com/amyreese/sphinx-mdinclude/blob/main/requirements.txt).

When building the docs I get the following:
$ sphinx-build -E -W -c ./docs/source/ -b html ./docs/source/ ./docs/build/
Running Sphinx v5.1.1
building [mo]: targets for 0 po files that are out of date
building [html]: targets for 9 source files that are out of date
updating environment: [new config] 9 added, 0 changed, 0 removed
reading sources... [100%] tests

Warning, treated as error:
README.md:3:Duplicate explicit target name: "here".

Chaging here to be unique no longer shows the issue so I might take that approach.
Details

OS: Ubuntu 20.04.5 LTS
Python version: Python 3.8.10
Project version: Sphinx==5.1.1, sphinx-mdinclude==0.5.2
Can you repro on main? Didn't reproduced on main
Can you repro in a clean virtualenv? Yes

this might be as simple as keeping track of how many links were seen with that target name, and automatically adding a -X suffix for subsequent links, such as here-2 for your specific example. Would need to make sure it only happens for links where both the target name and URL are included.


Cannot set SameSite property of ApplicationCookie

Issue / Steps to reproduce the problem
I have the following code in ConfigureServices(...), in Startup.cs, in a .NET 5.0 MVC application:
services.ConfigureApplicationCookie(options =>
{
options.Cookie.IsEssential = true;
options.Cookie.SameSite = SameSiteMode.Strict;
});
But after login, the server issued a cookie with SameSite=None. I see the SameSite property is overridden here:
https://github.com/IdentityServer/IdentityServer4/blob/07574678bed9e499b190d0b000b182323eb89566/src/AspNetIdentity/src/IdentityServerBuilderExtensions.cs#L51
and related to issue #2595 External authentication failing on iOS 12
Actual behavior:
The change restricted the SameSite to None so I can never set it to Strict or Lax.
Expected behavior:
It should be possible to set SameSite to Strict or Lax.

Have you read this on how to control the cookie settings?
https://identityserver4.readthedocs.io/en/latest/topics/signin.html#overriding-cookie-handler-configuration

Thanks for pointing to the documents. I'm able to change SameSite after I moved the code below services.AddIdentityServer(...).


[NEW ICON] Pinia

Skill Name
Pinia
Why?
The new official store library for Vue. I think Vue as a developer will be interested
Reference Image

Not exactly a specific skill, I think vue would cover this


KOGITO-7663: Fix inconsistency in filename template description and default value

Jira: https://issues.redhat.com/projects/KOGITO/issues/KOGITO-7663

This pattern is inherited from the DMN and BPMN extensions, that made the generated SVG compatible with Business Central.
I'm +1 to change it for SWF, though.
cc @ederign

Hi @zdrapela before merging this, we need to double check if this is not someone hardcoded on dev UI. @Sara4994 could you please confirm?

@ederign just checked quarkus dev UI, it is not hardcoded.

@zdrapela confilicts, could you please report a follow-up JIRA to add some sort of integration tests into VSCode suite for the SVG gen?

@domhanak It was some issue with prettier, but now all checks are OK.
Now I found out, that README mentions ${fileBasenameNoExtension}-svg.svg as a default value (instead of current default ${fileBasename}.svg). BPMN and DMN also use ${fileBasenameNoExtension}-svg.svg, so it might be better to go that way.

Deferring to @ederign.

@paulovmr would mind to take a look ?

Hi @zdrapela , the kogito-addons-quarkus-process-svg extension looks at a file with the same name as the workflow definition to make the SVG available through a REST service, so we should use ${fileBasenameNoExtension}.svg by default. If you could please change that on the README as well. Thanks!

The (build) kie-tools-stunner-editors check has failed. Please check the logs.

Hi @paulovmr, thank you for explaining. I changed that to ${fileBasenameNoExtension}.svg in the default value (same as it was before this PR), in the description and in README.

I think we can ignore the sonar check. It was broken works now and does not make sense for this PR.


Search function not working on iPad Air 2 (HA app)

What version of Music Assistant has the issue?
2.3.1
What version of the Home Assistant Integration have you got installed?
2024.10.3
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
The search function is not working on my Dashboard iPad Air 2 since i got it in August.
How to reproduce
Use ipad in horizontal mode
Music Providers
Yt music
Player Providers
YouTube
Full log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2024.10.3
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

found this, keyboard input in HA app works fine, its only for music assistant
https://github.com/music-assistant/hass-music-assistant/issues/1276

Vuetify lib is the problem on ios16


Helm yaml errors

To Repro:

Ingest the Redis HA official helm chart into a new KOTS vendor app
Attempt to deploy the application

Expected: Deploy succeeds. Helm charts with multi-resources and one of them is empty deploys without problem.
Actual: Deploy fails. Admin Console incorrectly reports the chart as invalid YAML. Upon inspection, it appears to be because there are multi-documents, and https://github.com/helm/charts/blob/master/stable/redis-ha/templates/redis-ha-announce-service.yaml#L6 one of them has only whitespace between --- lines.


Convertion with avif takes a huge amount of time

Hi.
I'm using your dependency instead of node canvas with pdfjs-dist. I just want to report the huge amount of time required to convert a page of a PDF to an avif.
Reproduction:
const viewport = page.getViewport({
  scale: 2,
});

const { width, height } = viewport;
const canvasAndContext = canvas.create(width, height);
const renderContext = {
  canvasContext: canvasAndContext.context,
  viewport,
};

await page.render(renderContext).promise;

// ...

canvasAndContext.canvas.toBuffer(`image/avif`),

// ... then await promises.writeFile(...)

It takes approx. 55 seconds to generate 4 PDF pages, while it takes 900ms to generate 4 pages with JPEG.

Encoding lossless AVIF is very slow without hardware accelerators. You need to adjust the quality via toBuffer() avif config: https://github.com/Brooooooklyn/canvas/blob/main/src/avif.rs#L11-L21


added search bar for blogs

Created a search bar feature for blogs for issue #133.

I think you have not pulled the changes from origin as recently the blog section has been updated.

I have pulled it now but I can still see only three blogs and below that there are images.

There is some issue on your side. If you go and check the website through the link provided you may see the changes.

fixed.
https://github.com/Suchitra-Sahoo/AgriLearnNetwork/assets/113294672/4a4a49c3-5063-49be-8185-44ca29b8f4be

Make a change that suppose the user searches a specific blog then the shape of the card shouldn't change even if there is a single result.

@Suchitra-Sahoo I am working on this issue.


feat(header): Add component

This adds the header component, based on the GOV.UK equivalent. The original macro in the GOV.UK Frontend is only intended for use on GOV.UK, and isn't that customisable, so our own version has been added here.
I added some specific visual regression tests for hover states.
Unfortunately, there was a problem where the window.matchMedia() call by the JavaScript used in the header component was behaving inconsistently for the Pixel 5 project. For now, I've skipped the relevant tests for mobile Chromium, and will look to create a simpler repro for this to file a bug upstream. Edit: I created https://github.com/microsoft/playwright/issues/18919 regarding this.

Nice one a lot of work here it's a shame you had to rewrite the header. Let's hope that there are not many components like this.

Have put this back to draft for the minute. We were missing <meta name="viewport" content="width=device-width, initial-scale=1, viewport-fit=cover"> in our HTML templates for the examples server which was causing the Playwright mobile projects to render incorrectly.
Will do a separate PR to fix this as it'll change all the screenshots.

I've rebased this on to #29 and updated and improved the tests. The skips have been removed as these are no longer required. Additional tests for mobile and with JavaScript disabled have also been added.
I've set up two new tags – @desktop and @mobile – to restrict tests to desktop or mobile. (test.skip() was an option, but the tags seemed like a cleaner way to do it.)


Generate CIFTI derivatives

@satra said (#658)

saving to CIFTI

@effigies:

This seems reasonable, but out of my experience so far. If you have example code that converts the type of data you'd be interested in, that would be useful. (I assume CIFTI can handle either structural or functional data, as standard NIfTI can...)

This issue propose storing derivatives to CIFTI format.

Quick question - is there a labels atlas for the MNI template fmriprep uses - MNI152NLin2009cAsym, similar to mindboggle's OASIS-TRT-20 joint fusion atlas?

We could map that one into MNI if you'd need it
On Feb 6, 2018 11:45, "Mathias Goncalves"<EMAIL_ADDRESS>wrote:

Quick question - is there a labels atlas for the MNI template fmriprep
uses - MNI152NLin2009cAsym, similar to mindboggle's OASIS-TRT-20 joint
fusion atlas https://osf.io/d2cmy/?action=download&version=1?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/poldracklab/fmriprep/issues/794#issuecomment-363542394,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkhxjuH_DzLqTM2wcssIOqLRUgWboXEks5tSKvugaJpZM4QGhHW
.

What's the role of such labels in context of CIFTI files?

@oesteban @chrisfilo we could use the labels to generate the corresponding brain structure label and ijk coordinates

I see - thanks for clarifying. It might be worth splitting this task to cover different CIFTI outputs in separate PRs.

we should always have the output CIFTI's subcortical structures in the same space as the template - as more are added (#487) we'll have to generate new labels for that template, so I agree this should be broken up in separate PRs. I'm hoping to add support for common space (fsaverage5 surf, MNI152NLin2009cAsym vols) and hopefully subject space (piggybacking once #957 is merged)
@oesteban how would we go about doing this mapping?

The first I'd do is running a registration using: niworkflows.interfaces.mni.RobustMNINormalization. Default settings should be pretty good. I assume the template comes with a good brain mask, so everything should be pretty straightforward.
Then we would resample the template, and keep the transform somewhere (maybe in niworkflows?). It'll be extremely useful :)

Closed via #1001


How do I specify build dependencies of default features

Trying to write my first port, I encountered the following issue. My library has support for different XML parsing libraries, and just needs one of them. So while there is a Feature: expat and a Feature: libxml2 that depend on the correspoding libraries, if I specify
Default-Features: expat
and then run vcpkg depend-info, I don't see expat listed as a built dependency, as I would have expected. However, adding:
Build-Depends: expat
seems wrong as well, as then the library would depend on expat even though it would only be using libxml2.
When asking a question please also include where you looked for an answer (so we can update the documentation if needed).

https://github.com/microsoft/vcpkg/blob/master/docs/specifications/feature-packages.md
https://github.com/microsoft/vcpkg/blob/master/docs/maintainers/control-files.md
https://github.com/microsoft/vcpkg/blob/master/docs/maintainers/portfile-functions.md

Could you post the full CONTROL file?

Also as a follow up, can your library tolerate both the expat and libxml2 features to be specified and can the library tolerate having neither specified?

So far, this is the [CONTROL] file for now, interestingly it does not complain about the Build-Depends line at all. I'll change it back to always depend on expat if necessary (as the library will be fine linking against both libraries). However, no xml parsing library is not an option.

So as for your question about why depend-info was not returning what you expected that is a bug which is being addressed by #7643. Do expat and libxml2 provide the same interface?

Hi @fbergmann
Do you have any other questions about this?

I'm closing this issue for now since inactive in a long time.


[capture-promotion] Update style of old pass.

No one has touched this pass in years and I am going to do a little work in it
to help with concurrency. So I am formatting the pass before I make further
changes.
All I did was I renamed CamelCase -> camelCase and git-clang-format the diff. I
used clangd to incrementally perform the renaming using some integration into my
editor.

Specifically, concurrent function types should always capture by value. Rather than implement this in SILGen, we are just going to have said condition be a requirement of Canonical SIL (with a SILVerifier check), not of Raw SIL and have capture promotion just handle it. This prevents us from needing to add additional code to SILGen to handle this case and can keep the complexity in SIL where it is easier to maintain.

@swift-ci smoke test and merge

@swift-ci smoke test and merge


preLaunchTask cannot be found for launch config in devcontainer.json settings

VSCode Version: 1.62.3
Local OS Version: RHEL 8 - Linux x64 4.18.0-305.19.1.el8_4.x86_64
Reproduces in: Remote - Containers
Name of Dev Container Definition with Issue: Go (I haven't tried if it happens with other definitions)

Steps to Reproduce:

Add a task in the tasks.json file in workspace/.vscode/
Add a launch object inside the settings section of devcontainer.json, with includes a preLaunchTask referencing the task above.
Run the debug action and a pop-up appears saying the task cannot be found.

Notes:

If the same launch configuration is created in a launch.json file in workspace/.vscode/ instead of under the settings section of devcontainer.json, the task can then be found and it runs properly.

tasks.json
{
    "version": "2.0.0",
    "tasks": [
        {
            "label": "echohi",
            "type": "shell",
            "problemMatcher": [],
            "command": "echo Hi"
        }
    ]
}

devcontainer.json
{
	"name": "Go",
	"build": {
		"dockerfile": "Dockerfile",
		"args": {
			"VARIANT": "1.17",
			"NODE_VERSION": "none"
		}
	},
	"runArgs": [ "--cap-add=SYS_PTRACE", "--security-opt", "seccomp=unconfined" ],

	"settings": {
		"launch": {
			"version": "0.2.0",
			"configurations": [
				{
					"name": "testconfig",
					"type": "go",
					"request": "launch",
					"mode": "debug",
					"preLaunchTask": "echohi",
					"program": "${workspaceFolder}/bin",
					"showLog": true
				},
			]
		},
		"go.toolsManagement.checkForUpdates": "local",
		"go.useLanguageServer": true,
		"go.gopath": "/go",
		"go.goroot": "/usr/local/go",
	},

	"extensions": [
		"golang.Go"
	],
	"remoteUser": "vscode",
	"containerUser": "vscode",
	"workspaceMount": "source=${localWorkspaceFolder},target=${containerWorkspaceFolder},type=bind,Z",
}

You should add launch configurations to .vscode/launch.json.  I don't believe what you are doing is supported there, is it @chrmarti ?


Feature Request: Ability to Enable/Disable feature flags through code

It would be great if we can have the ability to Enable/Disable feature flags through the code. So we can use our own UI rather than going into Azure Portal.

@jaliyaudagedara please let me know if #417 answers your question.

@zhenlan, thanks for the reply! Right, so in that case, technically this should work, isn't it?
Azure App Configuration client library for .NET - Version 1.0.2

Yes, the SDK you mentioned above will work.
And I agree - we should have better documentation for this too. I have this tracked internally (FYI, @AlexandraKemperMS). Thanks for the suggestion, @jaliyaudagedara.

Awesome, thank you @zhenlan!


Change deprecated clap functions to their alternatives.

Currently, in the printer and replicator, we implicitly use deprecated unit variant clap::ArgAction::StoreValue. This causes our workflows to fail, as we can see here. I have changed it to explicitly use clap::ArgAction::Set instead.

@macher259 as a followup, please bump the clap version in Cargo.toml. The clap::ArgAction feature you use here is only available since 3.2.0 and the code doesn't compile with older versions (I still had 3.1.18 in my Cargo.lock specified).