Can not see container logs

Hello,
I would like to enable container logs and I've enabled the values like shown below :
{
mode: daemonset
containerLogs:
enabled: true
image:
repository: otel/opentelemetry-collector-contrib
command:
name: otelcol-contrib
}
but after my agent & standalone are running, there are no contents in the path I mount.( yet I can see the container & log folders were created )
I can see the logs normally from the pod :
{
2022-05-06T08:49:59.335Z        info    builder/receivers_builder.go:73 Receiver started.       {"kind": "receiver", "name": "zipkin"}
2022-05-06T08:49:59.335Z        info    healthcheck/handler.go:129      Health Check state change       {"kind": "extension", "name": "health_check", "status": "ready"}
2022-05-06T08:49:59.335Z        info    service/collector.go:249        Starting otelcol-contrib...     {"Version": "0.49.0", "NumCPU": 4}
2022-05-06T08:49:59.335Z        info    service/collector.go:145        Everything is ready. Begin running and processing data.
2022-05-06T08:50:06.348Z        INFO    loggingexporter/logging_exporter.go:56  MetricsExporter {"#metrics": 14}
2022-05-06T08:50:16.365Z        INFO    loggingexporter/logging_exporter.go:56  MetricsExporter {"#metrics": 22}
2022-05-06T08:50:26.383Z        INFO    loggingexporter/logging_exporter.go:56  MetricsExporter {"#metrics": 22}
2022-05-06T08:50:36.400Z        INFO    loggingexporter/logging_exporter.go:56  MetricsExporter {"#metrics": 22}
2022-05-06T08:50:46.418Z        INFO    loggingexporter/logging_exporter.go:56  MetricsExporter {"#metrics": 22}
2022-05-06T08:50:56.436Z        INFO    loggingexporter/logging_exporter.go:56  MetricsExporter {"#metrics": 22}
}
Could any one help troubleshoot?
Thanks a lot.

@NanyinK Thanks for the issue. We moved it here bc it seemed related to the helm charts.
Couple questions:

What version of the charts are you using?
What does the collector's configmap look like?
Does the opened issue contain the entire values.yaml?

I have a suspicion that the collector has the filelog receiver configured but it's not in the logs pipeline.

Hi @TylerHelmuth ,
Please take a look at the answers below :

What version of the charts are you using?
0.6.9
What does the collector's configmap look like?
It's the default one, I didnt change anything in the configmap. I just modified the mode & the host mount path in _pod.tpl
Does the opened issue contain the entire values.yaml?
I'm afraid I didnt get this question, shall I upload the values.yaml here?

Besides the things I mentioned above, anything else I need to change?
Thanks a lot for your help

FYI, this is the agent configmap :
Name:         ot-collector-agent-opentelemetry-collector-agent
Namespace:    ot
Labels:       app.kubernetes.io/instance=ot-collector-agent
app.kubernetes.io/managed-by=Helm
app.kubernetes.io/name=opentelemetry-collector
app.kubernetes.io/version=0.49.0
helm.sh/chart=opentelemetry-collector-0.15.1
Annotations:  meta.helm.sh/release-name: ot-collector-agent
meta.helm.sh/release-namespace: ot
Data
relay:
exporters:
logging: {}
extensions:
health_check: {}
memory_ballast:
size_mib: "819"
processors:
batch: {}
memory_limiter:
check_interval: 5s
limit_mib: 1638
spike_limit_mib: 512
receivers:
filelog:
exclude:
- /var/log/pods/ot_ot-collector-agent-opentelemetry-collector*/opentelemetry-collector/.log
include:
- /var/log/pods///.log
include_file_name: false
include_file_path: true
operators:
- id: get-format
routes:
- expr: $$body matches "^\{"
output: parser-docker
- expr: $$body matches "^[^ Z]+ "
output: parser-crio
- expr: $$body matches "^[^ Z]+Z"
output: parser-containerd
type: router
- id: parser-crio
output: extract_metadata_from_filepath
regex: ^(?P[^ Z]+) (?Pstdout|stderr) (?P[^ ]) (?P.)$
timestamp:
layout: "2006-01-02T15:04:05.000000000-07:00"
layout_type: gotime
parse_from: time
type: regex_parser
- id: parser-containerd
output: extract_metadata_from_filepath
regex: ^(?P[^ ^Z]+Z) (?Pstdout|stderr) (?P[^ ]) (?P.)$
timestamp:
layout: '%Y-%m-%dT%H:%M:%S.%LZ'
parse_from: time
type: regex_parser
- id: parser-docker
output: extract_metadata_from_filepath
timestamp:
layout: '%Y-%m-%dT%H:%M:%S.%LZ'
parse_from: time
type: json_parser
- id: extract_metadata_from_filepath
parse_from: $$attributes["file.path"]
regex: ^./(?P[^]+)(?P<pod_name>[^]+)(?P[a-f0-9-]+)/(?P<container_name>[^.]+)/(?P<run_id>\d+).log$
type: regex_parser
- attributes:
k8s.container.name: EXPR($.container_name)
k8s.namespace.name: EXPR($.namespace)
k8s.pod.name: EXPR($.pod_name)
k8s.pod.uid: EXPR($.uid)
run_id: EXPR($.run_id)
stream: EXPR($.stream)
type: metadata
- id: clean-up-log-body
ops:
- move:
from: log
to: $
type: restructure
start_at: beginning
jaeger:
protocols:
grpc:
endpoint: <IP_ADDRESS>:14250
thrift_compact:
endpoint: <IP_ADDRESS>:6831
thrift_http:
endpoint: <IP_ADDRESS>:14268
otlp:
protocols:
grpc:
endpoint: <IP_ADDRESS>:4317
http:
endpoint: <IP_ADDRESS>:4318
prometheus:
config:
scrape_configs:
- job_name: opentelemetry-collector
scrape_interval: 10s
static_configs:
- targets:
- ${MY_POD_IP}:8888
zipkin:
endpoint: <IP_ADDRESS>:9411
service:
extensions:

health_check
memory_ballast
pipelines:
logs:
exporters:

logging
processors:
memory_limiter
batch
receivers:
otlp
metrics:
exporters:
logging
processors:
memory_limiter
batch
receivers:
otlp
prometheus
traces:
exporters:
logging
processors:
memory_limiter
batch
receivers:
otlp
jaeger
zipkin
telemetry:
metrics:
address: <IP_ADDRESS>:8888

Events:

I believe my suspicion is correct, the filelog receiver is not in the logs pipeline.
This can be fixed by adding it to your .Values.config
'''yaml
config:
service:
pipelines:
logs:
receivers:
- otlp
- filelog
'''

Also, it is strange that your chart version is 0.6.9 as that is the opentelemetry-operator chart latest release, but your values.yaml and configmap look like they are from the opentelemetry-collector chart.  What command are you using to install the chart?

Hello @TylerHelmuth
I'm afraid that it has already been added there by default (in the _config.tpl)

is it the correct file to add ?
and for the version, sorry it's 0.15.1

You're correct that it is there in _config.tpl, but in the same file where the template is used, you can see that whatever is in .Values.config overrides the template. Checkout #199 for an explanation on why this is happening and how to fix it.

Hi @TylerHelmuth
I think I get you, yet could you tell me how to add it to my .Values.config?
Sorry I'm new to helm, and not familiar with it. I’m also googling to find the way.

also I tired to add it here (in the last line)
config:
  exporters:
    logging: {}
  extensions:
    health_check: {}
    memory_ballast: {}
  processors:
    batch: {}
    # If set to null, will be overridden with values based on k8s resource limits
    memory_limiter: null
  receivers:
    jaeger:
      protocols:
        grpc:
          endpoint: <IP_ADDRESS>:14250
        thrift_http:
          endpoint: <IP_ADDRESS>:14268
        thrift_compact:
          endpoint: <IP_ADDRESS>:6831
    otlp:
      protocols:
        grpc:
          endpoint: <IP_ADDRESS>:4317
        http:
          endpoint: <IP_ADDRESS>:4318
    prometheus:
      config:
        scrape_configs:
          - job_name: opentelemetry-collector
            scrape_interval: 10s
            static_configs:
              - targets:
                  - ${MY_POD_IP}:8888
    zipkin:
      endpoint: <IP_ADDRESS>:9411
  service:
    telemetry:
      metrics:
        address: <IP_ADDRESS>:8888
    extensions:
      - health_check
      - memory_ballast
    pipelines:
      logs:
        exporters:
          - logging
        processors:
          - memory_limiter
          - batch
        receivers:
          - otlp
          - filelog

but I get this error when running it
2022/05/09 01:49:46 proto: duplicate proto type registered: jaeger.api_v2.PostSpansRequest
2022/05/09 01:49:46 proto: duplicate proto type registered: jaeger.api_v2.PostSpansResponse
2022-05-09T01:49:46.456Z        info    builder/exporters_builder.go:255        Exporter was built.     {"kind": "exporter", "name": "logging"}
2022-05-09T01:49:46.456Z        info    memorylimiterprocessor/memorylimiter.go:111     Memory limiter configured       {"kind": "processor", "name": "memory_limiter", "pipeline": "traces", "limit_mib": 1638, "spike_limit_mib": 512, "check_interval": 5}
2022-05-09T01:49:46.456Z        info    builder/pipelines_builder.go:224        Pipeline was built.     {"kind": "pipeline", "name": "traces"}
2022-05-09T01:49:46.456Z        info    builder/pipelines_builder.go:224        Pipeline was built.     {"kind": "pipeline", "name": "logs"}
2022-05-09T01:49:46.457Z        info    builder/pipelines_builder.go:224        Pipeline was built.     {"kind": "pipeline", "name": "metrics"}
Error: cannot build receivers: cannot create receiver filelog: unmarshal to regex_parser: unrecognized prefix
2022/05/09 01:49:46 collector server run finished with error: cannot build receivers: cannot create receiver filelog: unmarshal to regex_parser: unrecognized prefix

ot-collector-agent.tar.gz
this is my helm chart, could you please take a look if needed.

I also changed the value in examples (add logfile there)
[charts/opentelemetry-collector/examples/daemonset-collector-logs/values.yaml]
but the issue is still there.

I'm not sure how you are using helm, but it is non-standard to have the charts in a folder named ot-collector-agent.  Normally you add a chart repository to helm using the helm repo add command.  This will let you interact with the chart via helm install or helm template commands.  Also, chart users normally only configure the output of a chart via the values.yaml file; it is not best practice to be modifying anything in the chart.
For this chart and this use case, the best practice would be:

Add the opentelemetry-helm-charts repo:
helm repo add open-telemetry https://open-telemetry.github.io/opentelemetry-helm-charts
Create a values.yaml file that contains the configuration you want.  In this instance, I think your values.yaml would contain

mode: daemonset

config:
  service:
    pipelines:
      logs:
        receivers:
          - otlp
          - filelog

containerLogs:
  enabled: true

Run either a helm install or helm template command with the opentelemetry-collector chart, passing in your values .yaml.
helm install open-telemetry/opentelemetry-collector --values <path to your values.yaml>
helm template open-telemetry/opentelemetry-collector --values <path to your values.yaml>

