Co2signal

Description:
This PR adds the CO2 signal component. The sensor reports the CO2 intensity (in CO2eq/kWh) in a specific region based upon the actual generation in that region and the imports and exports with neighboring regions. For instance, the sensor could be used to start devices if the CO2 emissions go below a certain threshold (e.g. charging an electrical vehicle).
The data can be checked on https://www.electricitymap.org/.
A token for the API is available via https://www.co2signal.com/.
Related issue (if applicable):
Not applicable
Pull request in home-assistant.io with documentation (if applicable):
Todo
home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: co2signal
    token: SECRET # get yours for free (for personal use) from co2signal.org
    country_code: BE
    refresh_rate: 15 # minutes between polling

Checklist:

[ ] The code change is tested and works locally (as custom component).
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hi @danielsjf,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @danielsjf,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Could not find any related documentation PR in our documentation repository. Adding docs-missing label.


feat(turborepo): Using file hashing for package watching

Description
We leverage the file hash watcher to make watch mode more stable. Basically, once we get a list of changed packages, we query the file hash watcher to see if there are file hashes available. If so, we check if we've seen these file hashes before. If they are, then we don't re-run.
This does have some caveats. For one, file hashes are not exactly correct for whether a task should be re-run. We really should be checking task hashes, but that will require more daemon infrastructure. Also files are most likely the only changes a user will make with watch mode (will they edit env vars? Probably not), so this is good enough.
Also, this will only work on the second run, since we need to get the file hash and store it. Maybe even the third if hashing takes too long to get.
Testing Instructions
I tested on the next.js repo. You can confirm the behavior by looking at the daemon logs and seeing that we don't get a re-run when running touch foo multiple times, since the file has the same content.
Closes TURBO-3010

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#8104  👈
#8064
main

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

Join @NicholasLYang and the rest of your teammates on  Graphite

This does have some caveats. For one, file hashes are not exactly correct for whether a task should be re-run. We really should be checking task hashes, but that will require more daemon infrastructure. Also files are most likely the only changes a user will make with watch mode (will they edit env vars? Probably not), so this is good enough.

I don't even know how you can change env vars of a running process :)


Merge from main to develop for latest release

Checklist: (Put an x in all the boxes that apply)

[X] I have read the CONTRIBUTING document.
[X] My code follows the code style of this project.
[X] I have updated the documentation accordingly.
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.
[X] I have checked that there aren't any other open Pull Requests from the same change.
[X] The develop branch has been set as the base branch to merge changes of the pull request.
[X] I have updated the CHANGELOG.md file as appropriate.

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bring develop commit history in line with main
What is the current behavior? (You can also link to an open issue here)
What is the new behavior? (if this is a feature change)
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information:

No longer required. Used a branch from my fork instead to resolve merge conflicts.


Too many nullables

I think the library uses too many nullables. For example, here I'm explicitly setting a custom view. So it is not null. Of cause it only returns not null if I did set a custom view. So the way it works now I have to suppress many lint warnings which makes the code unreadable.

It forces you to code defensively, and the library can't guarantee that the result won't be null if there's any case where it does.

Yeah, the usage there is valid.


UE failed to ping UPF after separating RAN and 5GC

@herlesupreeth
Hello, Herle and every other lovely guy.
I have tried to externalize ueransim and deploy it in a separate VM that resides outside the 5GC SA networks.
ueransim part was left as much the way as it used to be.
My version of open5gs is v2.4.7, ueransim version is v3.2.6.
This is the specific topology:

Both of the gNB and UE were up, but I found that the UE failed to ping <IP_ADDRESS>, that is the data plane of UPF.

gNB log:

ip route table in gNB

NIC list in gNB

UE log:

ip route table in UE

NIC list in UE

What's weird was that UE was able to ping <IP_ADDRESS> through the NIC eth0.

And if you look at the proper examples in the integrated version set up before, you can find that the name of eth0 should be eth0@uesimtun0 instead of eth0@if14. In that way, the traffic enters eth0 will finally pass through the GTP-U tunnel.
the proper one:

From here, I will show what have been configured in the VMs.
I've done these things in the VM-5GC, aka 5GC SA networks:

In the .env file, change UPF_ADVERTISE_IP -> DOCKER_HOST_IP
In the sa-deploy.yaml file, uncomment these 2 code blocks in order to enforce port mappings.

    # ports:
    #   - "38412:38412/sctp"

    # ports:
    #   - "2152:2152/udp"

I've made a new folder ran/ for the individual RAN.
Here is a breakdown of the file system.
ran/
  gnb/
    Dockerfile
    open5gs-gnb.yaml
  ue/
    Dockerfile
    open5gs-ue.yaml
  nr-gnb.yaml
  nr-ue.yaml

I will list a series of aforementioned configuration out here:
ran/gnb/Dockerfile
# These are some built images used as prerequisites
FROM docker_ueransim:builder AS ueransim_builder
FROM docker_ueransim:ueransim_provenance

# I just put every single environmental constants in place,
# then extract the original shell codes and put them into a static Dockerfile
COPY --from=ueransim_builder /UERANSIM/build/nr-gnb /ueransim/nr-gnb
COPY open5gs-gnb.yaml /ueransim
WORKDIR /ueransim
CMD     export IP_ADDR=$(awk 'END{print $1}' /etc/hosts) && \
        ./nr-gnb -c open5gs-gnb.yaml & bash

ran/gnb/open5gs-gnb.yaml
......
......
linkIp: <IP_ADDRESS>  # gNB's local IP address for Radio Link Simulation (Usually same with local IP)
ngapIp: <IP_ADDRESS>   # gNB's local IP address for N2 Interface (Usually same with local IP)
gtpIp: <IP_ADDRESS>   # gNB's local IP address for N3 Interface (Usually same with local IP)

# List of AMF address information
amfConfigs:
  - address: <IP_ADDRESS>
    port: 38412
......
......

ran/ue/Dockerfile
FROM docker_ueransim:builder AS ueransim_builder
FROM docker_ueransim:ueransim_provenance

COPY --from=ueransim_builder /UERANSIM/build/nr-ue /ueransim/nr-ue
COPY open5gs-ue.yaml /ueransim
WORKDIR /ueransim
CMD     export IP_ADDR=$(awk 'END{print $1}' /etc/hosts) && \
        ./nr-ue -c open5gs-ue.yaml & bash

ran/ue/open5gs-ue.yaml
......
......
# List of gNB IP addresses for Radio Link Simulation
gnbSearchList:
  - <IP_ADDRESS>
......
......

ran/nr-gnb.yaml
version: '3'
services:
  nr_gnb:
    image: docker_ueransim_gnb
    container_name: nr_gnb
    stdin_open: true
    tty: true
    volumes:
      - /etc/timezone:/etc/timezone:ro
      - /etc/localtime:/etc/localtime:ro
    expose:
      - "38412/sctp"
      - "2152/udp"
      - "4997/udp"
    cap_add:
      - NET_ADMIN
    devices:
      - "/dev/net/tun"
    privileged: true
    networks:
      default:
        ipv4_address: <IP_ADDRESS>
networks:
  default:
    ipam:
      config:
        - subnet: <IP_ADDRESS>/24
          gateway: <IP_ADDRESS>

ran/nr-ue.yaml
version: '3'
services:
  nr_ue:
    image: docker_ueransim_ue
    container_name: nr_ue
    stdin_open: true
    tty: true
    volumes:
      - /etc/timezone:/etc/timezone:ro
      - /etc/localtime:/etc/localtime:ro
    expose:
      - "4997/udp"
    cap_add:
      - NET_ADMIN
    privileged: true
    networks:
      default:
        ipv4_address: <IP_ADDRESS>
networks:
  default:
    name: ran_default

In light of the fact that there is only ONE IP allocated per VM, so that there is no more host IP space for a vlan (ipvlan or macvlan) driver, though I've made it through by using a ipvlan for the target scenario. I have to use the bridge driver to save IP addresses.
I hope whoever is looking at this would come up with a favorable advice or solution. Thank you in advance.

Can you please attach a trace for the issue taken on both the VMs?
My suspicion is that since both (gNB + UE) and 5GC are running in two VMs behind a NATed IPs (192.168.100.X) there maybe a need for static route in 5GC host machine to reach gNB

@herlesupreeth
I have been weighing up the possibility of the circumstances you have suggested. But in fact, if there is any problem with the visibility for 5GC to reach gNB, then it can't be done for UE to ping UPF through eth0, for which I have shown a clip in above statement. So I guess the nature of the problem is that the eth0 failed to bind the uesimtun0, so there is not any traffic that have been sent through uesimtun0. But unfortunately, the UERANSIM pathetically indicated that the tunnel has been established.
I've used tcpdump to capture the ICMP packets of ping. Actually, when we used the uesimtun0 as the NIC ingress, the tcpdump didn't receive any packets. On the contrary, when we used eth0 as the NIC ingress, ICMP traffic could be seen on the screen.
From the perspective of RAN when pinging through eth0

From the perspective of 5GC when ping through eth0

@RaulB16
Sorry, just recognize your comment.
I've adopted a new docker network driver named IPVLAN.
It acts like the UE/gNB uses a IP address in the same subnet as the host of 5GC.
For example, UE (<IP_ADDRESS>) gNB (<IP_ADDRESS>) 5GC host (<IP_ADDRESS>).
The 5GC NFs lurk behind the 5GC host as before under a bridge network driver.


volume 4 is missing

there is a partial media folder, but no content. The latest can be found in the IT-Infrastructure github
https://github.com/IHE/IT-Infrastructure/tree/master/ForPublication

Added converted Volume 4 files


Refactor update surface to use RelationalCommand instead of DbCommand

Factoring change to remove raw DbCommand use from Update
@AndriySvyryd @ajcvickers

:shipit:


Notebook reorg

Starting the new notebooks folder with user guides

Really nice intro guide!!
Since I can't comment on the notebook inline in the diff on github, some notes along the way:

intro_guide.ipynb:

The w, s, e, n, I might be explicit here and call them west, south, east, north, I think that's more readable, and more clear for people less familiar with the concept
Places "You can also access tiles of places with a text-based search, and contextily will do its best to find you a solution:" -> maybe mention this is using geopy under the hood as acknowledgment ?
Places -> de is a strange variable name (at least it's not clear to me what it would be an abbreviation of)
For a future release, but shall we publicly expose _calculate_zoom ? Or rather add a verbose option to bounds2img / add_basemap (in any case worth opening an issue for this probably)

General thing: how shall we import contextily? You now did import contextily as cx, but I think previously we used import contextily as ctx in the notebook? The few examples in the reference guide (https://contextily.readthedocs.io/en/latest/reference.html) also use ctx, but those can be updated of course. (and also just import contextily is still an option of course ;-))
Something else: we now have a single python file examle at https://github.com/geopandas/contextily/blob/master/examples/plot_map.py, it might be good to integrate that in one of the notebooks (or decide it is already covered by the intro guide, actually), and then remove that?

Just pushed a couple of minor changes on your points @jorisvandenbossche, thanks very much!
On the last bit: I agree, that is probably deprecated as all the functionality is now on the guide intro.
On the import, mea culpa :) I've started using cx as it's even shorter and comfier to use. My view would be to use that one throughout going forward, but I'm open to different views, it's mostly typing laziness...

If using an abbreviation, I would personally keep it on ctx since that is already used in several places (eg the geopandas docs, probably other existing material using it). But in the end, also not big of a deal.


Question about useFast=False when loading PhoBERT Tokenizer

Hi @datquocnguyen ,
I remember you mentioned in some documents that when we use transformer v4, we should add useFast=False when loading the Tokenizer.
Is that still true for now? As I couldn't find that document again and your latest README seems to be updated too.
Another question is: what is the difference between useFast = True and useFast = False in this case? Anything change in the output?
Thank you very much.

"useFast = True" and "useFast = False" produce the same output tokenization.
"useFast = True" (by default) would help run the latest examples in https://github.com/huggingface/transformers/tree/main/examples/pytorch and the likes with the fast PhoBERT Tokenizer (installation shown in the readme), while "useFast = False" will be used for examples available in https://github.com/huggingface/transformers/tree/main/examples/legacy

Thank you.
On Sat, Nov 26, 2022 at 5:31 PM Dat Quoc Nguyen @.***>
wrote:

"useFast = True" and "useFast = False" produce the same output
tokenization.
"useFast = True" (by default) would help run the latest examples in
https://github.com/huggingface/transformers/tree/main/examples/pytorch
and the likes with the fast PhoBERT Tokenizer (installation shown in the
readme), while "useFast = False" will be used for examples available in
https://github.com/huggingface/transformers/tree/main/examples/legacy
—
Reply to this email directly, view it on GitHub
https://github.com/VinAIResearch/PhoBERT/issues/43#issuecomment-1328021423,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAFS22CGRKIHEXRLFA6YETWKHRJDANCNFSM6AAAAAASL4O5TI
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Sent from my iPhone


Addressed 2 issues + added a section to an effected guide.

I addressed the following issues:

#42446 in the tensorflow repo.
#802 in the keras-io repo.

I also added to the training_with_built_in_methods guide a demonstration of implementing a custom loss class using LossFunctionWrapper. I'm not sure if this is best practice since this class is not exported with @keras_export, but I believe it would be helpful for people who need this guide.
This is my first time contributing to an open source project, so I would appreciate comments and criticism :)
P.S. should I update the Last modified section in the 2 effected guides?

LGTM, thanks for the PR!

Cool, should I add the .md & .ipynb files now?
And should I update the date in the Last modified section of the tutorials?

Cool, should I add the .md & .ipynb files now?

Please do! You can just run the add_guide command.

And should I update the date in the Last modified section of the tutorials?

Sure!


PYIC-5032: Create common vc class

Proposed changes
What changed

Why did it change

Issue tracking

PYIC-XXXX

Checklists
Environment variables or secrets

[ ] No environment variables or secrets were added or changed

[ ] Documented in the README
[ ] Added to deployment repository
[ ] Added to local startup repository

Other considerations

[ ] Update README with any new instructions or tasks

NB: Ran E2E tests and they all pass


The sui-move-analyzer plug-in is installed in win10 vscode. Only code highlighting, no code error prompts. How to get code error prompts?

The sui-move-analyzer plug-in is installed in win10 vscode. Only code highlighting, no code error prompts. How to get code error prompts?

+1

Please ask on Sui's discord. The move analyzer in this repo is not maintained. It has been forked and evolved elsewhere.


default.nix is impure

https://github.com/nagy/nur-packages/blob/6d4ca3a1f1725bbb9bbe921411e4221eaa97658a/default.nix#L8 is impure, as it depends on the system nixpkgs. The easiest solution would be to just copy by-name-overlay.nix to this repository.

Should be fixed now.


Remove-GceFirewall does not take a firewall object properly.

Remove-GceFirewall will not take cross project firewall objects. It will assume the given object is of the given or default project.

Closed in PR #210.


[FALSE-POSITIVE] ||com^

Domains or links
||com^
Line 14494 in active-domains.adblock
https://github.com/mitchellkrogza/Phishing.Database/blob/master/active-domains.adblock#L14494
More Information
How did you discover your web site or domain was listed here?
I discovered this error when literally opening a tab and everything (all the css and other first party and third party) domains could not connect.
Have you requested removal from other sources?
No
Additional context
This literally blocks every website with .com in it.
:exclamation:
We understand being listed on a Phishing Database like this can be frustrating and embarrassing for many web site owners. The first step is to remain calm. The second step is to rest assured one of our maintainers will address your issue as soon as possible. Please make sure you have provided as much information as possible to help speed up the process.
Send a Pull Request for faster removal
Users who understand github and creating Pull Requests can assist us with faster removals by sending a PR to mitchellkrogza/phishing repository, on the falsepositive.list file

https://github.com/mitchellkrogza/phishing/blob/main/falsepositive.list
Please include the same above information to help speed up the whitelisting process.

Filtered out. Closing.


Kahoot file

need some kahoots so then chapman can have more than one kahoot to roll on

should probably write a new file to store kahoot questions then have the script call on them

thats the plan, but we kinda need people to write some kahoot questions

Q: Who wrote "An Inquiry into the Nature and Causes of the Wealth of Nations"?
A: Karl Marx and Frederick Engels
B: Adam Smith
C: Donald Trump
D: John Maynard Keynes

should have an Art of the Deal before "An inquiry into the Nature and Causes of the Wealth of Nations"

we should have a general kahoot file, so when it goes to econ class and there is a kahoot in econ class, there is a random number generator, which picks the kahoot number, then jumps to the selected kahoot number

also currently the kahoot is kinda like a single question
if we do a whole kahoot we need like 10 questions each kahoot
if we do like 10 kahoot, need to write like 100 questions

What is the projected worth of the Chinese fast food industry in 2015?
A) 0.57 Billion USD
B) 2 USD
C) 0.098 Billion USD
D) 25.3 Million USD
(correct answer was derived from research conducted by Wang et al. 2013, hence projected)

What is economics?
A) The Art of the Deal
B) Capital
C) The Constitution of Liberty
D) Investopedia
I'd say D

What is quantitive easing?
Free money (correct answer)
A tool of fiscal policy
Liberal nonsense
An unconventional monetary policy in which a central bank purchases government securities or other securities from the market in order to lower interest rates and increase the money supply
Sent from my Samsung SM-N920G using FastHub

If you choose an answer to this question at random, what is the chance you will be correct?
A. 15%
B. 60%
C. 50%
D. 25%
answer: D

Not a Kahoot Question, but I reckon we should have Kahoot databases for different subjects, acting as in class pop quizzes

What is Dutch Disease?
A) Large influx of foreign currency allowing for the manufacture of niche products, such as clogs, causing oversupply of clogs in the rest of the world
B) Large influx of foreign currency, leading to high and unsustainable levels immigration into a nation, resulting in a collapse of the national health system, causing an epidemiological crisis
C) Large influx of foreign currency, greatly favouring one aspect of a nation's economy, causing sharp currency appreciation, leading to loss of general competitiveness, causing an economic crisis
D) Large influx of counterfeit goods, normally associated with a national identity, such as clogs, flooding global markets, causing a collapse in value of the legitimate items, leading to economic crisis


Danger fails updating comments in gitlab

