There was some unexpected exception in the combinedAudioExecutor!

General Troubleshooting
Hey there! Before you report a bug or suggest a new feature,
please make sure to follow these steps first!

[x] I have checked for similar issues.
[x] I have updated to the latest JDA version.
[ ] I have checked the branches or the maintainers' PRs for upcoming features/bug fixes.

This is not the place to learn Java. Please refer to StackOverflow
for your general programming questions.

Issue
Issue Type

[x] Bug Report
[ ] Feature Request

For questions join the official JDA discord server: https://discord.gg/0hMr4ce0tIl3SLv5

Description
I'm doing some stuff in the handleCombinedAudio(CombinedAudio audio) method however it is never ran because this exceptions happens first, I can't really give you anymore except than that stacktrace
[JDA AudioConnection Guild:<PHONE_NUMBER>85280512 Combined Thread] ERROR net.dv8tion.jda.core.audio.AudioConnection - There was some unexpected exception in the combinedAudioExecutor!
java.lang.ArrayIndexOutOfBoundsException: 960
	at net.dv8tion.jda.core.audio.AudioConnection.lambda$setupCombinedExecutor$7(AudioConnection.java:509)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.runAndReset(FutureTask.java:308)
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$301(ScheduledThreadPoolExecutor.java:180)
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:294)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)

Either not using latest version (not possible since you checked it in your troubleshooting checkmark list) or java.lang.ArrayIndexOutOfBoundsException: 960. So you have to give us the code snippet for using handleCombinedAudio.
Either way, the github issue tracker is reserved for feature requests and bug reports. Your issue is neither and should be sorted out inside of our discord server. Go to discord.gg/0hMr4ce0tIl3SLv5 to receive the needed help.

Even when I comment out my code, so leave the handleCombinedAudio() empty proof: http://prntscr.com/kps2f3
I get this error (the one above) now I think i'd be smart enough to see any of my own methods in the stacktrace. There isn't.

Running version JDA: 3.7.1_411
Lavaplayer: 1.3.7
Java version: 8_171
Whenever the bot joins the Guild Voicechannel I make sure the receiveinghandler is set. and right now all the methods are empty
public boolean canReceiveCombined() {
	return true;
}

public boolean canReceiveUser() {
	return true;
}

public void handleCombinedAudio(CombinedAudio combinedAudio) {
}

public void handleUserAudio(UserAudio userAudio) {}

Note that the overrides were removed some I'm not marking someone.

This is being run straight out of the IntelliJ Idea Environment, built with maven.
the only code relating to the ReceivingHandler is`when I set it as receiver:
	@Override
	public void onGuildVoiceJoin(GuildVoiceJoinEvent event) {
		if (event.getMember().getUser().isBot() && event.getMember().getUser().equals(DiscordBotHandler.jda.getSelfUser())) {
			event.getGuild().getAudioManager().setReceivingHandler(new AudioPlayerReceiveHandler());
		}
	}

Can you try just testing setting the Receiving handler before joining the channel rather than in the voicejoin handler?

I have tried that @kantenkugel same output, I've decided to use UserAudio for now since all I really want to do is record the audio so I can just store each person's speaking per file, that's not so bad rn.

@Articdive I also noticed this issue but since the stuff that I'm doing in handleCombinedAudio shrugged the exception, I figured it was just a blip in the machine. I tracked it down to this thread, but since the Exception is just ArrayIndexOutOfBounds, I can't pin point which array got messed up.
You're saying that you can get around this by directly using UserAudio and performing your own mixing?

I think the exception is caused here.

@Sanduhr32 have you been able to reproduce this? I only see it when I've deployed the bot. If I could reproduce this 100% of the time I could probably create a PR to fix it.

I am already aware of the origin of this exception, however I'm just curious why this now happens all of a sudden when this code has not changed in months.

I used to see it in my logs every now and then, but recently it started happening a lot more, like enough where I had to turn off logs because it was writing too much to disk.
Discord also started appending some kind of header to the audio packets, I don't know if related or not.

https://github.com/DV8FromTheWorld/JDA/issues/755#issuecomment-418165036

From looking at the logs, it seems like the problem went away! @Articdive can you confirm?

I'm on my commute to work as of now. I can confirm later today in tge next
16 hours
On Mon, 3 Sep 2018, 21:40 Jose V. Trigueros<EMAIL_ADDRESS>wrote:

#755 (comment)
https://github.com/DV8FromTheWorld/JDA/issues/755#issuecomment-418165036
From looking at the logs, it seems like the problem went away! @Articdive
https://github.com/Articdive can you confirm?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DV8FromTheWorld/JDA/issues/755#issuecomment-418183012,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AM6KjdKeTXhEaKgEY5kWrOD3VbIuahxjks5uXYW1gaJpZM4WWtyN
.

This has been fixed

Closing the issue then


override equal/hashcode to only consider some specific columns for stream(watch)

Hi.
I have a table with multiple columns, so that some its specific columns are updating every X seconds.
I'm using the watch method to monitor this table:
Stream<List<TableData>> methodName(...) => ((select(table).where(...)).watch()).distinct(const ListEquality().equals);

The above stream results in unwanted UI updates, because it includes unwanted columns in equal/hashcode too.
But I want to be able to monitor changes of some specific columns not all of them.
I can modify equal/hashcode methods manually in the generated db.g.dart file. But this modification will be overridden when I run flutter pub run build_runner build again (for possibly modifying generated db file in the future).
What is the best solution for this case?

I don't think there should be an option to only consider a subset of columns for equals and hashCode.
If you need to ignore some columns for a specific stream only, you can use the elementEquality parameter for ListEquality:
class MyRowEquality implements Equality<TableData> {
  const MyRowEquality();

  @override
  bool equals(TodoItem e1, TodoItem e2) {
    return e1.col1 == e2.col1 && e1.col2 == e2.col2;
  }

  @override
  int hash(TodoItem e) => Object.hash(e.col1, e.col2);

  @override
  bool isValidKey(Object? o) => o is TableData;
}

And then
Stream<List<TableData>> methodName(...) => ((select(table).where(...)).watch()).distinct(const ListEquality(MyRowEquality()).equals);

If you need to ignore some columns all the time (which again, feels a bit weird for a hashCode/equals implementation), you might want to write the result class yourself.

Thank you :)


0.10.0 | "Apps" are not exposed to Homekit anymore

Describe The Bug:

Hi,
I just upgraded to 0.10.0 and reconfigured my TV to the platform configuration (manuel as described in the last comment of #25). It workes so far, however only channels are exposed in homekit. Apps are shown in the homebridge logs (HOOBS for me) as configured but not exposed.
To Reproduce:

I reinstalled the plugin and pasted the example configuartion (with my credentials) into the platforms part of the JASON. Further I deleted the TV in Homekit and added it again.
Expected behavior:

Apps like Netflix are exposed.
Logs:
Currently TV showes Netflix App:
//configuration:
...
apps: [ 'Netflix', 'YouTube', 'Disney+', 'Prime Video', 'Plex', 'TV' ],
...

//log:
[PhilipsAndroidTV] [TV] Current application has been changed to NA.

Plugin Config:
    "platforms": [
        {
            "debug": false,
            "configVersion": 1,
            "tvs": [
                {
                    "ip": "***",
                    "mac": "***",
                    "name": "TV",
                    "apiUser": "***",
                    "apiPass": "***",
                    "apiVersion": 6,
                    "alternativePlayPause": true,
                    "dedicatedMuteSwitch": false,
                    "dedicatedVolumeLightbulb": false,
                    "apps": [
                        "Netflix",
                        "YouTube",
                        "Disney+",
                        "Prime Video",
                        "Plex",
                        "TV"
                    ],
                    "channels": {
                        "useFavorites": false,
                        "favoriteListId": "1",
                        "includeAll": false,
                        "channels": [
                            "Das Erste HD",
                            "ZDF HD",
                            "SAT.1",
                            "RTL",
                            "ProSieben",
                            "WELT",
                            "N24 Doku"
                        ]
                    }
                }
            ],
            "platform": "PhilipsAndroidTV",
            "plugin_map": {
                "plugin_name": "homebridge-philips-android-tv"
            }
        },

Screenshots:

Environment:

Plugin Version: 0.10.0
Homebridge Version:  1.3.3
Node.js Version:  14.17.0
NPM Version:  3.3.5
Operating System: Raspbian

I found the issue. 0.10.1 fixes that.

Thanks for the quick fix! Homebridge/HOOBS doesn't show the update yet. I'll get back to this when it does.

Apps are now inputs, but HomeKit is not able to change the app. Channels still work fine.
That's the log if I try for example YouTube:
4.6.2021, 14:56:34 at PhilipsTVAccessory.launchActivity (/home/hoobs/.hoobs/node_modules/homebridge-philips-android-tv/src/PhilipsTVAccessory.ts:447:45)
4.6.2021, 14:56:34 at ActiveIdentifier.emit (events.js:376:20)
4.6.2021, 14:56:34 at /usr/local/lib/node_modules/@hoobs/hoobs/node_modules/hap-nodejs/src/lib/Characteristic.ts:1630:16
4.6.2021, 14:56:34 at new Promise (<anonymous>)
4.6.2021, 14:56:34 at ActiveIdentifier.<anonymous> (/usr/local/lib/node_modules/@hoobs/hoobs/node_modules/hap-nodejs/src/lib/Characteristic.ts:1628:14)
4.6.2021, 14:56:34 at step (/usr/local/lib/node_modules/@hoobs/hoobs/node_modules/tslib/tslib.js:143:27)
4.6.2021, 14:56:34 at Object.next (/usr/local/lib/node_modules/@hoobs/hoobs/node_modules/tslib/tslib.js:124:57)
4.6.2021, 14:56:34 at /usr/local/lib/node_modules/@hoobs/hoobs/node_modules/tslib/tslib.js:117:75
4.6.2021, 14:56:34 at new Promise (<anonymous>)
4.6.2021, 14:56:34 at Object.__awaiter (/usr/local/lib/node_modules/@hoobs/hoobs/node_modules/tslib/tslib.js:113:16)
4.6.2021, 14:56:34 at ActiveIdentifier.Characteristic.handleSetRequest (/usr/local/lib/node_modules/@hoobs/hoobs/node_modules/hap-nodejs/dist/lib/Characteristic.js:791:24)
4.6.2021, 14:56:34 at Accessory.<anonymous> (/usr/local/lib/node_modules/@hoobs/hoobs/node_modules/hap-nodejs/src/lib/Accessory.ts:1741:29)
4.6.2021, 14:56:34 at step (/usr/local/lib/node_modules/@hoobs/hoobs/node_modules/tslib/tslib.js:143:27)
4.6.2021, 14:56:34 at Object.next (/usr/local/lib/node_modules/@hoobs/hoobs/node_modules/tslib/tslib.js:124:57)
4.6.2021, 14:56:34 at /usr/local/lib/node_modules/@hoobs/hoobs/node_modules/tslib/tslib.js:117:75
4.6.2021, 14:56:34 at new Promise (<anonymous>)
4.6.2021, 14:56:34 (node:27820) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 5)
4.6.2021, 14:56:37 [TV 5688@Active Identifier] The write handler for the characteristic 'Active Identifier' on the accessory 'TV 5688' was slow to respond!
4.6.2021, 14:56:43 [TV 5688@Active Identifier] The write handler for the characteristic 'Active Identifier' on the accessory 'TV 5688' didn't respond at all!. Please check that you properly call the callback!```