Hi @TylerHelmuth
Sorry but after I did what you say I still got the same error
2022/05/10 01:19:01 proto: duplicate proto type registered: jaeger.api_v2.PostSpansRequest
2022/05/10 01:19:01 proto: duplicate proto type registered: jaeger.api_v2.PostSpansResponse
2022-05-10T01:19:01.231Z        info    builder/exporters_builder.go:255        Exporter was built.     {"kind": "exporter", "name": "logging"}
2022-05-10T01:19:01.231Z        info    memorylimiterprocessor/memorylimiter.go:111     Memory limiter configured       {"kind": "processor", "name": "memory_limiter",
 "pipeline": "traces", "limit_mib": 1638, "spike_limit_mib": 512, "check_interval": 5}
2022-05-10T01:19:01.231Z        info    builder/pipelines_builder.go:224        Pipeline was built.     {"kind": "pipeline", "name": "traces"}
2022-05-10T01:19:01.231Z        info    builder/pipelines_builder.go:224        Pipeline was built.     {"kind": "pipeline", "name": "logs"}
2022-05-10T01:19:01.231Z        info    builder/pipelines_builder.go:224        Pipeline was built.     {"kind": "pipeline", "name": "metrics"}
2022-05-10T01:19:01.231Z        info    builder/receivers_builder.go:226        Receiver was built.     {"kind": "receiver", "name": "jaeger", "datatype": "traces"}
2022-05-10T01:19:01.231Z        info    builder/receivers_builder.go:226        Receiver was built.     {"kind": "receiver", "name": "prometheus", "datatype": "metrics
"}
2022-05-10T01:19:01.231Z        info    builder/receivers_builder.go:226        Receiver was built.     {"kind": "receiver", "name": "zipkin", "datatype": "traces"}
Error: cannot build receivers: cannot create receiver filelog: unmarshal to regex_parser: unrecognized prefix
2022/05/10 01:19:01 collector server run finished with error: cannot build receivers: cannot create receiver filelog: unmarshal to regex_parser: unrecognized prefix

This is the command I used :
 helm install ot-agent open-telemetry/opentelemetry-collector --values values.yaml
I copied the content from yours and ran that.

I dug into this today and found that a release of opentelemetry-logs-collection, which filelog uses, had some breaking changes that are affecting the chart's configuration of the filelog receiver.  I will get the charts updated.  Thanks for finding the issue!

@NanyinK can you try with chart version 0.16.2?

Hi @TylerHelmuth
It works, thank you !

Hi @TylerHelmuth
It works, thank you !

Awesome! Can this issue be closed?

@TylerHelmuth
Sure mate, it can be closed.


Added address components (#48)

Please review the option to get the address components when making a rooftop match https://developers.google.com/maps/documentation/geocoding/intro#Types

Let's just always return the address components. Could you remove the added config value from this PR?

Closing due to inactivity.


Replace blockquotes with {. notice}

Better formatting.
Need to do it in a md compatible way.

Done.
Is compatible with kramdown
Multilines have to go in blockquotes though.


Got ArgumentError with unknown keyword in Ruby 3.4.0-preview2

👓 What did you see?

✅ What did you expect to see?
No error
📦 Which tool/library version are you using?
ruby 3.4.0-preview2
cucumber-ruby 9.2.0
🔬 How could we reproduce it?
I run bundle exec cucumber but the following is easier:

irb
Hash.new(strict: false).freeze (simplified from https://github.com/cucumber/cucumber-ruby/blob/v9.2.0/lib/cucumber/multiline_argument/data_table.rb#L78)

Ruby 3.3:

Ruby 3.4:

📚 Any additional context?
Trying to fix CI for ruby-head for https://github.com/egonSchiele/contracts.ruby

This is fixed in main - It will be released in cucumber v10.


针对vue2老项目开发环境用vite替代webpack

Clear and concise description of the problem
针对维护了3年的vue2的老项目，开发环境打算用vite替代webpack。
但是插件并不完善，需要改很多地方（如

I don't understand chinese but if I am correctly understanding this issue by using machine translation, I think this is out of scope for vite. It's something that can be solved by improving vite-plugin-vue2 or implemented by a new plugin.

Close as this is unrelated to Vite's core. Please open the issue to the upstream instead.


How to install pci controller？

If you are using Harvester 1.1.0+, you can enable it under Advanced->Addons
Otherwise you can use this helm chart: https://github.com/harvester/charts/tree/master/charts/harvester-pcidevices-controller


Metadata updates

Change all author occurrences of edmaca to saveenr
Change all ms.author occurrences of edmaca to saveenr
Change all manager of  arindamc, sukvg, saveenr or blank to jhubbard

@MarkMcGeeAtAquent,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@MarkMcGeeAtAquent : Thanks for your contribution to the Azure documentation! The author, @saveenr, has been notified to review your proposed change.

@MarkMcGeeAtAquent : Thanks for your contribution to the Azure documentation! The author, @saveenr, has been notified to review your proposed change.

#sign-off

@MarkMcGeeAtAquent : Thanks for your contribution to the Azure documentation! The author, @saveenr, has been notified to review your proposed change.

#sign-off

#sign-off

@MarkMcGeeAtAquent : Thanks for your contribution to the Azure documentation! The author, @saveenr, has been notified to review your proposed change.

@MarkMcGeeAtAquent : Thanks for your contribution to the Azure documentation! The author, @saveenr, has been notified to review your proposed change.

#sign-off

@MarkMcGeeAtAquent : Thanks for your contribution to the Azure documentation! The author, @saveenr, has been notified to review your proposed change.


Only a space is added if I try to apply auto suggestions when context is empty

Hi everyone,
First of all, big fan of KeyboardKit! I'm upgrading the library on the app that I work on, but unfortunately I noticed an issue with the library in the version 8.4.2.
In my application, we add default auto suggestions when the text is empty do the user can start his sentence easily. This is was no issue on the 7th version of the library, but unfortunately I found that it became an issue in the current one.
I've looked into the code and I noticed a few things that could help:
UITextDocumentProxy+Words file, line 69 -> func replaceCurrentWord(with replacement: String) { ... }
The first guard in the function unwraps currentWord into word, but currentWord is nil since the whole text context is empty. So this guard fails and the execution leads the app to only add a space then.
What should be the process now? Should I open a PR?
Here's a video to the issue that I'm having:
https://github.com/KeyboardKit/KeyboardKit/assets/3586658/7d21ffad-31e9-4c33-971f-9e0b2fe50fea

Hi @rubenspessoa
Happy to hear that you like the library and thank you for letting me know about your problems.
KeyboardKit 8.5.1 adjusted some autocomplete bugs when it comes to empty text. Can you upgrade and see if it solves the problem?

Hi @danielsaidi,
Thanks for your response. I’ve updated to the very latest version but unfortunately can still reproduce the bug. I’ll share the stacktrace and my suspicion:
1. KeyboardContext+ProxyBridging.insertAutocompleteSuggestion(_ suggestion: Autocomplete.Suggestion,, tryInsertSpace: Bool = true)
2. UITextDocumentProxy+Autocomplete.insertAutocompleteSuggestion(_ suggestion: Autocomplete.Suggestion, tryInsertSpace: Bool = true)
3. UITextDocumentProxy+Words.replaceCurrentWord(with replacement: String)

That’s where the problem occurs, in the first line of the replaceCurrentWord method we have:
guard let word = currentWord else { return }
But currentWord resolves as nil when the context is empty. It's a computed variable that's created as following:
var currentWord: String? {
  let pre = currentWordPreCursorPart
  let post = currentWordPostCursorPart
  if pre == nil && post == nil { return nil }
  return (pre ?? "") + (post ?? "")
}

And since the context is empty, pre == nil resolves to true, post == nil also resolves to true and the currentWord returns nil.
Since it resolves to nil, replaceCurrentWord returns early and, following the normal app flow, just the space is added to the context.
I'm not sure if any changes here could result in major consequences in the app functionality, but I hope this helps the team to spot the issue and improve the library even further! Thanks guys, let me know if I can help in anything.

Hi @rubenspessoa
Thank you for clarifying! I will look into this 👍

This will be fixed in 8.7 and available in the v8.7 branch later this week.


Enable actions on HSM group modifications with patterns

Filling the functions that add/remove nodes from the different HSM groups. This goes with the code on the branch feature/hsm_group_lifetime in the manta git repo that will get a pull request soon.

Goes with PR 68 on Manta: https://github.com/eth-cscs/manta/pull/68


Stop the audio on PositionalAudio unmount

Stop audio when the component unmounts.  Addresses issue here: https://github.com/react-spring/drei/issues/68

Oh makes sense! I'm on a little camping trip will publish later 🙂


split out matchmaker part 2 - moving matchmaker from wasm to dylib

In preparation for #579, we're moved the matchmaker WASM to a dylib that can currently be loaded by a intent gossiper node in place of a WASM file.
In this PR, we're adding a Matchmaker derive macro that can be used to automatically derive the binding code between the matchmaker runner and the implementation with a state that is managed by the runner. Thanks for this, we no longer need to (de-)serialize the matchmaker's state on each run.

71d8098535306b865e00f3cfeb35acd79349a11a: clean-up redundant async runtime on gossiper RPC
2db32bd3c778d2cea5d6d22b9dc88dfc01c28012: renamed "vm_macro" crate to just "macros" as it's no longer just for VM as we use it for matchmaker dylib now too
2142dc09309598b054b536e0e6dc0ef58bdf5e5a: re-purposed and updated the matchmaker macro to deriving macro, adding constructor, destructor and add_intent binding code
97f643ad4f61249793679a60689b91e2dfe881fc: use the macro
20291d2d8edeefb0ea41d8c4a657bcabcf8d01cc: update docs

pls update wasm


Is there a way to force shortURL to use https?

Hello,
We use https everywhere but the Branch generated links (short or long) are always using http. We could replace the string to force https, but is there a better way to force Branch to generate https links?
Thanks!
S.

There's got to be a better way, right?
I think that https should be the default, or at least an option.  I'm talking with the backend team about the ramifications about changing this.  There could be some weird edge cases that I don't know about before changing this.

any news about this?

@SebC99 We've been working on this transition for a while but it's a big change so it's taking a while. Note that we already generate links on https if they are branch domains (bnc.lt and app.link). We only generate http links on custom domains and then redirect to https version on a branch domain. Hopefully we will roll this out soon. Hang tight, more to come!

This is actually not an SDK issue but more of a backend one. We will add it to our product update email once it's live. Closing here.


Fix embedded web server on host platforms

Might make it a bit easier to work on for you, @nivoc!
With this change if you run:
build/jag simulate

the embedded web server now works.

@kasperl Nice - Thanks!


issue 8: testcafe for help page

Implemented testcafe test for help page and modified navbar slightly to give help button an id

Test currently fails.


Starting in code mode breaks some demos

As far as I can tell, we might be starting e.g. with a zero-sized iframe.

Fixed


Considered trying to get this in to golint or go vet?

Just curious, have you considered trying to get this in to golint or go vet? Why or why not?

By "this" do you mean every check? Most of the checks in here were rejected from go vet or don't quite fit. And golint is a poor choice because it is about following established style, not finding bugs.
go vet is interested in checking code that compiles, works without crashing but is subtly wrong. staticcheck, on the other hand, is interested in checking code before it's ever run.

The regexp check was once considered for go vet, but then rejected because of the above reason
There's an open issue for checking time.Format – if that ever makes it into go vet, we'll likely remove it from staticcheck. There hasn't been any discussion about its suitability yet.
The time.Sleep check was once considered for go vet, then for go lint, but rejected because they tried to make it too general, and ran into too many false positives
The template parsing probably falls into the same category as regexp
The waitgroup checks might fit into go vet. I have no interest of upstreaming them at this point, however. Someone else is of course free to do so.

Cool. Thanks for an exhaustive answer!


server side encryption defaulted to False

S3 server side encryption (as opposed to client side encryption) requires additional changes to blobstore in order to properly handle SSE-S3 or SSE-KMS or SSE-C)
ref: http://docs.aws.amazon.com/AmazonS3/latest/dev/serv-side-encryption.html
s3_util.py was "hardcoding" encrypt_key=True -- which enables server side encryption - although
client side encryption is expected to be True, we cannot expect the same of server side encryption.
this change was necessary for us to move forward with our use-case.

that said, a proper support of server side encryption by WAL-e utils needs a bit more work before it is usable. In the meantime, it is perhaps incorrect to assume all users will have server-side encryption turned on.
Please consider this PR. I can provide more details if needed. We are storing several TB of WAL's/base backups of postgres servers using WAL-e. We can think of covering the SSE-* properly in the next round.
Regards,
Venkat

Perhaps it's incorrect, but the horse is out of the barn, and I don't see sufficient reason here to regress this default to un-encrypted keys.


us-co-archuleta

says downlad data but link is empty
http://www.archuletacounty.org/index.aspx?nid=209

Now in sources https://github.com/openaddresses/openaddresses/blob/839fba4480145573ea02b092b4cb49bafac1f964/sources/us/co/archuleta.json


Improve test coverage of KMS sign and verify

The deploy step is complaining about test coverage so I added more tests to
fix it. https://codecov.io/gh/spulec/moto/commit/25aad70481d2b99b80b1a89c90ad21c20628c2a2

By adding this coverage, I tested sending hash digests to sign and verify and they seem to be working fine, so I removed the warning.

The failing test doesn't look related to my changes:

Correct, it isn't - a botocore release broke Moto. Our master branch contains a fix - would you mind merging this in? Just to double check that all tests pass with these changes

@bblommers all good now! Thanks for checking that out.


[kwokctl] Add --external flag

What would you like to be added?

The --external flag means ignore --name and respect --kubeconfig/--master
Add warning log to kwokctl snapshot export, use kwokctl --external snapshot save --format k8s instead
#797

Why is this needed?
NONE

/assign

/assign


Add Spherical ROM as Default in Unit Tests

Add spherical ROM model
Use spherical ROM as default in unit tests
Add standalone unit test for ROM models (currently only spherical rom)
Upload cache folder for computed FK results

Requested fixes all done. One thing though, is there anything in test_pybullet_csp.py that's not deterministic? I'm observing inconsistent results between different runs.


Loading json file created using AirNow api won't work and kepler keeps spinning

Describe the bug
When i try to upload a json file created using AirNow api, kepler keeps on spinning without any feedback.
File
air.json.zip

@macrigiuseppe this seems to be related to another underlying issue with displaying error feedback when processing. I created another issue here #294 that encompasses this issue and #253 too.

Please follow development here: https://github.com/uber/kepler.gl/issues/307


add storybook tests for BreakpointProvider

add test for Default breakpoints
add test for Bootstrap 3 breakpoints

Storybook Hub is building storybooks for every commit in this PR.
Latest Storybook: https://f0319018-fdb0-402f-ac25-15fe9755e3bd.sbook.io/
( Visit here for more info )

This comment is automatically added by Storybook Hub

@nathanielpiche I put up two basic storybook tests for the BreakpointProvider. If you can think of any other useful tests that we can implement in the storybook, let me know! I wasn't sure what the most useful way to visually test breakpoints was, so I'd appreciate any input you have 😄
I'm also thinking about improving the debug mode for breakpoints. Something along the lines of throwing an ::after pseudo element in the top-right of each Column that displays the current breakpoint it's using. Maybe also do the same thing for the Page component, but instead of showing the current breakpoint, the Page component would display the current view size. That way you could quickly tell if it's breaking when it's supposed to break.

Pretty straightforwards tests.
👍 for the debug improvements.
Should we add a story showcasing withBreakpoints?

Sure! Any idea what to demonstrate on that story?

Here is what I can think of:

Displaying the current breakpoints.
Two sets of breakpoints in the same page?
Dynamic breakpoints?

I can't really see the usefulness (is that a word?) of the last two though.

I don't personally see the value in the last two either, but I'm sure there's some edge case where it would be useful. I'll add them to the storybook :+1:


Transition to jethro

@agherzan @floion

Reviewed 8 of 8 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Add back grid lines for DataArray plots

I think these were lost with the big refactor.

I probably forgot that when I branched the xarray case, LGTM


Compatibility with IIS and .Net Core 5

Hello,
I'm currently trying to deploy a .Net Core 5 Server on my local machine using an IIS.
Everything works fine while debugging and testing in Visual Studio. But when I deploy the application to IIS the endpoint is not found and returns an 404.
Everything works fine while starting it in Visual Studio, but the moment I deploy it, the endpoint isn't found.
Just to check I kept the example controller "WeatherForecast" and it's working fine.
My config:
Configure
app.UseTus(httpContext =>             {                 var tusConfigurator = httpContext.RequestServices.GetService<ITusConfigurator>();                 return tusConfigurator.CreateTusConfiguration();             });
TusConfigurator
`
public class TusConfigurator : ITusConfigurator
{
private readonly ILogger Logger;
private readonly IConfiguration Config;
    public TusConfigurator(ILogger<TusConfigurator> logger, IConfiguration config)
    {
        Logger = logger;
        Config = config;
    }

    public DefaultTusConfiguration CreateTusConfiguration()
    {

        var diskPath = Config.GetValue<string>("TusConfig:Path");

        var config = new DefaultTusConfiguration
        {
            Store = new TusDiskStore(diskPath),
            UrlPath = "/files"
        };

        config.Events = new tusdotnet.Models.Configuration.Events
        {
            OnAuthorizeAsync = ctx => OnAuthorizeEvent(ctx),
            OnFileCompleteAsync = async ctx => await OnUplaodCompleteEvent(ctx)
        };

        return config;
    }

    public Task OnAuthorizeEvent(AuthorizeContext ctx)
    {
        Logger.LogInformation("OnAuth");

        return Task.CompletedTask;
    }

    public async Task OnUplaodCompleteEvent(FileCompleteContext ctx)
    {
        {...}
    }

}

`
The logs from the IIS also state that there is no matching route, but the api- and options- request works:

[2021-01-20 10:43:28.7120][INFO ][10][HostingApplicationDiagnostics.LogRequestStarting][] Request starting HTTP/1.1 OPTIONS http://inge/api/files - -
[2021-01-20 10:43:28.7120][DEBUG][10][CorsService.IsOriginAllowed][] The request has an origin header: 'http://localhost:4200'.
[2021-01-20 10:43:28.7120][INFO ][10][CorsService.IsOriginAllowed][] CORS policy execution successful.
[2021-01-20 10:43:28.7120][DEBUG][10][CorsService.ApplyResult][] The request is a preflight request.
[2021-01-20 10:43:28.7120][INFO ][10][HostingApplicationDiagnostics.LogRequestFinished][] Request finished HTTP/1.1 OPTIONS http://inge/api/files - - - 204 - - 4.6887ms
[2021-01-20 10:43:28.7120][INFO ][10][HostingApplicationDiagnostics.LogRequestStarting][] Request starting HTTP/1.1 POST http://inge/api/files - 0
[2021-01-20 10:43:28.7120][DEBUG][10][CorsService.IsOriginAllowed][] The request has an origin header: 'http://localhost:4200'.
[2021-01-20 10:43:28.7120][INFO ][10][CorsService.IsOriginAllowed][] CORS policy execution successful.
[2021-01-20 10:43:28.7215][DEBUG][10][DfaMatcher.MatchAsync][] No candidates found for the request path '/files'
[2021-01-20 10:43:28.7215][DEBUG][10][EndpointRoutingMiddleware.Invoke][] Request did not match any endpoints
[2021-01-20 10:43:28.7215][INFO ][11][HostingApplicationDiagnostics.LogRequestFinished][] Request finished HTTP/1.1 POST http://inge/api/files - 0 - 404 - - 5.2501ms

Full log file:
iis.log
I will try it with .Net Core 3.1, but would like to use Core 5.
Hopefully you can help me!

Same thing with .Net Core 3.1
Just for the record, my web.config. I added the part for the request limit, which was from your wiki
<?xml version="1.0" encoding="utf-8"?> <configuration>   <location path="." inheritInChildApplications="false">     <system.webServer>       <security>         <requestFiltering>           <requestLimits maxAllowedContentLength="2147483647"></requestLimits>         </requestFiltering>       </security>       <handlers>         <add name="aspNetCore" path="*" verb="*" modules="AspNetCoreModuleV2" resourceType="Unspecified" />       </handlers>       <aspNetCore processPath="dotnet" arguments=".\PwC.Care.IngestionEngine.Server.dll" stdoutLogEnabled="false" stdoutLogFile=".\logs\stdout" hostingModel="inprocess" />     </system.webServer>   </location> </configuration>

Same thing with .Net Core 3.1
Just for the record, my web.config. I added the part for the request limit, which was from your wiki
<?xml version="1.0" encoding="utf-8"?> <configuration>   <location path="." inheritInChildApplications="false">     <system.webServer>       <security>         <requestFiltering>           <requestLimits maxAllowedContentLength="2147483647"></requestLimits>         </requestFiltering>       </security>       <handlers>         <add name="aspNetCore" path="*" verb="*" modules="AspNetCoreModuleV2" resourceType="Unspecified" />       </handlers>       <aspNetCore processPath="dotnet" arguments=".\PwC.Care.IngestionEngine.Server.dll" stdoutLogEnabled="false" stdoutLogFile=".\logs\stdout" hostingModel="inprocess" />     </system.webServer>   </location> </configuration>

I put some logging into the middleware and configurator:
Middleware

app.UseTus(httpContext =>
{
logger.LogInformation("Create context in startup");
var tusConfigurator = httpContext.RequestServices.GetService();
return tusConfigurator.CreateTusConfiguration();
});
Constructor:
public TusConfigurator(ILogger logger, IConfiguration config)
{
Logger = logger;
Config = config;
Logger.LogInformation("Init Configurator");
}

[2021-01-20 12:51:50.6517][INFO ][ 4][Log.ExecutedEndpoint][] Executed endpoint 'PwC.Care.IngestionEngine.Server.Controllers.WeatherForecastController.Get (PwC.Care.IngestionEngine.Server)'
[2021-01-20 12:51:50.6598][INFO ][ 4][HostingApplicationDiagnostics.LogRequestFinished][] Request finished in 199.9126ms 200 application/json; charset=utf-8
[2021-01-20 12:52:04.3589][INFO ][ 7][HostingApplicationDiagnostics.LogRequestStarting][] Request starting HTTP/1.1 OPTIONS http://inge/api/files
[2021-01-20 12:52:04.3648][DEBUG][ 7][CORSLoggerExtensions.RequestHasOriginHeader][] The request has an origin header: 'http://localhost:4200'.
[2021-01-20 12:52:04.3648][INFO ][ 7][CORSLoggerExtensions.PolicySuccess][] CORS policy execution successful.
[2021-01-20 12:52:04.3648][DEBUG][ 7][CORSLoggerExtensions.IsPreflightRequest][] The request is a preflight request.
[2021-01-20 12:52:04.3648][INFO ][ 7][HostingApplicationDiagnostics.LogRequestFinished][] Request finished in 17.3554ms 204
[2021-01-20 12:52:04.3825][INFO ][ 9][HostingApplicationDiagnostics.LogRequestStarting][] Request starting HTTP/1.1 POST http://inge/api/files  0
[2021-01-20 12:52:04.3825][DEBUG][ 9][CORSLoggerExtensions.RequestHasOriginHeader][] The request has an origin header: 'http://localhost:4200'.
[2021-01-20 12:52:04.3825][INFO ][ 9][CORSLoggerExtensions.PolicySuccess][] CORS policy execution successful.
[2021-01-20 12:52:04.3825][INFO ][ 9][Startup.b__4][] Create context in startup
[2021-01-20 12:52:04.3825][INFO ][ 9][TusConfigurator..ctor][] Init Configurator
[2021-01-20 12:52:04.3934][DEBUG][ 9][Logger.CandidatesNotFound][] No candidates found for the request path '/files'
[2021-01-20 12:52:04.3934][DEBUG][ 9][Log.MatchFailure][] Request did not match any endpoints
[2021-01-20 12:52:04.3934][INFO ][ 9][HostingApplicationDiagnostics.LogRequestFinished][] Request finished in 21.0977ms 404

full log:
iis.log
Despite being configured properly (I even tested it without the service class...) the route isn't found

I put some logging into the middleware and configurator:
Middleware

app.UseTus(httpContext =>
{
logger.LogInformation("Create context in startup");
var tusConfigurator = httpContext.RequestServices.GetService();
return tusConfigurator.CreateTusConfiguration();
});
Constructor:
public TusConfigurator(ILogger logger, IConfiguration config)
{
Logger = logger;
Config = config;
Logger.LogInformation("Init Configurator");
}

[2021-01-20 12:51:50.6517][INFO ][ 4][Log.ExecutedEndpoint][] Executed endpoint 'PwC.Care.IngestionEngine.Server.Controllers.WeatherForecastController.Get (PwC.Care.IngestionEngine.Server)'
[2021-01-20 12:51:50.6598][INFO ][ 4][HostingApplicationDiagnostics.LogRequestFinished][] Request finished in 199.9126ms 200 application/json; charset=utf-8
[2021-01-20 12:52:04.3589][INFO ][ 7][HostingApplicationDiagnostics.LogRequestStarting][] Request starting HTTP/1.1 OPTIONS http://inge/api/files
[2021-01-20 12:52:04.3648][DEBUG][ 7][CORSLoggerExtensions.RequestHasOriginHeader][] The request has an origin header: 'http://localhost:4200'.
[2021-01-20 12:52:04.3648][INFO ][ 7][CORSLoggerExtensions.PolicySuccess][] CORS policy execution successful.
[2021-01-20 12:52:04.3648][DEBUG][ 7][CORSLoggerExtensions.IsPreflightRequest][] The request is a preflight request.
[2021-01-20 12:52:04.3648][INFO ][ 7][HostingApplicationDiagnostics.LogRequestFinished][] Request finished in 17.3554ms 204
[2021-01-20 12:52:04.3825][INFO ][ 9][HostingApplicationDiagnostics.LogRequestStarting][] Request starting HTTP/1.1 POST http://inge/api/files  0
[2021-01-20 12:52:04.3825][DEBUG][ 9][CORSLoggerExtensions.RequestHasOriginHeader][] The request has an origin header: 'http://localhost:4200'.
[2021-01-20 12:52:04.3825][INFO ][ 9][CORSLoggerExtensions.PolicySuccess][] CORS policy execution successful.
[2021-01-20 12:52:04.3825][INFO ][ 9][Startup.b__4][] Create context in startup
[2021-01-20 12:52:04.3825][INFO ][ 9][TusConfigurator..ctor][] Init Configurator
[2021-01-20 12:52:04.3934][DEBUG][ 9][Logger.CandidatesNotFound][] No candidates found for the request path '/files'
[2021-01-20 12:52:04.3934][DEBUG][ 9][Log.MatchFailure][] Request did not match any endpoints
[2021-01-20 12:52:04.3934][INFO ][ 9][HostingApplicationDiagnostics.LogRequestFinished][] Request finished in 21.0977ms 404

full log:
iis.log
Despite being configured properly (I even tested it without the service class...) the route isn't found

Hi!
I would say that this has to do with the order that middlewares are setup in your startup. As you can see in the log it's the EndpointRoutingMiddleware that says that the endpoint is not found. tusdotnet does not (at least not yet :) ) integrate with endpoint routing in ASP.NET Core so when the endpoint middleware kicks in it does not know of the route and cannot find any endpoint for it.
The order of middlewares in the test sites in this repo are as follows:
 // httpContext parameter can be used to create a tus configuration based on current user, domain, host, port or whatever.
// In this case we just return the same configuration for everyone.
app.UseTus(httpContext => Task.FromResult(httpContext.RequestServices.GetService<DefaultTusConfiguration>()));

app.UseRouting();

// All GET requests to tusdotnet are forwarded so that you can handle file downloads.
// This is done because the file's metadata is domain specific and thus cannot be handled
// in a generic way by tusdotnet.
app.UseEndpoints(endpoints => endpoints.MapGet("/files/{fileId}", DownloadFileEndpoint.HandleRoute));

Hi!
I would say that this has to do with the order that middlewares are setup in your startup. As you can see in the log it's the EndpointRoutingMiddleware that says that the endpoint is not found. tusdotnet does not (at least not yet :) ) integrate with endpoint routing in ASP.NET Core so when the endpoint middleware kicks in it does not know of the route and cannot find any endpoint for it.
The order of middlewares in the test sites in this repo are as follows:
 // httpContext parameter can be used to create a tus configuration based on current user, domain, host, port or whatever.
// In this case we just return the same configuration for everyone.
app.UseTus(httpContext => Task.FromResult(httpContext.RequestServices.GetService<DefaultTusConfiguration>()));

app.UseRouting();

// All GET requests to tusdotnet are forwarded so that you can handle file downloads.
// This is done because the file's metadata is domain specific and thus cannot be handled
// in a generic way by tusdotnet.
app.UseEndpoints(endpoints => endpoints.MapGet("/files/{fileId}", DownloadFileEndpoint.HandleRoute));

@smatsson , I am also using .Net Core 5. Everything works ok but while refactoring my code I noticed that middleware configuration was a bit off (according to MS recomendations at https://docs.microsoft.com/en-us/aspnet/core/fundamentals/middleware/?view=aspnetcore-5.0
What I wanted to put to your attention is that it says the recomended order is;
UseRouting()
UseCORS()
UseAuthentication()
UseAuthorization()
Now if I adjust the order as above, I will have to move UseTus() above UseRouting(). Then when trying to upload it ask for user/pass since user is not yet authenticated.
If I put it after UseAuthentication() it worked (even if its after UseRouting()) for me and didn't had the problem @SimonNeP  mentioned here.
any thoughts?

Hi @mkgn! I'm not really sure what you are asking? tusdotnet does not use endpoint routing and thus there is no need to add UseRouting() nor UseEndpoints() to your deploy pipeline. At best it will just be a performance penalty. At worst the route for tusdotnet will not be found.


Spelling

This PR corrects misspellings identified by the check-spelling action.
The misspellings have been reported at https://github.com/jsoref/hugegraph/commit/e94f980442e0f48d5019674f1fe72fbbdd06d75d#commitcomment-69669718
The action reports that the changes in this PR would make it happy: https://github.com/jsoref/hugegraph/commit/45806c1db3b6e736a27d761288ca492208c0a0e3
Note: this PR does not include the action. If you're interested in running a spell check on every PR and push, that can be offered separately.

recheck


[AUTOPATCHER] Patches CVE-2022-22825 for expat

        Patches CVE-2022-22825 for expat.
        This patch and PR was automatically generated and needs human verification before merging.

Will be resolved by #1934


chore: bump dependencies.

Bump Kustomize and Kubernetes dependencies to the newest versions.

Unfortunately this is causing some dependency issues with one of troubleshoot dependencies. I am closing this.


Document possible statuses in /var/cpanel/elevate

Please document possible values of stage and status set in /var/cpanel/elevate for proper integration with Virtuozzo's cPanel almaconvert script.

We'll update the docs but here's what it should look like.
Status can be running, paused, success, failed
Stage can be 1-6. These are documented here. https://cpanel.github.io/elevate/#sumup-of-upgrade-process
Stage 6 is when it's fully complete and it'll look like this:
   "stage_number" : 6,
   "status" : "success"


Remappings from the Keyboard Manager do not work while Environmental Variables window is open

Microsoft PowerToys version
0.81.1
Installation method
Microsoft Store
Running as admin
None
Area(s) with issue?
Environment Variables, Keyboard Manager
Steps to reproduce

Turn off caps lock (if activated)
Remap a key in the keyboard manager (e.g.: the 'Caps Lock' key becomes Ctrl, 'Ctrl(Left)' key to 'Disable')
Launch Environment variables
(Allow the software to make changes to the device)
Push the Caps Lock key on the keyboard

I access the environmental variables using this button:

✔ Expected Behavior
Pushing the 'Caps Lock' key on my keyboard should act as Ctrl (as remapped through the keyboard manager).
❌ Actual Behavior
The 'Caps Lock' key returns to its original function while the environmental variables window is open (and remains like this even for other applications). The 'Caps Lock' key becomes Ctrl again when this window is closed.
Other Software
No response

Update:
Even when running environmental variables from the search bar, this issue appears. As soon as I enter the window below, remapping stops working until I close that window.

I am not reproducing this with 0.84.1 on Windows 11.

Thank you for the check! Tried it with 0.84.1 and the problem is gone. Closing the issue ;)


Configuring ports with env vars

:tada: This PR is included in version 2.2.12 :tada:
Your semantic-release bot :package::rocket:


Multiple TabbyChat boxes in-game

Would be very useful to be able to have multiple tabbed floating "windows" or boxes in-game allowing multiple channels to be viewed onscreen at once.
Each box could have the same set of tabs available across the top, but each able to have a different tab selected/open.

As much as I would love to do it, the way I want to implement this require javafx. It's going to have to wait Mojang gets everyone to use Java 8.
I want to put something like this in GuiPanel so I can figure out if a component is visible.
    public boolean isVisible(GuiComponent component, @Nullable Point point) {
        Rectangle rect = component.getBounds();

        Shape s = new javafx.scene.shape.Rectangle(rect.x, rect.y, rect.width, rect.height);

        boolean wait = true;
        for (GuiComponent comp : this) {
            if (wait) {
                if (comp == component) {
                    wait = false;
                }
                continue;
            }
            if (comp == component) {
                return point == null ? true : s.contains(new Point2D(point.x, point.y));
            }
            rect = component.getBounds();
            Shape r = new javafx.scene.shape.Rectangle(rect.x, rect.y, rect.width, rect.height);

            s = Shape.subtract(s, r);
            if (!s.getLayoutBounds().contains(new BoundingBox(0, 0, 0, 0))) {
                return false;
            }

        }
        return true;
    }


set temporary redirect on blast-vip to standard checkout

This PR comments out the route for the blast-vip promo page and sets a 302 redirect to the main blastform checkout.
This promotion ends on Dec. 31, so we no longer want people who come to this page to be able to sign-up for The Blast at the discounted rate.
I left the code in the file, but commented out, as I expect we'll resurrect this promotion or another one in the future.
To test —
Go to local.../blast-vip and make sure that it redirects to https://checkout.texastribune.org/blastform with a 302.

WFM. Left one comment.


Support Enum Field

[x] enum field definition

Tests:

[x]  update enum
[x] bulk-update enum
[x] create model including enum  default value
[x] filter clause including enum

Codecov Report

Merging #564 (945782c) into master (4ed267e) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #564   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          185       185
  Lines        15298     15349   +51
=========================================
+ Hits         15298     15349   +51

Impacted Files
Coverage Δ

ormar/fields/__init__.py
100.00% <ø> (ø)

ormar/queryset/queryset.py
100.00% <ø> (ø)

ormar/fields/model_fields.py
100.00% <100.00%> (ø)

ormar/models/mixins/save_mixin.py
100.00% <100.00%> (ø)

tests/test_model_definition/test_columns.py
100.00% <100.00%> (ø)

tests/test_queries/test_queryset_level_methods.py
100.00% <100.00%> (ø)


DOWN status of application  still show the UP status in eureka page?

Hi ,
I encountered a problem, the eureka page is shown like this:

Application
AMIs
Availability Zones
Status

GATEWAY
n/a (2)
(2)
UP (2) - WCMIS208:GateWay:10020 , WCMIS209:GateWay:10020

but 'GATEWAY' has been closed on 'WCMIS208', so its status should be 'DOWN' or not displayed.
Although this does not affect the normal use, but I still want to where is the problem? thank you all

What do you mean it has been closed?  Is Eureka in "self-preservation mode"?  If it is it won't evict any down instances.

Thanks, you are right, and I saw this:

EMERGENCY! EUREKA MAY BE INCORRECTLY CLAIMING INSTANCES ARE UP WHEN THEY'RE NOT. RENEWALS ARE LESSER THAN THRESHOLD AND HENCE THE INSTANCES ARE NOT BEING EXPIRED JUST TO BE SAFE.
https://github.com/Netflix/eureka/wiki/Understanding-Eureka-Peer-to-Peer-Communication
http://www.itmuch.com/spring-cloud-sum/understanding-eureka-self-preservation/

So, can I manually remove it? Thanks  again

It is not recommended but you can disable self preservation mode by setting eureka.server.enableSelfPreservation to false

I got it, Thanks very much,


Memcached Issue in 3.16

On Google App Engine when trying to work memcache for session. i am facing issue

A PHP Error was encountered
Severity: Warning
Message:  Unknown: Failed to write session data (user). Please verify that the current setting of session.save_path is correct (localhost:11211)
Filename: Unknown
Line Number: 0
Backtrace:
My Config File :
$config['sess_driver'] = 'memcached';
$config['sess_cookie_name'] = 'ci_session';
$config['sess_expiration'] = 0;
$config['sess_save_path'] = 'tcp://localhost:11211';
$config['sess_match_ip'] = FALSE;
$config['sess_time_to_update'] = 0;
$config['sess_regenerate_destroy'] = FALSE;
I have updated driver latest file according to master.

We don't have a "3.16" version; "3.1.6" is not the same thing as "3.16".
We do have proper tags; pulling master is not the correct way to update.
This is a bug tracker and all the info you've provided amounts only to "This configuration doesn't work for me" ... That's not a bug report; please post on our forums instead.


Alternative Rendering

I use the zf-api-problem, zf-mvc-auth module in combination with apigility. Since I have to ship a regular webapplication also within the project I use the zf-mvc-auth events to handle both api and webapp authentication stuff. Everything works quite nice on the apigility side.
However my webapplication is based on session for login/logout. So all protected routes within my webapp are automatically handled by zf-mvc-auth and my listeners that generate everything needed. As soon as I try to access a protected resource a 403 Error is created which is correct also, but this response is always a application/problem+json, which is obviously correct for the api but incorrect on side of the webapp where I want to have a regular error site. Is there some way to change or configure the way how the problems get rendered?
BR Daniel

This repository has been closed and moved to laminas-api-tools/api-tools-api-problem; a new issue has been opened at https://github.com/laminas-api-tools/api-tools-api-problem/issues/8.


On mobile or small screens, the subnavbar dropdown jumps to the first subnavbar link

Thanks for the awesome theme! I am working on a wiki instance, where we will likely use yours as the default skin.
On problem I noticed is that on small screens, e.g. on a mobile, the subnavbar turns into a drop-down menu (that's pretty cool!), but on navigating to any page, the drop-down button still has the first link in the subnavbar pre-selected . This makes it impossible to use this drop-down button to navigate to the first link itself, since re-clicking the pre-selected item does not work apparently.
In the screenshot from the borkweb wiki, for instance, the button shows All Locations, since that is the first clickable subnavbar item, but if I actually want to navigate to the All Locations page, clicking on the pre-selected item from the drop-down does not work.
Any help or  suggestion for a workaround (the obvious one is to have the Main Page as the first link, but the topbar already has links to the Main Page, making this look rather redundant) would be highly appreciated. Thanks!

Ok

Ok


training with one GPU

excuse me,i want to know how to train with only one GPU, can you help me? thanks a lot.

Hi, we highly recommend using 4 or 8 GPUs to train the model, since the performance is very sensitive to the batch size.
To train with one GPU, you could set a smaller batch size and linearly reduce the learning rate.

Hello,i’m sorry to bother you again,there is a problem that has puzzled me for a long time. I really can't figure out where to correct it.I trained with one  gpu. I made the following modifications to the code(train.py):

and

and

and

The problem is that my training results are poor.I don't know what the problem is.

打扰一下，我想知道如何只用一个GPU进行训练，你能帮我吗？多谢。
你好 请问你解决了吗 我也只有一个GPU可用 需要怎么改呢

do you solve this problem?i want to know how to deal with that too.


Mobile view for transaction history

Description
The transfer screen (for desktop) with new transaction history is being implemented as part of the current UX improvement iteration. We will also need to have a design for the mobile view.
Tasks

[ ] @sashseurat to design a solution draft

[ ] Review with decision

[ ] Implementation

@taleldayekh Isn't there a possibility to have some small adjustments for mobile as we have done them before? I would like to avoid having a separate screen design for mobile.

@christianbrb, a quick fix could be to stack the input fields and button on top of each other on three rows if everyone is ok with that.

@taleldayekh Sounds good :) Let's do it like that


Update .nvmrc to latest node 16 LTS version

Needed to enable this PR to pass: https://github.com/dependabot/fetch-metadata/pull/421

I double-checked that our actions files pickup the version specified in .nvmrc and indeed they do: https://github.com/dependabot/fetch-metadata/commit/c40140bc51e29fc87519f16aed72c5c39e1a4a2a


Update manifests/e/Eugeny/Tabby/1.0.168/Eugeny.Tabby.locale.en-US.yaml

Fixes an issue where this package has multiple different monikers
Microsoft Reviewers: Open in CodeFlow

[Policy] Reset Labels


Show multiple tag values

@jlafleche

LGTM, can you try submitting this PR to the main Sentry repo?


Added GET option to SET command

Fixes https://github.com/DiceDB/dice/issues/207
This PR adds a new feature of integrating the GET option in the SET command.

Hey @soumya-codes , yes I remember, you reviewed the changes in my previous PR - https://github.com/DiceDB/dice/pull/246#discussion_r1706127435 . You told that, we need Put at only one place (in the end). I tried to do that, but with it, the tests were failing i.e. the new value wasn't getting assigned to the key, so that's why I had to add Put at 2 places in the GET option.

Since, previously, the XX and NX option tests were failing with the GET option, so that's why I thought that this can be done. Could you please guide that how can I do it with the Put at only 1 place, like only in the end. Actually, I'm not asking for the complete solution, but just if possible, could you please give a hint, so that I can try fix that...

For the sake of continuity I would highly recommend we continue on the same PR. cc @JyotinderSingh


Outline-focus - followup for better style isolation

Description

create new mixin instead of updated existing widely used mixin

Screenshots

outline-focus - unaffected

before
after

button focus - regression resolved

before
after

Test notes

Steps for testing

Visit the button page and click the primary button and verify focused state: http://localhost:4000/pages/element/button
Visit the outline-item page and click any of the icons and verify focused state on the row: http://localhost:4000/pages/element/button

+1 I'm cool with this


increase font size accross admin, improve color contrast

add new color-grey variable, adjust existing variables
increase body font size to 85%
add 20px to sidebar width
adjust layout on image focal point label to avoid text overlap
add new dark color variables and use for improved contrast on text

browser testing in progress now, but @thibaudcolas if you could take a look in the meantime I would appreciate it

Spotted one instance that still has the same issue as from #5270:

I've pushed some changes, including a fix for the above issue

This is working very well, I was initially a bit concerned about the global font size increase but it actually feels great to use.

Merged! This will be released in Wagtail 2.6.
Here is a before/after GIF for posterity:


popovers: Improve keyboard navigation

When you open a popover (even with the mouse) from the left or right sidebar, we should keyboard-enable the popover by selecting the first field, which should allow for keyboard navigation.
See also #15439.

@zulipbot claim

https://github.com/zulip/zulip/pull/15722 solves this.


Add filler option

Source name
Torrentio Anime
Source language
English
Describe your suggested feature
Can you Add filler options to Torrentio Anime
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I will fill out all of the requested information in this form.

Filters are already present in Torrentio Anime. Update to the latest version. If you are not getting the latest update use the correct repo.

I'd like to point out that there are no filler options.

Yeah same thing is happening with me.
filters are not showing in 14.9

You guys might be using another repo, where torrentio is maintained by someone else.
The official latest version is  14.7

There is no such option

https://github.com/user-attachments/assets/a9086dd6-9dd2-4fb2-81d4-c6b167baf16c
There is no such option

Thnx ni3x bro I was using wrong repo
Also Shoaib bhai the filter option is right there tho on the top right corner.

I am saying filler

Wait.... What's filler option?
What does it do?? I may have never seen that before 🤔

It just mark filler ep


Only allow swiping for some rows...

Hi there,
I wonder how it is possible to enable swiping in a list view only for some rows based on a condition? If this possible? And if yes, how?
Regards,
Christian

Found it out by myself :-)

and how did you do this? please post your answer here =)