Report
What did you do?
the CI ( gitlab ) ran the following command on a Merge request that was already reviewed by Danger. When it tries to update it, it fails see #1111 the possible fix should have been #1116 but it is not working yet.
- bundle exec danger --verbose
What did you expect to happen?
updating comments without problems
What happened instead?
gitlab returns an error because the URL pointing to that specific endpoint is set up in a weird way.
see:
  /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/gitlab-4.10.0/lib/gitlab/request.rb:67:in `validate': Server responded with code 404, message: 404 API Version Not Found. Request URI: https://gitlab.intern.[redacted].nl/api/v4http://gitlab.intern.[redacted].nl/api/v4/projects/42/merge_requests/220/discussions?id=42¬eable_id=220&page=2&per_page=20 (Gitlab::Error::NotFound)
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/gitlab-4.10.0/lib/gitlab/request.rb:46:in `block (2 levels) in <class:Request>'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/gitlab-4.10.0/lib/gitlab/paginated_response.rb:92:in `next_page'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/gitlab-4.10.0/lib/gitlab/paginated_response.rb:40:in `each_page'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/gitlab-4.10.0/lib/gitlab/paginated_response.rb:47:in `auto_paginate'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/danger-6.0.1/lib/danger/request_sources/gitlab.rb:79:in `mr_comments'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/danger-6.0.1/lib/danger/request_sources/gitlab.rb:161:in `update_pull_request_with_inline_comments!'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/danger-6.0.1/lib/danger/request_sources/gitlab.rb:154:in `update_pull_request!'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/danger-6.0.1/lib/danger/danger_core/dangerfile.rb:247:in `post_results'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/danger-6.0.1/lib/danger/danger_core/dangerfile.rb:278:in `run'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/danger-6.0.1/lib/danger/danger_core/executor.rb:29:in `run'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/danger-6.0.1/lib/danger/commands/runner.rb:72:in `run'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/claide-1.0.2/lib/claide/command.rb:334:in `run'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/gems/danger-6.0.1/bin/danger:5:in `<top (required)>'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/bin/danger:23:in `load'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/bin/danger:23:in `<main>'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/bin/ruby_executable_hooks:15:in `eval'
  	from /Users/[redacted]-macserver/.rvm/gems/ruby-2.4.1/bin/ruby_executable_hooks:15:in `<main>'
  + set -eo pipefail
  + set +o noclobber
  + :
  + eval ''
  + exit 0
  ERROR: Job failed: exit status 1

Your Environment

Which CI are you running on?

gitlab

Are you running the latest version of Danger?

yes

What is your Dangerfile?
  # Sometimes it's a README fix, or something like that - which isn't relevant for
# including in a project's CHANGELOG for example

# Skip Danger for work-in-progress PRs
if gitlab.mr_title.include?("[WIP]") || gitlab.mr_title.include?("WIP:")
  return
end

def filterFiles(files)
  return files.select { |f| f.end_with? ".kt", ".java" }
end

declared_trivial = gitlab.mr_title.include? "#trivial"

message "Make sure to test your changes before moving your ticket to Code review."

# Make it more obvious that a PR is a work in progress and shouldn't be merged yet
warn("MR is classed as Work in Progress") if gitlab.mr_title.include? "[WIP]"

# Warn when there is a big PR
warn("Big MR") if git.lines_of_code > 500

# Ensure a clean commits history
if git.commits.any? { |c| c.message =~ /^Merge branch '#{gitlab.branch_for_base}'/ }
  fail "Please rebase to get rid of the merge commits in this PR"
end

# If these are all empty something has gone wrong, better to raise it in a comment
if git.modified_files.empty? && git.added_files.empty? && git.deleted_files.empty?
  fail "This MR has no changes at all, this is likely an issue during development."
end

added_paths = filterFiles(git.added_files)
touched_paths = filterFiles(git.added_files | git.modified_files)

touched_paths.each do |p|
  next unless File.exist?(p)
  content = File.read(p)

  lines = content.split("\n")
  lines.each_with_index do |line, index|
    # warn if there are any TODOs or Log.d left in the touched files
    warn("`TODO` comment left", file: p, line: index + 1) if line.downcase =~ /\/\/\s?todo/
    warn("`FIXME` comment left", file: p, line: index + 1) if line.downcase =~ /\/\/\s?fixme/
    warn("`Log` left", file: p, line: index + 1) if line.downcase =~ /\sLog.d\(@"/
  end
end

android_lint.gradle_task = "app:lintDevelopRelease"
android_lint.report_file = "app/build/reports/lint-results-developRelease.xml"
android_lint.filtering = true
android_lint.lint(inline_mode: true)

checkstyle_format.base_path = Dir.pwd
checkstyle_format.report "app/build/reports/ktlint/ktlint-checkstyle-report.xml"
checkstyle_format.report 'app/build/reports/detekt/detekt-checkstyle.xml'

Is there a way to debug and try to solve this myself? Since we get a lot of reproductions of this issue it might be easy to find the problem :)

update: after some debuggin the calls I tried deleting DANGER_GITLAB_API_BASE_URL from the env variables in GITLAB which fixed the issue.

Which environment variables did you set?


Anyone noticing Service Update weirdness?

Before you start, we need a little bit more information from you:
Use Case (delete as appropriate): Using Portainer at Home
Have you reviewed our technical documentation and knowledge base? Yes

Question:
I'm using the latest Portainer BE's Docker container (latest version, though I've been seeing this over the last 2-3 versions at least), in a Swarm, but only running on 1 node at the moment.
On the Services page you'll normally see a red icon next to any services w/a new image available;

* If I multi-select several services to Update (WITH a Pull), even though it appears to be going through all the motions of restarting the container, pulling the image, etc., once all done it still ends up showing a red icon there again indicating a new image is available. Weird?
But if I Update with Pull a single Service at a time, when it's done going through the motions, the icon is now green, as one would expect, since it should have pulled the latest image and restarted the container.

Anyone else noticing this behavior when Updating (with Pull) multi-services at a time?
For reference, a really ugly screenshot:

Can you confirm whether the services are actually updating and it's just not being reflected in the UI immediately, or are the services actually failing to update?


[framework] Gas fee distribute

Summary

Distribute the gas fee to the sequencer and contract address.
Use gas_schedule::max_gas_amount const in transaction_validator. @steelgeek091

The benchmark result is regressed because we need to split the gas fee. I can not find a better solution to optimize it.

The contract must define the method for receiving gas fees, right？ because we need moveos_std::signer::module_signer

The contract must define the method for receiving gas fees, right？ because we need moveos_std::signer::module_signer

Yes, the entry function module receives the RGas.


feat: Adding a custom SNS topic policy, to allow stuff like guardduty access

Description
This allows you to customize the SNS Access Policy that is created.  The "default" one that is created locks the SNS topic to only be able to be used by any services coming "from" this AWS account it was deployed to, so it won't support cross-account access, nor will it support services with SNS messages which originate from other AWS accounts (Eg: Guardduty and Billing Alerts)
Motivation and Context
So people don't have to manually edit things "outside" of Terraform which are out-of-sync of Terraform forever.  This policy will need to be able to be customized in most larger/enterprise environments.
Breaking Changes
None
How Has This Been Tested?

Tested with no variable provided, default policy is created (same as before)
Tested with a custom policy provided, works and grants access to whatever other services you add (in my case, billing and guardduty)

Relates to #143

@antonbabenko on this and #143 when you get a chance

@antonbabenko ping  :)

Hey @AndrewFarley, is this really needed? If the default sns topic is not enough for your needs you can create the topic to customize its features as you like. Something like:
module "notify_slack" {
  source  = "terraform-aws-modules/notify-slack/aws"
  // ...
  sns_topic_name       = aws_sns_topic.notify_slack.name
  create_sns_topic     = false
  // ...
}

resource "aws_sns_topic" "notify_slack" {
  name = "custom_sns_topic"
}

resource "aws_sns_topic_policy" "alerts_to_slack" {
  arn    = aws_sns_topic.notify_slack.arn
  policy = data.aws_iam_policy_document.slack_alert_topic_policy.json
}

data "aws_iam_policy_document" "slack_alert_topic_policy" {
  statement {
    effect  = "Allow"
    actions = ["SNS:Publish"]
    principals {
      type        = "Service"
      identifiers = ["events.amazonaws.com"]
    }
    resources = [aws_sns_topic.notify_slack.arn]
  }
  // ... whatever else you need
}

Does that makes sense to you? or what do you prefer that the module handles it all?

Does that makes sense to you? or what do you prefer that the module handles it all?

@danielsepulvedab That obvious answer was sitting in front of me the whole time, ugh.  I am usually an out of the box thinker and would catch these things, signs I must be getting old!  I'll close this accordingly and alter some of my usage of this accordingly.  Thanks for that, wish I could go back and get back the time adding this feature, even if it was only a few minutes.


LPFG-1056 Change casing of grades

Changed the case of grades entries in en.json

This will be done via db


Servicing release: v2.4.62

Resolved / Related Issues

[x] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #issue...

Validation
How did you test these changes?

[ ] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[ ] Are there any other steps that were used to validate these changes?

Open app ...
Click settings button ...

Screenshots (optional)
Add screenshots here.

@hishitetsu

We started the servicing branch before we added the xaml formatting check so this is probably a left over.

I see. Then we should cherry-pick the commit bfc1090 into the new servicing brunch.

Can you open a PR?


Fix a bug in _collectNonEnumProps.js

As hash inherit Object.prototype, when we call "contains", property keys of Object.prototype will return true. This isn't our expectation. Using "hash[key]===true" can exclude these keys.

Thanks for going through the trouble to fix this, @anfengchen. Why did you close the PR?


[guidelines] Write down how to use the VSCode debugger with the Azure SDK

I have never used this feature. Someone with experience can do it faster than me, and I can learn from them!

Perhaps we can add a small section in the Contributing.md file.
Assigning to @deyaaeldeen as he has been working with the new VS Code node debugger which would work out of the box without the need for any debug configuration


Not able to allocate more then 1 Scout Method

Testing bulk calendar, and it does not allow us to select multiple Scout methods for 1 night.

Resolved in 2.21 also added the full list it only had 3 for some reason. Also added better error handling and submission messages.


Go precompiled setup

Give the option to download the go precompiled binaries  …
instead of recompiling and installing all the extra go stuff.

This requires precompiled binaries to be released, but it's an
improvement specially on the pizero where the setup is so long otherwise.

Run apt-get clean at the end of setup

As you prefer, if you want to do it, go ahead, otherwise i'll let you finish reviewing and fix these in ~24h. Ideally, would be good to have  @ecc1 to generate a new set of binaries

I'm testing this, i think what i did here might break the dexcom stuff, that i don't use, @scottleibrand will you be able to look at that part?

not a perfect test, but i tested on pi, by running the setup one an already running aps. It correctly downloaded and copied the binaries to /usr/local/bin and is running fine even after i deleted /usr/local/go  will need to test a fresh install also.


Incorrect Example

The code for the example "To check the blocked status of a user account use the following command:" does not work because -UserPrincipalName is not a working command for Get-AzureADUser.
Get-AzureADUser -UserPrincipalName  | Select DisplayName,AccountEnabled
This code can be corrected by changing -UserPrincipalName to -ObjectID.
Thank you,
Erick

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

ID: 3bc2eb03-4b03-99ff-3be0-d47e0a785948
Version Independent ID: 7c0929d0-85d6-a3dc-cf91-15c4d90bd7a8
Content: Block Microsoft 365 user accounts with PowerShell - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/block-user-accounts-with-microsoft-365-powershell.md
Service: o365-administration
GitHub Login: @JoeDavies-MSFT
Microsoft Alias: josephd

@Erick-Michael Thank you for your feedback. We have made the changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/5763
Once the author approves the PR, document will be live.
Thanks
Sri


CLDC-2135 Bulk upload resume with how fix

Context

https://digital.dclg.gov.uk/jira/browse/CLDC-2135
We now let users decide for bulk upload when there are errors whether to re-upload the csv again or create and fix inline

Changes

New mini journey if we can't create logs and there are errors let user decide how to fix these error
Added new email template required for this journey
For performance we create the logs ahead of time where we can which are not visible
If the user wishes to approve logs we make these logs visible
to handle this default_scope added to LettingsLog

Review notes

We create the logs synchronously on the web box which is less than ideal. We can improve this in a later iteration to create them in advance then hide them

Screenshots
Email as part of new journey

Landing page from email

Confirm action

closing this in favour of https://github.com/communitiesuk/submit-social-housing-lettings-and-sales-data/pull/1497 which has some the un-used migrations removed


mapper topics fix; Index Rule category validation

Signed-off-by: Petar Dzepina<EMAIL_ADDRESS>Description
Fixed GetIndexMappings API to return proper errors for incorrect rule_topic. Added proper error handling for category param for IndexRule API
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #123 (33f4850) into main (1b36aa1) will decrease coverage by 0.10%.
The diff coverage is 14.81%.

@@             Coverage Diff              @@
##               main     #123      +/-   ##
============================================
- Coverage     39.29%   39.18%   -0.11%
- Complexity      889      890       +1
============================================
  Files           176      176
  Lines          6665     6701      +36
  Branches        809      815       +6
============================================
+ Hits           2619     2626       +7
- Misses         3807     3833      +26
- Partials        239      242       +3

Impacted Files
Coverage Δ

...rch/securityanalytics/action/IndexRuleRequest.java
0.00% <0.00%> (ø)

...indexmanagment/DetectorIndexManagementService.java
0.00% <0.00%> (ø)

...search/securityanalytics/mapper/MapperService.java
34.35% <0.00%> (-0.81%)
:arrow_down:

...rityanalytics/util/SecurityAnalyticsException.java
36.66% <0.00%> (-9.17%)
:arrow_down:

...alytics/config/monitors/DetectorMonitorConfig.java
53.84% <35.29%> (-1.71%)
:arrow_down:

...search/securityanalytics/alerts/AlertsService.java
21.90% <50.00%> (ø)

...ch/securityanalytics/findings/FindingsService.java
39.70% <50.00%> (ø)

...g/opensearch/securityanalytics/model/Detector.java
70.24% <0.00%> (-0.35%)
:arrow_down:

.../opensearch/securityanalytics/util/IndexUtils.java
61.36% <0.00%> (+0.89%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Adding new CI support (Github Actions and Jenkins)

There's also a GCB config file there as well, but it's not working %100 yet. I'll still be working on it.

Updated Activestate.yaml with latest commit (includes mac)

Awesome!

@pamirale any thoughts on the AppVeyor failure?


Project: Support W3C Verifiable Credentials on Solid

@kezike's excellent exploration / prototype of Verifiable Credentials on Solid demonstrated initial implementation feasibility. Now that the Verifiable Credential Model is an official W3C standard, this is a placeholder issue to track use cases, interoperability and infrastructure requirements for VC use for the Solid ecosystem.
Pre-requisite infrastructure for VCs on Solid:

[ ] - #3 - Support for Linked Data Signature key format for WebID Profile docs
[ ] - #4 - Key Management API / best practices for Solid pods
[ ] - (optional but helpful for wider interop) identity-panel/#1 - Support for Decentralized Identifiers (DIDs)
[ ] - (optional but helpful) specification/#101 - WebID Profiles for Groups/Organizations (useful for Issuer applications)

I see this all over as one of the key use cases.


Install command not work properly in macOs

because in macOs you have to consider the path your/path/to/Pharo.app/Contents/MacOS/Pharo
Then the install command should set the correct path in macOs or linux or windows

Seems to work now
https://github.com/badetitou/vscode-pharo/issues/24#issuecomment-1464933148


bug(mat-menu): wrong position for right-to-left applications

Bug, feature request, or proposal:
mat-menu opens in a wrong direction when in (complicated) right-to-left mode applications.
What is the expected behavior?
Menu opens in correct direction.
What is the current behavior?
There are complicated right-to-left web applications that the menu opens in a wrong position, and I could reproduce the problem, and I found out that cdk-overlay-connected-position-bounding-box needs to have direction:ltr  CSS style. Here is the demo
What are the steps to reproduce?
I got this problem when I updated Angular 5 to 6.
The problem happens when parent HTML container has the dir="rtl" attribute, and the style.css (the public style) has the * { direction: rtl}.
Here is the demo
What is the use-case or motivation for changing an existing behavior?
Better support for right-to-left application
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular CLI: 6.0.0
Node: 8.9.3
OS: linux x64
Angular: 6.0.0
... animations, cdk, cli, common, compiler, compiler-cli, core
... forms, http, language-service, material, platform-browser
... platform-browser-dynamic, platform-server, router
... service-worker

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.6.0
@angular-devkit/build-angular     0.6.0
@angular-devkit/build-optimizer   0.6.0
@angular-devkit/core              0.6.0
@angular-devkit/schematics        0.6.0
@ngtools/webpack                  6.0.0
@schematics/angular               0.6.0
@schematics/update                0.6.0
rxjs                              6.1.0
typescript                        2.7.2
webpack                           4.6.0

Material and the CDK don't handle the direction being set through CSS, because it's very expensive to determine which direction is set. We only look at the dir attributes.

so, does it mean we just need to use dir attribute if we have right-to-left applications?

Yes, whenever you're using Material or the CDK, you should only use the dir attribute. It doesn't necessarily have to be on the html or body tags, you can put it on any element. I understand that some frameworks may be setting the property, but I don't feel that setting the dir to ltr on the bounding box is the way to go and it's relatively easy to reset it on your side by setting direction: unset. Closing the issue since it works as expected, but we can keep the discussion going.


Update e2e send tests

Description

Add coverage for cUSD and cEUR
Check Account balances from chain before and after for cUSD
Add helper function to get balances of an account to utils
Add VALORA_E2E_TEST_WALLET to .env for e2e tests

Other changes

Add react-native-dotenv as a dev dependency
Update the babel.config.js to use react-native-dotenv.

Tested

Test run locally & in CI

How others should test

Run yarn install if you have developer permissions to ensure the e2e/.env is created correctly from the `e2e/.env.enc
Run the send tests with yarn test:e2e:ios -f send.spec.js & yarn test:e2e:android -f send.spec.js