Fixed, and callback called faster - in 0.10.6. Sorry for that.


SpecWorms maintenance

A quick check shows around 100 codes that are now different to accepted in WoRMS. What to do about them:

Leave them all as they are: the ICES subset of WoRMs would not distinguish unaccepted codes. Some databases using this subset count on it holding accepted codes, as I understand it.
Maintaining it: deprecate unaccepted codes or maybe add a new flag for unaccepted codes different to deprecated. Would allow to be used as Quality Check of data submitted, but means big maintenance effort and a responsibility we do not want to have.
Slowly move away databases using RECO list as quality check. They could use ERMS database which has flags.

We're not going to maintain the SpecWoRMS list regarding the accepted/unaccepted flag. We'll add a relevant disclaimer in the long description of the code type.
Inform relevant teams, and suggest using ERMS DB on sql10 for the 'accepted' flag, if relevant. ERMS DB can be implemented as a complete replacement for SpecWoRMS, but it will affect search in Data Portal.

@HjalteParner and @HenrikK-N please be aware of this and let's discuss further options if needed in the respective sprints.

... A bit surprised about your take on this!. I think WORMS took over from ERMS ...

It is a problem if the AphiaIds changes over time, so for now from the top of my head it is good, that none of the 'SpecWorm' codes in Vocab are being deprecated. So 100 species out of the 2486 in Vocab are in WoRMS now not accepted?

To my knowledge AphiaIDs do not change over time!
What can happen is that an AphiaID can be deprecated, which is quite logic in the taxonomic world (and elsewhere) where as an example one species can be split into 2 or more species etc. when the taxonomists gets more knowledgeable by time.
This is perfectly fine and expected!
In my opinion we should just use the deprecated flag in RECO accordingly. We can setup a routine automatically which do this routinely at a given frequency.
I shouldn't per default have any effects on the respective systems (internally or externally) using this vocabulary.
I do as such not see the issue and do not think we have an issue - except setting up a routines to catch deprecated codes and set the flag accordingly.

Sorry for my wrong formunation AphiaIds do not change over time. But sometimes the AphiaId for an assessment species change I understand. It is not an easy solution to set previously used codes to deprecated in RECO, and let the users deal with the issue, because then the previously used code will not be valid in the RDBES.

@HenrikK-N, that's an RDBES issue an not an RECO issue!
RECO users have different use cases, and RDBES should not dictate other systems - if so we will complicate things and not for the good

Hi Hjalte and all
I was just flagging that deprecated AphiaId will create work and make us agree on some standard procedures for updating/re-upload historical data, which include deprecated AphiaIds. But I agree with you, it would be best to follow the code list we have decided to use and the countries uploading also are following. I am sure we can agree on standard procedures for how countries should update/re-upload historical data for deprecated AphiaId. I also talked briefly with the RDBES Core Group, and they also recommended to keep the AphiaId up-to-date, regarding deprecated AphiaId.


💅 noUnusedVariables still checking function arguments

Environment information
There is a doc: https://biomejs.dev/linter/rules/no-unused-variables/
It said: "From v1.9.0, the rule won’t check unused function parameters any more. If you want to report unused function parameters, enable noUnusedFunctionParameters."

Today we have a version 1.9.4, so the issue still exists (take a look at playground)

Rule name
lint/correctness/noUnusedVariables
Playground link
https://biomejs.dev/playground/?lintRules=all&code=YwBvAG4AcwB0ACAAdABlAHMAdAAgAD0AIAAoAHYAMQA6ACAAdQBuAGsAbgBvAHcAbgAsACAAdgAyADoAIABzAHQAcgBpAG4AZwApADoAIABzAHQAcgBpAG4AZwAgAD0APgAgAHsACgAgACAAIAAgAHIAZQB0AHUAcgBuACAAdgAyADsACgB9ADsACgAKAHQAZQBzAHQAKAAwACwAIAAnADAAJwApAA%3D%3D
Expected result
No issues on that playground
Code of Conduct

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

Oh, so sad. So, I'm looking forward to the second version

Looking forward to a fix here 😄


remote connect question.

how to work with remote connect ?
i use TeamServer in kali , client.py in windows , found can not connect.

Hello,
Indeed, it was not implemented. I add it in this commit. I created a new release 0.0.1 that include this feature.
You can now launch:
python GUI.py --ip <TeamServer IP> --dev
The --dev is to ignore the hostname that is store in the SSL certificat for the GRPC connection, as the default certificat is created with localhost.
Tell me if it fix you issue.

it work  ,thanks.


Support peer in MQ consumer side plugin, backend support that in entry analysis

@ascrutae Now, only exit span supports #peer, but in MQ scenario, when there is no reference in message, backend needs MQ peer(ip, hostnamt) to make topology map better.
So, please add peer, representing MQ address, in Entry span. Then backend could support to build a new service relation source based on this.

This becomes a required feature. Because in zookeeper watcher scenario, we need this. @ascrutae
ref #2211


Scanning parameters

Reviewer1 mentions that

Information about the scanning parameters should appear in the methods text not the figure legends.

We could add all log files of the shown scans in the Supplementary Materials and generate a table to show the relevant scanning parameters, like in the EAWAG manuscript

Most of the details from the log files have been added with https://github.com/microct-ana-unibe-ch/microvasculature-manuscript/commit/7097e1f214a4d36b88f56fec67a962adc6da8d70
We're still missing logfiles for figure 5b and 5c.


questions about train_contrastive_feature.py and convert.py

hello mr. Jumpat
first, i want to say thank you for your wonderful contribution to 3dgs field as 3dgs is a new thing and what you did is amazing.
i'm an undergraduate student working on my final project and i'm very new to this field so i think i'm slow to understand how your code works.
i have few noob questions:

i don't understand what train_contrastive_feature.py do, would you please explain it to me in an easy terms?
if i want to use my own images as the data, would it be sufficient to just do convert.py , let the data structure format be it, and just proceed to the next steps? to be more clear, after converting, do i have to change the data structure format to be the same as the example?

for now i can only try to understand the codes without implementing it right away bcs the machine that i'm working on is on repair. i might ask more questions in the future during implementation, i sincerely hope you're okay with that. i'm sorry for the long questions, thank youu and i'm looking forward to your reply!

Hi. Thanks for your interest and just feel free to ask questions.
The train_contrastive_feature.py script learns a 3D feature for each 3D Gaussian from the auto extracted masks and SAM features, then these features are used for retrieving the corresponding 3D Gaussians in the inference stage through feature matching.
After convert.py the data structure will be similar to our example. But you may still need to do something like modifying configs or renaming folders.


10x-np: Fix checkver

Excavator has been updating to the wrong versions sporadically https://github.com/ScoopInstaller/Nonportable/commits/master/bucket/10x-np.json
Used the solution from the spotify manifest to get the current version from the installer itself.

[x] I have read the Contributing Guide.

/verify


agixt_1      | TypeError: string indices must be integers

Description
when running task chain
error shows
here's the logs:
logs.txt
Steps to Reproduce the Bug
Run smart task chain
Expected Behavior
no error
Operating System

[X] Linux
[ ] Microsoft Windows
[ ] Apple MacOS
[ ] Android
[ ] iOS
[ ] Other

Python Version

[ ] Python <= 3.9
[X] Python 3.10
[ ] Python 3.11

Environment Type - Connection

[X] Local - You run AGiXT in your home network
[ ] Remote - You access AGiXT through the internet

Runtime environment

[X] Using docker compose
[ ] Using local
[ ] Custom setup (please describe above!)

Acknowledgements