TPUs not avail in us-central1 and us-west4 for pipelines

Expected Behavior
TPUs should be available in us-central1 and us-west4 for custom model training in a Vertex AI Pipeline
Actual Behavior
This notebook has consistently daily failed since 11/7: Training [DAG failed], due to lack of TPU availability
https://console.cloud.google.com/cloud-build/builds;region=us-central1/a8146e40-67c8-45ed-8757-8e0637bda3f8?project=1012616486416
Steps to Reproduce the Problem
google_cloud_pipeline_components_TPU_model_train_upload_deploy.ipynb
1.
1.
1.
Specifications

Version:
Platform:

reached out to PM on potential capacity issue


WFM api framework - HTTP RESTfull api for WFM (ex-webapi)

Motivation
Allow to use express based API to operate on WFM objects.
Note: this changes are WFM specific and it will require integration with WFM module = sparate PR in angular.js repository
Notes
I have done some investigation for pagination and I did not found anything that will meet all of our criteria. I wrote simple pagination mechanism for this module. We can extract this to separate module if needed somewhere else but I think it makes sense to have it here.
Progress

[x] Api definitions
[x] Extract to separate module
[x] Pagination
[x] Security
[x] Documentation
[ ] Unit tests

Coverage remained the same at 87.662% when pulling f8a361c3ccb57d9589819b04bda38271858a69bc on wtrocki:RAINCATCH-1145 into 0cf1e1f492e2dc9389fc7af81ae24da4c2d7f7a0 on feedhenry-raincatcher:master.

