Add record elimination, tests and docs

Okay here's my stab at it.

The compiler insists that class VariantRecordElim, class VariantMatch must be exported – is this true? it seems a bit ugly ...
I just used StrMap.lookup, would you prefer FFI?
Let me know if you want anything renamed, or if you want the helper classes merged, etc.

And of course typeclass polymorphism doesn't work from inside records, but I can't even begin to think of how to accomplish that. Otherwise it seems to work well!

I was playing around with RowList and I think I found a nice abstraction: https://gist.github.com/MonoidMusician/fe7d35c437c23833a5ca5b67a5ca3744
We could maybe put the classes in typelevel and just use them in variant, what do you think about that?
The next layer of abstraction would be a ZipWith kind of thing, for applyRecord and such, since we really are just matching on corresponding rows and checking a relation.
Grant and pick
I decided to change the name of elim to grant, since the Variant is giving it's value to a Record, and now there's a dual(?) called pick where the Variant decides which row in the Record it gets applied to. Does that make sense? Basically the difference is where the function is – Variant or Record.

Okay, I think this is ready to be merged – thanks for the unsafeGet PR!

Thanks for the PR!

Thank you!


👋🏻  VSCode Extension

Hi there! 👋🏻 amazing tool btw!
As I tend to use vscode (and was a bit jealous of that vim integration!) I've created a little vscode extension that runs Wollemi on file save.
Project link: https://github.com/jamesjarvis/vs-wollemi
VSCode marketplace link: https://marketplace.visualstudio.com/items?itemName=jamesjarvis.vs-wollemi
I was wondering if I could get your opinions on it? Thanks!

Hi James,
Nice job, I just test your plugin and looks promising. So far we've been using wk-j.save-and-run plugin to integrate that but we would love to have a specialized plugin.

Nice! I have not used vscode so it took me awhile to figure out that vs-wollemi.shell needed to be set for the extension to work for me. Now that I have that set the extension seems to work just like I would expect. Thanks @jamesjarvis!

@brian-bice thanks for the feedback! I left the option in as it worked locally (on my MacBook) without needing config, however on other systems it seems to require a shell to be specified.
I think I'll update the docs to highlight that this setting may require setting 😁


[Feature]: Publish NuGet Public IP addresses/CIDRs for whitelisting

Related Problem
I am using AWS CodeBuild to build a .NET 8 project which relies on some public NuGet packages. I'd like to lock down the Security Group on my CodeBuild project to ONLY allow outbound HTTPS/443 traffic to NuGet, since that is the only necessary internet bound communications that is needed.
Security Groups require whitelisting by CIDR blocks so I am looking for a place where I can find the NuGet's public CIDR blocks for whitelisting.
The Elevator Pitch
Make publicly available the CIDR blocks of that would be hit when restoring public packages from nuget, so clients can implement least-privledge on network infrastructure.
Additional Context and Details
Originally posted here: https://stackoverflow.com/questions/78804804/nuget-public-ip-addresses-for-whitelisting

Hey @skirk-mpr, apologies for the delay.
The set of IPs used for NuGet.org are not fixed. In fact, there is not published list of IP ranges that we maintain. This is because NuGet.org has a variety of cloud infrastructure dependencies and these internal have many different IP ranges.
To give a sample of why this is a hard problem for us, I'll list some dependencies of ours (easily discoverable via some simple DNS queries):

api.nuget.org - this points to a CDN. Not only are there multiple IP addresses associated with a single CDN URL (this is part of a traditional CDN implementation), there are multiple CDNs we use, for high availability and performance reasons. So, two distinct problems causing the set of IP ranges to be hard to document as fixed.
www.nuget.org - this points to Azure API Management. This may have a fixed set of IPs, but it is owned by another team, not the NuGet.org team, and may be dynamic or pooled with other customers (I am not sure).
Our search services, such as azuresearch-usnc.nuget.org - this is again cloud infrastructure, but running on Azure App Service. These IPs are not managed by us but instead by an Azure team. We do not require a fixed IP and only rely on a stable DNS name.

I'm sorry that I do not have a good answer for you. If possible, could you allow-list *.nuget.org DNS names? That is the only feasible option I can provide you given our current infrastructure.
Here is Visual Studio document aligned this my suggestion:
https://learn.microsoft.com/en-us/visualstudio/install/install-and-use-visual-studio-behind-a-firewall-or-proxy-server?view=vs-2022#urls-to-add-to-an-allowlist-and-ports-and-protocols-to-open
See the row for NuGet and NuGet package discovery.


decorator for tasks

I was wondering if we should have a decorator for tasks where we can signal things. For instance:
@brigade.plugins.task(dry_run=True, format_agruments=["filename"])
def napalm_configure(task, filename=None, configuration=None, replace=False):
    ....

@brigade.plugins.task(dry_run=None, format_agruments=[])
def print_result(task, data, vars=None, failed=None, task_id=None):