[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of AGiXT.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

Fixed I'll make a pr

can close this after merge made a pr https://github.com/Josh-XT/AGiXT/pull/835


Update hlfsupport-installing.rst

Implemented changes from https://zenhub.ibm.com/workspaces/ibm-blockchain-platform-saas--software-59b193b1106772601c6574a3/issues/ibp/optools/4916

@joshhus sorry for the delay - can you update the commit so that's signed please?

@joshhus I think there is yes, but not 100% sure


WIP So close yet so far

1st Query Script:
-- Use the kodb database
USE kodb;
-- Update the kind column based on the values from the remote database (1298) UPDATE k
SET k.kind = r.kind
FROM Item k
JOIN [1298].dbo.Item r ON k.Num = r.Num;
2nd query script to fix left over bardish (ITEM_CLASS_POLEARM)etc: UPDATE ITEM
SET Kind = 52
WHERE Kind = 100;
Use the provided skill userdata editor:
https://discord.com/channels/385253339971190794/1004036880444162089/1134510865496932412
Copy 1264 data\skills_magic_main_en.tbl and rename it to skills_magic_main.tbl
Copy 1264 Fx directory
TODO: Inbe4 ShitStorm of fixing every item kind on Ebenezer.
TODO: Figure out mage skills ice_target partparticle version 6 1 bool too much or less. (ReadFile(hFile, &m_bParticleYAxisFix, sizeof(bool), &dwRWC, NULL);)
Description
WIP Experimental branch help is needed with FX loading.
@Steve help needed......
💔Thank you!

ps mage userdata table:
nation 2
race 13
Class 209


Master nodes fail to join cluster if multiple nodes are joined concurrently

If multiple master node servers try to join the cluster concurrently, the rke2-server service fails on one or more master nodes, with the following error:
"Failed to start Rancher Kubernetes Engine v2 (server)."
When the rke2-service fails on a host, Ansible considers the failure as "Fatal" and stops executing the following tasks on the respective host.

The results is that the node will then join the cluster, because systemd keep restarting the service until the activation succeeds, but the playbook stops executing the tasks on that particular host.
I resolved this issue adding a "retry" on the task "Start RKE2 service on the rest of the nodes" in the file "remaining_nodes.yml".
Here's the commit on my forked project: https://github.com/GabriFedi97/ansible-role-rke2/commit/70abe0d5241aa4bc11387e239f2e7f94cd4fa9e3
There is an issue on rke2 that explains the problem: https://github.com/rancher/rke2/issues/349

Cool, thanks @GabriFedi97 for info. I will check it ...

Fixed in https://github.com/lablabs/ansible-role-rke2/pull/40


Button Action of InAppMessages deeplink not working

Expected Behavior
Button Action should call deeplink
Actual Behavior
Button Action is just dismissing the InAppMessageController not delivering another action
Its working fine with v4.1.0 but not with 6.0.3
Steps to Reproduce the Problem

Created message from leanplum portal

It will be visible on the app
Click on Add Credit Button
Should take it to another screen

Specifications

Version: 6.0.3
Platform: iOS
Subsystem:

Hello,
I see that this is a custom action. Please, check the changelog regarding In-app messages and Actions in 5.0.0:
https://docs.leanplum.com/changelog/custom-in-app-templates-migration-ios-500
If the in-app leading to the action is also a custom one, please ensure you are calling context.actionDismissed() when dismissing the message.


liga-audio: audio support for liga

UDF
UDT
DSL to manipulate Audio (Jupyter Integration)

Liga is not designed to be domain specific.


Adds attributes support for the Price Model

This PR adds attribute data to the Price model.
This will be useful for storing price related meta data for conditions such as title and description related to pricing. This is in relation to a discussion #2007 to allow adding titles and descriptions for Shipping rates and Rate breaks.
Since the column on the database is nullable, it should not break or affect any existing UI. The attributes related to pricing can be added on a need basis as mentioned in the Use case on the discussion here: #2007
A PR is in the works

I don't believe this is the solution we're looking for. Using attributes would be very difficult to implement in the admin UI.
Let's continue the discussions on the Feature Request discussion.


Raise proper error sbdb, fixes #611

Fixes issue #611 by changing the way of dealing with the error. The error message keeps the same.

Thanks for the pull request @Ivan9528. I saw you fetched @Himanshu-Garg pull request in order to keep his commits and contributions. However since #611 was opened, different changes into main code have been done. You rebased this branch onto master (that is perfect) but somehow you deleted useful lines of code. Also you added others that were not used.
I would recommend you to revert your branch to commit 00209294df89de874029551b869515023d88368e so you just can apply the changes requested in #611 in order to merge.
Before pushing again, you can run tox -e check to see if the format and tests are successful :+1:

Thanks @jorgepiloto, just wanted to say that this is probably half my fault 😅 I was instructing @Ivan9528 how to do the git stuff properly, and didn't realize some stuff was removed in the process. No big deal though :) We can bring it back!

Oh! No problem then :smile:

Thank you for everything! I am trying to add the stuff which is left and correct the quality issues, I will push as soon as I make it! :)

Alright, do not worry this is going in the right direction :+1:
There is another method that was partially removed, in this case def plot(self, label=None, use_3d=False, static=False). Notice that the argument static=False has been removed. This is why the tests are not passing. Also, other lines related to plot() method documentation have disappeared. But since you already solved the other method, this will be easy now :smile:
I will leave you here the same requests here as @Juanlu001 asked for in #611

It took some effort, but here we are :) Thanks a lot @Ivan9528 for this small contribution and especially for your patience, which is arguably one of the most important traits when working in open source. Merging!


RTL mode

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
When I enable RTL in the language settings, the theme does not fold right
Expected Behavior
No response
Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version
No response
Anything else?
No response

@RezaAccountant Thanks for contacting us.
We are not supporting RTL at the Theme level, and we are not planning to do so in the near future. But you can use the IsRtl parameter in some of our components. If there is something missing please tell us about them.
And if you can, explain your requirements in more detail, or mention another library that has what you need.

Ok thank you

I want add style for change layout to RTL.
what is the best solution for add style sheet ? Core project or web project?

You can start by changing the global styles available in the app.scss file of the Core project. the main thing to change is the direction of the root elements to rtl.
But remember, changing the whole layout to RTL is not a simple task. you need to update a lot of other stuff for the app to look clean in a right-to-left direction.
Check this address for more information: https://rtlstyling.com/posts/rtl-styling/


break_on_function_many_child_isolates_test failing on ubsan bot

break_on_function_many_child_isolates_test is failing on the ubsan bot. It's a NPE at jit/compiler.cc:246
Logs: https://dart-ci.appspot.com/log/vm-kernel-ubsan-linux-release-x64/dartk-ubsan-linux-release-x64/1038/service_2/break_on_function_many_child_isolates_test/service
This test has previously been flaky, but the new failure looks different. Might be related to isolate groups?

Another one today: https://dart-ci.appspot.com/log/vm-kernel-ubsan-linux-release-x64/dartk-ubsan-linux-release-x64/1046/service_2/break_on_function_many_child_isolates_test/service
cc @bkonyi service protocol
cc @aam @mkustermann isolate groups

Seems like a benign, trivial issue. Made cl/215547


(PDB-3234) Explicitly install python 2 on debs

It looks like Ubuntu 16.10 may no longer have "python" available by
default, so make sure it's explicitly installed on Debian derivatives in
the pre-suite.

Test FAILed

CLA signed by all contributors.

Test FAILed

Test PASSed


Unexpected error parsing or serializing query

I use the following YQL statement:
select * from doc_source
where ([{"defaultIndex": "doc_fields","grammar": "web","stem": true,"allowEmpty": true}]userInput(@search_terms));
When the search_terms value is m&m's then the yqlRepresentation method of the Query class generates the exception:
Unexpected error parsing or serializing query: com.yahoo.prelude.query.AndSegmentItem incompatible with com.yahoo.prelude.query.TaggableItem
The root cause is the casting in the VespaSerializer::leafAnnotations method:
TaggableItem connectedTo = (TaggableItem) item.getConnectedItem();
when type of the item.getConnectedItem() is AndSegmentItem class.
I  guess that the serialization problem affects not only the yqlRepresentation method.
The problem doesn't exist in Vespa 6 version.

Hi @jamesbond7
Are you using the latest Vespa version?
Can you share your document schema?
Can you add '&tracelevel=3' to the request and add the trace here?
Thanks!

We use the 7.254.8 version.
The document schema sample
The test query request is the following one:
http://vespa7-dev1-srv-1.da-vespa.wixprod.net:8080/search/?yql=select * from test where ([{"defaultIndex"%3A "test"%2C"grammar"%3A "web"%2C"stem"%3A true%2C"allowEmpty"%3A true}]userInput(%40inp))%3B&inp=m%26m's&presentation.summary=base_summary&hits=10&user=debugging&tracelevel=3
The trace output

Thanks, I am able to reproduce this, thanks for reporting.
Your best option to handle terms with other characters than text characters is to add a specialtokens config, see https://docs.vespa.ai/documentation/query-rewriting.html#special-tokens

Query tokens are built from text characters, as defined by isTextCharacter in com.yahoo.text.Text. To query for terms with other characters, like c++ or .net, use special tokens. Unlike query rewriting/phrasing, special tokens modifies data at feeding time, changes to configuration must hence be followed by document re-feed.