Coverage remained the same at 87.662% when pulling 2505addcb7be6810ef45b727ddd439214e6724cc on wtrocki:RAINCATCH-1145 into 0cf1e1f492e2dc9389fc7af81ae24da4c2d7f7a0 on feedhenry-raincatcher:master.

Coverage remained the same at 87.662% when pulling 2505addcb7be6810ef45b727ddd439214e6724cc on wtrocki:RAINCATCH-1145 into 0cf1e1f492e2dc9389fc7af81ae24da4c2d7f7a0 on feedhenry-raincatcher:master.

Coverage decreased (-0.3%) to 87.313% when pulling d423b82424350919a116b74bd9dd88fe3f6973ee on wtrocki:RAINCATCH-1145 into 0cf1e1f492e2dc9389fc7af81ae24da4c2d7f7a0 on feedhenry-raincatcher:master.

Changes Unknown when pulling ab13ceefe4200fa3a5bb4d1fbbef63c23cdb40e8 on wtrocki:RAINCATCH-1145 into ** on feedhenry-raincatcher:master**.

Changes Unknown when pulling ab13ceefe4200fa3a5bb4d1fbbef63c23cdb40e8 on wtrocki:RAINCATCH-1145 into ** on feedhenry-raincatcher:master**.

Changes Unknown when pulling de0fba1c54d5b1e551edc2640249785ffd57134b on wtrocki:RAINCATCH-1145 into ** on feedhenry-raincatcher:master**.

