Update pipeline

@derekchiang whenever you have a chance. This is probably ready for a review.
Summary of changes:

The high level change here is that pipelines can now be update. It uses the same rpc as CreatePipeline it just adds a flag Update. This made the implementation easier since the rest of the API is identical.
When a pipeline is updated its previous output is Archived this prevents those commits from showing up in ListCommit and FlushCommit but the data can still be accessed.
To get the synchronization of updating the pipeline and archiving the commits right I had to add StopCommit and StartCommit methods.

Per conversation with JD.
Update pipeline shouldn't archive in all cases such as changing
parallelism, debug, or stopped. Would also makes sense to add a
dont-archive flag of some sort for users
On Mon, Aug 8, 2016 at 11:56 AM, Joe Doliner<EMAIL_ADDRESS>wrote:

@derekchiang https://github.com/derekchiang whenever you have a chance.
This is probably ready for a review.
Summary of changes:

The high level change here is that pipelines can now be update. It
uses the same rpc as CreatePipeline it just adds a flag Update. This
made the implementation easier since the rest of the API is identical.
When a pipeline is updated its previous output is Archived this
prevents those commits from showing up in ListCommit and FlushCommit
but the data can still be accessed.
To get the synchronization of updating the pipeline and archiving
the commits right I had to add StopCommit and StartCommit methods.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pachyderm/pachyderm/pull/694#issuecomment-238340277,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtjwGLrTIA5bt6cjq8iCsDdM0OA7nB_ks5qd3vdgaJpZM4JXtd-
.

2 things that aren't implemented here:

commit archiving isn't persisted because the way our persistence works we only persist commits once and archiving happens after the first persistence. This should be trivial to fix post pfs refactor though so it doesn't seem worth it to fix it in the current system.
Certain operations shouldn't archive commits but do (such as changing the parallelism of a pipeline) there's currently a flag --no-archive that will prevent this but users shouldn't have to figure out if they want to archive or not.

I loooooove this feature, and can't wait to start using it.
My comments center around the CLI API and test coverage.

CLI
Should we expose the new client functions to the CLI (ArchiveCommit() / StartPipeline() / StopPipeline()) ?
I think we're in the habit of doing so. I could understand how these new functions are mostly used in the case provided by calling CreatePipeline() to update a pipeline, but I could see users wanting to start / stop a pipeline manually / etc.

Test Coverage
I like the tests you've written -- those are some real thorough tests! But I'm not sure we have coverage for everything that is affected by the change.
There's an archived field on both CommitInfo and DiffInfo ... which means anything that reports results based on commits / diffs can be affected, so should be tested.
Some of these I do think we have coverage for already based on your tests (e.g. FlushCommit(), GetFile(), PutFile()). And some of the coverage I'm asking for is a bit redundant (quite a lot of things depend on the behavior of inspectFile() for instance) ... but I think still make for good end-end tests:

Commits

InspectCommit() should report the right size (it should include only the diffs that are not archived)
ListCommit() should list only non archived commits when the 'normal' type is specified (you test the 'all' case but I didn't see the 'normal' case ... maybe I missed it)

File

InspectFile() should only list files that are created by non archived commits, and report the right size
ListFile() should only list files from non archived commits

Repo

ListRepo() should report correct sizes omitting the sizes from archived commits
same for InspectRepo()

Concerns
If we do expose these helper functions (ArchiveCommit() in particular), there are cases in which a user could shoot themselves in the foot. Mostly because I believe inspectFile() traverses back on commits and looks for delete flags and doesn't check for the archived flag.
If user's stay on the 'happy path' and only use CreatePipeline() to archive commits this can't happen ... since by restarting pipelines, we start basically a fresh branch that contains no archived commits as parents in the history. That's really the assumption a user could violate w the manual tools.
But this is something to consider. Especially in light of the PFS refactor. We've put some thought into how we detect deletions efficiently w our queries. As long as a whole 'branch' gets archived at once I think we're ok in the new paradigm. But again, if a user decides to archive commits that are ancestors of non archived commits, that's bad, and would have undefined behavior as is in the refactor.
Perhaps as a compromise near term (esp since we want this to land short term) we:
a) don't expose these manual tools to the user, and
b) consider how we'd handle this in the pfs refactor. Maybe either:

detecting 'bad' usage of ArchiveCommit() (and disallow it unless you archive a whole branch)
or update our commit querying logic to handle archiving (but this seems complicated and bad)

Hopefully my descriptions make sense. We can discuss more in slack if thats helpful.

I fixed all of your inline comments. I'm a little confused by your longer comment though. I think you've missed the point of what archiving a commit is supposed to mean and are treating it like a synonym for delete which it isn't.

Some of these I do think we have coverage for already based on your tests (e.g. FlushCommit(), GetFile(), PutFile()). And some of the coverage I'm asking for is a bit redundant (quite a lot of things depend on the behavior of inspectFile() for instance) ... but I think still make for good end-end tests:

inspectFile isn't changed by this PR so the things that depend on it will have exactly the same behavior as before. Theoretically we could test that adding this flag didn't change this behavior but I'm kind of willing to take that one on faith since the function didn't change.

InspectCommit() should report the right size (it should include only the diffs that are not archived)

So if I understand correctly this would just mean that if a commit is archived we report 0 for its size rather than the actual size that it's taking up. To me that doesn't seem like the behavior we want. Also it shouldn't be possible for some diffs to be archived while others aren't, and with the pfs refactor that will be enforced.

ListCommit() should list only non archived commits when the 'normal' type is specified (you test the 'all' case but I didn't see the 'normal' case ... maybe I missed it)

https://github.com/pachyderm/pachyderm/pull/694/files#diff-169b88a58cc85335a06d29c2e7678bc9R966

InspectFile() should only list files that are created by non archived commits, and report the right size
ListFile() should only list files from non archived commits

This would defeat the purpose of archiving. If we wanted to make the data in them inaccessible we would just delete them.

ListRepo() should report correct sizes omitting the sizes from archived commits same for InspectRepo()

This one's kind of interesting, on the one hand it's a bit weird if you have a huge repo and then you do ListCommit and see nothing. But on the other hand I think having the repos report honestly how much space is actually being used to store the repo is important. So I think what we're doing now counting the archived commits as part of the repos size is the right thing to do.

@sjezewski another quick look when you have a chance to make sure you're satisfied with everything.

Ah, ok. I can see how my comments were confusing. And you're right, there was some conflating of Archive and Delete.
I was assuming we would expose the helper functions via CLI, and in that case I think I could construct a state where the results are confusing. I realize now the tests I was asking for are mostly addressing these sorts of problematic states. Below I describe what I mean in more detail for GetFile()
It makes sense what you said about repo sizes, (though if I was debugging something and only wanted to report the size of the non-archived commits it would be nice to have a flag for that).
Anyway, as long as we don't expose the CLI helpers to the user (which we aren't currently), then this PR LGTM

Just as an example of a confusing state the user could construct w access to ArchiveCommit via the CLI, consider the commit structure:
A --> B --> C --> D
Where each commit appends to the same file. After having created this commit structure, a user decides to Archive commits A and B. (Not saying this corresponds to a clear use case, just is something that they could do).
Now what happens when you call GetFile() on commit D with no From commit? Normally we'd expect it to traverse all the commits to construct the content. That is what will happen. But is that what we'd expect? Probably not? But I'm really not sure. That just seems confusing.

Exposing the commands seems reasonable to me. I'll add that before I merge. The example you describe is a bit confusing in that you have commits that are archived while their parents aren't. In terms of GetFile behavior though it doesn't have any effect because archive is supposed to leave things in a gettable state.


Publish artifact to clojars

It would be ideal if build-clj artifacts were published to Clojars, to allow it to be used as a transitive dependency in other build-related libraries that extend it (such as, though by no means limited to, this one).
The reason for this is that currently tools.deps doesn't support transitive dependencies to libraries that only have git coordinates (including, but not limited to, build-clj).

Sure, but because the tools.build JAR is automagically brought in by the CLI tools without needing to list it anywhere in a deps.edn file, it is not subject to the same limitations as something "3rd party" (such as build-clj).
I stand by the original request - it would be ideal if build-clj artifacts were published to Clojars.


cmd/compile: ssa.html sources line numbers are misaligned

Reproduce: generate ssa.html for any function. Expand the sources column. Observe that that line numbers are no longer aligned with the source code. This is a regression from this cycle, I believe.
cc @bradford-hamilton

Hey @josharian, more than happy to put some eyes on this and see whats going on.


Local, Facebook methods not compatible

I'm using all of the auth methods on our accounts portal, and we want to provide the user the option to link their social accounts, however if the user uses local auth with a email address of their choice and then we login?type=facebook facebook over rights the email attribute that local auth uses, if the facebook email was different we have issues.
Can we have each of the auth methods store their lookup variables in their own discrete names, i.e for facebook:  fb_facebookId, fb_email, fb_first_name, etc.
local: loc_email, loc_password, etc.
twitter: tw_twitterId
... etc. etc.
This way one auth method would not overwrite other methods attributes.

What do we think the fix is here? Maybe a separate Auth object per strategy?

Quickest, easiest fix which I implemented on my fork was to rename the
email attribute for local Auth to localEmail this way it doesn't get over
written by the email set by Facebook.  Another good change would be to
change the Facebook email field to facebookEmail similar to the way the
Google email attribute is stored.
David
On Jan 28, 2016 2:03 PM, "wayne-o"<EMAIL_ADDRESS>wrote:

What do we think the fix is here? Maybe a separate Auth object per
strategy?
—
Reply to this email directly or view it on GitHub
https://github.com/waterlock/waterlock/issues/119#issuecomment-176342625
.

I actually already started on this and went with prepending facebook on the FB specific properties - will do the same with google etc.
I am concerned that this is a major change - do we think we need to bump the version to 2.0 when we push this change? I suspect this will break for people else.
Can anyone advice?

The problem is now that if I register using lcoal auth and an email of<EMAIL_ADDRESS>and then later login with facebook auth where my facebook account is linked ot my email<EMAIL_ADDRESS>i get an error at findOrCreateAuth because that method tries to find the user by their FB ID.
One fix would be to find the user by email.
Another would be to email the user and ask them to re-confirm that email address and lock out authenticating via that FB account until it is confirmed - they would still be able to access the site by the email address.
Option #1 would be easier by a metric ton but is it safe to do this? I just checked and FB (not looked into google etc) do ask to confirm an email address if one is added to your account. So you do need to own that email at least for FB.
Thoughts?

I am now of the mindset that we should have a sign auth per provider. so user -> auth would need to become 1*n
This would definitely be a breaking change and would require some thoughts on migrating to
Does anyone have any opinion on this?