Basically arguments of the decorators indicate that:
a. Whether the task supports dry_run or not or if it doesn't apply
b. Which arguments should be auto-formatted (an alternative to #53)
c. Others that might come, maybe?

Could perhaps also use decorators later to handle the try/except statements from https://github.com/napalm-automation/brigade/pull/77#issuecomment-363729919 as a way of avoiding having redundant code in various plugins.

I am going to close this one for now as I don't see an immediate need. Right now most of the logic can be enclosed in the start method (https://github.com/brigade-automation/brigade/pull/98/files#diff-118676f40dc730bc62bf5d116683f4abR42). The main purpose for a decorator would be to allow "configuring" the behavior of tasks. For instance, #97, which I am not sure we want.


Chat Functionality

Is it possible to add a common chat kind of functionality/ sending some different event to participants to this flow?

@nikhilPlays Chat functionality can be implemented using the data channels in the twilio-videio sdk, if your goal is a full featured chat system such as XMPP, then that is not available with twilio-video today.


wall times are all messed up. Everything is negative hours

This is a report I generated with the JSON output. Using the latest jar to generate the reports

--packages qubole:sparklens:0.3.2-s_2.11 \

Report Link

@vijay-jangir Thanks for reporting this issue.
We analysed the sparklens Json and noticed that 364 out of total 535 stages failed to report information about all the tasks. Out of expected 128152 tasks 45867 were not reported. At our end we plan to add another sparklens analyser which can report such problems. It might be good idea to grep the logs for "event dropped" to validate this claim. Increasing the size of spark.scheduler.listenerbus.eventqueue.size from default 10K to something higher like 20K or 30K will probably help.  Since this will take up some memory at driver end, it will be good idea to increase the driver memory a bit if under pressure.

See https://issues.apache.org/jira/browse/SPARK-18881 and https://issues.apache.org/jira/browse/SPARK-18838 for more context around this issue.


python310Packages.sip: 6.7.5 -> 6.7.7

fixes the build on Python 3.11
Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

cc @eduardosm @kilianar

This broke pyqt5, fixed by https://github.com/NixOS/nixpkgs/pull/217781.


DS-560 - [Swagger] Select is not working in GET /{id} API

This commit rectifies a mismatch between Swagger UI documentation and API implementation, enabling proper support for the select parameter in the GET /{id} API. The enhancement ensures that the API aligns with documented specifications, allowing clients to request specific fields and receive customized responses

@jugnuagrawal

@jugnu-appveen Modified the code to ensure that the select parameter is accepted only as comma-separated values:


Add Terraform demo to session materials

Add initial set of files for using Terraform with OpenStack. This branch is not ready for merging into master yet; only creating this PR as a placeholder.

I'm merging this PR as it works in my lab (3 test runs, all successful). We'll commit final changes resulting from porting this to the corporate lab after it's been merged into master.


Allow braced sections

Actual
new CopyWebpackPlugin([
	{
		from: 'file1'
	},

	{
		from: 'file2'
	},
	{
		from: 'dir/'
	}
])

Expected
new CopyWebpackPlugin([
	{
		from: '{file1,file2,dir/}'
	}
])

The current error:
ERROR in [copy-webpack-plugin] unable to locate './{file1,file2}' at '/{file1,file2}'

I've already tried the glob option, but it does not work as well https://github.com/kevlened/copy-webpack-plugin/issues/87

This is a bug. Here's a workaround:
new CopyWebpackPlugin([
	{
		from: '{file1,file2,dir/}',
		fromType: 'glob'
	}
])


Feature request: Replace the query editor with Monaco

The core vscode editor and its shortcuts are quite versatile, and appear to be a more polished superset of the current querybook editor. It has features the querybook editor could benefit from, like multiple cursor support.

I think multiple cursor support is available currently with codemirror?
My personal preference would be to upgrade to https://codemirror.net/6/ since it is more modular and (potentially) simpler to upgrade


Create issue on failed nightly

Automatically create a github issue when nightly run fails. Maintainers are notified when the issue is created.

this should resolve #23


Remove CallerIpAddress from the documentation

The CallerIpAddress is not provided in the logs anymore so it needs to be removed

@stefang931 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

@bwren can this be merged? Stefan works here


[5.3] implement migrateExpiredJobs with eval

Regarding issue #12653

Can you describe in detail how the Lua script works line by line in this PR. Also why do you send negative values into queue later in your integration test?

From what I understand, he is using negative values to make sure the getTime() - $delay timestamp is in the past and not in the future. Using positive values (like the 1000) would tell the worker that there is no job that is ready to be worked on.

why do you send negative values into queue later in your integration test?
To have expired jobs without waiting using sleep() and slowing down the tests.

I did a stress test. The results are looking good for this pull request and poor for the current driver. Please take a look:
halaei/redis-benchmark

Might be an idea to actually target 5.2 here rather than 5.3?

This requires Redis version > 2.4 or 2.6 I guess. Plus access to eval for
the Redis user.
On Mar 10, 2016 5:16 PM, "Graham Campbell"<EMAIL_ADDRESS>wrote:

Might be an idea to actually target 5.2 here rather than 5.3?
—
Reply to this email directly or view it on GitHub
https://github.com/laravel/framework/pull/12655#issuecomment-194845464.

Maybe best to keep this in 5.3 then, and note the upgrade required.

The last commit is to make the Redis queue more reliable. Basically, it is about the same idea as described in http://redis.io/commands/rpoplpush#pattern-reliable-queue.

@taylorotwell I am done.

Should I close this and have another pull request without conflict?

@halaei sure, go ahead and open a new PR.


ELR Failure - RI, 10/19/2022

Email from Jason Garrett RI DOH:
Hi @oslynn ,
I am emailing at this time because RI has identified an ELR that failed upon entry into the RI system.  This is with accession# cb23c0a5-3984-4ba2-8f9c-141db7d7bca4 and failed on entry yesterday 10/19.  I had our vendor investigate the raw message and the case for failure is listed below.  Can you please review and verify if this is a good result or was it one used when testing your system with Jason Suchon yesterday?  If it is a good result, can this please be corrected and resubmitted?  If it was a test message to test the SFTP, no further actions are required.

Thank you,
Jason Garrett, BSN, MPH

Email from Jason G:
Good morning Ott,
This morning RI received a list of ELR failures from the ReportStream feed due to missing required data elements in the HL7 messaging.  There are 7 total failed messages from yesterday, and all with the same reason code.  Below is a screenshot of the reason for failure and list of accession numbers.  Please work to have these corrected and resubmitted as soon as possible, as implement the fix to prevent these errors going forward.  If you need additional information, please let us know and I can open a ticket with our NBS host vendor.
Accession #: 05204ec7-476e-4af2-a80d-8a8f88aa4651
Accession #: 9ecadb3d-57b1-4530-a8b8-857fc9e67646
Accession #: 9c96a25a-aabe-4322-b768-851ebe7d8c60
Accession #: 5872ddd8-e380-4d1a-a446-42f1cf18e288
Accession #: ef515e92-26b7-41ad-82c8-ae14bb328f41
Accession #: 04aa517d-f52e-4478-86bb-9f2c26e155fa
Accession #: 19b6b8ed-b23f-4e15-815a-64e9f1a356ec
Success/Failure
Description
Failure
The code for Ordered Test name is missing. Blank identifiers in segments HL7LabReport.HL7PATIENT_RESULT.ORDER_OBSERVATION.ObservationRequest.UniversalServiceIdentifier
Thank you,
Jason Garrett, BSN, MPH

Email from Jason G:
Thank you all.  I have also looped in Jason from our ASP vendor, who conducted the onboarding, in case there are any back end questions that he may be able to assist with.  Ott, if you would like one cumulative file of all the Advanced Molecular Diagnostic accessions #’s which failed, please let me know.  I would be more than happy to get that for you.  In the meantime, is that feed able to be turned off until the fix is completed and implemented so RI does not keep getting numerous failure alerts?
Thank you,
Jason G
RIDOH

Email from Jason G:
Hi @oslynn tt,
There were continued Advanced Molecular Diagnostics ELR failures over this weekend due to the feed continuing to stay active while you work on the fix.  I have attached a cumulative list of message ID’s and test accession #’s for your FYI of all AMD failures.  All of these results will need correct and resubmission to RI once your fix is in place.  If RI continues to received AMD failures, I will add to his aggregate file and forward to ReportStream for FYI purposes and easier identification of which messages need to be retriggered for submission to RI.
Thank you,
Jason Garrett
RI Dept of Health

Email from Ott:
Good morning, Mr. Garrett,
I will deactivate the feed while I am working on the fix.  Is that ok with you?  I will reactivate it as soon as I have a fix in place.
Very best regards,
-Ott Sathngam
CDC SWE

2022-10-24 - Email from RI (Jason Garrett)
Hi Ott,
There were continued Advanced Molecular Diagnostics ELR failures over this weekend due to the feed continuing to stay active while you work on the fix.  I have attached a cumulative list of message ID’s and test accession #’s for your FYI of all AMD failures.  All of these results will need correct and resubmission to RI once your fix is in place.  If RI continues to received AMD failures, I will add to his aggregate file and forward to ReportStream for FYI purposes and easier identification of which messages need to be retriggered for submission to RI.
[https://app.zenhub.com/files/304423150/ba6e7b37-c5c1-4141-8532-cb44fc8cc815/download
Thank you,
Jason Garrett
RI Dept of Health

2022-10-24.  Configure prod setting database for RI-DOH to block messages that doesn't have valid OBX-3.2.
Notified RI with email below:
Good after noon, Mr. Garrett,
I configured our system to not send the message that doesn't have valid "Test Ordered LOINC Long Name" (OBX-3.2).
Therefore, you will still get other messages.  Further, I will be working with senders to resolve those invalid messages.
After we get them resolved, I will resubmit all of them.
Please let me know, if you still get those invalid missing OBX-3.2 messages.
Thank you for your patient,
-Ott Sathngam
CDC SWE

Email from Jason G:
Hi Ott,
Failed message info from yesterday, 10/24, below.
Processed Time
Status
Message ID
Reporting Facility
Accession#
Exception Text
10/24/22 4:01 PM
Failure
c11377a5-907d-42f8-b71b-ac1ceabcf00f-6
Advanced Molecular Diagnostics
c11377a5-907d-42f8-b71b-ac1ceabcf00f
Error creating lab. Blank identifiers in CE or CWE segments 1 or 4. See Activity Details.
10/24/22 4:01 PM
Failure
d3b3f773-bf60-4d39-afc6-b00214861e4a-6
Advanced Molecular Diagnostics
d3b3f773-bf60-4d39-afc6-b00214861e4a
Error creating lab. Blank identifiers in CE or CWE segments 1 or 4. See Activity Details.
10/24/22 4:01 PM
Failure
928d6586-7ee3-4602-b4fb-ab16802358d8-6
Advanced Molecular Diagnostics
928d6586-7ee3-4602-b4fb-ab16802358d8
Error creating lab. Blank identifiers in CE or CWE segments 1 or 4. See Activity Details.
Best,
Jason

2022-10-24:
Hi Ott,
Failed message info from yesterday, 10/24, below.
Processed Time
Status
Message ID
Reporting Facility
Accession#
Exception Text
10/24/22 4:01 PM
Failure
c11377a5-907d-42f8-b71b-ac1ceabcf00f-6
Advanced Molecular Diagnostics
c11377a5-907d-42f8-b71b-ac1ceabcf00f
Error creating lab. Blank identifiers in CE or CWE segments 1 or 4. See Activity Details.
10/24/22 4:01 PM
Failure
d3b3f773-bf60-4d39-afc6-b00214861e4a-6
Advanced Molecular Diagnostics
d3b3f773-bf60-4d39-afc6-b00214861e4a
Error creating lab. Blank identifiers in CE or CWE segments 1 or 4. See Activity Details.
10/24/22 4:01 PM
Failure
928d6586-7ee3-4602-b4fb-ab16802358d8-6
Advanced Molecular Diagnostics
928d6586-7ee3-4602-b4fb-ab16802358d8
Error creating lab. Blank identifiers in CE or CWE segments 1 or 4. See Activity Details.
Best,
Jason

2022-10-29.  Resubmitted all the messages with corrected OBX-3.2 to Jason.

2022-10-31: Feed back from RI all messages resubmitted on 2022-10-29 missing OBR-4.1, 4.2.

2022-11-1: resubmitted one corrected OBR-4 message to RI to see if it is good.
Good morning, Jason,
I corrected the OBR-4-1, 2, 3, 7 for one message and uploaded to your prod folder to see if it is good for you.
The message id: cb23c0a5-3984-4ba2-8f9c-141db7d7bca4
Please let me know if it is good for you.  I will process and resubmit the rest of the messages.
Very respectfully,
-Ott Sathngam
CDC SWE

Thank you, Ott,
I will be on the lookout for this at the next ELR push time and provide some feedback.
In addition, I did notice during my review of the failures from 10/29 that there are duplicate accession #’s coming through with different reporting facility names, i.e. Advanced Molecular Diagnostics (AMD) and CDC PRIME.  It is my assumption that these are all AMD accessions, but my question is to why they are also coming through with CDC PRIME as the reporting facility?  Examples below:
Processed Time
Status
Message ID
Reporting Facility
Accession#
Exception Text
10/22/22 4:01 PM
Failure
027b9348-8718-45cd-9d3c-7f8e55c24d99-6
Advanced Molecular Diagnostics
027b9348-8718-45cd-9d3c-7f8e55c24d99
Error creating lab. Blank identifiers in CE or CWE segments 1 or 4. See Activity Details.
10/29/22 4:01 PM
Failure
027b9348-8718-45cd-9d3c-7f8e55c24d99-6
CDC PRIME
027b9348-8718-45cd-9d3c-7f8e55c24d99
Error creating lab. Blank identifiers in CE or CWE segments 1 or 4. See Activity Details.
10/29/22 4:01 PM
Failure
027b9348-8718-45cd-9d3c-7f8e55c24d99-6
Advanced Molecular Diagnostics
027b9348-8718-45cd-9d3c-7f8e55c24d99
Error creating lab. Blank identifiers in CE or CWE segments 1 or 4. See Activity Details.
Processed Time
Status
Message ID
Reporting Facility
Accession#
Exception Text
10/20/22 6:01 PM
Failure
04aa517d-f52e-4478-86bb-9f2c26e155fa-6
Advanced Molecular Diagnostics
04aa517d-f52e-4478-86bb-9f2c26e155fa
Error creating lab. Blank identifiers in CE or CWE segments 1 or 4. See Activity Details.
10/29/22 4:01 PM
Failure
04aa517d-f52e-4478-86bb-9f2c26e155fa-6
Advanced Molecular Diagnostics
04aa517d-f52e-4478-86bb-9f2c26e155fa
Error creating lab. Blank identifiers in CE or CWE segments 1 or 4. See Activity Details.
10/29/22 4:01 PM
Failure
04aa517d-f52e-4478-86bb-9f2c26e155fa-6
CDC PRIME
04aa517d-f52e-4478-86bb-9f2c26e155fa
Error creating lab. Blank identifiers in CE or CWE segments 1 or 4. See Activity Details.
Thank you,
Jason G
RIDOH

Email from Jason G:
Hi Ott,
Thank you for this update and thank you for the time and efforts to correct the feed.   Please feel free to contact me with any questions you have as well.
Best,
Jason


Garbage collection issues with luar.slice?

The results of luar.slice can sometimes be random, and I suspect it's because the created slice is not properly anchored so that the Go GC doesn't collect it.

This should be fixed then. Closing.


java.net.BindException: Address already in use

I get the following exception when opening the "Open Task" dialog, also the issue list from youtrack is not updated any more.
This seems to happens since the update to version 0.2 of the youtrack plugin.
com.github.jk1.ytplugin.navigator.components.SourceNavigatorComponent@16de606b: Address already in use
java.net.BindException: Address already in use
	at java.net.PlainSocketImpl.$$YJP$$socketBind(Native Method)
	at java.net.PlainSocketImpl.socketBind(PlainSocketImpl.java)
	at java.net.AbstractPlainSocketImpl.bind(AbstractPlainSocketImpl.java:382)
	at java.net.ServerSocket.bind(ServerSocket.java:375)
	at java.net.ServerSocket.bind(ServerSocket.java:329)
	at fi.iki.elonen.NanoHTTPD$ServerRunnable.run(NanoHTTPD.java:1687)
	at java.lang.Thread.run(Thread.java:745)

On which port does the nanoHttpd run? Maybe I have something else running on that port?

It tries to listen on ports from 63330 till 63339. It's still a bug though, as occupied port is not expected to affect issue list.

A graceful fallback has been implemeted

Any chance of creating a new release any time soon? I'm looking forward to continue using the plugin ...

@tgoetz, would you mind trying a fixed build first? I failed to get the very same exception as you did even with required ports occupied. What I've implemented here is a general fallback, so I'd like to know if it really fixes the problem for you before releasing it.
Here's the build with a fix: https://www.dropbox.com/s/vuef7qvbkrszllo/ytplugin-0.2.1.zip?dl=0
Thanks in advance.

I tried the build, now I get the following warning when starting IDEA:
"YouTrack plugin error: Can't listen on ports 63330 to 63339. 'Open in IDE' feature will be disabled". It looks like the list of tasks does not get refreshed also, at least I can't find newly created task in the list when opening the "Open Task " dialog.

Do these newly created tasks match a search query in YouTrack repository settings?

Hm? I simply started typing the ticket description ...

Here's the search query I am talking about: https://www.dropbox.com/s/9s9ltabu1im4kre/tasks.png?dl=0
Task management in Idea is designed to filter tasks for "Open Task" dialog by this query.

Ok, got it! The task was unassigned and the filter contained "assigned to me". Thanks for the hint!

That explains much) I'm glad we finally got this sorted out.

Yeah, me too. Thanks again for your support!


Allow dependencies to be specified when compiling, using maven coords

Comments welcome...
This utilizes some dependency resolution code taken from the spring cloud thin launcher. It is now possible to compile code that has dependencies, supplying them via maven coordinates:
String source =
"import org.joda.time.*;\n"+
"public class A implements java.util.function.Supplier {\n"+
"  public String get() {\n"+
"    DateTime dt = new DateTime();\n" +
"    int month = dt.getMonthOfYear();\n"+
"    return String.valueOf(month>0);\n"+
"  }\n"+
"}";
CompilationResult cr = rjc.compile("A", source, "maven://joda-time:joda-time:2.9.9");

The final argument to compile is varargs so you can supply multiple (of the form maven://groupId:artifactId:version)
If dependencies cannot be resolved there will be messages in the compilation result (accessible in a similar way to other compile errors). If compilation succeeds the resolved dependencies are accessible on the CompilationResult:
List<File> resolvedAdditionalDependencies = cr.getResolvedAdditionalDependencies();

So for the dependency maven://joda-time:joda-time:2.9.9 the resolved dependency file might be: /Users/aclement/.m2/repository/joda-time/joda-time/2.9.9/joda-time-2.9.9.jar.
The challenge after this might be how to package those dependencies alongside the compiled code for later use. If simply storing the maven coordinates someone is going to have to resolve them later.

re: the challenge of packaging... in the case of Docker containers, we could rely on the thin launcher "dry run" feature to populate a maven repo within the container

merged (with minor changes based on comments above) via https://github.com/spring-cloud/spring-cloud-function/commit/2fdfda941676d438c6b1e012b3dd35acdd5069a6


Feature : Add / POST route

Home post route
This route will be used when user will  be successfully login in
Details :
This API will simply return the logged in users data

git fetch origin
git checkout 15-feature-add-post-route


Good catch, go.mod is outdated. Will bump it now. Thanks.

Have released v1.1.0.


Allow file_from_env and file_from_cl to override file in FileSource?

Code:
class ConfigRoot(BaseConfig):  # type: ignore
    mail: dict[str, MailConfig]
    redis: RedisConfig
    CONFIG_SOURCES = FileSource(file="config.toml", file_from_env="CONFIG_FILE")

What I'm looking for is for a way to make it so that if CONFIG_FILE is not set, fallback to config.toml. Right now file is prioritized, then env, then cl. There should probably be a Value error thrown if more than one of the flags are truthy? As they can not work together?
https://github.com/Zuehlke/ConfZ/blob/ecd5b0bd9ec2667cb638fb2d16554a6908463068/confz/loaders/file_loader.py#L21-L32
For me, it seems the fix can be as simple as swapping the elif with if statements, and making from env and from cl mutually exclusive, for example? Or make the params order sensitive somehow?
Alternatives:
Having multiple file sources all marked as optional seems like a bit of an iffy approach? As we end up with either a validation error or an empty object I need to check, instead some error regarding resolution.
Example:
class ConfigRoot(BaseConfig):  # type: ignore
    mail: dict[str, MailConfig]
    redis: RedisConfig
    CONFIG_SOURCES = [
        FileSource(file_from_env="CONFIG_FILE", optional=True),
        FileSource(file="config.toml", optional=True),
    ]

In this scenario, neither CONFIG_FILE is set and config.toml is available. We then end up with a pydantic validation error instead on any loading related error.

Thanks for the detailed description! I agree, it would be useful to have a "default value" for the config file path, if no env-var / cli-arg is provided.
I think it would be a bit more work than just swapping the elif with if; we end up with quite a bit of cases. But also not too complicated I guess.
If anyone suggests a PR, I'm happy to review it.


Fixed: 404 in ansible-setup

Broken link  to the old  platform-setup document.
code block in openshift setup.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: fleeto
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: sdake
If they are not already assigned, you can assign the PR to them by writing /assign @sdake in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS

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


[Bug]: On sharing workspace with appviewer rights, cannot see data on launch from the appviewer login.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I share a workspace with appviewer rights to user<EMAIL_ADDRESS>and login with me-qa1 user, I do not see any data on launching the app.
Steps To Reproduce

create an application with google sheets DS.
Create a query and bind some data to a table
Deploy the app
Share the workspace with<EMAIL_ADDRESS>user, and give the user App Viewer rights
Login with me-qa1 and launch the app
Note no data is seen

https://ddpa7mfuvv.vmaker.com/record/DFrFimmwHlSJ0xYo
Public Sample App
No response
Environment
Deploy Preview
Issue video log
No response
Version
Cloud

@btsgh parking this till rbac is completed

@btsgh This is not reproducible anymore,
The user listed here has been given app-viewer access only, the data could be seen in the table

Still able to reproduce the issue @sondermanish @ayushpahwa
Private Zenhub Video

This issue is fixed in new environments, but in existing env still able to reproduce it. Since you have mentioned above that this is a problem that you are aware of, rechecking here.
From my perspective, this can be faced by a customer with existing GSheets connection, and upgraded to Enterprise to use Custom environments, so I would want to reopen the issue.


Now calling the xcodebuild -create-xcframework command with the -headers option

Updated the make/framework.mk and scripts/gen_xcframework.sh files so that the xcodebuild -create-xcframework command is called with the -headers option. Fixes #2180.

Nice work @adil-hussain-84 - this does indeed produce valid XCFramework artefacts which seem to comply with Apple's specification. The /usr/bin/codesign tool is still happy that its a valid framework, and thus Xcode 15 will compile projects using the framework without issue.
👍 for merge

@tomball FYI I have approved this PR but still see the message "At least 1 approving review is required by reviewers with write access." - does someone else need to approve, or will this be picked up automatically?

Yes, someone from  Github's Google org needs to approve, which I just did.
Instead of merging this PR, the next process step will cause the PR to be
staged internally as a change request, requiring another engineer to
approve, and then it gets automatically merged (Google has to be
super-careful about how changes are integrated into its source base). This
should happen automatically, but I'm always a little amazed when it works.
:-)
On Tue, Sep 19, 2023 at 2:12 PM Rory Sinclair @.***>
wrote:

@tomball https://github.com/tomball FYI I have approved this PR but
still see the message "At least 1 approving review is required by reviewers
with write access." - does someone else need to approve, or will this be
picked up automatically?
—
Reply to this email directly, view it on GitHub
https://github.com/google/j2objc/pull/2181#issuecomment-1726472668, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW2JL5ACDY666CURW2TJQ3X3IDEDANCNFSM6AAAAAA455WHSQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Haha sounds good @tomball 👌

Copybara is our repo-syncing service. We're good here, and thanks to you both for your contribution and help!

I did very little other than try out @adil-hussain-84 ’s changes, but you’re most welcome 👌

@tomball @mungler: Thank you both for reviewing, approving and merging 👍👍 Sadly, it's looking like this not the last pull request from me related to XCFrameworks 😭 A number of the XCFrameworks that are being built have the same module name (i.e. JRE) in their module.modulemap file. The implication of this is that two modules with the same cannot be linked into the same target. I see that their is an open Issue and Pull Request related to this already. I'll post in there and draw attention to it.


page 59: app.js cannot listen on port 80

Before typing this I also tried using the files up github instead of the ones I typed/pasted and nothing worked until I used an unprivileged port.

@ofsoundmined - When you had this problem, were you running the playbook against one of your own servers, or against a VM that you built locally, e.g. with the Vagrantfile as shown in the book?
The examples seem to work fine if I run them using the Vagrant-built VM, so maybe it's a privilege escalation error? Could be related to the fix I posted to #77.

To be able to run on port 80, the node.js app has to be run with elevated privileges (e.g. sudo node app.js vs node app.js — or the equivalent for forever).
Note that this example runs the forever commands using become/sudo, to make sure it can listen on port 80. If you are building a more secure app that should not run under root privileges, you should probably opt for a higher port (e.g. 8080) and then proxy it using Nginx or some other web server that can proxy traffic to the backend.


Add Konsole's Dark Pastels theme.

I just stole the colours from the dialog.

Could you please add the link to the source?

Would this be ok? I'm not sure where to link it.
Also, now that I've found the source maybe I'll need to update the colors a bit
https://invent.kde.org/utilities/konsole/-/blob/master/data/color-schemes/DarkPastels.colorscheme


Disabled search is still generating alerts

I'm still receiving alerts and emails about a search that is currently disabled and I'm wondering if there's a way to remedy this?

The server is likely working thru a backlog of jobs. What are the stats you see on the /health page?


typo: exernal

typo
@brief in the following line adjust the value of high speed exernal crystal (hext)
should be
@brief in the following line adjust the value of high speed external crystal (hext)

Got it, Thanks for your advice.


Refactor: remove the blue highlight of buttons on mobile

Hey @LukeberryPi ,
I removed the blue highlight of buttons on mobile. When you press a button, the blue highlight stays on the button and it's annoying, not least because it's square whereas the button is rounded.

good catch! two questions:

could you post a picture of this in effect? as in, how does this square manifest itself on mobile?
i saw you created a styles.css file, but this project uses tailwindcss. please implement your solution by using tailwind functionality

thanks!

@LukeberryPi This is the documentation for the css property added
https://developer.mozilla.org/en-US/docs/Web/CSS/-webkit-tap-highlight-color
It basically gives the mobile user a visual feedback that the button has been pressed:
https://user-images.githubusercontent.com/42651514/226192549-d717501f-b410-48bc-a680-661a49832a9a.mp4
I personally don't like removing that kind of feedback, since most users are already used to it, and removing it for every link and button will make having clicked / not clicked versions to every one of them a necessity just to replace what the tap highlight does.
And also currently it doesn't have a very good cross-browser compatibility:

since there seems to be no gain on removing this visual effect + it's an accessibility standard, i will be closing this.


Allow per component sharing content

Let us share different text per share button.
Right now we define everything for all the buttons in the parent wrapper, which can be confusing cause you don't really know without trying (or looking through the code) what share button uses what data. Also it makes it impossible to create different sharing messages adapted to the medium.
Ideally, from a marketing perspective, content you share on Twitter and for ex Facebook should be vastly different.

@nicolasbeauvais Thanks for closing this issue without further comment. You can perhaps also deliberate to disable Issues completely or make it closed source project.


Export system Foundation for Darwin platforms

Export system Foundation on Darwin platforms. This allows us to lower the supported platforms and allow this package to be used as a dependency on Darwin.

While I was here: deleted Package-5.8.swift since it's no longer used and really diverged from the main file.

Closing this for now, since we're going a different direction in the short term.


Docker readme out of date

The link to "Volumes" is broken.
Also, it is unclear if I need to run:
$ DATA_DIR=${HOME}/respond-data
$ mkdir -p ${DATA_DIR}/sites
$ mkdir -p ${DATA_DIR}/mysql

I am not familiar with Docker.  @chirino , you initially submitted the pull requests that added Docker support.  Can you point us in the right direction on this?

The commands are required if you want to store content in /app/sites and /var/lib/mysql to your docker host instead of storing inside the docker image.

#411 for fixing the broken link and adding more details

Closing issues not related to R6.


fix(http): using hostname as default resource name

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


python312Packages.bc-detect-secrets: 1.5.17 -> 1.5.18

Diff: https://github.com/bridgecrewio/detect-secrets/compare/refs/tags/1.5.17...1.5.18

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 353689

x86_64-linux

:white_check_mark: 6 packages built:

checkov
checkov.dist
python311Packages.bc-detect-secrets
python311Packages.bc-detect-secrets.dist
python312Packages.bc-detect-secrets
python312Packages.bc-detect-secrets.dist


Uploaded the Cheat Sheet for Verticapy

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Comment Highlighting Missing

Hi, Thank you for your great work! I recently switched from the REST Client to httpYac and I really like it. However, I noticed that one feature I miss is the highlighting of comments.
In the REST Client extension, the comments are highlighted clearly, but in httpYac, they are not as distinct:
REST Client:

httpYac:

Is this something that can be resolved through the settings? I searched but couldn't find any solution. Could you please take a look at this issue? Thank you in advance!

Not through the settings. The order of the regex does not fit somehow. I have fixed it the easy way and have not yet looked for the reason why the request-line does not close the area correctly. But it works


Mocking Entry method of DbContext is hard

Many of methods in DbContext are virtual. So they can be mocked using Moq or similar frameworks. But Entry is not virtual. [Searching the net] (http://www.asp.net/web-api/overview/testing-and-debugging/mocking-entity-framework-when-unit-testing-aspnet-web-api-2#dependency) made me accept that the better way to mock this method is to write a MarkAsModified method in our DbContext. I think this is not a clean way. Is it possible to make Entry method as virtual or create IDbContext interface?

@alisabzevari

@alisabzevari It seems Entry() is already virtual in EF7. I will assume that you are asking about making in virtual in EF6 as well, correct?

Yes I was talking about EF6. I thought they migrated EF6 to github. Seems like it is in codeplex yet.

Yeah, we haven't moved EF6 tracking to GitHub, at least not yet. In the meanwhile I copied this issue to https://entityframework.codeplex.com/workitem/2741.

@alisabzevari can you see https://entityframework.codeplex.com/workitem/2741 as we have a question on the request


ResponseSelector

Hi , when i want to use rasa with "ResponseSelector" I get this error.

Hi, we'll be providing a different way to enter responses directly that supports multilingual bots very soon

But the problem is not the retrieval selector?? , is the language?

Are you sure this has to do with the ResponseSelector component? i.e. when you remove it from your pipeline, the problem disappears?
Please show us the contents of your credentials settings.

I solved the problem, I had a mistake. thank you.

Good to hear!


importer: Detect duplicate data importing by Lightning.

What problem does this PR solve?
Issue Number: part of https://github.com/pingcap/br/issues/1110
Problem Summary:
We need detect duplicate keys from source data when we use Lightning to import data to an initial TiKV cluster. The disk of Lightning is limited, so we can not keep all source data in one disk of Lightning.
What is changed and how it works?
Proposal: xxx
We will use TiKV cluster to store all data which may be imported by difference Lightning. Every lightning instance will use an  unique commit-ts for their request. When importing is end, we will collect the duplicate keys from TiKV, which have a multiple version in MVCC.
What's Changed:
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note

No release note.

/merge


As an Operator, I can view the logfiles of a failed task to gain a better understanding of what went wrong

@mike-gangl As I see it there are probably two main ways to view logs of a failed task:
1). Through some SPS interface (EMS?)

In the near term I don’t really have an opinion on how to do this.
Longer-term we should avoid going straight to whatever the implementation detail is (e.g. HySDS)… I think MAAP throws related stderr and stdout into a field that you can get via WPS. Ideally an operator can see the log/stderr/stdout info in the Job dashboard as part of the results.


Adsk Contrib - Better manage Imath dependency

Signed-off-by: Patrick Hodoul<EMAIL_ADDRESS>Using Imath in the library could sometime conflict with the Imath from OpenImageIO. When you try on ubuntu 20.04 using the system OpenImageIO the compilation of OpenColorIO fails because of symbol clashes on Imath (i.e. 3.1.x from OpenColorIO 2.1.x vs. 3.0.x expected by OpenImageIO 2.1.x).
The pull request changes the code to use the OpenImageIO one for apps using OpenImageIO. I also removed some useless Imath dependencies.

I did my tests on ubuntu 20.04 and macOS.

The PR is a candidate for RB-2.1


Fix drush core-rsync not expanding ~ to $HOME

~ expansion is a shell feature that doesnt work with rsync -e flag.

Not saying this will solve all drush issues, but it's at least one bug.
You can test that ~ does not expand with:
 rsync -e 'ssh -o PasswordAuthentication=no -i ~/.vagrant.d/insecure_private_key' vagrant@drupalvm-debug.dev:~/.bashrc foobar

This is what drush issues under the hood using a core-rsync. Eg:
drush core-rsync @drupalvm.drupalvm.dev:index.php testing.php

Update, apparently $_SERVER['HOME'] doesnt exist on windows, so I use drush helper function instead.

Nice; never knew about that shortcut.


Follow user help with example

I was trying to make the code follow a user and i can't get it to work. Could anyone help me show an example of following a user with this api.

If you are already logged in this will work:
const userId = (await ig.user.searchExact('username')).pk;
const res = await ig.friendship.create(userId);
console.log(JSON.stringify(res));

@Nerixyz  There is a problem that is the login is successful but the code is not working, i tried console logging the user search and that returned a null.


richer examples, metamask conenction, integration with web3 library

in another issue @contrastpro have provided a working example for metamask connection
https://github.com/WalletConnect/WalletConnectFlutterV2/issues/113#issuecomment-1620083931
and @4xMafole provided an example for integration with web3 dart library
https://github.com/WalletConnect/WalletConnectFlutterV2/issues/122#issuecomment-1632359925
Can it be integrated into the examples so new users would easily find it and also it would be maintained in case of API changes?

Thank you for the issue!
I will be building out the working examples next week and combining them with the WalletConnectModal example.
As such, here's this same issue over there.


Undo DFC hack when mtgjson patches it or Gatherer is fixed.

https://github.com/PennyDreadfulMTG/Penny-Dreadful-Tools/pull/4159

When this page knows Mayor of Avabruck is 'a' and Howlpack Alpha is 'b' we're probably good – http://gatherer.wizards.com/Pages/Card/Details.aspx?multiverseid=222183

This is fixed in Gatherer but waiting on a rebuild in mtgjson – https://github.com/mtgjson/mtgjson/issues/548

Fixed in mtgjson's 3.15.2.

Actually no, there's at least one card still broken (Bruna, the Fading Light). Dangit.

Still broken.

There is no plan to fix this in current mtgjson as it is being wholesale replaced. When that happens we'll have entirely new bugs to contend with and may switch to scryfall as our source. I'm going to close this and assume that we'll remove this hack if/when we move to a new data source.


Painting Fence Algorithm in Java

💥 Proposal
I would like to contribute Painting Fence Algorithm in Java for Gssoc21.
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


How to run catvehicle in Gazebo using Autoware's Astar obstacle avoid?

I have recently started using Autoware and I am unable to figure out how to run the catvehicle in Gazebo and control it through Autoware. I want to use the built in Astar package to detect obstacles in Gazebo world and be able to drive the vehicle autonomously. I am able to run the vehicle in RViz as per this tutorial.
Steps so far:

Start Autoware -> Gazebo
Open RViz with Gazebo.config

I see the world in RViz and Gazebo model seems fine and /point_raw topic is being published, but I could not find anything to move beyond this point.
Any help would be appreciated.
Thank you.

@sumjos did you find a solution?

Autoware does not support gazebo (catvehicle) since kinetic.
I am trying to integrate with LGSVL Simulator with Autoware.
#1724
Please wait a bit!!

LGSVL simulator is integrated. Also, Old gazebo will be replace the new Gazebo
https://github.com/CPFL/Autoware/pull/1930


Supermarket SSL error

Description
I'm able to access Chef Server but I cannot access public supermarket due to SSL error. When I try to execute for example
knife cookbook site install mysql
The output is
knife cookbook site install mysql Installing mysql to /home/security/chef-repo/cookbooks Checking out the master branch. Pristine copy branch (chef-vendor-mysql) exists, switching to it. ERROR: SSL Validation failure connecting to host: supermarket.chef.io - SSL_connect returned=1 errno=0 state=error: certificate verify failed ERROR: Could not establish a secure connection to the server. Use knife ssl checkto troubleshoot your SSL configuration. If your Chef Server uses a self-signed certificate, you can useknife ssl fetch` to make knife trust the server's certificates.
Original Exception: OpenSSL::SSL::SSLError: SSL Error connecting to https://supermarket.chef.io/api/v1/cookbooks/mysql - SSL_connect returned=1 errno=0 state=error: certificate verify failed
[security@work cookbooks]$ !494
export SSL_CERT_FILE=/home/security/chef-repo/.chef/trusted_certs/server_a2a_it.crt
[security@work cookbooks]$
`
ChefDK Version
[security@work cookbooks]$ chef -v Chef Development Kit Version: 1.1.16 chef-client version: 12.17.44 delivery version: master (83358fb62c0f711c70ad5a81030a6cae4017f103) berks version: 5.2.0 kitchen version: 1.14.2 [security@work cookbooks]$
Platform Version
Tell us which operating system distribution and version ChefDK is running on.
Replication Case
Tell us what steps to take to replicate your problem. See How to create a Minimal, Complete, and Verifiable example for information on how to create a good replication case.
Stacktrace
Please include the stacktrace.out output or link to a gist of it, if there is one.
NOTE: CHEFDK BUGS ONLY
This issue tracker is for the code contained within this repo -- chefdk.

Server issues
Chef-client issues
Cookbook Issues (see the https://github.com/chef-cookbooks repos or search Supermarket or GitHub/Google)

I am having the same issue any luck Rosario?

Hi Napster,
to be honest I detected that problem trying to change the configuration of the workstation related to the certificates to be pushed on the clients during the bootstrap. Without receiving any support, at that time I resolved this issue re-provisioning the workstation from the scratch. Why do you get this problem?which changes have you applied to your infrastructure to be in this situation? Maybe it could lead to a solution

Closing this out as this is most likely due to either a transient network issue or misconfiguration of SSL and/or local proxy. Feel free to open a new issue with current versions of the DK if this is still a problem.


Scrap javascript rendered page.

Hello, can i use colly to scrap javascript rendered page?
or any idea to scrap it?
i already try. but still not working. :(

see #4


Templates modification

Hi! Sorry if that's something obvious, but I can't figure out how can I modify the templates - file names and the template itself?

You mean within the installed plugin (inside the IDE?) That is currently not supported.
But you can fork the repo and do your customisations and create a new plugin from that

Yeah, thanks will do this way. But it will be good to have some customization options in future releases, at least "to generate .html" and the template customization

@ton77v thanks. I'll keep an eye on this feature


Sometimes bumps versions of required packages in setup.py

Dear @peritus,
first things first: Thanks for conceiving and maintaining this fine piece of software. We use it intensively in our daily workflow.
We had this issue from time to time and it's always confusing like "who broke the dependencies in setup.py again?", but we couldn't make up an appropriate example for showing it to you.
But here you are, the best canonical example we can get: bumpversion bumped itself from 0.5.3 to 0.5.4 into non-existence ;]. To see this in action, please have a look at the commit https://github.com/ip-tools/uspto-opendata-python/commit/2dd8e237.
Do you have an idea what we can do to prevent this problem of bumpversion bumping versions of package dependencies?
Thanks in advance for your efforts!
With kind regards,
Andreas.
P.S.: Also the "keyring" package got bumped from 10.5.3 to 10.5.4, which obviously was not intended as well.
P.P.S.: See also the active .bumpversion.cfg file of the project.

You should use the search option to restrict replacements.

Hey @amotl, hope you're doing well, wherever you are these days :) Haven't maintained this much in the past years, and I'm kind of astonished that it is heavily used still.
So, @ionelmc is basically right. You can use the search = option to specify a more specific string for the search/replace option.

Hey guys, thanks for your quick reply!
I just updated the projects' .bumpversion.cfg honoring your suggestions and it still seems to work. Very cool, that minor issue had annoyed me regularly and i wasn't aware of the search option yet.
Thanks a bunch!

Dear Filip, dear bumpversion community,
it just occurred again: bumpversion tried to upgrade from docopt==0.6.2 to docopt==0.6.3 when just attempting to bump our own phenodata module from 0.6.2 to 0.6.3.
This happens despite having configured the respective .bumpversion.cfg appropriately - at least having believed so.
We did some minor investigations and want to share our outcome.

Configuring the "search" option like seen in the .bumpversion.cfg referenced above does not work. The first line after "search = " must not be empty, you have to do it that way:
 search = version='{current_version}'
          __version__ = '{current_version}'
          version = u'{current_version}'
          release = u'{current_version}'

This is on us.

After fixing this, it still didn't work. Bummer! We took the chance to have a look under the hood of the def replace workhorse. So:

We didn't find appropriate code being capable of replacing multiple distinct search expressions at L243 at all. Obviously, the current code will not find any replacements when configured with the search patterns listed above as they are treated as a single opaque string and used as a full search expression.

Things start going haywire after checking no replacement has taken place at L249, where bumpversion is running a second replacement attempt presumably as a fallback. Here, just current_version.original is used as the search string, so bumpversion actually will perform the replacements as if nothing special has been configured at all. This essentially renders the feature useless or even slightly dangerous.

Please bear with us if we got something wrong when reading bumpversions code. Maybe we missed the place where everything is happening correctly and we just have misconfigured our instance. Otherwise, we will be happy to help getting this issue finally right.
While the world definitvely will keep on spinning anyway, we would be happy if you see any slight chance to improve the situation as we are regularly running into this. Through their subtle nature and the fact packaging usually runs after CI, anomalies like these have a reasonable chance of making it beyond QA and everyone feels sad about having published broken eggs ;].
Cheers,
Andreas.

Hi amotl,
First, as peritus commented development of this project has been dead for a while. You should open this bug/feature request in bumpversion2.
As a workaround (and perhaps good practice) I would suggest you to use a version file. Basically create a "__version.py" file in module folder. The contents will be just the version:
__version__ = "0.0.1"

The, from your other files, import the version one and pick the version from that, e.g. setup.py:
...
setup(
    name='pymof',
    version=pymof.__version__,
   ...

Then your bumpersion.cfg only has to modify one file (the __version.py).
This makes bumpversion less prone to messing up and also makes it easier for manual housekeeping of versions (if needed at some point).

Importing stuff in setup.py is a packaging anti-pattern. Stop doing that, it will make your package uninstallable as soon as you start doing imports and stuff in your __init__.py.

Thanks for your suggestions, @arcanefoam! However, there are always things to bump beyond the world of Python, that's the reason we enjoy the agnostic attitude of bumpversion very much in general ;].
We haven't been aware of bumpversion2 yet, so we will definitively head over there. Thanks again!

May i humbly ask you where we can find bumpversion2 already, @arcanefoam?

Misspelled the name:
https://github.com/c4urself/bump2version

Thanks a bunch! As far as i can see, this fork also has the problem of not honoring the "search" option appropriately. We will cross-post the issue there and see what we might be able to contribute to mitigate this problem.
Do you officially endorse heading over to bump2version or would you be happy to accept an eventual pull request to your original version also, @peritus? Do you plan to reintegrate the changes happening in the fork into your mainline version or shall we consider it officially abandoned?
How ever this might turn out: Thank you so much for conceiving this fine piece of software, @peritus!


Elaborated little on the code

Hi Marcus!
Elaborated a little on the code you made.
What do you think about these changes?
Also for the future when you want to update your local work use git rebase upstream/master instead so we get a clean history. https://gist.github.com/ravibhure/a7e0918ff4937c9ea1c456698dcd58aa

Didn't see this yesterday so it got me a little by surprise.
Where do you want check_zram_mounts() to go ?

As I understood the documentation the mounts can be checked using the provided zramctl tool.
I did however not manage to get it working in docker.

The install script.sh wants the machine to be rebooted after installation.
Maybe not needed or the state can be achived in another way?

Are you sure it is working, does the mounts show up in zramctl?
You also need to rewrite your test to be of destructive kind.

Besides, this line also needs to be removed:
echo "ZRAM not installed - test successful."

Use the skip functionality of bats-core instead.
See the comments in original PR.

I comment out the zramctl as I didn't got it working.
See this code as WIP, was easier then comment in original PR.

Also you are in "control" for this PR. It's your repo.

on 1)  yes we could also use zramctl but standard df as used in check_zram_mounts() is also fine.
So we actually do not (have to) use zramctl.
on 2) yes it asks to reboot but in fact start/stop works without rebooting so I don't do (we will of course recommend that to users at the end of installation but we don't need to for our testing)
on 3) I'll retry running inside docker with/without using zramctl