With this config you will be able to search for 'm&m', without matching documents with the single character 'm':
<config name="vespa.configdefinition.specialtokens">     <tokenlist>       <item>         <name>default</name>         <tokens>           <item>             <token>m&m</token>           </item>         </tokens>       </item>     </tokenlist>   </config>
=>
select * from sources * where (default contains \"m&m\" AND default contains \"\\'s\");

Special tokens must be supported by the tokenizer, and unfortunately the default open source tokenizer component does not support it. We have an issue on that but haven't gotten around to it yet unfortunately.

Thanks for the proposed solution.
Is it possible just to fix the yqlRepresentation method output? To return a regular yql string instead of the exception message?

Fixed in https://github.com/vespa-engine/vespa/pull/14003 (will be released next week).
The difference from Vespa 6 is that we changed the default from phrase segmentation to and segmentation. If you want the earlier behavior, which will also avoid this bug, add
query-command: phrase-segmenting
to the field(s) in question.


flutter 3.3.10 occasional crashing on android 9 vivo mobiles

Here is crash stack:

This graph is for the major crash models and Android versions:

Here is the flutter doctor:

Hi @BuildAndRelease
Please provide the information in text format instead of a screenshot for better readability and investigation. You may want to refer to writing and formatting syntax for beautifully formatted issue description.
Also, is this issue reproducible locally in release mode? If so, can you please share the output of flutter run -v and a completed and minimal reproducible code sample? And please also try on the latest stable/master channel to see if the issue still persists or not.


Enable data-plane-self-registration on clustered environments

Feature Request
Currently data-plane-self-registration works well in single instance environments. We want to make it work in clustered environments
Which Areas Would Be Affected?
data-plane, configuration
Why Is the Feature Desired?
Are there any requirements?
Solution Proposal
we should have a "cluster id" configuration settings that will permit to identify a cluster of data-planes, and the self registration will use that id instead of the "runtime" ones that's used currently

closed by #4402


[Question] Special characters in prefixes

Desired outcome: setting prefix to ( ͡° ͜ʖ ͡°)
Command tried: lp user dzineit permission set prefix.1.( ͡° ͜ʖ ͡°)
Result: "[22:33:37 INFO]: [LP] Set prefix.1.( to true for dzineit in context server=͡°, world=͜ʖ."
How would I set the prefix to ( ͡° ͜ʖ ͡°) globally without LuckPerms interpreting it as specifying a server and world?

Put quotes around the prefix.
Also there is a better command for setting prefixes: /lp user  meta addprefix

Great, thanks


Use 1 validator per CR

Description
K8S-11317
We initially wanted to have the validation logic per version, which makes sense on paper, but not very realistic in reality, as most new validation logic needs to be added to all versions, and sometimes have to think about cross-version checks.
So now instead we use 1 custom-validator per CR, that can handle multiple version.

For older versions, just use ConvertTo

We still need the 1 validation endpoint per version in the ValidationWebhookConfiguration

It's how the framework works:

You define the webhook for a given version of the CR, so when the webhook receives the request it will convert it to that version.

So if you define only 1 webhook and point all the versions there, all but the 1 version will succeed, others will just fail due to a conversion error within the framework.

Sidenote: Will break https://github.com/Dynatrace/dynatrace-operator/pull/3853 but I can live with that, should be an easy merge conflict. (it's also my PR :P )
How can this be tested?
Deploy 1 dk/ec for each version (broken or not) and check if the webhook reacts accordingly
Example: 3 different dk versions, all intentionally broken, and the webhook rejects them accordingly
Warning: ActiveGate specification missing memory limits. Can cause excess memory usage.
Error from server (Forbidden): error when creating "./.vscode/deploy/dynakube_zib50933.yaml": admission webhook "v1beta3.dynakube.webhook.dynatrace.com" denied the request:
1 error(s) found in the DynaKube
 1. The length limit for the name of a DynaKube is 40, because it is the base for the name of resources related to the DynaKube. (example: dkName-activegate-<some-hash>)
	The limit is necessary because kubernetes uses the name of some resources (example: StatefulSet) for the label value, which has a limit of 63 characters. (see https://kubernetes.io/docs/concepts/overview/working-with-objects/labels/#syntax-and-character-set)
Error from server (Forbidden): error when creating "./.vscode/deploy/dynakube_zib50933.yaml": admission webhook "v1beta2.dynakube.webhook.dynatrace.com" denied the request:
1 error(s) found in the DynaKube
 1. The DynaKube's specification has an invalid API URL value set.
	Make sure you correctly specify the URL in your custom resource (including the /api postfix).

Error from server (Forbidden): error when creating "./.vscode/deploy/dynakube_zib50933.yaml": admission webhook "v1beta1.dynakube.webhook.dynatrace.com" denied the request:
1 error(s) found in the DynaKube
 1. The DynaKube's specification has an invalid API URL value set.
	Make sure you correctly specify the URL in your custom resource (including the /api postfix).

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 26.41509% with 39 lines in your changes missing coverage. Please review.

Project coverage is 64.24%. Comparing base (08d5959) to head (8bdfde4).
Report is 3 commits behind head on main.

Files with missing lines
Patch %
Lines

pkg/api/validation/dynakube/validation.go
33.33%
13 Missing and 1 partial :warning:

pkg/api/v1beta2/dynakube/feature_flags.go
0.00%
9 Missing :warning:

pkg/api/validation/edgeconnect/validation.go
43.75%
8 Missing and 1 partial :warning:

cmd/webhook/builder.go
0.00%
7 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3856      +/-   ##
==========================================
- Coverage   65.56%   64.24%   -1.33%
==========================================
  Files         404      368      -36
  Lines       21819    20469    -1350
==========================================
- Hits        14306    13150    -1156
+ Misses       6161     6037     -124
+ Partials     1352     1282      -70

Flag
Coverage Δ

unittests
64.24% <26.41%> (-1.33%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

code LGTM, i've tested EdgeConnect and it worked.


Write progress/debug output to sderr

https://stackoverflow.com/questions/3385201
https://stackoverflow.com/questions/26130795
https://stackoverflow.com/questions/57016157
https://www.jstorimer.com/blogs/workingwithcode/7766119-when-to-use-stderr-instead-of-stdout

v1 is merged, will be released after a short pre-release period.


add: kicad-diff-git

Progress

[x] Edit packagelist
[x] Add initial pacscript
[ ] Contact devs
[ ] Add maintainer to pacscript

@Henryws thanks for reviewing. Is this what you meant?

What should we do about this?

@Henryws Sorry what should we do about what

The CI failing.

Seems to be an error resulting from pip install wxpython:
ModuleNotFoundError: No module named 'attrdict'

The solution I've found online is to use Python 3.10. For things like this, I usually use pyenv.

So why can't we use 3.10?

It probably has to do with the packaged version in Ubuntu. Maybe the pacstall script worked on my computer because I had pyenv, which injects its own version of python into $PATH. I'm traveling right now. Maybe when I'm back I could figure out a script with pyenv?

Stale.


Can't view logs when pod on "error" state

What did you do?
i can't open logs when my containers on error state

What did you expect to see?
what ever state container after started ,we should view container's log
Environment

System information:
ubuntu20.04

Kubegems version:
v1.21

Kubernetes version:
v1.23.8

Any further updates? @chenshunliang

This feature is merged in v1.22.0-beta.0 .


Refactor/pump tooltip refactor

This PR applies feedback by @tordans and @julizet:

Changes text to: Status in OpenStreetMap aktualisieren
Click outside of tooltip closes the tooltip

I like it. No need for the X IMHO. ❤
Browser metadata
Path:      /
Browser:   Firefox 101.0 on Mac OS 10.15
Viewport:  1560 x 970 @1x
Language:  en
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved


fixes map sizing issues and table pagination

Overview

fixes #328
fixes #314
fixes #313
added a class to contributor meta-data (in the map/sidebar layout) which will force a line break when the string is longer than the containing sidebar.

Demo

Testing Instructions

view a facility and check it's static map.

Should no longer be warped.

view a facility contributor list table, resize screen and scroll table.

Pagination should not scroll left or right.

Heads up that there are some linter errors preventing CI from building this branch:
/usr/local/src/src/components/FacilityListItemsTable.jsx
    9:8  error  'TableRow' is defined but never used                     no-unused-vars
  194:9  error  Expected indentation of 12 space characters but found 8  react/jsx-indent
  213:1  error  Expected indentation of 12 spaces but found 8            indent
  213:9  error  Expected indentation of 12 space characters but found 8  react/jsx-indent

✖ 4 problems (4 errors, 0 warnings)
  3 errors and 0 warnings potentially fixable with the `--fix` option.

Sorry about that, I always forget this project has a linter.

I rebased and updated some styling to that message. I was able to keep the message within the area it was originally placed.


provide client.WithLease to user

Signed-off-by: yanxuean<EMAIL_ADDRESS>@dmcgowan @crosbymichael @stevvooe @Random-Liu
Can we provide withLease() to the user for using?
We need to import image from tar in cri-containerd. so we need it.
ref https://github.com/containerd/containerd/pull/1690#discussion_r152505174

Codecov Report

Merging #1799 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1799      +/-   ##
==========================================
+ Coverage   49.33%   49.38%   +0.05%
==========================================
  Files          77       77
  Lines        7692     7692
==========================================
+ Hits         3795     3799       +4
+ Misses       3252     3248       -4
  Partials      645      645

Flag
Coverage Δ

#linux
49.4% <ø> (ø)
:arrow_up:

#windows
44.95% <ø> (+0.06%)
:arrow_up:

Impacted Files
Coverage Δ

metadata/content.go
32% <0%> (+0.94%)
:arrow_up:

Continue to review full report at Codecov.

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

@dmcgowan  PTAL

We could add the logic on our side, but it's good to reuse the same helper funciton.

LGTM

LGTM


Question: Location of RRDP-related source code

This issue is a question, not a suggestion of a problem with the source code. Sorry.
I am using the testbed functionality of krill and running krill and routinator in a docker container with it as the root certificate authority.
Configuring krill
https://github.com/yushoyamaguchi/rpki_container/blob/main/my_krill1/krill_start7.sh
Configuration of routinator
https://github.com/yushoyamaguchi/rpki_container/blob/main/my_routinator1/routinator_start1.sh
The communication between krill and routinator is successful, but in the container on the routinator side I get
routinator -v --config /etc/routinator/routinator.conf update

When the above code was executed, the following error was printed.
[WARN] RRDP https://krill.example.org/rrdp/notification.xml: error sending request for url (https://krill.example.org/rrdp/notification. xml): error trying to connect: invalid peer certificate contents: invalid peer certificate: UnknownIssuer
[INFO] RRDP https://krill.example.org/rrdp/notification.xml: Update failed and there is no current copy.
[INFO] RRDP https://krill.example.org/rrdp/notification.xml: Falling back to rsync.
[INFO] rsyncing from rsync://krill.example.org/repo/.

For the first error, the error message(UnknownIssuer) was not included in the routinator source code.
Where are these error messages being output from?
I am using a certificate which is generated by krill testbed.
ref:https://blog.apnic.net/2022/05/27/making-a-krill-sandbox/

The first error comes from the HTTP library we are using – reqwest.
If you are not using a “proper” certificate (e.g., one generated by Let’s Encrypt), you need to add the issuer to Routinator via the rrdp-root-cert option. If that is still rejected, you can create a certificate works using these instructions: https://github.com/partim/random-sysadmin-docs/blob/main/generate-self-signed-certificate.md

Thank you very much.
I'll try.

If you increase the log level to debug with -vv, Routinator reports the servers it talks to and what files it processes from them. Alternatively, if you run it in server mode and enable the HTTP endpoint, the /status path shows all the repositories it collected data from. E.g., https://routinator.do.nlnetlabs.nl/status

@partim
I'm sorry for repeating questions.
If it is possible, please tell me about code for processing when rrdp-root-cert options are given.

The code that processes the lost of added root certificates starts here: https://github.com/NLnetLabs/routinator/blob/main/src/collector/rrdp.rs#L1330 – ultimately, the certificate is given to the reqwest HTTP library which does all the actual work.
The only reason I could think of why adding your own certificate should break access to an RRDP server is if your certificate is broken. There should be a log message about this, though?

Thank you for your advice.
That is the difference between using --rrdp-root-cert option and not using it.
No use

Being said that UnknownIssuer about certificate.
Use

It seems that the process is in progress, but the verification has not been completed.
By the way, I specify a certificate in the way like this pic using --rrdp-root-cert option .

Is this the correct type of certificate to specify in --rrdp-root-cert option?

The rrdp-root-cert option is working as expected – Routinator updates successfully in the second example (“snapshot update completed”). The reason you have two ROAs in the first example is that Routinator falls back to fetching via rsync. Looks like there is a discrepancy between the rsync and RRDP data.

As you said, configure of rrdp repository in krill was not correct.
Thank you very much for your amazing support.


starlette middleware decorator is deprecated

The example code for starlette proposes to use the middleware decorator: https://secure.readthedocs.io/en/latest/frameworks.html#starlette
Anyhow, this decorator is deprecated, and will be removed in version 1.0.0 - at least there is a warning about that.
The message is:
[...]/site-packages/starlette/applications.py:248: DeprecationWarning: The `middleware` decorator is deprecated, and will be removed in version 1.0.0. Refer to https://www.starlette.io/middleware/#using-middleware for recommended approach.

It would be good to update the help.
I'm using this now, but would not say that I'm confident this is the correct solution:
class SecureHeadersMiddleware(BaseHTTPMiddleware):
    async def dispatch(self, request, call_next):
        response = await call_next(request)
        secure_headers.framework.starlette(response)
        return response

app.add_middleware(SecureHeadersMiddleware)

Thank you @fnep! I’ll test and adjust the documentation.

@cak My code from above is probably not the best for an example. There are other issues with it that i discovered later.
Im using this now:
class SecurityHeadersMiddleware:
    """Middleware to add security headers to the response."""

    def __init__(self, app: ASGIApp) -> None:
        self.app = app

    async def __call__(self, scope: Scope, receive: Receive, send: Send) -> None:
        if scope["type"] != "http":
            return await self.app(scope, receive, send)

        async def send_with_security_headers(message: Message) -> None:
            if message["type"] == "http.response.start":
                headers = MutableHeaders(scope=message)
                for key, value in secure_headers.headers_tuple():
                    headers.append(key, value)

            await send(message)

        await self.app(scope, receive, send_with_security_headers)

Hi @fnep , I have updated the docs on the new version v.1.0.0 of secure.py: https://github.com/TypeError/secure/blob/update-v1.0/docs/frameworks.md#starlette
from starlette.applications import Starlette
from starlette.middleware.base import BaseHTTPMiddleware
from starlette.responses import Response

from secure import Secure

secure_headers = Secure.with_default_headers()

async def homepage(request):
    return Response("Hello, world")

class SecurityHeadersMiddleware(BaseHTTPMiddleware):
    async def dispatch(self, request, call_next):
        response = await call_next(request)
        await secure_headers.set_headers_async(response)
        return response

app = Starlette(debug=True)
app.add_route("/", homepage)
app.add_middleware(SecurityHeadersMiddleware)

Thank you for the update. I need to check out what will change with 1.0. :)
Anyhow, i think my answer is outdated these days. Probably the class SecurityHeadersMiddleware from my later comment is the better choice. It looks like Starlette also plans to deprecate BaseHTTPMiddleware. https://github.com/encode/starlette/discussions/2160

Thank you again for opening the issue! Absolutely, if you’d like to check out the new documentation for v1.0.0, it’s available at https://github.com/TypeError/secure/tree/update-v1.0. I plan to merge it into main tomorrow (9/27) and release v1.0.0 on PyPI on Saturday (9/28). I’ll close this issue once it’s merged. Any feedback is greatly appreciated!

I just upgraded my project to v1.0 and really appreciated the cleanup. 👍

Outstanding! Thanks so much for the feedback! I’m really glad to hear the upgrade went smoothly. Thanks again for filing the issue and helping make secure.py even better! Happy coding! 🚀


Aliens are taking over my project!!!

No matter what I do, the database name remains BlazorHero.CleanArchitecture. I even tried context.Database.Migrate() in Program.cs and the name is still BlazorHero.CleanArchitecture. Aliens are taking over the project!!

  "Logging": {
    "LogLevel": {
      "Default": "Information",
      "Microsoft": "Warning",
      "Hangfire": "Warning",
      "Microsoft.Hosting.Lifetime": "Information"
    }
  },
  "AllowedHosts": "*",
  "AppConfiguration": {
    "Secret": "S0M3RAN0MS3CR3T!1!MAG1C!1!"
  },
  "ConnectionStrings": {
    "DefaultConnection": "Data Source=(localdb)\\mssqllocaldb;Initial Catalog=BrokenDatabase;Integrated Security=True;MultipleActiveResultSets=True"
  },
  "MailConfiguration": {
    "From"<EMAIL_ADDRESS>    "Host": "smtp.ethereal.email",
    "Port": 587,
    "UserName"<EMAIL_ADDRESS>    "Password": "vAKmWQB8CyPUBg8rBQ",
    "DisplayName": "Mukesh Murugan"
  },
  //For Serilog Configuration, refer https://codewithmukesh.com/blog/serilog-in-aspnet-core-3-1/
  "Serilog": {
    "MinimumLevel": {
      "Default": "Information",
      "Override": {
        "Microsoft": "Error",
        "Microsoft.Hosting.Lifetime": "Information",
        "System": "Information",
        "Hangfire": "Warning"
      }
    },
    "WriteTo": [
      {
        "Name": "Console",
        "Args": {
          "outputTemplate": "{Timestamp} [{Level}] {Message}{NewLine:1}"
        }
      },
      {
        "Name": "File",
        "Args": {
          "path": "Logs\\log.txt",
          "rollingInterval": "Day"
        }
      }
    ],
    "Enrich": [
      "FromLogContext",
      "WithMachineName",
      "WithProcessId",
      "WithThreadId"
    ],
    "Properties": {
      "Application": "BlazorHero.CleanArchitecture.Server"
    }
  }
}```

Found the problem. There was appsettings.Development.json that was not included.


@jjonkman  suggested me to compile using OpenFAST libraries. I am able to compile without any errors. Thanks

@jjonkman  suggested me to compile using OpenFAST libraries. I am able to compile without any errors. Thanks
Originally posted by @abhinaygoga in https://github.com/OpenFAST/openfast/issues/152#issuecomment-697143140
Hello @abhinaygoga
Please, could you tell me what do you mean with OpenFAST Libraries? I think I'm having the same problem.

duplicate of #625 ?
Closing

duplicate of #625 ?
Closing

Hey Andy,
I am replying from my phone. If I remember correctly, I think the thread
you are talking about is to compile standalone BeamDyn driver!?
Whatever it is, FAST and OpenFAST have connecting libraries. You can find a
folder named FAST libraries in the downloaded FAST library (folder). If it
is for standalone BeamDyn compilation, you can just take the BeamDyn
libraries ,fortran code and relevant files from OpenFAST and copy it in
your BeamDyn compilation folder. Then try to compile.
Hope this helps. Cheers
Regards
Abhinay Goga
On Wed, Jan 13, 2021, 17:27 Andy Platt<EMAIL_ADDRESS>wrote:

duplicate of #625 https://github.com/OpenFAST/openfast/issues/625 ?
Closing
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenFAST/openfast/issues/624#issuecomment-759563854,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOUINTP3H4TD6VB3QNIPV7LSZXCYFANCNFSM4VZO3XOA
.

Hey Andy,
I am replying from my phone. If I remember correctly, I think the thread
you are talking about is to compile standalone BeamDyn driver!?
Whatever it is, FAST and OpenFAST have connecting libraries. You can find a
folder named FAST libraries in the downloaded FAST library (folder). If it
is for standalone BeamDyn compilation, you can just take the BeamDyn
libraries ,fortran code and relevant files from OpenFAST and copy it in
your BeamDyn compilation folder. Then try to compile.
Hope this helps. Cheers
Regards
Abhinay Goga
On Wed, Jan 13, 2021, 17:27 Andy Platt<EMAIL_ADDRESS>wrote:

duplicate of #625 https://github.com/OpenFAST/openfast/issues/625 ?
Closing
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenFAST/openfast/issues/624#issuecomment-759563854,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOUINTP3H4TD6VB3QNIPV7LSZXCYFANCNFSM4VZO3XOA
.


[To rel/1.2] Test CI

coverage: 47.946% (+0.02%) from 47.924% when pulling bcad4a0650b885f11d51a0b575c35c15697412ae on ty/new1.2 into 53ff13b713373a416f66a9b1262aa8717d7b41a6 on rel/1.2.


Dropdown mistakenly reclaims focus when clrCloseMenuOnItemClick is set to false

Describe the bug
How to reproduce

See the last example on clarity documentation > dropdowns,
where clrCloseMenuOnItemClick is set to false.
Additionally an input field has been added next to the dropdown example
When the user opens the dropdown, eg selects one of the items,
then clicks on the input element,

See error: shortly the input elements gains focus but instantly it is reclaimed by the dropdown menu.
The dropdown mistakenly reclaims focus when clrCloseMenuOnItemClick is set to false.

Expected behavior
focus is hold by the element the user clicks on - in this case the input field.
Versions
App

Clarity: 2.2.3, v3

Device:

Type: Notebook
OS: Windows 10
Browser Chrome 77

@gnomeontherun Hi Jeremy, are there any news on this issue? thx upfront!

This is the expected interaction/behavior. Clicking away from the dropdown closes the dropdown and returns focus to the element that opened the dropdown.
This behavior mirrors the behavior of native browser elements as well. See the video below. Opening the select options and clicking away does not focus the text input. It closes the select options menu. A second click is required to focus the text input.
https://user-images.githubusercontent.com/2728359/153304815-29fe4a6a-fe57-44d2-8143-23fc47961ebe.mov


window.print() ?

how to open a printer dialog ?
javascript does not work. I guess because its a web view.

Any ideas of this ?
Have been lookng into the golang / c support for it and there are a few ones out there to allow printing the actual webview and opening the OS native Printer Dialog.
I can add some links to show how it works ?

The macOS webview doesn't automatically handle the JavaScript window.print
method. You have to implement the WebUIDelegate's webView:printFrameView:
method.

Thanks !!
Is this feasible to do in the core code @zserge ?

Would be huge for me. Been doing horrible work around with PDF templates to print so far. But being able to print from just the screen would be insanly huge for me at least

method

The macOS webview doesn't automatically handle the JavaScript window.print method. You have to implement the WebUIDelegate's webView:printFrameView: method.

That no longer works. Its deprecated: https://developer.apple.com/documentation/webkit/webuidelegate/1387787-webview?language=objc

@gedw99
I have been out of touch with this for a few years now... let me see if I can dig up some pointers. :)
OK, I am not familiar with the createPDF method you linked. The way I did it for pywebview (a Python alter ego of this project) was using a private API method, which Apple has now (as of macOS 11.0) made public:
https://developer.apple.com/documentation/webkit/wkwebview/3516861-printoperation
There still seems to be no delegate that handles windrow.print(). So you need to override it on the JS side (for every page-load), hook to a handler with invoke and call the above method from the handler.
You can refer this commit from pywebview. It includes also the pagination settings then used by Safari.
P.S. This post from SO claims that the above method has some limitations for scrolled content, etc. I don't remember having met with any (may be the pagination code handles it). If you do, you might want to check out the repo linked in the post.

Thanks @shivaprsd for kindly responding .
I checked that link and it seems it’s only supported on Darwin. No iOS .
also I run the pdf engine ( a golang engine ) inside the browser as wasm and on iOS and desktop as obviously non wasm .
the gioui platform makes this easy .
So I was hoping to just calll the print dialogue giving it my generated pdf . I don’t rely on the webview functionality at all .
—-
Yesterday I was thinking the best approach as via a webview but I now realise that although more work for me and other developers , the pdf approach gives full control to output anything and on any platform .
Hope I am not confusing you and apologies for the change of direction. That’s due diligence though

Have you checked the private version (_printOperaition...) on iOS? I thought WKWebView was portable.
In any case this can print only the webview itself with that. It appears you are converting your content somehow to a PDF and then trying to print that. In that case I am afraid I can't help you. You may need to dig in to PDFKit or AppKit or something.
From what I can see, the createPDF method you linked seems to be for converting the webview content into PDF.  I think you need a need to open a generic print service that accepts PDFs. Have you tried googling that?


dev: add safe singleton and create2deployer to our deployment scripts

Feature Request
Describe the Feature Request
There is a new RIP for rollups which is to simplify adding a list of important contracts that are factories, namely arachnid, createX, create2deployer and safe singleton.

There are 4 contracts in the RIP, so I think we could add the missing two: https://github.com/ethereum/RIPs/pull/30/files#diff-8fb76c4cfd3e4dc31a49e0ecb56ea00561f0f2af4b4d4353c52b62e95fc6a8bbR39
Describe Preferred Solution

For each tx:

create constants with signer and full tx as per the other pre eip155
add their deployment following the others (in deploy_kakarot.py or a in to be created deploy_pre_eip155.py

Might need further scoping, cc'ing @enitrat

vetoing this as they're just permissioned versions of their trustless counterparts CREATEX and Arachnid proxy


"Apply" button times out when a gateway is down

Describe the bug
In a dual WAN configuration, my main WAN is currently down.
When making configuration changes such as firewall rules or IPSEC, clicking on the apply button ends up in a ERR_CONNECT_TIMEOUT in Chrome.
Manually disabling the broken gateway, web UI changes apply in 1 second (as should be).
Marking gateway as down doesn't help.
To Reproduce
In a 2 gateways setup :

Go to 'Gateways' and click 'Mark gateway as down', apply
Alternatively, break your WAN access (DSL router out of sync in my case)
Try to make changes such as firewall rule
Click apply
Wait for the timeout

Expected behavior
Changes should be immediate, regardless of gateway status.
Screenshots
N/A
Relevant log files
I tried to find something in the webUI log, but came out empty-handed. Please let me know where to look.
Additional context
My DSL connection went down (issue on ISP side).
 Each setting change was very long to apply, usually Chrome times out.
Environment
OPNsense 20.1.6 (amd64, OpenSSL) on VMWare VM (2 vCPU 4Go RAM)

I am testing opnsense as a replacement firewall so, no actual WAN connections are plugged
so I ran into this issue.  I was not able to get anything in error_log file from php, the file does not get created.
By clicking on the green triangle to disable the gateway cause the apply to work normally.
As far as I could determine by truss on the lighttpd process the system_gateways.php crashes.

could be related to https://github.com/opnsense/core/commit/f80081f110f45c8a50e11e556cb978488ca19c87

I have OPNsense 20.7.b_224-amd64
I expect this change is more recent

I switched back to production OPNsense 20.1.8_1-amd64
and tried a few more tests, occasionally the process would complete and the success message
would be displayed taking about 32 seconds (just looking at clock)

This issue has been automatically timed-out (after 180 days of inactivity).
For more information about the policies for this repository,
please read https://github.com/opnsense/core/blob/master/CONTRIBUTING.md for further details.
If someone wants to step up and work on this issue,
just let us know, so we can reopen the issue and assign an owner to it.


Collectfast is no longer maintained

While trying to test #4477, I went over the collectfast repo today and noticed that the repo is archived. There was an issue opened a while back to try to look for new maintainers, which didn't really go anywhere. There is also a note at the top of the readme for that.
I don't think it's causing any problems, but if anything arises, we'll probably have to drop the dependency. It's not heavily integrated nor is it linked to any options, so it should be relatively simple to remove. I couldn't find a fork that looked actively maintained and I don't know of a good replacement.
Why keep it? I haven't used it in a while, but I remembered that it provided a massive speed boost to collectstatic which went down from a couple of minutes to a few seconds.
Opening this for visibility, let's see if it breaks

I'm experiencing issues with Django 4.2 and collectfast with connection to s3 just freeze at the end. So switched off it for now but this increase our collect static from several minutes up to a hour because collectstatic copy all images once again there.
It will be really nice if you can fork and start supporting collectfast as maintainers.

I've forked this and spent a bit of time getting it working with 4.2+
https://github.com/jasongi/collectfasta

Hi @jasongi ,
May I ask you then open Issues in your fork in case you plan to actively support it?

@DmytroLitvinov sure - I didn’t realise issues weren’t automatically enabled after forking 😅


Updated package.json

Changed 'peerDependencies' to 'optionalDependencies'.
Please describe the changes this PR makes and why it should be merged:
This removes the warnings in the console when doing npm install.
Semantic versioning classification:

[ ] This PR changes the library's interface (methods or parameters added)

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)

[x] This PR only includes non-code changes, like changes to documentation, README, etc.

These are peer deps for a reason 🤔

peer vs optional, I vote optional so things like npm list don't stderr 👍

they are peer dependencies because optional dependencies are always installed unless the build fails. please see extensive discussion in #1222


InfectionAnnotator

InfectionAnnotator is a replacement for CountAnnotator. It differs in a few key ways:

The approach is heavily influenced by CountAnnotator. Writing this, I tried to implement what I thought were the main take-aways from CountAnnotator, but with hindsight, and more tailored to the kind of stuff we need with IBIS vs. with GRITS.
InfectionAnnotator stricter with what it includes. It only includes items which seem to describe an infection, hospitalization, death, or generic "case", and which include a number (including things like "a case of", detecting singular definite and indefinite articles).
In other words, it generally favors precision over recall.
It leverages spaCy's parse tree heavily, which allows it to be more precise with the kinds of syntactic forms it defines.
This means that there are syntactic arrangements it has not been told to look for, which it just doesn't pick up. This is fine, because we can implement these later, and I'd rather not pick up things accidentally. I think I hit the stuff that constitutes the majority of the things we try to annotate.
Some additional attributes that CountAnnotator generates are not yet implemented — things describing the number, like "average", "min", and "max". Those will be implemented in future versions.
I'd love to do a walkthrough of how it works.
I already have some ideas for how we might want to change this, but I figured it was better to freeze it as is, and to use those ideas in future iterations.

[From the old PR:]
After revising it a whole bunch and adding tests, I think this PR is ready to begin a final run up to merging.

There are features not yet implemented from CountAnnotator
I have thoughts about some additional changes, especially in terms of coming up with a pattern for housing multiple separate types of stuff we extract, because I want to keep different workflows modular.
Docstrings need to be written properly; I'll do that on Monday.
There's probably a bunch of other issues which will come up, but this is a good point to start a review :)

@nathanathan This now works with the updates to spaCy document handling, and the code has been somewhat reorganized.


Emoji's showing Square Boxes

Hello,
I use the package to generate screenshot for various links and webpages. I am getting an issue rendering Emojis, as whenever I take a SS, the emojis are all turned to boxes. Do I need to install any specific plugin or anything

Seems like a duplicate of #8

Going to close this out in favor of #8. Thanks for raising!


adding a "prebuild versions available for ..." matrix to the readme

It would be super useful if the readme could mention which versions of the library have a prebuild binary available for which combination of os/node version. Because having to compile this is a dead stop for anyone who isn't a hardcore dev with visual studio installed. (especially the part where Node, a scripting language, needs Python, a completely different scripting language, just to install its own modules)
(trying to install for latest LTS 16.12.0 on Windows, and it sees no precompiled binaries, and then it hard crashes because I don't have a full source compile build chain set up, nor will that ever be the case =)

The available prebuilt binaries are listed as assets under each release: https://github.com/JoshuaWise/better-sqlite3/releases/tag/v7.5.1

Please still update the README.md  though, discoverability is  hard and having it in the README is so much easier for folks to find.


👌 IMPROVE: Making in-page TOC title configurable

This PR makes in-page TOC title configurable with the introduction of the following key:
html_theme_options = {
	"toc_title": "Some other title",
}

The default value of the title has been kept as Contents.
It will throw an error if the title is set to empty (I reckon a title is always necessary?)
@choldgraf is there any exception handling format we need to follow for this repo. or the way I have handled it is ok?
fixes https://github.com/executablebooks/sphinx-book-theme/issues/297

thanks very much!


Fixed loading local file bug

This PR fix an error when assigning a local path to asset src.

also, just after this merge, pushed new fresh version to npm :)


New Login from Bulgaria

I keep on getting an email notification that there is a new login from Bulgaria whenever I use the spot app. This hasn't happened before, and I've changed my password already, but since I use spot, I keep on getting these notifications.
Is there a service that is used so that you can login that is hosted in Bulgaria? Is it just me that is getting random logins? Any help would be much appreciated!

Spot triggers a new login e-mail every time it needs to get a new short-lived token. https://github.com/xou816/spot/issues/374 https://github.com/xou816/spot/issues/163
It authenticates directly against the Spotify infrastructure.
If your e-mail say the login is originating from Bulgaria, then Spotify probably thinks that your public IP belongs to Bulgaria. Are you using a VPN? That could be the cause for incorrect geolocation. (Assuming that you aren't located in Bulgaria.)

No, I am not using a VPN, but maybe my ISP is doing something with that. At least I know it's not an intruder ;)

Hi! I'll work on @Louis9902 's solution eventually, this should solve the receiving way too many emails part. As for the location, that's definitely out of our hands :)

@xou816 Great! Actually, I encountered this problem in multiple places (the Bulgaria issue). Maybe my ISP has some weird connections with Bulgaria


Problems with attaining Token.

When using the URL
https://account.withings.com/oauth2_user/authorize2?response_type=code&client_id=183e03e1f363110b3551f96765c98c10e8f1aa647a37067a1cb64bbbaf491626&state=OK&scope=user.metrics&redirect_uri=https://wieloryb.uk.to/withings/withings.html&
to request the token, I now get the error: {"errors":[{"message":"redirect_uri_mismatch: The redirect URI provided is missing or does not match partner callback url"}]}
Indeed the redirect URI gives
Not Found
This worked on one of my computers, but now I am setting up again in Docker it no longer works.

Hi there,
We use GitHub issues as a place to track bugs and other development-related issues.
Please see the link below to our dedicated support line:
Help Center :  Click Here
Ticket ID: WR240
Note: Click on the live chat icon at the bottom corner of the page to start a conversation.


🌍Turkish Language update!🌍

🐌Turkish language option🐌

I make some changes, as I want the config file stick to English, as well as the Readme! Other than that, thanks for your contribution


user given input date changing to timezone

In some cases, i dont want to convert user given date time to timezone instead i want as it is date time given by the user. Is there any setting for this.

I seem to be having the same issue, my user in EST is choosing 00:00 but it's coming through as 05:00 for some reason.

If all you have is a timestamp and a timezone offset, this is not enough to unambiguously determine the timezone name. See relevant stackoverflow question. You should get this info from the user, as using just the offset is inadequate.
You need the timezone name in order for moment-timezone to be able to set the correct timezone, as most timezone offsets contain multiple actual named timezones.

I dont understand your answer, what im asking is if user given input 06/10/2015 10:30:00 in the $scope value im getting the value 06/10/2015 04:00:00 (since my system tz is Asia\Calcutta (+5:30)), but iwan the user given input as it is what he has selected, that means in my $scope variable i want 06/10/2015 10:30:00. Is there any setting for this to work.

If you don't provide a timezone, then your machine local timezone will be used. However, everybody's local timezone may be different, so 12:00 UTC will be shown as varying times in different time zones. If you want to show the same time to your other users as a specific user's selected date, then use their named timezone.
For example, if a user is in the Asia/Hong_Kong timezone, their offset is +8:00. However, just because their offset is +8:00 does not mean they are in the Asia/Hong_Kong timezone. Therefore, in order to reliably display times in the Asia/Hong_Kong timezone for everybody, just the offset is not enough (as this maps to multiple possible named time zones).
However, If all you want is to display times in a specific timezone, (such as UTC) then just pass the name of the timezone you are interested in, and you don't need to worry about what timezone your users are in.

yeah, it cleared.


Understanding Demand

I know that I asked this question in item #62 about what Demand was and the response was CFM. But I don't think that is correct. I have never seen a Demand that exceeded 100. Mostly for me its about 35-45, but can go up to 100. My actual CFM limits are much larger (see below).
I am wondering if it actually is the percent that is set for the zone? If you have a zoned system you set each of the zones airflow in the Test section. Don't get me started why they are there.  You can set these for (in CFM):

Heating
Cooling
Blower (I guess for fan only use)

The maximum is based on the air handler you have and the installer decides how much they want each zone to get at a maximum.
Looking over the s30api_async.py file I did not see any place to obtain these fan set points. So if my hunch about percent is correct, then there seems to be no way, other than manually entering the values, to actually determine the CFM for a zone at a given time.

The configuration data for the zone CFM is probably in the "parameters" of the equipment.  Since this is metadata driven there are no predefined elements.  Once you know the PID then you can get or set that data.
It sounds like the zone demand attribute for a zones system is the % of the CFM allocated to the zone whereas in an unzoned system it is the CFM of the blower.   I have not looked at that element in a while because the equipment diagnostics provide a set of CFM related data.

I have one zoned and one un-zoned. The un-zoned has not been on since I started data collection so I will report back what I find next time it fires up.
Since the airflow maxes are operation dependent and fixed, I think what I will do is to capture them in Grafana variables and then multiply against the Demand return via the API for the dashboard. And if it turns out the non zoned one is CFM its variables will all be 1.

Did a quick experiment: Single zone S30 was set to 78F with ambient at 75F, heating mode. Unit came on and after a few minutes set the Demand at 50. After about 10 minutes I increased the temperature to 80F and then after a few minutes the Demand as 100 and stayed that way for the 10 minutes I ran it.
So in my case both my zoned and un-zoned S30's seem to report the percent of max airflow the unit is configured for. In the advanced settings for the none zoned S30 under Heat Pump I can see there are both low and high settings for heating, cooling and blower.
So maybe when the S30 is connected to a heat pump the Demand reports the current percent of airflow for that operation. Perhaps other type of systems report direct airflow.

That's good information.  I'll get the docs updated.


Error when trying to self-host LuckPermsWeb on node v18

The error error:0308010C:digital envelope routines::unsupported appears when trying to install the server on Node v18.
Downgrading to Node v16 fixes the issue.
StackOverflow Article: https://stackoverflow.com/questions/69692842/error-message-error0308010cdigital-envelope-routinesunsupported
https://discord.com/channels/241667244927483904/807749585493688320/1068829506015604756

Thanks, I have restricted which Node version can be used with this commit https://github.com/LuckPerms/LuckPermsWeb/commit/14c89128bddceb8d23e6c4af8c9f8ca016bc597f
Ideally the project should be compatible with latest versions of Node so I will keep this issue open until that is resolved.


DLLs missing and Fusion not starting up

Describe the bug
I started the application for the first time, and I just had many DLLs missing errors thrown at me.
To Reproduce
Steps to reproduce the behavior:

Install the application by following the instructions
Startup the application

Expected behavior
A working instance of the Fusion app.
Desktop:

Version number of Autodesk Fusion 360: 2.0.12376
Graphics card model + Graphics card driver (Version number): AMD RX580 + mesa from Arch Repos
Operating system (OpenSUSE Leap 15.3, Fedora 34, ...): Arch Linux
Desktop environment (XFCE 4.16, Gnome 40, ...): Openbox

Additional context
Got the output while trying to execute F360DIR="$HOME/.wineprefixes/fusion360/" env WINEPREFIX="${F360DIR}" wine C:\\windows\\command\\start.exe /Unix "${F360DIR}/drive_c/ProgramData/Microsoft/Windows/Start Menu/Programs/Autodesk/Autodesk Fusion 360.lnk" from (https://github.com/cryinkfly/Autodesk-Fusion-360-for-Linux/issues/44#issuecomment-898090000).
I've pasted the output on the clipboard site as Github did not allow me to paste such a big output.
https://online-clipboard.online/online-clipboard/
Please use code 9865 to retrieve the text.

@Rohit-Rangaraj

Got the output while trying to execute F360DIR="$HOME/.wineprefixes/fusion360/" env WINEPREFIX="${F360DIR}" wine C:\windows\command\start.exe /Unix "${F360DIR}/drive_c/ProgramData/Microsoft/Windows/Start Menu/Programs/Autodesk/Autodesk Fusion 360.lnk" from (https://github.com/cryinkfly/Autodesk-Fusion-360-for-Linux/issues/44#issuecomment-898090000).

Please use my .desktop files and the launcher.sh! 🙂

Yea, I'm having the same issue, and I'm using the launcher script too:
https://gist.github.com/chabad360/e9cace26c87ae23ad8812dd90a52f628

@cryinkfly Any updates on this?

This seems to be a wine bug, see https://bugs.archlinux.org/task/73994.

@op3 How do I fix it? Should I do a clean install again?

I've downgraded to wine-staging 7.2 and I'm in the process of doing a clean install, it seems to be actually working this time (so far).

I've downgraded to wine-staging 7.2 and I've done a clean install, that seems to have done the trick.

Yes, there seem to be serious problems with the latest wine version at the moment!
That's why I opened a case for this: https://github.com/cryinkfly/Autodesk-Fusion-360-for-Linux/issues/202

So I looked it up and ran more tests!
So it is actually due to the Wine Version 7.3!!!
But there is a newer version released this week...Wine version 7.4 and with it this bug got rid of!!!!!
Here:  https://github.com/cryinkfly/Autodesk-Fusion-360-for-Linux/issues/202#issuecomment-1066049410

@cryinkfly Now it is throwing me this error.
wine: could not open working directory L"unix\\home\\rohit\\.config\\fusion-360\\bin\\", starting in the Windows directory. Application could not be started, or no application associated with the specified file. ShellExecuteEx failed: File not found.

@Rohit-Rangaraj this looks like an issue with the desktop file or something related.
Can you try in a terminal to launch Fusion with bash $HOME/.config/fusion-360/bin/launcher.sh to see if this works at least.

@gromain I tried that and got the following error.
SSL_INIT
Loaded CA certificate '/etc/ssl/certs/ca-certificates.crt'
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 404 Not Found
2022-03-25 01:06:47 ERROR 404: Not Found.

Do nothing!
wine: could not open working directory L"unix\\home\\rohit\\", starting in the Windows directory.
Application could not be started, or no application associated with the specified file.
ShellExecuteEx failed: File not found.```


PR for the script that does the binary build ...

Ok Jonathan ... I went through and made sure variables were set properly, indentations, spacing and have ran it many times.
The other two files are needed too:

git_rm_stale_hints.sh - would fail on Windows night run which we agreed we should do before building the binaries
Makefile - this was due to Z3 in some cases not working so put in the extra check

I got my code review / PR protective suit on, so go ahead and fire away ... I can take it. :)

