EFLOW VM doesn't contain proper host name in hosts file

Not sure if this a bug or not, but we noticed a quirk of the hosts file on the VM that is causing us trouble in connecting to the edgeHub module. Here's the content of /etc/hosts in the EFLOW VM:

# Begin /etc/hosts (network card version)

::1         ipv6-localhost ipv6-loopback
<IP_ADDRESS>   localhost.localdomain
<IP_ADDRESS>   localhost
<IP_ADDRESS>   efl-vm
# End /etc/hosts (network card version)

Notably, there's no entry there that contains the actual host name of the EFLOW VM, in our case W10IOT-EFLOW-458b3451. On the standard Ubuntu and Raspbian systems we've tested with, there's been an entry included in the hosts file like "<IP_ADDRESS>   DeviceHostname"
Is this intentional? By the way, looking at my Preview Preview EFLOW VM, it looks like there was such an entry included.
Not having this entry there seems to result in a failure to connect to the edgeHub. We are using Microsoft.Azure.Devices.Client.ModuleClient.CreateFromEnvironmentAsync(TransportSettings) and TransportSettings are created as new Microsoft.Azure.Devices.Client.ITransportSettings[] { new AmqpTransportSettings(Microsoft.Azure.Devices.Client.TransportType.Amqp_Tcp_Only) }
which are doing based on the example here
Once we manually add the missing entry to the hosts file, we're able to connect to the edgeHub successfully.

Thanks @iconics-rich for your detailed information.
Could you please provide us the following information?

What was the deployment method used? (WAC/PowerShell)
What version of EFLOW are you using? You can check the version by following these instructions:
a.	Open Settings on Windows 10
b.	Select Add or Remove Programs
c.	Select Azure IoT Edge app
d.	Check the version number
Are you using an external or default switch?
Could you please send us the logs? You can get the logs using the Get-EflowLogs command
Could you please send us the following files?
a) /etc/hostname
b) /etc/hosts
c) /etc/iotedge/config.yaml (make sure no secret information is shared)

Thanks,
Francisco

Hi @fcabrera23 ,
Sorry for the delay. Here's the info you requested:

What was the deployment method used? (WAC/PowerShell)

I used WAC on Windows 10 (21H1), but I'm seeing similar behavior on a Server 2019 VM where PowerShell was used.

What version of EFLOW are you using?

<IP_ADDRESS>

Are you using an external or default switch?

External
I've also attached file EFLOW logs and files you requested.
eflowlogs.zip
files.zip

Thanks @iconics-rich - We are currently investigating this issue.  Could you please provide us  /var/log/cloud-init.log and /var/log/cloud-init-output.log files?

Here you go.
cloud-init.zip
Thanks,
Rich

HI @iconics-rich,
Thanks for your providing the logs.  We have successfully identified the problem and will be fixed in our next update.
Thanks for your help,
Francisco

Sounds great, thank you for your help here Francisco.

@iconics-rich,
Thanks for your patience. We have released our Generally Available version - Please check out our latest release.

@fcabrera23 , That's great news! A quick question on this item from those release notes:

Windows Admin Center (WAC) Azure IoT Edge extension updated to 1.61.0 to support new EFLOW GA functionalities – WAC IoT Edge extension remains in preview.

I'm not sure what the implications of that are. Does the WAC extension being in preview mean that an EFLOW VM created via WAC will be different to one created via PowerShell?

@iconics-rich
The EFLOW VM creation is exactly the same when using PowerShell and WAC. What we mean by saying that WAC Extension is still in Preview, is that the features that are included there for managing the VM using WAC extension are in Preview. We plan to keep improving and adding features in the following updates to our extension.
Again, to make it clear, EFLOW is GA, but the Azure IoT Edge Extension will continue Preview while we add more features.

Ah ok, that makes sense. Thanks for clarifying.
--
Rich Henderson, US QA Manager
t: +1<PHONE_NUMBER> • iconics.comhttps://iconics.com
ICONICS – A Group Company of Mitsubishi Electric
Register HEREhttps://iconics.com/connect2021 for ICONICS Connect 2021!
@.***D76911.A6323DF0]https://iconics.com/newsletter [A picture containing drawing  Description automatically generated] https://www.linkedin.com/company/iconics  [A picture containing animal, bird  Description automatically generated] https://www.twitter.com/ICONICS  [A picture containing kit, drawing  Description automatically generated] https://www.facebook.com/ICONICSInc  [A picture containing drawing  Description automatically generated] https://www.youtube.com/user/IconicsVideo
From: Francisco Cabrera @.>
Sent: Wednesday, June 23, 2021 7:04 PM
To: Azure/iotedge-eflow @.>
Cc: Richard Henderson<EMAIL_ADDRESS>Mention @.***>
Subject: Re: [Azure/iotedge-eflow] EFLOW VM doesn't contain proper host name in hosts file (#30)
@iconics-richhttps://github.com/iconics-rich
The EFLOW VM creation is exactly the same when using PowerShell and WAC. What we mean by saying that WAC Extension is still in Preview, is that the features that are included there for managing the VM using WAC extension are in Preview. We plan to keep improving and adding features in the following updates to our extension.
Again, to make it clear, EFLOW is GA, but the Azure IoT Edge Extension will continue Preview while we add more features.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/iotedge-eflow/issues/30#issuecomment-867214044, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AP6AL6FEUYOSWB5V3HVIRWDTUJR4RANCNFSM46Q2CUUQ.


Alter the query instead of the repo

I've been struggling w/ other libraries that are expecting an object with Ecto's standard method signatures.  This strategy leaves the existing functionality untouched, but exposes the private helper methods in their own module so they can be pulled in separately.
A few points to note:

add_prefix_to_query and add_prefix were both renamed to just set_tenant to keep the api easier. If there is an important reason why that might be a bad thing, please let me know. But I did alter the version targeting Ecto models to look for meta instead of struct. It seemed to work w/out that change, but I'm not sure why.
If you're ok w/ this change, I can append this strategy to the docs.

A few other thoughts based on work I'm doing in my project:
I would like to enforce the specification of a tenant, and raise an appropriate error if one is missing. To do this, I've created a repo that wraps the 'real' ecto repo. It will then check for the prefix and raise if it is missing.  Other methods are then defdelegate-d to the repo.
I'm not thrilled with how this looks. I figure there is probably a metaprogramming solution that will do this much more elegantly, and without me having to be so intimately tied to the stuff being generated by Ecto. But so far it's working.
defmodule RestConnectder.ApartmentexRepo.TenantMissingError do
  defexception message: "No tenant specified"
end

defmodule RestConnectder.ApartmentexRepo do
  alias RestConnectder.ApartmentexRepo.TenantMissingError
  alias RestConnectder.Repo
  alias Ecto.Changeset

  defdelegate __adapter__, to: Repo
  defdelegate __log__(entry), to: Repo
  defdelegate config(), to: Repo
  defdelegate start_link(opts \\ []), to: Repo
  defdelegate stop(pid, timeout \\ 5000), to: Repo
  defdelegate transaction(fun_or_multi, opts \\ []), to: Repo
  defdelegate in_transaction?(), to: Repo
  defdelegate rollback(value), to: Repo
  defdelegate aggregate(queryable, aggregate, field, opts \\ []), to: Repo
  defdelegate preload(struct_or_structs, preloads, opts \\ []), to: Repo
  defdelegate __pool__, to: Repo
  defdelegate __sql__, to: Repo

  def all(queryable, opts \\ []) do
    assert_tenant(queryable)
    Repo.all(queryable, opts)
  end

  def get(queryable, id, opts \\ []) do
    assert_tenant(queryable)
    Repo.get(queryable, id, opts)
  end

  def get!(queryable, id, opts \\ []) do
    assert_tenant(queryable)
    Repo.get!(queryable, id, opts)
  end

  def get_by(queryable, clauses, opts \\ []) do
    assert_tenant(queryable)
    Repo.get_by(queryable, clauses, opts)
  end

  def get_by!(queryable, clauses, opts \\ []) do
    assert_tenant(queryable)
    Repo.get_by!(queryable, clauses, opts)
  end

  def one(queryable, opts \\ []) do
    assert_tenant(queryable)
    Repo.one(queryable, opts)
  end

  def one!(queryable, opts \\ []) do
    assert_tenant(queryable)
    Repo.one!(queryable, opts)
  end

  def insert_all(schema_or_source, entries, opts \\ []) do
    assert_tenant(schema_or_source)
    Repo.insert_all(schema_or_source, entries, opts)
  end

  def update_all(queryable, updates, opts \\ []) do
    assert_tenant(queryable)
    Repo.update_all(queryable, updates, opts)
  end

  def delete_all(queryable, opts \\ []) do
    assert_tenant(queryable)
    Repo.delete_all(queryable, opts)
  end

  def insert(struct, opts \\ []) do
    assert_tenant(struct)
    Repo.insert(struct, opts)
  end

  def update(struct, opts \\ []) do
    assert_tenant(struct)
    Repo.update(struct, opts)
  end

  def insert_or_update(changeset, opts \\ []) do
    assert_tenant(changeset)
    Repo.insert_or_update(changeset, opts)
  end

  def delete(struct, opts \\ []) do
    assert_tenant(struct)
    Repo.delete(struct, opts)
  end

  def insert!(struct, opts \\ []) do
    assert_tenant(struct)
    Repo.insert!(struct, opts)
  end

  def update!(struct, opts \\ []) do
    assert_tenant(struct)
    Repo.update!(struct, opts)
  end

  def insert_or_update!(changeset, opts \\ []) do
    assert_tenant(changeset)
    Repo.insert_or_update!(changeset, opts)
  end

  def delete!(struct, opts \\ []) do
    assert_tenant(struct)
    Repo.delete!(struct, opts)
  end

  def assert_tenant(%Changeset{} = changeset) do
    assert_tenant(changeset.data)
  end

  def assert_tenant(%{__meta__: _} = model) do
    case Ecto.get_meta(model, :prefix) do
      nil -> raise TenantMissingError
      _ -> nil
    end
  end

  def assert_tenant(queryable) do
    case queryable |> Ecto.Queryable.to_query |> Map.fetch(:prefix) do
    {:ok, _} -> nil
    {:error} -> raise TenantMissingError
    end
  end
end

In my case, all of the tables are in schemas except for the tenants themselves, so when I create a tenant, I pass in the 'real' Repo. But ideally, this could be configured in such a way that you could tell it which tables require tenants, and which do not.
Thoughts/Suggestions?

Sorry, here's a slightly improved macro solution:
defmodule RestConnectder.RepoDelegation do
  defmacro __using__(opts) do
    quote bind_quoted: [opts: opts] do
      @repo Keyword.fetch!(opts, :repo)
      # @untenanted_models Keyword.get(opts, :repo, [])

      # From Ecto.Repo
      defdelegate __adapter__, to: @repo
      defdelegate __log__(entry), to: @repo
      defdelegate config(), to: @repo
      defdelegate start_link(opts \\ []), to: @repo
      defdelegate stop(pid, timeout \\ 5000), to: @repo
      defdelegate transaction(fun_or_multi, opts \\ []), to: @repo
      defdelegate in_transaction?(), to: @repo
      defdelegate rollback(value), to: @repo
      defdelegate aggregate(queryable, aggregate, field, opts \\ []), to: @repo
      defdelegate preload(struct_or_structs, preloads, opts \\ []), to: @repo

      # From Ecto.Adapters.SQL
      defdelegate __pool__, to: @repo
      defdelegate __sql__, to: @repo

      def assert_tenant(%Ecto.Changeset{} = changeset) do
        assert_tenant(changeset.data)
      end

      def assert_tenant(%{__meta__: _} = model) do
        case Ecto.get_meta(model, :prefix) do
          nil -> raise RestConnectder.ApartmentexRepo.TenantMissingError
          _ -> nil
        end
      end

      def assert_tenant(queryable) do
        case queryable |> Ecto.Queryable.to_query |> Map.fetch(:prefix) do
        {:ok, _} -> nil
        {:error} -> raise RestConnectder.ApartmentexRepo.TenantMissingError
        end
      end
    end
  end
end

defmodule RestConnectder.ApartmentexRepo do
  @behaviour Ecto.Repo
  use RestConnectder.RepoDelegation, repo: RestConnectder.Repo #, untenanted_models: [RestConnectder.Company]
  alias RestConnectder.Repo
  alias Ecto.Changeset

  def all(queryable, opts \\ []) do
    assert_tenant(queryable)
    Repo.all(queryable, opts)
  end

  def get(queryable, id, opts \\ []) do
    assert_tenant(queryable)
    Repo.get(queryable, id, opts)
  end

  def get!(queryable, id, opts \\ []) do
    assert_tenant(queryable)
    Repo.get!(queryable, id, opts)
  end

  def get_by(queryable, clauses, opts \\ []) do
    assert_tenant(queryable)
    Repo.get_by(queryable, clauses, opts)
  end

  def get_by!(queryable, clauses, opts \\ []) do
    assert_tenant(queryable)
    Repo.get_by!(queryable, clauses, opts)
  end

  def one(queryable, opts \\ []) do
    assert_tenant(queryable)
    Repo.one(queryable, opts)
  end

  def one!(queryable, opts \\ []) do
    assert_tenant(queryable)
    Repo.one!(queryable, opts)
  end

  def insert_all(schema_or_source, entries, opts \\ []) do
    assert_tenant(schema_or_source)
    Repo.insert_all(schema_or_source, entries, opts)
  end

  def update_all(queryable, updates, opts \\ []) do
    assert_tenant(queryable)
    Repo.update_all(queryable, updates, opts)
  end

  def delete_all(queryable, opts \\ []) do
    assert_tenant(queryable)
    Repo.delete_all(queryable, opts)
  end

  def insert(struct, opts \\ []) do
    assert_tenant(struct)
    Repo.insert(struct, opts)
  end

  def update(struct, opts \\ []) do
    assert_tenant(struct)
    Repo.update(struct, opts)
  end

  def insert_or_update(changeset, opts \\ []) do
    assert_tenant(changeset)
    Repo.insert_or_update(changeset, opts)
  end

  def delete(struct, opts \\ []) do
    assert_tenant(struct)
    Repo.delete(struct, opts)
  end

  def insert!(struct, opts \\ []) do
    assert_tenant(struct)
    Repo.insert!(struct, opts)
  end

  def update!(struct, opts \\ []) do
    assert_tenant(struct)
    Repo.update!(struct, opts)
  end

  def insert_or_update!(changeset, opts \\ []) do
    assert_tenant(changeset)
    Repo.insert_or_update!(changeset, opts)
  end

  def delete!(struct, opts \\ []) do
    assert_tenant(struct)
    Repo.delete!(struct, opts)
  end
end

once you can pass in the models that should and should not be tenanted, I think this entire thing can be simplified to just:
defmodule RestConnectder.ApartmentexRepo do
  @behaviour Ecto.Repo
  use RestConnectder.RepoDelegation, repo: RestConnectder.Repo, untenanted_models: [RestConnectder.Company]

Wondering if we can just import Ecto.Repo, then do something like enumerating through info(:functions) to patch the functions with the guard. That or proxy module that checks the guard before passing function/arity direct to Ecto.Repo. I'm on a road trip, but will be back Monday to take a look :-)

@Dania02525 I've got this sorted out, I think. It'll be on a separate PR. No rush. Enjoy your road trip!

Hi @Dania02525, have you had a chance to look into this at all?  I tried the 'enumerating' option you were thinking first, but in the end I skipped it because:

the opts \ [] means that each overridden function would have mean 2 methods created (all you get is delete!\3, delete!\2 from the info(:functions) call
Not all of the methods are being overridden, which means I'd need to maintain a list of which were just delegated, and which got the set_tenant call. That impairs the code's ability to be resilient to Ecto changes anyway, so I figured I may as well make it easier to read and debug

I was just starting to work on a pull request to swap the repo and the queryable places because repo as first param blocks apartmentex api of being piped. But your solution do the job too.

@danielfoxp2  - https://github.com/promptworks/tenantex is what I wound up with.


FPS test

Hi! Great code you have here!
I just find that your FPS test script speed.py may not be precise. Since you did not count time after all CUDA threads finish. Although this should only affect time count by 1/args.iter, considering also the use of time.time, on my machine the FPS is 104.24/100.5 before/after the fix (resnet34), seeming larger than random deviations. So I'll still propose the fix in a PR for your review.
For reference:
https://pytorch.org/docs/stable/generated/torch.cuda.synchronize.html?highlight=synchro#torch.cuda.synchronize
https://www.webucator.com/article/python-clocks-explained/

time.perf_counter() will give you the most accurate results when testing the difference between two times
time.time() should NOT be used for comparing relative times. It’s not reliable because it’s adjustable.

command I used: python -m utils.speed --cfg=cfgs/laneatt_tusimple_resnet34.yml

You are correct. However, I am not sure if I will accept pull requests into this repository since it should contain the official implementation used to obtain the results described in the paper, especially when the differences are that small. I might add a warning to the README.md explaining that, though. Thank you for your contribution.

However, I am not sure if I will accept pull requests into this repository since it should contain the official implementation used to obtain the results described in the paper,

Right, that is very thoughtful. Let's not merge that PR. I agree with closing it and add a ref in the readme.


thoughts

could add more about relationship between encryption types and SSL SSH -
Could add more to more clearly explain that we don't recommend using external drives, but what do we recommend

could add: https://pmc.ncbi.nlm.nih.gov/articles/PMC2965886/


Add badge capabilities to users

Similar to how github profiles have badges/achievements, this adds the capabilities to Gitea to be able to utilize that in the future

I don't get it, whats the use case for this?

I don't get it, whats the use case for this?

@noerw for community instances, where they may wish to "award" some badge for certain situations.

I don't get it, whats the use case for this?

@noerw for community instances, where they may wish to "award" some badge for certain situations.

Sorry to be that guy, but this feels very feature-creepy to me.
Gamification as a network effect multiplier kinda makes sense for a platform like GitHub, but should this really be a Gitea feature? (yes I'm biased; i don't like gamification).
Currently this PR is rather slim LoC-wise, but until it's a working feature (configure badges via UI or config file, updating badges via cron task or adding hooks into every other route), I'd estimate some extra 500LoC, lower bound.

Badges can be useful also in private instances to motivate developers etc
I don't think we need UI in gitea to manage that, only API. Service to manage badges and add for users can be developed outside of Gitea core.

Badges can be useful also in private instances to motivate developers etc

Developers do meaningful work when they are intrinsically motivated, not by some made up metric or status icon. But what do I know.

I don't think we need UI in gitea to manage that, only API. Service to manage badges and add for users can be developed outside of Gitea core.

Doing hundreds of API calls to check if a badge should be applied seems more than wasteful to me.
I'll shut up, seems like I'm in the minority here

It's probably not the best way to reward developers, but I see no problem in at least supporting it on a invisible api-only level.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@82f89ff). Click here to learn what that means.
The diff coverage is 80.00%.

:exclamation: Current head 0145a5e differs from pull request most recent head bf0763a. Consider uploading reports for the commit bf0763a to get more accurate results

@@           Coverage Diff           @@
##             main   #20607   +/-   ##
=======================================
  Coverage        ?   47.07%
=======================================
  Files           ?      989
  Lines           ?   136380
  Branches        ?        0
=======================================
  Hits            ?    64202
  Misses          ?    64302
  Partials        ?     7876

Impacted Files
Coverage Δ

routers/web/user/profile.go
46.74% <42.85%> (ø)

models/user.go
32.84% <100.00%> (ø)

models/user/badge.go
100.00% <100.00%> (ø)

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

Could you put a screenshot?

Would be nice to have followup PR to add API that would be accessible only for admin to define badges and add user badges

Is this released? I haven't found it on the release notes

@bauermarkus yup, it's in 1.18, however it's just the absolute base code needed to present badges stored in the DB to show on user's profiles. I have a followup PR where you can manage badges with the API.


Return command status for set-config commands

Codecov Report
Attention: Patch coverage is 0% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 34.93%. Comparing base (b286c2d) to head (3040616).

Files
Patch %
Lines

deployment/cluster.go
0.00%
11 Missing :warning:

deployment/aeroinfo.go
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #52      +/-   ##
==========================================
- Coverage   34.96%   34.93%   -0.04%
==========================================
  Files          23       23
  Lines        5477     5482       +5
==========================================
  Hits         1915     1915
- Misses       3427     3432       +5
  Partials      135      135

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


Better UI on showing transferred amount (specially when send-to-many used)

Problem : When many transactions are sent from an adress, e. x. Pool rewards, you as a user should dig deep to find your relevant tx and transferred amount!!.. That's really confusing for newcomers to find out how much they have been paid!
Requested feature: Making it more accessible to see tx fee, amount, balance after each received stx transaction

This is effectively blocked on the API which currently doesn't easily expose stx transfer amounts for certain types of transactions. It's in development, tracked here: https://github.com/blockstack/stacks-blockchain-api/issues/369

Thanks for your reply... I took a look at the provided link but it was hard for me to confirm if that is the same problem as i mentioned here!... Anyway, i would be satisfied to see it is already under the focus

This should be fixed now


fix(v2): getting started url

Motivation
(Write your motivation here.)
When opening the initial page and clicking on the Getting Started  button, the link was broken. This is because we change the initial docs path recently.
imgur
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan

Open the docusaurus v2 initial page
Click on the getting started button and

Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)
#2652

I think that I created from a base different of the master, I'll create another.


Set up Integrations: Chat, LibGuides, Databases