I don't mind using chech_zram_mounts() just make sure that it's in parity with zramctl.
When you feel ready for a "final" review squash together the commits and rebase on top of master.
Looking forward to get this feature onboard, it just need to be solid. 👍

I squashed and finalized code in my PR #576.
https://travis-ci.com/mstormi/openhabian/builds/120515410 is the build log of that, see line 534f report successful testing


[FLOC 3251] add key pair step to AWS instructions

Fixes 3251
Adds a step to our instructions on how to create nodes using AWS. Previously we did not mention there is a step on choosing an existing key pair or create a new one.

:+1: Thanks!


Documented code merge

I have now added proper comments to all of the code. I have merged all of the changes so far.

A point to note is that, the Linker file is to be generalized. If I fail to do that, will be providing a hard-coded Linker file for every tutorial set.

A point to note is that, the Linker file is to be generalized. If I fail to do that, will be providing a hard-coded Linker file for every tutorial set.


Proposal: Allow Single Dollar Sign as Valid Identifier

Swift should continue supporting use of $ as an identifier but when used with non-numeric name consider it an error.
→ View Proposal

Any comments on this proposal?

This proposal looks in scope for Swift 3 for discussion, but I think it would be helpful if you improved the writing to make it more clear: I think what you're trying to say is that "$[0-9]+" is reserved for closure parameters, that $ by itself should be a valid identifier, and that "$[a-zA-Z_....]" (where the second character is something that would start a valid identifier) should be invalid.

