React Native 0.62.1 [iOS] - Empty config file in JS

Hey folks,
Just upgraded my project to react-native 0.62.1
In addition, I migrated react-native-config from 0.11.5 all the way to 1.0.0
I've updated my info.plist variables to match the new formatting, removed the old pre-processing scripts, added my build pre-actions copy and xcconfig setup. I believe my info.plist is actually pulling in variables properly, huzzah!
However, inside my actual JS, my Config

Is giving me a blank object.
I'm still loading in the envfiles the exact same way as before.

If other folks upgrading are experiencing this, or if anyone has direction for me, it would be hugely appreciate.

Hi  @ahartzog  Do you have a reproducible demo?

@luancurti I do not, unfortunately other demands on my time mean I can't spin up a brand on 0.62
If this issue is common as people upgrade to 0.62, I'd like to see where we land. If it isn't, then I'll just have to find which esoteric hidden mis-configuration in my project is causing the issue.
Thank you for the response!

I have the same issue. Updated from RN 0.59.10 to 0.62.2 and react-native-config 0.11.5 to 1.0.0

@ahartzog try to remove envfile from /tmp directory.
On mac go to Finder -> Go -> Go to Folder -> /tmp
The ruby script tries to find env file firstly from tmp/envfile. Then if it doesn't find anything searches in ENVFILE variable.
Hope this helps

@makarkotlov good call on clearing out the react-native-config, I had forgotten to do that when updating from manual linkage.
Running the link on it isn't changing anything in the iOS config for me (and autolinking should work right in theory, according to @luancurti in another post)
I'm still not seeing my env variables in my JS - I've noticed that the pre-build script

Creates that envfile inside private/tmp - and then I still have no idea what this file is supposed to be doing or pointing to. Should I point it to my actual env file with an "absolute" url e.g.

Somebody mentioned that this file just works as a pointer to the actual file - I've also tried deleting it like the prior poster mentions but as far as I can tell this has no effect.
Still trying to figure this out here - would appreciate any additional ideas.

I went ahead and tried @maxkomarychev 's new library, react-native-ultimate-config.
So far so good, iOS worked easily.  JS config works, build configs work.
I would consider moving back to this when it's actually a part of react-native-community and supported, but until then I'll be with RNUC.

I am facing issue in reading the config value it is undefined on reading Config.API_URL with the import Config from "react-native-config";
RN 0.61.5
react-native-config 1.0.0
Only issue is with Ios and works fine with Android platform

I am facing issue in reading the config value it is undefined on reading Config.API_URL with the import Config from "react-native-config";
RN 0.61.5
react-native-config 1.0.0
Only issue is with Ios and works fine with Android platform

Could you please give us more info?
How many .env files do you have?
Could you please check content of your tmp/envfile on your PC / Mac ?

I am facing issue in reading the config value it is undefined on reading Config.API_URL with the import Config from "react-native-config";
RN 0.61.5
react-native-config 1.0.0
Only issue is with Ios and works fine with Android platform

Could you please give us more info?
How many .env files do you have?
Do you use pre-action script?
Could you please check content of your tmp/envfile on your PC / Mac ?

i had to restart my machine, it works for me.

For me (RN0.62.2 + v1.0.0), it's not that the config is empty - it's returning the contents of my react-native-config.js, not my .env file...
import Cfg from 'react-native-config';
console.warn('CONFIG', Cfg);

prints:
'CONFIG', { dependencies: { 'react-native-flipper': { platforms: { ios: null } } } }

I suspect it's loading this.

@wbercx But the file with dependencies you've mentioned is called react-native.config.js. Not react-native-config.js.

@wbercx But the file with dependencies you've mentioned is called react-native.config.js. Not react-native-config.js.

🤦 You're absolutely right. Thanks for pointing that out.
It's been a while since one character cost me this much time haha.

If you are using Pods just add (without react-native link react-native-config)
pod 'react-native-config', :path => '../node_modules/react-native-config/react-native-config.podspec'
in your Podfile

I installed the package according to the instructions, and it worked... for a while. After switching branches, with code changes, etc., the config started returning an empty object.
I tried reinstalling modules and pods, clearing the cache, in the end I got the config data, only writing explicitly
echo ".env" > /tmp/envfile

Hello, I'm having the same problem with the latest version, but for Android on Windows.
OS - Windows 10 Pro 64-bit
Node - 12.16.3
RN - 0.62.2
react-native-config - v1.2.0
I've tried everything in the documentation and supposed fixes from random issues.
React-native-config v1.1.0 works, v1.2.0 returns an empty object.
Anyone else having this issue?

Unless I name one of my files to .env I'm also getting an empty Config on iOS.
react-native: 0.61.5
react-native-config:1.2.1

I was able to get it to work by changing the Pre-action Build script to:
echo ".env.stage" > /tmp/envfile

echo ".env.stage" > /tmp/envfile

@MattyK14 Thanks that worked for me!

The README.md instructions seem to be misleading on this one. While at one point it mentions the tmp/envfile in step 6 of the iOS setup, the actual iOS Multi-Scheme section instructs us to use cp ${PROJECT_DIR}/../.env.staging .env as pre-action.
However, I could not get the pre-action work using the cp command. Switching the command to echo ".env.stage" > /tmp/envfile as suggested before works perfectly.

IOS:
<EMAIL_ADDRESS><EMAIL_ADDRESS>this in pre-action worked for me

echo ".env.staging" > .env

this not:

echo ".env.stage" > /tmp/envfile

I have installed react-native-config according to docs and I have empty config.
Sure, echo ".env.staging" > /tmp/envfile - fix this issue.
but why we have such a case?

so I have to correct something
it's very strange sometimes it works sometimes not.
for 9 days this worked:
echo ".env.staging" > .env
for 2 days this worked
cp ${PROJECT_DIR}/../.env.staging .env
and yesterday no one of the above commands worked:
also not this:
echo ".env.staging" > /tmp/envfile
I think the pre-actions are being executed too late,
because on the second run it works sometimes
at the moment i'm copying my stage env variables manually into .env

Hello guys!
I am having this exact same issue with react-native 0.63.2 and react-native-config 1.3.2.
Is there a workarround for this?
Cannot make it work on iOS, Android works fine
Thanks!

My problem was that I mistakenly added echo ".env.staging" > /tmp/envfile to the Run scripts. When changed to use Build scripts instead got this to work.
(Could not get the more clean cp ${PROJECT_DIR}/../.env.staging .env version to work, though.)

@ronkot Thanks for your reply, I have already tried that without success :(
My project structure is
root/.env
root/.env.production
root/ios/etc
root/android/etc
Should I write
echo ".env.production" > /tmp/envfile
or
echo "../.env.production" > /tmp/envfile
?

same error here with the solution from the docs

Is this resolved?
Still having the issue of the empty config file after following the official documentation in step number 6 (iOS).
React Native v0.63.2
React Native Config v1.3.3

I was having this issue today but i was getting values had changed in my .env file. The only way to get the proper env vars was to delete everything in /tmp (mac os) and recompile the app. Now it works.

Make sure you remove /tmp/envfile if you've changed from the old echo  method to the new cp one. I was pulling my hair out trying to figure out why and it turns out the Ruby file still checks for /tmp/envfile https://github.com/luggit/react-native-config/blob/master/ios/ReactNativeConfig/ReadDotEnv.rb#L15

It happened to me as well.
I had followed this tutorial where you add pre-run script to your schema. In the tutorial the script looked like this:
cp "${PROJECT_DIR}/../.env.development" "${PROJECT_DIR}/../.env.development"
You can probably guess that the second .development was unnecessary.
I removed it and followed the same pronciple for the other schemas as well. It worked.


RHEL repo links return 404

The links for RHEL packages  in the section https://docs.microsoft.com/en-us/microsoft-365/security/defender-endpoint/linux-install-manually?view=o365-worldwide#rhel-and-variants-centos-fedora-oracle-linux-and-amazon-linux-2
result in a 404 response.
The bash example for Centos is incorrect.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bab0b894-0199-7515-b060-a6724e2c9b9e
Version Independent ID: 988ff83c-c4b5-d2f3-023e-2d8dd191b7ae
Content: Deploy Microsoft Defender for Endpoint on Linux manually
Content Source: microsoft-365/security/defender-endpoint/linux-install-manually.md
Product: m365-security
Technology: mde
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@WarrenTR65 Thank you for your feedback. The link is restored now. Could you please elaborate on which example is incorrect?
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @WarrenTR65 Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


feat: add surname and name in subject referential

Looks OK for the code but your commit message does not match what I see in the code

Looks OK for the code but your commit message does not match what I see in the code

Maybe it's better to say :Add username and groupName for different user and group in db ?

Looks OK for the code but your commit message does not match what I see in the code

Maybe it's better to say :Add username and groupName for different user and group in db ?

Add username and group name to tests subjects


Duplicate Videos Selected

Two of the same videos are being displayed.

added conditional that if the video links are the same to jump to  label which will run the get random video again.


When we have triggers of pipelines we get very long jobId which cause error in minio

sub pipeline should be support also

we decided that deep of trigger call can be up to 3-4 therefore the issue is not relevant.


injector: update envoy container's UID and document it

Description:
Updates the UID set for the sidecar proxy and documents
it as a part of a newly created app requirements guide.
Signed-off-by: Shashank Ram<EMAIL_ADDRESS>
Affected area:

New Functionality      [ ]
Documentation          [X]
Install                [ ]
Control Plane          [ ]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [X]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
No

Codecov Report

Merging #2557 (0d41819) into main (8492993) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2557      +/-   ##
==========================================
+ Coverage   61.11%   61.12%   +0.01%
==========================================
  Files         157      157
  Lines        7144     7144
==========================================
+ Hits         4366     4367       +1
+ Misses       2761     2760       -1
  Partials       17       17

Flag
Coverage Δ

unittests
61.12% <ø> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/injector/iptables.go
100.00% <ø> (ø)

pkg/envoy/route/config.go
96.00% <0.00%> (+0.79%)
:arrow_up:

Continue to review full report at Codecov.

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


Explicitly mention the build steps

While manual run of antrea components, explicitly mention the steps to build the binaries.

/skip-all

/skip-all

simple rebase

/skip-all


Added missing validations and improved decoding

Added the following validations

Commands priority must be an integer
Data streams' name must be not empty
Action triggers' name must be not empty
Http request must define a url with a host and you should provide a content-type header or application/text will be used
Improved creation of a time interval, now it's a time span

Pull Request Test Coverage Report for Build 799

216 of 216 (100.0%)  changed or added relevant lines in 16 files are covered.
136 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-1.7%) to 98.279%

Files with Coverage Reduction
New Missed Lines
%

