WARN SettingsEditor2: Settings not included in settingsLayout.ts: gettingStarted.hiddenCategories

Opening settings running out of sources:
WARN SettingsEditor2: Settings not included in settingsLayout.ts: gettingStarted.hiddenCategories

Thanks! Renamed it to be under workbench.


perf(sourcemap): improve perf of search_original_line_and_column

this seems to show a decent speed up on perf.
the thinking behind this change is:

subsequent calls to search_original_line_and_column will never look at back at lines it has already searched
binary search is faster in this case as typically idx is only increased by a small amount

Given this, we can skip checking a bunch of lines (that will never return true) improving performance.
This is a very hot path.
$ hyperfine --warmup=3 ./target/release-with-debug/examples/sourcemap-main ./target/release-with-debug/examples/sourcemap
Benchmark 1: ./target/release-with-debug/examples/sourcemap-main
  Time (mean ± σ):      79.5 ms ±   5.5 ms    [User: 65.1 ms, System: 12.2 ms]
  Range (min … max):    76.8 ms … 111.7 ms    37 runs

  Warning: Statistical outliers were detected. Consider re-running this benchmark on a quiet system without any interferences from other programs. It might help to use the '--warmup' or '--prepare' options.

Benchmark 2: ./target/release-with-debug/examples/sourcemap
  Time (mean ± σ):      72.9 ms ±   0.9 ms    [User: 59.2 ms, System: 12.2 ms]
  Range (min … max):    70.6 ms …  75.0 ms    40 runs

Summary
  ./target/release-with-debug/examples/sourcemap ran
    1.09 ± 0.08 times faster than ./target/release-with-debug/examples/sourcemap-main

#7926  👈 (View in Graphite)
main

How to use the Graphite Merge Queue
Add either label to this PR to merge it via the merge queue:

0-merge - adds this PR to the back of the merge queue
hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

there's the op for more perf gains here.
SourceMapBuilder::tokens is just a Vec with no initial size. we can probably guess the size based on the length of the text to avoid some allocations - even a rough guess should be ok.
poc:

Summary
  ./target/release-with-debug/examples/sourcemap ran # reserving vec capacity
    1.03 ± 0.04 times faster than ./target/release-with-debug/examples/sourcemap-with-opt1 # this PR
    1.11 ± 0.05 times faster than ./target/release-with-debug/examples/sourcemap-main  # buiilt against main

https://github.com/oxc-project/oxc-sourcemap/blob/0e5ec37a00d6c563727814b9ec41232b226eff90/src/sourcemap_builder.rs#L19

Very nice!
Did I write this code?? It looks familiar. It also looks embarrassingly messy! I'd probably write it differently if I wrote it today. I think I've got a bit better at coding over past 6 months...

yes and no i think? i think @ underfin wrote this originally, then you made some perf improvements in https://github.com/oxc-project/oxc/pull/4641

If you've enjoyed this and you'd be interested in pushing for further gains, I have a few thoughts of other optimizations we might be able to make, which I guess I didn't see first time around.

👀 let me know what you can see

👀 let me know what you can see

I'll open an issue once we have this PR merged.

Unrelated to this PR, but I just want to say that the conversations you two have in your PRs/issues are super helpful to somebody unfamiliar with Rust (or any low level languages).
Thanks for the explanations @overlookmotel and @camc314 for always asking questions.

Unrelated to this PR, but I just want to say that the conversations you two have in your PRs/issues are super helpful to somebody unfamiliar with Rust (or any low level languages).
Thanks for the explanations @overlookmotel and @camc314 for always asking questions.

❤ this is amazing to hear. overlookmotel always has the answers
now just to make rust a part of my day job 🙂

@camc314 I hope you don't mind, but I thought I'd give it a go. I've pushed another commit with an optimized implementation of search_original_line_forwards.
I didn't want to use #[cold], but I couldn't find another way to get the assembly in the shape I wanted - fastest path for "on the same line as last time", followed by the unrolled loop covering next 16 lines.
https://godbolt.org/z/heh1cnYa4 (the unrolled loop is the part where same 3 lines repeat over and over).
Let's see what CodSpeed says!

OK, the perf gain is back to where it was originally, and we don't have the problem of pathologically bad perf if code has moved a long distance any more.
If you think my code looks OK, would you like to apply the same scheme to the backwards search?


Using Android SDK 1.49.1, switching between front and back, the 3D model cannot be rendered.

Describe the bug
Using Android SDK 1.49.1, switching between front and back, the 3D model cannot be rendered.
Smartphone (please complete the following information):

Device: 22041211AC
OS: Android 12

Additional context
https://github.com/google/filament/assets/7814811/6ae4038b-3764-4492-ab71-84d135c449fd

The issue has been fixed already and will be part of the next update.


autodetect: Allow more characters in cluster name

Description
The normal workflow for eks user is to run below command to update
kubeconfig. However, the cluster name in kubeconfig file is following
full ARN format, hence if user uses cilium cli for installation, it
will fail cluster name validation as per below.
This commit is to extend cluster name regex to allow few more characters
e.g. '/', ':' and '-'.
aws eks update-kubeconfig --name tammach-17899

$ cilium install
🔮 Auto-detected Kubernetes kind: EKS
ℹ  using Cilium version "v1.11.1"
🔮 Auto-detected cluster name: arn:aws:eks:ap-southeast-2:926702434684:cluster/tammach-17899
🔮 Auto-detected IPAM mode: eni
🔮 Auto-detected datapath mode: aws-eni
❌ Cluster name "arn:aws:eks:ap-southeast-2:926702434684:cluster/tammach-17899" is not valid, must match regular expression: ^[a-z0-9]([-a-z0-9]*[a-z0-9])$
↩ Rolling back installation...

Signed-off-by: Tam Mach<EMAIL_ADDRESS>Testing
Testing was done for the same cluster name above.

Local testing
$ cilium install
🔮 Auto-detected Kubernetes kind: EKS
ℹ  using Cilium version "v1.11.1"
🔮 Auto-detected cluster name: arn:aws:eks:ap-southeast-2:926702434684:cluster/tammach-17899
🔮 Auto-detected IPAM mode: eni
🔮 Auto-detected datapath mode: aws-eni
🔑 Created CA in secret cilium-ca
🔑 Generating certificates for Hubble...
🚀 Creating Service accounts...
🚀 Creating Cluster roles...
🚀 Creating ConfigMap for Cilium version 1.11.1...
🚀 Creating EKS Node Init DaemonSet...
🚀 Creating Agent DaemonSet...
🚀 Creating Operator Deployment...
⌛ Waiting for Cilium to be installed and ready...
♻  Restarting unmanaged pods...
♻  Restarted unmanaged pod kube-system/coredns-68f7974869-cq2m5
♻  Restarted unmanaged pod kube-system/coredns-68f7974869-xpwnz
✅ Cilium was successfully installed! Run 'cilium status' to view installation health

Closed, as cilium codebase is only supporting cluster name with original set of characters.


Improve the way script waits for printer

The way this script works is that we send a message to the printer, and receive a response in a listener function, which then triggers sending the next message to the printer.
Problem here is that Python thinks we're done as soon as we've send our message, and exists the script. To prevent this I've added an ugly hack by calling sleep(60), which keeps our script running for at least a minute. During this time the listener is still active and can respond to messages from the printer.
It would be great if we could somehow get rid of this hack and implement a better way of making sure we're done before exiting.

Implemented threading to help with this. A pull request has been created. Please test and give feedback on results.


Move most compliance validation to pre-run

Because it is important that when possible, the compliance run
get associated with the converge  that preceded it (via run-id) this PR updates
compliance mode to pre-validate for most common issues before the
converge occurs.
This is a change of behavior, in that previously we would
wait until it was time to send the report after the converge before
validating, which would cause the report to not get captured for current
converge (run-id) if config errors were present.
This also adds error numbers to each of the failure conditions we
detect, in order to simplify providing the operator help with resolving
the errors.
Resolves #11106 and #11105
Types of changes

[x] Chore (non-breaking change that does not add functionality or fix an issue)

Not quite ready for merge, I meant to open this as a draft pending final real-world testing.

The big concern I have here is that if a bad configuration get applied by chef-client, to itself, then that will now cause runs to fail way too early for chef-client to do anything about it.  We have this problem, of course, with other config items, but they're currently pretty minimal and obvious and necessary in a very hard sense.  Is this being driven by a solid customer desire to see this?

There's a pretty easy out for any issues with the compliance phase: remove the attribute or fix the attribute with the token or server URL in it. It's all attribute controlled so any failure can be fixed by either correcting attributes or removing attributes. Why make it a hard fail. There has been a customer driven desire to have this phase be a real phase and not a bolt on experience like it was with the audit cookbook. They view the compliance phase as being just as important (sometimes more important) than their infra converges. The same way we fail the client if we can't reach the server or fetch cookbooks we should be failing compliance hard if it's configured incorrectly. Anytime the user expressly configures something it shouldn't silently fail.

Okay, yeah also the PR was updated to run at run_started which is after build_node, so the user has normal attributes and policyfiles, the only thing that hasn't run yet is cookbook attributes.  And Tim pointed out to me in zoom that the way that it consumes node attributes and not just Chef::Config state gives escape hatches for users.  That all makes me less worried about it.
I do agree compliance mode should fail hard.
If we encounter any future difficulties though we could move it to after compile time before converge starts, to allow cookbook attributes to fix it, or have it just fail hard and noisy at the end.

Oh wait these attributes could be set in a cookbook. We need to account for that.

oh right just for validation.  so you need to move it to converge_start then.  that'll also cover even compile time of recipes.
it might be simpler to just have it blow up loudly at the end...  otherwise someone will try to set the variables inside a lazy block somewhere and get sad.
i'm definitely on board with the idea that compliance blows up the whole run and the engineering on audit mode where by default it was all compartmentalized was terrible.  the compliance phase is just the last phase in the run and if it throws the whole thing fails is entirely fine with me.