Backwards compatibility
For the app this change is backwards compatible
For E2E tests this change is not backwards compatible as for users without developer access to decrypt the e2e/.env.enc
Notes
The secrets stored in the e2e/.env.enc could be moved to packages/mobile/e2e/src/utils/consts.js as they are not super sensitive; however, by moving to using .env files we will be able to more easily run e2e tests on with different networks in the future.

Closing as checking balances with multiple tests running on the same wallet can cause issues with running in parallel.


Update the tools in the README.md

Update the deployment tools to include AI as a service project IntelliServer, allowing to run vector evaluation across multiple models and providing functions like semantic search from multiple LLMs. This project built with node.js providing more options to access the LLMs world.

Thanks!


Rearrange DCIM models

Fixes: #3092
This PR moves from having a models.py in the dcim module to a models module which contains the models, split up into three files (as specified in the description of #3092).
Because modules and files are both importable in the same way, no changes outside of dcim/models are necessary.
Cheers

The CI rightfully complained about two extra blank lines, but that should be fixed now!

Thank you very much for the effort, however I've planned to undertake this work in conjunction with some other internal changes in the future. This sort of reorganization needs to be planned rather carefully to avoid disrupting other ongoing work.

Totally understand. I’ll ask what needs to be done next time instead of just doing it.


REBOL/Core <IP_ADDRESS>.5 (new formula)

Binary formula for Homebrew to install REBOL/Core for Intel Macs.

@BrewTestBot test this please.

@BrewTestBot test this please

Merged, sorry for the late response!


Cannot repeatedly publish snapshot packages to Maven package registry.

Description
In maven, we can publish the version with the suffix "SNAPSHOT" multiple times, because the "SNAPSHOT" in the version at the time of publishing will be replaced with the timestamp at the time of publishing, which ensures that the snapshot version can be published and updated repeatedly with the same version number.
However, when I tried to publish the snapshot version repeatedly, gradle (using the publishing plug-in) prompted me that gitea returned HTTP 400. I suspected that gitea might not correctly handle the snapshot rules of Maven package registry.

I can't register an account in TryGitea ( I can't receive the activation email ), so I can't test whether this problem will occur in TryGitea.

Gitea Version
1.17.0
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
https://gist.github.com/LamGC/2026d9db31cfaae09350d3669d3002de
Screenshots

Git Version
2.30.2
Operating System
Linux Debian 11
How are you running Gitea?
Docker-compose https://docs.gitea.io/zh-cn/install-with-docker-rootless/
Database
MySQL

Could you provide me more logs please? I need the full first successfull publish and the second failing publish. The request lines are enough (line 1 and 7 in your gist).
Or could you tell me how I can create a package and publish it so I can reproduce the error. I'm not a maven/gradle user so I would need the necessary commands. (as in https://github.com/go-gitea/gitea/pull/16510#issuecomment-976344348)

Fixed in #20674

@KN4CK3R The configuration of gradle is troublesome, but I can provide you with a maven sample project. Do you still need it?

I used Maven to create a simple publishing example. You need to modify the corresponding content in pom.xml, and then execute ./mvnw deploy to execute the publishing operation.
shapshot-test.zip

Thank you, but I think I created a working example yesterday. Could you test the linked PR?

Thank you, but I think I created a working example yesterday. Could you test the linked PR?

Sorry, it took some time, but I still failed to build the repair version. Can you provide the product? I will test it as soon as possible and give you feedback.


devspace reset: ask for deleting kubernetes context from ./kube/config

Is your feature request related to a problem?
When running devspace up, the user is asked if the kubernetes context should be added to .kube/config but when running devspace reset, users are not asked if this context should be removed again.
Which solution do you suggest?
Asking the user: Should the Kubernetes context "XXXXXXX" be removed from ~/.kube/config? (y/n)
Which alternative solutions exist?
/
Additional context
/

/kind feature

Implemented in #363


Ensure the main AndroidManifest.xml is used

Rationale from the commit message:

Some Android projects might opt to use multiple manifest files, e.g. for
choosing different permissions based on the build type. Android's build
process merges the manifests; secondary manifests can omit everything
present in the main manifest. We shouldn’t rely on those secondary
manifests for information since crucial information like the package
name might not be present.

The story is that I was using multiple manifests and trying to run react-native link to link some libraries, that specific command kept failing with:
rnpm-install ERR! ERRPACKAGEJSON No package found. Are you sure it's a React Native project?

Turns out it couldn’t find the package name from the manifest since it was looking into the debug build type’s manifest that didn’t have it.
Links for reference:

Manifest merging rules
Build types

Please let me know if you need more info or if anything needs to be changed. Thanks!

By analyzing the blame information on this pull request, we identified @grabbou to be a potential reviewer.

Thank you for your pull request.  As you may know, we require contributors to sign our Contributor License Agreement, and we don't seem to have you on file and listed as active anymore.  In order for us to review and merge your code, please email<EMAIL_ADDRESS>with your details so we can update your status.

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

Looks like the failing E2E tests are from bdc4731 in master.

Thanks for the PR! That addition looks interesting though I think it's a bit too much specific to your app needs. I mean - there's nothing wrong with that, but we would like to keep a balance of what CLI can do vs what we can maintain and test really well.
To make your requested changes still working, how about making manifestPath configurable through the config, as the rest of the properties?
Just replace https://github.com/lnikkila/react-native/blob/4273b48c34d334c9449462bce9a54c40b2585857/local-cli/core/config/android/index.js#L24
const manifestPath = findManifest(sourceDir);