source/Cosmos-API-Actions/ActionEvaluationsRESTfulControllerSpecification.class.st
24
74.74%

source/Cosmos-API-Actions/ActionsRESTfulControllerSpecification.class.st
30
83.43%

source/Cosmos-API-Triggers/TriggersRESTfulControllerSpecification.class.st
38
85.5%

source/Cosmos-API-Commands/CommandsRESTfulControllerSpecification.class.st
44
67.65%

Totals

Change from base Build 797:
-1.7%

Covered Lines:
7765

Relevant Lines:
7901

💛  - Coveralls


Issue from website page Flutter for Android 开发者 (Developer)

三星s10手机，flutter textfield焦点获取不成功。
Translate:  Samsung s10 mobile phone, flutter textfield focus is not successful.

@waerdai
Are you reporting an issue on Samsung S10 device that textfield focus is not working as expected ? If so, can you provide flutter doctor -v and a complete minimal reproducible code sample that shows this issue ?
Thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Idea: Multi-Location Cluster

Since you already support a HA setup, would it be possible to have location groups?
Spanning nodes and workers over several locations would help if one location burned down :D

Interesting idea. It should be possible I think because if I remember correctly the private networks work across locations. I've added a card for this at https://github.com/vitobotta/hetzner-k3s/projects/1#card-77987216
It wouldn't be a good idea to spread the masters across locations because etcd is latency sensitive. But it Thanks for the suggestion!

Hi, I've released v0.5.4 that allows you to specify a different location for each node pool, provided that all node pools (workers and masters) are in the same network zone. Currently, means that either all the nodes are located in Ashburn, or none of them, since the Ashburn location is in a different network zone from that of the other locations. Can you please try a multi-zone cluster and let me know if you run into any issues? Thanks.

Forgot to mention, the masters must all be in the same location because etcd is latency sensitive.

Sorry totally lost track of time :(

Np, can I close this now?


[Feature Request] verify command for each app

It might be useful to have a command that verifies that an app was installed correctly? like looking at the deployments/CRDs and making sure they have started?
Plus we could then setup a load of tests in CI for each app that uses the verify command?

For this it would be useful to have something like a ArkadeApp-Interface with the common operations:
type ArkadeApp interface {
  Install()
  Verify()
  Delete()
}

this Interface should be used by all Operations from the cmd Package. You suggested something in #21 for a different reason. I think we should work on an Interface and some basic types like Helm2App and Helm3App.


dist-overrides.xml is missing entries

The cli, java-main, cmr-aether, cmr-js and cmr-webdav modules are missing from the system overrides file cmr/api/src/main/resources/com/redhat/ceylon/cmr/api/dist-overrides.xml. We should add them. (Is there a way w could write a test for this? A bit difficult probably)

@FroMage another one for you.

Already done.


[Feature] Add page cache metrics

What type of PR is this：

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_admit_test

@Mergifyio branch-2.3

@Mergifyio backport branch-2.4

@Mergifyio backport branch-2.4

@Mergifyio backport branch-2.4


[WFLY-18562] re-evaluated test exclusions for ts.preview and ts.boota…

…ble.preview profiles
Issue: https://issues.redhat.com/browse/WFLY-18562

I need to test this on CI. Locally it fails on ModelControllerMBeanTestCase due to java.lang.IllegalArgumentException: Default value not supported for ArrayType and TabularType exception on "jboss.as:management-root=server" -> getOperation("addSubsystemMicrometer") -> getParameterInfo("exposedSubsystems") and I need to consult this with Kabir as I can't understand why it fails.

@istudens If that's the last thing blocking this (I saw some other CI failures but didn't investigate, nor have I done a code review), I suggest filing a WFLY against the Micrometer subsystem, and disabling the test with ts.preview with a comment noting the JIRA.
I suspect this actually has nothing to do with Preview, except that we include Micrometer in the default config there, so this test picks up issues there. We support Micrometer in standard WF though, it's just not in the default config.
I suspect this isn't a Micrometer bug per se, it's just it has an unusual thing, where an attribute of ModelType.LIST has a default value. That's a valid thing but it sounds like JMX might not like that.

This needs https://issues.redhat.com/browse/WFCORE-6604 to fix the JMX issue.

@istudens The WFCORE-6604 fix is integrated into full WF now.

Thanks @istudens!


Map error when loading

hi, since couple days i have map error with puppet on,
without puppet t loads fine,
this is the error from puppet though.
Exception from asynchronous event: System.NullReferenceException: Object reference not set to an instance of an object
at Puppeteer.State.Load () [0x00026] in <0a91468c87ce45c5a446d6046464cf9d>:0
at Puppeteer.State.get_Instance () [0x00007] in <0a91468c87ce45c5a446d6046464cf9d>:0
at Puppeteer.Tools+<>c.b__33_0 (Verse.Pawn pawn) [0x00000] in <0a91468c87ce45c5a446d6046464cf9d>:0
at HarmonyLib.CollectionExtensions.Do[T] (System.Collections.Generic.IEnumerable1[T] sequence, System.Action1[T] action) [0x00014] in <255414689d344b1385ae719f6cce8c55>:0
at Puppeteer.Tools.GameInit () [0x00007] in <0a91468c87ce45c5a446d6046464cf9d>:0
at Puppeteer.Game_LoadGame_Patch.Postfix () [0x0000b] in <0a91468c87ce45c5a446d6046464cf9d>:0
at (wrapper dynamic-method) Verse.Game.Verse.Game.LoadGame_Patch1(Verse.Game)
at (wrapper dynamic-method) Verse.SavedGameLoaderNow.Verse.SavedGameLoaderNow.LoadGameFromSaveFileNow_Patch1(string)
at Verse.Root_Play+<>c.b__1_1 () [0x00005] in <81af9f8a18324e17b286924a43555a7c>:0
at Verse.LongEventHandler.RunEventFromAnotherThread (System.Action action) [0x00008] in <81af9f8a18324e17b286924a43555a7c>:0
UnityEngine.StackTraceUtility:ExtractStackTrace ()
(wrapper dynamic-method) Verse.Log:Verse.Log.Error_Patch1 (string)
Verse.LongEventHandler:RunEventFromAnotherThread (System.Action)
Verse.LongEventHandler/<>c:b__27_0 ()
System.Threading.ThreadHelper:ThreadStart_Context (object)
System.Threading.ExecutionContext:RunInternal (System.Threading.ExecutionContext,System.Threading.ContextCallback,object,bool)
System.Threading.ExecutionContext:Run (System.Threading.ExecutionContext,System.Threading.ContextCallback,object,bool)
System.Threading.ExecutionContext:Run (System.Threading.ExecutionContext,System.Threading.ContextCallback,object)
System.Threading.ThreadHelper:ThreadStart ()

after i deleted puppet completly out of config and made a new token it seem to be fixed for now. my old token was from september 2021 :D


RSDK-2399 Remove Pre Map Representation API Methods & Add Final API Methods

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


get_genotype_vcf.py review

Some points to consider in get_genotype_vcf.py

Extracting the chr values from the bim (used to clean up the variants selected for variance ratio estimation) will leave empty values for the X, Y and MT chromosomes. However, you don't explicit exclude these chromosomes when creating the initial variant subset, and so you may end up with missing chr data being passed to SAIGE. I'd recommend either excluding these chromosomes from variant selection (if they are potentially going to skew variance ratio estimation), or adjusting your regex to (\d+|X|Y|MT).

I believe the current checkpointing approach (e.g. here) is not going to save you anything (unless there's some hail magic I'm missing). Should you not check for existence of the checkpoint first and then run e.g. densification if and only if the checkpoint is not already available?

You may wish to predicate the downsampling to 1% based on the number of variants left, to avoid accidentally downsampling to less than 2000 variants?

Ideally r2 and bp window would be parameters you can adjust. Why did you choose these values in the first instance (NB they seem fine to me), and can you add that information to the README?

Could you briefly explain the logic behind this sampling approach / add it to the README?

Just noting that you use MAF > Threshold. It's done consistently in the code and README, so is fine, but I feel like most studies I've seen include the lower threshold in their tested range (i.e. keep MAF >= threshold).

I believe the current checkpointing approach is not going to save you anything

Just chiming in to say that this script might be a bit checkpoint-heavy at the moment out of an abundance of caution. The MT densification step is quite a lot of processing, but the filtering between these two checkpoints is pretty minor - the first checkpoint can probably be safely removed
The key here is checkpointing prior to the LD pruning method call, as that includes a chunk more sample processing with no checkpoints, which was causing Hail/Java OOM errors. We added an overly conservative number of checkpoints, and if time allows this can/should be pared down to the minimum number of checkpoints to get the job done.
The checkpoint_mt method does perform the 'if it exists, read, otherwise write' check

The checkpoint_mt method does perform the 'if it exists, read, otherwise write' check

I was actually meaning the line above, but that's where my hail is rusty re: when things get evaluated. My interpretation of the code is that we always take the VDS and split out multiallelics + densify, but then we only save that result to a checkpoint if it doesn't already exist (and in fact we go ahead and replace the matrixtable we just spent a bunch of time calculating with the old checkpoint data if it does exist). But does the hail evaluation magic save us here?

Ah sorry... yes, or at least that's my understanding. Everything including the densification is done using lazy-loading AFAIK, but admittedly I've not spent much time with VDSs compared with MTs

Hi @katiedelange just going through this now, I'll try to respond to this one point a the time!
Also as I mentioned to you on Slack I am adding TO DOs to improve the pipeline to this GDoc linked in the repo README.

Could you briefly explain the logic behind this sampling approach / add it to the README?

The logic for this is simply that the sample rows method (as far as I can tell) expects a proportion of values to retain, vs the actual number, so for example you can say randomly select 20% of variants, not randomly select 1,000 variants.
So I am getting to my desired vre_n_markers by approximately dividing that number by the total number (multiplied by 1.1 to make sure I am in excess of it) and then take the vre_n_markers head to get to the precise number.
There might be a better way to do this but this was what I came up with at the time :)
Probably enough to describe this briefly in the comment above this line? Or I can add to the README if you think it's better

@katiedelange

Extracting the chr values from the bim (used to clean up the variants selected for variance ratio estimation) will leave empty values for the X, Y and MT chromosomes. However, you don't explicit exclude these chromosomes when creating the initial variant subset, and so you may end up with missing chr data being passed to SAIGE. I'd recommend either excluding these chromosomes from variant selection (if they are potentially going to skew variance ratio estimation), or adjusting your regex to (\d+|X|Y|MT).