I just read the whole script and the logic seems good so no major changes ahead. Good work!
There's a few bits & nits which I'm going to flag, but nothing major.
Thanks,
Jonathan

Ok - @protz ... I implemented all the changes.  Please check it over and merge if you see  fit.  One note ... there was an extra Makefile change due to the power pack not being needed. examples/hello/Makefile
Also - I cancelled the fstar CI build .. was killing off my CI builds so other build run. Will run separate one

One question ...
From line 133 of my script, is this the proper thing to do?
git checkout master
git reset --hard HEAD
git pull origin master
If a linux build runs and then I got to the Windows side of things, this will pop up a merge editor window (if I do the git pull origin call manually). I just tried it ... Linux build finished. I then went to a Cygwin window and ran these three steps. The git pull origin master pops up the merge editor.
I have seen it work several times, but sometimes I have seen git get into a weird state where requires a merge and it isn't able to. That weirdness might be me being in the window causing it.

After more testing of it ... it turns out that the git stuff is working as expected. I literally ran a Windows run, then a Linux run, then a Windows run then a Linux run again ... all passed and worked as expected. :)

that git series of commands most likely won't work if you have two concurrent jobs at the same time fighting to commit... I'll be on a plane tomorrow so I'll try to review this then... but the concurrency seems unlikely