Is LibGuides OAI-PMH a possibility? Admins, head to Tools > Data Exports and click Get OAI Base URL to find the unique base URL for your LibGuides site.

@nleboeuftrujillo i have just about completed the information for the questionnaire which is in Google Drive at https://docs.google.com/document/d/19yxW-PguJsf7ctCs5AYUgOFQqPs50QxTcD5elVPIWq4/edit?usp=sharing
there are a couple pieces where I am in search of clarifications so can cover those in a future meeting. i linked the IR .doc file in the other google doc - have asked Caroline about the collection name to confirm. thanks!


samples: cpp: add a Hello, world! C++ app

Create a separate hello world C++ app for Zephyr.

/me likes! I'd say it probably belongs to the cpp/ samples folder though, as I'd prefer if each Hello World project (to your point, definitely to hope seeing other languages in the future) stays as naked as possible.
 Would be nice to refresh it with C11 Threads, no?

/me likes! I'd say it probably belongs to the cpp/ samples folder though, as I'd prefer if each Hello World project (to your point, definitely to hope seeing other languages in the future) stays as naked as possible.

I can see the appeal of keeping languages fenced off, but that would be a lot of duplication for what is otherwise almost identical. I didn't think of updating the docs, but it would be nice to just cycle through a few different tabs on the same page to see how to accomplish the same trivial thing in a different language.

 Would be nice to refresh it with C11 Threads, no?

Why settle for C11 threads and not C++11 threads? The SDK needs some patching, which is probably the biggest challenge (because it's very time intensive). But otherwise #43729 has been good to go for a long time.

I'd personally create a separate sample, hello_world_cpp maybe

Back to the the "tabs" idea though, having one actual sample per language certainly shouldn't prevent us from creating some kind of page cataloguing the various Zephyr Hello Worlds (eh, there might even be some out-of-tree ones worth referencing?)

That would work too. It's well beyond my level of .rst / sphinx, but maybe later on down the road.
I'll change this so that it targets the cpp samples subdir

I have another rev to push. Would like to see if it's possible to use iostream with newlib

I haven't done a massive build for every platform yet. Hopefully nothing too serious.

It appears as though the iostream approach causes twister to fail on virtually all platforms even though the correct output appears on the terminal. Maybe it isn't flushed immediately or something along those lines. It would be strange if there were unusual characters being written to the output, but that doesn't appear to be the case.

Works on my desk. For excluded platforms, patches welcome.

@aescolar - hopefully this aligns with the goals of #65872 - I exclude native_posix and suggest native_sim in yaml comments.

Still need to disable native_posix unfortunately, @aescolar, but I made your other requested changes.
ERROR   - native_posix              samples/cpp/hello_world/sample.cpp.helloworld       FAILED : Timeout
INFO    - /home/cfriedt/zephyrproject/zephyr/twister-out/native_posix/samples/cpp/hello_world/sample.cpp.helloworld/handler.log
ERROR   - *** Booting Zephyr OS build zephyr-v3.5.0-1864-gf13c2bd3ca3a ***

INFO    - /home/cfriedt/zephyrproject/zephyr/twister-out/native_posix/samples/cpp/hello_world/sample.cpp.helloworld/handler.log
INFO    - Total complete:   40/  40  100%  skipped:   19, failed:    1, error:    0

Still need to disable native_posix unfortunately, @aescolar, but I made your other requested changes.

@cfriedt It does not sound right.
The PR as it is now works fine with native_posix* here :
zephyr$ twister -p native_sim -p native_sim_64 -p native_posix -p native_posix_64  -T samples/cpp/hello_world/ -v
Renaming output directory to /persistant/scratch/zephyrproject/zephyr/twister-out.3
INFO    - Using Ninja..
INFO    - Zephyr version: zephyr-v3.5.0-1864-gf1f578704222
INFO    - Using 'zephyr' toolchain.
INFO    - Building initial testsuite list...
INFO    - Writing JSON report /persistant/scratch/zephyrproject/zephyr/twister-out/testplan.json
INFO    - JOBS: 32
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - 1/4 native_sim                samples/cpp/hello_world/sample.cpp.helloworld      PASSED (native 2.612s)
INFO    - 2/4 native_sim_64             samples/cpp/hello_world/sample.cpp.helloworld      PASSED (native 2.611s)
INFO    - 3/4 native_posix_64           samples/cpp/hello_world/sample.cpp.helloworld      PASSED (native 2.611s)
INFO    - 4/4 native_posix              samples/cpp/hello_world/sample.cpp.helloworld      PASSED (native 2.612s)

Just enabling it
diff --git a/samples/cpp/hello_world/sample.yaml b/samples/cpp/hello_world/sample.yaml
index d86d20caa0..ce29b0df04 100644
--- a/samples/cpp/hello_world/sample.yaml
+++ b/samples/cpp/hello_world/sample.yaml
@@ -22,5 +22,3 @@ tests:
       - intel_ish_5_4_1
       - intel_ish_5_6_0
       - intel_ish_5_8_0
-      # please use native_sim instead
-      - native_posix

The PR is here for reference. Please do whatever you like with it @kartben.

Thanks. I will have a look and try to push it over the finish line. To clarify I was mainly/only asking that the wording "all supported boards" is not used in combination with an hyperlink taking one to the list of all Zephyr's supported boards, as this can be quite error-prone to newcomers who wouldn't understand why their board is in fact not (currently) working.

Done! I reworded to say "many supported boards", removed the hyperlink, and rebased.
Does that work @kartben?


Different results between released model and released ply

I tried to infer and fuse on DTU dataset, but found the predicted results are different from your released ply models.
For example:
Your released point cloud for scan1 (mvsnet001_l3.ply): 26727801 points, fscore 0.191357
Predicted point cloud for scan1: 19405933 points, fscore 0.2162721
When I visualize the two point clouds, the completeness of predicted point cloud is much lower than released point cloud, especially around edges.
My environment:
Fusibile: compiled with cuda 11.4, sm86
Pytorch: 1.8.2+cu111
Python: 3.8.12
Here is my log file for scan1
log.txt

I tried to set align_corners=True for grid_sample, and I got more points (22287197), but still less than 26727801.

Maybe you can try Pytorch1.2.0, cudatoolkit 10.0 and python3.6.

I tried to infer and fuse on DTU dataset, but found the predicted results are different from your released ply models.
For example: Your released point cloud for scan1 (mvsnet001_l3.ply): 26727801 points, fscore 0.191357 Predicted point cloud for scan1: 19405933 points, fscore 0.2162721
When I visualize the two point clouds, the completeness of predicted point cloud is much lower than released point cloud, especially around edges.
My environment: Fusibile: compiled with cuda 11.4, sm86 Pytorch: 1.8.2+cu111 Python: 3.8.12
Here is my log file for scan1 log.txt

Hi, I used gipuma to fuse the depth maps, but I the number I got is only half as the author provided. So if there is any parameters of setting I need to change?


MINIFICPP-2071 - Change default configuration class to yaml

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically main)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI results for build issues and submit an update to your PR as soon as possible.

I'd leave this on AdaptiveConfiguration, and let that fall back to YAML when necessary.


chore(ONYX-138): remove recommended filter

This PR resolves ONYX-138
Description

Before
After

https://github.com/artsy/eigen/assets/36167539/723991a9-215c-4e4e-9774-3a50db490979
https://github.com/artsy/eigen/assets/36167539/bf338026-1f78-4a61-8310-5fad52e7adf0

PR Checklist

[x] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

remove recommended filter from the show page -daria

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (remove recommended filter from the show page -daria)

  Generated by :no_entry_sign: dangerJS against 7418f0ad54e287e78f1c496e8318d09ef5d1ae15


Videos on course pages need closed captioning option available in controls

Steps to Reproduce

Go to a course page. E.g.: https://mitxonline.mit.edu/courses/course-v1:MITxT+JPAL102x/
Click play on the video

Expected Behavior

There should be a closed captioning value (CC) available in the video controls
For reference, look at the video controls on xPRO
Please use the same controls that we use for xPRO

Actual Behavior

There is no closed captioning option on the videos

Related Issues

https://github.com/mitodl/mitxonline/issues/80

PR: https://github.com/mitodl/mitxonline/pull/95

Can this be closed?

Yes, the ticket has made it to the end-user and deployed on prod.


[Snyk] Fix for 4 vulnerabilities

Snyk has created this PR to fix 4 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

package.json

⚠ Warning
Failed to update the package-lock.json, please update manually before merging.

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Asymmetric Resource Consumption (Amplification) SNYK-JS-BODYPARSER-7926860
  696

Cross-site Scripting SNYK-JS-EXPRESS-7926867
  541

Cross-site Scripting SNYK-JS-SEND-7926862
  391

Cross-site Scripting SNYK-JS-SERVESTATIC-7926865
  391

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting

Checkmarx One – Scan Summary & Details – 04120436-8957-422c-ac77-8903585fe10f
New Issues

Severity
Issue
Source File / Package
Checkmarx Insight

CVE-2021-23440
Npm-set-value-2.0.0
Vulnerable Package

CVE-2021-23440
Npm-set-value-0.4.3
Vulnerable Package

CVE-2021-32050
Npm-mongodb-3.5.9
Vulnerable Package

CVE-2021-4264
Npm-dustjs-linkedin-2.5.0
Vulnerable Package

CVE-2022-24441
Npm-snyk-1.278.1
Vulnerable Package

CVE-2022-25881
Npm-http-cache-semantics-4.1.0
Vulnerable Package

CVE-2022-25883
Npm-semver-6.3.0
Vulnerable Package

CVE-2022-25883
Npm-semver-5.7.0
Vulnerable Package

CVE-2022-25883
Npm-semver-5.7.1
Vulnerable Package

CVE-2022-25883
Npm-semver-1.1.4
Vulnerable Package

CVE-2022-25883
Npm-semver-5.5.0
Vulnerable Package

CVE-2022-33171
Npm-typeorm-0.2.24
Vulnerable Package

CVE-2022-48285
Npm-jszip-3.2.2
Vulnerable Package

CVE-2023-26115
Npm-word-wrap-1.2.3
Vulnerable Package

CVE-2023-26136
Npm-tough-cookie-2.4.3
Vulnerable Package

CVE-2023-3696
Npm-mongoose-4.2.4
Vulnerable Package

CVE-2023-42282
Npm-ip-1.1.5
Vulnerable Package

CVE-2023-45133
Npm-babel-traverse-6.26.0
Vulnerable Package

CVE-2023-46234
Npm-browserify-sign-4.0.4
Vulnerable Package

CVE-2024-29415
Npm-ip-1.1.5
Vulnerable Package

CVE-2024-33883
Npm-ejs-0.8.8
Vulnerable Package

CVE-2024-33883
Npm-ejs-1.0.0
Vulnerable Package

CVE-2024-4068
Npm-braces-2.3.2
Vulnerable Package