@lattner as you said. Rewording to make it clear.

https://github.com/apple/swift/pull/3901

@lattner with PR https://github.com/apple/swift/pull/3901/files already merged. This can be closed.

I had suggested keeping the Dollar as valid identifier. If my PR is to be closed then it should have been done in June when I requested it and not wait until now by keeping it open. I created this PR in June then someone comes in August and merge their code that changes language syntax breaking a lot of existing code by doing opposite of what I suggested.

Hi @ankurp,
The core team discussed this today and would like to run this proposal for review by the swift-evolution community.  Three requests:

please update it to the new proposal template: https://github.com/apple/swift-evolution/blob/master/0000-template.md
please do an editing pass to improve typos and the writing, for example there is a typo in the title.
Please mention that $ was never documented in the TSPL book.

Thanks!

Thanks will update soon.

Thanks!  Here is a link to TSPL:
https://developer.apple.com/library/content/documentation/Swift/Conceptual/Swift_Programming_Language/LexicalStructure.html#//apple_ref/swift/grammar/identifier

Updated the proposal to be more clear. Looking forward to whatever the community decides.

Thanks!  I'll merge this, make a few tweaks, and kick off review.


Added support to Google Play Services plugin

Just added support to this amazing plugin that interact with google play game services.
https://github.com/ptgamr/cordova-google-play-game
Thanks