Changes Unknown when pulling 16d8bb05f40f622e19862f27632543bde2cffe2e on wtrocki:RAINCATCH-1145 into ** on feedhenry-raincatcher:master**.

Changes Unknown when pulling 8efbf0c3601475414bb6bbd1cbf5c89081bf49f4 on wtrocki:RAINCATCH-1145 into ** on feedhenry-raincatcher:master**.

Changes Unknown when pulling 112d2ca528b47e19c5506321f3eb0aa555200d8d on wtrocki:RAINCATCH-1145 into ** on feedhenry-raincatcher:master**.

Changes Unknown when pulling 112d2ca528b47e19c5506321f3eb0aa555200d8d on wtrocki:RAINCATCH-1145 into ** on feedhenry-raincatcher:master**.

Changes Unknown when pulling 6388fe3f9545c4cade47f3e0bec88320cba29951 on wtrocki:RAINCATCH-1145 into ** on feedhenry-raincatcher:master**.

Changes Unknown when pulling 6388fe3f9545c4cade47f3e0bec88320cba29951 on wtrocki:RAINCATCH-1145 into ** on feedhenry-raincatcher:master**.


Create

"Create an Azure AD" for step 10 states "Under the Settings blade, select Keys." and then 11 states, "Create a new key, and save it."  Should there be a create button?  Or, should we be create a key pair outside of the Azure Active Directory blade?

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

ID: bf203c92-3847-c2f9-f966-edaca7b59818
Version Independent ID: 76f7519d-3040-646d-93a0-2657812d81e9
Content: Azure Active Directory B2C: Add an Azure AD provider by using custom policies
Content Source: articles/active-directory-b2c/active-directory-b2c-setup-aad-custom.md
Service: active-directory-b2c
GitHub Login: @parakhj
Microsoft Alias: parakhj

Changed step 11 to: Enter the key description, select a duration, and then click Save. The value of the key is displayed. Copy it because you will use it in the steps in the next section.
#please-close

@evh69 We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Issue with latest version of strawberry 0.115.0

Prerequisites

[x] Is it a bug?
[ ] Is it a new feature?
[ ] Is it a a question?
[x] Can you reproduce the problem?
[x] Are you running the latest version?
[x] Did you check for similar issues?
[ ] Did you perform a cursory search?

For more information, see the CONTRIBUTING guide.
Description
Description of the bug/feature/question
Steps to Reproduce
upgrade to strawberry-graphql==0.115.0
Expected behavior
no conflict
the error
  File "/Volumes/SSD/projects/petroly/petroly-django/env3.10/lib/python3.10/site-packages/gqlauth/bases/mixins.py", line 195, in __init_subclass__
    for name, ret_type in parent_resolver.base_resolver.annotations.items():
AttributeError: 'StrawberryResolver' object has no attribute 'annotations'. Did you mean: '__annotations__'?

Paste the packages you are using, you can get this information from executing pip freeze.

we need temp modify the setup.py
https://github.com/nrbnlulu/strawberry-django-auth/blob/2c92b9f4285659854436c7b5059b50a650bc7d57/setup.py#L54

Hey, maybe a strawberry version check at that line will do as well.
I will look into it later today.

This will be fixed in a future release of strawberry https://github.com/strawberry-graphql/strawberry/issues/1989

fixed after https://github.com/strawberry-graphql/strawberry/releases/tag/0.116.2


hyperref Functionality Support for Fast Mode

Current Situation
Hi, I know that hyperref is turned off in the fast mode, as can been seen here.
Problem of the fast Mode with hyperref

Class: beaulivre
LaTeX Engine: TeX Live 2022 on macOS (with all updates)

However, turning that off completely can introduce some trouble in the document. I have been using this code (repo here) in the preamble to do hyperref stuff only when fast mode is false:
\newcommand{\turnoffhypercolor}{\relax}

\ExplSyntaxOn
\bool_if:NF \l__colorclass_fast_bool
  {
    \hypersetup
      {
        , pdfauthor  = { Wuqiong~Zhao~(Teddy~van~Jerry) }
        , pdftitle   = { mmCEsim~Documentation~\&~Tutorials }
        , pdfsubject = { mmCEsim }
        , colorlinks = true
        , urlcolor   = blue
        , linkcolor  = purple
        , citecolor  = red
      }
    \renewcommand{\turnoffhypercolor}{\hypersetup{hidelinks}}
  }
\ExplSyntaxOff

But in the document, we may sometimes use the \hyperref macro, for example
% Though \ref would do, we are just using this as an exmaple.
View \hyperref[sec:foo]{this section} for details.
\section{Foo}\label{sec:foo}

Check the fast mode in the main document can be tedious.
Possible Solution
I have tested by changing this line in the generated colorist-fancy.sty and it worked properly. With my limited LaTeX knowledge, I reckon hyperref in the draft mode should work since no actual link is activated.
\bool_if:NTF \l__colorist_fast_bool
  {
    \RequirePackage { hyperref } % <- added
    \hypersetup { draft }        % <- added
    % original setup for `\href` or `\texorpdfstring` removed
  }
  {
    \RequirePackage { hyperref }
    \RequirePackage { bookmark }
    \hypersetup{ hidelinks, linktoc = all }
    \bookmarksetup{ numbered }
    % other stuff
  }

The original way is to provide frequently used macros, but that can miss some (for instance \hyperref or \hypertarget) which can be dangerous, so maybe draft mode of hyperref should be more desirable? I noticed that the compiling speed is quite okay.

Additional Question
I really love this template and am wondering if pull requests are accepted. Since I have noticed that all commit messages are written in that format, I am not quite sure what to do if I submit a pull request to provide more functionality or enhance the documentation.
Here is a small list I propose to work on:

Add hyperref option interface, as is in the beamer class, so that I can use colorlinks without influencing the colour in ToC;
Change (or offer another type) of ToC style for subsection like that in memdesign which aligns those in a row so a lot of space would be saved. This is especially useful for the beaulivre class since a book can be quite long;
Enhanced documentation with examples.

Thanks again for your work in providing this beautiful template.

Thank you for this proposal! I have changed it in Update 2023-01-02 (9757f76419347d722d2f6d699e6104f71724a5a3).
I didn't think about it too much when creating the fast mode. There was originally an efficiency issue with the command \CreateTheorem, which motivated me to introduce the fast mode, and the other changes like the one with hyperref are just "made along the way". However, when this has been fixed with the introduction of the package create-theorem, I hardly use that fast mode ever since, left it less tested and maintained.
Regarding pull requests, I would be most delightful to receive other people's contributions to this project. However, to be frank, I'm not very familiar with the feature of GitHub other than using the official GUI application GitHub Desktop, so I know very little about this process.
Some notes on the next steps of this project:

New styles. Recently I have been mainly focusing on the parallel project minimalist. If you compare the commits, you would probably find that recent changes of the current project are mostly the same as there. For now, minimalist already contains 4 styles plain, classical, stream and flow (the latter two yet to be integrated into the .dtx file), while coloristhas only one style currently named fancy. A good news is that there are 5 more styles already in my mind (two of which resemble the stream and flow styles of minimalist, the other three come from a few beautiful designs I've seen recently) yet to be realized, which should be finished in the next two years.
Doc and demo. The documentations are, unfortunately, completely out-of-date and require a thorough rewriting -- and it is also the case for minimalist and ProjLib. However, I still haven't yet came up with a good framework to begin with, nor have sufficient time to develop the ideas. For the TeX Live 2023 coming this April, I would probably just remove the currently documentations but keep the code to be the newest version. However, documentations and demos are always on the schedule.
Code clear up. The current code is build up gradually in two years and there is a major update early last year with the transition to expl3. The resulting code is poorly formatted and less organized comparing to smaller (and newer) project like create-theorem.
Details. The design of ToC, the font issues, the fast mode, the behaviors under different engines...

Thank you for your detailed explanations, and now I have a big picture of this project (as well as minimalist).
I personally love fancy and colourful stuff, so I am currently using the colorist series most. I assume that minimalist will suit many course reports perfectly!
Pull requests are quite simple and can be manipulated on github.com readily. The process will be:

I commit changes in my fork of colorist or minimalist;
I start the pull request, and you will see that on github.com;
You review my changes and make comments;
I make changes according to your comments, until you find them satisfactory;
You then click the button to merge the changes;
A result different from 5 can also be rejecting the changes.

I have gained some experience writing LaTeX2e documents (and publishing a naive package to CTAN) and I am learning expl3 recently. So I am willing to contribute to this project (as well as minimalist), maybe starting with small issues including enhancement of documentation which I am capable to help.

Thank you for this explanation! Seems that this would be a nice interacting process :)
To leave you some impression of minimalist, here are a few documents I've created recently:

style plain;
style flow, with sections and theorems sharing the same numbering, and colored proofs;
and style stream, with numbering in the left margin and allowing empty section titles, sections and theorems sharing a same "stream" of numbering.

(These files are just for demonstrating the typographic result, I do not claim the copyright of their content.)
As for colorist, there would also be a style possibly named classical similar to the book Linear Algebra Done Right; a style possibly named simple similar to the various titles from the "Sciences Sup" collections in French; I'm also considering learning from the styles of the new book Explorations in Number Theory, various titles from the series "Oxford Graduate Texts in Mathematics" and some other nice-looking books -- once there is a good design on paper, writing the code won't be too hard -- but of course, all these have to wait until new documentations and demo documents are prepared.


[Feature Request] Spooky  8-ball

The goal of this task is to create an 8-ball that will generate random answers in a spookifying way, to support the halloween theme.

A user will ask a the bot a question via a command !8ball, is discord python awesome? and the bot will answer just like an 8-ball. (hhmm)

[ ] list of spooky answers,
[ ] 8-ball image or emoji

Hey , can I give this a shot ?

Hey @WrightKD how is it coming? Are you still working on this?

Closing until next October.

Id like to take a shot at this, if no one else has.

it's all yours,

merged

No PR for this has been created

Has this been completed? If not I am down to take this on. Please do let me know.


python310Packages.pathspec: 0.9.0 -> 0.10.1

Description of changes
https://github.com/cpburnz/python-pathspec/blob/v0.10.1/CHANGES.rst
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg build awsebcli
Appears broken by this.