CVE-2024-4068
Npm-braces-3.0.2
Vulnerable Package

CVE-2024-45296
Npm-path-to-regexp-0.1.3
Vulnerable Package

CVE-2024-45590
Npm-body-parser-1.9.0
Vulnerable Package

MongoDB_NoSQL_Injection
/routes/index.js: 39
Attack Vector

MongoDB_NoSQL_Injection
/routes/index.js: 39
Attack Vector

MongoDB_NoSQL_Injection
/routes/index.js: 191
Attack Vector

MongoDB_NoSQL_Injection
/routes/index.js: 219
Attack Vector

CVE-2017-20162
Npm-ms-0.7.3
Vulnerable Package

CVE-2017-20162
Npm-ms-0.6.2
Vulnerable Package

CVE-2017-20162
Npm-ms-0.7.1
Vulnerable Package

CVE-2022-22984
<EMAIL_ADDRESS>Vulnerable Package

CVE-2022-22984
Npm-snyk-python-plugin-1.16.0
Vulnerable Package

CVE-2022-22984
Npm-snyk-sbt-plugin-2.11.0
Vulnerable Package

CVE-2022-22984
Npm-snyk-gradle-plugin-3.2.2
Vulnerable Package

CVE-2022-22984
Npm-snyk-mvn-plugin-2.7.0
Vulnerable Package

CVE-2022-22984
Npm-snyk-docker-plugin-1.33.1
Vulnerable Package

CVE-2023-0842
Npm-xml2js-0.4.19
Vulnerable Package

CVE-2023-0842
Npm-xml2js-0.4.23
Vulnerable Package

CVE-2023-28155
Npm-request-2.88.0
Vulnerable Package

CVE-2024-29041
Npm-express-4.12.4
Vulnerable Package

CVE-2024-4067
Npm-micromatch-3.1.10
Vulnerable Package

CVE-2024-42459
Npm-elliptic-6.4.1
Vulnerable Package

CVE-2024-42460
Npm-elliptic-6.4.1
Vulnerable Package

CVE-2024-42461
Npm-elliptic-6.4.1
Vulnerable Package

Cx9b9f022f-6dff
Npm-marked-0.3.5
Vulnerable Package

Privacy_Violation
/routes/index.js: 43
Attack Vector

Unpinned Actions Full Length Commit SHA
/codeql-analysis.yml: 57
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/codeql-analysis.yml: 71
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/snyk-code.yml: 15
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/snyk-code.yml: 8
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/snyk-test-sarif.yml: 15
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/codeql-analysis.yml: 46
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/snyk-test-sarif.yml: 8
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/snyk-code-manual.yml: 13
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Fixed Issues

Severity
Issue
Source File / Package

Code_Injection
/routes/index.js: 213

Code_Injection
/routes/index.js: 155

Code_Injection
/routes/index.js: 213

Code_Injection
/routes/index.js: 155

Code_Injection
/routes/index.js: 213

Code_Injection
/routes/index.js: 155

Code_Injection
/routes/index.js: 213

Code_Injection
/routes/index.js: 155

Cx3ec48c7c-8c0c
Npm-handlebars-4.0.11

CVE-2007-2379
Npm-jquery-2.2.4

CVE-2014-6071
Npm-jquery-2.2.4

Cx65603961-769c
Npm-debug-3.1.0

Cx65603961-769c
Npm-debug-4.1.1

Cx65603961-769c
Npm-debug-2.6.9

Cx65603961-769c
Npm-debug-3.2.6

Cx65603961-769c
Npm-debug-2.2.0

Cx73e331a8-9e08
Npm-parse-url-5.0.1

Host Namespace is Shared
/docker-compose.yml: 15

Host Namespace is Shared
/docker-compose.yml: 3

Host Namespace is Shared
/docker-compose.yml: 20

Networks Not Set
/docker-compose.yml: 15

Networks Not Set
/docker-compose.yml: 3

Networks Not Set
/docker-compose.yml: 20

Update Instruction Alone
/Dockerfile: 9

Open_Redirect
/routes/index.js: 213

Open_Redirect
/routes/index.js: 71

Open_Redirect
/routes/index.js: 304

Open_Redirect
/routes/index.js: 91

Open_Redirect
/routes/index.js: 213

Open_Redirect
/routes/index.js: 71

Open_Redirect
/routes/index.js: 304

Open_Redirect
/routes/index.js: 91

Open_Redirect
/routes/index.js: 213

Open_Redirect
/routes/index.js: 71

Open_Redirect
/routes/index.js: 304

Open_Redirect
/routes/index.js: 91

Use_Of_Hardcoded_Password
/routes/index.js: 315

Use_Of_Hardcoded_Password
/routes/index.js: 315

Use_Of_Hardcoded_Password
/routes/index.js: 315

Use_Of_Hardcoded_Password
/routes/index.js: 315


Options flow is broken

voluptuous.error.SchemaError: unsupported schema data type 'Platform'

https://github.com/toreamun/victorsmartkill-homeassistant/releases/tag/2022.2.2


Duplicate count check: on Oracle datasource wrong query to select failed rows

``Hello,
I found a bug relative to selection of failed rows when executing checks on Oracle datasource.
Soda builds the query using LIMIT keyword, but it doesn't exist in Oracle.
Query error: oracle_beneficiario_with_connstring_dev.MYTABLE.duplicate_count[MYCOLUMN].failed_rows.aggregated: ORA-00933: SQL command not properly ended

WITH frequencies AS (
    SELECT MYCOLUMN, COUNT(*) AS frequency
    FROM MYTABLE
    WHERE MYCOLUMN IS NOT NULL
    GROUP BY MYCOLUMN)
SELECT *
FROM frequencies
WHERE frequency > 1
ORDER BY frequency DESC
LIMIT 10

The correct query should use the ROWNUM keyword:
WITH frequencies AS (
    SELECT MYCOLUMN, COUNT(*) AS frequency
    FROM MYTABLE
    WHERE MYCOLUMN IS NOT NULL
    GROUP BY MYCOLUMN)
SELECT *
FROM frequencies
WHERE frequency > 1
  AND ROWNUM <= 10
ORDER BY frequency DESC

Hi @stelapo! Thanks for letting us know. We're happy to accept any contribution to the project, so if you're already aware of the solution, feel free to open up a PR and we'll help you through getting it merged.
If not we'll of course try to fix it at some point. :)

Fixed in #2101

Hi,
this issue has been closed, but the problem is still present. I tried with the last version 3.3.9
Soda builds the query to select failed rows samples using LIMIT keyword, but it doesn't exist in Oracle.
Thank you


Provide execute permissions to others

Other users in the system doesn't have execute permissions for docker-compose.

this kind of defeats the purpose of the umask '0027'. Might be better to adapt that (or set with an attribute to allow override)

@hrak thanks for the suggestion and incorporated review comment.


[Question] Why does this not work?

Question

i get the error that the variable {database{target.name}} doesnt exist
Plugin Version
latest
Additional Information
No response

Please remove the outer braces like 'database{target.name}'.

thank you 👍


7.1.8 - Surface_Laptop_3_1867_1868

Unfortunately you can't include : (colon in the path name). This needs to be corrected.

Having the same problem Surface Laptop 4 drivers. Both Win10 & Win11 drivers

Fix coming in the morning.

Hotfix inbound shortly

Hotfix here - https://github.com/maurice-daly/DriverAutomationTool/blob/master/Current Branch/7.1.8/DriverAutomationTool.exe

Hotfix here - https://github.com/maurice-daly/DriverAutomationTool/blob/master/Current Branch/7.1.8/DriverAutomationTool.exe

Thanks, able to download now. Have a great day!


Code has multiple issues

In the following block, I'm not sure, but the !wpsContext.hasError seems flipped from what it should be. Additionally, it doesn't explain what the protocol for the return is. What values are accepted here and what is the recipient? Additionally, the logging is to the console and not the context, (e.g., context.log), which means that the logs will likely only appear somewhere the developer isn't looking. There also isn't a good explanation of how to write this method correctly. There is a vague mention of Abuse Protection, but the example doesn't really line up with the documentation on abuse protection.
module.exports = async function (context, req, wpsContext) {
  if (!wpsContext.hasError || wpsContext.isPreflight)
  {
    console.log(`invalid request: ${wpsContext.response.message}.`);
    return wpsContext.response;
  }
  console.log(`user: ${wpsContext.connectionContext.userId} is connecting.`);
  return { body: {"userId": wpsContext.connectionContext.userId} }; // no explanation of what is allowed in the return here or who is receiving it
};

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

ID: 0c4c897d-50cd-34e9-c2e7-7ca235090b8c
Version Independent ID: 65d9edff-959a-e5d7-1164-947261bb592a
Content: Reference - Azure Web PubSub trigger and bindings for Azure Functions
Content Source: articles/azure-web-pubsub/reference-functions-bindings.md
Service: azure-web-pubsub
GitHub Login: @vicancy
Microsoft Alias: lianwei

@oadrian2  Thanks for the feedbacks. I'll fix the logging and checks.
Take a look at the picture from in the beginning, you would notice WebPubSubContext is a wrapper of HttpTrigger to response Web PubSub service requests related to customer's operations, so it's return service. In this particular sample, it's responding the connect event and the response is to assign the connection a userId. See here for more details. And the AbuseProtection is explained in the same page.
This article is about how to write with Azure Functions, so probably not suitable to help you understand Web PubSub concepts. Suggest check an E2E sample for better understanding:

Chat app with Static Web Apps
Chat app with Function Apps

@oadrian2
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

The code sample up on GitHub has the same issues and should likely be corrected as well.
See: https://github.com/Azure/awps-swa-sample

Do you refer to log method? It was for local debug purpose. And I'll scan all samples to have an update.

Also, there doesn't appear to be a message property on the response object.
    console.log(`invalid request: ${wpsContext.response.message}.`);

The code sample up on GitHub has the same issues and should likely be corrected as well.
See: https://github.com/Azure/awps-swa-sample

Do you refer to log method? It was for local debug purpose. And I'll scan all samples to have an update.

I was referring to the flipped hasError check

Also, there doesn't appear to be a message property on the response object. See the response object below. I'm also not sure what that line would even do. If it's based on the C# transport object, then it lines up closer to the simple response object, which doesn't have a message property. What is the response object on the context a response to?
    console.log(`invalid request: ${wpsContext.response.message}.`);

I'm not able to find this piece of code. I'm wondering there's some sample snippets missed to get updated between our preview version and final version. The final version uses wpsContext.response.body. Would you help point the location to let me provide an update?
    console.log(`invalid request: ${wpsContext.response.message}.`);

If you're interest in the original object definition. You may check WebPubSubContext and the customized response json converter

The code sample up on GitHub has the same issues and should likely be corrected as well.
See: https://github.com/Azure/awps-swa-sample

Do you refer to log method? It was for local debug purpose. And I'll scan all samples to have an update.