Thanks for the PR @paulogr, but it's missing one thing: can you include the ngCordova.plugins.googlePlayGame module as a dependency in the /plugins/module.js file?

Hi @pbernasconi !
Just did what you ask me and create mocked services for the plugin.
Let me know if its ok.
Thank you.


ci: set up angular-automatic-lock-bot for the repo

Begins process for #15532
Completes, for cli, https://github.com/angular/dev-infra/issues/16

Merged per @vikerman instruction offline


builtins: implement ST_MakePoint and ST_MakePointM

PR: https://github.com/cockroachdb/cockroach/pull/61105
From release notes:

The geometry builtins ST_MakePoint and ST_MakePointM have been implemented and provide a mechanism for easily creating new points. [#61105][#61105] {% comment %}doc{% endcomment %}

Documented on functions and operators and mentioned elsewhere in docs via #10555


[SR-7151] Generalize CaseIterable Synthesis for Availability

Previous ID
SR-7151

Radar
rdar://problem/43569215

Original Reporter
@CodaFi

Type
Improvement

Status
In Progress

Resolution

Additional Detail from JIRA

Votes
2

Component/s
Compiler

Labels
Improvement, StarterBug

Assignee
None

Priority
Medium

md5: 508eab090d1f0026b4428b290a2d40be

relates to:

SR-9094 Unavailable enum case with synthesized raw value causes error

Issue Description:
As of this issue, the current implementation of SE-0194 does not synthesize a conformance if any cases contain an availability attribute.  This restriction may be lifted at any time by an implementation that maintains the source-order invariant.
enum Rhapsody: CaseIterable {
  @available(*, unavailable)
  case inBlue
  case hungarian, bohemian
  @available(*, deprecated)
  case onATheme
}
Rhapsody.allCases // [hungarian, bohemian, onATheme]

Any news on this issue? Swift 5.7, still a problem.


CI add codecov token to uploader

Adding codecov token to the CI hoping that it'd help with the codecov upload issues.

@BenjaminBossan this might make you happier 😁


5th Idea Meeting

Review 예정
캡스톤 아이디어 회의
Thursday, January 6 · 10:00 – 11:00pm
Google Meet joining info
Video call link: https://meet.google.com/bkm-aiwt-xnv
Or dial: (US) +1<PHONE_NUMBER> PIN: 510 191 537#

New Idea

스마트 적재함


VS Enterprise Subs but my Microsoft 365 E5 Developer tenant was still deleted because of a period of inactivity

I won’t bore you with the sad saga of this, but needless to say, after several case incidents being raised with various support teams across Visual Studio,  and Azure, my E5 dev tenant was still deleted.
as I mentioned in the subject, I have an active Visual Studio Enterprise subscription from my employer. I use it as a tech architect supporting various Microsoft technologies.
My request is that the statements made 3 times through this page are made even clearer so that Tech Support staff are left in no doubt about the status of Office Dev Program Services and how they are impacted by end users with relevant Visual Studio subscriptions.  This might help other people from having a similar experience as mine in future.
many thanks
pete

@Lauragra Hi, the issue I raised may be relevant to the PR that you recently authored. Do you know who is best to assign it to please?

My tenant is loaderladdy.onmicrosoft.com. I’ve been trying to delete it so I can recreate it and restore my VS Enterprise M365 E5 benefit which is disabled. Deleting a tenant ain’t easy. I hope the background to this was useful to help improve the backend help desk instructions.  Many thanks


Support accessing OSS with proxy

What changes are proposed in this pull request?
Support accessing OSS through proxy by configuring alluxio properties or system properties.
Why are the changes needed?
When accessing OSS through a proxy, the OSS client cannot recognize the proxy configuration in system property and environment variables. So it has to proactively set proxy-related configurations in the configuration.
Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

change in user-facing APIs
addition or removal of property keys
webui

@jiacheliu3 thanks for your comment, I have fixed the description of the properties and added the log if proxy is enabled.

alluxio-bot, merge this please


Galaxy animation added

Title and Issue number

I have added the project .Can it be merged now?
Close #4729

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[] I have commented my code, particularly in hard-to-understand areas
[] I have created a helpful and easy to understand README.md

@MohitGupta121 ,@TusharKesarwani Please review my PR.


Add Almalinux dockerfile for manylinux2_28 builds

Add dev/Dockerfile-almalinux-8-complete using amd64/almalinux:8 as base
Update build_all.sh and push_all.sh to generate and push almalinux8 image
Install versioned ROCm packages eg. rocm-dev6.1.0 to avoid issues with "yum update" pulling really old (circa rocm5.2.3 rocm-dev packages from epel (example in comments)

Tested via: http://ml-ci-internal.amd.com:8080/blue/organizations/jenkins/pytorch%2Fdev%2Fmanylinux_rocm_wheels_test/detail/manylinux_rocm_wheels_test/79/pipeline/54/#step-55-log-737
[2024-07-15T05:55:35.214Z] #18 0.467 + yum install -y rocm-dev6.1.0 rocm-libs6.1.0

Example of yum update considering really old ROCm5.2.3 packages from epel:
[2024-07-12T08:29:42.418Z] #13 1.301 Error:

[2024-07-12T08:29:42.418Z] #13 1.301  Problem 1: cannot install the best update candidate for package rocm-device-libs-<IP_ADDRESS>100-82.el8.x86_64

[2024-07-12T08:29:42.418Z] #13 1.301   - nothing provides clang(major) = 14 needed by rocm-device-libs-5.2.3-1.el8.x86_64 from epel

[2024-07-12T08:29:42.418Z] #13 1.301  Problem 2: package rocm-opencl-5.2.3-1.el8.x86_64 from epel requires comgr(rocm) = 5.2, but none of the providers can be installed

[2024-07-12T08:29:42.418Z] #13 1.301   - cannot install the best update candidate for package rocm-opencl-<IP_ADDRESS>100-82.el8.x86_64

[2024-07-12T08:29:42.418Z] #13 1.301   - nothing provides libclang-cpp.so.14()(64bit) needed by rocm-comgr-5.2.3-1.el8.x86_64 from epel

[2024-07-12T08:29:42.418Z] #13 1.301   - nothing provides liblldCommon.so.14()(64bit) needed by rocm-comgr-5.2.3-1.el8.x86_64 from epel

[2024-07-12T08:29:42.418Z] #13 1.301   - nothing provides liblldELF.so.14()(64bit) needed by rocm-comgr-5.2.3-1.el8.x86_64 from epel


PAI become unavailable when users open too many job detail-pages

Currently, PAI job-detail page will auto-refresh every 10 seconds. When user submit too many job instances, every refresh will take a long time to retrieve data. And this will block other requests send to rest-server
Options to solve this problem:

Disable auto-refresh in job detail page
Add cache in rest-server
Add PAI service alert. When PAI become unavailable, rest-server will consume more than 4GB memory and cpu usage is over 100%. Maybe we can trigger an alert for this. (Collecting rest-server logs will also helpful)

In the short term, I suggest to:

Only call API to get GPU index when mouse hovering
Disable auto-refresh when number of containers is larger than 64


Instagram Stories on v2

Ask your Question

Is instagram stories working on this version?

v2 was a great opportunity to release instagram stories
but we were focused on RN 0.60
if you're interested, I can help you with instructions to rollback instagram stories and release a v3
we just need to bring these changes: https://github.com/react-native-community/react-native-share/pull/505

@jgcmarins , yes I am. I would like to

was any work done on this?

was any work done on this?

As i remember, we needed to remove some implementations related to stories on this version. But, we released some updates on the last weeks. @hbarylskyi you can give a try with that to help us know what is the actual state of this feature?

@MateusAndrade I'm looking into the issue. I don't have much experience with rn libraries. How do you test the code in the library? copy it to a react-native project's node_modules manually and run functions from that react-native project?

@MateusAndrade I'm looking into the issue. I don't have much experience with rn libraries. How do you test the code in the library? copy it to a react-native project's node_modules manually and run functions from that react-native project?

You can clone this repo, run a yarnat the root folder, and then at a example folder run a yarnand a yarn dev-sync. This command will copy everything from the root folder, to the node_modules folder at the example app.

@MateusAndrade thank you!

I managed to build and install in release mode after I disabled Dead Code Stripping in example target. Disabling it in RNShare didn't make any difference. So not sure if it's a solution since users would have to change their apps to make them build with this library. Any thoughts @MateusAndrade? Btw, here's the error I was getting:
Undefined symbols for architecture x86_64:
  "_RCTSetLogFunction", referenced from:
      -[exampleTests testRendersWelcomeScreen] in exampleTests.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

@hbarylskyi you got this trying to run the example with react-native run-ios??  :)

@MateusAndrade I ran the example from Xcode.
First i did a git revert ec7ff153015da25ff7f861643c30bcef705a632d. Then I belive I got the error. Then disabled Dead Code Stripping. Should I push my changes to a fork so that you can try it?

@MateusAndrade I ran the example from Xcode.
First i did a git revert ec7ff153015da25ff7f861643c30bcef705a632d. Then I belive I got the error. Then disabled Dead Code Stripping. Should I push my changes to a fork so that you can try it?

Sure! This will help me a lot. :smile:

Ok. Will do later today.

@MateusAndrade #600 👽


Raise an error on 0-dim input to matmul

Close #4684.

Jenkins CI test (for commit 2e46a9b2f8feaaa656e3d2fd042cd7be95ea4949) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins CI test (for commit d46130fed928651309e930c2c84709bcd28bb5fc) succeeded without errors!

LGTM!

Jenkins CI test (for commit d46130fed928651309e930c2c84709bcd28bb5fc) succeeded without errors!


Failed at cmake ..

Faced this issue while installing brainfuck in ubuntu x86_64 at this step
cmake ..

Output :
Could NOT find Doxygen (missing:  DOXYGEN_EXECUTABLE)
CMake Error at CMakeLists.txt:29 (message):
  Doxygen is needed to build the documentation.

Doxygen is required when building the documentation. To disable documentation, run cmake with the following option:
$  cmake -DWITH_DOCUMENTATION=FALSE ..

This option is enabled by default on the develop branch. Also, since this is the develop branch, the interpreter won't work as expected. If you are looking for a working interpreter, take a look the master branch: https://github.com/fabianishere/brainfuck.
Thank you!


API for saving application data (config files, caches) to disk

Tracking issue, details tbd: we should have some API that lets the use save and load data from some set of named locations, like the 'program files' location, the 'temporary directory' location, etc.

There is also the topic of applications being able to store configuration files and other runtime created files in platform idiomatic locations. While that I feel is probably more in the wheelhouse of druid-shell it's at least tangentially relevant to this discussion.

Originally posted by @ForLoveOfCats in https://github.com/linebender/druid/issues/1047#issuecomment-647021531

This sounds mostly like a job for the directories crate?
Not sure if druid should be concerned with this.

That might be true. I'm not thinking of things like getting the user's home directory or documents directory or things like that for actual open/save, but rather dumping of application state; something like the config_dir() feature in directories, but maybe populated with the values used by the installer.

It really annoys me when applications places application specific data outside its own directory, but I suppose we could have a option for that, that application developers could use.  It might also be good to have API to store widget(s) and window(s) data. For example recreating the window(s) on next launch with the same size and position, or custom toolbars where users have added/dragged/removed buttons.

@rhzk persistence between launches is another issue that will maybe/likely/possibly require coordination with platform APIs; on the mac, for instance, there's a bunch of stuff builtin where they will remember what windows you had open and where they were positioned, and attempt to launch you into that state.


external web links open in a new window instead of browser

when clicking any link inside of the app the link opens in a new windows of the web wrapper instead of the default browser! here is a screenshot:

brilliant catch!
It should be fixed in 1.6 version.
Sorry about such quick version bump, wasn't sure I will be able to fix it so quickly.

 Thanks for the fixes and sweet updates.


testGameWidget should allow passing Subject

Problem to solve
Currently, if one is testing with testGameWidget in order to test the subject within verify it requires the developer to traverse the game tree and find the subject they are interested in.
flame.testGameWidget('example test',
  setUp: (game, tester) async {
    final fooComponent = FooComponent();
    final booComponent = BooComponent();
    await booComponent.add(fooComponent);
    await game.add(booComponent);
  },
  verify: (game, tester) {
    final fooComponent = game.descendants().whereType<FooComponent>.first;
    // The above is error prone, depends on how the given game tree is structured.

    expect(fooComponent.something, isTrue);
  },
);

Proposal
Something as below would be useful. Including a build parameter and updating the signature to:
  void testGameWidget<G extends Game, S>(
    String description, {
    Future<void> Function(G game, WidgetTester tester)? setUp,
    required Future<S> Function(G game, WidgetTester tester) build,
    Future<void> Function(G game, S subject, WidgetTester tester)? verify,
  })

or
  void testGameWidget<G extends Game, S>(
    String description, {
    Future<void> Function(G game, WidgetTester tester)? setUp,
    required Future<S> Function(G game, WidgetTester tester) build,
    Future<void> Function(S subject, WidgetTester tester)? verify,
  })

Although with the latter signature I think it would be challenging to test/verify something that requires both the game and the subject. The subject could be the game and then we would have to traverse (same problem).
Another proposal could be with an act parameter:
  void testGameWidget<G extends Game, S>(
    String description, {
    required Future<S> Function(G game, WidgetTester tester)? setUp,
    Future<void> Function(G game, S subject, WidgetTester tester) act,
    Future<void> Function(G game, S subject, WidgetTester tester)? verify,
  })

Note: the above snippets are a quick proposal, it is open to change. I didn't use typedef for readability.
More information

@wolfenrain had an offline conversation regarding this. Currently thinking that it is hard to define an API for it since sometimes you do indeed still want to maybe verify other items on your game as well.
Currently thinking if an additional test would be better, for example, testComponent. That aims to only test a single component,

You can always do
testWithFlameGame(
  'example test',
  (game) async {
    final fooComponent = FooComponent();
    final booComponent = BooComponent();
    booComponent.add(fooComponent);
    game.add(booComponent);
    await game.ready();

    expect(fooComponent.something, isTrue);
  },
);

Yes of course @st-pasha that's also possible. However, I believe for golden tests one does need the verify method. The idea of this is just as an enhancement they are currently workarounds one can do.

However, I believe for golden tests one does need the verify method.

Indeed; but for golden tests you don't need to search for a specific component inside the component tree -- the golden test renders the entire Game widget.

Adding this here:
Currently, some FlameBloc tests will pass in the setUp testGameWidgets but fail in test with an error outlining a BloC is not in context.
I will try to provide minimal code that reproduces this, however, if someone is willing to take this task I'll be happy to.


Cannot see nfs home directory from jupyter hub

@danielballan :
fluerasu@jupyterhub:~$ pwd
/home/fluerasu
fluerasu@jupyterhub:~$ ls /nfs/fluerasu
ls: cannot access /nfs/fluerasu: No such file or directory
fluerasu@jupyterhub:~$

On the jupyterhub host, your /home/fluerasu mirrors your NFS home directory. There is no /nfs/fluerasu, per se, but its contents should be available to you in /home/fluerasu.

that's precisely the problem. It doesn't seem to mirror my home directory. I cannot see, for instance ~/shared which is a link to a common directory we're all using (which is where I linked also the Analysis_pipelines

Ah, I see.
What happens if you open a notebook and type
!ls ~/shared

Your problem is due to an annoying inconsistency with the controls network. Some machines see the home directories under /nfs; other see them under /home. Thus, your ~/shared link works on machines and fails on others. This is not related to Jupyterhub; it is related to the differences between how xf11id-ws1 and xf11id-srv1 are configured.
I took the liberty of created a new link, creatively named ~/shared2. One or the other will work, depending on which machine you are on. Forward any concerns about this situation to the sysadmins. :- )
dallan@xf11id-ws1:/home/fluerasu$ ls -alF shared*
lrwxrwxrwx 1 fluerasu fluerasu 18 Sep 12 04:55 shared -> /nfs/xf11id/shared/
lrwxrwxrwx 1 root     root     19 Oct 29 11:29 shared2 -> /home/xf11id/shared/