Fix relative redirection on https

Location header can now be relative: https://httpwg.org/specs/rfc9110.html#field.location
Fix #393

Wouldn't this break the tests since it requires LuaSec (LuaSocket doesn't support HTTPS)?

It did not. If it should have broken a test which test exactly do you think should have failed?

Exactly this HTTP redirection test. Isn't LuaSocket lacked support for HTTPS?

indeed the httptest.lua test will fail if luasec is not installed (raising on the assertion from http.lua)
current httptest.lua is also making additional hypotheses about having a luasocket-cgi runing on localhost:3128 serving the cgi folder tho
both assume the test environment to be a bit specific and test socket and http code at the same time.
in order to provide a reliable test without dependence to a third party, httptest.lua could use a ad-hoc fake socket that match luasocket api
see cpython/Lib/test/test_httplib.py as example


Add script for creating teams in bulk in github orgs

This PR adds a bash script to create teams in bulk to help Org owners to get up and running when setting up teams as part of their migration effort.

@larsxschneider @nicklegan
I just rewrote branch history to remove commit with accidentally committed token and forced push the changes. Can you please have a final review before I merge this please? 🙇

thanks folks 🚢 now..


No release notes for <IP_ADDRESS>

Our dependency update tool reported about the <IP_ADDRESS> version of Webtrekk but there is no changelog nor release notes yet.
Is this a stable release or a SNAPSHOT?

Hi,
release <IP_ADDRESS> is the latest stable one, we will update the release notes soon. It was hotfix for 5.1.4, where you could not use the setEverId functionality as expected, so please use <IP_ADDRESS> instead.


Docs using type aliases for features

Yeah, I hope'd we'd get away without doing it, and inside the module all is well. Outside it doesn't work, so I fixed it.

Codecov Report
Base: 83.47% // Head: 83.43% // Decreases project coverage by -0.04% :warning:

Coverage data is based on head (490a329) compared to base (a1eb79c).
Patch coverage: 90.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #481      +/-   ##
===========================================
- Coverage    83.47%   83.43%   -0.05%
===========================================
  Files           60       60
  Lines         5676     5679       +3
===========================================
  Hits          4738     4738
- Misses         938      941       +3

Impacted Files
Coverage Δ

core/eolearn/core/utils/parsing.py
90.47% <90.00%> (-0.59%)
:arrow_down:

core/eolearn/core/extra/ray.py
92.68% <0.00%> (-2.44%)
:arrow_down:

core/eolearn/core/graph.py
94.94% <0.00%> (-1.02%)
:arrow_down:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Review and update doc describing how we source the plugin description info

FYI, this is a recent PR I made of my own accord while exploring plugin creation: https://github.com/napari/docs/pull/143#discussion_r1179488353

Updated made here.


Apache Flink Evaluation

Evaluate Apache Flink framework for Texera
Important
This issue evaluates the Apache Flink version of 1.6.0 with pre-built support for Apache Hadoop 2.8 installed.
Check your version if there is any inconsistence

According to the Stack Overflow answer and our observation. The current Flink framework does not provide ways to implement pause/resume functionality.
We have arrived the conclusion that the current Flink framework is not suitable for Texera backend implementation.
The Stack Overflow reference: https://stackoverflow.com/questions/51885621/pausing-a-dataflow-in-apache-flink-and-viewing-the-state-of-operators


ValueError: Attempted relative import beyond toplevel package

Here's another error I've been getting.
ValueError                                Traceback (most recent call last)
<ipython-input-10-9f8306731f52> in <module>()
      2 import sys
      3 sys.path.append(auto_ml_path)
----> 4 from auto_ml import Predictor
      5
      6 col_desc_dictionary = {'target': 'output'}

/Users/alex/Desktop/Machine Learning/auto_ml/auto_ml/__init__.py in <module>()
----> 1 from .predictor import Predictor

/Users/alex/Desktop/Machine Learning/auto_ml/auto_ml/predictor.py in <module>()
     27     from auto_ml import utils
     28 except:
---> 29     from .. auto_ml import utils
     30 try:
     31     from auto_ml import date_feature_engineering

The issue was solved for me by just eliminating the problematic import statement.
Is this statement (and other try/except blocks with imports from parent directories) necessary?

would love a PR to clean this up a bit! thanks for filing the bug- the issue you reported should be fixed now.


Classification

This pull request a SYSTEM_HIGH_CLASSIFICATION variable for passing in a classification to client applications.  The request also contains some housekeeping that assisted in development of client applications.  Per Igor, despite this technically being a breaking change, making this effort available to the community.

@clabas we appreciate your pull request and would like to put it forward for our GOSS Board to determine if this should be included in the product. We have just formalized our patch process that can be found at https://ozone.nextcentury.com/patch_process. Please ensure that the following has been included so that it can be presented to the board:

Add comments in the code to provide detailed developer-level documentation of your changes
Verify that the entire code base builds without issue after all changes are made
Document and provide unit/regression tests that verify that your new code functions and can be tested by others
Submit complete and adequate documentation with the pull request explaining the feature(s) and how it works

Please be sure to include in your documentation how this will be a breaking change.
Thank you again for your contribution to this project.


Can't flash Arduino with firmware

Hi, i have some problems with installing firmware to Arduino Mega through FarmBotOS.
Arduino connected to RaspberryPi3 cable and green LED onto RAMP board started to flash.
Also, as i see Raspberry succesfully connected to WebApp and MQTT:

but when I try to flash firmware, the error "Flash firmware failed" appears:

I've connected to Raspberry3 via Serial Console and got next error in terminal:
iex(farmbot@farmbot-000000008aa873b0.local)3> warning: CeleryScript Error: {:noproc, {GenServer, :call, [FarmbotCore.Firmware.UARTCore, {:flash_firmware, "arduino"}, 600000]}}
  (elixir 1.10.4) lib/gen_server.ex:1013: GenServer.call/3
  (farmbot_celery_script 14.1.2) lib/farmbot_celery_script/sys_calls.ex:372: FarmbotCeleryScript.SysCalls.ok_or_error/3
  (farmbot_celery_script 14.1.2) lib/farmbot_celery_script/step_runner.ex:40: FarmbotCeleryScript.StepRunner.execute/3
  (farmbot_celery_script 14.1.2) lib/farmbot_celery_script/step_runner.ex:15: FarmbotCeleryScript.StepRunner.step/3

13:31:41.721 [info]  Starting firmware flash.

13:31:41.859 [info]  Using default reset function

13:31:41.888 [debug] Writing firmware to MCU... ["-patmega2560", "-cwiring", "-P/dev/ttyAMA0", "-b115200", "-D", "-V", "-v", "-Uflash:w:/srv/erlang/lib/farmbot_core-14.1.2/priv/arduino_firmware.hex:i"]

13:31:41.983 [info]  Begin flash attempt 1

13:32:12.667 [info]  Attempt 1 failed.

13:32:12.790 [info]  "\navrdude: Version 6.1-svn-20130917, compiled on May 28 2021 at 14:31:52\n         Copyright (c) 2000-2005 Brian Dean, http://www.bdmicro.com/\n         Copyright (c) 2007-2009 Joerg Wunsch\n\n         System wide configuration file is \"/etc/avrdude.conf\"\n         User configuration file is \"/root/.avrduderc\"\n         User configuration file does not exist or is not a regular file, skipping\n\n         Using Port                    : /dev/ttyAMA0\n         Using Programmer              : wiring\n         Overriding Baud Rate          : 115200\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_getsync(): timeout communicating with programmer\n\navrdude done.  Thank you.\n\n"

Could you suggest whats wrong and how i can flash the proper firmware to Arduino?

@gueux You are running an outdated version of FBOS. Older versions of FBOS 14.x had some firmware issues that were recently fixed. Please upgrade and let me know if that fixes the problem.
You can upgrade by reflashing FBOS, or by running sudo docker-compose run rake releases:publish from your server's shell and then upgrading from inside of the web app.

@RickCarlino i've updated farmbot os to the latest 14.1.3-rc2 version, but unfortunately issue is the same one.
Thats what i get when click onto FlashFirmware:
iex(farmbot@farmbot-000000008aa873b0.local)3> RingLogger.attach
:ok

13:52:17.976 [info]  CeleryScript OK
iex(farmbot@farmbot-000000008aa873b0.local)4> warning: CeleryScript Error: {:noproc, {GenServer, :call, [FarmbotCore.Firmware.UARTCore, {:flash_firmware, "arduino"}, 600000]}}
  (elixir 1.10.4) lib/gen_server.ex:1013: GenServer.call/3
  (farmbot_celery_script 14.1.3-rc2) lib/farmbot_celery_script/sys_calls.ex:372: FarmbotCeleryScript.SysCalls.ok_or_error/3
  (farmbot_celery_script 14.1.3-rc2) lib/farmbot_celery_script/step_runner.ex:40: FarmbotCeleryScript.StepRunner.execute/3
  (farmbot_celery_script 14.1.3-rc2) lib/farmbot_celery_script/step_runner.ex:15: FarmbotCeleryScript.StepRunner.step/3