Build #7552 passed and successfully uploaded a new binary. This is good for me.
Thanks for solving this long-standing issue! This will be highly beneficial to everyone.


[#2687] Linux detection fixes.

Problem?
We should properly detect LSB-compliant distros and thus Ubuntu.
We should not set linux as OS unnecessary at the end of the Linux section in paver.sh's detect_os function.
Solution?
Use lsb_release binary instead of /etc/lsb-release file as an indicator of a LSB-compliant distro.
Removed superfluous lines.
Have also updated version, something which was forgotten when refactoring OS detection.
How to test?
Please review changes.
Run the tests.
reviewer: @adiroiban

needs-review

all good. thanks!
approved-at  98a3879


content block for twig components

I tried to use some of my existing components. I use the content block of the component e.g. as the button label:
<{{tag}}
    {{ attributes.defaults({ class: classes|join(' ') }) }}
    {% if title %} name='{{ title }}'{% endif %}
    {% if url %} href='{{ url }}'{% endif %}
    {% if url and target %} target='{{ target }}'{% endif %}
    {% if tooltip %} data-content='{{ tooltip }}'{% endif %}
>
    <span>
        {% block content %}{% endblock %}
    </span>

</{{tag}}>

How can I pass text for the block in the story.js ? Is it just missing from the docs or not supported yet?
I know the project is very young and in active development but it would be helpful to know if this is coming or not or if I just missed something in the docs.

Hi, you can provide a custom template in your story to handle a such case: https://github.com/sensiolabs/StorybookBundle/blob/main/docs/features/csf-stories.md#using-a-component-in-custom-template

Thank you - I didn't read properly


Docs: Fix CLI syntax for --bullets

Thanks for this useful tool!

👍  This clarification helps, I spent longer than I am willing to admit publicly working this out


getRecords with defined header produce wrong results

Bug Report

Information
Description

Version
9.11.0

PHP version
PHP 8.1.23

OS Platform
Ubuntu 22.04.2 LTS

Summary
I was trying to use getRecords with header set based on provided example in documentation. Problem is that it is not working
as I expected. It returns wrong result.
Standalone code, or other way to reproduce the problem
Here is sample code (copied exactly from documentation - https://csv.thephpleague.com/9.0/reader/tabular-data-reader/).
use League\Csv\Reader;

$csv = <<<CSV
Abel,14,M,2004
Abiga,6,F,2004
Aboubacar,8,M,2004
Aboubakar,6,M,2004
CSV;

$reader = Reader::createFromString($csv);
$resultSet = Statement::create()->process($reader);
$records = $resultSet->getRecords([3 => 'Year', 0 => 'Firstname', 4 => 'Yolo']);

Expected result
Based on documentation I expect that $records will contain items like:
[
     "Year" => "2004",
     "Firstname" => "Abel",
     "Yolo" => null,
 ]

Actual result
But actual result is:
[
     "Year" => "Abel",
     "Firstname" => "14",
     "Yolo" => "M",
 ]

Am I missing something?

@adalbertus the documentation is in sync with the master which means tried a feature no yet in production. As state in the documentation this feature will be available when version 9.12 is released. Hopefully next week depending on my free time and my last tests to validate the new behaviour.


python3Packages.pyrad: 2.4-2023-06-13 -> 2.4-2024-07-24

Update pyrad to latest commit (4 month old) as they cleaned up pyproject to remove support for old python version and remove problematic dependencies (netaddr 0.8)

Things done

Built on platform(s)

[x] 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

[x] 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
[x] 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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 355422 run on x86_64-linux 1

4 packages built:

python311Packages.pyrad
python311Packages.pyrad.dist
python312Packages.pyrad
python312Packages.pyrad.dist


Layer assignment is different based on document creation method

It was pointed out to me recently that this remains an issue when making documents in a different ways.
Observed Behaviour:

If you create a document via the create document widget, the inital layer(s) which are set in the definition.json file, are the layer(s) is being stored in the new document.
However if you create a document with the add or edit a map feature button, only the public layer is applied and the initial layer(s) appear to be ignored.

This is not ideal, but I think I know how we arrived at this outcome. Features being added to the map by someone who isn't logged in (depending on auth config) or doesn't otherwise have privileges need to be added to public to be accepted. There is a conversation needed about this and Issue #442 as well.

At the time this logic was developed, Nunaliit did not have utilities. Since utilities have been developed, it is possible to specify "assignLayerOnDocumentCreation" on a module.
The idea of assigning layers based on schema was good at a time when we could only filter a map based on layers. However, this approach is starting to feel dated.
The feature where any map feature should be added to the public layer also seems outdated. This should be specified on a per-module basis.

Removed hard coded assignment of layer "public" from document editor. Added ability to specify a layer id only for documents with geometries, in utility "assignLayerOnDocumentCreation"


Alargean/compose sample updates

📃 Description

🤔 Motivation and Context

🧪 How Has This Been Tested?

⚒ Does the PR contain new tests or refactored old ones?

📷 Screenshots (if appropriate)

📦 Types of changes

[ ] New feature (non-breaking change which adds functionality)
[ ] Enhancement to a current functionality
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Bug fix (non-breaking change which fixes an issue. Please add issue link here too).

✅ Checklist

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] This PR contains new tests that cover the new code.
[ ] This PR refactor previous tests to cover the new code.
[ ] This PR is part of a set of PRs that build a bigger feature. If so, please, add here links to previously merged or open PRs.

Hi @AlinArgeanu Good job on the sample updates, especially the new SliderState. 👍 There is an existing issue with the Companion Pane sample, which should not show the top bar on the bottom screen in the dual-landscape mode. And the dark theme probably needs to be disabled since it doesn't fit the new light-blue theme color.
The updated Compose sample(better be updated to match the following one)

The current Kotlin sample

@AlinArgeanu I think the PR is almost ready to merge. Just one more thing, apart from the document, the screenshots in the READMEs here also need to be updated. Would you please include them in this PR?

@AlinArgeanu thanks for your awesome work!


Setting additional (data-)attributes on the rendered script/link tag

It would be nice to be able to set additional attributes on the rendered script/link tags.
In our case, we want to specify "rel" and "title", and also "data-contrast" to indicate that our stylesheet is for the contrast option.
It would be good to change the HTL helper, for example by allowing the specification of a map (or similar), so that any attributes can be specified and we don't have to deal with a hardcoded list of supported attributes. This way, we don't have to do code changes when something new needs to be supported.

@henrykuijpers is this still relevant? can you have a look at the PR?

Oops! totally forgot about this one. Yes, this is still relevant. :) I'll review the PR. :)