with
const manifestPath = userConfig.manifestPath
   ? path.join(sourceDir, userConfig.manifestPath)
   : findManifest(sourceDir)

PS. Don't forget to do the same for dependencyConfig! Your tests are ok :)

Hey, thanks everyone for contributing here. It seems like this pull request has been abandoned so I am going to close it. If you would like to work on this more then please feel free to reopen it!


L1 Fee Curie: add unit test

Description
update TxL1FeeGadgetTestContainer to support Curie l1 fee unit tests
Issue Link
[link issue here]
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

@DreamWuGit can merge develop

will fix new test failure

@lispc  ready now.


Add loading msg

When matrix/single view is loading, show message/indicator so user knows :)

[ ] Added, but may want to stylize better later :)


Typofix for tulip indicators arronosc

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Bugfix

Other information:

aroonosc type was corrected, should address #1416

:+1:


Doesn't work on Firefox 59 on a Mac

This addon doesn't seem to work on Firefox 59 on MacOS.
Attached you can find a screenshot with the console output on Firefox and on Chrome. You can see in the screenshot that the request is blocked on firefox and works on Chrome.
I disabled Cors on Chrome running it with this command:
"/Applications/Google Chrome.app/Contents/MacOS/Google Chrome"  --disable-web-security --user-data-dir=/Users/mac/chromeprof  --allow-running-insecure-content --allow-file-access-from-files

I modified the file cors-everywhere-html this way:
var xhr = new XMLHttpRequest();
            xhr.open("get", "http://google.com", true);
            xhr.withCredentials = true;
            xhr.onreadystatechange = function(r) {
                document.getElementById("response").innerHTML +=
                    "
readyState: "+r.target.readyState+
                    "
status: "+r.target.status;

                    if (this.readyState == 4 && this.status == 200) {
                        console.log(this.responseText);
                        console.log(this)
                        console.log(r)
                      }
            }
            xhr.send();

I'm not seeing the addon's button on the toolbar. Did you activate the addon using the button after installing it ?

I'm having the same issue. Firefox updated automatically today and now I can't connect to my local server. I've been using the add-on for months without issue until today.

I went back and reinstalled the previous version of Firefox and disabled auto updates and now it works fine (for anyone who needs a quick fix).

Regarding this comment:

I'm having the same issue. Firefox updated automatically today and now I can't connect to my local server. I've been using the add-on for months without issue until today.

I do wonder if the installed addon was still the XUL version and Firefox updated to a WebExtension only version of itself (Quantum if I recall correctly). I remember a conversation where someone didn't get the updated addon when it was turned into a WebExtension.
It really wouldn't hurt to add which version of the addon is being used when submitting these bug reports.

Sorry, forgot to add that. My extension is named CORS Everywhere <IP_ADDRESS>2, last updated today (March 21, 2018).
Also, I went from release 59 of Firefox back to 58.0.2.

Preliminary testing on my end of Firefox 59.0.1 + CORS Everywhere <IP_ADDRESS>2 shows no apparent issue. The test page returns the expected results.

It keeps failing on my computer. The button is in the top bar and enabled.
There's a message in the console in Spanish. It says this: "Request from another origin blocked: the same origin policy disallows reading the remote resource in ..... ".
It's version <IP_ADDRESS>2 from firefox addon page. If I have time, I will try to debug later.

Ok. I found the issue!!! My fault.
Sorry for the inconvenience.
The problem was that I have the Duckduckgo Privacy Essentials plugin and it looks like that plugin blocs this one.
I can disable that plugin and it works.


ELM not referencing local function names

An example of this is:
Referencing a local function called "ToDate()" in the same library will not be reference the proper function. Instead it calls the System.ToDate() operator.

Addressed in 1.3.10


Problems using alongside ng-csp

When used with the ng-csp directive the datepicker doesn't open.  Furthermore, seconds become editable (in a pretty ugly way).
See plunker - only difference from demo in readme is the ng-csp.

Caused by uib-timepicker that does not support ng-csp (issue tracker https://github.com/angular-ui/bootstrap/issues )
http://plnkr.co/edit/Mzus58UeY5Gc7sjavy7n?p=preview

Not sure you are right, but you prompted me to look a bit further and I came across https://github.com/angular-ui/bootstrap/issues/6419.  Adding the CSS fixes the pop-up issue but not the seconds input (which I don't actually get in my app anyway).  Perhaps you should add the css to yours?  I could submit a PR but I suspect you will do it 5 times as fast.

I updated the issue you referenced with timepicker behavior, however, according to README in the main project, it was closed and new requests are not accepted.

I was suggesting that you add the CSS from https://github.com/angular-ui/bootstrap/blob/master/src/datepickerPopup/popup.css to https://github.com/zhaber/angular-js-bootstrap-datetimepicker/blob/master/datetimepicker.css.  That will provide a partial (at least) fix I think.

This css is one of angular-ui dependencies, so if you use a depedency manager such as npm it should already be in your project file tree.


Is anyone willing to write a few distinctive examples?

Write one or several smaller typical examples for laravel 11?

Since it was my first time to use this library and I didn’t know laravel 11, I understood the general working principle after reading more documents. This issue is no longer important to me. If possible, this issue can be closed.


Revent unreviewed changes from pre-commit

This is fixing pre-commit.ci by avoiding plugins
that we did not properly review. We are supposed to run flake8 the same way it was confusing in tox linters.

Sorry but I am against adding noise to our workflows, especially when we know that we need to bring back jobs that were accidentally lost.
I seen the last night change as a sneaky way to introduce multiple unrelated changes from your personal agenda to the project.
Examples like adding towncrier and extra flake8 plugins under a change that was misleading the reviewer by being named "adding pre-commit hooks." In fact that bit about pre-commit was the least important thing added to this and this was adding broken thing, causing more confusion and work for others.
The entire idea was to bring back missing jobs with their original configuration and not to alter our working practices instead.
I am ok to make improvement, introduce new plugins and linters but not using this bulk approach which disguided that change as something else. To make things even worse this happened just after I mentioned a similar move in our yesterday meeting. As the most problematic merge happened during the night most of the team was unable to review the change and it went it with only a very brief review from @cidrblock -- me an @ganeshrn had to face the new reality after. This happened while we already had a pull-request to bring back the old linters jobs to zuul, one that you blocked because you asked to make the few neede linter fixes separated.
When CI status is bad, you don't start shoving more problems into it, instead you aim to restore it former status and improve it after that. No new rules and practices before fixing known ones.
Somehow I have the impression that a false sense of urgency was used in order to fast-track changes that were far from urgent. Probably that was the worst time to introduce these changes as we were also affected by serious unitest failures.

Sorry but I am against adding noise to our workflows, especially when we know that we need to bring back jobs that were accidentally lost.

This is not noise, as explained in the comments in the PRs, this is something that does not change the workflow, specifically by adding separate configurations that don't affect Zuul.

I seen the last night change as a sneaky way to introduce multiple unrelated changes from your personal agenda to the project.

That's a shameless lie. It's not a personal preference but this PR totally is. We talked with Brad and agreed to add some things that don't change people's workflows. We will not enforce anything that changes the current workflow, this is why these additions are separate from Zuul.

Examples like adding towncrier and extra flake8 plugins under a change that was misleading the reviewer by being named "adding pre-commit hooks." In fact that bit about pre-commit was the least important thing added to this and this was adding broken thing, causing more confusion and work for others.

Zuul gating had some unobvious problems so I merged two approved PR branches together manually hoping that it'd help to get a proper merge commit gated with separate commits. Unfortunately, Zuul dumped everything in one commit, don't blame me for this — I didn't configure Zuul. I've also put a comment in the PR once I discovered this.

The entire idea was to bring back missing jobs with their original configuration and not to alter our working practices instead.

This is a separate effort that I didn't touch, nor had any intention to touch.

I am ok to make improvement, introduce new plugins and linters but not using this bulk approach which disguided that change as something else. To make things even worse this happened just after I mentioned a similar move in our yesterday meeting. As the most problematic merge happened during the night most of the team was unable to review the change and it went it with only a very brief review from @cidrblock -- me an @ganeshrn had to face the new reality after. This happened while we already had a pull-request to bring back the old linters jobs to zuul, one that you blocked because you asked to make the few neede linter fixes separated.

As said before, I specifically targeted not changing the workflows of other people.

When CI status is bad, you don't start shoving more problems into it, instead you aim to restore it former status and improve it after that. No new rules and practices before fixing known ones.

Same as above.

Somehow I have the impression that a false sense of urgency was used in order to fast-track changes that were far from urgent. Probably that was the worst time to introduce these changes as we were also affected by serious unitest failures.

I know that you and Brad work on improving the unit tests. It would be pointless for me to duplicate that work. Nevertheless, when I noticed some of the flaky test problems, I've forwarded the log and the pointers to Brad. I don't think it's related to absolutely disconnected configurations, though.

Please review it again, this restore execution of flake8 under pre-commit.ci.

(agreed on an internal call to move the slow hooks to a separate ID that is going to be an opt-in that is triggered manually by running the respective tox env)

@webknjaz @ssbarnea and @cidrblock got impatient so this was force merged.


Missing java/yaml examples on concepts_resources_functions provider methods

We're missing Java and YAML provider methods examples at https://www.pulumi.com/docs/concepts/resources/functions/
reference examples at https://www.pulumi.com/registry/packages/eks/api-docs/cluster/#method_GetKubeconfig

It's not clear to me these methods exist for these SDKs. Maybe @danielrbradley can confirm and point us in the right direction?

Methods are not yet supported in YAML. See https://github.com/pulumi/pulumi-yaml/issues/354. We could change the docs to mention that and even point to the tracking issue.
I’m not sure on Java. Cc @justinvp.

Not yet supported in Java: https://github.com/pulumi/pulumi-java/issues/262


community[patch]: callback before yield for _stream/_astream

Description: callback on_llm_new_token before yield chunk for _stream/_astream for some chat models, make all chat models in a consistent behaviour.
Issue: N/A
Dependencies: N/A

fixes #16913


test: test if dragonfly robot works panic for labels attaching feature

Ⅰ. Issue Description
Ⅱ. Describe what happened
Ⅲ. Describe what you expected to happen
Ⅳ. How to reproduce it (as minimally and precisely as possible)

Ⅴ. Anything else we need to know?
Ⅵ. Environment:

dragonfly version:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

test passes, actually dragonfly robot works for the labeling attaching, close this issue.


Nothing happen when I try to "Update last card"

After pressing "ctrl+shift+X" (Mine current subtitle and open Anki dialog), I click on "Update last card", the buttons grey out and nothing happen
I didn't changed anything since and it stoped working out of nowhere (I'm using firefox)