I was referring to the flipped hasError check (and now also returning the wpsContext.response value or printing a non-existent message property, which seems nonsensical).

Suggest to return wpsContext.response directly which is the pre-build http response in node version. And log body is simply for easier debug.

The code sample up on GitHub has the same issues and should likely be corrected as well.
See: https://github.com/Azure/awps-swa-sample

Do you refer to log method? It was for local debug purpose. And I'll scan all samples to have an update.

I was referring to the flipped hasError check (and now also returning the wpsContext.response value or printing a non-existent message property, which seems nonsensical).

Suggest to return wpsContext.response directly which is the pre-build http response in node version. And log body is simply for easier debug.

Won't the body be '' always at that point since it's an empty response object? Also, shouldn't the response be something other than 200 OK since it's a failure state? According to the docs you sent, it seems that the proper failure response is 401 Not Authorized. You could make the argument for 500 as well, but that's not specified as an official option.

There is an errorMessage property on response objects when the type is EventErrorResponse. When is the response object on the WebPubSubContext populated? What populates it or should be populating it?

The code sample up on GitHub has the same issues and should likely be corrected as well.
See: https://github.com/Azure/awps-swa-sample

Do you refer to log method? It was for local debug purpose. And I'll scan all samples to have an update.

I was referring to the flipped hasError check (and now also returning the wpsContext.response value or printing a non-existent message property, which seems nonsensical).

Suggest to return wpsContext.response directly which is the pre-build http response in node version. And log body is simply for easier debug.

Won't the body be '' always at that point since it's an empty response object? Also, shouldn't the response be something other than 200 OK since it's a failure state? According to the docs you sent, it seems that the proper failure response is 401 Not Authorized. You could make the argument for 500 as well, but that's not specified as an official option.

When it's invalid request, the response isn't empty as it's the purpose of this function extension to help customer build a proper response in advance, otherwise Customer would have to build the response(include status code, body, header) themselves, especially for the preflight has a protocol about response. And customer check the hasError property to know whether it's a valid request easily.

There is an errorMessage property on response objects when the type is EventErrorResponse. When is the response object on the WebPubSubContext populated? What populates it or should be populating it?

It works for WebPubSubTrigger. Please check here. Both WebPubSubTrigger and WebPubSubContext works for the service request. The difference is that WebPubSubTrigger is the trigger binding and WebPubSubContext is the input binging based on HttpTrigger. WebPubSubContext works for static web apps that only HttpTrigger is supported trigger.
https://learn.microsoft.com/en-us/azure/azure-web-pubsub/reference-functions-bindings?toc=%2Fazure%2Fazure-functions%2Ftoc.json&tabs=javascript#return-response

As an example of some test WPS code (albeit in V4 model), I came up with this:
app.post('message', {
  extraInputs: [webPubSubContextInput],
  extraOutputs: [webPubSubActionsOutput],
  handler: async (_request, context) => {
    const wpsContext = context.extraInputs.get(webPubSubContextInput) as WebPubSubContext;

    if (wpsContext.hasError) return wpsContext.response;

    const request = wpsContext.request as UserEventRequest;

    context.extraOutputs.set(webPubSubActionsOutput, {
      actionName: 'sendToAll',
      data: request.data,
      dataType: request.dataType,
    });

    return wpsContext.response;
  },
});

I'm not sure in what contexts I would get isPreflight: true. It seems it's only used for validate.

It's related to the routing configure.

If your upstream is set like: https://<server-endpoint>/api/{event}, then preflight will send to https://<server-endpoint>/api/validate.
If your upstream is set like: https://<server-endpoint>/api/events, then first request would be preflight.

Also see doc here: https://learn.microsoft.com/en-us/azure/azure-web-pubsub/howto-develop-eventhandler#upstream-and-validation

This looks like it has been resolved. Closing! Feel free to request reopen if this is still being worked on. #please-close


Preprocessing.md: fix typo

Hello @blancadesal!
Thanks for the contribution.
To modify to the tutorials, you should edit the ipynb notebook then generate the updated markdown files as specified here.
Probably it was not very clear. @bilgeyucel is currenty improving the contributing guide (https://github.com/deepset-ai/haystack-tutorials/pull/119).

Oh, ok! Thanks for the link. I've amended my commit; however, when I ran pre-commit, black-jupyter and md version of tutorials formatted a whole bunch (20+) of files. Was that to be expected? It feels wrong to include them in this PR as they are unrelated to the typo fix.

It seems strange. 🤔
I think you should only include the ipynb notebook and the generated markdown in this PR.

@TuanaCelik @bilgeyucel this PR seems good to me.
Please approve the workflows to run checks.
:heart_decoration:

however, when I ran pre-commit, black-jupyter and md version of tutorials formatted a whole bunch (20+) of files. Was that to be expected? It feels wrong to include them in this PR as they are unrelated to the typo fix.

Hmm, it does seem like this is the only way to make CI pass though, at least locally.

Hey @blancadesal, thank you for your contribution! I had to checkout to your branch to update the last_updated timestamp one last time. This problem is basically because of a bug in our workflows, but we're working to solve it so this problem doesn't happen again. Again, thank you, and this PR is merged! 💯 🚀

Thank you for sorting this out @bilgeyucel !


[META] Repo size over 1GB

Is your feature request related to a problem? Please describe.
not exactly a feature request, but i noticed the repo is over 1 GB right after cloning, which feels pretty big for a docs project. maybe it comes from large files or old artifacts hanging around in the .git history.

Describe the solution you'd like
i think maybe a history cleanup to get rid of any big or outdated files could be worth it.
Additional context
i checked for large files in the repo history, and here’s what came up:
git rev-list --objects --all | git cat-file --batch-check='%(objecttype) %(objectname) %(objectsize) %(rest)' | sed -n 's/^blob //p' | sort --numeric-sort --key=2 | cut -c 1-12,41- | numfmt --field=2 --to=iec | tail -n 10

8d775968d852    9.3M website/build/vi/ts.worker.js
f7d42abd0ce4    9.3M website/build/zh-CN/ts.worker.js
8a3b535b2b91      10M docs/assets/ledger-mobile.gif
9873bfdbc146      12M website/build/assets/images/drawing-da1faed841bf6cdd04ef58ac661fc935.svg
b6bc8ffaf007      14M website/build/assets/images/drawing-26c4dc1344e7ded8ff6d12b6652e6788.svg
1c5372b39452      15M website/build/assets/images/drawing-c60718c18458bad7704e13baf87fb838.svg
bde38a8768dc      15M website/static/docs/assets/welcome-pages/drawing.svg
3a29c7e23132      20M website/build/assets/images/drawing-9dfd2072c04c4601eb4a98aa5b13a8ec.svg
a2d06f34d776      20M website/static/docs/assets/welcome-pages/drawing.svg
373540bf40ea      25M website/static/docs/assets/QAPIScreen.gif

most of these files aren’t in the main repo anymore, they’re still in .git and adding to the size. the only large file still in use is drawing.svg (20 MB), which is still too big for the web ideally it’d be around 50 or 200KB max.
also, when i checked dir sizes at root, .git alone was already 1 GB 👀
du -sh * .[^.]* | sort -h

4.0K	.gitignore
4.0K	CODE_OF_CONDUCT.md
4.0K	CONTRIBUTING.md
4.0K	LICENSE-MIT.txt
4.0K	LOCALIZATION.md
4.0K	README.md
4.0K	yarn.lock
 12K	LICENSE-APACHE.txt
 48K	.github
 80K	blog
3.0M	docs
125M	website
1.0G	.git

feel free to dive in further!

I think this is because we are deploying the docs site to the build branch. If you only checkout the master branch you shouldn't get 1 GB download.

you shouldn't get 1 GB download

hi @bucanero
just tested out cloning only master branch like suggested, turns out size is only 203.63 MB✅
and not 1 GB. so seems like the repo should be fine as is. thx again for taking the time to look into this!

hi @bucanero just tested out cloning only master branch like suggested, turns out size is only 203.63 MB✅ and not 1 GB. so seems like the repo should be fine as is. thx again for taking the time to look into this!

thanks for reporting your results and reporting this issue. 👍
To improve the experience for other users and contributors, I have changed the building process so the repository no longer has a build branch. So now you can clone the full repository, and you'll get only the 203 Mb download:
git clone https://github.com/near/docs
Cloning into 'docs'...
remote: Enumerating objects: 29120, done.
remote: Counting objects: 100% (1212/1212), done.
remote: Compressing objects: 100% (710/710), done.
remote: Total 29120 (delta 695), reused 862 (delta 486), pack-reused 27908 (from 1)
Receiving objects: 100% (29120/29120), 203.46 MiB | 12.57 MiB/s, done.
Resolving deltas: 100% (19013/19013), done.


Parsing for composite meshes

What is a composite mesh in this context?

What is a composite mesh in this context?

It is the abutting mesh. The UWyo PR called it abutting/composite meshes, so I chose to refer to it as a composite mesh in the exawind-driver.

I have just never heard them called composite meshes before, just abutting. When I hear composite I think like arbitrary elements. Composite is a much broader word than abutting so I would prefer abutting. But I'm not opposed enough to request a change. I can adjust my vernacular as well.

I will change it to abutting then.


[BERT] BERT Tokenizer PR followups

[ ] Add a de-tokenizer API which returns the original string representation given tokens
[ ] Replace existing tokenization code in script/bert with the new tokenizer API

ping @kenjewu if you are interested

ok, i will try to finish it and #492 .


Add text-based UI for hammer

This required some refactoring to create an object tree to keep references to all of the components that had previously been let loose in their own goroutines without any references.
This provides a key handler for increasing/decreasing ops/s (using +/-).

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 60.26%. Comparing base (07aa77d) to head (819f2f0).
Report is 28 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #114      +/-   ##
==========================================
+ Coverage   55.99%   60.26%   +4.26%
==========================================
  Files           8        8
  Lines         759      609     -150
==========================================
- Hits          425      367      -58
+ Misses        265      173      -92
  Partials       69       69

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


Must specify the name in the CustomMetadataType.CustomMetadata format Error

This issue a is copy of this question which I've posted on Salesforce Stack Exchange. I've also posted the question in the Custom Metadata Types Sucess Group here. I'll update this issue if someone posts a solution on either site.
I've just deployed the Custom Metadata Loader app to my sandbox and am trying to load my first batch of records for my Custom Metadata Type Interchange_Table__mdt.
When I select the CSV containing my records, the Metadata Type from the type picklist and click Create custom metadata, I get the following error

Web service callout failed: WebService returned a SOAP Fault: Must specify the name in the CustomMetadataType.CustomMetadata format, Name:Interchange_Table., Delimiter:. faultcode=soapenv:Client faultactor= Error is in expression '{!createCustomMetadata}' in component in page custommetadatarecorduploader:
Class.MetadataService.MetadataPort.createMetadata: line 9254, column 1 Class.MetadataUtil.createMetadataAndValidate: line 116, column 1 Class.MetadataUtil.transformToCustomMetadataAndCreate: line 83, column 1 Class.CustomMetadataUploadController.importCSVFileAndCreateCmdRecords: line 120, column 1 Class.CustomMetadataUploadController.createCustomMetadata: line 45, column 1

My spreadsheet looks like this

and I can't see any significant difference between this and the sample data which is included in the Github repository which looks like this so I can't work out what's causing the error?

The label in your sample has a lot of dots and since the app tries to default that to the dev name, it fails. You can add a name column to your csv and it should work.


test: fix cilium policy action for clusterwide policies

Fixes #13378

test-focus K8sServicesTest

test-focus K8sServicesTest

test-me-please

I wonder why this doesn't affect the other host firewall test (for fromCIDR+toPorts) thinking

Yeah, I wonder about this too, but honestly, I don't have a really good answer for this. My guess would be that something changed in the test env for Services.go that lead to some request failing when applying policies as we started seeing this fail only recently.
Do you have ideas around how a policy addition/removal dictates traffic, as in are there any unnecessary/expected drops when the policy is recalculated and updated(with and without host policies)?

Did you check if we have other tests using CCNPs that may be affected?

No, I didn't check any other tests related to CCNP. :sweat_smile:
Maybe test-me-please run can give us some info around them. But we haven't seen any tests related to CCNP failing apart from this in recent times.


up pillow for compatibility with python3.10

Python 11 has been already released. I guess we need to update deps to support the latest version too)