upyterhub:~$ ls ~/shared
/home/fluerasu/shared
fluerasu@jupyterhub:~$ cd /home/fluerasu/shared
bash: cd: /home/fluerasu/shared: No such file or directory
fluerasu@jupyterhub:~$

shared2 was owned by root:root. I just changed it to fluerasu:fluerasu.  Should work now. Try it again?

This has been solved by Rob Petkus mounting the /xf11id nfs on jupyterhub.

I am not happy with this long-term, but it's a fine solution until we have a better-defined policy for how to share notebooks.

This is not a good idea and should not be a long term option.
Sent from my iPad
On Nov 12, 2015, at 1:42 PM, Dan Allan <notifications<EMAIL_ADDRESS>wrote:
I am not happy with this long-term, but it's a fine solution until we have a better-defined policy for how to share notebooks.
—
Reply to this email directly or view it on GitHubhttps://github.com/NSLS-II/Bug-Reports/issues/45#issuecomment-156197284.

I talked to Rob about alternatives. We should get a careful consensus
before proceeding.
On Thu, Nov 12, 2015 at 5:35 PM Annie Heroux<EMAIL_ADDRESS>wrote:

This is not a good idea and should not be a long term option.
Sent from my iPad
On Nov 12, 2015, at 1:42 PM, Dan Allan<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
I am not happy with this long-term, but it's a fine solution until we have
a better-defined policy for how to share notebooks.
—
Reply to this email directly or view it on GitHub<
https://github.com/NSLS-II/Bug-Reports/issues/45#issuecomment-156197284>.
—
Reply to this email directly or view it on GitHub
https://github.com/NSLS-II/Bug-Reports/issues/45#issuecomment-156259012.