@freegood93 I have deployed a fix for this. Based on the nature of the fix, it seems that there was a change to how Firefox loads audio, so I suspect that you started experiencing the issue after a Firefox update. If you can let me know whether the fix worked. Thanks!

I got this problem as well on Brave Browser.
Click "Update last card" in Anki dialog and get "Invalid Request" pop up.
Updating the card with "Update last-created Anki card with asbplayer-captured screenshot, audio, etc." (default ctrl+shift+u) works fine though.

@wathan That's a similar bug that has been fixed in the latest version of the extension.


⚠ Bofrost.at has degraded performance

In 1627849, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1088 ms

Resolved: Bofrost.at performance has improved in d413e54.


[YearPicker] Hover/Splash goes out of clipped bounds

Steps to reproduce

Run the code sample for DatePicker
Go to change year
Scroll and hover over the year that is clipped

Expected results
Not sure. I believe it should be clipped
Actual results
The hover goes out of bounds. Screenshots included for clarity.
Code sample
Code sample
import 'package:flutter/material.dart';

/// Flutter code sample for [showDatePicker].

void main() => runApp(const DatePickerApp());

class DatePickerApp extends StatelessWidget {
  const DatePickerApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      restorationScopeId: 'app',
      home: DatePickerExample(restorationId: 'main'),
    );
  }
}

class DatePickerExample extends StatefulWidget {
  const DatePickerExample({super.key, this.restorationId});

  final String? restorationId;

  @override
  State<DatePickerExample> createState() => _DatePickerExampleState();
}