i've just do this fix to get this script working on my laptop)


Updated MAINTAINERS.md format.

Coming from https://github.com/opensearch-project/.github/issues/121, updated MAINTAINERS.md to match opensearch-project recommended format.

@dblock could you take care of the changelog?
Looks like 1.0.1 integration tests are also failing, we could take care of that in a different PR.

We can skip changelog for this one, thx.

@saratvemulapalli Care to open an issue for the failure?


Can't pick photo from google photos, can it possible?

I was tried to pick a photo from Google photos but it generates an error

@vitulgoyani hi. Would you mind post your logs? Also which version from the picker are you using?
Just make sure beforehand that you’ve internet connection and the internet permission in your iOS/Android plist/manifest file respectively.

I/flutter (14598): [FilePicker] Unsupported operation. Method not found. The exception thrown was: NoSuchMethodError: The method 'split' was called on null.
I/flutter (14598): Receiver: null
I/flutter (14598): Tried calling: split("/")
And I used the latest version from git

it was working fine with google drive but not work with google photos

@vitulgoyani  looks like a dup of #58 that should have already been fixed. What are the steps to replicate the issue?

@vitulgoyani is this still a problem for you?

Closing due to no further info. Feel free to reopen it if you still experience the same issue.


torsocks error

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[ ] I've searched the bugtracker for similar questions including closed ones

Question

WRITE QUESTION HERE
Hi, i am trying to use youtube-dl with torsocks to allow myself to download videos anonymously.  However i can not get it to work. What is the correct way to do so? thanks

--proxy socks5://...

thanks it works using installed tor service, but hangs in case of tails OS. Is that normal?

I used ip address and port provided in torrc file


Fail on invalid config

what

remove || statement

why

Causes this action to succeed when in fact it has failed

@Benbentwo can you resolve the merge conflict?


Use latest marketplace image in master branch

Always use latest image when deploying master branch pai
Use specific version of marketplace image in each release version

Coverage remained the same at 33.933% when pulling 26aa0b87a417d7c035b80c8383dad6dbb280a54b on mintao/marketplace-image into 9d7c1aca76269d61e36ab46feca1d667a64154e1 on master.

Coverage remained the same at 33.933% when pulling 26aa0b87a417d7c035b80c8383dad6dbb280a54b on mintao/marketplace-image into 9d7c1aca76269d61e36ab46feca1d667a64154e1 on master.


Adds ripgrep to documentation. Closes #2491

I just merged #2493, which I think is possibly a better approach, since a lot of the time people don't read documentation 😅


Create BubbleSort.py

Other sort solutions coming

@prateekiiest well you see, python gives a really less clustered code, but the problem with it is that for someone who is looking at these codes, is highly unlikely to know python itself. A C or C++ implementation would be great as in IIEST, thats the first thing people are taught. And of-course someone in third or fourth year, likely knowing python, wont be looking for bubble sort or binary search implementations. Infact, in contests we mostly use STL for these purposes.

Ok, I will implement the rest in C

It will do good if a seperate readmes for each algorithm folders are created. There we can upload the videos, links
Specific resources for specific algos

Look at the gh-pages branch on the repo @prateekiiest
I have almost completed creating the basic template, for a blog , that will classify algos under some sections like Sorting, graphs, dp, etc. and there you can upload markdown or videos or pdfs whatever. I've almost completed the structure in my laptop, which I'll be pushing probably tomorrow, as I got a little busy with other stuff. You can clone that and enhance the site

Sure
On Dec 26, 2016 9:37 PM, "Soumit Bose"<EMAIL_ADDRESS>wrote:

Look at the gh-pages branch on the repo @prateekiiest
https://github.com/prateekiiest
I have almost completed creating the basic template, for a blog , that
will classify algos under some sections like Sorting, graphs, dp, etc. and
there you can upload markdown or videos or pdfs whatever. I've almost
completed the structure in my laptop, which I'll be pushing probably
tomorrow, as I got a little busy with other stuff. You can clone that and
enhance the site
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeIIEST/Algorithms/pull/12#issuecomment-269223185,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APSucpqNtcDLBwOErA3x_ch3tKYbgOkJks5rL-Y_gaJpZM4LVhYl
.


Resource Form Page interface updates

~"Local Generation Only" - if this is going in the production version, then we should come up with a different title because to a teacher this probably wont mean much~
[ ] "Generate Resource" button needs styling to match the rest of the site

Regarding the top one, see this part of the template:
    {% if debug %}

Local Generation Only

      <fieldset>
        <legend>Header Text</legend>
        <input type="text" name="header_text" placeholder="Example School: Room Four" class="long-text-field">
      </fieldset>

      {% if resource.copies %}
        <fieldset>
          <legend>Number of Copies</legend>
          <input type="number" name="copies" value=1 min=1 max=50>
        </fieldset>
      {% endif %}

      <input type="submit" value="Generate Resource" class="mb-3" />
    {% else %}
      {% if resource.copies %}

        The download of this resource includes 30 unique copies.

      {% endif %}

      <input type="submit" value="Download Resource" class="mb-3"/>
    {% endif %}

If it is local (i.e. debug), then it displays the local generation header text, displays the copies option, and a button with 'Generate'.
If it's production, then it displays alert stating it's 30 copies (if the resource allows copies), and a button with 'Download'.
Is this what you were implying?

Have removed the first item as it was already solved.


fix: replacing macro {napi_build_version} in binary filename

fix: replacing macro {napi_build_version} in binary filename by using binary.napi_build_versions in package.json. Fixes: #554
Added unit test for offending module: "sqlite3": "5.1.6"
Tested on mac for arm64 and x64 rebuilds
Related:
https://github.com/electron-userland/electron-builder/issues/7473
Original PR credit to https://github.com/electron/rebuild/pull/1076 (I just added tests as requested)

Friendly bump. @MarshallOfSound.


removing dependency on cluster centers k in autoencoder.py and adding…

… new latent dimension variable ld

Merged commits into a single one as requested.

lgtm


Cleaned up format and removed some unneeeded castings.

Changed some stuff, suggest testing before merging.

While I appreciate your effort for code cleanup, I don't see removing castings as significant enough to merge with the main branch. If there is something else I missed, please respond here.

Aw :C


Update RPMI spec

Update Fast-channel description and some chapters.

There are few nits, but I also have a PR coming so i can take them up


Getting host/port is wrong on configuring eufy web socket instance

Installed Eufy through HACS
While setting up integration, getting host/port is wrong error on configuring eufy web socket instance with options 'localhost' and '3000'