Multiple fields don’t work correctly at once

I just noticed a screencast in this article where having 2 fields select meant you could override both with a single editor tab. That’s now how it should work. 😰
https://applech2.com/archives/20210216-ghosttext-for-safari.html

This must be specific to the sublime text plugin. My plugin doesn't behave like that. It opens each field in new tab. See GIF -

It’s a little strange because each field as its own websocket if I remember correctly, so this feels like messages would be mixed in the browser

It’s a little strange because each field as its own websocket if I remember correctly, so this feels like messages would be mixed in the browser

Umm.. I actually couldn't understand what you tried to say... :sweat_smile:

@fregante Can you reproduce this issue using the Sublime text plugin? I just checked and the issue doesn't seem to occur at all. It behaves as intended!

It seems that the video in the article uses Atom, which is similar to the VS Code plugin as well. I haven’t had time to replicate

ST seems to work correctly, but VS Code is super buggy in every way:

the connection doesn't close when closing the editor’s tab, you have to close the whole window
closing the tab will lose the textarea content altogether
enabling 2 fields will work for a moment, but then it will stop as you switch back and forth

So it appears that it's an issue with Atom and VS Code. Closing this issue here


Imagebuilder doesn't copy .dockerignore into the image

https://github.com/openshift/imagebuilder/blob/fbd4f31c568b767447302e08fb3531b33bcbf837/dockerclient/client.go#L136
This is inconsistent with docker. I'm trying to add imagebuilder as an option in our CI to validate we work with it before getting our component into OSBS, but part of our jobs validates that there's no unstaged changes after re-generating some files (to ensure commited generated files look the same in CI as what the developer produced), but this causes issues with that since the .dockerignore is missing in the image after building.