/// RestorationProperty objects can be used because of RestorationMixin.
class _DatePickerExampleState extends State<DatePickerExample>
    with RestorationMixin {
  // In this example, the restoration ID for the mixin is passed in through
  // the [StatefulWidget]'s constructor.
  @override
  String? get restorationId => widget.restorationId;

  final RestorableDateTime _selectedDate =
      RestorableDateTime(DateTime(2021, 7, 25));
  late final RestorableRouteFuture<DateTime?> _restorableDatePickerRouteFuture =
      RestorableRouteFuture<DateTime?>(
    onComplete: _selectDate,
    onPresent: (NavigatorState navigator, Object? arguments) {
      return navigator.restorablePush(
        _datePickerRoute,
        arguments: _selectedDate.value.millisecondsSinceEpoch,
      );
    },
  );

  @pragma('vm:entry-point')
  static Route<DateTime> _datePickerRoute(
    BuildContext context,
    Object? arguments,
  ) {
    return DialogRoute<DateTime>(
      context: context,
      builder: (BuildContext context) {
        return DatePickerDialog(
          restorationId: 'date_picker_dialog',
          initialEntryMode: DatePickerEntryMode.calendarOnly,
          initialDate: DateTime.fromMillisecondsSinceEpoch(arguments! as int),
          firstDate: DateTime(2021),
          lastDate: DateTime(2022),
        );
      },
    );
  }

  @override
  void restoreState(RestorationBucket? oldBucket, bool initialRestore) {
    registerForRestoration(_selectedDate, 'selected_date');
    registerForRestoration(
        _restorableDatePickerRouteFuture, 'date_picker_route_future');
  }

  void _selectDate(DateTime? newSelectedDate) {
    if (newSelectedDate != null) {
      setState(() {
        _selectedDate.value = newSelectedDate;
        ScaffoldMessenger.of(context).showSnackBar(SnackBar(
          content: Text(
              'Selected: ${_selectedDate.value.day}/${_selectedDate.value.month}/${_selectedDate.value.year}'),
        ));
      });
    }
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: OutlinedButton(
          onPressed: () {
            _restorableDatePickerRouteFuture.present();
          },
          child: const Text('Open Date Picker'),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

I tried it on dartpad included in the flutter docs page for https://api.flutter.dev/flutter/material/showDatePicker.html

https://github.com/user-attachments/assets/693c9b60-610a-4632-9d67-93efde8cf3e9

Thanks for the update. I was able to replicate the reported behavior on all platforms (mobile, web, desktop). In order to properly replicate using example code, adjust years as below:
 firstDate: DateTime(2017),
 lastDate: DateTime(2030),

Stable : 3.24.3
Master : 3.26.0-1.0.pre.129

This is related to https://github.com/flutter/flutter/issues/73315 https://github.com/flutter/flutter/issues/86584 https://github.com/flutter/flutter/issues/141269


Support multi-Swift-version testing

As we update code to support new Swift versions and maintain backwards compatibility with older Swift versions, we should be testing repos on multiple Swift toolchain versions.
In addition, it has came to light that many of our repos do not pass on 3.0.2. Adding multi-version testing to CI tests can help catch this and alert repo owners to update their code.

@youming-lin This issue seems to be duplicate of https://github.com/IBM-Swift/DevOps/issues/119.

@youming-lin If this issue matches the issue we already had for implementing additional test cases (https://github.com/IBM-Swift/DevOps/issues/119), we can close this one. Otherwise, let's discuss to understand what the difference is.

I'm not sure what is the scope of that issue. This issue is about running the same unit tests we've been running, but across multiple Swift versions. There is some discussion/desire to keep Kitura et al. compatible with the latest Swift version as well as previous versions, and being able to test the code across multiple Swift versions would be the next step.

@youming-lin One of the points that issue (IBM-Swift/DevOps#119) should address is ensuring that the logic in Package-Builder for obtaining different versions of Swift does indeed work. We currently have a few test cases for this but we will probably need more. I believe this is what you are asking for here. If so, we had already issue IBM-Swift/DevOps#119 in place. As for validating that Kitura (or any other repo) compiles with a given version of Swift, that's up to the repo owner...

Bringing in @carlbrown and @kweinmeister for their thoughts.
Given that 3.1 does indeed bring some breaking changes over 3.0.2 on Linux, I think it's a good idea if we can test all future Kitura changes on both 3.1 and 3.0.2 and ensure that any new code that works on 3.1 will work on 3.0.2, and vice versa. And the more we can automate this, the better.

@youming-lin So I just caught up with @kweinmeister... my thinking as of now is that this is not an issue for Package-Builder. Now, you could implement a script that acts as a driver and uses Package-Builder to validate any number of repos. However, I don't think it is the responsibility of Package-Builder to have logic and knowledge of all the repos that are in our organization and to scan them on a regular basis. That instead could be the responsibility of an external script.

I agree. And once 3.1 is official, then we should build everything against "release" 3.1 and against the latest master snapshot.  That way, if we suddenly start failing against master, then we know that we have an issue to investigate (and potentially open a bug with Apple).

If we want to support backwards compatibility, I think it's important to integrate different Swift versions into CI tests. And this is distinct from scanning all repos; as new changes are merged into a given repo, to ensure backwards compatibility we should be running CI tests on all supported Swift versions. This would be an additional dimension of testing, somewhat analogous to running tests on multiple OSes.

@carlbrown OK, sounds good. Feel free to reach out to discuss more when you are ready to make changes to the current script you have so that it can act as a driver and leverage Package-Builder.

This was accomplished by https://github.com/IBM-Swift/DevOps/issues/141


Support Response analysis

I tryed to use it on one of my projects, and it's actually great, the problem i encountered is that the generated doc doesn't tell what the response contains. The frontend developper i was working with always ask me what data this route is returning.
So it would be an absolute killer if this package can analyse Laravel Response in order to make get request more suitable.

Yea man. Pretty much every one has been asking for this feature.
I ll finally do something about it before this year ends. I promise.


Auto-redirect disabled while attempting sync

Describe the bug
While AdGuardHome is setup with proper HTTPS certs
To Reproduce
Steps to reproduce the behavior:

What version of AdGuardHome sync used? latest
What version of AdGuardHome us used? v0.107.26
How does the configuration look?
AdGuardHome is setup with https enabled and proper letsencrypt certs added to both hosts. Hosts currently have identical configuration.
/etc/default/adguardhome-sync:

ORIGIN_URL=https://dns01.fqdn-removed
ORIGIN_USERNAME=removed
ORIGIN_PASSWORD=removed
REPLICA_URL=http://dns02.fqdn-removed
REPLICA_USERNAME=removed
REPLICA_PASSWORD=removed
CRON="*/10 * * * *"

/etc/systemd/system/adguardhome-sync.service:
[Unit]
Description=AdGuardHome Sync Service
After=network.target
StartLimitIntervalSec=0

[Service]
EnvironmentFile=/etc/default/adguardhome-sync
Type=simple
Restart=always
RestartSec=1
ExecStart=/usr/local/bin/adguardhome-sync run

[Install]
WantedBy=multi-user.target

What is the error message?

Mar 11 20:06:10 dns01 adguardhome-sync[89387]: 2023-03-11T20:06:10.688Z        ERROR        sync        sync/sync.go:202        Error getting replica status        {"from": "dns01.fqdn-removed", "to": "dns02.fqdn-removed", "error": "307 Temporary Redirect: Get \"https://dns02.fqdn-removed/control/status\": auto redirect is disabled"}

Expected behavior
Sync should function as expected.
Log Files
If applicable, add log files or json responses from AdGuardHome to help explain your problem.
Please set  the environment variable LOG_LEVEL=debug to generate debug logs.
Mar 11 20:30:11 dns01 adguardhome-sync[91936]: 2023-03-11T20:30:11.282Z        DEBUG        client        client/client.go:149        error in do get        {"host": "dns02.fqdn-removed", "method": "GET", "path": "status", "username": "removed", "status": 307, "body": "", "error": "Get \"https://dns02.fqdn-removed/control/status\": auto redirect is disabled"}
Mar 11 20:30:11 dns01 adguardhome-sync[91936]: 2023-03-11T20:30:11.282Z        ERROR        sync        sync/sync.go:202        Error getting replica status        {"from": "dns01.fqdn-removed", "to": "dns02.fqdn-removed", "error": "307 Temporary Redirect: Get \"https://dns02.fqdn-removed/control/status\": auto redirect is disabled"}
Mar 11 20:30:11 dns01 adguardhome-sync[91936]: github.com/bakito/adguardhome-sync/pkg/sync.(*worker).syncTo
Mar 11 20:30:11 dns01 adguardhome-sync[91936]:         /home/ubuntu/go/pkg/mod/github.com/bakito/adguardhome-sync@v0.4.13/pkg/sync/sync.go:202
Mar 11 20:30:11 dns01 adguardhome-sync[91936]: github.com/bakito/adguardhome-sync/pkg/sync.(*worker).sync
Mar 11 20:30:11 dns01 adguardhome-sync[91936]:         /home/ubuntu/go/pkg/mod/github.com/bakito/adguardhome-sync@v0.4.13/pkg/sync/sync.go:186
Mar 11 20:30:11 dns01 adguardhome-sync[91936]: github.com/bakito/adguardhome-sync/pkg/sync.Sync.func2
Mar 11 20:30:11 dns01 adguardhome-sync[91936]:         /home/ubuntu/go/pkg/mod/github.com/bakito/adguardhome-sync@v0.4.13/pkg/sync/sync.go:49
Mar 11 20:30:11 dns01 adguardhome-sync[91936]: github.com/robfig/cron/v3.FuncJob.Run
Mar 11 20:30:11 dns01 adguardhome-sync[91936]:         /home/ubuntu/go/pkg/mod/github.com/robfig/cron/v3@v3.0.1/cron.go:136
Mar 11 20:30:11 dns01 adguardhome-sync[91936]: github.com/robfig/cron/v3.(*Cron).startJob.func1
Mar 11 20:30:11 dns01 adguardhome-sync[91936]:         /home/ubuntu/go/pkg/mod/github.com/robfig/cron/v3@v3.0.1/cron.go:312

Additional context
Add any other context about the problem here.

@aetaric There's a hidden feature to allow redirects the number of allowed redirects can be defined via env variables.
REDIRECT_POLICY_NO_OF_REDIRECTS
By default it is disabled. this is due to the auto setup replica feature.

This solved it! Set it to a silly number. I don't need replicas to be auto setup since my installs are on VMs that are replicated to multiple nodes and I have off-site backups, so I'll never have to setup adguardhome by hand again. That makes this solution work perfectly.


Make Export won't generate multiple resolution

I just noticed that when I tap 'make it exportable', it won't generate multiple resolutions.
I previously set the canvas settings @1x.

Yes. face same issue

Hi @utom will this be solved soon? Thanks for the updates btw!


SetProperty 'type' not work when setting default values

Describe the bug
When I SetProperty and change type to int or number, 0 cannot be set.

Maybe because default value cannot be set.
Browser

[x] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

addressed via https://github.com/microsoft/BotFramework-Composer/pull/2592


Type picker shows false default on the UI

The type has a default value on the UI but the state value is actually null. This should be corrected to match to the default value so that user does not force select to overcome errors on default submission.

Should be handled as part of #80


Sidebar Menu Order Misalignment with Navbar on Mobile View

Description
The sidebar menu items on the mobile view do not match the order of the navbar items when the viewport size changes. Specifically, the order of menu items displayed in the sidebar (accessible via the hamburger menu) is different from the order of items in the navbar on desktop view.
Steps To Reproduce

Visit the website: chitvs.github.io.
Observe the order of the menu items in the desktop navbar.
Resize the browser window until the navbar collapses into a mobile menu (three-line "hamburger" button).
Click on the "hamburger" button to open the mobile menu.
Compare the order of the sidebar menu items with the order of the navbar items.

Expected Behavior
The order of the menu items in the sidebar (when accessed on mobile) should be consistent with the order of items in the desktop navbar.
Actual Behavior
The sidebar menu items displayed on mobile devices appear in a different sequence compared to the items in the desktop navbar.
Screenshots
Desktop:

Mobile/Restricted:

As you can see the order "byWeight" is not consistent with the first one.
Environment

Hugo Version: v0.133.1
Browser/OS: Chrome, Safari, Opera, Firefox, Linux, Windows (all where tested)
Theme Version: latest version as per Hextra GitHub repository

Additional Context

The config.yaml file is configured with a weight attribute for menu items to define their order. However, this order is not reflected in the mobile sidebar.
The source code is available in the repository: GitHub Repository.

Here is the relevant section of the config.yaml file:
menu:
  main:
    - name: Blog
      pageRef: /blog
      weight: 1
    - name: Projects
      url: /projects
      weight: 2
    - name: Photography
      url: /photography
      weight: 3
    - name: Bookshelf
      url: /bookshelf
      weight: 4
    - name: Search
      weight: 5
      params:
        type: search
    - name: GitHub
      weight: 6
      url: "https://github.com/chitvs"
      params:
        icon: github
    - name: LinkedIn
      weight: 7
      url: "https://linkedin.com/in/alessandro-chitarrini"
      params:
        icon: linkedin

Requested Assistance

Consistency in Ordering: Help with ensuring that the sidebar menu items on mobile devices appear in the same order as they do in the desktop navbar.
Potential Solutions: Recommendations for any necessary adjustments or configurations to resolve this issue.

Thank you for your assistance with this matter.
Best regards,
Alessandro Chitarrini

i am seeing this as well. furthermore, the hamburger menu on mobile has added a page that I have specifically not included on the desktop menu

i am seeing this as well. furthermore, the hamburger menu on mobile has added a page that I have specifically not included on the desktop menu

I hope someone can help us.

I no longer use Hextra for unrelated reasons, but the issue still persists. It's a minor detail given how good the theme is overall. However, please note that I'm unable to reproduce the issue on my personal website anymore.


After re-introspecting w/ existing queries, return type changes to incorrect type

I have a query like GetUsers which is set to return user, but after I introspect, return type changes to character. Once this happens, when you click "add field", it causes all the values to be from the character schema instead.
I was able to "solve" this by deleting the query and making the exact same thing again - then the return type becomes correct.
Additionally, when you start building up these deep resolvers, you can imagine that re-creating these every time becomes a huge pain.
Is there no way to create the queries as text in files rather than this cumbersome tree UI?
Thanks!

Another interesting behavior was that at one point, I added two resolvers to my User object and then did an Introspect. Afterward, this time, the return type did not change, but only 1 of the two properties were available after Introspection. Then I created a brand new query of the same type and both were available.
There must be something going, maybe, with the .asset file after introspection. I will try to keep an eye on the file and report back anything that comes up.

hi,
i am just seeing this. does the return always change whenever you introspect? or it only happens occasionally/randomly? if so, can you try to recreate the situation it happens so i can pinpoint the source?
also, if you don't want to use the tree ui, you can always create your queries in text as files and simply use HttpHandler.PostAsync to make the call to your server.

Hey Gazuntype,
Honestly I just stopped using the GUI completely, and I use some of the code your GUI uses to create some API around .gql files. But the solution I have right now is kind of hacky.
I am basically just doing this - I defined some functions to make a small API:
public struct CreateQueryOptions
        {
            public string name;
            public GraphApi.Query.Type queryType;
            public string root;
            public string returnType;
            public string projection;
            public object queryArgs;
        }

        public static GraphApi.Query CreateQuery(
            CreateQueryOptions options
        )
        {
            GraphApi.Query query = new GraphApi.Query{
                    fields = new List<GraphApi.Field>(),
                    isComplete = false,
                    name = options.name,
                    query = options.projection
                        .Replace(System.Environment.NewLine, "\n"),
                    queryOptions = new List<string>(),
                    queryString = options.root,
                    returnType = options.returnType,
                    type = options.queryType
            };

            if (options.queryArgs != null)
            {
                query.SetArgs(options.queryArgs);
            }

            return query;
        }

and then I use them like this:
       public async Task<Models.User> GetUser(
            string userName = null
        )
        {
            CreateQueryOptions options = new CreateQueryOptions {
                name = "GetUsers",
                queryType = GraphApi.Query.Type.Query,
                root = "users",
                returnType = "User",
                projection =
                    @"
    id
    name
    userName
    emailAddress
    firstName
    lastName
                    ",
                queryArgs = new {
                    where = new {
                        userName,
                    }
                }
            };

            GraphApi.Query getUser = CreateQuery(options);

            UnityWebRequest request = await rttGraphQLApi.Post(getUser);

            if (request.isNetworkError)
            {
                Debug.Log("Error running GetUserByEmail: " + request.error);
                return null;
            }

            string data = request.downloadHandler.text;

            // GetUserByEmailResults parsed =  JsonConvert.DeserializeObject<GetUserByEmailResults>(data);
            JObject parsed  = JObject.Parse(data);
            IList<JToken> results = parsed["data"]["users"].Children().ToList();
            IList<Models.User> users = new List<Models.User>();
            foreach (JToken result in results)
            {
                Models.User user = result.ToObject<Models.User>();
                users.Add(user);
            }

            if (users.Count > 0)
            {
                Debug.LogFormat("Found {0} total user accounts matching.", users.Count);
                Models.User user = users[0];
                return user;
            }
            else
            {
                Debug.Log("No users found with specified email.");
            }

            return null;
        }

As far as recreating the problem, I just had a mixture of Queries and Mutations against a relatively sophisticated schema.

Personally I did a hacky thing in the GraphApiEditor.cs to be able to change the query return type if the index was lost.
GraphApiEditor.cs (new line 83)
                    if (String.IsNullOrEmpty(query.returnType)){
                        if (GUILayout.Button("Delete")){
                            graph.DeleteQuery(queryList, i);
                        }
                        continue;
                    }

Instead of
GraphApiEditor.cs (line 77)
                    if (String.IsNullOrEmpty(query.returnType)){ //move this
                        index = EditorGUILayout.Popup(type, index, options);
                        query.queryString = options[index];
                        EditorGUILayout.LabelField(options[index]);
                        if (GUILayout.Button($"Confirm {type}")){
                            graph.GetQueryReturnType(query, options[index]);
                        }
                       //here
                        if (GUILayout.Button("Delete")){
                            graph.DeleteQuery(queryList, i);
                        }

                        continue;
                    }

Basically only put the Delete button wherer there's no return Type, so you can change this thing again if it broke

This still hasnt been fixed  in almost a year, @Gazuntype do you intend to resolve this issue? It causes the GUI to be unusable


Update Cargo.toml

Mentioning "0" would mean cargo would pick the latest dependency which is versioned "0.*" which would include versions incompatible with the one that this crate was written with and may break your create.

Thanks for fixing this!

I'll release a new version tomorrow.


RFC: Attributes in #print

Proposal
In Lean 3, #print would show the attributes on a definition, such as @[reducible] or @[simp]. We should get this back. I think the only blocker here is a technical one.
@[reducible, simp] def foo := 1

#print foo
-- @[reducible] def Lean.Compiler.foo : Nat :=
-- 1

Currently a few attributes like @[reducible] are supported, but not others like @[simp], as demonstrated above.
Community Feedback
This comes up periodically on Zulip, here is the latest thread.
Impact
Add :+1: to issues you consider important. If others benefit from the changes in this proposal being added, please ask them to add :+1: to it.

This RFC looks incomplete: it should discuss whether the intention is to print the attributes at the time of declaration or at the time of use, and the pros and cons of this decision.

@digama0, would you be able to write something explaining why you prefer that choice? (Which, indeed, seems the more useful one to me.)


ISPN-14863 Implement RESP "cluster nodes" command

https://issues.redhat.com/browse/ISPN-14963

Updated with suggestions.

I need to find another way to check this beyond using the Lettuce client to validate the response.

Doing a comparison by running with echo 'cluster-enabled yes' | redis-server -, my server is version Redis 7.0.10 (00000000/0) 64 bit and it is assigned a few slots automatically. Running the command with redis-cli:
<IP_ADDRESS>:6379> CLUSTER NODES
308345ab50e60cdf538e30d2fd5e3de671abfb0c :6379@16379 myself,master - 0 0 0 connected 238 5282 5798 9755 12291 12539
<IP_ADDRESS>:6379>

Running with ISPN:
<IP_ADDRESS>:11222> CLUSTER NODES
jbolina-64043 <IP_ADDRESS>:7800@7800,,shard-id=jbolina-64043 myself,master - 0<PHONE_NUMBER> 1 connected 0-255
<IP_ADDRESS>:11222>

The ISPN output includes the shard-id in the auxiliary field. Per the doc, this field is always included in version >= 7.2.0.
I need to update to return the correct address in the protocol server.

Doing a test with three servers (server-1, server-2, server-3):
<IP_ADDRESS>:11222> cluster nodes
server-1 <IP_ADDRESS>:11222@60836,,shard-id=server-1 myself,master - 0<PHONE_NUMBER> 9 connected 0-255
server-2 <IP_ADDRESS>:11232@60836,,shard-id=server-2 master - 0<PHONE_NUMBER> 9 connected 0-255
server-3 <IP_ADDRESS>:11322@60836,,shard-id=server-3 master - 0<PHONE_NUMBER> 9 connected 0-255
<IP_ADDRESS>:11222>

I think that should be it, going to squash the commits. I guess a point of attention is the way I did to retrieve the resp server.

I need to find another way to check this beyond using the Lettuce client to validate the response.

Maybe introduce a Vert.x Redis client test ?

I think adding the Vert.x might be worth it, especially since we're missing more throughout coverage in cluster mode, checking the CRC16 implementation. Although, the client seems to use the CLUSTER SLOTS command (link).
The CLUSTER SLOTS is next on my list. I'll add the client then.

Pushed a small change. I was not retrieving the segments owned (that's why all nodes had 0-255). Now we have:
jbolina-22538 <IP_ADDRESS>:11222@54392,,shard-id=jbolina-22538 myself,master - 0<PHONE_NUMBER> 5 connected 74-136 142-145 157-168 181-181 187-210 212-217 219-219 225-226 230-247
jbolina-61882 <IP_ADDRESS>:11322@54392,,shard-id=jbolina-61882 master - 0<PHONE_NUMBER> 5 connected 0-73 137-141 146-156 169-180 182-186 211-211 218-218 220-224 227-229 248-255

@tristantarrant @wburns something else ?

Integrated into main, thanks @jabolina !


added requirements

added requirements.txt for folks running it in a virtual environment

thanks for doing this!


Error in gatsby docs tutorial part six

I'm just going through gatsby tutorials right now (absolutely amazing btw, probably one of the most intuitive static site generators I've used) and I think there might be a slight error in one of the snippets used in part six of the tutorial.
In this snippet:

Looking through the schema in GraphiQL, it only takes a string I don't believe date can take the argument formatString, and thus will throw an error.
I'm happy to do a pull request and amend it if that's the case. If not, sorry for being completely ignorant!

Oops, sorry! Posted this in the wrong repo! (Too many tabs open). Closing it now.


Update notif users

Updated the users array per issue #2. Commented out the empty users.

Thank you for doing this.
If this is your first pr with us please fill this out and we'll send you some awesome stickers and a nice hand written card.
https://goo.gl/forms/r02wt0pBNhkxYciI3


Add DeVinci chat app as demo example

Thank you for this great project and your work. I built this chat app prototype around WebLLM and deployed it on the decentralized cloud platform Internet Computer (so the frontend and the backend are hosted decentrally). If you like, this PR adds it as a demo example. Thanks.

thank you, @tqchen :)


tfgrid_directory [development_tfgrid_directory]

https://github.com/threefoldtech/jumpscaleX_threebot/tree/provisioning-actors/ThreeBotPackages/threefold/tfgrid_directory/actors

[ ] wait till development_bcdb_sqlite is done & migrate the models (* -> **)
[ ] put on right branch
[ ] ask kristof to review the actors & the way how we use the model
[ ] show kristof how it get info from deployed zos-nodes
[ ] create a actor method (using myjobs) to pull info from old TFDir, if node already exists don't overwrite
[ ] create example in non jumpscale pythong how to call the redis client and use actors from gedis to call actor and verify info

start feature development on development branch until development_bcdb_sqlite is ready. Then migrate models to use sqlite.
Make sure to develop on branch development_tfgrid_directory

I created a preliminary actor for migration but some stuff is still missing like figuring out the threebot id that will be used for migrated farms.
I can see there's a discussion going on here https://github.com/threefoldtech/tf_cockpit/issues/12

@serboctor where is your code for the migration ?

@zaibon https://github.com/threefoldtech/jumpscaleX_threebot/blob/development_tfgrid_directory/ThreeBotPackages/threefold/directory/actors/farms.py#L87

@despiegk For the last point in the ticket description do you just want a script that shows how to call gedis actors using redis? I don't get the verify info part.


feat: change screen size queries to min-width

What I did

Converted media queries to min-width based queries for standard breakpoints.  I feel this is the proper approach for responsively designed websites.  Breakpoints matching css cascading to larger sizes so that it is additive :

[no-break-point]
 ⬇
[320px] /* mobilePortraitBreakpoint */
 ⬇
[576px] /* mobileTabletBreakpoint */
 ⬇
[768px]
 ⬇
[992px]
 ⬇
[1200px]
 ⬇
etc...

/* no breakpoint */
div {
 color: blue;
}

@media and screen(min-width: 320px) {
  /* mobilePortraitBreakpoint */
  div {
    padding: 0.5em;
  }
}

@media and screen(min-width: 576px) {
  /* mobileTabletBreakpoint */
  div {
    padding: 1em;
  }
}

etc...

Ordered tokens smallest to biggest, and added comments for what I think might be special breakpoints tokens which are based on max-width such as mobile and mobileXL.
Added matches property to return all current  matching BreakPointKey.  Resulting in a stack such as ['mobile', 'mobilePortrait', 'mobileLandscape', 'mobileLandscape', 'tabletPortait']  if the screen size is < 992px as defined in the token value:

export const tabletLandscapeBreakpoint = unsafeCSS`992px`;

The implementation of matches allows you to set a class based on if a specific breakpoint is or isn't in the stack, then we know we've gone beyond the size in which the interface needs to react.
/* Looking for if `tabletLandscape` breakpoint has been reached. If not we add `is-mobile` to classes */
const mobile = !this.#screenSize.matches.has('tabletLandscape');
const classes = { 'is-mobile': mobile };

The reasoning behind this change is that the size attribute is only set to the current breakpoint giving nothing to base the logic of checking all matched breakpoints from if the current state returned was less than the tabletLandscapeBreakpoint.  Because min-width is additive, the matches will include all breakpoints that match.
This is also working around the idea of this.mobile in the context of the controller which is based on the max-width breakpoint value of 700px defining what is or isn't mobile.  I'm not convinced we should have a specific definition of mobile at a specific breakpoint, rather being more content-specific on when things shift.
In the case of secondary-nav that content specific happens under 992px per spec due to limited width for content at a smallish screen size.  Not that 992px specifies what is and what isn't mobile.

@zeroedin happy for you to fold this into your secondary-nav PR, so long as it doesn't break any other components.

@zeroedin happy for you to fold this into your secondary-nav PR, so long as it doesn't break any other components.

As it stands right now, the only other component that is actively using RHDSScreenSizeController is rh-dialog.  It uses the mobile max-width breakpoint, which this PR does not change.
I'll fold this into feat/secondary-nav.  I'll leave this PR open as a draft for any further comments or considerations.

Closing this as it has been implemented.


Avoid assigning agents with rare capabilities to tasks that can be done by many agents

Currently, the  agents2tasks algorithm is focused on minimizing the cost of assignments.
I have noticed that you tend to set all assignment costs equal to 1.  So the cost optimization is irrelevant.  I assume that you therefor want to optimize for the maximum number of tasks assigned in each timeslot ( funny thing: you never actually specified what you want to optimize for, all you said was 'solve' the problem )
If cost is equal, the current algorithm simply assigns tasks in first come, first served order.  This can cause agents to that have rare capabilities to be assigned to tasks that could have been dome by many other agents, which can in turn mean that they are not available for the  later tasks that only they could have done.
Here is a simple example that demonstrate the problem:

Because of the order in which the agents and tasks have been specified, Alice will be assigned to the cleaning task, when it would have been much more efficient to assign her to the later, teacher task that only she can do.
Solution: re-order the tasks so that agents with multiple capabilities are NOT assigned to tasks that many others could do.

So I can understand it better.

Do you want me to re-order so that the agent with multiple task are at the end of the  "a" records
Remove from "a" records where multitasks are task are assigned those that can be done by others?

This issue is assigned to you in test.  Please do not reassign it to me until you have tested it.

Do you want me to ...

All you need to do is test this.  No changes to the input are required.
The sorting described was implemented yesterday in commit https://github.com/JamesBremner/Agents2Tasks/commit/16e6ba72a730729da1bcc1cb24e96b0d7925c276

Works as expected. Input:
a Alice 1 teacher cleaner
a Bob 1 cleaner
t<PHONE_NUMBER>00 cleaner teacher
produced output
A<PHONE_NUMBER>00 Alice to teacher
A<PHONE_NUMBER>00 Bob to cleaner


Suggestion: example for raw data

Looks cool!
An example for "Raw data can be passed to the contract" would be helpful.  And as a follow-on, if using pipes and rawdata work, an example of that would be quite nice.  (The idea is imagine codesim callmethod A | codesim callmethod B)

At the moment it is just the raw data sent with a transaction (in hex string format).
Good idea on having pipes. Perhaps it would make more sense once state management is in place?