Remove dep on bolt and fix 2fa stuff

Codecov Report

Merging #106 into master will decrease coverage by 5.5%.
The diff coverage is 8%.

@@            Coverage Diff             @@
##           master     #106      +/-   ##
==========================================
- Coverage   82.66%   77.15%   -5.51%
==========================================
  Files          31       33       +2
  Lines         819      880      +61
  Branches      144      160      +16
==========================================
+ Hits          677      679       +2
- Misses        131      192      +61
+ Partials       11        9       -2

Impacted Files
Coverage Δ

packages/cli/src/utils/logger.ts
12.5% <ø> (ø)
:arrow_up:

.../src/utils/bolt-replacements/getDependencyGraph.ts
90.62% <ø> (ø)
:arrow_up:

packages/get-workspaces/src/index.ts
96.42% <ø> (ø)
:arrow_up:

packages/cli/src/utils/errors.ts
0% <0%> (ø)

...ckages/cli/src/commands/release/publishPackages.ts
0% <0%> (ø)

packages/cli/src/commands/bump/index.js
96.82% <100%> (+0.05%)
:arrow_up:

packages/cli/src/commands/release/index.ts
77.27% <37.5%> (ø)

packages/cli/src/commands/release/npm-utils.ts
7.01% <7.01%> (ø)

... and 1 more

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 e55fa3f...1f27298. Read the comment docs.


DiscoverByShowSlaveHosts - from true to false still not using processlist

I'm having this issue with a set of 3 orchestrator nodes (3.1.2) tied together with raft.
At first, the setting for DiscoverByShowSlaveHosts on all 3 hosts was set to true. Unfortunately, my hostnames are not fully populated (report_host is not defined) which made me go back to DiscoverByShowSlaveHosts=false.
I changed that setting, did a systemctl reload orchestrator on all three nodes but it looks like it's still not taking effect.
Am I missing something?
Oh, and I was also wondering, is there a way to have orchestrator spit out its config from the cli so I can validate that the setting has stuck?
Thank you!
Config:
{
  "Debug": true,
  "EnableSyslog": true,
  "ListenAddress": ":3000",
  "DefaultInstancePort": 3306,
  "DiscoverByShowSlaveHosts": false,
  "InstancePollSeconds": 5,
  "UnseenInstanceForgetHours": 240,
  "ProblemIgnoreHostnameFilters": [

  ],
  "VerifyReplicationFilters": false,
  "AuditLogFile": "",
  "AuditToSyslog": false,
  "RemoveTextFromHostnameDisplay": ":3306",
  "ClusterNameToAlias": {
    "<IP_ADDRESS>": "test suite"
  },
  "SlaveLagQuery": "",
  "DetectClusterAliasQuery": "SELECT SUBSTRING_INDEX(@@hostname, \".\", 1)",
  "DataCenterPattern": "",
  "DetectRegionQuery": "SELECT LEFT(@@hostname, 3);",
  "DetectDataCenterQuery": "SELECT LEFT(@@hostname, 3);",
  "PhysicalEnvironmentPattern": "[.]([^.]+[.][^.]+)[.]mydomain[.]com",
  "StatusEndpoint": "/api/status",
  "StatusSimpleHealth": true,
  "PseudoGTIDMonotonicHint": "asc:",
  "AutoPseudoGTID": true,
  "BinlogEventsChunkSize": 10000,
  "ReduceReplicationAnalysisCount": true,
  "FailureDetectionPeriodBlockMinutes": 60,
  "RecoveryPeriodBlockSeconds": 3600,
  "RecoveryIgnoreHostnameFilters": [

  ],
  "RecoverMasterClusterFilters": [
    "*"
  ],
  "RecoverIntermediateMasterClusterFilters": [
    "*"
  ],
  "OnFailureDetectionProcesses": [
    "echo \"Detected {failureType} on {failureCluster}. Affected replicas: {countSlaves}\" >> /opt/orchestrator/log/recovery.log"
  ],
  "PreGracefulTakeoverProcesses": [
    "echo \"Planned takeover about to take place on {failureCluster}. Master will switch to read_only\" >> /opt/orchestrator/log/recovery.log"
      ],
  "PreFailoverProcesses": [
    "echo \"Will recover from {failureType} on {failureCluster}\" >> /opt/orchestrator/log/recovery.log"
  ],
  "PostFailoverProcesses": [
    "echo \"(for all types) Recovered from {failureType} on {failureCluster}. Failed: {failedHost}:{failedPort}; Successor: {successorHost}:{successorPort}\" >> /opt/orchestrator/log/recovery.log"
  ],
  "PostUnsuccessfulFailoverProcesses": [

  ],
  "PostMasterFailoverProcesses": [
    "echo \"Recovered from {failureType} on {failureCluster}. Failed: {failedHost}:{failedPort}; Promoted: {successorHost}:{successorPort}\" >> /opt/orchestrator/log/recovery.log"
  ],
  "PostIntermediateMasterFailoverProcesses": [
    "/usr/local/orchestrator/resources/bin/orchestrator-client -c set-writeable -i {successorHost}",
    "/opt/orchestrator/scripts/fail_over.sh {failedHost} {successorHost} >> /opt/orchestrator/log/fail_over.log",
    "echo \"Recovered from {failureType} on {failureCluster}. Failed: {failedHost}:{failedPort}; Successor: {successorHost}:{successorPort}\" >> /opt/orchestrator/log/recovery.log"
  ],
  "PostTakeMasterProcesses": [
    "echo \"Running PostTakeMasterProcesses Hook\" >> /opt/orchestrator/log/recovery.log"
  ],
  "PostGracefulTakeoverProcesses": [
    "echo \"Planned takeover complete\" >> /opt/orchestrator/log/recovery.log"
  ],
  "PreventCrossDataCenterMasterFailover": true,
  "PreventCrossRegionMasterFailover": true,
  "BackendDB": "sqlite",
  "SQLite3DataFile": "/var/lib/orchestrator/orchestrator.sqlite3",
  "MySQLTopologyUser": "orc_user",
  "MySQLTopologyPassword": "...",
  "MySQLTopologyCredentialsConfigFile": "",
  "MySQLTopologySSLPrivateKeyFile": "",
  "MySQLTopologySSLCertFile": "",
  "MySQLTopologySSLCAFile": "",
  "MySQLTopologySSLSkipVerify": true,
  "MySQLTopologyUseMutualTLS": false,
  "MySQLConnectTimeoutSeconds": 1,
  "MySQLOrchestratorPassword": "...",
  "UseSSL": false,
  "UseMutualTLS": false,
  "SSLSkipVerify": false,
  "SSLPrivateKeyFile": "",
  "SSLCertFile": "",
  "SSLCAFile": "",
  "SSLValidOUs": [

  ],
  "AuthenticationMethod": "proxy",
  "AuthUserHeader": "X-Forwarded-User",
  "PowerAuthGroups": [
    "DevOps_Prod_Role"
  ],
  "ServeAgentsHttp": false,
  "AgentsServerPort": ":3001",
  "AgentsUseSSL": false,
  "AgentsUseMutualTLS": false,
  "AgentSSLSkipVerify": false,
  "AgentSSLPrivateKeyFile": "",
  "AgentSSLCertFile": "",
  "AgentSSLCAFile": "",
  "AgentSSLValidOUs": [

  ],
  "RaftEnabled": true,
  "RaftDataDir": "/var/lib/orchestrator",
  "RaftBind": "some_host",
  "DefaultRaftPort": 10008,
  "RaftNodes": [
    "1",
    "2",
    "3"
  ]
}