How/why was this merged?  I've not looked at it at all, but it doesn't appear anyone has?

NVM, losing my mind in reviews.  This is an issue, not a PR.  I now crawl back under my rock.


feat: Add --set-node-options=true option

This completely bypasses spawn-wrap, potentially eliminating some
irritating Windows bugs and avoiding use of process.binding.

Coverage increased (+0.01%) to 98.514% when pulling 6b78a7ba5b999dc106f5b600a3db80d143891cac on coreyfarrell:set-node-options into 5c1b7a9c43771f3439af44a1104e5426519e1123 on istanbuljs:master.

This is cool.  When did the support for env.NODE_OPTIONS land?

This is cool. When did the support for env.NODE_OPTIONS land?

node.js 8.0.0 added the option: https://nodejs.org/dist/latest-v8.x/docs/api/cli.html#cli_node_options_options
node.js < 12 does not support quoting the path given to --require so it is not possible to directly do:
process.env.NODE_OPTIONS = `--require ${pathWithSpaces}`

I was thinking about the case where someone clears the environment variable before calling spawn, like spawn(process.execPath, ['foo.js'], { env: { NODE_OPTIONS: 'different' } }).
We could update nyc's wrap.js to munge ChildProcess.spawn kind of like how spawn-wrap does, but without writing a shim file, such that it always puts the environment variable back in.
Then the only hazard would be something like spawn('bash', ['escape.sh']) where escape.sh does NODE_OPTIONS= node foo.js, but that's enough of an explicit step that we should probably respect it anyway.

I was thinking about the case where someone clears the environment variable before calling spawn, like spawn(process.execPath, ['foo.js'], { env: { NODE_OPTIONS: 'different' } }).
We could update nyc's wrap.js to munge ChildProcess.spawn kind of like how spawn-wrap does, but without writing a shim file, such that it always puts the environment variable back in.

I thought of this, I've created https://github.com/cfware/node-preload as a potential alternative to spawn-wrap.  The functionality has enough platform/version edge cases that it probably deserves it's own testing.  I'm actually looking to get it to a state where it might be possible to get the functionality into the node.js core.  My module is not published to npm yet, I want to get a little feedback first.

Then the only hazard would be something like spawn('bash', ['escape.sh']) where escape.sh does NODE_OPTIONS= node foo.js, but that's enough of an explicit step that we should probably respect it anyway.

Agreed.  Also we can't control everything.


don't propagate all transport errors back up from MessageDistribution

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/608/
Test PASSed.


I'd like to join this project's community

Your public repo
https://github.com/tomatcree01/gov4git-identity-public.git
Your public branch
main

Hi, have you contributed yet?  Are you on Discord?

@GlenWeyl, how can I find the Plurality server on Discord?

https://discord.gg/bM3ZYA6b is discord.

Hi, have you contributed yet? Are you on Discord?

Sorry for the late response,
I joined the discord but I haven't contributed yet. Im trying to familiarize myself currently. I want to test some of the gov4git app to see if I can break something when I get some time to.

Approved.

 Follow up
On Monday, 18-Mar-24 18:56:03 UTC by Gov4Git dev
@tomatcree01 was added to the community.


Topic as a Trait

Adds a trait to represent the functionality a pub/sub topic may require to allow for multiple bus implementations which may have different delimiters, wildcards, or matchany syntaxes

I need to fix the nats topic mapping stuff I think, some missing things I just realized as I walked away

Added a test case for mapping to/from a nats subject string to something that implements a Topic.
From a subject string to a Topic should require zero to few allocations (no string copying).


Updates for logging and to not send IPv6 addresses

Codecov Report

Merging #187 (ceffb51) into int (ca4f7f6) will increase coverage by 0.06%.
The diff coverage is 100.00%.

:exclamation: Current head ceffb51 differs from pull request most recent head 35097f3. Consider uploading reports for the commit 35097f3 to get more accurate results

@@            Coverage Diff             @@
##              int     #187      +/-   ##
==========================================
+ Coverage   68.00%   68.06%   +0.06%
==========================================
  Files          14       14
  Lines        1472     1475       +3
==========================================
+ Hits         1001     1004       +3
  Misses        364      364
  Partials      107      107

Impacted Files
Coverage Δ

eventhandler.go
60.91% <100.00%> (+0.45%)
:arrow_up:

Continue to review full report at Codecov.

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


incubator/schema-registry: Change delimter in sed

For password fields. ` is less common than / in passwords, and
confluent kafka's API key generator almost always includes a /

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped

@benjigoldberg thoughts on this?