This is the log
2022-02-08 13:31:10 ERROR (MainThread) [custom_components.eufy_security.config_flow] eufy_security Exception in login : Cannot connect to host <IP_ADDRESS>:3000 ssl:default [Connect call failed ('<IP_ADDRESS>', 3000)] - traceback: Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/aiohttp/connector.py", line 986, in _wrap_create_connection
return await self._loop.create_connection(*args, **kwargs)  # type: ignore[return-value]  # noqa
File "/usr/local/lib/python3.9/asyncio/base_events.py", line 1056, in create_connection
raise exceptions[0]
File "/usr/local/lib/python3.9/asyncio/base_events.py", line 1041, in create_connection
sock = await self._connect_sock(
File "/usr/local/lib/python3.9/asyncio/base_events.py", line 955, in _connect_sock
await self.sock_connect(sock, address)
File "/usr/local/lib/python3.9/asyncio/selector_events.py", line 502, in sock_connect
return await fut
File "/usr/local/lib/python3.9/asyncio/selector_events.py", line 537, in _sock_connect_cb
raise OSError(err, f'Connect call failed {address}')
ConnectionRefusedError: [Errno 111] Connect call failed ('<IP_ADDRESS>', 3000)
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/config/custom_components/eufy_security/config_flow.py", line 191, in _test_credentials
await eufy_ws.connect()
File "/config/custom_components/eufy_security/websocket.py", line 40, in connect
self.ws: aiohttp.ClientWebSocketResponse = await self.session.ws_connect(
File "/usr/local/lib/python3.9/site-packages/aiohttp/client.py", line 776, in _ws_connect
resp = await self.request(
File "/usr/local/lib/python3.9/site-packages/aiohttp/client.py", line 535, in _request
conn = await self._connector.connect(
File "/usr/local/lib/python3.9/site-packages/aiohttp/connector.py", line 542, in connect
proto = await self._create_connection(req, traces, timeout)
File "/usr/local/lib/python3.9/site-packages/aiohttp/connector.py", line 907, in _create_connection
_, proto = await self._create_direct_connection(req, traces, timeout)
File "/usr/local/lib/python3.9/site-packages/aiohttp/connector.py", line 1206, in _create_direct_connection
raise last_exc
File "/usr/local/lib/python3.9/site-packages/aiohttp/connector.py", line 1175, in _create_direct_connection
transp, proto = await self._wrap_create_connection(
File "/usr/local/lib/python3.9/site-packages/aiohttp/connector.py", line 992, in _wrap_create_connection
raise client_error(req.connection_key, exc) from exc
aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host <IP_ADDRESS>:3000 ssl:default [Connect call failed ('<IP_ADDRESS>', 3000)]

can you share the logs of add-on?

I am also having this issue. My add-on logs are below. Dont worry about T81. Thats a solo cam. Also, for context, I do have z-wave JS installed. So I am setting the eufy add-on to port 3001.
2022-02-08 23:18:47.528  INFO  Eufy Security server listening on host <IP_ADDRESS>, port 3000
2022-02-08 23:18:49.236  INFO  Connected to station T8010P1321290ABC on host <IP_ADDRESS> and port 29706
2022-02-08 23:18:49.663  INFO  Push notification connection successfully established
2022-02-08 23:18:49.783  INFO  Connected to station T8123P1021350A81 on host <IP_ADDRESS> and port 22159
2022-02-08 23:18:56.327  ERROR Status return code not 200
{
status: 423,
statusText: 'Locked'
}
2022-02-08 23:18:56.332  INFO  No stations found.
2022-02-08 23:19:04.838  INFO  Initiated closing of connection to station T8123P1021350A81 for saving battery.
2022-02-08 23:19:04.840  INFO  Disconnected from station T8123P1021350A81
Here is my HA log for the integration:
2022-02-08 18:15:41 WARNING (MainThread) [homeassistant.config_entries] Config entry 'localhost' for eufy_security integration not ready yet: Exception: Host/port is not reachable localhost 3000!; Retrying in background

can you share the logs of add-on?

Where can I find it? I checked supervisor log and there's nothing..

I think I'm having similar issue. One day, all cameras in HA has gone.
I deleted HACS integration and ws addon and re-installed them.
My addon logs are as follows:
2022-02-11 23:08:41.863  INFO  Eufy Security server listening on host <IP_ADDRESS>, port 3000
2022-02-11 23:08:48.846  INFO  Push notification connection successfully established
2022-02-11 23:08:49.056  INFO  No stations found.
2022-02-11 23:08:50.212  INFO  No devices found.

You should check here: https://github.com/bropat/eufy-security-ws/issues/74

Please get latest version of addon and try again

Please get latest version of addon and try again

I tried with the latest version and i'm still getting the same issue.. there's nothing in supervisor log..

Tried with the latest version. Re-installed everything. Working now.

Issue is not resolved yet.. still showing host/port is wrong error when configuration starts


JSlider single-pin fix

Modifying jslider to fix single-pin assignment issue.

Confirmed working by @TheTrainGoes


Development: Refactor exercise import components

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines and ensured that the layout is responsive.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I documented the TypeScript code using JSDoc style.

Motivation and Context
The different import components in the client were all just copy-and-paste components.
Description
This PR refactors the import components to use a common abstract superclass.
This PR focusses on the exercise imports, other components like LectureImportComponent will be part of a followup.
I directly included a small bugfix: It was not possible to sort by exam title, trying this would lead to an empty search result.
Steps for Testing
Check that all exercise import components still work:
Prerequisites:

1 Instructor
1 Course with exercises of each type

Navigate to Course Management  -> Course -> Exercises
Import exercises of each type and make sure the import still works.
Play around with the different sorting options and the course / exam filter and make sure that everything works.

Exam Mode Testing
Prerequisites:

1 Instructor
1 Exam

Navigate to the exam management page
Check that importing exercises into the different exercise groups of the exam still works (for all exercise types)

Play around with the different sorting options and the course / exam filter and make sure that everything works.

Review Progress
Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test

Codecov Report

Merging #6026 (6df1ad9) into develop (b6f6ae5) will increase coverage by 3.99%.
The diff coverage is 77.77%.

:exclamation: Current head 6df1ad9 differs from pull request most recent head 1e2826c. Consider uploading reports for the commit 1e2826c to get more accurate results

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #6026      +/-   ##
=============================================
+ Coverage      75.98%   79.98%   +3.99%
- Complexity         0    12650   +12650
=============================================
  Files           1151     2181    +1030
  Lines          45847    86689   +40842
  Branches        8593    12974    +4381
=============================================
+ Hits           34839    69341   +34502
- Misses          5711     9637    +3926
- Partials        5297     7711    +2414

Impacted Files
Coverage Δ

...ercises/shared/import/exercise-import.component.ts
82.14% <64.10%> (ø)

...n/java/de/tum/in/www1/artemis/domain/Exercise.java
85.52% <100.00%> (ø)

...anage/exercise-groups/exercise-groups.component.ts
82.00% <100.00%> (+1.32%)
:arrow_up:

...ses/modeling/manage/modeling-exercise.component.ts
94.91% <100.00%> (+0.17%)
:arrow_up:

...ogramming/manage/programming-exercise.component.ts
87.37% <100.00%> (+0.24%)
:arrow_up:

...p/exercises/quiz/manage/quiz-exercise.component.ts
89.74% <100.00%> (+0.17%)
:arrow_up:

...xt/manage/text-exercise/text-exercise.component.ts
91.83% <100.00%> (+0.34%)
:arrow_up:

...webapp/app/grading-system/bonus/bonus.component.ts
82.39% <100.00%> (ø)

...ge/services/programming-exercise-paging.service.ts
62.50% <0.00%> (-12.50%)
:arrow_down:

...n/www1/artemis/repository/GradeStepRepository.java
100.00% <0.00%> (ø)

... and 1033 more

Flag
Coverage Δ

client
75.92% <77.41%> (-0.07%)
:arrow_down:

server
84.51% <100.00%> (?)

Continue to review full report at Codecov.

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


Add cake to homebrew

Would love to run a brew command to install this app!
brew install cake-build

Created from a comment coming from this thread

We already publish Cake to home brew:
https://github.com/Homebrew/homebrew-core/blob/master/Formula/cake.rb
😄

It's already on brew as formula cake

You can find more information here:
https://formulae.brew.sh/formula/cake

@nitrocode I am going to go ahead and close this, as I believe that this is covered.  Let us know if you have any follow up.


Fix enabling of OGRE_GLSUPPORT_USE_EGL_HEADLESS

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
The OGRE_GLSUPPORT_USE_EGL_HEADLESS is enabled in https://github.com/conda-forge/ogre-next-feedstock/blob/f79e06a8e17d003a68db271467fe87e8aa84142d/recipe/build.sh#L53, but as EGL is not listed as dependency, I do not think this is actually working, and indeed EGL is not listed as a linked dependnecy on any ogre plugin.
Installed packages
.

Environment info
.

Probably we can enabling this while dropping CDTs, to avoid the problem described in https://github.com/conda-forge/opencv-feedstock/issues/401 .


A Question about public databases

Is a token required to access the public database of a CKContainer? I ask only shared and private databases are mentioned

Yeah public databases aren't supported yet. I'll add that to the roadmap.

thanks for clearing that up

Just to circle back, this should work though I haven't tested using the standard web auth via Apple ID method but also via a Server-to-Server token. I'm not sure I support the server-to-server token yet. So if you want to test that out feel free but the current Apple ID method should still work. Obviously with a Public db that seems unnecessary hence the server-to-server token.
If you are interested, you can take look at how both methods work here:
https://developer.apple.com/library/archive/documentation/DataManagement/Conceptual/CloudKitWebServicesReference/SettingUpWebServices.html#//apple_ref/doc/uid/TP40015240-CH24-SW6

Hi Leo,
Thanks for following up about this. Correct me if I’m wrong (please!) but the documentation you sent appears to suggest that I may be able to communicate with the public database without the need for Apple ID authentication (server token only). Do you concur? This would be perfect for what I’m trying to do. The reason I found MistKit is because I need CloudKit access from an app clip and those don’t support it yet. Up until I saw the documentation you shared with me I was under the impression that this would be impossible without having the user sign in to their Apple ID from the app clip, which would be nuts in terms of user experience. Anyway, if the train of thought I lay out here is feasible, I’d love to help you extend MistKit to accommodate AppClip island hopping to CloudKit

On Nov 1, 2021, at 5:20 PM, leogdion @.***> wrote:

Just to circle back, this should work though I haven't tested using the standard web auth via Apple ID method but also via a Server-to-Server token. I'm not sure I support the server-to-server token yet. So if you want to test that out feel free but the current Apple ID method should still work. Obviously with a Public db that seems unnecessary hence the server-to-server token.
If you are interested, you can take look at how both methods work here:
https://developer.apple.com/library/archive/documentation/DataManagement/Conceptual/CloudKitWebServicesReference/SettingUpWebServices.html#//apple_ref/doc/uid/TP40015240-CH24-SW6
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.


tty: this setting doesn't get set in service

v0.3.0
Enabling a service with this yaml code doesn't work:
tty: true

When I do a docker inspect on the container created with the yaml file, the container shows:
"Tty": false

It's not enabled in the most recent commit related to service.
https://github.com/rancherio/os/commit/b2ece712376e72069f15d85fe8bf413922b6bd32

Verified fix in v0.3.1-rc1


I11 - Add a general Email module

Module can be used to send emails at various stages of a workflow. Credentials are provided by local file, and email transmission is triggered by the presence/arrival of a trigger-file which can be user configured.
To test the module:

Pull down this branch
Create a 'results/credentials/credentials.json' file (in accordance with the workflow/Snakefile docstring); note that this replicates a separate module which would like to a local file during normal execution.
run snakemake --use-cores 1 --use-conda

To test the module in grapevne:

Obtain and build the latest version of GRAPEVNE
Pull down this branch
redirect grapevne to the workflows folder in this repo (use the local copy, since the module had not yet been merged in)
connect the module to some Local file modules in order to provide credential and trigger files.

I've updated the module to permit glob-matched email attachments, and to support monitoring of successive email 'triggers' to permit multiple sends (these are useful when monitoring long jobs); there is a test config file with short timeouts that can be run as snakemake --cores 4 --configfile config/config_test.yaml.

Squashed the last two commits to remove email from the commit history.

I know this has already been squashed and merged but I have added two relatively minor comments and, also, try as I might I can't get it to work in GRAPEVNE

@rowannicholls Thanks for taking a look at this - I think you've found an issue that was initially overlooked. Although everything seemed to work on my test-bed, trying it on a clean install did raise an unexpected issue when testing in GRAPEVNE (the email standard library module was not being bundled with the packaged python/snakemake environment). I will add this to the new PR.
In the meantime (and just for clarity) to test the module in grapevne:

Follow the setup instructions in the Email module's docstring to create an 'app password'. You will need the username and password saved into your credentials.json file.
Use the LinkLocalFile module to provide the credentials file to the Email module.
Connect the same LinkLocalFile module to the 'send' port (you can use it as a trigger by changing the 'send email' trigger filename in the Email module paramers, or use the new Touch module)

Here's a screenshot of my setup:


📃: Add Snake Game

:red_circle: Title : Snake Game
:red_circle: Aim : Made for fun and exploring 2D game development in Python
:red_circle: Brief Explanation : In this game, you control a snake that grows longer every time it eats food, and you have to avoid crashing into the walls or the snake's own body.
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name : Ishant Sahu
What is your participant role? GSSOC-EXT , Hacktoberfest

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Hi @UTSAVS26,
I had like to work on this issue. I have experience with python and I believe I can contribute effectively.

Hi @UTSAVS26,
I had like to work on this issue. I have experience with python and I believe I can contribute effectively.

You can raise another issue as this one is already assigned.


Uploder on mobile not working

Uploder on iOS is not working

Related: https://github.com/react-dropzone/react-dropzone/issues/1196

Is it iOS or Safari in general?  https://stackoverflow.com/questions/76486986/react-dropzone-ondrop-is-not-getting-invoked-in-safari

It was tested on Chrome and Safari on iOS version 17.5.1. Both are unresponsive.


Added few more queries to utlization card

@rawagner Please review.

@rawagner Incorporated all the review comments. Please review.

besides the unnecessary comma, LGTM

@rawagner Removed the comma. Please review.


Adaptor is not a function

Hello,
I'm trying MathJax on my website, but with the following code I get an 'adaptor is not a function' error.
// the MathJax core
const MathJax = require("../../../../node_modules/mathjax3/mathjax3/mathjax.js").MathJax;

// Tex input
const TeX = require("../../../../node_modules/mathjax3/mathjax3/input/tex.js").TeX;
// HTML output
const CHTML = require("../../../../node_modules/mathjax3/mathjax3/output/chtml.js").CHTML;
// Use browser DOM
const adaptor = require("../../../../node_modules/mathjax3/mathjax3/adaptors/browserAdaptor").browserAdaptor();
// Register the HTML document handler
require("../../../../node_modules/mathjax3/mathjax3/handlers/html.js").RegisterHTMLHandler(adaptor());

I tried to change the route to:
const adaptor = require("../../../../node_modules/mathjax3/mathjax3/adaptors/browserAdaptor.js").browserAdaptor();

But it wouldn't work.

In the line
const adaptor = require("../../../../node_modules/mathjax3/mathjax3/adaptors/browserAdaptor").browserAdaptor();

you have already called the  browserAdaptor() function to create the adapter, so adaptor is already the adaptor you need, not the function to create one.  So you only need to do
require("../../../../node_modules/mathjax3/mathjax3/handlers/html.js").RegisterHTMLHandler(adaptor);

(without the parentheses after adaptor) to pass the adapter to the handler.
If you took this code from an example documented somewhere, can you let us know where?  It may need to be corrected.

Hi,
It compiled! Now I can try it.
Actually, I got that code from one of your demos.

Thanks for the reference to the example.  I have fixed it.
Glad it is compiling for you now.


[Suggestion] Add BlueBird Example to Docs?

Hey guys, long time no see.. Haven't been around here  in months. I  miss you all. My aRPG is coming along nicely.
I am using Bluebird promises extensively now since I figured out how to use it with MYSQL. It's a lifesaver and cleans up so many of my try/catch blocks.
For example:
function importFriends() {
    db.query('SELECT * from rpg_friends where character_id = ' + user.RPG.character_id, function(error, results) { // Import RPG_Friends into the buffer
        try {
            user.Friends = results;
            importBlocks();

        } catch (e) {
            gamelog(e)
        }
    });
}

Can be converted into:
function importFriends() {
	db.queryAsync('SELECT * from rpg_friends where character_id = ' + user.RPG.character_id).then((results) => {
		user.Friends = results;
		importBlocks();
	}).catch((e) => {
		gamelog(e);
	});
}

All I had to do was add:
Promise.promisifyAll(require("mysql/lib/Pool").prototype)
Promise.promisifyAll(require("mysql/lib/Connection").prototype)

Right above the mysql.createPool method
I think maybe some type of Bluebird example in our docs would be beneficial as this will help new people who are wanting to use  one of the best mysql libraries out there. It's also way safer for your program using promises in case some type of mysql error has happened so it won't shut down. (Why the try/catch usage is so important, but promises takes care of that for you).

See https://github.com/felixge/node-mysql/issues/929


Core styling

Addressing the first phase of CSS rework for Dataflow, this addresses the main features required by the Dataflow UI. I've avoided anything that might change functionality, focussing entirely on CSS for this iteration.
The run-state of each program tile may need revisiting as part of other work, and an overall indicator for run state is not yet available for styling.

I assume that the new document button that appears in the right panel will eventually be removed. Is this better placed in its own story?

The new document button that appears in the right panel is scheduled for removal. It's controlled by configuration in app-config.json, so removal is just a matter of deleting that line.

The pencil icon on document topbar is missing, but I assume this is better placed in a new story covering document title editing.

That feature was just merged to the dataflow branch this morning, so it should be available (without any Dataflow-specific styling) after merging/rebasing the core-styling branch to dataflow.

I didn't see the data collection badge (the little yellow ball) appear in the right panel gutter when running a program.

We had talked about needing to store some application-specific metadata properties for this, but @dstrawberrygirl is taking advantage of the fact that the thumbnails are live to approach this without the additional metadata. If we decide to make the thumbnails less live (which has been discussed), we may need to revisit the document metadata approach.


avoid unnecessary use of inline function expressions

This PR doesn't do too much, it was mostly for increasing readability, but it probably is faster given it avoids the look up at run time.
Also removes the unnecessary use of IFE's.

First of all, thanks so much for your PR :+1:
While this code might be more efficient because it doesn't create a closure inside the loop, the problem is that this code changes the value of cmd for all the the wrapper functions each time you go through the iteration. That's why you need the closure.
That is why the travis tests are failing (check out the logs), because it's trying to call the last cmd in the loop for every single call!
Thanks for the PR, please submit more if you come across anything else :smile:

I didn't have time to check the implementation.  I'll have a deeper look
into this.
Cheers for the extensive response.
On 3 Apr 2015 05:57, "Sean Lavine"<EMAIL_ADDRESS>wrote:

First of all, thanks so much for your PR [image: :+1:]
While this code might be more efficient because it doesn't create a
closure inside the loop, the problem is that this code changes the value of
cmd for all the the wrapper functions each time you go through the
iteration. That's why you need the closure.
That is why the travis tests are failing (check out the logs), because
it's trying to call the last cmd in the loop for every single call!
Thanks for the PR, please submit more if you come across anything else [image:
:smile:]
—
Reply to this email directly or view it on GitHub
https://github.com/freewil/node-bitcoin/pull/25#issuecomment-89009135.


⚠ API - Spain (ES) has degraded performance

In 3d3556d, API - Spain (ES) ($URL_API) experienced degraded performance:

HTTP code: 200
Response time: 9348 ms

Resolved: API - Spain (ES) performance has improved in d3e6b66.


Adjust circle scaling for optimal viewing

circles are probably too big right now

I'm wondering what we have for ideas here. Getting too much smaller is going to leave us with pretty limited pattern on the national map, especially in the smaller categories. I bet we can go to ~65% current before this is an issue, but that doesn't solve our issue of thermoelectric on the northeast coast (#197 ) because they still aren't small enough there.
I am confident we can make things better than they are now, but I don't think we should have #197 as acceptance criteria for this one.

Agreed; as noted in #197, resolving this issue should only improve, not solve, the problems noted in #197.

OK, what are the general thoughts on this size change.
USA, thermo, desktop
Largest circle size = 15 (current)

Largest circle size = 10

USA, thermo, mobile
Largest circle size = 15 (current)

Largest circle size = 10

New Jersey, thermo, mobile
Largest circle size = 15 (current)

Largest circle size = 10

USA, irrigation, desktop
Largest circle size = 15 (current)

Largest circle size = 10

I think it's an improvement, especially at the national view level.
For state views, what would you think about rescaling the circles to a different maximum? This could either be state specific (harder) or simply involve having a smaller maximum, e.g., largest circle size = 5

Hmm I guess that we are no longer bound to keeping them the same on zoomed scales...huh I think that might be the way to go!


Commands ignoring channel

The displayed commands ignore the selected channel.

But Nix is broken and nix-shell '<nixos-unstable> -p hello uses stable channel (https://github.com/NixOS/nix/issues/6984)!

What do you suggest? Adding -I nixpkgs=channel:nixos-<channel> to the command line?

This is a complicated UX design problem.
Most users probably only look up the package name and command to use with their default channel (e.g. nixos stable). Then the current command is fine.
Another use case is when a user intentionally set a different channel (as i did here) and expect to use that. Then it has to be added to the command.
nix-shell -I nixpkgs=channel:nixos-unstable -p hello
It will become even more complicated for configuration.nix.
This shows again how complicated Nix is. There are 12* different official ways to install a package. Even more with third-party tools like home-manager, snow and nix-software-center. No surprise users give up.
*3 supported channels * 4 install methods (nix-shell, nix-env, environment.systemPackages, users.users..packages)

I've said this before but I don't think nixos-search should spoonfeed users or teach them how to use Nix. The "How to install" section is meant as a quick reminder if you're not sure which name is the attribute name or you just want something to quickly copy-and-paste, but it shouldn't try to cover every possible scenario. Users should be able to fill in the gaps if they want something more specific.


500 Database [] not configured.

Sorry for my poor report, I'm new to this, my database config is displayed and ive hit a snag with the eror above.
You can see the error, ive turned on debug, at http://office.lbpfan950.net:81 if you sign in with the default credentials

Use gist - it's part of github.
I won't and I'm sure many others won't access a strange url like what you
provided
On 20/06/2016 7:22 AM, "lbpfan950"<EMAIL_ADDRESS>wrote:

Sorry for my poor report, I'm new to this, my database config is displayed
and ive hit a snag with the eror above.
You can see the error, ive turned on debug, at
http://office.lbpfan950.net:81 if you sign in with the default credentials
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/TechnicPack/TechnicSolder/issues/585, or mute the
thread
https://github.com/notifications/unsubscribe/AHMWcAlfvPqnevIQQXuJES2RtCchlUFnks5qNbMNgaJpZM4I5QVl
.

the error isnt simply one, it is the debug screen


Guidelines for scanning new projects

There need to be guidelines for including projects that are scanned. Here is a PR for including repositories to be scanned #151

For the most part we can scan whatever! If the list gets too long we'll need to better shard out the job, but I have no concerns yet unless we get to around 5x the size.

For the most part we can scan whatever! If the list gets too long we'll need to better shard out the job, but I have no concerns yet unless we get to around 5x the size.

+1, Goal is to scale this system infinitely, we shouldn't be saying no to any project.

Thanks

For the most part we can scan whatever! If the list gets too long we'll need to better shard out the job, but I have no concerns yet unless we get to around 5x the size.

+1, Goal is to scale this system infinitely, we shouldn't be saying no to any project.


[1.3.6-release:#5864][Feature]add FlinkX plugin

Purpose of the pull request
This pr adds FlinkX plugin
Brief change log
add FlinkX (version:1.11,not stable release) plugin
Verify this pull request
This change added tests and can be verified as follows:
1、added FlinkX plugin tests for end-to end via starting using local mode.
2、modifed FlinkX startup script like this way:

do you develop base on the dev branch?

do you develop base on the dev branch?

No.this pr is based on the 1.3.6 branch,someone told me to merge into the dev branch.