(tested against the set of packages in #191558)

The pointers are much appreciated. I'm doing more testing now locally.
As an aside, as I was building awsebcli, a sphinx test that checked that connecting to an unknown domain name was failing for me. It turns out, I have AT&T Fiber, and the router they installed uses a custom DNS server that returns some AT&T IP for unknown domain names...

It seems that pathspec was already updated. It's in staging-next. I'll update this PR to address the remaining feedback.

Do we need to add setuptools to nativeBuildInputs?

Yes, great catch! I built this on staging to verify, so it took a few hours, but it does fail without it.

I think this will be needed by https://github.com/NixOS/nixpkgs/pull/191339.
I will try to rebase onto staging-next following the instructions in CONTRIBUTING.md. Apologies in advance if I mess it up.

It doesn't seem needed.

Ok, let's keep this here for now. I did only build this on staging and not staging-next and less recently than you. Thank you for checking!

fyi: In this case adding setuptools becomes necessary when you switch to pyproject format.

I rebased and updated the commit message.


TypeError: Network request failed when using fetch in Android

Description

Fetch request fails only on Android, succeeds on iOS.
Fails only on my team servers. Success on other servers. So, I changed https to http (my team server and client) and it succeeded.
Same problem with axios.
android:usesCleartextTraffic="true" => Also not helpful
Flipper ...?? https://github.com/facebook/react-native/issues/28551 => Also not helpful

React Native version:
System:
OS: macOS 11.3
CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
Memory: 731.05 MB / 16.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 14.15.1 - /usr/local/bin/node
Yarn: 1.22.10 - /usr/local/bin/yarn
npm: 7.10.0 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.10.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 14.4, DriverKit 20.2, macOS 11.1, tvOS 14.3, watchOS 7.2
Android SDK: Not Found
IDEs:
Android Studio: 4.1 AI-201.87<IP_ADDRESS>99119
Xcode: 12.4/12D4e - /usr/bin/xcodebuild
Languages:
Java: 11.0.10 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.1 => 17.0.1
react-native: 0.64.0 => 0.64.0
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps To Reproduce

set up project
npx react-native init AwesomeTSProject --template react-native-template-typescript

App.tsx
async function helloServer() {
try {
const response = await fetch("https://www.*****.com/greeter/hello", {
headers: new Headers({ "Content-Type": "application/json" }),
});
const json = await response.json();
console.log(json);
} catch (error) {
console.log(error);
}
}

export default function App() {
useEffect(() => {
helloServer();
}, []);
return (

Open up App.tsx to start working on your app!

);
}

npm run android (test with device)

Expected Results
Server response should be logged.

It was a SSL certificate issue. :(

Same issue exists. The API call is working in a real device, but not in the emulator.
Now, my concern is, Should I worry that it doesn't work on emulator? It will be good if someone can answer this question, so that I can work with the real device until this issue is sorted out.


Support MKDIR and RMDIR commands for ProFTPd

FTP OS: Unix
FTP Server: ProFTPD
Now that we support detection of ProFTPd via the ServerType, we should support MKDIR and RMDIR internally to speed up those operations on ProFTPd.
This issue has more info: https://github.com/robinrodricks/FluentFTP/issues/88#issuecomment-304430823

Done in 25.1.0.


Small bugs

Weird ascii character in README.md broke pip install .
Wrong name in __init__.py broke from matrixprofile import *

Thanks @peterdhansen !


DiffWithPreviousCommand — getPreviousDiffUris(, , undefined) errror

GitLens Version: 9.7.1
VSCode Version: 1.33.1
OS Version: Windows 10 with WSL (ubuntu 16.04)

vscode version info
Version: 1.33.1 (system setup)
Electron: 3.1.6
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.17763

I've been using gitlens for a long time, it works very well. At last auto upgrade, when I click a new file (not exists in previous repos) in file changed list, will get a error:
DiffWithPreviousCommand — getPreviousDiffUris(d:/src/apps/app, d:\src\apps\app\database\migrations\2019_05_05_195442_create_receipts_table.php, undefined)
Error: File has no history

It's in previous version has no problem 😂

@andares I'm not sure I understand the steps to reproduce this issue, can you please provide some detailed steps that I can follow to see this issue? Thanks!

I try to show it with screenshots 😄
step 1
I have a repos

step 2
create a file

step 3
check file by gitlens view

step 4
check output info


feat: Mount GCS volumes

I gave it a try to add support to mount GCS volumes as explained in https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/cloud_run_v2_service#example-usage---cloudrunv2-service-mount-gcs
Tbh I wasn't able to test the generated output so some help would be appreciated.

Closing this PR as we're trying to favor the newer module https://github.com/gr4vy/terraform-gcp-cloud-run-service


[0.66-stable] feat: pruning unused methods from shared libraries

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[ ] I am making a fix / change for the macOS implementation of react-native
[x] I am making a change required for Microsoft usage of react-native

Summary
Many of the shared libraries built in RNM used in React Android export methods that are neither referenced nor linked to. This change sets the default visibility of the exported methods to hidden, and ensures that only the methods being referenced are public.
Size improvements:
libmapbufferjni.so: -12 KB
libhermes-executor-release.so: -24 KB
libhermes-executor-debug.so: -24 KB
libturbomodulejsijni.so: -28 KB
librntester_appmodules.so: -56 KB
libreactnativeutilsjni.so: -204 KB
libreactnativejni.so: -204 KB
libfabricjni.so: -408 KB
Changelog

[Android] [Fixed] - Pruned unused methods from shared libraries

any chance we can upstream this? Also, any specific reasons for doing this on 0.66 branch directly and not main then cherry pick?

also, I see that you are putting the changes in the pre-existing Build patch folder, could you make it into its own folder? Accumulating different changes in the same patch makes it harder to de-fork after

@kelset Ah, sure. I'll send a PR to main then.
As for upstreaming - I think we can, but may need to check with Facebook once if this is something they want. Or do you recommend sending a PR anyway?

As for upstreaming - I think we can, but may need to check with Facebook once if this is something they want. Or do you recommend sending a PR anyway?

I think it'd be better to submit the PR and then ask for feedback on it directly - main reason being that the approach used in this repo with the patches means that it's harder to read/see the actual changes. So the alternative, which would be to send the engineers from Meta a link to this PR and ask for feedback, is a bit harder to parse which might lead to them taking more time.
Instead, by doing the PR on upstream too, you would be able to make it easier for them to check it out + their CI will validate it (and also, you can then cross-reference more easily the PR on this side with the one on upstream, which will make it easier to defork if/once it lands upstream)

closing this in favor of #1186


Chess: Move B4 to C5

Please do not change the title. Just click "Submit new issue". You don't need to do anything else :D

Sorry @UTkbxRME7c9C, you can't move twice in a row! You can ask someone to play the next turn :D


Compilation Caching Not Working As Expected When Moving Between Commits (forge verify-bytecode)

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (c486fca 2024-05-06T00:20:07.911403000Z)

What command(s) is the bug in?
forge verify-bytecode 0xba2492e52F45651B60B8B38d4Ea5E2390C64Ffb1 SystemConfig --rpc-url $MAINNET_RPC_URL --etherscan-api-key $ETHERSCAN_API_KEY

Operating System
macOS (Apple Silicon)
Describe the bug
Steps to reproduce
I've noticed two cases of unexpected behavior.

Redundant compilation on Optimism repo tag: op-contracts/v1.1.0

# Clone optimism monorepo
git clone https://github.com/ethereum-optimism/optimism.git

cd optimism

# Clean env and install dependencies then navigate to foundry project and install deps
git checkout op-contracts/v1.1.0 &&
        pnpm clean && pnpm install && pnpm build &&
        cd packages/contracts-bedrock &&
        rm -rf lib/ && forge clean && forge install

# Run forge verify-bytecode (watch as it compiles the code, this is expected)
# You'll have to add in your own API keys for this command.
forge verify-bytecode 0xba2492e52F45651B60B8B38d4Ea5E2390C64Ffb1 SystemConfig --rpc-url $MAINNET_RPC_URL --etherscan-api-key $ETHERSCAN_API_KEY

# e.g.
# Compiler output:
# [⠒] Compiling...
# [⠢] Compiling 82 files with Solc 0.8.25
# [⠆] Compiling 286 files with Solc 0.8.15
# [⠰] Compiling 1 files with Solc 0.5.17
# [⠒] Compiling 34 files with Solc 0.8.19
# [⠑] Solc 0.5.17 finished in 82.85ms
# [⠆] Solc 0.8.19 finished in 2.16s
# [⠒] Solc 0.8.25 finished in 2.63s
# [⠢] Solc 0.8.15 finished in 74.53s
# Compiler run successful!
# Creation code did not match - this may be due to varying compiler settings
# EVM version mismatch: local=paris, onchain=london
# Runtime code matched with status partial

# Run the same command again
forge verify-bytecode 0xba2492e52F45651B60B8B38d4Ea5E2390C64Ffb1 SystemConfig --rpc-url $MAINNET_RPC_URL --etherscan-api-key $ETHERSCAN_API_KEY

# You'll notice it doesn't use the cache, instead it recompiles again. Producing the same output as above.

Wrong cache is used across tags: op-contracts/v1.3.0 and op-contracts/v1.1.0

# Clear any git changes in the optimism repo
git restore .

# Clean env and install dependencies then navigate to foundry project and install deps
git checkout op-contracts/v1.3.0 &&
        pnpm clean && pnpm install && pnpm build &&
        cd packages/contracts-bedrock &&
        rm -rf lib/ && forge clean && forge install

# Run forge verify-bytecode (watch as it compiles the code, this is expected)
# You'll have to add in your own API keys for this command.
forge verify-bytecode 0xba2492e52F45651B60B8B38d4Ea5E2390C64Ffb1 SystemConfig --rpc-url $MAINNET_RPC_URL --etherscan-api-key $ETHERSCAN_API_KEY
# Notice it fully compiles as expected

# Navigate back to optimism/ and then clear any local git changes
cd ../../ && git restore .

git checkout op-contracts/v1.1.0

cd packages/contracts-bedrock

# Run the same command
forge verify-bytecode 0xba2492e52F45651B60B8B38d4Ea5E2390C64Ffb1 SystemConfig --rpc-url $MAINNET_RPC_URL --etherscan-api-key $ETHERSCAN_API_KEY

# Notice no compilation output. It reuses compilation output from op-contracts/v1.3.0

This is due to https://github.com/foundry-rs/foundry/pull/7358.
On v1.1.0 build_info = true in foundry.toml, which disables caching and forces compilation. Whereas, on v1.3.0 the build_info flag is removed from foundry.toml, thus enabling the cache.

@yash-atreya do you expect this issue to be worked on anytime soon?

we've improved build-info tracking and enabled cache for runs with --build-info flag, so this should be resolved I believe https://github.com/foundry-rs/foundry/pull/8164

Following up on this, @blmalone could you check if you are still running into issues?

@zerosnacks I tried to test this to confirm it's working but it seems another bug has been introduced: https://github.com/foundry-rs/foundry/issues/8555

For reference, related PR for #8555 : https://github.com/foundry-rs/foundry/pull/8560

Going to mark this as resolved as I'm not seeing the same behavior anymore.


Intake plugin not pulling in venv

When using terminal in virtual environment with python=3.12 and all dependancies added:
uv pip install pandas matplotlib zarr fsspec s3fs intake intake_xarray intake_parquet ipython jinja2
ipython and jinjia2 are not in the pip install at the top of the page:
ipython is imported into the code, but I don't believe it is actually used so could be taken out of imports.
jinja2 is used but is not in the pip install list at the top of the page.
I then run the code as it is written in the document
when I get to line:
dataset = catalog.level1.shots(url=shot.url, group='amc').to_dask()
I get error message:
ValueError: No plugins loaded for this entry:
intake_xarray.xzarr.ZarrSource
which I believe is from .to_dask(). It does still work when I am in collab however.
Added issue before noticing it was a python version error. The code works for python=3.11 and hence why it works in collab.

Pip installs that were missing have now been added.
The fact that it doesn't work for python 3.12 to 3.13 has not been resolved.

I've investigated this and it is because the intake-xarray plugin imports distutils, which was removed in python 3.12.
I'll open an issue there and see how that goes.

It looks to me like we should remake the intake catalogues using just intake, rather than the plugin.
See https://github.com/intake/intake/issues/850 for details.


[ACR] Improve repository commands error messages

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ ] I adhere to the Command Guidelines.

View a preview at https://prompt.ws/r/Azure/azure-cli/6047
This is an experimental preview for @microsoft users.


Make send/request form routes fullscreen

r? @keybase/react-hackers

@MarcoPolo thanks for looking, I still have to add a fix to make the safearea + keyboardavoidingview play nice with eachother

[ ] fix keyboard covering confirm button on create new account / link existing account

OK, I spent some more time trying to make it work on the create new / link existing dialogs with no luck, so I reverted the changes to their routing and WalletPopup. Now this all works except for when you make a new account from the "Send to other account" form, in which case the bottom button is janky on newer iPhones and notched android phones

@keybase/react-hackers PTAL


Incorrect coverage if using chained operators

Describe the bug
When using chained operators in an if statement (4 > i > 2), the code coverage report incorrectly shows that lines of code didn't run when they did.
To Reproduce
How can we reproduce the problem?  Please be specific. Don't link to a failing CI job. Answer the questions below:

What version of Python are you using? Python 3.8.10
What version of coverage.py shows the problem? The output of coverage debug sys is helpful. Coverage 6.4.4
What versions of what packages do you have installed? The output of pip freeze is helpful. Run in a clean virtual environment, so minimal packages were installed. Output of pip freeze is below.

coverage==6.4.4
pkg_resources==0.0.0

What code shows the problem?  Give us a specific commit of a specific repo that we can check out. If you've already worked around the problem, please provide a commit before that fix.

Consider the two following code samples:
test_chained.py
i = 0
while True:
    if 4 > i > 2:
        break

    i += 1

test_no_chained.py
i = 0
while True:
    if 4 > i and i > 2:
        break

    i += 1

Both samples perform identical functions, so the code coverage should be the same.

What commands did you run?

Output of coverage run test_chained.py && coverage report --show-missing
Name              Stmts   Miss  Cover   Missing
-----------------------------------------------
test_chained.py       4      1    75%   4
-----------------------------------------------
TOTAL                 4      1    75%

Output of coverage run test_no_chained.py && coverage report --show-missing
Name                 Stmts   Miss  Cover   Missing
--------------------------------------------------
test_no_chained.py       4      0   100%
--------------------------------------------------
TOTAL                    4      0   100%

Expected behavior
For the code samples given, the coverage run results should be the same since the code performs the same function.  Both code samples must run Line 4 (which calls break) because if they didn't, the code would never terminate (but it does in both cases).  It seems that code coverage is reported incorrectly if compound operators are used.
Additional context
N/A

It looks like this is a result of compiler optimizations that vary across versions:
chained.py:
def chained():
    i = 0
    while True:
        if 4 > i > 2:
            break

        i += 1

def no_chained():
    i = 0
    while True:
        if 4 > i and i > 2:
            break

        i += 1

chained()
no_chained()

Running 3.7 through 3.11:
% for py in py37 py38 py39 py310 py311; do
   echo; echo === $py ==============
   .tox/$py/bin/coverage run --branch /tmp/chained.py
   .tox/$py/bin/coverage report -m
done

=== py37 ==============
Name                      Stmts   Miss Branch BrPart  Cover   Missing
---------------------------------------------------------------------
/private/tmp/chained.py      14      0      4      0   100%
---------------------------------------------------------------------
TOTAL                        14      0      4      0   100%

=== py38 ==============
Name                      Stmts   Miss Branch BrPart  Cover   Missing
---------------------------------------------------------------------
/private/tmp/chained.py      12      1      4      1    88%   5
---------------------------------------------------------------------
TOTAL                        12      1      4      1    88%

=== py39 ==============
Name                      Stmts   Miss Branch BrPart  Cover   Missing
---------------------------------------------------------------------
/private/tmp/chained.py      12      1      4      1    88%   5
---------------------------------------------------------------------
TOTAL                        12      1      4      1    88%

=== py310 ==============
Name                      Stmts   Miss Branch BrPart  Cover   Missing
---------------------------------------------------------------------
/private/tmp/chained.py      14      0      4      0   100%
---------------------------------------------------------------------
TOTAL                        14      0      4      0   100%

=== py311 ==============
Name                      Stmts   Miss Branch BrPart  Cover   Missing
---------------------------------------------------------------------
/private/tmp/chained.py      14      0      4      0   100%
---------------------------------------------------------------------
TOTAL                        14      0      4      0   100%

We can look at the compiled bytecode for 3.8 and see that line 5 can't be reached.  The jump to jumps are optimized away:
% python3.8 -m dis /tmp/chained.py
  1           0 LOAD_CONST               0 (<code object chained at 0x105356030, file "/tmp/chained.py", line 1>)
              2 LOAD_CONST               1 ('chained')
              4 MAKE_FUNCTION            0
              6 STORE_NAME               0 (chained)

  9           8 LOAD_CONST               2 (<code object no_chained at 0x105356190, file "/tmp/chained.py", line 9>)
             10 LOAD_CONST               3 ('no_chained')
             12 MAKE_FUNCTION            0
             14 STORE_NAME               1 (no_chained)

 17          16 LOAD_NAME                0 (chained)
             18 CALL_FUNCTION            0
             20 POP_TOP

 18          22 LOAD_NAME                1 (no_chained)
             24 CALL_FUNCTION            0
             26 POP_TOP
             28 LOAD_CONST               4 (None)
             30 RETURN_VALUE

Disassembly of <code object chained at 0x105356030, file "/tmp/chained.py", line 1>:
  2           0 LOAD_CONST               1 (0)
              2 STORE_FAST               0 (i)

  4     >>    4 LOAD_CONST               2 (4)
              6 LOAD_FAST                0 (i)
              8 DUP_TOP
             10 ROT_THREE
             12 COMPARE_OP               4 (>)
             14 POP_JUMP_IF_FALSE       24
             16 LOAD_CONST               3 (2)
             18 COMPARE_OP               4 (>)
             20 POP_JUMP_IF_FALSE       30
             22 JUMP_ABSOLUTE           40
        >>   24 POP_TOP
             26 JUMP_FORWARD             2 (to 30)

  5          28 JUMP_ABSOLUTE           40

  7     >>   30 LOAD_FAST                0 (i)
             32 LOAD_CONST               4 (1)
             34 INPLACE_ADD
             36 STORE_FAST               0 (i)
             38 JUMP_ABSOLUTE            4
        >>   40 LOAD_CONST               0 (None)
             42 RETURN_VALUE

Disassembly of <code object no_chained at 0x105356190, file "/tmp/chained.py", line 9>:
 10           0 LOAD_CONST               1 (0)
              2 STORE_FAST               0 (i)

 12     >>    4 LOAD_CONST               2 (4)
              6 LOAD_FAST                0 (i)
              8 COMPARE_OP               4 (>)
             10 POP_JUMP_IF_FALSE       22
             12 LOAD_FAST                0 (i)
             14 LOAD_CONST               3 (2)
             16 COMPARE_OP               4 (>)
             18 POP_JUMP_IF_FALSE       22

 13          20 JUMP_ABSOLUTE           32

 15     >>   22 LOAD_FAST                0 (i)
             24 LOAD_CONST               4 (1)
             26 INPLACE_ADD
             28 STORE_FAST               0 (i)
             30 JUMP_ABSOLUTE            4
        >>   32 LOAD_CONST               0 (None)
             34 RETURN_VALUE

Thanks for the quick and detailed reply!  I see what you mean, and in a sense I suppose optimizing the code this way isn't even a bug, just a way to skip an unnecessary instruction.
That said, I do wish there were a way that coverage.py could recognize this behavior, since it's a bit counterintuitive to users since that part of the code is in fact reached (it's just that Python recognizes that if it enters that part of the loop, it can skip ahead directly).  However, I'm not familiar with how coverage.py works under the hood, so I'm assuming this would probably take pretty significant changes and may not be worth adding for a likely uncommon edge case.
Regardless, thanks for your time and help investigating and explaining this issue.


deleteOne is not a function

It exists in Mongodb-Native, but does not exist in monk?
http://mongodb.github.io/node-mongodb-native/2.0/api/Collection.html#deleteOne
Remove has been deprecated and written to use deleteOne instead.

Thanks for this great lib.

Yes. But I really don't like the API of having 2 methods to do the same thing.
I think I'll modify the monk remove to use the mongo deleteOne and deleteMany depending on wether there is the option multi or single