(if we don't already by default fail hard at the end of the run talking to the data collector we probably should do that as well)

This has been updated to move the validations to occur at converge_start, to ensure that all cookbook attributes that could affect validations are loaded as well. Also updated cspell.json and fixed the other found spellcheck issue.


DOM Exception 12 for legacy browsers

While developing applications for legacy Samsung Android (4.2.2) (Device: GT-P5110). We were experiencing DOM Exception 12 errors while setting responseType.
Fix SimpleWebRequest.ts line 371
const responseType = SimpleWebRequestBase._getResponseType(acceptType);
try {
    this._xhr.responseType = responseType;
} catch (e) {
    // WebKit added support for the json responseType value on 09/03/2013
    // https://bugs.webkit.org/show_bug.cgi?id=73648.
    // Versions of Safari prior to 7 and Android 4 Samsung borwsers are
    // known to throw an Error when setting the value "json" as the response type.
    //
    // The json response type can be ignored if not supported, because JSON payloads
    // are handled by mapBody anyway
    if (responseType !== 'json') {
        throw e;
    }
}

Current behavior: DOM Exception 12
After fix: SimpleRestClients were working correctly with JSON payloads

@berickson1 I think this issue can be closed? Or am I wrong?


Video cannot be played in landscape on native iOS app

What happened?
open live streaming, the video cannot be played in landscape on iOS simulator
What was the expected behaviour?
video can be played in landscape
Was it tested on latest Detox?

[ ] I have tested this issue on the latest Detox release and it still reproduces.

Did your test throw out a timeout?

[X] I have followed the instructions under Identifying which synchronization mechanism causes us to wait too much.

Help us reproduce this issue!
Play a video in landscape
In what environment did this happen?
Detox version: 20.5
React Native version: 0.67
Has Fabric (React Native's new rendering system) enabled: (yes/no)  yes
Node version: 16.20
Device model: Simulator
iOS version: 16.2
macOS version: 13.1
Xcode version: 14.2
Test-runner (select one): jest / other
Detox logs
Detox logs
paste logs here!

Device logs
Device logs
paste your device.log here!

More data, please!
No response

@aaronpliu it would be great to have a small reproduction project to play with. Does it work when you're running the app on simulator with landscape mode manually (without Detox)?

yes.  it's good to play with landscape without Detox.

As said, since I don't have anything to reproduce this issue on, it would be very helpful if you could provide me a mini project that reproduces this issue with Detox so I'll investigate this bug on. @aaronpliu


Machine deployment fails in air-gapped environment when using containerd runtime

when using the containerd runtime the machine controller runs the following command when deploying new machines:
yum-config-manager --add-repo=https://download.docker.com/linux/centos/docker-ce.repo
That is line 120 here: https://github.com/kubermatic/machine-controller/blob/release/v1.43/pkg/containerruntime/containerd.go
In a completely air-gapped installation this fails, causing the setup.service on the worker machines to fail, so the machine deployment hangs with "machine has no node ref" errors.

This is a very urgent issue. It's blocking SVA for their air-gapped showcase at KV Brandenburg.

We don't have that problem in the offline-reference setup. Have you configured containerd mirrors in the same way?
containerRuntime:
  containerd:
    registries:
      "*":
        mirrors:
        - nexus-url
      docker.io:
        mirrors:
        - nexus-url

https://github.com/kubermatic/offline-reference-setup/blob/main/infra/aws/kkp-cluster/kubeone.yaml#L24-L29

This issue can be closed since OSM can be used to customize these configurations. WDYT @holgers66?

I agree. The OSM solved the issue for SVA, so this can be closed.


reduce build time: Initial small wins

I also fixed the benchmarks en passant; I noticed they were broken for a while.

rebased.

Didn't we have git-lfs before?

Now it appears we are throttled again, this time downloading onnxruntime. We should probably cache it on the CI image.


Improvement:

index.html created
index.rst updated
README.rst updated

tests.log

@em230418 a screenshot may help

I approve to merge it now


[flutter_tools] remove context from WebWorkflow

Description
Reduces context usage in WebWorkflow class, removes Mocks from testing.

This one isn't used in google3, so the change is actually non-breaking


docs: Improve documentation

GitHub Issue (If applicable): #

PR Type
What kind of change does this PR introduce?

What is the current behavior?

What is the new behavior?

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tested code with current supported SDKs
[ ] Tested the changes where applicable:

[ ] UWP
[ ] WinUI
[ ] iOS
[ ] Android
[ ] WASM
[ ] MacOS

[ ] Updated the documentation as needed:

[ ] General Doc Update
[ ] Controls Doc Update
[ ] Extensions Doc Update
[ ] controls-styles.md
[ ] lightweight-styling.md (LightWeight Styling Resource Keys)

[ ] Runtime Tests and/or UI Tests for the changes have been added (for bug fixes / features) (if applicable)
[ ] Contains NO breaking changes
[ ] Associated with an issue (GitHub or internal)
[ ] Commits must be following the Conventional Commits specification.

Other information

Internal Issue (If applicable):

Replaced by https://github.com/unoplatform/uno.toolkit.ui/pull/1012.
CI doesn't pass from forks. :confused:


Add padding to text ouput code display

for errors, etc. see #46 for the rest.

this should do it for now:


Is it possible to add analytics queries to the plugin functionality?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Please delete paragraphs that you did not use before submitting.

Hi @Anatoliy-Kas !
Could you please elaborate a bit on your usecase?  We would like to understand what features exactly you are looking for and how are you using analytics currently.


common-updates

azure-common ImplUtils update with
isNullOrEmpty and extractAndFetch  methods.
updates azonfig to use extractAndFetch from ImplUtils

Can one of the admins verify this patch?

Please update the branch and merge.


Add support for alternative languages. For example russian symbols in URI.

Hi.
We are using this fix for russian letters in url for long time already.
Works like a charm

Thank you for you contribution! This is looking pretty good.
Would it be possible to add a few test cases to verify and document the intended behavior?

sure i will add a few test later this week

Oops - apologies, I only just spotted this PR -- I didn't mean to tread on your toes @sadgb !
I've made a different PR with a different approach (#44) but it addresses the same issue (which is basically #19, as well as #34)

Superseded by https://github.com/substancelab/route_downcaser/pull/44


Trailing spaces are lost for memo fields converting dbf with including data.

ℹ Computer information

VFP version:  9.0 sp2
FoxBin2PRG version: latest
Last FoxBin2PRG version w/o bug (if available):

📝 Provide detailed reproduction steps (if any)
Trailing spaces are lost for memo fields converting dbf with including data.
Convert SpaceBug.DBF to reproduce problem
I fixed this problem in added foxbin2prg by replacing space with a placeholder and restore them later.
✔ Expected result
Keep Trailing spaces for memo fields
❌ Actual result
Trailing spaces are lost for memo fields
📷 Screenshots
foxbin2prg.PRG.txt
SpaceBug.DBF.txt
SpaceBug.FPT.txt

@bjornhoeksel
Have you tried this? For me this fails on luValue = .Encode(luValue, .T.), after that the trailing space is removed.

@bjornhoeksel  There are two problems. .ENCODE cleans up trailing spaces, and the TEXT TO construct (only this one) removes trailing spaces if the spaces are the only signs on a last line.
So I look up ENCODE.

ok. Now I'm confused
The only working part on your code is the removed RTRIM. There is a problem in the TEXT TO, but it is not valid for memos.
.Encode(luValue, .T.)  wraps luValue in <![CDATA /> tag anyway, so the trailing space is preserved.
I just get the right db2 with the spaces by removing the RTRIM. The STRTRAN does nothing.
		<RECORD>
			<QRY_JOIN1><![CDATA[action.src_code = contact.src_code AND
action.act_who = qry_who.sc_code AND
action.act_type = qry_type.sc_code                                                                                ]]></QRY_JOIN1>
		</RECORD>

Would you mind to check this?

The problem with TEXT TO is a problem for VARCHAR, if they are have trailing spaces. Char has spaces anyway, but on varchar one might store some spaces, and they will come out of the field, since this is a length, not a space problem.

Hi
Maybe some explanations with some some small samples makes tings clearer.
Fix is two part remove rtrim to keep last spaces last line. And replace spaces with placeholder to restore them later.  Is needed for memos because  trailing spaces can matter.
Problem is with TEXT TO trailing space before line return are removed
See sample prg’s correct and wrong. These places text on clipboard that you can paste. You will see by wrong that spaces are missing.
correct.PRG.txt
wrong.PRG.txt

@Bjornhoecksel
Please send me the config used on the table. Please use
DO FOXBIN2PRG.PRG WITH '-C','path-to-the-table\SpaceBug.cfg'
to create a config file with all settings.
If you use SpaceBug.DBF.cfg, this too.

@lscheffler
Hi
This is the config I used.
Only change in this is that dbf files include data records
FOXBIN2PRG.CFG.txt

@bjornhoeksel
Please create the config as mentioned above. FoxBin2Prg has a chain of inheritance. I like to be sure to have the settings valid.

@lscheffler
SpaceBug.cfg.txt
Created fconfig file  with command DO FOXBIN2PRG.PRG WITH '-C','c:\av win\SpaceBug.cfg'
c:\av win\ is location table

@bjornhoeksel
Thank you.
FYI. If you check the file you send me, there is a list of the inheritance. So, with the first file, I get my default settings involved.  Fix is two part remove rtrim to keep last spaces last line. And replace spaces with placeholder to restore them later. Is needed for memos because trailing spaces can matter.
Problem is with TEXT TO trailing space before line return are removed See sample prg’s correct and wrong. These places text on clipboard that you can paste. You will see by wrong that spaces are missing. correct.PRG.txt wrong.PRG.txt

Oh. I learned the problem in TEXT .. ENDTEXT.  Only the problem is no problem here.
In line 26694 there is luValue = .Encode(luValue, .T.). The important part is  the .T. .   This wraps luValue into a CDATA tag (see around line 26736), that means the spaces are not trailing anymore. So the problem in TEXT .. ENDTEXT does not occur.   So the remove of RTRIM keeps the spaces first, and .ENCODE wraps it.
The problem occurs for VARCHAR and VARBINARY if they have trailing spaces stored. Then the data is not wrapped and cut off.    For a normal CHAR it does not matter if the space is stored.
My solution is not to add (doubtfull) replace strings but to alter the TEXT .. ENDTEXT like
TEXT TO lcText TEXTMERGE NOSHOW flags 1+2 PRETEXT 1+2 additive
<<>>			<<'<' + lcField + '>' + luValue + '</' + lcField + '>'>>
ENDTEXT

this removes the cutoff of the trailing spaces by <<luValue>>, since luValue is now inside a concatenation.
The RTRIM is removed.
Please try the prg attached
foxbin2prg.zip

@lscheffler
Hi
I tried the prg file you send with spacebug.dbf
But the result is incorrect for me.  If I convert back to dbf with the new prg file you send
The first 2 lines in memo have no space after  AND
Result new prg
action.src_code = contact.src_code AND
action.act_who = qry_who.sc_code AND
action.act_type = qry_type.sc_code
Result my version
action.src_code = contact.src_code AND
action.act_who = qry_who.sc_code AND
action.act_type = qry_type.sc_code
This is a problem because this result in a expression that is not valid anymore in memo field because of the missing spaces.

@lscheffler
The sample are not working in chat here all trailing spaces are eaten.
But difference is dat my version has space after and for first 2 lines

Maybe you generate some files with the data, zip and post (zip files work here)

SpaceBug.zip
@lscheffler
Here the zip file with the table maybe some data is lost with send as .txt file

I have run DO FOXBIN2PRG.PRG WITH 'BIN2PRG','SpaceBug.DBF' against the table using the zip and the config you send. See attached, see line 47 with the spaces
This is the prg a send you. just a bit different TEXT ENDTEXT and RTRIM removed.
SpaceBug.zip

I have run a test against varchar with trailing spaces, (creating text, deleting table, recreating table) and the right count of spaces is recreated

@lscheffler
Hi I downloaded the Spacebug .zp you send.
But I see  there is the same problem in de spacebug.db2.
I think we had some misunderstanding.
The problem is not with line 47 but  45 and 46 a space is missing after the AND on those lines
Can you check that?
The version I created keep these spaces but maybe not the prettiest solution.

Sorry, I closed by accident.

@bjornhoeksel
Those spaces. okay. I never recognized them at all. I look

Then I'm pro removing the whole TEXT .. ENDTEXT for char type and the likes and just concatenate the string in full
lcText = lcText + 0H0D0A + '			<' + lcField + '>' + luValue + '</' + lcField + '>'
This removes the oddness of TEXT  - and as far as I see the flags it's just the same + with the trailing spaces
Looks like TEXT .. ENDTEXT is not meant for such operations.
See file attached
SpaceBug.zip
I have expanded the table a bit, you can see it working with other types, in special keeping the two spaces in varchar

@lscheffler
Hi
I checked it and this change work good.
Yes I think its best to avoid TEXT .. ENDTEXT in this situations to avoid strange behavior.

@bjornhoeksel  new version is on Thor

@lscheffler
Thanks!

@lscheffler
Hi I test the latest version. I saw still a problem.
Now the spaces of last line are lost because of the rtrim.
The rtrim  was not removed in code for memo.
The first 2 lines are ok now.
	Changed by: SF 07.4.2022
!
! Changed by: SF
! https://github.com/fdbozzo/foxbin2prg/issues/81 / Trailing spaces are lost for memo fields converting dbf with including data.
!	RTRIM removed trsiling spaces from memo
!
!
luValue = .Encode(Rtrim(luValue), .T.)

@bjornhoeksel How ever I managed to activate this line again.

@bjornhoeksel ReRun ...

@lscheffler
Yes! it works now correct


Please respond to the destructive changes made by LuaSnip.

He removed global variables such as Luasnip_current_nodes.

PR create


onMessage displayNotification not working in foreground on iOS

I have a standard FCM + Notifee setup where Notifee is displaying remote notifications fine in background, but not in foreground.
messaging().setBackgroundMessageHandler(displayRemoteNotification);
messaging().onMessage(displayRemoteNotification);

The callback is called properly in both cases, but displayNotification doesn't yield expected heads-up notification in foreground (onMessage)
const displayRemoteNotification = async (remoteMessage: any) => {
  console.log('👍 Logs fine in both foreground and background');

  let { title, body } = remoteMessage.data;

  notifee.displayNotification({
    title,
    body,
  });
};

Any help appreciated
As always many thx to the developers for open sourcing this saving me days of work

looks like displayNotification doesn't display a notification if title and body are empty. Makes sense I guess

Hey @icecapp :wave: - it does make sense but then it would be better if it threw an error or something. If you do a try/catch was there an error sent back to your code? If not, might make a good PR to remove the confusion of whether the API was called correctly or not


Histogram too spiky

The histogram from VAPOR is very spiky, while other tools generate much smoother histograms.
Using the same data set (/glade/p/cisl/vast/vapor/Bugs/vorticity/wx.vdc) at 1024^3 size, the histogram should be smooth, while VAPOR presents a spiky one.
To reproduce:

Open VAPOR, load the above mentioned data set. Create a volume renderer.
Change to the highest fidelity, go to Appearance tab, change min and max to be 0.1 and 42.
Click "update histogram."
Compare the VAPOR histogram with Sam's histogram with different number of bins.

The following screenshots are from Google Spreadsheet.

No longer applicable.


Allow passing options from template to the MenuBuilder

We were building a menu where we needed to pass some variables from our view to the knp_menu_render function. However in the builder for that menu, the $options array was always empty. With these changes the options successfully get passed through

Hello, I took a second look and noticed additional locations where the options wouldn't make their way to the MenuBuilder so made more modifications and added tests to cover the modified methods.

On a closer look, and after briefly discussing with stof, the idea seems not quite good.
The same change was already proposed in the past, and it was rejected, see for example PR #90 and #337

OK, so can you explain how the docs show passing options to your menu builder but that's not actually possible? Did we mis-understand the docs or how to do this? It seems like a bug but I don't know enough to know what should be happening.

@gnat42 passing options from the template to the menu builder is possible when using the knp_menu_get Twig function to get the menu instance.

Thanks @stof much appreciated!
I'll look to see if a PR for the docs are appropriate to clarify how to do this and submit if its warranted.


Add additional date format helpers from ember-cli-dates?

Hello,
I'm the maintainer of ember-cli-dates. There's definitely a lot of overlap between these two projects as pointed out in johnotander/ember-cli-dates/issues/19, so I was hoping to combine efforts and collaborate on a single, cohesive project. This project has significantly more functionality, but is missing some of the date helpers that ember-cli-dates currently provides.
So, I'm wondering if you all have any interest in adding some of the helpers like day-of-the-week, month-and-day, and month-and-year, etc.? I'd be happy to open up a PR if this is something that this project would be interesting in incorporating. Or, perhaps the helpers could be broken off into an ember-moment-helpers addon, which end users can optionally install in order to incorporate a collection of helpers regarding dates and times?
Any thoughts or insight would be greatly appreciated, I'd love to collaborate and combine efforts.
Best,
John Otander

So, I'm wondering if you all have any interest in adding some of the helpers like day-of-the-week, month-and-day, and month-and-year, etc.? I'd be happy to open up a PR if this is something that this project would be interesting in incorporating. Or, perhaps the helpers could be broken off into an ember-moment-helpers addon, which end users can optionally install in order to incorporate a collection of helpers regarding dates and times?

adding it here seems appropriate for now. We will soon have better support for peerAddons, which would allow ember-moment and ember-moment-helpers to nicely exist.

Any thoughts or insight would be greatly appreciated, I'd love to collaborate and combine efforts.

+10000

:+1:

day-of-the-week: {{moment today 'dddd'}} // Wednesday
month-and-day: {{moment today 'MMMM Do'}} // March 11th
month-and-year: {{moment today 'MMM YYYY'}} // Mar 2015
So, current implementation supports all the mentioned helpers and it may be enough.
http://momentjs.com/docs/#/displaying/
Maybe renaming {{moment}} to something like {{moment-format}} will make more obvious the availability of this feature?
I can also add a few more examples to the documentation if you think it would help.

@johnotander any progress on this?  I'd be willing to help

:+1:, I'd love to see more opinions on @btecu's latest comment. It may makes sense not to add new helpers if the same functionality can be achieved simply by better documenting how to use ember-moment in conjunction with the features moment already provides.

Everything is already renamed.  What's left?

Ah, didn't see that haha. Maybe this issue should be closed and ember-cli-dates README updated with a mention that ember-moment now incorporates that functionality?

Want to update the docs? :)


Add syntax highlighting to attachments example

While reading the README I noticed the attachments section wasn't syntax highlighted.
This PR adds a code fence around it.

Thank you @mhuhot :)


GPAW 21.1.0

compile with:
intel-compiler: icc
intel-mpi: mpiicc
intel-mkl (includes blas, blacs, lapack, scalapack (32 and 64 bit built automatically) and fftw3-mpi wrapper)
fftw3-mpi wrapper: in mkl/interfaces/fftw3x_cdft/ "make libintel64 CFLAGS="-fPIC" (optional interface=ilp64 for 64bit)
libxc-intel
python-intel; including numpy and scipy
-- need to fetch gpaw setups and set GPAW_SETUP_PATH
-- ase 3.21.1 (a separate nix installation)

Ehh moved to the official [PKG-REQ]


Duration audio Infinity

In the following function I send the URL, my problem is that the audio.duration gives me Infinity.
function recordingEndedCallback (urlBlob) {
  var audio = new Audio()
  audio.src = urlBlob
  audio.addEventListener('loadedmetadata', () => {
    console.log(audio.duration)
  }, false)
}

Did you record using the MediaStreamRecorder?
With the MediaStreamRecorder I got the same 'error', if I use StereoAudioRecorder everything works fine, the inconvenient is the file size
10 seconds:
350 KB with StereoAudioRecorder (numberOfAudioChannels: 1)
67 KB with MediaStreamRecorder

I want to capture in mp3 format to upload it to firebase but I get error in infinite audio

@gdurga can you provide a part of code how you get duration?

@vladpoltorin Sure! Here is the code
First record the audio with the MediaStreamRecorder to reduce the file size, like this:
    recorder = RecordRTC(microphone, {
        recorderType: MediaStreamRecorder,
        type: "audio",
        bufferSize: 4096,
        sampleRate: 44100,
        checkForInactiveTracks: true
    });

Then convert the file using ffmpeg to fix the infinite audio duration
ffmpeg -i <original_file> -c copy -fflags +genpts <fixed_duration_file>
The resulting file are pretty much the same file size of the original and it has a defined duration

Also, when I try to record video with MediaStreamRecorder or MultiStreamRecorder I get infinity duration. StereoAudioRecorder works fine, giving the duration of the audio recording and WhammyRecorder results in a valid duration also.
@gdurga
Is there a way to get the original duration without using ffmpeg?

Hi @nikosgi, unfortunately the only way I found to get the duration using the MediaStreamRecorder is using ffmpeg to convert the file after the record :/
Using the StereoAudioRecorder works fine for me too but the file size will be much larger

Here is a very basic example with Django -> https://github.com/gdurga/Django-RecordRTC-example
Converts the file after the upload using the python os.system

Someone mentioned this as well, however I didn't try it myself yet:
https://github.com/legokichi/ts-ebml

Can someone make it work without type-script compilers?

When I record the screen using throw the recordRTC but video length is infinite. Could you please tell me what should I do?

Did anyone get a solution?

I have integrated below solution. (It's working fine in desktop Chrome Version 100.0.4896.75 (Official Build) (64-bit))
Still need confirmation from different OS and browser.

In order to get the duration inside recorded file we have to add EMBL dependency.
I have used webm-duration-fix npm.
By integrating above solution it's fixed my recording duration issue but still need to test in other OS and browsers.

I have integrated below solution. (It's working fine in desktop Chrome Version 100.0.4896.75 (Official Build) (64-bit)) Still need confirmation from different OS and browser.

In order to get the duration inside recorded file we have to add EMBL dependency. I have used webm-duration-fix npm.
By integrating above solution it's fixed my recording duration issue but still need to test in other OS and browsers.

I did some test in android and IOS latest version by today date (14/04/2022).
It's working fine.

can this be used for audio recording as well?

This works for me!
useEffect(() => {
    if (!audioRef.current) return
    const audioContext = new window.AudioContext()

    fetch(audioRef.current.src)
      .then(response => response.arrayBuffer())
      .then(arrayBuffer => {
        return new Promise((resolve, reject) => {
          audioContext.decodeAudioData(
            arrayBuffer,
            buffer => resolve(buffer),
            error => reject(error),
          )
        })
      })
      .then(buffer => {
        setDuration((buffer as AudioBuffer).duration)
      })
  }, [])


orderBy implementation

I definitely need some help with the description with this one,
Description
It's an implementation of the lodash's orderBy function.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x ] My code follows the code style of this project.
[x ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x ] I have checked that there isn't any PR doing the same
[x ] I have read the CONTRIBUTING document.

@iamsoorena Yes. Done. 👍


FIDO fails when requests to confirm on ssh, but works fine for ssh-keygen

For key generation i use following:
SSH_SK_PROVIDER=libwindowsfidobridge.so ssh-keygen -t ecdsa-sk -Oapplication=ssh:windows-fido-bridge-verify-required
Then it asks me to confirm the key, with success.

Then i add public key to the server, with verify-required before the key and save the file

And finally try to connect to my server via ssh -oSecurityKeyProvider=libwindowsfidobridge.so<EMAIL_ADDRESS>which results into this:

Confirm user presence for key ECDSA-SK SHA256:NkBUzghWtXZDS415hNnZlGu1aJIXFE1U0evIhsXej6U
[2022-01-25 23:37:11.057] [wfb-middleware] [debug] Parameters from OpenSSH:
[2022-01-25 23:37:11.057] [wfb-middleware] [debug]     Algorithm: 0
[2022-01-25 23:37:11.057] [wfb-middleware] [debug]     Data:
[2022-01-25 23:37:11.057] [wfb-middleware] [debug]       |        0  1  2  3  4  5  6  7  8  9  a  b  c  d  e  f
[2022-01-25 23:37:11.057] [wfb-middleware] [debug]       | 0000: 00 00 00 20 5a 15 64 a4 8a 45 b0 0d 77 50 e0 d0  ... Z.d..E..wP..
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0010: 6a cc 7a ec fe ea a4 43 52 45 1a 5f bb e1 0d ac  j.z....CRE._....
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0020: 2f 29 d9 d5 32 00 00 00 05 61 64 6d 69 6e 00 00  /)..2....admin..
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0030: 00 0e 73 73 68 2d 63 6f 6e 6e 65 63 74 69 6f 6e  ..ssh-connection
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0040: 00 00 00 09 70 75 62 6c 69 63 6b 65 79 01 00 00  ....publickey...
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0050: 00 22 73 6b 2d 65 63 64 73 61 2d 73 68 61 32 2d  ."sk-ecdsa-sha2-
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0060: 6e 69 73 74 70 32 35 36 40 6f 70 65 6e 73 73 68  nistp256@openssh
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0070: 2e 63 6f 6d 00 00 00 a2 00 00 00 22 73 6b 2d 65  .com......."sk-e
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0080: 63 64 73 61 2d 73 68 61 32 2d 6e 69 73 74 70 32  cdsa-sha2-nistp2
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0090: 35 36 40 6f 70 65 6e 73 73 68 2e 63 6f 6d 00 00<EMAIL_ADDRESS>[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 00a0: 00 08 6e 69 73 74 70 32 35 36 00 00 00 41 04 34  ..nistp256...A.4
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 00b0: 88 81 ea e6 1f 08 12 1b 78 6d 9e 9f 13 cd 66 a7  ........xm....f.
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 00c0: d2 6a a6 1f 12 c1 fc 30 2a 79 3b 6f d9 1e 0a 9e  .j.....0*y;o....
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 00d0: f1 c8 1e 7b 86 5e 4d 8f 3d de b0 ff 9d 0a a1 25  ...{.^M.=......%
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 00e0: 2a 63 1f 33 bc f4 5c 22 71 90 a9 bd 87 40 8b 00<EMAIL_ADDRESS>[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 00f0: 00 00 27 73 73 68 3a 77 69 6e 64 6f 77 73 2d 66  ..'ssh:windows-f
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0100: 69 64 6f 2d 62 72 69 64 67 65 2d 76 65 72 69 66  ido-bridge-verif
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0110: 79 2d 72 65 71 75 69 72 65 64                    y-required
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]     Application: "ssh:windows-fido-bridge-verify-required"
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]     Key handle:
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       |        0  1  2  3  4  5  6  7  8  9  a  b  c  d  e  f
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0000: ec 68 9d 51 79 ab 48 b1 16 72 21 1c f3 fc 09 6c  .h.Qy.H..r!....l
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0010: 19 86 68 aa 98 89 06 ad 9c ec e4 07 9b 5c 2e 1f  ..h..........\..
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0020: 75 8f d1 10 27 41 b1 d8 1b 99 17 b9 b3 b4 40 ff<EMAIL_ADDRESS>[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0030: 8a 82 3d ea 6f ad eb fd d2 cb 63 01 75 bb 90 c4  ..=.o.....c.u...
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]       | 0040: f9 87 e8 18 f4 00                                ......
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]     Flags: 0b00000001
[2022-01-25 23:37:11.058] [wfb-middleware] [debug]     PIN: (not present)
[2022-01-25 23:37:11.058] [wfb-middleware] [debug] Forcing user verification because the environment variable "WINDOWS_FIDO_BRIDGE_FORCE_USER_VERIFICATION" is set to any value
terminate called after throwing an instance of 'std::system_error'
  what():  Failed to get WebAuthN assertion (NotAllowedError): The operation was canceled by the user.
terminate called after throwing an instance of 'std::runtime_error'
  what():  End of stream
client_converse: receive: unexpected internal error
reap_helper: helper exited abnormally
sign_and_send_pubkey: signing failed for ECDSA-SK "/home/w7rus/.ssh/id_ecdsa_sk": unexpected internal error

I use OnlyKey as a FIDO key, running debug multiple times shows no difference in the output.

I got it working using the command in this alias I made.
alias ssho="ssh -oIdentitiesOnly=yes -oSecurityKeyProvider=/usr/local/lib/libwindowsfidobridge.so -i ~/.ssh/onlykey"
You want it to use a specific key, and disregard what's loaded in the agent already. IdentitiesOnly=yes does this. I also have to specify the full path to the libwindowsfidobridge.so, presumably because it's not in my path, but that's fine.


[leak-tracking] Cover testwidgets with leak tracking in test/gestures

Description
This PR changes testWidgets by testWidgetsWithLeakTracking in flutter/test/gestures tests according to the https://github.com/dart-lang/leak_tracker/issues/134.
Tests

Updates transformed_monodrag_test.dart to use testWidgetsWithLeakTracking.
Updates transformed_scale_test.dart to use testWidgetsWithLeakTracking.

Pre-launch Checklist

[X] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[X] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[X] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[X] I signed the [CLA].
[X] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making, or this PR is [test-exempt].
[X] All existing and new tests are passing.

It seems you have auto-formatting on. Flutter does not auto-format: https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo#formatting
Can you revert the format changes?

@droidbg , it seems you reverted leak tracking too :)


Sharding is posting 1 shard server count

My bot is on thousands of servers and has 4 shards. BLAPI is sending to server listings that support shards and don't that it has 4 shards and is only on 912 servers (the first shard's server count).
I am running the latest version of BLAPI from GitHub and I'm not using manual posting.
Why is this happening and how can I fix this?

@pizzafox Could you try using postman to send your stats post directly to the BotBlock endpoint?
Would like to ensure this isn't an issue I'm causing.

Iirc i havent added shard posting to manual post yet, so that won't help
I'll check if im doing something wrong in a few days, when im back

@T0TProduction I meant using postman, not your lib. To ensure it isn't an issue with the BotBlock API.

Ran via Postman. Results:
{
  "success": {
    "discordbot.world": [
      200,
      "{\"success\":true,\"message\":\"Stats updated\"}",
      "{\"shards\":[911,909,897,849]}"
    ],
    "botsfordiscord.com": [
      0,
      false,
      "{\"server_count\":3566}"
    ],
    "bots.ondiscord.xyz": [
      204,
      "",
      "{\"guildCount\":3566}"
    ],
    "discordbots.org": [
      200,
      "{}",
      "{\"shard_count\":4,\"shards\":[911,909,897,849]}"
    ],
    "discord.services": [
      200,
      "{\"success\": true}",
      "{\"server_count\":3566}"
    ],
    "bots.discord.pw": [
      204,
      "",
      "{\"server_count\":3566,\"shard_count\":4}"
    ],
    "botlist.space": [
      200,
      "{\n    \"code\": 200,\n    \"message\": \"Successful at posting server count\"\n}",
      "{\"shards\":[911,909,897,849]}"
    ]
  },
  "failure": []
}

@pizzafox you posted all servercounts within a single post?
Ofc blapi cant do that, you only have one servercount per code running
So sites that "support sharding" but need you to post every single count inside a combined post, obv won't work
(I also strongly disagree that those sites support sharded botcount posting, because they literally don't, they just show the data if you yourself aggregated it 🤔)

What do you mean by "you only have one servercount per code running"?

Blapi is used in the code a single shard is running on, isnt it🤔

My first thought was that this was the case, but it isn't. The code is running on all shards.

Wait what🤔
How is it running on all shards
Isnt the whole sharding to split the load between smaller instances?

Yes it is. The BLAPI.handle function is getting run on every shard. Each shard handles around 900 servers. Is this the proper implentation for BLAPI with sharding?

Yes it is. BLAPI then posts that Count, for every single shard, and lists that really Support sharding then combine those postst
lists that need you to post an Array of guildcounts, Like you did in postman, won't work that way, but BLAPI won't know that array in this setup

So, in theory, the wrong guildcount should only be on a few select lists, while others display it correctly.
I'll make a list of those when im back if you want me to
Ofc there could still be something wrong on BLAPIs side

When BLAPI handles posting the server counts with shards to any listing, it shows 4 shards with the server count being ${the first shard's server count}. I just want my server count to be accurate.

OK that should not be the case for every(or any) site
I'll double check the code after reviewing all the issues when im back (soon™)

@pizzafox could you try posting different stats than your real ones (to see what changed and worked) via botBlock and postman, but in the form BLAPI would use? (4 posts with each only their own shard data, not the aggregated)
It looks to me that BLAPI forwards the correct data to botBlock

@T0TProduction The postman request made appears to have the following data in the request
server_count: 3566
shards: [911, 909, 897, 849]
shard_count: 4
It appears it was a single request as would be expected for bot lists that accept a shards array.

This is resolved with PR #14/#15. Waiting for minor bug fixes on it and then merge with master.

merged #14


⚠ Bofrost.lu has degraded performance

In f203e02, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1644 ms

Resolved: Bofrost.lu performance has improved in b72d463 after 6 minutes.


Add missing currencies + rate limits + allow creation of orders from JSON

Add missing currencies for some exchanges
Add rate limits for some exchanges
Allow creation of orders from JSON

I don't want to make the DTOs mutable, hence the final keywords need to stay. Also, I don't want to allow for annotations like @JsonIgnore. These topics have been discussed before, not that I expect you to have seen them, and I'll refrain from writing out all the reasons here for now.

It would be pratical to have DTOs mutable in order to be able to store them in JSON format, and then deserialise them when necessary (for exemple for testing purpose).
If you don't want to approve this modification, could you please at least commit the others?

You can serialize an immutable object to JSON. That should be no issue.
No, I am not able to select out the changes that we'd like to keep. Can you do it from your end?

@sebastienkoch will you be working on this PR?
if not then will have to close it


Wrong command for dropping Materialized Views

Describe the bug
When switching from   materialized='materialized_view' to   materialized='materialized_view' an incorrect statement is issued
11:40:17 Dropping relation `some`.`dbt_eongaro`.`test` because it is of type materializedview
11:40:17 On model.umc_poc.test: drop materializedview if exists `some`.`dbt_eongaro`.`test`
11:40:17 Databricks adapter: Error while running:
drop materializedview if exists `some`.`dbt_eongaro`.`test`

Steps To Reproduce

Create a new model with  `materialized='materialized_view'
dbt run
change model to `materialized='view'
dbt run
See error drop materializedview if exists `some`.`dbt_eongaro`.`test

Expected behavior
A clear and concise description of what you expected to happen.
System information
12:02:24 Running with dbt=1.7.7
12:02:24 Registered adapter: databricks=1.7.3

Sorry for the delay, the beta with the fix is out today (1.8.0b1).

https://github.com/databricks/dbt-databricks/discussions/595


[Package Issue]:  Microsoft.VC++2008Redist and Microsoft.VC++2005Redist

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Multiple installed packages found.
Brief description of your issue
Microsoft Visual C++ 2008 Redistributable package has 2 identical packages with different names and Microsoft Visual C++ 2005 Redistributable package name is inconsistent with the rest of Microsoft Visual C++ Redistributable packages.
Steps to reproduce
winget search c++
Actual behavior
Winget has the following duplicate packages:
Microsoft.VC++2008Redist-x86
Microsoft.VC++2008Redist-x64
Microsoft.VCRedist.2008.x86
Microsoft.VCRedist.2008.x64
Besides that, the following packages are not named according to the same logic as the rest of Microsoft Visual C++ Redistributable packages:
Microsoft.VCRedist.2008.x86
Microsoft.VCRedist.2008.x64
Microsoft.VCRedist.2005.x86
Microsoft.VCRedist.2005.x64
Expected behavior
I think Microsoft.VCRedist.2008.x86 and Microsoft.VCRedist.2008.x64 should be deleted.
Also, I believe Microsoft.VCRedist.2005.x64 and Microsoft.VCRedist.2005.x86 should be renamed to Microsoft.VC++2005Redist-x64 and Microsoft.VC++2005Redist-x86.
Environment
Gerenciador de Pacotes do Windows v1.3.2091
Copyright (c) Microsoft Corporation. Todos os direitos reservados.

Windows: Windows.Desktop v10.0.19044.2006
Arquitetura do Sistema: X64
Pacote: Microsoft.DesktopAppInstaller v1.18.2091.0

Screenshots and Logs
No response

Also, shouldn't Microsoft.VC++2015-2019 be merged with Microsoft.VC++2015-2022?

Microsoft Visual C++ 2008 Redistributable package has 2 identical packages with different names

The VC++ packages are currently being moved between identifiers. This is a bit of a process because we have to check to be sure any dependencies are updated before removing the old manifests. I would expect that Microsoft.VC++2008Redist-x64 and the similarly named 2005 one will be removed soon, resolving the issue

Also, I believe Microsoft.VCRedist.2005.x64 and Microsoft.VCRedist.2005.x86 should be renamed to Microsoft.VC++2005Redist-x64 and Microsoft.VC++2005Redist-x86.

I believe @SpecterShell was working on moving them all to the new format - `Microsoft.VCRedist... You have the right idea, just that the 2005 and 2008 ones are correct, and the others haven't been moved yet

Also, shouldn't Microsoft.VC++2015-2019 be merged with Microsoft.VC++2015-2022?

No, 2015-2019 and 2015-2022 are in fact two separate packages

No, 2015-2019 and 2015-2022 are in fact two separate packages

2015-2019 and 2015-2022 are the same package as they share the same UpgradeCode. If 2022 is installed upon 2019 the latter will be replaced.

No, 2015-2019 and 2015-2022 are in fact two separate packages

2015-2019 and 2015-2022 are the same package as they share the same UpgradeCode. If 2022 is installed upon 2019 the latter will be replaced.

Really? I wasn't aware of that

Thank you both for answering. Should I keep this issue open for tracking or it's unnecessary?
I'm now facing a problem where Winget doesn't recognize Visual C++ 2015/2022 properly (it shows as 485c6580-376a-450b-9a80-43c390b968a3), even when installed through Winget and restarting Windows between install/uninstall. Should I open a new issue for it?

You can leave this open. I will try to fix these issues during the process.

Nice progress:

It seems the only thing left is removing Microsoft.VC++2008Redist duplicates:

I've updated the issue since the only thing left is deleting Microsoft.VC++2008Redist

I've updated the issue since the only thing left is deleting Microsoft.VC++2008Redist

This is blocked because Apache.OpenOffice still references the old package identifier. There were PRs open for OpenOffice, but they were having some validation errors; I'll have to check on the status of that

This is blocked because Apache.OpenOffice still references the old package identifier. There were PRs open for OpenOffice, but they were having some validation errors; I'll have to check on the status of that

Where can I follow that?

I've updated the issue since the only thing left is deleting Microsoft.VC++2008Redist

This is blocked because Apache.OpenOffice still references the old package identifier. There were PRs open for OpenOffice, but they were having some validation errors; I'll have to check on the status of that

Done.

Thanks! This was fixed! (at least for me)


Make sure GH and ModelDB are in sync (same content)

Out of Sync

126637 -> GH has newer version
139653 -> GH has newer version
246546 -> different content (not sure which is appropriate)
256311 -> ModelDB has newer version
260971 -> different content (not sure which is appropriate)
265584 -> GH has modern MOD updates, ModelDB has changes not retrofitted to GH
3808   -> GH has newer version

The models: 126637, 139653, 256311, 3808, and 266774 are solved.
Models 246546, 260971, and 265584 still need further attention.

Model 267666 (new?) is missing from GitHub.

Model 267355 is also missing from GitHub.

Has this been resolved? I tried locally comparing all of the models from MODELDB with the ones from GH, and have found no differences between the two (modulo some unnecessary files/directories like __MACOSX, .DS_Store, __pycache__, etc.), nor any missing models.
For reference, I used this Bash script for downloading and comparing the files: https://gist.github.com/JCGoran/8e12f8a1aac70203c1f4e8a56171f088

@JCGoran - if the two "unchecked" versions are now synced / same then it's OK to close this ticket. I added you to the @modeldb slack channel on NEURON-Dev where you can see related discussion / post additional questions.

All ModelDB entries (not just NEURON models) should be in GitHub... Although having said this, I wonder if this is true for alternate model entries

@ramcdougal  I've made some ad-hoc Bash scripts which can be used to clean up the models on MODELDB (can probably be used on the GH ones as well, though I haven't tried it) in case it's of use: https://gist.github.com/JCGoran/c179761e47eee831134fa74f8b304061


Outcomes: Using PIDs where possible, fix journal titles, add D8.2.2

Using PIDs like doi.org where possible for persistency. Stripped weird ?newsletter stuff.
Also some of these had wrong journal names or wrong titles.  Please check the other end of the link before adding as both of these matter in academia.
Added D8.2.2 link to Zenodo.
Is there a source file for these that should be updated instead of outcomes.mdx? The bugs seems autogenerated.

Thanks for the suggestions/corrections. Those links were probably taken directly from a newsletter. I'll check the links in the next releases. By the way, we update outcomes.mdx file only.

Thanks for the suggestions/corrections. Those links were probably taken directly from a newsletter. I'll check the links in the next releases. By the way, we update outcomes.mdx file only.


Separate response support

Separate response is required for Border Agent handling COMM_PET and COMM_KA requests from commissioner. Empty Message is required by both client and server. Currently only client implemented this feature.
This PR,

[x] Moved Empty Message from CoapClient to CoapBase.
[x] Verify token before accept a separate response.
[x] Adjusted Empty Message related API.

Codecov Report

Merging #1571 into master will decrease coverage by 0.46%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1571      +/-   ##
==========================================
- Coverage   69.65%   69.19%   -0.47%
==========================================
  Files         156      156
  Lines       20122    20119       -3
  Branches     2483     2483
==========================================
- Hits        14016    13921      -95
- Misses       5201     5302     +101
+ Partials      905      896       -9

Impacted Files
Coverage Δ

src/core/coap/coap_client.hpp
73.33% <ø> (-0.36%)
:arrow_down:

src/core/coap/coap_base.hpp
42.85% <0%> (-57.15%)
:arrow_down:

src/core/coap/coap_base.cpp
64.51% <0%> (-35.49%)
:arrow_down:

src/core/coap/coap_client.cpp
53.25% <0%> (-14.71%)
:arrow_down:

examples/platforms/posix/random.c
85% <0%> (-15%)
:arrow_down:

src/core/net/ip6.cpp
53.91% <0%> (-10.93%)
:arrow_down:

src/core/coap/coap_server.cpp
82.39% <0%> (-7.05%)
:arrow_down:

src/core/net/netif.cpp
56.97% <0%> (-4.07%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

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


Explicitly mark nullable parameter

Implicitly nullable parameter declarations are deprecated in PHP 8.4. The proposed change is safe and is not considered by PHP as a signature change, see https://php.watch/versions/8.4/implicitly-marking-parameter-type-nullable-deprecated

Ping @shadowhand .
PHP 8.4 will be released soon :)

Same as #1039.

Thank you for contributing! 🎉
Leaving a note to say we're not going to worry about the test failures on PHP 5.6 and 7.0, due to the ? syntax. We will be dropping support for these versions.


feat: Lean.MVarId.cleanup configuration

Modifies cleanup so that it takes (1) an array of additional fvarids to preserve and (2) a flag to control whether to include indirect propositions.
(This is wanted in mathlib for the extract_goal tactic.)

(This is wanted in mathlib for the extract_goal tactic.)

@kmill Please include that in the cleanup docstring

@leodemoura Done


Reduced width of 'body' on GenericContentPage when InPageNav is false

When disabling the InPageNav on a GenericContentPage the width of the main body (that holds the markdown content) has a max-width defined. This causes the content to fill just over half of the page. Seems strange, but I don't know whether this is by design?
Behavior:

Expected behavior:

Source: https://developer-portal-sitecoretechnicalmarketing.vercel.app/digital-asset-management/dam
Steps to replicate:
Switch hasInPageNav: false to data\markdown\pages\solution\digital-asset-management\product\dam\index.md

@markvanaalst  this was intentional so that "article" like content is the same width regardless of InPageNav being present,  but I'll update the behavior.


Only write config in environment-to-ini if there are changes (#15861)

Backport #15861

Only write config in environment-to-ini if there are changes

Only write the new config in environment-to-ini if there are changes or the
destination is not the same as the customconf.
Fix #15719
Fix #15857
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>Co-authored-by: 6543<EMAIL_ADDRESS>

make lgtm work


Rename std::math::Exponentiate trait

Perhaps rename this trait to Exponent to avoid the use of a verb for a trait name.

There is now also a trait Exponent in std::math, which could be confusing.
Perhaps we rename this one to Pow or Power ? i.e:
pub trait Power {
    fn pow(self, exponent: Self) -> Self;
}

cc @otrho @mohammadfawaz

So IMO yes, Power is better than Exponentiate due to pow(), and we keep Exponent and exp().
In old school maths libraries exp uses a base of e and pow supplies the base, and this would be what we're doing there in math.h.
Interestingly Rust doesn't have a trait for pow() or exp(), I guess because there's no operator overriding involved.

Isn't this breaking too?

Isn't this breaking too?

That is indeed breaking.


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2023-10-08T01:36:48+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni_istio_release-1.18
b9fc3aead9f1c4882bb6e0fd3158a45408ff49bc
link
true
/test integ-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-cni


Unsafe command buffer

I had the need to use this library with UnsafecommandBufferBuilder. I created my own fork .
Since other people could benefit from that change I was wondering if it could be merged. My code is kind of dirty and has duplicated code but I can clean it up if you are willing to merge it.

Hi! I would be willing to merge it, just create a pull request once you have a version ready to merge.


Add categories for each of the SASDI themes

We need to add categories for each SASDI theme, as described in
http://www.sasdi.gov.za/sites/SASDI/Acts Policies and Procedures/List of appointed base data set custodians - 30 November 2020.pdf
Additionally, each new dataset should have at least one of these categories. Thus we will need to enforce this too

Hi Ricardo,
Not all data set will fit on the themes. Those are only the ones we've prioritized to implement but there are as many as 41 themes that need to be implemented and we anticipate additional themes coming from the Marine and Coastal Environment. The other data set, will have to fit table 15 of SANS1878 standard.


Source code for refinement

It's a very interesting work. The code here can only predict the accuracy. I would appreciate it if you could provide the source code for refinement.

We are still preparing for the refinement code to go public. It would still take a bit of time.

It's a very interesting work. The code here can only predict the accuracy. I would appreciate it if you could provide the source code for refinement.

Hello,
Thank you for your request! Those scripts will be added in this github as well, although may take a bit of time for cleanup. If you can leave your e-mail address here, I will let you know when it's ready.

It's a very interesting work. The code here can only predict the accuracy. I would appreciate it if you could provide the source code for refinement.

Hello,
Thank you for your request! Those scripts will be added in this github as well, although may take a bit of time for cleanup. If you can leave your e-mail address here, I will let you know when it's ready.

Thanks! My e-mail address is<EMAIL_ADDRESS>

Hi,
The code is upload to the "modeling" folder. Let me know if you have questions.


Make tweaks to enable Halite single-player mode.

Allow just a single bot to be specified when launching game environment, which enables single-player mode.  Count productive_squares_remaining every turn, and when in single-player mode, continue game loop while productive_squares_remaining > 0.  Fame and glory to those that completely mine a given board in the fewest frames.

You may want to modify promptNetworking() in main.cpp to only prompt for 1 player before allowing a user to continue.
Otherwise LGTM.

@nmalaguti is correct, but let me add a larger point.  I seem to get dropped into that "prompting" code loop whenever I invoke the game environment with malformed / incomplete command line arguments.  When that happens, I'm completely lost ... I have no idea what it is asking me for.  I have never successfully launched the game environ once I've entered the prompting loop.  So, I always just kill it and fix my input string.  Instead of trying to improve the prompts, I think that code should just be ripped out; if user enters incomplete / malformed command line arg's, just print the --help string and exit, which I think is the pretty standard Unix/Linux/shell way of dealing with errors of this kind.
That said, if you want to do the fix that he suggests, it's on line 147 of main.cpp.  Instead of
 if (np >= 2) {
it should be:
 if (np >= 1) {


View Formatting causing issues when added using list view webpart on modern page

Do we still have these kinds of issue ?
Is there a way to fix them ?
Currently i am trying to something like this :

But when i add this to modern page, it doesnt work

When i investigated, i came to know that this is because of the following CSS

What should i do in such scenario ? .

I think its because of float:left used in rowFormatter
{
"$schema": "https://developer.microsoft.com/json-schemas/sp/view-formatting.schema.json",
"hideColumnHeader": true,
"hideSelection": true,
"rowFormatter": {
"elmType": "div",
"style": {
"display": "inline-flex",
"align-items": "stretch",
"padding": "8px",
"margin-bottom": "16px",
"max-width": "200px",
"border-radius": "8px",
"box-shadow": "4px 4px 8px darkgrey",
"margin-right": "15px",
"float": "left"
},
"attributes": {
"class": "ms-bgColor-neutralLighter"
},
"children": [
{
"elmType": "img",
"attributes": {
"src": "https://qapostennorge.sharepoint.com/sites/CAL/SiteAssets/Facebook.png?web=1"
},
"style": {
"height": "64px",
"width": "64px",
"border-radius": "25%"
}
},
{
"elmType": "div",
"style": {
"display": "flex",
"flex-wrap": "wrap",
"margin-left": "10px"
},
"children": [
{
"elmType": "a",
"txtContent": "Facebook 1",
"attributes": {
"class": "ms-fontSize-s ms-fontWeight-semibold",
"href": "[$Link1]"
}
},
{
"elmType": "a",
"txtContent": "[$Link1.desc]",
"attributes": {
"class": "ms-fontSize-s ms-fontWeight-semibold",
"href": "[$Link1]",
"title":"[$Link1.desc]"
}
},
{
"elmType": "a",
"txtContent": "Facebook 1",
"attributes": {
"class": "ms-fontSize-s ms-fontWeight-semibold",
"href": "[$Link1]"
}
},
{
"elmType": "a",
"txtContent": "Facebook 1",
"attributes": {
"class": "ms-fontSize-s ms-fontWeight-semibold",
"href": "[$Link1]"
}
}
]
}
]
}
}

The Issue is still present, i need a fix here, too.
You can use tile-view.json to reproduce -> on a modern webPart page it "jumps".
In my Opionen it is caused by styles not being loaded on the webpart page:

Uncheck the overridden min-height 0 to reproduce webpart page behaviour
also hover classes like "ms-bgColor-themePrimary--hover" dont work on webPart Page

problem is not limited to the webpart
same issue also directly on the list view

For now we cant do anything as this issue is with main office ui fabric css

https://github.com/SharePoint/sp-dev-list-formatting/issues/149

The fix for this issue is done in the host application (Sharepoint) itself. It is in the deployment pipeline now and should be live in next couple of weeks!

The fix for this issue is done in the host application (Sharepoint) itself. It is in the deployment pipeline now and should be live in next couple of weeks!

Thanks svaibhav, is there any way we can track when this fix begins rolling out?

This is fixed for me:-)

This is fixed for me:-)

Oh, wow, it is for me too. I've been waiting for this, now to remember the many times I've wanted to use it. Hope they fix the class render issue soon.

@harshdamaniahd @furler This was rolled to 100% production around June 11, just in case you need this info.

Actions don't behave the same on a SP Page as they do on a List's page:
When you place the List on a Modern Page, the actions, like defaultclick, do not function the same as they do on the actual List. For example, on the List itself, when you have the defaultclick action, it behaves like the gif for this view formatting example, where the Details Pane slides out on the right side of the page. However, on a SP Page, you are taken to an entirely different tab to view the Item details. (Same experience for Edit functionality).


example of using Vue component as customize component in Canner

https://github.com/akxcv/vuera

I've tried to import vue components into Canner, and found out it still not a mature technology. Moreover, I think we should focus on build CMS for one community (React) instead of two at this moment, which might cause unexpected issues (such as dynamic import, etc.. problems).


Behavior of /[dynamic].ext routes together with /static.ext

Proposing a feature of an error (or a warning) being returned to the developer when trying to create a single /[dynamic].ext route in the same folder with /static.ext routes, or when creating multiple /[dynamic].ext routes in the same folder. Doing this may lead to an unexpected behavior and thus should be discouraged.

It should be obvious which file will be returned instead of Fresh choosing one over the other.
Related issue: https://github.com/denoland/fresh/issues/461

@synalice I think it uses the static routes first over the dynamic ones (priority wise).
It works fine creating a new project and adding static routes on the relative path to dynamic ones

@iamsahilsonawane, yes, I know it does choose static over the dynamic and works fine. What I'm saying is that most of the time it wouldn't be a wise decision to put dynamic routes together with static, and the developer should be warned about what's he doing.

I would be interested in helping with this (and getting involved generally)
@lucacasonato - you tagged good-first-issue
Would you (or anyone) be able to give any pointers as to where and how you would like the warning to be shown? I'm currently thinking it would show in the terminal output where the developer has started the process?

Would you (or anyone) be able to give any pointers as to where and how you would like the warning to be shown? I'm currently thinking it would show in the terminal output where the developer has started the process?

IMO yes, there should be a warning in the terminal. Also, these warnings should be reflected in the docs.

IMO yes, there should be a warning in the terminal. Also, these warnings should be reflected in the docs.

Ok sure I can have a go a figuring this out.


Improve local db usage and load_representative points

Let's not merge this until we test that it works for loading the representative points in the typical use case. I am not convinced that it does.


App crashes when selecting the default simulator

Steps to reproduce:

Make sure "Show default simulator" option is turned on in the preference
Launch the app
Select the very first simulator item from the sidebar, i.e. the one reads "Default"
App crashes

It seems that the selectedSimulators property in SimulatorsController is unable to locate the selected simulator(s) when a simulator has "booted" as its udid.

@dyang thank you for reporting this!

@davedelong Thanks for the quick fix!


What is the Acton idiomatic way of truncating digits in a float?

How can we truncate the number of digits in a float? Going via a string is certainly one way, but it feels meh? Going via integer?
a = 0.151
b = float("%.2f" % a)
c = int(a * 100) / 100

Should we have some more explicit function for float truncate? @sydow WDYT?

actor main(env):
print (round(3.14159265,2))
env.exit(0)
prints
3.14
31415926535000

On 10 Nov 2023, at 11:12, Kristian Larsson @.***> wrote:
How can we truncate the number of digits in a float? Going via a string is certainly one way, but it feels meh? Going via integer?
a = 0.151
b = float("%.2f" % a)
c = int(a * 100) / 100
Should we have some more explicit function for float truncate? @sydow WDYT?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Sorry, I missed to paste again the extended program with the line
print(round(31415926535,-3))

On 10 Nov 2023, at 11:19, Björn von Sydow @.***> wrote:
actor main(env):
print (round(3.14159265,2))
env.exit(0)
prints
3.14
31415926535000

On 10 Nov 2023, at 11:12, Kristian Larsson @.***> wrote:
How can we truncate the number of digits in a float? Going via a string is certainly one way, but it feels meh? Going via integer?
a = 0.151
b = float("%.2f" % a)
c = int(a * 100) / 100
Should we have some more explicit function for float truncate? @sydow WDYT?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Haha, that was easy. Thanks!

which exposes a bug!! :-)
The intention with negative second argument to round is to round the number so that it ends with that number of zeros…
It should print
31415927000
Back to work..

On 10 Nov 2023, at 11:20, Björn von Sydow @.***> wrote:
Sorry, I missed to paste again the extended program with the line
print(round(31415926535,-3))

On 10 Nov 2023, at 11:19, Björn von Sydow @.***> wrote:
actor main(env):
print (round(3.14159265,2))
env.exit(0)
prints
3.14
31415926535000

On 10 Nov 2023, at 11:12, Kristian Larsson @.***> wrote:
How can we truncate the number of digits in a float? Going via a string is certainly one way, but it feels meh? Going via integer?
a = 0.151
b = float("%.2f" % a)
c = int(a * 100) / 100
Should we have some more explicit function for float truncate? @sydow WDYT?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Added a small example of float round() in https://github.com/actonlang/acton/pull/1576.


Feature: Ability to connect to self signed certificated wss server

as in title, very self explained.

Hello there!
Weasel already does have that ability. In better words: your browser does.
In terms of connectivity and security features, Weasel inherits everything from your browser.
We don't need really to reinvent a very good working wheel.
Modern browsers avoid self-signed certificates by default for security reasons.
You should tell them that you know what you're doing, and they should trust that very certificate.
Here is how you can do it in Firefox (~ v.116.x.x):

Head to Privacy and Security settings (i.e. about:preferences#privacy)
Find Certificates section and press View Certificates
In the opened popup, switch to the Servers tab
Press Add Exception
Enter your server's location where the cert can be fetched from, then press Get Certificate
NOTE: You probably need to use https:// here instead of wss:// to make it work.
You can now view and Confirm Security Exception
OK and you're done!

I hope you find it helpful.


Update: Mozilla.Firefox.DeveloperEdition.mr version 133.0 (133.0b2)

Created by 🥟 Dumplings in workflow run #9186.

Log
Updated: 133.0b1 → 133.0b2
Submitting WinGet manifests
Found existing pull requests:
Update: Mozilla.Firefox.DeveloperEdition.be version 133.0 (133.0b2) - https://github.com/microsoft/winget-pkgs/pull/187117
Update: Mozilla.Firefox.DeveloperEdition version 133.0 (133.0b2) - https://github.com/microsoft/winget-pkgs/pull/187110
New version: Mozilla.Firefox.DeveloperEdition.mr version 133.0 (133.0b1) - https://github.com/microsoft/winget-pkgs/pull/186707
The existing pull requests are ignored as the version is updated while the real version isn't
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Fix #70: Remove symlink dependency for source files

This uses relative references in the source files rather than relying on the symlink.

As stated earlier, having the hard-coded paths is a must for the downstream projects. There is nothing wrong, imo, with having "full" paths in each file. It works like namespacing, and the verbosity is much nicer than relative paths too.

That's fine @perry-mitchell - this could potentially fix the symlinking issue, though. (#70)

@Jameskmonger I'm not wholly convinced there is a massive issue, as symlinks are quite trivial (even on Windows it seems). What doesn't seem trivial is how npm manages it's installation procedures - we had some quirks in buttercup-core-web with the symlink some time ago, but I don't completely recall exactly where all the issues were.
Regardless of this learning process, we've had to stick to using the symlinks to ensure that not only our paths are readable, but they work properly when run through browserify (a requirement for the Chrome extension). If they are proven to be a massive problem going forward, we'll definitely reassess their usefulness.

This is sorted in #73 where we've removed symlinks completely.


Stream periodically stops at client after track ends

This happens randomly between different tracks. After reloading page stream continues.
I see the same problem when using BASS: http://www.bass.radio42.com/help/html/322b7ddc-a9f5-21eb-c78a-f5937cd0d2f2.htm
Failed to decode media resource http://localhost:8000/stream. x27 [stream]
Failed to decode media resource http://localhost:8000/stream, error: Error Code: NS_ERROR_DOM_MEDIA_DECODE_ERR (0x806e0004)
Details: virtual mozilla::MediaResult __cdecl mozilla::FFmpegAudioDecoder<46465650>::DoDecode(mozilla::MediaRawData *, uint8_t *, int, bool *, mozilla::MediaDataDecoder::DecodedData &): FFmpeg audio error:-1094995529

Sometimes after page load the stream gives out the full length of the track instead of a per-second download:

And stops stream after finishing with the above error.
I even tried to export all the tracks to the same bitrate, but it didn't help.
const { PUBLIC_EVENTS, SHUFFLE_METHODS, Station } = require("@fridgefm/radio-core")
const http = require("http")

const station = new Station()
station.addFolder("./stream")

const reqListener = (req, res) => station.connectListener(req, res)
const radio = http.createServer(reqListener)

station.reorderPlaylist(SHUFFLE_METHODS.randomShuffle())

station.on(PUBLIC_EVENTS.NEXT_TRACK, async (track) => {
    const result = await track.getMetaAsync()
    console.log(result.title)
})

station.start()
radio.listen(8000)

It doesn't seem to be happening with your radio in the example. Maybe I'm setting something up wrong?

Hey, what are the contents of the stream folder? I assume there might be some issue considering synchronous folder walking

stream folder just contains bunch of .mp3 files without any subfolders

Could you try to use another folder with just 2-3 mp3 files in it and see if the problem reproduces? And also, are there any helpful logs from the server?

this is happening to apple devices listening to my streams too.

Wondering if this issue still exists, I'm too writing my own radio but in Go. I'm using ffmpeg for doing most of the heavy lifting like encoding, removing metadata (like idv3 tags etc). Occasionally, my client ( tag) also disconnects while listening. It mostly occurs when the server is changing songs (like for example one song has ended and its trying to load the next song). @ch1ller0 did you encounter such issues after the idv3 removal fix? Let me know. Thanks.

A hacky solution that I use is reconnect to stream when error occurs.
// audio -> audio element
audio.onerror = function (err) {
    console.error(err)

    audio.setAttribute('src', '/stream')
    audio.play().catch(console.error)
}


Upgrade workflows

[ ] Use $env:GITHUB_OUTPUT instead of ::set-output:: in workflows
Closes #143
[ ] Revert to use -bl switch for build binlogs
Binlogs created via MSBuildDebugEngine = 1 are incomplete. Will probably be fixed in MSBuild 17.4. See:

dotnet/msbuild/issues/7780
dotnet/msbuild/pull/7897
Closes #146

[ ] Codecov: Include net462 tests on Azure Pipelines
Closes #141

Codecov Report

Merging #148 (37b9ccd) into master (e6f7f3a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #148   +/-   ##
=======================================
  Coverage   81.36%   81.36%
=======================================
  Files          17       17
  Lines        1352     1352
  Branches      108      108
=======================================
  Hits         1100     1100
  Misses        227      227
  Partials       25       25


fix(atlassian jira): Select the correct Jira issue key in board sidebar

:star2: What does this PR do?
In the Jira Board issue sidebar, when an issue has a parent issue/epic, then when The Toggle button gets placed, it takes the parent issue number (being the first <a />) instead of the correct issue number.

The issue number in the description will be PROJ-1, which is wrong. This PR changes the code so it will pick the last one, so PROJ-3 here.
:bug: Recommendations for testing
I have tested this on a SaaS Jira instance (at https://*.atlassian.net/). I've tried to test this on public Jira boards but couldn't find any where the piece of code would be applicable to in the first place.
:memo: Links to relevant issues or information
Fixes #1912

@shantanuraj is there anything you need to get this PR moving?


feat(layer): add flag to enable/disable layer

Adds a new layer property introduced in https://github.com/ngageoint/opensphere/pull/899.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Coding standards

Add coding standards to documentation.

Is it really worth leaving this open? Failing that, getting rid of the milestone - there are plenty of other issues which are worthy but haven't been given a specific milestone.


Error in tutorial

let clients = listener.incoming();
let welcomes = clients.map(|(socket, _peer_addr)| {
    tokio_core::io::write_all(socket, b"hello!\n")
});
let handle = core.handle();
let server = welcomes.for_each(|future| {
    handle.spawn(future.then(|_| Ok(())));
    Ok(())
});

error: no method named `then` found for type `tokio_core::io::WriteAll<tokio_core::net::TcpStream, &[u8; 7]>` in the current scope

Ah, looks like there needs to be a use futures::Future; for that to work.

True, just figured out.

Note that we're planning on completely rewriting the tutorial soon, so hopefully issues like this will be fixed shortly!

We've now removed the tutorial in favor of https://tokio.rs/, so I'm going to close this.


Added new & improved underlining feature with new custom syntax

Some important information about my new PR:

Added a new custom syntax -underline-, to provide more dynamic underlining(anywhere within the text)

Now the sub, superscripts & underlining features, are included within the method: _buildTextWithFormatting

Merged, Thank You

Thank you for merging!
On Sun, Aug 4, 2024 at 9:02 PM Imad Laggoune @.***>
wrote:

Merged, Thank You
—
Reply to this email directly, view it on GitHub
https://github.com/Native-Bits-Team/M-Flow/pull/14#issuecomment-2267582000,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZW2422DW2JEORHM4OU6NQ3ZPZCQ7AVCNFSM6AAAAABL6SX4QKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRXGU4DEMBQGA
.
You are receiving this because you authored the thread.Message ID:
@.***>


v0.9.10-rc.5 Docker daemon does not start

creating a new cluster with v0.9.10-rc.5
I'm getting a lot of "File /usr/lib/systemd/system/systemd-timedated.service:31 configures an IP firewall (IPAddressDeny=any), but the local system does not support BPF/cgroup based firewalling. but it might be something to do with CoreOS
Etcd create successfully but controller fails
+00:11:56	ip-10-11-14-217.aws.MYCOMPANY.local: "docker.service: Failed with result 'exit-code'."
+00:12:02	ip-10-11-14-217.aws.MYCOMPANY.local: "docker.service: Start request repeated too quickly."
+00:12:02	ip-10-11-14-217.aws.MYCOMPANY.local: "docker.socket: Failed with result 'service-start-limit-hit'."
+00:13:04	ip-10-11-14-217.aws.MYCOMPANY.local: "kubelet.service: Failed with result 'exit-code'."

I went on the host and tried to start Docker
Job for docker.service failed because the control process exited with error code.
See "systemctl  status docker.service" and "journalctl  -xe" for details.```

```systemctl  status docker.service
● docker.service - Docker Application Container Engine
   Loaded: loaded (/run/systemd/system/docker.service; enabled; vendor preset: disabled)
  Drop-In: /etc/systemd/system/docker.service.d
           └─10-post-start-check.conf, 40-flannel.conf, 60-logfilelimit.conf
   Active: activating (auto-restart) (Result: exit-code) since Wed 2018-04-25 13:56:53 AEST; 5s ago
     Docs: http://docs.docker.com
  Process: 13185 ExecStartPost=/usr/bin/docker pull k8s.gcr.io/pause-amd64:3.1 (code=exited, status=1/FAILURE)
  Process: 13089 ExecStart=/usr/bin/env PATH=${TORCX_BINDIR}:${PATH} ${TORCX_BINDIR}/dockerd --host=fd:// --containerd=/var/run/docker/libcontainerd/docker-containerd.sock $DOCKER_SELINUX $DOCKER_OPTS $DOCKER_CGROUPS $DOCKER_OPT_BIP $
  Process: 13086 ExecStartPre=/usr/bin/systemctl is-active flanneld.service (code=exited, status=0/SUCCESS)
 Main PID: 13089 (code=exited, status=0/SUCCESS)

Apr 25 13:56:53 ip-10-11-14-217.aws.MYCOMPANY.local systemd[1]: docker.service: Failed with result 'exit-code'.
Apr 25 13:56:53 ip-10-11-14-217.aws.MYCOMPANY.local systemd[1]: Failed to start Docker Application Container Engine.

it cannot pull docker pull k8s.gcr.io/pause-amd64 because the daemon is not started
I'm not sure what are supposed to be the values of $DOCKER_SELINUX $DOCKER_OPTS $DOCKER_CGROUPS $DOCKER_OPT_BIP $
So couldn't dig further.
Attaching cluster.yaml (replaced some sensitive values)
kube-aws-v0.9.10-rc.5-cluster-debug.yaml.txt

Thanks @mumoshu , you put me me in the right direction.
https://k8s.gcr.io/v2/pause-amd64/manifests/3.1 is required to be downloaded for docker service to succeeds. k8s.gcr.io is blocked by our company firewall. It wasn't required previously.
It seems many images not come from k8s.gcr.io. I will move them to our own repo and edit cluster.yaml or get this repo whitelisted.
Note our internal repo is much faster, every public repo goes through a proxy and makes it very slow to boot up nodes. We had to bake images into the AMI so far to go around this issue as not all of them could be specified via the cluster.yaml. (and it's a lot of work to go fetch the images)


feat: add variable to configure httpd mutexes

Add MUTEX variable to enable configuration of httpd mutexes.

Context: httpd on Apple silicon has a bug. It will only start when the mutex is set to posixsem.


Adds script to check for node version.

Description
Adds a script to check that the proper node version is installed.
This PR also will introduce NVM to manage different node versions. Currently, there is no hard version set for the node version. Our node version is determined by our Trussworks CircleCI Docker Image - 10.X. The current stable version for recommended use is 10.16.3. The script ensures that Node version 10.16.3 is set. The .nvmrc ensures that nvm will automatically switch the version of Node when using this project.
Reviewer's Note
This work will enable us to use swagger-cli as a pre-commit hook properly as it depends on a recent version of Node.js
Setup

cd /transcom/mymove
make prereqs will inform you that you are missing nvm
brew install nvm will install nvm
Follow the brew install instructions for nvm:

You should create NVM's working directory if it doesn't exist:

  mkdir ~/.nvm

Add the following to ~/.bash_profile or your desired shell
configuration file:

  export NVM_DIR="$HOME/.nvm"
  [ -s "/usr/local/opt/nvm/nvm.sh" ] && . "/usr/local/opt/nvm/nvm.sh"  # This loads nvm
  [ -s "/usr/local/opt/nvm/etc/bash_completion" ] && . "/usr/local/opt/nvm/etc/bash_completion"  # This loads nvm bash_completion

source ~/.bash_profile
nvm install 10.16.3 && nvm use 10.16.3 will install and switch to node v10.16.3
make check_node_version will check the current node version

Code Review Verification Steps

[ ] Have been communicated to #dp3-engineering (will occur once passes review)
[X] Request review from a member of a different team.

@chrisrcoles - I still think there's a lot of value in this PR. Maybe we can't do nvm (yet) but having something explicitly check our node version seems like the right thing to have in our project.


Add Visual Studio MFC support to build dependencies

Improve build documentation to specify that MFC support must be installed with Visual Studio
Does this resolve #24 @forderud?

Thank you Jøger. I hereby confirm that installing MFC fixes the build problem.
However, did you consider instead removing the MFC dependency, so that the projects to a greater extent build "out of the box" with fewer configuration steps? ATL contains most of the COM stuff found in MFC, so there shouldn't be too hard to get rid of MFC.

Duplicates #29


License?

I just discovered your library, and I love the idea of it, but I don't see a license mentioned anywhere in the documentation or code. Unfortunately I really can't (legally) do anything with it unless you explicitly license it! Please take a moment to choose a license and add it to the repository in a LICENSE file. I (and I'm sure many others) would greatly appreciate it. Thanks in advance.

I guess I jumped the gun porting this to Ruby.
But +1 to this request so I can make it into a ruby gem.

Oh hey, Apache License 2.0 added 5 months ago. Thanks, @galaxykate!

Oh, yep, there should always be an apache license on it, but sometimes I
accidentally overwrite it when I check in a new version.   The intention is
that it's free/OS
On Tue, Feb 2, 2016 at 9:06 PM, David Eyk<EMAIL_ADDRESS>wrote:

Closed #6 https://github.com/galaxykate/tracery/issues/6.
—
Reply to this email directly or view it on GitHub
https://github.com/galaxykate/tracery/issues/6#event-536997791.

Sorry! Was looking at the tracery2 branch.

Ah, I must've forgotten to add the license there. Will do right away Thanks.
On Feb 4, 2016 6:47 AM, "Eli Brody"<EMAIL_ADDRESS>wrote:

Sorry! Was looking at the tracery2 branch.
—
Reply to this email directly or view it on GitHub
https://github.com/galaxykate/tracery/issues/6#issuecomment-179877810.


CDA requirements

Apparently, there should be requirement documents that the NHS111 adapter must fulfill based on what it does.  This is particularly crucial when suppliers need to integrate their products and must meet the risks assurance process. A number of elements of NHS 111, are Web service ITK and CDA functionality (which include GP System workflow), and DOS, the previous two can be retrieved from TRUD, DOS can be seen in the NHSD development website.  My question is about CDA requirements, how much of it we need to test against the adapter, as CDA requirements span across adapter (because adapter translates CDA) and GP systems (still need to adhere to the expected workflow).

Not sure i understand the question here, please provide more information on this. STATUS IS NEEDS MORE INFORMATION

I mean this package (download from TRUD), NHS Interoperability Toolkit CDA Functional Requirements
This is the requirement for the CDA document which is transported via ITK. But I am not sure whether NHS 111 is assured against this or not, so I ask this question

@Kusnaditjung can you confirm this is the document name you require:
NHS111DMS-Version2-0-RC1

From a discussion with NHSD architect (David Waters and Adam Lees), the CDA requirements will be needed for the adapter as it contains CDA documents, and NHS 111 appears to be CDA accreditated. All the errors need to communicate as a syncronous response.


Replaces enketo-xpathjs

Replaces enketo-xpathjs with openrosa-xpath-evaluator (beta version).

Woohoooo!! Thanks!! 🎉
I'll be doing some performance testing and am very curious.
This straightforward replacement will help a lot with figuring out a build system that still can create an IE11 build with the old evaluator.
Since it will break custom non-ODK XPath extensions and the existing build system,  I may try to do one more release before that (in the next few days) to squeeze in a feature a client is waiting for. A 6.0.0 release may be appropriate for this (not sure).


pluto_use on non-Pluto script

In Pluto, you can disable built-in keywords by using pluto_use.
pluto_use class = false

local class = require("30log")

-- ...

But in non-Pluto script (PUC Lua script), you can't disable the built-in keywords.
--- library.lua

local class = require("30log")

-- ...

--- main.pluto

local lib = require("library")

-- ...

Will throw following error.
pluto: error loading module 'library' from file 'library.lua':
	syntax error: ./library.lua:3: expected a class name, found '='
    3 | local class = require("30log")
      | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ here: 'class' has a different meaning in Pluto, but you can disable this: https://pluto.do/compat
stack traceback:
	[C]: in ?
	[C]: in function 'require'
	[C]: in ?

So, providing pluto_use for non-Pluto code would be convenient.
--- library.lua
-- @pluto_use: class = false

local class = require("30log")

-- ...

Although you can do workaround by putting pluto_use in the script where it requires library.
--- main.pluto

pluto_use class = false
local lib = require("library")
pluto_use class

-- ...

-- @pluto_use class = false is already valid as of Pluto 0.9.0

-- @pluto_use class = false is already valid as of Pluto 0.9.0

Good to know it's already planned for 0.9.0.


Implement serializing producer typing

:tada: This issue has been resolved in version 0.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


specs2 matchers for server

mostly request matchers and web server request matchers

done without combined matchers for now, not should on what is useful, once enough usages will request something i will implement


Importer chanegs wrt to download block and list style bullets

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix
Test URLs:

Before:

https://main--shredit--stericycle.aem.page/en-us/resource-center/infographics/6-tips-to-protect-your-business-from-fraud

After:

https://import-changes-download--shredit--stericycle.aem.page/en-us/resource-center/infographics/6-tips-to-protect-your-business-from-fraud

Blocked by https://github.com/aemsites/stericycle-shared/pull/283


Error 429 Too Many Requests

I keep receiving the following error, even though I'm pretty certain I'm not sending too many requests at a time. This error is new and I wasn't receiving it before, I don't think I have changed anything significant for this to occur.

I'm unsure how to navigate this, any help would be appreciated.

@teakopp does this have to do with us going public or is it the throttling of Lob?

PR #166 should fix this 🤞🏻

Closed, moved to front-end


add scrapers for WI county and demographic vaccines

dashboard source: https://www.dhs.wisconsin.gov/covid-19/vaccine-data.htm#summary
adds scrapers to track:

cumulative total_vaccine_initiated and total_vaccine_completed at the county level.
cumulative total_vaccine_initiated by race, age, sex, and ethnicity at the state level.

(see below) This currently does not track "unknown" values. They are stored in separate tables than the rest of the data, which means (with the method I've used) a new scraper would need to be made for each. I can do this if it is important/wanted. The proportion of "unknown" values for each section can be seen on the dashboard--at the moment they range from 0.5-8%.
Added tracking for unknown and other categories for race, sex, and ethnicity. Age and the county-level data don't report missing/unknown entries, but they appear to exist.  The sum of all the total_vaccine_initiated entries for the race, sex, and ethnicity trackers all total 1088339, which matches the corresponding value on the dashboard. For age and the county-level data these totals are lower (1088269 for age and 1081070 for the county-level). I don't know if this is normal/if there's an explanation for this, so I wanted to flag it.
It seems to be hinted that there might be county level demographic data somewhere, (the demographical tables were labelled such as ... vax/unvax county. But, the only data in these tables were state level, and there is no county level demographic data on their dashboard.

Thanks team!