grant
+-------------------------------------------------------------------------------------------------------------------------------------------------------+
| Grants for orc_user@%                                                                                |
+-------------------------------------------------------------------------------------------------------------------------------------------------------+
| GRANT RELOAD, PROCESS, SUPER, REPLICATION SLAVE ON *.* TO 'orc_user'@'%' IDENTIFIED BY PASSWORD '....' |
| GRANT SELECT ON `mysql`.`slave_master_info` TO 'orc_user'@'%'                                                                               |
+-------------------------------------------------------------------------------------------------------------------------------------------------------+

Following grant needed to be added
REPLICATION CLIENT


Avoid creating vite-timestamp files

Is your feature request related to a problem? Please describe.
I think it's good idea to avoid creating that files, because it's really unnecessary and annoying.
Describe the solution you'd like
On Discord we was talking about possible solution.
If there's no native way do avoid creating it, we can create script which will be removing that files during hmr reloading.

It was happening because of conflicts between project versions, it doesn't ocuring anymore.
If anybody still have problem with it, let's clone repo again.

The comment from above isn't true, it's not related to our project at all
I've created bash script for cleaning up this mess, which vite can't resolve over the years ...
I was basing on this comment:
https://github.com/vitejs/vite/issues/13267#issuecomment-1998598546

Vite in version 6 fix this issue, but we need to wait for:

https://github.com/laynezh/vite-plugin-lib-assets/issues/99


MissingReferenceException (seemingly harmless but common)

I frequently get the following error, often when I start / stop play mode, or load / unload a scene.
[Exception] MissingReferenceException: The object of type 'Object' has been destroyed but you are still trying to access it.
Your script should either check if it is null or you should not destroy the object.
0. FullInspector.Internal.fiLateBindings+EditorUtility.IsPersistent() at Assets/Plugins/FullInspector2/Core/fiLateBindings.cs:161
1. FullInspector.Internal.fiPersistentEditorStorage.Read[fiDropdownGraphMetadataSerializer]() at Assets/Plugins/FullInspector2/Modules/PersistentMetadata/fiPersistentEditorStorage.cs:32
2. FullInspector.Internal.fiPersistentEditorStorageMetadataProvider`2[TItem,TStorage].RestoreData() at Assets/Plugins/FullInspector2/Modules/PersistentMetadata/fiPersistentEditorStorageMetadataProvider.cs:14
3. FullInspector.fiPersistentMetadata.GetMetadataFor() at Assets/Plugins/FullInspector2/Modules/PersistentMetadata/fiPersistentMetadata.cs:52
4. FullInspector.Internal.fiGraphMetadataSerializer`1[TPersistentData].UnityEngine.ISerializationCallbackReceiver.OnBeforeSerialize() at Assets/Plugins/FullInspector2/Modules/PersistentMetadata/fiGraphMetadataSerializer.cs:32

I faced this and managed to make it go away.
It began after upgrading. Old scenes would show the warning when serializing/de-serializing.
What solved for me was to go to Window -> Full Inspector -> Developer -> View all Scene Game Objects
Then I simply deleted (clicking the X) the full inspector related objects (fiPersistendEditorStorage and fiBackupSceneStorage). They'll be re-created when you use something from Full Inspector in the scene again (like clicking a foldout).
I'm not sure if this isn't doing more harm than good, but so far everything seems to be working normally.
@jacobdufault, is deleting those objects okay?
P.S.:

I did the upgrade following the instructions
I was able to consistently reproduce this on a new, empty project using Unity 5.4.0p1

Install FullInspector 2.6.2
Have any object inheriting from BaseBehavior on a newly created scene
Save the scene
Open another scene
Upgrade to 2.6.3 or 2.6.4 (from the access page or the store, doesn't matter)
Load the scene created in step 1
Hit play and the warnings will show

During play, if you use the "View all Scene Game Objects" you'll notice that the fi* objects are indeed with their references empty.
When you stop, their references will be correctly set, so something happens during serialization/de-serialization that causes the fi* objects already on the scene to lose their references, but only during play mode.


New version: Bluebeam.Revu.21 version 21.0.50

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @PckgrBot,
This PR doesn't appear to have a manifest.


Multiple instances on windows possible

On windows it seems to be possible to run multiple instances of the application.
This could be solved using single-instance.

@Schrims would be great if you can do a PR. Looks like it is one-line fix, but have a big value! Unfortunately it is hard for me to test Cerebro on windows, so even if I do this fix I'll have to wait someone to test it.
Let me know if you can do it or if you need any help!


Ansible: Etcd not available to dict "groups"

https://github.com/kubernetes/contrib/blob/4d3c7d4e9599812872abc1ddd5536e290792efe7/ansible/roles/master/templates/apiserver.j2
When attempting to stand up a master node using:
INVENTORY=hosts ./setup.sh --tags=masters

The ansible setup for a master node fails when trying to write the config file to the api server:
TASK: [master | write the config file for the api server] *********************
fatal: [<IP_ADDRESS>] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'etcd'", 'failed': True}
fatal: [<IP_ADDRESS>] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'etcd'", 'failed': True}

FATAL: all hosts have already failed -- aborting

From the task file:
- name: write the config file for the api server
  template: src=apiserver.j2 dest={{ kube_config_dir }}/apiserver
  notify:
    - restart apiserver

The call in question seems to be that the groups dict does not have an etcd entry:
# Location of the etcd cluster
KUBE_ETCD_SERVERS="--etcd-servers={% for node in groups['etcd'] %}http://{{ node }}:2379{% if not loop.last %},{% endif %}{% endfor %}"

If this is an error on my part please excuse, and an explanation would be appreciated.

Your inventory file must define
[masters]
[etcd]
[nodes]
Looks like you didn't define etcd in your hosts file.

THANKS!
On Feb 1, 2016 11:48, "Eric Paris"<EMAIL_ADDRESS>wrote:

Your inventory file must define
[masters]
[etcd]
[nodes]
Looks like you didn't define etcd in your hosts file.
—
Reply to this email directly or view it on GitHub
https://github.com/kubernetes/contrib/issues/439#issuecomment-178150578.


COMP: Suppress LabelGeometryImageFilter deprecated warning

PR Checklist

[ ] No API changes were made (or the changes have been approved)
[ ] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)
[ ] Updated API documentation (or API not changed)
[ ] Added license to new files (if any)
[ ] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

This warning is still occurring on the dashboard:
https://open.cdash.org/viewBuildError.php?type=1&buildid=9617479
Any suggestions on how it can be suppressed?

Isn't the problem simply that ImageToImageFilter is used in many places?  If its deprecated, then uses of it will cause such a warning, which is the point, no?


Duplicate criterion error message is awful

Try creating two criteria with the same name for either Rubric or Flexible criteria modes. We don't seem to have a standard error message for this.
It really should be the same error message for both cases.

Dibs!


send email upon news flash failure

Send the error message once news flash cron run fails.

@Mano3 can you elaborate a bit more about this issue?

@atalyaalon done


Tiller-less support in Flux Helm Operator

The Helm Operator could work without Tiller by running helm template and kubectl apply. See https://jenkins-x.io/news/helm-without-tiller/

Seems like a good idea if it is optional, eg --set helmOperator.tiller: false and could serve as a good
stepping stone to helm 3 migration.

That’s one way to do that, another way is to just run tiller in the same pod as helm-operator and bind it to localhost. You still have to run tiller, but it makes it easier to operate (and it’s not going to use many resources on its own) and mitigates the security issues. You also still get to benefit from all of the features tiller gives you (garage collection, rollbacks, etc).
There’s a helm plugin that lets you do effectively this when running helm locally, but you don’t need it if you’re running them in a pod together: http://rimusz.net/tillerless-helm/

But the operator will also get those privileges, which will often be essentially cluster-admin.

If the HelmOperator pod is compromised today, the attacker has the certificate-auth secret material for connecting to tiller anyway. Giving a HelmOperator pod the k8s permission directly doesn't change the risk.

Helm 3 has just had its first alpha release without tiller:
https://github.com/helm/helm/releases/tag/v3.0.0-alpha.1
It would be great to look at what changes would be required to run flux with helm 3

There's a Helm 3 tracking issue at #2069.

This will be covered by Helm v3


Update humans.txt to mark Ben Thorner's departure

Today is Ben Thorner's last day at GDS. As per the documentation he very helpfully wrote 1 we'd like to make a special gesture to thank him for his exceptional positive impact to GDS.

Thanks both. I've made amends. Will merge when it's green.


[UI]: Firefly logo in App-Icon not aligned (macOS)

Platform
Desktop
Version
1.5.1
Description

The Firefly logo in the App-Icon is not properly aligned.
Visual/Optical alignment might be considered over pixel alignment.

Expected behaviour
Aligned properly
Actual behaviour
Not aligned
Can the issue reliably be reproduced?
Yes
Steps to reproduce the issue
No response
Operating System
Mac
Diagnostics
No response
Developer Profile
No
Profile Type
Software (Stronghold)
Staking
No response
Crash Status
None of the above
Error Messages
No response
Error Logs
No response
Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hey thats some serious eye for detail, will pass on to UX/UI team.