This is an excellent point and I should've thought about this harder! I don't have a clear sense of whether variants from these chromosomes may skew the VRE calculation or not! I will discuss with Wei and then either remove these or adjust the regex (again if I don't get around to this I have added this to the TO DO list so someone can potentially take this on later instead)

@katiedelange

You may wish to predicate the downsampling to 1% based on the number of variants left, to avoid accidentally downsampling to less than 2000 variants?

Yup another good point and something I have actually already encountered when testing (not many variants left when filtering for MAC>20 with 5 individuals haha), I have added this to the TO DO.

@katiedelange

Just noting that you use MAF > Threshold. It's done consistently in the code and README, so is fine, but I feel like most studies I've seen include the lower threshold in their tested range (i.e. keep MAF >= threshold).

Mmmh yes I noticed I changed this in this recent PR where I extract rare variants as well! Is this what you'd do, define common variants as MAF>=T, vs rare as MAF<T? Happy to do whatever you think makes most sense, and update readme accordingly.

@katiedelange

Ideally r2 and bp window would be parameters you can adjust. Why did you choose these values in the first instance (NB they seem fine to me), and can you add that information to the README?

I think this seems to just be default parameters in the method's description but I have added turn these into args to the TO DO list.
Also a bit random but just ran into this issue highlighting how long this step takes in Hail compared to Plink, which we have also observed. cc @MattWellie

@katiedelange and finally I think @MattWellie is better placed to respond to the checkpoint question, I definitely do not understand this well enough to have an opinion, but happy to submit a PR to change the code / remove checkpoints if appropriate.

@katiedelange this PR: https://github.com/populationgenomics/saige-tenk10k/pull/144 should hopefully address your comments from here?


Pods scheduled on non-host node in the cluster don't have internet access

Please find the inspection-report here: inspection-report-20201116_072520.tar.gz.
sudo iptables -P FORWARD ACCEPT does not solve my problem.

Hi @tamimjabban
If I understand the issue correctly you have two nodes and the pods in one of them cannot reach the internet. Is the problematic node segubtjabban001? I assume you already checked for firewalls.

@ktsakalozos The problem is with pods scheduled on the second machine (not segubtjabban001 as it is the main/host machine, and pods schedule there are able to connect to the internet as well as other pods, so there's not a problem here).


synopsysctl v1 not backwards compatible with synopsys v2020 blackduck installations

What happened:
While trying to administer a blackduck installation created in Aug 2019 the current version of synosysctl throws an error and is able to get the existing blackduck installation.
$ synopsysctl-v1.0.3 get blackduck blackduck -n blackduck-prod
ERRO[0001] synopsyctl failed: failed to get Blackduck values: unable to find release 'blackduck' in namespace 'blackduck-prod'

What you expected to happen:
I expected the new version to be backwards compatible with existing blackduck clusters.
$ synopsysctl-v2020.3.0 get blackduck blackduck -n blackduck-prod
NAME        AGE
blackduck   307d

How to reproduce it (as minimally and precisely as possible):
See the above output from both versions of synopsysctl.
Screenshots/additional context:
I was able to retrieve the old version of synopsysctl from this archived repo: https://github.com/blackducksoftware/synopsys-operator/releases/tag/2020.3.0
Environment:
Kubernetes version 1.14 hosted in AWS EKS:
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.2", GitCommit:"52c56ce7a8272c798dbc29846288d7cd9fbae032", GitTreeState:"clean", BuildDate:"2020-04-21T01:25:41Z", GoVersion:"go1.13.10", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.9-eks-f459c0", GitCommit:"f459c0672169dd35e77af56c24556530a05e9ab1", GitTreeState:"clean", BuildDate:"2020-03-18T04:24:17Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}

@alexclineargo it's an expected behavior. From synopsysctl 1.0.0 or later, we have moved out of Synopsys Operator, so the synopsysctl utility don't support both operator based and synopsysctl based deployment at the same time. If you want to administer Operator based Black Duck deployment, then you need to use pre synopsysctl 1.0.0. It's already covered as part of FAQ
https://synopsys.atlassian.net/wiki/spaces/BDLM/pages/488243643/Frequently+Asked+Questions#If-you-have-multiple-Black-Duck-instances-deployed-with-Synopsys-Operator-and-you-only-want-to-update-one-instance-with-synopsysctl-1.0.0-or-later%3F

Interesting. I didn't come across that wiki article while troubleshooting yesterday. This issue can serve as another documentation resource for others who might have this issue.
Closing.


Reimporter view to class based view

[sc-2909]
Changed function views Import and reimport to class based views

@FelixHernandez FYI: Something is broken with that PR Labeler test - especially since your PR handles what it was complaining about.
We'll ignore that test on this PR. :+1:


End to End TLS SSL - step #9 - add Azure AD authentication URL

Hi, @mnriem
There is a createPrivateKeyFromPem method in the com.azure.security.keyvault.jca.KeyVaultClient class. I see that IllegalArgumentException is thrown in it. But maven-checkstyle-plugin does not allow to throw exceptions directly in this library.
   	if (line == null || !line.contains("BEGIN PRIVATE KEY")) {
            throw new IllegalArgumentException("No PRIVATE KEY found");
        }

If necessary,or no other way,I will skip third-party checks to merged this pr.

Feel free to refactor this to instead not return a certificate if an error occurred looking for the PRIVATE_KEY marker


Are the policies that don't currently accept Exemptions going to in the future?

Are the policies that don't currently accept Exemptions going to in the future?

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

ID: ab87f953-2276-3c34-743b-e39f62d35b60
Version Independent ID: 15186f5f-03b1-e538-53d9-8ce936374fc0
Content: Exempt a Microsoft Defender for Cloud recommendation from a resource, subscription, management group, and secure score
Content Source: articles/defender-for-cloud/exempt-resource.md
Service: defender-for-cloud
GitHub Login: @bmansheim
Microsoft Alias: benmansheim

@m455954
Thanks for your feedback! We will investigate and update as appropriate.

@m455954
Thank you for your time and patience!
Unfortunately, we don't provide product release updates through our GitHub channels. For this, I'd recommend referring to our Azure Updates or Microsoft 365 Roadmap pages for this info.


[CognitiveServices] LUIS Programmatic Spec - Latest changes

This PR includes the following changes:

Those operations that returned empty HTTP 200 responses, now return an OperationStatus object
Batch adding Examples now return a partial success when some errors are detected
AvailableEndpoints update to use a Dictionary
Add deleteUtterances parameter to Delete Intent operation
Minor description updates

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Do me a quick favor?
node -e "fs.writeFileSync('./LUIS-Programmatic.json',JSON.stringify(require('./LUIS-Programmatic.json'),null,2))"
Can you  run that on the swagger file to make sure we've got the right formatting (the enum array probably isn't quite right there).
Past that, there's some weird error in the validator, but not sure what's causing that. We can commit this; we should probably follow up and make sure we're validating when they look at the glitch.

No modification for AutorestCI/azure-sdk-for-python

No modification for AutorestCI/azure-sdk-for-ruby


Sync API Modifications, New Subcommands, API Changes and Unit Tests

This PR makes several major changes to zookeepertcl:
Sync API
Currently, all sync API calls in zookeepertcl actually use the async functions in the Zookeeper  C client.  To make these sync API calls appear to block to the Tcl caller, zookeepertcl uses the function zootcl_wait, which has the unfortunate side effect of manually calling Tcl_DoOneEvent(TCL_ALL_EVENTS), i.e., it enters the event loop.  However, if the Tcl client code is already in the event loop, this could cause hard-to-track-down bugs because it requires that the client code is re-entrant.  Moreover, having a sync API that is actually async violates expectations; if a client wants a sync-like API that actually uses async calls, this can be implemented on top of the async API.  In order to avoid using zootcl_wait, then, this PR uses the actual sync calls in the Zookeeper C client.
New Subcommands
A few new subcommands have been added to Zookeeper objects:

destroy and close: these are synonyms and end up closing the connection to Zookeeper and deleting the zookeepertcl handle
server: this returns the hostname or IP of the Zookeeper server currently connected to.  In a clustered environment, this allows for the client to figure out which node in the cluster is being used

API Changes
For the get subcommand, this PR makes the following modification:

When the -data option is provided for an extant znode but the data is NULL, get returns 1 and unsets the variable specified by -data

In the current code, this scenario would return 0, although the README states

If -data is specified, dataVar is the name of an array that the data is stored into, and get returns 1 if the znode exists and 0 if it doesn't.

Unit Tests
This PR also adds a number of unit tests in the tests/ directory.  These tests require a functioning Zookeeper server / cluster and test the Zookeeper object subcommands, watch callbacks and some of the Tcl procs in the zookeeper namespace.

Destroy is basically a less obscure rename zkHandle ""?

Can you clarify the difference between the two cleanup paths? At least they should have a common subfunction to minimize code duplication.

Which cleanup paths do you mean?


Rule collection improvements

Introduces empty Rego packages to document the rule collections, these are in policy.release.<collection> package, but to reduce the release policy directory, placed in policy/release/collection directory.
The rule collection table has been moved to Release Policy documentation.
Ref. https://issues.redhat.com/browse/HACBS-1749

Preview can be seen here

Thanks for the feedback, will incorporate as these are easy fixes to make...


Attempt to disable NvmCacheState when NVM cache is not enabled.

It seems that NvmCacheState object assumes that when cacheDir is specified then NVM cache is enabled. In case of cachebench used with multiple tiers and no NVM cache, this assumption is causing an error message: "E1122 21:27:43.287637 1212 NvmCacheState.cpp:135] unable to deserialize nvm metadata file: no content in file : ...".

This change is

cachelib/allocator/CacheAllocator-inl.h, line 53 at r1 (raw file):
Previously, igchor (Igor Chorążewicz) wrote…
Hm, maybe instead of adding a new isNvmCacheEnabled parameter to nvmCacheState_ we could just convert nvmCacheState_ into a pointer (or std::optional) and move this initialization to initNvmCache? I think it would be quite natural to keep everything related to nvm in that method.

I tried std::optional - please take a look.

Converted nvmCacheState_ to std::optional to simplify NVM cache state handling when NVM cache state is not enabled.


[Proposal] Remove publish folder/scripts

Looking at what the publish scripts are doing isn't it better to completely remove them and add appropriate scripts in the package.json file in the src folder. Then just execute npm pack or npm publish commands from inside the src folder. I think much more intuitive, not to mention that those .sh scripts cannot be execute under Windows 😉

Duplicate of https://github.com/NativeScript/nativescript-plugin-seed/issues/77


error in step 6 rnn_eval

No previous checkpoints of non_pool.model exist
Exiting..
This problem is showing up in step 6. I have 4 files created in the checkpoint folder named model_file.meta, model_file.index, model_file.data-00000-of-00001 and checkpoint after running rnn_train.py. But when executing rnn_eval, why cant it find the model file?

Try giving the absolute path to the model in the rnn_eval.


Handler example

Could you add an example on how to use a click handle? This seems a bit awkward when I tried. I could not call another method in my controller from it.

Not true. You can access angular stuff from the outside even though it is not a "best practice"
Here is an example:
handlers: {
   click: function() {
      angular.element('#body-id').scope().someFunctionOnThatScope()
   }
}

By combining a few random ideas and suggestions, here's a snippet that is able to call a function in my Controller. ($scope.themes is my array of words, gathered from a ThemeMap object). For the click handler I create a copy of the val and use $scope.$apply on my controller's function:
            $scope.themes = [];
            for (var val in $scope.ThemeMap) {
                $scope.themes.push({
                    text: val,
                    weight :$scope.ThemeMap[val].length,
                    handlers:
                        {click: function() {
                            var newVal = angular.copy(val)
                            return function() {
                               $scope.$apply($scope.checkByTheme(newVal))
                           }
                        }()
                        }
                    })
            }


Add No Info to Display Message

Definition of Ready

[x] Does this meet the INVEST criteria?
[x] Does this have a clearly defined user?
[x] Does this have an estimate?
[x] Does this have acceptance criteria?
[x] Does this have sufficient context (wireframes, business rules, etc.)?
[x] Is this traceable to a feature, epic, or PI?

User Story
As a user
I want to see a message when a facility has no configurations
So that I can understand why there is not data to view
Acceptance Criteria
Given a user is within any table
When the selected item/data set has no data to display
Then the system displays the message "No data currently present."
Does this Story require 508 Testing?

[ ] Implementing a new authoring tool that will produce code / content for the application
[ ] Implementing a 3rd party widget that will be part of the application
[ ] Adding new / updating custom functionality
[ ] Adding new / updating a page template
[x] Adding new / replacing existing functionality (unless the replacement has already passed 508 testing)
[ ] Full Site Review prior to a scheduled EPA 508 review
[ ] Planning a video that will appear on the application
[ ] Producing a video that will appear on the application
[ ] Producing Training materials
[ ] Producing documents

Context
Relevant Facilities: Avon, RG1&2, Silver Gate, and Crosscut.
Testing
Test Case: TC314
Tester: @mosesdeeCVP
Definition of Done

[ ] Have all Acceptance Criteria been agreed to and validated by the scrum team?
[ ] Has the code been unit tested and peer reviewed?
[ ] Have the functional tests been executed?
[ ] Have all defects been reviewed and dispositioned, resolved, or deferred?
[ ] Has the user story been accepted by the Product Owner?
[ ] Is the user story ready to be deployed to the test/staging environments?

Test Passed


possible typo

Is documentation correct?
or should it be: ?
from:
response.socket
Request socket.
to:
response.socket
Response socket.
if so:
fixes #737

in each http request we only have one socket: https://nodejs.org/dist/latest-v4.x/docs/api/http.html#http_message_socket
so in koa, both request.socket and response.socket are point to it.


stop parsing without exception, when data source is not available

Codecov Report

Merging #12 (096125e) into development (2e95da8) will not change coverage.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##           development       #12   +/-   ##
=============================================
  Coverage       100.00%   100.00%
=============================================
  Files               15        15
  Lines              444       463   +19
  Branches            20        25    +5
=============================================
+ Hits               444       463   +19

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

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

Impacted Files
Coverage Δ

src/rki_covid_parser/__init__.py
100.00% <100.00%> (ø)

src/rki_covid_parser/parser.py
100.00% <100.00%> (ø)

tests/conftest.py
100.00% <100.00%> (ø)

tests/test_parsing.py
100.00% <100.00%> (ø)

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 2e95da8...096125e. Read the comment docs.


GetExpandedTextAds.cs example is broken.

The filter condition in GWQL should be ad_group_ad.ad.type.

Fixed in v2.9.0.


Add more LLMs

(1) nvidia api isn't working from backend
(2) add more OpenAI llms based on whatever is available

resolved


iFrame Fixes

Fixes #41

i think this got out of date needs a rebase. (and addressing the above comment)


update menu with stubs

Codecov Report

Merging #313 into experimental will increase coverage by -0.06%.

@@               Coverage Diff                @@
##           experimental     #313      +/-   ##
================================================
- Coverage         31.92%   31.87%   -0.06%
================================================
  Files                30       30
  Lines              2844     2849       +5
================================================
  Hits                908      908
- Misses             1936     1941       +5

Impacted Files
Coverage Δ

vent/menu.py
0% <ø> (ø)
:white_check_mark:

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 ed16de7...530898d. Read the comment docs.


Hyprland doesnt start

Already reported ? *

[X] I have searched the existing open and closed issues.

Regression?
No
System Info and Version
 System/Version info
pcilib: Error reading /sys/bus/pci/devices/0000:00:08.3/label: Operation not permitted
Hyprland, built from branch  at commit 4520b30d498daca8079365bdb909a8dea38e8d55  (version: bump to 0.44.1).
Date: Wed Oct 9 12:54:39 2024
Tag: v0.44.1, commits: 5309
built against aquamarine 0.4.2

flags: (if any)

System Information:
System name: Linux
Node name: gentoo
Release: 6.10.13-gentoo-dist
Version: #1 SMP PREEMPT_DYNAMIC Fri Oct  4 16:27:35 -00 2024

GPU information:
33:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Rembrandt [Radeon 680M] [1002:1681] (rev d1) (prog-if 00 [VGA controller])

os-release: NAME=Gentoo
ID=gentoo
PRETTY_NAME="Gentoo Linux"
ANSI_COLOR="1;32"
HOME_URL="https://www.gentoo.org/"
SUPPORT_URL="https://www.gentoo.org/support/"
BUG_REPORT_URL="https://bugs.gentoo.org/"
VERSION_ID="2.15"

plugins:
	unknown: not runtime

Description
Trying to launch hyprland, but it crashes with a pcilib error
How to reproduce
Type Hyprland in the tty
Crash reports, logs, images, videos
hyprlandCrashReport5392.txt

Well can you post what you are getting from the tty ?? I guess you may try something like
Hyprland > hyprlogs.log &
and then open another desktop environment and post the content of that file here. But wait for like 5 seconds until Hyprland process is finished.

log.txt, in the tty it also says pcilib: Error reading /sys/bus/pci/devices/0000:00:08.3/label: Operation not permitted Dunno why it doesnt show in the log
It may not actually be a hyrland issue.
doas lspci -k | grep -A 3 -E "(VGA|3D)"
pcilib: Error reading /sys/bus/pci/devices/0000:00:08.3/label: Operation not permitted
33:00.0 VGA compatible controller: Advanced Micro Devices, Inc. [AMD/ATI] Rembrandt [Radeon 680M] (rev d1)
	Subsystem: Lenovo Device 50b4
	Kernel driver in use: amdgpu
	Kernel modules: amdgpu

my guess is you compiled something wrong. Can you build aq and hl in debug and get a debug stacktrace https://wiki.hyprland.org/Crashes-and-Bugs/#obtaining-a-debug-stacktrace

It may be a clang issue. I'm using 19.1.2+libcxx. I tried compiling all the dependencies with gcc 14 and it now works

However now I'm getting ✖ Headers version mismatch. Please run hyprpm update to fix those. When I run hyprpm update


Updated parsing logic to keep schema intact

Can you update your .gitignore to add .savant/cache and then revert all of the changes to .savant ?


DRAFT: feat(plan): Support Create Table As Select

add NamedTableWriteRel type

add CreateTableAsSelect() in builder

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


Added papplMain feature

delete and default(set) respond successful-ok but fail to make changes.
Also, to allow a server to run, the system needs the driver callback to exist so had to use testpappl/pwg driver to fix it.
@michaelrsweet Could you suggest changes, if any?

@michaelrsweet The changes you requested have disappeared after a force-push due to renaming of the files. I have made the changes to not use testpappl/pwg-driver.c any more and make the entry points private wherever possible. For the system options, I think we can set some default options and use command line arguments to alter these based on user requirement. Should I do it this way?

@michaelrsweet I have made the changes you suggested.
"showDevices" has already been implemented in main.c by calling papplDeviceList without creating an additional function for the same.
Do we still need a separate function for it?

@JaiLuthra1 Sorry, I missed that (was reviewing from my iPhone...) I don't think it is a big deal either way, although from a purely design/maintenance perspective it is nicer to keep the same pattern and isolate the code accordingly.
Also, see my subsequent comment about not using "--sub-command" for sub-commands. Just as for lprint and Git, sub-commands should not be prefixed - only the options get a prefix.

Cleanup is now complete. I did a little renaming (main -> mainloop) for clarity, added the missing "drivers" sub-command, and cleaned out all of the non-lprint options so things are focused on running as a standalone printer application and not as a unit test program (which we have separately...)


FortranFile test failures on Travis with numpy dev version

The master branch builds seem to currently fail on Travis CI Python: 3.8-dev  and Python: 3.7 "--pre --upgrade" targets. This is probably connected to some changes in a new numpy dev version (1.18.0.dev0+cacdf26).
Likely needs to be fixed in numpy, but here's an Scipy issue for it in case complications turn up.

        Parameters
        ----------
        *dtypes : dtypes, optional
            Data type(s) specifying the size and endiness of the data.

        Returns
        -------
        data : ndarray
            A one-dimensional array object.

        Raises
        ------
        FortranEOFError
            To signal that no further records are available
        FortranFormattingError
            To signal that the end of the file was encountered
            part-way through a record

        Notes
        -----
        If the record contains a multi-dimensional array, you can specify
        the size in the dtype. For example::

            INTEGER var(5,4)

        can be read with::

            read_record('(4,5)i4').T

        Note that this function does **not** assume the file data is in Fortran
        column major order, so you need to (i) swap the order of dimensions
        when reading and (ii) transpose the resulting array.

        Alternatively, you can read the data as a 1D array and handle the
        ordering yourself. For example::

            read_record('i4').reshape(5, 4, order='F')

        For records that contain several variables or mixed types (as opposed
        to single scalar or array types), give them as separate arguments::

            double precision :: a
            integer :: b
            write(1) a, b

            record = f.read_record('<f4', '<i4')
            a = record[0]  # first number
            b = record[1]  # second number

        and if any of the variables are arrays, the shape can be specified as
        the third item in the relevant dtype::

            double precision :: a
            integer :: b(3,4)
            write(1) a, b

            record = f.read_record('<f4', np.dtype(('<i4', (4, 3))))
            a = record[0]
            b = record[1].T

        Numpy also supports a short syntax for this kind of type::

            record = f.read_record('<f4', '(3,3)<i4')

        See Also
        --------
        read_reals
        read_ints

        """
        dtype = kwargs.pop('dtype', None)
        if kwargs:
            raise ValueError("Unknown keyword arguments {}".format(tuple(kwargs.keys())))

        if dtype is not None:
            dtypes = dtypes + (dtype,)
        elif not dtypes:
            raise ValueError('Must specify at least one dtype')

        first_size = self._read_size(eof_ok=True)

        dtypes = tuple(np.dtype(dtype) for dtype in dtypes)
        block_size = sum(dtype.itemsize for dtype in dtypes)

        num_blocks, remainder = divmod(first_size, block_size)
        if remainder != 0:
            raise ValueError('Size obtained ({0}) is not a multiple of the '
                             'dtypes given ({1}).'.format(first_size, block_size))

        if len(dtypes) != 1 and first_size != block_size:
            # Fortran does not write mixed type array items in interleaved order,
            # and it's not possible to guess the sizes of the arrays that were written.
            # The user must specify the exact sizes of each of the arrays.
            raise ValueError('Size obtained ({0}) does not match with the expected '
                             'size ({1}) of multi-item record'.format(first_size, block_size))

        data = []
        for dtype in dtypes:
            r = np.fromfile(self._fp, dtype=dtype, count=num_blocks)
            if len(r) != num_blocks:
                raise FortranFormattingError(
                    "End of file in the middle of a record")
            if dtype.shape != ():
                # Squeeze outmost block dimension for array items
                if num_blocks == 1:
                    assert r.shape == (1,) + dtype.shape
                    r = r[0]

            data.append(r)

        second_size = self._read_size()
        if first_size != second_size:
>           raise IOError('Sizes do not agree in the header and footer for '
                          'this record - check header dtype')
E           OSError: Sizes do not agree in the header and footer for this record - check header dtype

block_size = 80
data       = [array([[ 0. ,  4. ,  2.3],
       [ 5. ,  6. ,  4.5],
       [20. , 20. ,  0. ]]), array([       0, -1048576], dtype=int32)]
dtype      = dtype(('<i4', (2,)))
dtypes     = (dtype(('<f8', (3, 3))), dtype(('<i4', (2,))))
first_size = 80
kwargs     = {}
num_blocks = 1
r          = array([       0, -1048576], dtype=int32)
remainder  = 0
second_size =<PHONE_NUMBER>
self       = <scipy.io._fortran.FortranFile object at 0x7f0879171370>

scipy/io/_fortran.py:287: OSError

git bisect says the issue appears after https://github.com/numpy/numpy/pull/14586

Fixed up now in numpy, CI should be green again.


Change how bitwise operators are used in filters, remove virtualcolumn dependency.

Description
Since my last feature request the bitwise operators have been added. These are VERY helpful and work almost perfect.
The problem now is that when we want to use them, you always have to make use of a virtual column. For example:
A simple query like:
SELECT * FROM "table" WHERE BITWISE_AND("flags", 4) = 0

Converts into:
{
  "queryType": "scan",
  ...
  "virtualColumns": [
    {
      "type": "expression",
      "name": "v0",
      "expression": "bitwiseAnd(\"flags\",4)",
      "outputType": "LONG"
    }
  ],
  ...
  "filter": {
    "type": "selector",
    "dimension": "v0",
    "value": "0",
    "extractionFn": null
  },
 ...
}

As you can see, due to the fact that we use a virtual column, we are able to use the bitwise expressions.
However, druid has some other places where records can be filtered. For example, the DruidInputSource and the Filtered Aggregator.
In these places it is not possible to use a virtual column, and thus we cannot use the bitwise expressions.
Example. Assume a table where we store the role a user can have in a "roles" field. This field is a digit, where each bit represents a role.
Thus:
1 = Content moderator
2 = Financial user
4 = Data analyst
8 = Sales
... etc
A person can have multiple roles of course. So somebody with a "roles" value of 5 is both Content moderator as data analyst.
Now, back to druid.
When I want to create a new druid dataSource and fill it with a native batch ingestion task with data from another druid dataSource, I cannot filter on the "roles" as I want to. See the "filter" part below:
{
  "type": "index_parallel",
  "spec": {
    "dataSchema": {
      "dataSource": "dataAnalystUsers",
      "timestampSpec": { ... },
      "dimensionsSpec": { ... },
      "metricsSpec": [ ... ],
      "granularitySpec": { ... }
    },
    "ioConfig": {
      "type": "index_parallel",
      "inputSource": {
        "type": "druid",
        "dataSource": "users",
        "interval": "2022-02-12/P1D",
        "filter": {
           'type': '????',
           'dimension': 'roles',
           'value': 4
        }
      }
    },
    "tuningConfig": { ... }
  }
}

As you can see, we have no way to filter here using the bitwise expressions. My suggestion is to create native filter types for the bitwise expressions:
Maybe something like below, but this is only a idea.
{
	"filter": {
		"type": "bitwiseAnd",
		"dimension": "<DIMENSION>",
		"value": "<VALUE>",
		"expected": "<VALUE>"
	}
}

I see now that an expression filter was added, which is a perfect solution for this problem. This solves the case described above. I will close this feature request.


PHP Autocomplete source file reading + caching broken

@complex857 I know the listed URL in the file is https://github.com/shawncplus/phpcomplete.vim but that looks abandoned, it also appears that the core version is way out of sync. In any case, the problem exists in both versions.
Attempting a simple class method completion fails with the error below, even though a thorough universal ctags file is loaded:

-- Omni completion (^O^N^P) Pattern not found

I tried to debug the error using :debug normal A^X^O and found an error occurring here:
https://github.com/vim/vim/blob/ab65fc77c5389f7d3f788bbdc3d931561feab131/runtime/autoload/phpcomplete.vim#L2097
Removing the ! from the silent! prefixes halted completion and produced:

E578: Not allowed to change text here

Firstly, the class, method, and their containing file are all correctly determined. With the ! retained, the error is skipped over, the new buffer is never switched to and filled with the class' source file, so when let content = join(getline(cfline, endline), "\n") is executed, it instead reads the current buffer (a completely different PHP script), ultimately filling s:cache_classstructures with bogus data. The completion then fails because no candidates are found in the bogus data.
I thought it might be #6017 again. This script seems to use buffers rather than windows (though I think windows were tried in https://github.com/shawncplus/phpcomplete.vim/issues/57).

Thank you for taking the time for looking into this!
I've managed to reproduce the issue (as you said it's quite a basic case) and your analysis is spot on.
It seems that switching windows has been forbidden since this commit https://github.com/vim/vim/commit/6adb9ea0a6ca01414f4b591f379b0f829a8273c0
Sadly the plugin itself relies extensively on this to be allowed, i'll have to think of some other solution (manipulating files somehow without having them in buffers is my first guess, but then we loose all the niceties like searchpairpos and syntax based comment / string ignores).
I don't see an easy fix for it right now, suggestions are welcomed!

Good to have the theory vindicated. It does look very active with the buffer use.
Is the reasoning for so much live reading of files to keep up with code changes ahead of updates to tags files? Or is it gleaning more into than tags files can hold?
I use joonty/vim-taggatron which updates the relevant tags files on every save. From what I can make out, this script reads only the on-disk copies then parses their contents to look for all the info that I thought was already in my tags file (variables, constants, functions, classes, methods, inheritance, namespaces), which would be double-handling the source files. Correct me if I'm wrong.
If sufficient info already is (or can be) put into tags files, then wouldn't this be an acceptable level of dependency, given some level of ctags is already required?

I'm not 100% sure since this was there already when I started contributing to the project.
I think it's mostly because ctag's PHP support back then (even with exuberant-ctags and some custom regexes) was very poor. You would not get any class related attributes in the generated files ("what class this method belongs to" and so on).
When the first ctag versions (universal-ctags, b4n patches for namespaces) appeared and I was adding extra features relying on these "better ctags", I've always tried to keep the code as functional as possible with the old ctags (I was thinking "who knows what you've on your system").
I did try to convert these extended ctags into some in-memory structure we can just query (reading files is super slow), but that didn't pan out, because lots of interesting information is in docblocks (back then there was no return-type type hints syntax, most projects never type-hinted their function arguments).
So I couldn't re-create the structured information I'd need. From what I can tell this is still the case (even if we'd accept that you'll need some newer ctags versions for anything to work).
What you'd need is something that's as powerful as felixfbecker/php-language-server, I remember some forks of the plugin that included similar (external process you send your buffer over and get completions), but the my idea was that I keep the plugin with minimum (preferably no) external dependency.

It seems that switching windows has been forbidden since this commit 6adb9ea

I don't see an easy fix for it right now, suggestions are welcomed!

Have a look at :h win_execute().  It lets you execute Ex commands in arbitrary windows with few or no side-effects.  It should not raise E565.

You could wrap the problematic code in a function, and call it with win_execute():
call win_execute(window ID, 'call Func()')

If you want to manipulate text without opening a buffer or window, using readfile() might be a good solution.  You can't run Vim commands on it, but if you only want to search for something it should work fine.

I've done a quick test like this:
function! phpcomplete#CompletePHP(findstart, base)
  let ret = win_execute(win_getid(), 'call s:test()')
endf

function! s:test()
  below 1new
endf

The error persists, but the window itself opens. That's why we had a silent! in there. I've tried executing the whole completion function indirectly with win_execute(), but it doesn't seem to solve the problem (completing a just a class name which is from only tags works still, so generally it runs it fine).
On further investigation I found something else wrong: It seems that when the new window is opened, the current window is not switched over in newer vim versions. In the same section I've added this:
echom 'before: '.winnr()
silent! below 1new
echom 'after: '.winnr()

On an older vim (osx catalina default)
VIM - Vi IMproved 8.1 Included patches: 1-503, 505-680, 682-2292
I get this in :messages
"t2.php" 4L, 26C
before 1
after 2

On a newer vim (last homebrew install) VIM - Vi IMproved 8.2 Included patches: 1-2400:
"t2.php" 4L, 26C
before 1
after 1

What's weird that the line right after the below 1new is adding content to that window and that seems to be adding it to the correct buffer. Other commands like getline() or search() are all executing on the original buffer which breaks the code.

The error persists, but the window itself opens.

Yes, because now it's no longer allowed to switch windows while a completion function is being processed.  This is caused by these 2 patches:

8.2.2426  allowing 'completefunc' to switch windows causes trouble
8.2.2427  can still switch windows for 'completefunc'

win_execute() doesn't change that.  But it seems that your below 1new works with silent!, so I guess your issue is later with some other commands which fail because you're no longer in the right window.  That's when win_execute() can help.  With it, you don't need to be in the right window; it can run your commands anywhere, without switching the currently focused window which is now disallowed.

Also, instead of creating a temporary window with :1new, which maybe can break the general windows layout, or trigger some undesirable autocmds, I wonder whether you could use a temporary hidden popup.  Again, you can run arbitrary commands in a popup with win_execute().  Unfortunately, the script is quite long, and I don't know anything about php, so I'm not sure what the right fix is here.
FWIW, this patch seems to fix the error, but I have no idea whether the suggestions are still correct or completely off:
diff --git a/runtime/autoload/phpcomplete.vim b/runtime/autoload/phpcomplete.vim
index 377baa843..838320aa6 100644
--- a/runtime/autoload/phpcomplete.vim
+++ b/runtime/autoload/phpcomplete.vim
@@ -2536,36 +2536,30 @@ endfunction!
 " }}}

 function! phpcomplete#GetCurrentNameSpace(file_lines) " {{{
-	let original_window = winnr()
-
-	silent! below 1new
-	silent! 0put =a:file_lines
-	normal! G
+	let popup_id = popup_create(a:file_lines, {'hidden': v:true})
+	call win_execute(popup_id, 'normal! G')

 	" clear out classes, functions and other blocks
 	while 1
-		let block_start_pos = searchpos('\c\(class\|trait\|function\|interface\)\s\+\_.\{-}\zs{', 'Web')
+		let block_start_pos = win_execute(popup_id, 'echo searchpos(''\c\(class\|trait\|function\|interface\)\s\+\_.\{-}\zs{'', "Web")')->trim("\<NL>")->eval()
 		if block_start_pos == [0, 0]
 			break
 		endif
-		let block_end_pos = searchpairpos('{', '', '}\|\%$', 'W', 'synIDattr(synID(line("."), col("."), 0), "name") =~? "string\\|comment"')
+		let block_end_pos = win_execute(popup_id, 'echo searchpairpos("{", "", ''}\|\%$'', "W", ''synIDattr(synID(line("."), col("."), 0), "name") =~? "string\\|comment"'')')->trim("\<NL>")->eval()

 		if block_end_pos != [0, 0]
 			" end of the block found, just delete it
-			silent! exec block_start_pos[0].','.block_end_pos[0].'d _'
+			call win_execute(popup_id, block_start_pos[0].','.block_end_pos[0].'d _', 'silent!')
 		else
 			" block pair not found, use block start as beginning and the end
 			" of the buffer instead
-			silent! exec block_start_pos[0].',$d _'
+			call win_execute(popup_id, block_start_pos[0].',$d _', 'silent!')
 		endif
 	endwhile
-	normal! G
+	call win_execute(popup_id, 'normal! G')

 	" grab the remains
-	let file_lines = reverse(getline(1, line('.') - 1))
-
-	silent! bw! %
-	exe original_window.'wincmd w'
+	let file_lines = win_execute(popup_id, "echo reverse(getline(1, line('.') - 1))")->trim("\<NL>")

 	let namespace_name_pattern = '[a-zA-Z_\x7f-\xff\\][a-zA-Z_0-9\x7f-\xff\\]*'
 	let i = 0

When I say "fixes the error", I mean this: no error is raised when I press C-x C-o in this test php file:
<?php

$a = "hello";

$b = 'hello';

$c = "hello \"buddy\"";

$d = 'hello \'buddy\'';

$e = <<<EOS
hello buddy
EOS;

$f = <<<'EOS'
hello buddy
EOS;

$g = <<<"EOS"
hello buddy
EOS;

# just to check we're correctly out of a string here
$zzz_end = 42;

There is still another silent! below 1new in the script; I don't know when it's executed, but it would probably need to be refactored in a similar way.

I think I forgot an eval():
diff --git a/runtime/autoload/phpcomplete.vim b/runtime/autoload/phpcomplete.vim
index 377baa843..838320aa6 100644
--- a/runtime/autoload/phpcomplete.vim
+++ b/runtime/autoload/phpcomplete.vim
@@ -2536,36 +2536,30 @@ endfunction!
 " }}}

 function! phpcomplete#GetCurrentNameSpace(file_lines) " {{{
-	let original_window = winnr()
-
-	silent! below 1new
-	silent! 0put =a:file_lines
-	normal! G
+	let popup_id = popup_create(a:file_lines, {'hidden': v:true})
+	call win_execute(popup_id, 'normal! G')

 	" clear out classes, functions and other blocks
 	while 1
-		let block_start_pos = searchpos('\c\(class\|trait\|function\|interface\)\s\+\_.\{-}\zs{', 'Web')
+		let block_start_pos = win_execute(popup_id, 'echo searchpos(''\c\(class\|trait\|function\|interface\)\s\+\_.\{-}\zs{'', "Web")')->trim("\<NL>")->eval()
 		if block_start_pos == [0, 0]
 			break
 		endif
-		let block_end_pos = searchpairpos('{', '', '}\|\%$', 'W', 'synIDattr(synID(line("."), col("."), 0), "name") =~? "string\\|comment"')
+		let block_end_pos = win_execute(popup_id, 'echo searchpairpos("{", "", ''}\|\%$'', "W", ''synIDattr(synID(line("."), col("."), 0), "name") =~? "string\\|comment"'')')->trim("\<NL>")->eval()

 		if block_end_pos != [0, 0]
 			" end of the block found, just delete it
-			silent! exec block_start_pos[0].','.block_end_pos[0].'d _'
+			call win_execute(popup_id, block_start_pos[0].','.block_end_pos[0].'d _', 'silent!')
 		else
 			" block pair not found, use block start as beginning and the end
 			" of the buffer instead
-			silent! exec block_start_pos[0].',$d _'
+			call win_execute(popup_id, block_start_pos[0].',$d _', 'silent!')
 		endif
 	endwhile
-	normal! G
+	call win_execute(popup_id, 'normal! G')

 	" grab the remains
-	let file_lines = reverse(getline(1, line('.') - 1))
-
-	silent! bw! %
-	exe original_window.'wincmd w'
+	let file_lines = win_execute(popup_id, "echo reverse(getline(1, line('.') - 1))")->trim("\<NL>")->eval()

 	let namespace_name_pattern = '[a-zA-Z_\x7f-\xff\\][a-zA-Z_0-9\x7f-\xff\\]*'
 	let i = 0

By moving these variables in the script-local namespace, you could get rid of the ugly trim("\<NL>")->eval()

block_start_pos
block_end_pos
file_lines

For example, instead of writing this:
let block_start_pos = win_execute(popup_id, 'echo searchpos(''\c\(class\|trait\|function\|interface\)\s\+\_.\{-}\zs{'', "Web")')->trim("\<NL>")->eval()

You could write this:
call win_execute(popup_id, 'let s:block_start_pos = searchpos(''\c\(class\|trait\|function\|interface\)\s\+\_.\{-}\zs{'', "Web")')

I forgot to close the hidden popup at the end.  Also, what I said about moving the variables in the script-local namespace was wrong.  You don't need to.  let name = value creates a function-local variable.  I was confused because that's not the case in Vim9.  I'll try to post a better patch later.

This one is better:
diff --git a/runtime/autoload/phpcomplete.vim b/runtime/autoload/phpcomplete.vim
index 377baa843..1a8e8aaf9 100644
--- a/runtime/autoload/phpcomplete.vim
+++ b/runtime/autoload/phpcomplete.vim
@@ -2536,36 +2536,31 @@ endfunction!
 " }}}

 function! phpcomplete#GetCurrentNameSpace(file_lines) " {{{
-	let original_window = winnr()
-
-	silent! below 1new
-	silent! 0put =a:file_lines
-	normal! G
+	let popup_id = popup_create(a:file_lines, {'hidden': v:true})
+	call win_execute(popup_id, 'normal! G')

 	" clear out classes, functions and other blocks
 	while 1
-		let block_start_pos = searchpos('\c\(class\|trait\|function\|interface\)\s\+\_.\{-}\zs{', 'Web')
+		call win_execute(popup_id, 'let block_start_pos = searchpos(''\c\(class\|trait\|function\|interface\)\s\+\_.\{-}\zs{'', "Web")')
 		if block_start_pos == [0, 0]
 			break
 		endif
-		let block_end_pos = searchpairpos('{', '', '}\|\%$', 'W', 'synIDattr(synID(line("."), col("."), 0), "name") =~? "string\\|comment"')
+		call win_execute(popup_id, 'let block_end_pos = searchpairpos("{", "", ''}\|\%$'', "W", ''synIDattr(synID(line("."), col("."), 0), "name") =~? "string\\|comment"'')')

 		if block_end_pos != [0, 0]
 			" end of the block found, just delete it
-			silent! exec block_start_pos[0].','.block_end_pos[0].'d _'
+			call win_execute(popup_id, block_start_pos[0].','.block_end_pos[0].'d _', 'silent!')
 		else
 			" block pair not found, use block start as beginning and the end
 			" of the buffer instead
-			silent! exec block_start_pos[0].',$d _'
+			call win_execute(popup_id, block_start_pos[0].',$d _', 'silent!')
 		endif
 	endwhile
-	normal! G
+	call win_execute(popup_id, 'normal! G')

 	" grab the remains
-	let file_lines = reverse(getline(1, line('.') - 1))
-
-	silent! bw! %
-	exe original_window.'wincmd w'
+	call win_execute(popup_id, "let file_lines = reverse(getline(1, line('.') - 1))")
+	call popup_close(popup_id)

 	let namespace_name_pattern = '[a-zA-Z_\x7f-\xff\\][a-zA-Z_0-9\x7f-\xff\\]*'
 	let i = 0

But there is still another :1new to refactor.

I tried to figure out the winid for the newly created window after the new, but it seems that only the buffer is created, not a window. Tried something like this:
silent! 1new phpcomplete_tempbuf
let bufid = bufnr('phpcomplete_tempbuf')
echom 'bufid: '.bufid
let winids = win_findbuf(bufid)
echom 'winids:'
echom winids

messages:
bufid: 17
winids:
[]

Thank you for the suggestion for hidden popup windows (I suck at vimscript 😓). I'll give it a try.

I've create a pull-request with the migrated code. All the previous tests are passing locally (old travis-ci integration seems to be broken).
Thank you @Roy-Orbison once again, for reporting it and @lacygoill for the popup window suggestion!
I'm a bit unfamiliar with github in this regard: I can't seem to add labels on the MR to mark it as "runtime" (I can see other MRs with that label). If there's some specific naming scheme or anything I can do that helps the process let me know.

Thanks to you all for jumping on this so quickly!

Fixed by https://github.com/vim/vim/commit/942db23c9cb7532d68048530d749eb84ca94d0cd


Enable selected DAP tests previously disabled.

Turn back on the DAP2 remote tests, but leave DAP4 remote tests disabled.
Turn on selected non-remote DAP4 tests
Replace d4crc32 with the equivalents in libdispatch

@DennisHeimbigner I notice this PR turns off nczarr_test/run_chunkcases.  Was this intentional? It was passing on my systems.

@DennisHeimbigner I notice this PR turns off nczarr_test/run_chunkcases.  Was this intentional? It was passing on my systems.

it was intentional because of errors.
it is turned on in a later pr
i should have said that applying prs in order matters

it was intentional because of errors.
it is turned on in a later pr
i should have said that applying prs in order matters


Add support for disabling.

Currently you have to disable the package, it should be possible to toggle the highlighting.

This is what toggle-packages is for. ;)
I don't think that a toggle in this package is necessary, because disabling the package is better, since it removes any overhead (load time, activation time, etc.). The only downside is ease-of-use for toggling, but toggle-packages fills the gap there.


Rebasing and hardlink fuckery

Rebasing from Fedora Linux 39.20231205.0 (Silverblue)
To sodalite:sodalite/current/x86_64/desktop
Following the rebase command given on Sodalite's github page, produces the following error :

error: Checkout gnome-control-center-45.2-1.fc39.x86_64: Hardlinking 65/84d9b066db7cfbb89f672be8d5a574a2948b969738a3d6af7f640c5757beb5.file to gnome-control-center: Die Datei existiert bereits

"The file already exists" for those not acquainted to German.
What causes this issue exactly, please ? How would that be fixable ?

You don't happen to have GParted installed, do you?
If you're wondering what exactly is happening anyways, Sodalite is providing its own /usr/bin/gnome-control-center as a workaround for various apps that fire up GNOME Control Center instead of Pantheon Switchboard (such as when GNOME Software warns you about using mobile data and offers you to open settings). Recently, GParted (and potentially more apps) have added GNOME Control Center as a dependency, and so now rpm-ostree is wanting to drag the actual GNOME Control Center in but it can't since /usr/bin/gnome-control-center already exists.
Could you provide the output of rpm-ostree status for me too? Thanks.

Aaah interesting, that makes sense. Thank you.
No, no gparted - vanilla Silverblue with nothing layered. I should have mentioned that (or maybe i left xrandr and elementary-icon-theme ? But i really avoid adding anything that way)
Ill provide once home

Update: I had Gnome-Teaks layered. Removal and reboot allows to rebase.
So this expand the list of Gnome apps that block any rebasing possibility. Granted that one isnt useful with Pantheon - Elementary Tweaks takes care of reskinning everything to the standard elementary appearance.


Support for completely custom context schemas

We use a completely different mobile schema (mostly same info, different keys).  I want to pass in a context to track* methods and have it use it and not add on the default mobile context.
Also, I would need SnowplowUtils to be public so that I can get to the functions you use to get OS version, etc.
I can do this PR but just want to ask if making SnowplowUtils available in the framework is ok -- or if you'd rather I make some other facade that calls into it? If a facade, what name would you suggest?

Hey @loufranco ! If I understand you correctly, that all sounds fine. All of the track methods allow contexts to be added, so it's just a case of making the SnowplowUtils public right? We actually ended up doing something similar in the JS Tracker:
https://github.com/snowplow/snowplow/wiki/3-Advanced-usage-of-the-JavaScript-Tracker#42-methods-which-can-be-used-from-inside-a-callback
Oh, we will also need a config variable somewhere to indicate that the standard mobile_context shouldn't be attached. (We should do this in a way that complements #37 - it might be worth doing #146 soon).
I don't think we need a facade, unless I'm missing something - @duncan any thoughts?

The next version of the Snowplow JS Tracker will have a contexts field in its constructing argmap, which specifies which contexts to auto-add to all events:
snowplow_name_here("newTracker", "cf", "d3rkrsqld9gmqf.cloudfront.net", {
  appId: "cfe23a",
  platform: "mob"
  cookieDomain: null,
  cookieName: "_sp534_",
  encodeBase64: false,
  respectDoNotTrack: false,
  userFingerprint: true,
  userFingerprintSeed:<PHONE_NUMBER>,
  pageUnloadTimer: 0,
  forceSecureTracker: true,
  writeCookies: true,
  post: true,
  contexts: {
    performanceTiming: true,
    gaCookies: true,
    geolocation: false
  }
});

We could do something similar in a builder pattern for the Obj-C Tracker...

For SnowplowUtils I don't think we need a facade, but it is getting sufficiently stuffed with things that it's going to be wanting a refactor at some point in the near future. Certainly all the context related methods could be lifted out and into something more public.

@duncan, @alexanderdean: I need something for the near term. Is it ok to add Utils to the public headers in the podspec and PR.  I'll worry about turning off the mobile context in a future PR.
If you want me to do something different (pull out methods into something more public), I'd appreciated more guidance on what you think makes sense for this project.  To be clear, I only need the get* functions that are called in setMobileContext to be public (e.g. getOsVersion). I can move those functions somewhere if you want and make that public (leaving Utils as is)

Hi @loufranco - it's fine, go ahead and make them public. Let's leave SnowplowUtils otherwise unchanged

@alexanderdean I wasn't going to change Utils, just add it to public_headers in the pod spec so I can see it in my app.  The methods I need are there, but can't see the header

👍 to making it public.

Hi @loufranco - sure, please go ahead


Unix socket support

Please rebase to main

oh god that rebase was scary

Needs testing with https://github.com/SlimeVR/SlimeVR-OpenVR-Driver/pull/27

I merged it it's better to start testing and fixing >:


Nothing is showing?

I have followed the instructions here including the steps needed for Flask server.
But all I am getting is just a few circles and no flights.
In the .env file I have included
VITE_SKIES_ADSB_HOST_EXISTING=<IP_ADDRESS>:30003
Since I already have dump1090-mutability running locally on a headless server. I even tested by telnet to port 30003 from another computer and I can see the SBS1 formatted messages.
And somehow after running the run_existing.sh script, port 5000 and port 5173(??) are responding to web requests.
Port 5000 just gives 404 no matter what I tried (e.g. http://ip:5000/ or http://ip:5000/skies-adsb/)
Port 5173 responded, but just a plain of circles even when I zoomed out I still didn't see any flights
BTW I modified the last line of run_existing.sh to npx vite --host so I can use the web browser on my Windows machine.
 ./run_existing.sh
 * Debug mode: off
WebSocket server settings:
  - Listen on :30006
  - No SSL/TLS support (no cert file)
  - proxying from :30006 to 192.168.x.x:30003
WARNING: This is a development server. Do not use it in a production deployment. Use a production WSGI server instead.
 * Running on all addresses (<IP_ADDRESS>)
 * Running on http://<IP_ADDRESS>:5000
 * Running on http://192.168.x.x5000
Press CTRL+C to quit

  VITE v5.1.5  ready in 1876 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: http://192.168.x.x:5173/
  ➜  press h + enter to show help
192.168.x.y - - [20/Mar/2024 13:14:26] "GET / HTTP/1.1" 404 -
192.168.x.y - - [20/Mar/2024 13:14:26] "GET /favicon.ico HTTP/1.1" 404 -
192.168.x.y - - [20/Mar/2024 13:14:57] "GET / HTTP/1.1" 404 -
192.168.x.y - - [20/Mar/2024 13:15:24] "GET /skies-adsb/ HTTP/1.1" 404 -

Sorry for the late reply

hi @Derek-K

Did you set your latitude and longitude in the .env file?

VITE_DEFAULT_ORIGIN_LATITUDE= VITE_DEFAULT_ORIGIN_LONGITUDE=

Yes I did

Please open your browser developer tools and then refresh the skies-adsb screen. Let the app run for a few minutes. Verify that there is air traffic nearby with any other adsb tracker. Please post here a copy of the console log output. NOTE: Be sure to scrub out any sensitive data you wish to keep private -- the console output will have your LAT/LNG and IP addresses.

And yes, I am sure there are planes nearby, see my second screenshot (I just blocked out the details of the planes)
Thanks

hi @Derek-K I need to see the console log output in order to debug the issue. Please post the console log output following the instructions below.
For Chrome:

Launch the app in Chrome
When the app loads press F12
Click on the Console tab
right-click any message in the console area and click "Save as..."

For Firefox

Launch the app in Chrome
When the app loads press F12
Click on the Console tab
right-click any message in the console area and click "Save all Messages to File"

Please post the console output here and please be sure to remove any private data you do not wish to share.
Thanks!


WA3 record: 'LightContext' object has no attribute 'get_resource'

When trying to record a series of input events on the cutting edge of the next branch, I'm finding that things aren't quite working as expected.
Particularly, running wa record -srt -w angrybirds_rio as described in the docs here yields the following output:
$ wa record -srt -w angrybirds_rio
INFO     Setting up target
INFO     Deploying angrybirds_rio
ERROR      File "/work/workload-automation/wa/framework/entrypoint.py", line 120, in main
ERROR        sys.exit(command.execute(config, args))
ERROR      File "/work/workload-automation/wa/commands/revent.py", line 127, in execute
ERROR        self.workload_record(args)
ERROR      File "/work/workload-automation/wa/commands/revent.py", line 202, in workload_record
ERROR        workload.apk.initialize(context)
ERROR      File "/work/workload-automation/wa/framework/workload.py", line 662, in initialize
ERROR        self.resolve_package(context)
ERROR      File "/work/workload-automation/wa/framework/workload.py", line 678, in resolve_package
ERROR        self.resolve_package_from_host(context)
ERROR      File "/work/workload-automation/wa/framework/workload.py", line 716, in resolve_package_from_host
ERROR        apk_file = context.get_resource(ApkFile(self.owner,
ERROR
ERROR    AttributeError('LightContext' object has no attribute 'get_resource')

Unlike ExecutionContext, the LightContext used in wa record doesn't seem to define get_resource and thus isn't playing nicely with the infrastructure in workload.py...

Should be fixed by https://github.com/ARM-software/workload-automation/commit/4a06c1373884c27aa6724e33ca62b2fe5de87573

That's giving me:
ERROR      File "/work/workload-automation/wa/framework/entrypoint.py", line 120, in main
ERROR        sys.exit(command.execute(config, args))
ERROR      File "/work/workload-automation/wa/commands/revent.py", line 127, in execute
ERROR        self.workload_record(args)
ERROR      File "/work/workload-automation/wa/commands/revent.py", line 202, in workload_record
ERROR        workload.apk.initialize(context)
ERROR      File "/work/workload-automation/wa/framework/workload.py", line 662, in initialize
ERROR        self.resolve_package(context)
ERROR      File "/work/workload-automation/wa/framework/workload.py", line 678, in resolve_package
ERROR        self.resolve_package_from_host(context)
ERROR      File "/work/workload-automation/wa/framework/workload.py", line 722, in resolve_package_from_host
ERROR        strict=self.strict)
ERROR      File "/work/workload-automation/wa/commands/revent.py", line 295, in get_resource
ERROR        return self.resolver.get_resource(resource, strict)
ERROR
ERROR    AttributeError('ResourceResolver' object has no attribute 'get_resource')

Looks like get_resource is implemented on ExecutionContext as a wrapper around resolver.get rather than on ResourceResolver.

Argh, sorry -- that's my bad. Should be fixed new.

That's that particular error solved. Looks like that wasn't the only issue with LightContext though:
$ wa record -srt -w angrybirds_rio
INFO     Setting up target
INFO     Deploying angrybirds_rio
ERROR      File "/work/workload-automation/wa/framework/entrypoint.py", line 120, in main
ERROR        sys.exit(command.execute(config, args))
ERROR      File "/work/workload-automation/wa/commands/revent.py", line 127, in execute
ERROR        self.workload_record(args)
ERROR      File "/work/workload-automation/wa/commands/revent.py", line 203, in workload_record
ERROR        workload.apk.setup(context)
ERROR      File "/work/workload-automation/wa/framework/workload.py", line 665, in setup
ERROR        context.update_metadata('app_version', self.apk_info.version_name)
ERROR
ERROR    AttributeError('LightContext' object has no attribute 'update_metadata')

I no longer need wa record for the moment so there's no pressure on my end for a quick fix, but thought it was worth keeping this tracked anyway.

This should be fixed with 21b80798a32da74be666c4bf43dda00c691c663b

Looks like that's solved it, thanks.


Some updates to the tests + handling of previously seen files

This changes the way previously seen files are handled, now a message is printed instead of an exception being raised.
I've also formatted the tests with black to make them more consistent.

Codecov Report

Merging #245 (f24a096) into devel (1227e47) will increase coverage by 0.89%.
The diff coverage is 92.61%.

@@            Coverage Diff             @@
##            devel     #245      +/-   ##
==========================================
+ Coverage   71.53%   72.43%   +0.89%
==========================================
  Files         130      135       +5
  Lines        7908     8021     +113
==========================================
+ Hits         5657     5810     +153
+ Misses       2251     2211      -40

Impacted Files
Coverage Δ

openghg/store/_emissions.py
97.84% <0.00%> (-1.08%)
:arrow_down:

openghg/store/_footprints.py
88.99% <0.00%> (-0.92%)
:arrow_down:

openghg/store/_obssurface.py
92.95% <0.00%> (-0.71%)
:arrow_down:

tests/analyse/conftest.py
100.00% <ø> (ø)

tests/helpers/__init__.py
100.00% <ø> (ø)

tests/objectstore/test_local_object_store.py
61.53% <0.00%> (ø)

tests/retrieve/test_ceda_export.py
100.00% <ø> (ø)

tests/retrieve/test_search.py
97.24% <ø> (ø)

tests/services/test_process.py
36.11% <ø> (ø)

tests/services/test_ranking.py
35.71% <ø> (ø)

... and 37 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 1227e47...f24a096. Read the comment docs.


VR Architecture error

UVR is not compatible with filenames containing special characters. Please rename the file ensuring it does not contain special characters and try again.

Thanks for reply!

The song file name cannot have any special characters in it. UVR is not compatible with special characters at this time. Please rename the file and try again.

This has been addressed. Please download the latest version of UVR.
Link:
https://github.com/Anjok07/ultimatevocalremovergui#installation


Incorporate the switch to Text in the dependencies

This is a fairly minimal incorporation of the changes to the other core pandoc packages. Pack and unpack invocations and view patterns were used to keep the types and logic in the package the same.
The compat module still exists, but the inserted pack statement means that the module will fail to compile on a version of pandoc without the switch to Text.

Great! Now we have all the pieces.
Merged into a new text branch for testing.


Master

This is great @eubruno.
@Stevenic this fixes a major bug with the current implementation. Can you please merge this?

@eubruno I'm thinking we should fork this and create a new one going forward.

Sorry for the delay guys...


Universal grammar does not include arcsin nor tanh

The universal grammar does not include the functions arcsin nor tanh, which makes it inappropriate for the two Feynman equations: II.35.21 and I.26.2.

ProGED provides functions that streamline the construction of grammars. The default settings are currently not optimized for any specific dataset. Users are encouraged to modify the grammars for their task, such as by adding functions like arcsin and tanh through the argument functions.

brencej commented 15 days ago •
ProGED provides functions that streamline the construction of grammars. The default settings are currently not optimized for any specific dataset. Users are encouraged to modify the grammars for their task, such as by adding functions like arcsin and tanh through the argument functions in construct_grammar_universal.


Added banking system project using python in mini projects.

1.This program is completely written by me as a learning project i did
it for first-ever experience with python language.
2.This program provides Deposite, Withdrawl and check balance faciltiy

@orendrasingh contribute here https://github.com/Ishaan28malik/Get-Dev-Resources


[Feature Request] Modify Model DB Interface for CRUD Operations

Tell us about your feature request
Modify the DB interface to show people how to make crud operations
Disclaimer

[X] I agree

Hi @Melkeydev I want to take this issue and want to work on it. Can you please help to provide more information about the issue.

@nahaktarun @Melkeydev wouldn’t mind if you worked on this. Let me know, and I can assign it to you.

The idea is to provide a basic example of how to interact with the database, how abstraction is done, and how functions that interact with the database are later referenced in other packages. There have been many questions around this topic.
This is just an example of how I envision this; it can be done in other ways.
database/models.go
package database

type Todo struct {
	Id uint64 `json:"id"`
	Todo string `json:"todo"`
}

func (s *service) CreateTodo(todo string) error {

	statment := `INSERT INTO todos(todo, done) values($1, $2);`
	_, err := s.db.Query(statment, todo)
	return err
}

func (s *service) GetTodo(id uint64) (Todo, error) {
	statement := `SELECT todo, done FROM todos WHERE id=$1;`

	var todo Todo
	todo.Id = id

	row := s.db.QueryRow(statement, id)
	err := row.Scan(&todo.Todo)
	if err != nil {
		return todo, err
	}

	return todo, nil
}

database.go
type Service interface {
       Health() map[string]string
	CreateTodo(todo string) error
        GetTodo(id uint64) (Todo, error)
}

server/server.go
type Server struct {
	port int
	db   database.Service
}

A database migration tool would also be nice for creating a test table in the database:
-- +goose Up
CREATE TABLE "todos" (
    "id" SERIAL NOT NULL,
    "todo" VARCHAR(255),
    PRIMARY KEY ("id")
);

-- +goose Down
DROP TABLE "todos";

We can also add two simple API endpoints in routes.go that can be hit with curl to add and retrieve data from db.
Now, the tricky part is that it needs to be done for all databases specified in the --driver flag. Additionally, good documentation needs to be written.
Tests would be nice, but they're not mandatory.

Do we need to add CRUD operations for the TODO? Can you explain more about API endpoints in the routes.go?

@Samarthasbhat I think adding CRUD operations would be a good idea. Then there are endpoints that can be tested and used as a base to build on in the future, maybe adding middleware, for example.
/api/v1/todos and /api/v1/todos/{id} for GET, POST, DELETE, PUT, and PATCH will do the job.


If the image fails to be pulled, whether the pvc created before should be delete

In my lab, the kubestr image pull failed. I notice the pvc didn't delete automatically.
So if the image fails to be pulled, whether the pvc created before should be delete.
Thank you.

@drummerglen Thanks for opening this issue. I think I was able to reproduce the problem; looks like the only way for kubestr to detect runtime errors like image pull failures is the way for the context to expire. Since the cleanup code shares the same context with the K8s API calls code, the cleanup step would fail because the context has expired by then. Does this match what you saw?
$ kubestr csicheck -i quay.io/isim/noexist:latest -s ebs-csi -v ebs-csi
Creating application
Cleaning up resources
Error deleteing PVC (kubestr-csi-original-pvcgxrmd) - (context deadline exceeded)
Error deleteing Pod (kubestr-csi-original-pod74std) - (context deadline exceeded)
CSI checker test:
  Pod failed to become ready: Pod did not transition into running state. Timeout:15m0s  Namespace:default, Name:kubestr-csi-original-pod74std: rate: Wait(n=1) would exceed context deadline  -  Error
Error: Pod failed to become ready: Pod did not transition into running state. Timeout:15m0s  Namespace:default, Name:kubestr-csi-original-pod74std: rate: Wait(n=1) would exceed context deadline

I think an easy fix will be to use a different context with a reset timeout value for the cleanup step.
Will you be interested in helping us with a PR? That will help to speed up the fix.

Relevant #103.

Fixed by #103.


ci: Enable remote cache in Windows CI

Description: Enable remote cache in Windows CI, hopefully in order to enable faster builds
Risk Level: Low
Testing: N/A
Docs Changes: N/A
Release Notes: N/A

INFO: 3163 processes: 2382 remote cache hit, 781 local.


security rules for firebase

Hey!
Would you be able to show me what security rules look like for the cloud database? The app does not work with the usual rules.

Same issue here, does not work. Useless code.


drop CI for kperf as it is archived

What this PR does, why we need it

With kperf repository archived (https://github.com/knative/community/pull/1433) we can drop CI for it

/assign @upodroid
/assign @dprotaso

/lgtm
/approve


Defining things like what an identifier is

I can't seem to find any documentation on this, so I thought I'd try here.
In many grammar specifications, rules like NAME or NUMBER are used. I can see these defined in the file Tokens, but how do I define these? Is it safe to do:
identifier: characters_for_an_identifier

Or are there better ways of doing this? I'm curious because different languages define what an "identifier" is, so I was curious how this is handled, and where these rules/tokens are (actually) defined.

Tokens like NAME or NUMBER come from the tokenizer and the parser has no control over them. In order to change what constitutes an identifier, the tokenizer would have to be changed to handle NAME tokens differently.
pegen uses the python tokenizer by default, which has a strict definition of what an identifier is, but you could pass a different tokenizer when instantiating a parser object, if you really want to change that.

@lysnikolaou I might need a different tokenizer owing to the language I'm trying to parse having some unique lexical rules in regards to strings and such. The language is fully Unicode aware, so I have that to deal with. Are there any examples of overriding/replacing the tokenizer or should I just look at the default implementation?


Delete application angular-quick1 from this environment

The command jx delete app was run by felipe.ceballos and it generated this Pull Request

ok