13:52:25.145 [error] CeleryScript error [#Reference<0.2907558172.537657345.28763>]: "{:noproc, {GenServer, :call, [FarmbotCore.Firmware.UARTCore, {:flash_firmware, \"arduino\"}, 600000]}}"

13:52:25.246 [info]  Failed to execute command: {:noproc, {GenServer, :call, [FarmbotCore.Firmware.UARTCore, {:flash_firmware, "arduino"}, 600000]}}

13:52:25.247 [error] CeleryScript error [#Reference<0.2907558172.537657345.28763>]: "{:noproc, {GenServer, :call, [FarmbotCore.Firmware.UARTCore, {:flash_firmware, \"arduino\"}, 600000]}}"

And thats what happened when click onto RestartFirmware:
13:53:32.754 [info]  CeleryScript OK
iex(farmbot@farmbot-000000008aa873b0.local)4> warning: CeleryScript Exception:
  :erlang.send(FarmbotCore.Firmware.UARTCore, :reset_state)
  (farmbot_core 14.1.3-rc2) lib/firmware/uart_core.ex:80: FarmbotCore.Firmware.UARTCore.restart_firmware/1
  (farmbot 14.1.3-rc2) lib/farmbot_os/sys_calls.ex:195: FarmbotOS.SysCalls.firmware_reboot/0
  (farmbot_celery_script 14.1.3-rc2) lib/farmbot_celery_script/sys_calls.ex:372: FarmbotCeleryScript.SysCalls.ok_or_error/3
  (farmbot_celery_script 14.1.3-rc2) lib/farmbot_celery_script/step_runner.ex:40: FarmbotCeleryScript.StepRunner.execute/3
  (farmbot_celery_script 14.1.3-rc2) lib/farmbot_celery_script/step_runner.ex:15: FarmbotCeleryScript.StepRunner.step/3

13:53:36.955 [error] CeleryScript error [#Reference<0.2907558172.537657345.62551>]: "argument error"

13:53:37.042 [info]  Failed to execute command: argument error

13:53:37.042 [error] CeleryScript error [#Reference<0.2907558172.537657345.62551>]: "argument error"

BTW, i can manually compile and upload latest firmware through Arduino IDE, so seems board and usb cable is suitable.

After some of arduino restart this error appears:
14:35:26.806 [info]  Rebooting inactive Farmduino. 2859697

14:35:26.931 [info]  Starting firmware flash.

14:35:27.081 [info]  Using default reset function

14:35:27.097 [debug] Writing firmware to MCU... ["-patmega2560", "-cwiring", "-P/dev/ttyAMA0", "-b115200", "-D", "-V", "-v", "-Uflash:w:/srv/erlang/lib/farmbot_core-14.1.3-rc2/priv/arduino_firmware.hex:i"]

14:35:27.200 [info]  Begin flash attempt 1
iex(farmbot@farmbot-000000008aa873b0.local)17>
nil

14:35:51.626 [info]  Attempt 1 failed.

14:35:51.748 [info]  "\navrdude: Version 6.1-svn-20130917, compiled on May 28 2021 at 14:31:52\n         Copyright (c) 2000-2005 Brian Dean, http://www.bdmicro.com/\n         Copyright (c) 2007-2009 Joerg Wunsch\n\n         System wide configuration file is \"/etc/avrdude.conf\"\n         User configuration file is \"/root/.avrduderc\"\n         User configuration file does not exist or is not a regular file, skipping\n\n         Using Port                    : /dev/ttyAMA0\n         Using Programmer              : wiring\n         Overriding Baud Rate          : 115200\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_getsync(): timeout communicating with programmer\n\navrdude done.  Thank you.\n\n"

14:35:51.870 [info]  Begin flash attempt 2

@RickCarlino  Seems i've found the reason of this problem. FarmbotOS use the wrong port definition for avrdude programmator tool.
iex(farmbot@farmbot-000000008aa873b0.local)5> {:ok, fun} = FarmbotCore.Firmware.Resetter.find_reset_fun("")
{:ok, #Function<0.91348444/0 in FarmbotCore.Firmware.Resetter.use_default/0>}
iex(farmbot@farmbot-000000008aa873b0.local)6> Avrdude.flash("/srv/erlang/lib/farmbot_core-14.1.3-rc2/priv/arduino_firmware.hex", "/dev/ttyAMA0", fun)
{"\navrdude: Version 6.1-svn-20130917, compiled on May 28 2021 at 14:31:52\n         Copyright (c) 2000-2005 Brian Dean, http://www.bdmicro.com/\n         Copyright (c) 2007-2009 Joerg Wunsch\n\n         System wide configuration file is \"/etc/avrdude.conf\"\n         User configuration file is \"/root/.avrduderc\"\n         User configuration file does not exist or is not a regular file, skipping\n\n         Using Port                    : /dev/ttyAMA0\n         Using Programmer              : wiring\n         Overriding Baud Rate          : 115200\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_ReceiveMessage(): timeout\navrdude: stk500v2_getsync(): timeout communicating with programmer\n\navrdude done.  Thank you.\n\n",
 1}

iex(farmbot@farmbot-000000008aa873b0.local)9> {:ok, fun} = FarmbotCore.Firmware.Resetter.find_reset_fun("")
{:ok, #Function<0.91348444/0 in FarmbotCore.Firmware.Resetter.use_default/0>}
iex(farmbot@farmbot-000000008aa873b0.local)10> Avrdude.flash("/srv/erlang/lib/farmbot_core-14.1.3-rc2/priv/arduino_firmware.hex", "/dev/ttyUSB0", fun)
{"\navrdude: Version 6.1-svn-20130917, compiled on May 28 2021 at 14:31:52\n         Copyright (c) 2000-2005 Brian Dean, http://www.bdmicro.com/\n         Copyright (c) 2007-2009 Joerg Wunsch\n\n         System wide configuration file is \"/etc/avrdude.conf\"\n         User configuration file is \"/root/.avrduderc\"\n         User configuration file does not exist or is not a regular file, skipping\n\n         Using Port                    : /dev/ttyUSB0\n         Using Programmer              : wiring\n         Overriding Baud Rate          : 115200\n         AVR Part                      : ATmega2560\n         Chip Erase delay              : 9000 us\n         PAGEL                         : PD7\n         BS2                           : PA0\n         RESET disposition             : dedicated\n         RETRY pulse                   : SCK\n         serial program mode           : yes\n         parallel program mode         : yes\n         Timeout                       : 200\n         StabDelay                     : 100\n         CmdexeDelay                   : 25\n         SyncLoops                     : 32\n         ByteDelay                     : 0\n         PollIndex                     : 3\n         PollValue                     : 0x53\n         Memory Detail                 :\n\n                                  Block Poll               Page                       Polled\n           Memory Type Mode Delay Size  Indx Paged  Size   Size #Pages MinW  MaxW   ReadBack\n           ----------- ---- ----- ----- ---- ------ ------ ---- ------ ----- ----- ---------\n           eeprom        65    10     8    0 no       4096    8      0  9000  9000 0x00 0x00\n           flash         65    10   256    0 yes    262144  256   1024  4500  4500 0x00 0x00\n           lfuse          0     0     0    0 no          1    0      0  9000  9000 0x00 0x00\n           hfuse          0     0     0    0 no          1    0      0  9000  9000 0x00 0x00\n           efuse          0     0     0    0 no          1    0      0  9000  9000 0x00 0x00\n           lock           0     0     0    0 no          1    0      0  9000  9000 0x00 0x00\n           calibration    0     0     0    0 no          1    0      0     0     0 0x00 0x00\n           signature      0     0     0    0 no          3    0      0     0     0 0x00 0x00\n\n         Programmer Type : Wiring\n         Description     : Wiring\n         Programmer Model: AVRISP\n         Hardware Version: 15\n         Firmware Version Master : 2.10\n         Vtarget         : 0.0 V\n         SCK period      : 260.5 us\n\navrdude: AVR device initialized and ready to accept instructions\n\nReading | ################################################## | 100% 0.02s\n\navrdude: Device signature = 0x1e9801\navrdude: reading input file \"/srv/erlang/lib/farmbot_core-14.1.3-rc2/priv/arduino_firmware.hex\"\navrdude: writing flash (48968 bytes):\n\nWriting | ################################################## | 100% 7.96s\n\navrdude: 48968 bytes of flash written\n\navrdude done.  Thank you.\n\n",
 0}

Seems there is no fallback port option for trying both ports  /dev/ttyAMA0 and /dev/ttyUSB0

@gueux I will assume you are using an Arduino that you did not purchase from FarmBot, Inc. If you are using non-official hardware, you need to manually set the firmware path.
Here is how you can manually set your UART device path (ttyUSB0):

If you do not know your Arduino's device path, reboot the device. It will provide a log that tells you all available UART device paths (usually ttyAMA0 or ttyUSB0).
Enable ADVANCED SETTINGS.
Change your Arduino path to ttyUSB0 or whatever the correct path is.


8/8/2022 PM Publish

Docs Build status updates of commit 89738c7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-7.5.0/quickstart-migrate-azurerm-to-az-automatically.md
:white_check_mark:Succeeded
View (azps-7.5.0)

docs-conceptual/azps-8.1.0/quickstart-migrate-azurerm-to-az-automatically.md
:white_check_mark:Succeeded
View (azps-8.1.0)

docs-conceptual/azps-8.2.0/quickstart-migrate-azurerm-to-az-automatically.md
:white_check_mark:Succeeded
View (azps-8.2.0)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Trying to sort by product updated_at

Hi, I am trying to sort the product by its updated_at. I already change the database type to timestamp, but I notice the updated_at only updated when I change info from the general section. It doesn't update at all when I change information in like description. How should I modify that?

@lYesterdaYl
could you mention your bagisto version?

Not getting any activity from the last 11 days so closing this issue.
Feel free to reopen anytime,


enh: improve tables query dust app

Description
https://dust.tt/w/78bda07b39/a/b4f205e453

fix regexp to account for newly allowed - character
improve support for comma-separated array string columns

Risk

Deploy Plan

@spolu PMRW


Custom date interval behavior confusing

The Year, Week, Month and Day date interval types all add one extra day to the end date so that the date interval is inclusive on the beginning date and exclusive on the end date.  This makes a lot of sense and lets me easily write code like this:
class KPIReport(luigi.Task):
    date_interval = luigi.DateIntervalParameter()

    def run(self):
        start = datetime.datetime.combine(self.date_interval.date_a, datetime.time())
        end = datetime.datetime.combine(self.date_interval.date_b, datetime.time())
        num_members = User.find({'signup_time': {'$gte': start, '$lt': end}})
        # etc...

So now if I run python reports.py KPIReport --date-interval 2015-03-20 everything works out and I get a report for that single day.
However now if I want to run the same report on an arbitrary range, such as the last 7 days, I might think I can simply run python reports.py KPIReport --date-interval 2015-03-14-2015-03-20.  But now instead of ending at 23:59:59 on the 20th my date interval ends at 23:59:59 on the 19th.
It seems to me that the Custom date interval should add one day to the end date just as all of the other implementations do, or is there a different best practice for this scenario?

This is by design. There are some particular reasons:

If you think of a date itself not as an extent in time but as a timestamp then 2012-03-20 is equal to 2012-03-20T00:00:00 which is the 0th second of 2012-03-20.
In a lot of situations it's a lot more convenient to express it as half open intervals because you don't have to think about how many days are in a month. For instance if you run a bimonthly report then it's easier to run it for 2012-02-01-2012-02-15 and 2012-02-15-2012-03-01 rather than 2012-01-01-2012-02-14 and 2012-02-15-2012-02-29 (now you need to know the number of days of Feb 2012)

Found some web sites on the topic btw:
http://stackoverflow.com/questions/4264947/what-is-best-practice-for-representing-time-intervals-in-a-data-warehouse
http://wrschneider.blogspot.com/2014/01/time-intervals-and-other-ranges-should.html

Also compare how Python uses half-open notation for array slicing (R doesn't though, afaik)


Filter with intervals not work

HTTP REQUEST
curl http://localhost:3000/database/public/table?created_at='$gte.2017-11-03'

SQL CREATED:
SELECT json_agg(s) FROM (SELECT * FROM "database"."public"."table" WHERE "created_at" >= $1 ) s parameters: [2017-11-03]

curl http://localhost:3000/database/public/table?created_at='$gte.2017-11-03'&created_at='$lte.2017-11-05'

SQL CREATED
SELECT json_agg(s) FROM (SELECT * FROM "database"."public"."table" WHERE "created_at" >= $1 ) s parameters: [2017-11-03]

SQL EXPECTED
SELECT json_agg(s) FROM (SELECT * FROM  "database"."public"."table" WHERE "created_at" >= $1 AND "created_at" <= $2 ) s parameters: [2017-11-03 2017-11-05]

It is not a bug it is a feature request key on query string is unique

@felipeweb It's a bug.
https://stackoverflow.com/questions/1746507/authoritative-position-of-duplicate-http-get-query-keys
https://docs.oracle.com/javaee/6/api/javax/servlet/ServletRequest.html#getParameterValues(java.lang.String)

Yes it is a bug


Activity page - Refinery fuel gas combustion

Description:
Wireframe:
Following #253 we have a full stack example of how to generate an activity emissions form using the API form builder service.The implementation here should be very similar to the GSC activitiy page.
Documentation on the full stack steps necessary to create an activity page are in our docs
Use the excel file (BCIERS Reporting Fields V2 in the SME channel files  - Reporting App directory) provided by IRC to determine what the necessary activity, source type, gas type, methodology, reporting fields & related configurations that need to be added are.
Acceptance Criteria:
Given I am on the refinery fuel gas combustion activity form page
When I first land on the page
Then the activity schema is fetched & displayed
Given the schema has rendered correctly
When I make selections that have dependencies (gas type, methodology)
Then the correct options and fields as defined from the configuration are shown
Development Checklist:

[x] Any activities, source types, gas types, methodologies or reporting fields needed for the configuration that are not already in the database are added
[x] The configuration relationships have correctly been added to the database
[x] The many to many reporting field data (if needed) has correctly been added to the database
[x] The base schema for the activity & source types have been added to the database
[x] A routing directory/page has been added to both the industry_user & industry_user_admin flows for the activity
[x] A parent component (named for the activity) has been added to the components in the activity directory
[x] The page fetches the correct initial activity schema from the API service on initial load
[x] A uiSchema exists in the components/activities/uiSchema directory for the activity & source types (will move to the database eventually, but can live here for now)
[x] Conditional fields render dynamically as data is entered
[x] On selecting a source type (or if there is only one, on loading the page) The first set of data fields is displayed (defaultEmptySourceTypeState)
[x] Matches the wireframe

Definition of Ready (Note: If any of these points are not applicable, mark N/A)

[ ] User story is included
[ ] User role and type are identified
[ ] Acceptance criteria are included
[ ] Wireframes are included (if required)
[ ] Design / Solution is accepted by Product Owner
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

Definition of Done (Note: If any of these points are not applicable, mark N/A)

[ ] Acceptance criteria are tested by the CI pipeline
[ ] UI meets accessibility requirements
[ ] Configuration changes are documented, documentation and designs are updated
[ ] Passes code peer-review
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Ticket is ready to be merged to main branch
[ ] Can be demoed in Sprint Review
[ ] Bugs or future work cards are identified and created
[ ] Reviewed and approved by Product Owner

Notes:

@dleard can you share the URL. I took a stab at reporting/reports/1/facilities/1/activities/refinery_fuel_gas_combustion but that didn't work.

Oops, sorry I shortened it a bit. It's reporting/reports/1/facilities/1/activities/refinery_fuel_gas

Reviewed in Dev. Works as expected. Matches BCIERS data field Spreadsheet. Will move to QA (in Test) for future review.


Not all variants are available (Tailwind + Vite + React)

What version of @tailwindcss/jit are you using?
v0.1.13
What version of Node.js are you using?
v14.15.1
What browser are you using?
Safari
What operating system are you using?
macOS
Reproduction repository
https://github.com/edgarasben/bulletproof-ui/
I am trying to access all variants from JIT documentation, but they do not seem to work.
I've tried these few from the documentation
<input class="disabled:opacity-75">
and
<button class="md:dark:disabled:focus:hover:bg-gray-400">
My setup is: Tailwind + Vite + React
What am I getting wrong here? Do I need to extra configuration?

Hey! I can't reproduce, seems to be working as expected — when I add the disabled attribute to the input the disabled:opacity-25 class kicks in just like it's supposed to:
https://user-images.githubusercontent.com/4323180/112728421-9ace1800-8efd-11eb-93e2-2922a0d604be.mov
Same with the button, once I add the dark class to the HTML element and make the button disabled, the styles kick in on hover+focus at the md breakpoint:

Going to close since I think this is maybe just expecting the feature to work differently — feel free to open another issue with a really clear/minimal reproduction though if I'm misunderstanding.

Hi, Adam! You are right, it's working perfectly. It's just me getting confused about the disabled state. Thank you!


Allow to test update query with query editor

Context: hitting "Run query" within the "Update request payload" section doesn't seem to do anything/send a query.

We discussed it internally today. To execute Run query for the Update request payload we will need to generate test payload and send it to the data source.
We are open to work on it in the upcoming releases if we see more requests from the community.

@asimonok Could you please double check if it's possible to hide the Run query button?


Possibly an Error in Guide

In the guid on this page to following code is written:
use vulkano::descriptor::descriptor_set::PersistentDescriptorSet;

let layout = pipeline.layout().descriptor_set_layout(0).unwrap();
let set = Arc::new(PersistentDescriptorSet::start(layout.clone())
    .add_buffer(data_buffer.clone()).unwrap()
    .build().unwrap()
);

two things are not ideal here:

the pipeline variable was called compute_pipeline in the previous page
use vulkano::descriptor::PipelineLayoutAbstract is required

Would be great to get this fixed, just stumbled on it myself and @nilsmartel's solution fixed it for me.


Remove Newtonsoft.Json dependency

Hi there,
Would it be possible to remove the dependency on Newtonsoft.Json?
The built-in JSON serializer of .NET has seen much improvement since .NET 6, which makes this dependency obselete.
Messages that internally get serialized in this library are simple enough to not depend on Newtonsoft's library it seems.
I'm happy to provide a PR if needed.

Seems like this is already done, lol. My bad!


style(transformer): re-order dependencies

Nit. Move oxc_* dependencies to first in list.

#6659  👈
main

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

Join @overlookmotel and the rest of your teammates on  Graphite

Cosmetic change, so merging without review.

Merge activity

Oct 18, 10:38 AM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


⚠ Bofrost.de has degraded performance

In b5ec922, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1471 ms

Resolved: Bofrost.de performance has improved in c6545fb.


Honda: check regen paddle

Describe the bug
TL;DR
Honda Insight's brake regen adjustment paddles cause OP to throw a CruiseDisabled error instead of canceling cruise control.
More Info
The Honda Insight has adjustable regenerative braking. This is controlled by paddles on the steering wheel. Hitting either paddle when cruise is engaged will disengage cruise.
Disengaging cruise control in this fashion causes OP to trigger a "cruise not engaged" message and sound the alarm, instead of gracefully exiting cruise control via the "Cancel" button. This is governed in the Honda interface, L332.
It appears that the car sends a generic "Cruise Control disabled" message; there's a lot of junk in Cabana for this trip which isn't mapped to any signals, but one of those could be the issue.
Desired Outcome
When using the steering wheel paddles to disable cruise control, OP should treat this as a standard disablement instead of an emergency.
Which car does this affect?
Honda Insight EX 2019
Provide a route where the issue occurs
8958942b212e8cd9|2023-11-30--08-18-06--10
openpilot version
On release3, latest release.
Additional info
No response

We will want to add this signal to panda safety/Honda carstate. Check out safety_gm.h and gm/carstate.py for reference.
Let us know if you plan on adding this!

Need any direction to get started?

Kinda. Unfortunately I haven't had time to start work on this (I'm in
Eastern time but I work UTC hours, so I'm a bit of a zombie right now).
Is there documentation on the "standard OP libraries) anywhere?
On Thu, Dec 7, 2023, 14:15 Shane Smiskol @.***> wrote:

Need any direction to get started?
—
Reply to this email directly, view it on GitHub
https://github.com/commaai/openpilot/issues/30559#issuecomment-1845958037,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFCPBBA3LI2H3SLXZBLXBNDYIIIVVAVCNFSM6AAAAABABKVL36VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBVHE2TQMBTG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

So the goal here to detect when the regen paddle has been pressed, to both know that stock cruise disengaging is okay, as well as implementing our own safety that enforces openpilot disengages at the same time (even if not using stock ACC).
To do this, we need to have openpilot understand that this exists and what it does (setting regenPaddle in carstate.py), as well as in panda to enforce this safety behavior.
Our abstractions should be pretty good for these types of changes, you just need to set this signal in both places. Finding the signal might be a different, smaller project. Cabana is helpful to reverse engineer CAN messages.
Is there any specific questions you had?

No, sir, that is exactly what I needed to know - thank you very much. I
triggered the behavior a bunch of times in past drives in order to find the
signal - unfortunately, I'm getting segfaults when loading videos in
Cabana, but I'm able to work past this (the warning banners still show in
the Cabana UI).
On Tue, Dec 12, 2023 at 4:43 AM Shane Smiskol @.***>
wrote:

So the goal here to detect when the regen paddle has been pressed, to both
know that stock cruise disengaging is okay, as well as implementing our own
safety that enforces openpilot disengages at the same time (even if not
using stock ACC).
To do this, we need to have openpilot understand that this exists and what
it does (setting regenPaddle in carstate.py
https://github.com/commaai/openpilot/blob/e687be939e7cf67c4a75b87320ab058941d4316f/selfdrive/car/gm/carstate.py#L72-L74),
as well as in panda
https://github.com/commaai/panda/blob/892ca5a0f19c07a4786373e360027ab6ec57f723/board/safety/safety_gm.h#L124-L126
to enforce this safety behavior.
Our abstractions should be pretty good for these types of changes, you
just need to set this signal in both places. Finding the signal might be a
different, smaller project. Cabana
https://github.com/commaai/openpilot/tree/master/tools/cabana is
helpful to reverse engineer CAN messages.
Is there any specific questions you had?
—
Reply to this email directly, view it on GitHub
https://github.com/commaai/openpilot/issues/30559#issuecomment-1851666073,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFCPBBD4R5QICXLZSSQIFWTYJARMXAVCNFSM6AAAAABABKVL36VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJRGY3DMMBXGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

@plygrnd let me know if there's any issues moving forward.


Load in (barry) emoticon

Slap into the www/img directory then change the index.html accordingly.

Haha :smile: :sparkling_heart:


Dvd Drive not list

please support dvd drive into drive list.

Duplicate of https://github.com/resin-io-modules/drivelist/issues/162


Setup global parameters

 Here is an overview of what got changed by this pull request:

Clones added
============
- src/tests/possumwood/subnetwork_from_file.cpp  3
- src/tests/possumwood/root_network_ports.cpp  4
- src/tests/possumwood/subnetwork_rename.cpp  1

Clones removed
==============
+ src/tests/possumwood/json.cpp  -3

See the complete overview on Codacy


Add editorconfig

editorconfig is a great way to set some file formatting conventions for a project and have your text editor automatically apply them. This saves the hassle of having to fix and re-review pull requests because of simple indentation issues or Windows-created files with CRLF line endings.

Cheers! :beers:


PBREffect

With almost all engines supporting this, would be nice to have PBR effect support.

Pharr, Jakob, and Humphreys, ,Physically Based Rendering: From Theory to Implementation, Morgan Kaufmann, website code


[Bug] Make the challenge service "AWS-aware" using the method applied to the organization service

Is there an existing issue for this?

[X] I have searched the existing issues

What product(s) are you seeing the problem on?
OpenChallenges
Current behavior
Contributes to https://sagebionetworks.jira.com/browse/IT-3800
Expected behavior
No response
Anything else?
No response
Commit ID
No response
Are you developing inside the dev container?

[X] I am using the dev container

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Done


Highlights/Top Issues - Slider

https://www.figma.com/design/MxA95ZHK5NqGX4ptFQljRJ/⭐-Design-System-3.0?node-id=6814-101318&t=W0vvSvMREkXryYaI-11 - Design Mockup

Closed in #863


Remove copy to spreadsheet button

No longer needed. It was only a stopgap measure before we had automatic prevalence data imports.

Do we have a good way to communicate that folks using old sheets need to
upgrade to one with auto-importing?
On Sat, Dec 26, 2020 at 12:08 PM netlify[bot]<EMAIL_ADDRESS>wrote:

Deploy preview for microcov ready!

Reply to this email directly, view it on GitHub
https://github.com/microcovid/microcovid/pull/600#issuecomment-751391464,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABM25LNN36C7VVZQDJN6CC3SWY7CNANCNFSM4VKFTVXQ
.

@beshaya good question. It was only ever designed to work with a manual data entry for a very certain version of the spreadsheet. I don't think it works with old versions before that. So... shrug I'm assuming almost nobody uses it. But I don't have the data to back that up. And there's a chance I could be mis remembering.
Would you suggest we add a little notice for spreadsheet users for a while to encourage them to switch to the latest version?

@beshaya good question. It was only ever designed to work with a manual data entry for a very certain version of the spreadsheet. I don't think it works with old versions before that. So... shrug I'm assuming almost nobody uses it. But I don't have the data to back that up. And there's a chance I could be mis remembering.
Would you suggest we add a little notice for spreadsheet users for a while to encourage them to switch to the latest version?

Yeah, could we keep this button for, say, 2 weeks, and have it pop up a warning to upgrade?

ALSO WE SHOULD ADD ANALYTICS TO THAT BUTTON

Oops. Didn't mean to close this.

Oops. Didn't mean to close this.

@beshaya ready for your review.

@beshaya ready for your review.


Remove Coveralls

We don’t actually use it that much and it’s getting kind of annoying in pull requests.

:+1:


Results as root

I understand this is an artifact of docker and the way the container is running, but i was wondering if there was anyway to change the way the results are getting stored. Currently they are being written as root and this is causing some issues for our end users who are running analysis but cant see the results. Is there a way to fix this other than an admin chowning the files afterwards?

I understand this is an artifact of docker and the way the container is running, but i was wondering if there was anyway to change the way the results are getting stored. Currently they are being written as root and this is causing some issues for our end users who are running analysis but cant see the results. Is there a way to fix this other than an admin chowning the files afterwards?

line 184 in run_docker.py , you should be able to change the output path
output_target_path = os.path.join(_ROOT_MOUNT_DIRECTORY, 'output')

This is addressed by pull request #87.

"cant see the results" did not happen to me. Although the results are written as root:root, the r (read) permission is given to everyone. So, the end-user should be able to read or copy the file. At least it is the case for me on Ubuntu 18.04.6 LTS.

Thank you, i will repull and test this myself. Appreciate the responses.

Used a fresh pull and it builds rather quickly. The results are indeed being written as 644 and owned by root, but can still be read by the 'world.' Thank you.


feat(Debounce): customisable debounce

wrapped out scoped functions in a factory
created a provider
injected the factory in the directives

closing this as it's too old!


Revert "Include Schemas in NodeContext"

This reverts commit 58a6a3ffe4fa0a2b70600f1f417a6d7382b96ed0.

Should be reverted in 5.7 as well?

Should be reverted in 5.7 as well?

It's reverting for 5.7 as we decided not to change behavior of has_schema_privilege after all.

It's reverting for 5.7

sure, sorry 🤦


Update Containerfile

Remove context dependency when build
Change install way for charon to wheel

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 66.447%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1915

Relevant Lines:
2882

💛  - Coveralls


feat: zeva-2112 - refined emails

@kuanfandevops a new environment variable called FRONTEND_URL_BASE needs to be added; value should be "https://zeva-dev.apps.silver.devops.gov.bc.ca" in dev, "https://zeva-test.apps.silver.devops.gov.bc.ca" in test, and "https://zeroemissionvehicles.gov.bc.ca" in prod. Thanks!


Raise error for incorrect JSON serialization

Raise error when lines = False and batch_size < Dataset.num_rows in Dataset.to_json().
Issue: #7037
Related PRs:
#7039 #7181

PTAL @lhoestq @albertvillanova

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Multi-GPU Training - sagemaker + accelerate

I'm trying use the following code
from sagemaker.estimator import Estimator

estimator = Estimator(image_uri='aaa.dkr.ecr.us-east-1.amazonaws.com/training',
                      role='arn:aws:iam::aaa:role/sagemaker-role',
                      instance_count=1,
                      entry_point='./train.py',
                      instance_type='ml.p4d.24xlarge')

estimator.fit()

Inside the train.py script there is a usage of Accelerator with initialization in the following way:
    accelerator = Accelerator(
        gradient_accumulation_steps=args.gradient_accumulation_steps,
        mixed_precision=args.mixed_precision,
        log_with=args.report_to,
        logging_dir=logging_dir,
    )

However, when monitoring, we can see that the training code utilizes one gpu only.
What is missing? Is there a proper way to do accelerate config inside the custom container for Sagemaker?

cc @philschmid

@xenia-kra the Estimator/ SageMaker is using python train.py .... to execute your script. That should be visible in the logs of your jobs. Thats why you are only leveraging one GPU.

@philschmid so it's not possible to use multi-gpu for custom container at al? Which side is the blocker in this integration - sagemaker or accelerate?


[1.16.5] Crash when trying to paint Inventory Hopper

To be honest, I don't know if Inventory Hoppers are paintable, but I decided to test it and my server crashed because of said test 😂😂
Crash Report = https://mclo.gs/ErWGbyu
Should I upload the latest server log? If so, let me know in the comments!

*playing the game*
*clicks inventory hopper with paint bucket*
💥💥💥💥

✨🎉


ctrl+w is closing 2 editors when custom webview editor image preview is focused

#81698
It's closing the webview editor as well as the next editor. There may also be an issue with editorIsOpen context key because this keybinding triggers when only the webview is open.

I've tested this on MacOS and Ubuntu can can't repo this. Any specific steps required?

Seems fixed now


Add Jackson XML proguard rules to the SDK

We can't build the samples with the following Proguard rules
-keepnames class com.fasterxml.jackson.** { *; }
-dontwarn com.fasterxml.jackson.databind.**

We need to figure out how we can add these rules to the SDK so that applications will not need to add the rules themselves

Already


solove the problem that selinux doesn't work

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage
/sig storage

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

In the last version,  go-selinux use  conditional compilation by a file name suffix. It doesn't need to add tags.

@chenyufeifei Thank you for raising pull request. please fix golang lint error.


Update documentation to add cache explaination

This updates the documentation explaining how to setup any cache that implements the Doctrine interface. As the Symfony documentation don't clearly shows how to do this, I intend to add it here.

This is a bit complex.
We coule simplify usage of cache for class metadata by creating a service serializer.cache that is null by default.

I agree.
Do you have the time to make the necessary changes?

I have maked the Pull Request 1484

@jlHertel your PR related to the validation service, while this PR is about the serializer ;) Thanks for the work!

@SpacePossum, Ok, you're right.
Any subjection to merge this? It will improve the documentation showing how to better setup serializer. Or maybe should I write something more simple?

Your docs helped me to get things going faster, however I've no powers here (to merge) :)

Ok, I will wait for someone to merge it.
In the meantime, you can look at my blog for some tips.

Thank you @jlHertel.


Toggle overflow menu

NOTE: This PR includes https://github.com/carbon-design-system/carbon-components/pull/273, because this PR depends on https://github.com/carbon-design-system/carbon-components/pull/273. The change for this PR itself can be seen in: https://github.com/carbon-design-system/carbon-components/pull/426/commits/4a750317fc20506724e605f0d147b8053b62365a
Overview
Resolves #423.
This PR looks at the state of overflow menu's open/close state right before the trigger button is clicked to determine if such clicking action should open or close the overflow menu. Older code looks at such state after the the menu, which is a separate element from trigger button, gets out of focus upon clicking on trigger button and thus closed, which caused #423.
Testing / Reviewing
Testing should make sure overflow menu is not broken.

These changes look great! Thanks for fixing this @asudoh. If you have time, could you also take a look at https://github.com/carbon-design-system/carbon-components/issues/436 and perhaps bundle that fix in?


Additional setting, that enables the popover to automatically avoid bottom screen edge

There is a 3rd setting for popoverType now. Choosing it will make popover attempt to select proper type, .up or .down, depending on how much room there is left to the bottom screen edge. Defaults to .down if anything goes wrong.
Usage:
popover.popoverType = .auto
The default type (which is .down at the moment) was not changed, because this new .auto option might probably need better testing on different orientations.

Thank you for reflecting. This is a wonderful job 👍


211 code docs looks incorrect

https://github.com/warpnet/salt-lint/wiki/211
This code documentation looks incorrect.
In the top of the file pillar.get['item'] is described, but salt fails with that.
Shouldn't be this pillar['item'] ?

I think you're right, @arbulu89.
Additionally, I'm curious if this documentation could be expanded a bit for 211? I'm working in a legacy salt 2017 codebase, and while onboarding salt-lint, I found the incorrect pattern is used frequently. How wrong is, e.g. {{ pillar.get('data_volume_encryption', 'none') }}? –the [211] [HIGH] pillar.get or grains.get should be formatted differently warning suggests it's very wrong. Why?

You're absolutely right. Thank you for the feedback, we'll try to look into this soon.

The rule should have spotted the usage of pillar.get['key'] or grains.get['key']. All example on the wiki are actually correct examples and will be rendered in Jinja without any problems.
Using pillar.get['key'] results in:
local:
    Data failed to compile:
----------
    Rendering SLS 'base:example' failed: Jinja variable 'method object' has no attribute ‘key'

Using grains.get['key'] results in:
local:
    Data failed to compile:
----------
    Rendering SLS 'base:example' failed: Jinja variable 'builtin_function_or_method object' has no attribute 'key'


add repository and bugs properties to all packages package json files

WHY are these changes introduced?
In addition to https://github.com/Shopify/cli/pull/424.
Included a link to the repository on the NPM package (etc.) itself.
WHAT is this pull request doing?
dds repository and bugs properties to package.json of all packages.
How to test your changes?

Measuring impact
How do we know this change was effective? Please choose one:

[ ] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[x] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[x] I've considered possible documentation changes

Hi @gonzaloriestra
I thought it might as well be helpful to include this in the rest of your packages.

Thanks for you help! I think it only makes sense for published packages, which are these ones if I'm not wrong:

@shopify/cli
@shopify/cli-hydrogen
@shopify/create-app
@shopify/create-hydrogen
@shopify/app
@shopify/cli-kit
@shopify/theme
@shopify/shopify-cli-extensions

Replaced by https://github.com/Shopify/cli/pull/432.


Add a search to the developer list page

Currently the developer list page is just a list of developers, would be good to have certain fields filterable.

will love to work on this feature

@rcoffie Perfect, I've assigned the issue to you! Let me know any questions or ideas for the initial implementation 👍


Enhanced logging to help troubleshooting, DNS failures

Do you already have an issue opened with F5 support? No
Description
I was working with a customer last week troubleshooting their IAC deployment of BIG-IP, we were getting inconsistent results where the DO declaration would succeed in clustering the devices but the CFE declaration would result in error code 400. The root cause was an external DNS server and an external gateway route being applied during DO which resulted in DNS queries failing after that point. The behaviour of CFE extension was inconsistent, GET to  /declare would generally work. /inspect would time out /reset would time out with not helpful logs in restjavad and no errors in restnoded.
It would be great if CFE would report dns failures to help troubleshoot.
I can provide the TF project to reproduce.
Environment information
For bugs, enter the following information:

Cloud Failover Extension Version: 1.11.0
BIG-IP version: <IP_ADDRESS> or <IP_ADDRESS>
Cloud provider: AWS

Severity Level
For bugs, enter the bug severity level. Do not set any labels.
Severity: 3

Sorted with changing log level to Silly! https://clouddocs.f5.com/products/extensions/f5-cloud-failover/latest/userguide/logging.html?highlight=silly


Issue with annotation rules that are considered as existing rules

Describe the bug
This bug appears while trying to add an annotation rule on weak annotation mode.
On the picture below, I try to add a rule from the query text:"jobs". This rule isn't part of the rules I already set, however it is considered by Argilla as "already saved as rule".

After two clicks on the "Workforce & organizational" button, the message disappears and it becomes possible to add the rule, reaching the second picture below. The number of rules increased from 10 to 11 as well as the total coverage.

Environment (please complete the following information):

OS [e.g. iOS]: iOS
Browser [e.g. chrome, safari]: Chrome

Additional context
Argilla instance used by Datamaran

Same here using argilla server version 1.2.1 and Chrome Version 111.0.5563.64 (Official Build) (x86_64)
https://user-images.githubusercontent.com/2077497/225859107-8f945dfb-93ce-4f9c-b2a0-dacd5b3f51c1.mp4

A correction of this problem have been implemented on this issue #2117


When clicking the icon in the taskbar it opens with a considerable delay

Still figuring out a good solution currently.

Thanks anyway, the idea it's great... It's still usable... But that 1
second delay every time gets in my nerves.
If there's any solution in the future let us know... It's a great app
El dom., 13 jun. 2021 4:18 p. m., Pike @.***> escribió:

Still figuring out a good solution currently.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tjackenpacken/taskbar-groups/issues/145#issuecomment-860264881,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APWPMELH4NR2RAPJAABCRO3TSUHBVANCNFSM46UBOJSA
.

https://github.com/PikeNote/taskbar-groups-pike-beta/issues/2
That is what I am planning thus far, but implementation will be a bit slow due to school work and such, but my goal is to implement that after some other features are done.

Awesome men, thanks for the update! I really appreciate it
El dom., 13 jun. 2021 4:28 p. m., Pike @.***> escribió:

PikeNote#2 https://github.com/PikeNote/taskbar-groups-pike-beta/issues/2
That is what I am planning thus far, but implementation will be a bit slow
due to school work and such, but my goal is to implement that after some
other features are done.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tjackenpacken/taskbar-groups/issues/145#issuecomment-860265956,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APWPMEJD46OJ3JWYKDVQSTDTSUIFTANCNFSM46UBOJSA
.

I also experience this issue, so much delay before opening apps!


How to deal with Postgres Enum columns?

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the SQLModel documentation, with the integrated search.
[X] I already searched in Google "How to X in SQLModel" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to SQLModel but to Pydantic.
[X] I already checked if it is not related to SQLModel but to SQLAlchemy.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from typing import List, Optional
from enum import Enum

import sqlalchemy as sa
import sqlmodel

class Status(str, Enum):
    active = "active"
    banned = "banned"

status_enum = postgresql.ENUM("active", "banned", "c", name="status_enum")

class User(sqlmodel.SQLModel, table=True):
    __tablename__ = "auth_user"
    id: int = sqlmodel.Field(primary_key=True)
    status: Status = sqlmodel.Field(status_enum)
    password: str

Description
How to use Postgres Enums together with SQLModel?
Operating System
Linux
Operating System Details
No response
SQLModel Version
0.0.3
Python Version
3.8.11
Additional Context
No response

@gregsifr, could you please reopen the issue for the reasons stated above?

Apologies, I misunderstood and thought that since there is a PR for it there's no point to keep it open.

Hey there! I think this was possibly solved in https://github.com/tiangolo/sqlmodel/pull/165. That's available since SQLModel 0.0.7. Could you confirm?


Feature request:  do not reload latest checkpoint on each DNNRegressor.predict() call

Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Each DNNRegressor.predict (or predict_scores) call reloads model parameters from the latest saved model checkpoint, even if the checkpoint hasn't changed between predict() calls.  This slows down generation of predictions.  It will be helpful to be able to disable reloading of model parameters after the initial loading, and/or to be able to reload the latest checkpoint manually via a separate function call.

@honkentuber can you comment on this one?

train, evaluate, and predict always build the full graph and load from checkpoint. You can get multiple predictions, while loading the graph only once, by feeding multiple batches via input_fn and iterating over the results.
@ispirmustafa @martinwicke

You can also use the generator_input_fn if your inputs come via a generator.
We like to keep the Estimator state 100% on disk. This is crucial for distributed training since we wouldn't want to maintain a consistent state ourselves.

The specific use case is:

input data for prediction arrives at random points in time;
DNNRegressor.predict() is invoked to generate prediction.
As it stands, there is approximately 3-4 seconds latency for each prediction when running on Google App Engine flex with model_dir in a Google Storage bucket.  The latency seems to be to a substantial extent due to reloading the model from a gs:// path.   Given that data for prediction arrives in online mode (i.e. it's not available ahead of time), how can input_fn be written to return multiple batches?  The ultimate goal is to minimize latency when generating each prediction, from several seconds down to milliseconds.

You can also try contrib/predictor.


Implement Basic caching using redis and mongodb

can you assign it to me @iamgoeldhruv
i am interested to work on this issue

@AyushDharDubey
please merge my other PRs and assign me this issue. looking forward to work on this one, thanks.

hi @disvid,
before we assign the issue to you, we’d like you to share your approach, just a rough idea of how you’re planning to implement the aforementioned.
also, if there’s anything unclear, this would be a good time to seek guidance.


PlaceHolderTexture2D calculates wrong size in Combination with TextureArray2D

Godot version
4.0 Alpha 15
System information
Win 10
Issue description
If you make a Sprite with its texture set to a PlaceHolderTexture2D and use it to render a Texture2DArray it ends up bigger than the PlaceHolderTexture2D.
Steps to reproduce
Example:

PlaceHolderTexture2D is 16x16
Texture2DArray is a bunch 16x16 images
-> The rendered sprite ends up being 16x16*4 instead of 16x16
I have tried around a little bit and it always seems to be 4 times the size it should be.

Minimal reproduction project
TextureArrayTest.zip

I think I am doing something wrong and there might be issues with the minimal reproduction project. I have rechecked and the size of the placeholder appears to be correct. Might be an issue with the shader.


Select all checkbox only works one-time

When creating a "select-all" checkbox with some simple jQuery, "awesome-bootstrap-checkbox" only works one time!
On the first click i give all checkboxes the checked attribute, and on the second click i remove all checked attributes (also tried setting checked attribute to false, but same result).
The first way around works perfectly, but when repeating the process (thus selecting all for the second time), nothing happens to the awesome-bootstrap-checkbox replacements. When checking the DOM, the input fields themself are working.
Here is my example:
https://jsfiddle.net/abkvzt7v/

@jbweb you probably should use 'prop' method instead of 'attr'.
Working fiddle: https://jsfiddle.net/4yqyowpj/

@nikrabaev, Thanks!


0.7.0 on 5.3.3: Integration issues with createUiLayer on application didFinishLaunchingWithOptions

Currently integrating our app using a combination of https://github.com/blitzagency/ios-unity5 and the SDK
When I run our app with cardboard, I get the following error message:
2016-04-14 15:45:37.689 MyiOSApp[1221:929504] -> registered mono modules 0x196529c
-> applicationDidFinishLaunching()
2016-04-14 15:46:11.748 MyiOSApp[1221:929504] +[GOOTypography robotoMediumWithSize:]: unrecognized selector sent to class 0x1c55050
2016-04-14 15:46:11.954 MyiOSApp[1221:929504] Uncaught exception: NSInvalidArgumentException: +[GOOTypography robotoMediumWithSize:]: unrecognized selector sent to class 0x1c55050
(
	0   CoreFoundation                      0x23fae303 <redacted> + 150
	1   libobjc.A.dylib                     0x2377adff objc_exception_throw + 38
	2   CoreFoundation                      0x23fb39d1 <redacted> + 0
	3   CoreFoundation                      0x23fb170f <redacted> + 702
	4   CoreFoundation                      0x23ee0c88 _CF_forwarding_prep_0 + 24
	5   MyiOSApp                        0x017b3831 -[GCSTransitionView loadView] + 1908
	6   MyiOSApp                        0x017b3063 -[GCSTransitionView initWithFrame:delegate:] + 190
	7   MyiOSApp                        0x017abdff -[GCSOverlayViewController init] + 326
	8   MyiOSApp                        0x017a4517 -[GVRUnityController init] + 170
	9   MyiOSApp                        0x017a4daf createUiLayer + 278
	10  MyiOSApp                        0x004d08a5 -[CardboardAppController createUnityView] + 168
	11  MyiOSApp                        0x003b4413 -[UnityAppController application:didFinishLaunchingWithOptions:] + 1022
	12  UIKit                               0x281ba64f <redacted> + 394
	13  UIKit                               0x283e3033 <redacted> + 3086
	14  UIKit                               0x283e6f0f <redacted> + 1570
	15  UIKit                               0x283fac15 <redacted> + 36
	16  UIKit                               0x283e43f7 <redacted> + 134
	17  FrontBoardServices                  0x252bfc75 <redacted> + 232
	18  FrontBoardServices                  0x252bff61 <redacted> + 44
	19  CoreFoundation                      0x23f71257 <redacted> + 14
	20  CoreFoundation                      0x23f70e47 <redacted> + 454
	21  CoreFoundation                      0x23f6f1af <redacted> + 806
	22  CoreFoundation                      0x23ec1bb9 CFRunLoopRunSpecific + 516
	23  CoreFoundation                      0x23ec19ad CFRunLoopRunInMode + 108
	24  UIKit                               0x281b3a17 <redacted> + 526
	25  UIKit                               0x281adfb5 UIApplicationMain + 144
	26  MyiOSApp                        0x00c898f4 main + 428
	27  libdyld.dylib                       0x23b74873 <redacted> + 2
)
2016-04-14 15:46:11.956 MyiOSApp[1221:929504] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '+[GOOTypography robotoMediumWithSize:]: unrecognized selector sent to class 0x1c55050'
*** First throw call stack:
(0x23fae2eb 0x2377adff 0x23fb39d1 0x23fb170f 0x23ee0c88 0x17b3831 0x17b3063 0x17abdff 0x17a4517 0x17a4daf 0x4d08a5 0x3b4413 0x281ba64f 0x283e3033 0x283e6f0f 0x283fac15 0x283e43f7 0x252bfc75 0x252bff61 0x23f71257 0x23f70e47 0x23f6f1af 0x23ec1bb9 0x23ec19ad 0x281b3a17 0x281adfb5 0xc898f4 0x23b74873)
libc++abi.dylib: terminating with uncaught exception of type NSException

Any thoughts or issues? No records of "GOOTypography" exists anywhere with googling. Am I missing a plugin on the transferral/integration process? Much thanks.

There should be some .bundles in Assets/Plugins/iOS which can't from importing the latest SDK.  Did you update your project from an earlier version?

Indeed, I am using all 5 of the bundles and imported them, and this didn't do too much. I was assuming it was MaterialTypography.bundle that you suspect missing?

OK, so the actual error appears to be this:
2016-04-14 15:46:11.748 MyiOSApp[1221:929504] +[GOOTypography robotoMediumWithSize:]: unrecognized selector sent to class 0x1c55050
I'll ask one of the iOS engineers on the team about this.

Can you double check that the -ObjC flag is enabled in the Xcode project's "Other Linker Flags"?  The Cardboard plugin is supposed to add that automatically, but something might have prevented it.

I think including the ObjC flag did it, thanks!
Also another note for people trying to upgrade, I also marked minimum iOS version to ** 9.0** as I got a compile error setting up ObjC in the libvrunity.a file.

Other Linker Flags NO OK


Analysis of Local Transportation

After to analyze six kinds of suspects reimbursement we found R$240 million to be investigated. There are 1,031 congresspeople and 24,517 companies in this list.

@cuducos and @jtemporal,
Thank you so much for your patience and sorry about my messy. I am really learning a lot with this project and already found a course of Git/Github.  :)
Do you think these kinds of suspicious reimbursements could be add on Rosie? If yes, they would be automatically updated for Senate data? Or I should do the same analyses to Senate dataset?

Hi @fabiocorreacordeiro, many thanks for the fixes. IMHO it's ok to merge — what do you think @jtemporal? I'll leave the Merge button for you this time.

About Rosie and the future of your analysis
When Rosie runs she always start with no dataset at all and generate them from the original sources. In the casa of the analysis of your notebook, for example, she will download fresh XML from the Chamber of Deputies, convert them to CSV, clean and merge it into something similar to `2017-03-15-reimbursements.xz' — but that will have all data released by the government until the day Rosie is running at. In sum, yes, Rosie automatically update its datasets every time she runs ; ) She's pretty cleaver, huh?
Considering your hypothesis they are very good and very revealing. I do believe they will foster very interesting analysis, new hypothesis and strategies but… there's always a but… personally I'm not confident about taking legal action bases solely in your notebook. But this is not a critique to your work — I mean it. Let me walk you through my rational for that.
Worst case scenario I can see lawyers in court dismissing your assumptions with bullshit like but my client drives a truck which uses diesel, way more expensive than gas, or my client drives a lot to stay in touch with their electorate, he works 24/7 which a taxi driver usually don't etc. Or yet my client don't rent an Onix, he rents a bulletproof sedan which costs X thousands a week.
Beyond that if the law say they can spend up to 6k BRL with gas…well… we're fucked.
Or not. Because I do believe your analysis has a great appeal to be the base of media stories. In other words we don't use it to sue anybody because legally speaking I think it would not worth it. But we use you analysis to call the law itself into question. We have good connection with journalists  and I'm pretty sure many of them would love to write stories based on your analysis, isn't it @rodolfo-viana?
And… if I'm wrong, to want to be credited if journalists reach us or do you prefer to remain anonymous. I can say a contributor called Fabio Correa Cordeiro did it, but if you prefer I go with awesome contribution came from the community ; )
And… again: many many thanks for your contribution, mate! Cheers

Hi @cuducos and @jtemporal,
I completely agree that some hypotheses are immoral but not illegal. To contact journalists could be a good strategy. I will talk with my sister, she worked on O Globo and could help us.
But think about the "Red Flag" and "Yellow Flag" cases (one single reimbursement above the max estimate cost). They look very similar the meal expenses above of the mean. Maybe in these cases, we could try to add on Rosie code.
If some journalist uses the notebook I would like to be credited. Will be an honor!

Hi @fabiocorreacordeiro, @cuducos, @jtemporal,
Cuducos is right. Jornalists work not only with what is illegal but also with what violates moral principles. The purpose is to show society who are the people they are voting for. I myself write a lot of pieces containing no information on illegal activities but full of data that hopefully delivery to society resource to debate.
This is an example (not related to Serenata or the Chamber of Deputies) of a story on controversial-but-not-necessarily-illegal actions.
So, in short, your work is gold for media. :)
And every serious journalist -- and I assume your sister is one of them -- will take these data of yours and reach congresspersons for a reply, a comment, an explanation, their versions of these expenses. So media outlets take for themselves the legal responsability of checking the facts with people involved.
As a journalist, I would just consider a few things (and this recommendation goes to every analysis sent to journalists):

Media does not care so much about data concerning past legislature.  That's because few former congresspersons would comment on actions they took years ago. "So why bother?," would a reporter think.

Journalists in general are good at writing and awful at numbers. So deliver the results in the simplest way possible. A table with congressperson names, states, parties, reimbursement values, names of the company and links to the receipt would work fine. Other that than you may confuse the reporter.

Always make it clear how you want to be credited (e.g. "Fabio Correa Cordeiro, cientista e integrante da Operação Serenata de Amor, projeto que audita gastos da cota parlamentar dos deputados").

I hope this may help you with a perspective from a media worker. :)

@fabiocorreacordeiro this looks awesome! congrats 🎉
before I merge it (which I will pretty soon) I would like to request a minor check from you, when I was looking the .ipynb file on my browser - along side with other issues - I noticed a problem with the notebook as the image below shows:

Since there is a written conclusion in the .py would you mind re-running the notebook again, saving it and then committing it again? I believe it was a problem when it was saved ;)
Other than that great work o/

@jtemporal,
I really don't know why the conclusion didn't appear, but after rewrite finally worked :)
@rodolfo-viana,
Thanks for the tips.
I am thinking about to do some data visualization. I hope to help the journalists and the general public to have a better understanding. On these visualizations, we could separate the suspicious reimbursements per legislature.
At the same time, I will start the same analyses on senate dataset.
Thanks, everybody!

@rodolfo-viana — many many thanks for the awesome comments! We should add that somewhere in our documentation <3
@fabiocorreacordeiro again: your contribution is gold. But I would like to take the opportunity to discuss my argument a little bit further, precisely given your answer:

But think about the "Red Flag" and "Yellow Flag" cases (one single reimbursement above the max estimate cost). They look very similar the meal expenses above of the mean. Maybe in these cases, we could try to add on Rosie code.

Short answer: no, it's not similar. The core difference is that the meal outliers classifier takes an outlier (a suspicion) based on the total amount spend at the same location. So if we're talking about a restaurant we are talking about the same food being served. This kind of analysis is coherent with data that are factual: how much each congresspeople paid for a meal when they were at that exactly same restaurant.
Your analysis is very good but it depends on assumptions: how much does a taxi driver drive a day, how much does a car cost, how much does fuel cost. These are not facts and can be called into question way easier than… facts. As I said before: surely Onyx costs that price, but what if they use a different model equipped differently (bulletproof car rentals costs way more)? what if they go for trucks with diesel instead of gas? what if a representative from the country side has to travel 20km between each voter residence to be in touch with their electorate?
So the key point here is that your assumptions gave us a good leverage to reflect on data — and that is really awesome. But on the other hand in courts (or elsewhere) these assumptions are easily called into question. That's why I suggest whoever is reading this takes your analysis mostly form the moral perspective, and not from the legal perspective.


chore(tests): enable end-to-end test workflow

Issue number: #1435
Summary
This PR enables our new E2E framework to run on push and manually.
It also (1) refreshes the contributing guide to include our testing definition, and (2) maintainers playbook to include more info on the E2E test structure.
Changes

Please provide a summary of what's being changed

User experience

Please share what the user experience looks like before and after this change

Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of this change
[ ] Changes have been tested
[x] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

View rendered CONTRIBUTING.md

Codecov Report

Merging #1470 (6148c3d) into develop (122f989) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop    #1470      +/-   ##
===========================================
- Coverage    99.89%   99.89%   -0.01%
===========================================
  Files          123      123
  Lines         5499     5498       -1
  Branches       629      629
===========================================
- Hits          5493     5492       -1
  Misses           2        2
  Partials         4        4

Impacted Files
Coverage Δ

aws_lambda_powertools/event_handler/appsync.py
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.

It was a trap! I thought it was ready for review!

It was a trap! I thought it was ready for review!

HAHAHAHA almost there!

Ready to go @rubenfonseca !


Inc cache num

Note: the cache was increased to check that the fix works, see the CI runs related to commit https://github.com/PennyLaneAI/qml/pull/428/commits/fbac9bd63a0c5facb605d695edf3d9c84f915df2.


explore paranoia action

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.876%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1493

Relevant Lines:
1780

💛  - Coveralls


Extend Ruff linter config to import sorting with isort

See Frischmuth's pyproject.toml ("I" is the relevant ruleset for import sorting).
This should get tested locally by team members before adding it to main – there were some (non-obvious) discrepancies between how Ruff picked up on settings on my system vs. how it did (not) on GitHub. ... IIRC my Ruff infered src and known-first-party without them having to be set explicitly.

there were some (non-obvious) discrepancies between how Ruff picked up on settings on my system vs. how it did (not) on GitHub. ... IIRC my Ruff infered src and known-first-party without them having to be set explicitly

could you explain what exactly that means? I fail to see exactly what exactly I should be testing for


Public Server Compatibility && Make the Bot Compatible with the new and public server

ideally make the bot multi server compatible in a generic method

Closing issue as this has been completed

@Sarahtonein  If you link issues to PRs they can be automatically closed once they are merged


add launchd documentation to the cluster-on-startup page

There are currently no explicit instructions for MacOS/launchd on our cluster-on-startup page. We should probably add this case to the documentation.

Do we need to provide a special script or can launchd be made to use our SysV init script?

With launchd it is only a configuration document, not a script. So it would be different.

It might be good to mention lunchy as a friendly wrapper for launchctl.

I'm going to wait until #643 gets through code review to tackle this. Do we include a .plist file for launchd? If not we can probably use Homebrew's as a reference point. (And perhaps recommend that people install through Homebrew, or at least mention it?)


Site bindings get overwritten with defaults during :config

Resolves #146

@ksubrama simple change to fix issue with binding usage.


Allow API docs to be viewed locally and on snapshots

Also fixes an issue with history back

snapshot for this PR available at:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/DOCS/refs/pull/1253/merge

snapshot for this PR available at:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/DOCS/refs/pull/1253/merge


Format the whole codebase with JuliaFormatter.jl

Codecov Report

Merging #71 (5faccb0) into main (a2e6b60) will decrease coverage by 0.11%.
The diff coverage is 82.48%.

:exclamation: Current head 5faccb0 differs from pull request most recent head 585e514. Consider uploading reports for the commit 585e514 to get more accurate results

@@            Coverage Diff             @@
##             main      #71      +/-   ##
==========================================
- Coverage   88.34%   88.22%   -0.12%
==========================================
  Files          22       22
  Lines         884      892       +8
==========================================
+ Hits          781      787       +6
- Misses        103      105       +2

Impacted Files
Coverage Δ

src/algorithms.jl
100.00% <ø> (ø)

src/diffusions.jl
80.80% <42.85%> (ø)

src/solution_plotting.jl
67.56% <50.00%> (ø)

src/ProbNumDiffEq.jl
72.22% <61.53%> (ø)

src/preconditioning.jl
65.00% <69.23%> (ø)

src/filtering.jl
88.00% <75.00%> (ø)

src/gaussians.jl
70.00% <75.00%> (+3.33%)
:arrow_up:

src/initialization/rungekutta.jl
85.24% <77.77%> (-2.89%)
:arrow_down:

src/solution.jl
93.58% <80.00%> (ø)

src/alg_utils.jl
77.77% <100.00%> (ø)

... and 12 more

Continue to review full report at Codecov.

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


Improve font loading

Currently, the Salt website has a bad case of FOUT when loading. We should optimize the way we load the fonts:

Can we make the unstyled case less jarring?

It seems to be using a serif font, which obviously looks totally different. Can we add some system fonts and sans-serif as fallbacks?
I think we can setup @font-face rules to configure system fonts to have similar metrics to OpenSans by fiddling with letter spacing and stuff like that. The idea is we can have a fallback font that displays immediately but is visually as close as possible to the proper OpenSans font. When that gets swapped in, the change is less noticable and will also cause less layout shifts.

Can we reduce the download? For example, by:

Reducing the number of font files we load by limiting the number of weights and styles we use in our designs
Maybe we can subset the font files to cut down file sizes
Explore using variable font version of OpenSans instead

Can we improve caching of the font files?

There might be some HTTP headers we can configure on the web server to maximise how long browsers will cache the font files, once they have been loaded.

Further reading:

How to avoid layout shifts caused by web fonts

Fixed by #1509


Missing resistor?

If you calculate the DAC output impedance for the different input values, you will find that the value is close to R/3 except for S[2:0]==3'b111. If you add another resistor of value R before the the first 1/2R resistor, the output impedance becomes exactly R/3 for all S values, which makes more sense.
Keeping a constant output impedance is important in DAC design to prevent non linearities. So I think there is probably a missing R in your schematic.

I read your comment and drew the Thevenin circuit. As you say, if there is an R connected to +VDC, the impedance will be R/3 at any tap. But that resistor really doesn't exist!
I double-checked the R2R DAC because if the impedance of the R2R DAC is R/2 instead of R, then the impedance at any tap would be R/3. However, the poly resistor elements in the analog shifter and the R2R DAC have same width and height. Unless Yamaha did something special, the resistances should all be the same.

The number of resistors connected to +VDC is five, not six that makes all the tap impedance R/3. There is a box that I have marked with "??", but I am not sure if this is a resistor connected to +VDC.

Thank you for double checking the die shot. The narrow poly stripe marked as ?? does not seem to be connected from the left side, is it? If it was, the other side is indeed the right net. But the resistance would not be R, as the stripe is too narrow and too long.
I simulated the circuit and indeed the impedance for S[2:0] codes from 7 to 1 is 0.5R, 0.375R, 0.344R, 0.336R and from that point on 0.333R.  I have tried fixing the circuit in different ways with the components present in the schematic (swapping resistors, changing the switch polarity...) But the only solution that makes a constant output impedance is adding an R-valued resistor here.

Yes. The stripe I marked with ?? doesn't look like a resistor at all - only one node is connected.
I looked at the die shot again, but couldn't find the resistor you're talking about. Honestly, I'm starting to wonder why it's not there either. As I said, it's possible that the impedance of the R2R DAC is 1/2R, but it doesn't seem to be a correct assumption considering the width and height of the poly resistor. I think its impedance is R, resulting in the output impedance of the tap being very strange.


upgrade from 0.46.5 to 0.47.0 breaks a react project

Upgrade from 0.46.5 to 0.47.0 breaks my build (a react SPA project). Below is the diff before and after, running unocss cli:

and the only places with the word container:

and my unocss.config.ts file:

I don't know what your problem is, does it mean that many styles are generated?

and the only places with the word container:

Uno will scan your file source code, extract tokens as much as possible and generate styles.

I don't know what your problem is, does it mean that many styles are generated?

Yes, extra css styles are generated, but none of them exists in class="xxx" or className="xxx".
Like the word container, it on exists as a variable name in my source code. So I expect it should not be wrongly extracted.

if I uncomment presetAttributify to include it back, more strange changes will happen:

https://github.com/unocss/unocss/blob/b19f5955112988b7a0f4916fc3273d68e69d159c/packages/core/src/extractors/split.ts#L25-L31
It comes from the split extractor. If you don't want use it, you can add it to blocklist.

if I uncomment presetAttributify to include it back, more strange changes will happen:

You'd better provide a minimal reproduction and explain the operation steps.

No problem, will provide a codesandbox reproduction after holiday. Happy new year 🎆

No problem, will provide a codesandbox reproduction after holiday. Happy new year 🎆

Happy new year 🎉

here is the reproduction repo (https://github.com/yckbilly1929/uno-demo)
switching from "~0.46.5" to "0.47.0" and rerun pnpm build, you will see that there is difference in src/uno-gen.css

Hi, thanks your reproduction repo, i reproduced it. It is indeed a bug, maybe the wrong token was extracted when uno scanned the file. I'll fix it in my free time, maybe a little later.

@yckbilly1929  Use presetOnly with presetAttributify, you should use prefix before your tokens, like un-bg-red.
As for your repo(https://github.com/yckbilly1929/uno-demo), i see those css was generated without un-(prefix) when i run build commond. Maybe that was a bug before, but you need to follow the meaning of this property when you use.

issue still exists in 0.50.0 🤔


Remove the codea extension for better readability.

Looks pretty straightforward, this change for the dependency picker only?

Just saw your reply and this makes me realize there's a problem because this path doesn't include the project folder. I'll see what I can do. It's not major, it only means the explorer won't refresh properly after adding a dependency.


Feature/cax 1 prs@166 OpenAPI contract and integration test

Created one controller endpoints, and integration test for OpenAPI contract. The other query endpoint will follow in a subsequent PR.
The implementation of the service returns a stubbed data set for now.

The module naming seems messy, I will try to improve this.


Fixes #288 Auto close DrawerLayout on selecting particular stream

This PR fixes issue #288. DrawerLayout is now closing itself as soon as click is made.

Closing it for now. Will reopen soon with ExpandableListView bug fixed.


No Social Badges

Hey,
I have a little issue. The social badges arent showing up in my projet.

Do you know what it could be?
Thanks a lot, awesome work :)

same is the case with me did you find a solution @Cyb3rGh05t

Hey @Cyb3rGh05t I guess I found the issue, you must have not used the "FontAwesome" icon toolkit. bootstrap-social works on it.
Here's the CDN of free version  :

yep.. the icons are not part of the project


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 2322 ms

Resolved: Bofrost.lu performance has improved in ceb5c44 after 1 hour, 4 minutes.


ENH(shells): window functions for linear and cubic interpolation

Adds window functions linear_windows() and cubic_windows() that correspond to linear and cubic spline interpolation, respectively.
Documentation is updated to give an overview of available window functions.
Closes: #118
Added: New linear_windows() and cubic_windows() window functions for shells.

Plot of the new window functions from the docs: