Spanish - Dropdown options are not translated until page refresh in room settings and track distance when language changed in another session

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to Workspace - Reimburse Expenses
Observe Track Distance Unit values
Change language in another session
Observe Track Distance Unit values
Go to room settings
Observe the Notify me about new changes
Repeat step 3
Check the Notify me about new changes

Expected Result:
Track Distance Unit dropdown values should be changed to new selected language
Actual Result:
Track Distance Unit dropdown values are not changing to new selected language
Workaround:
unknown
Platform:

Where is this issue occurring?

Web

Version Number: 1.2.41-1
Reproducible in staging?: y
Reproducible in production?: y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/208438382-2757761a-1bdd-4a80-adb1-4d14e5fb672e.mov
https://user-images.githubusercontent.com/43996225/208438397-2766a8b5-9b3b-48ca-889f-32f4b112261e.mp4
https://user-images.githubusercontent.com/43996225/208440352-9d5ae039-158f-4449-bce1-f63771c2bea2.mov
Expensify/Expensify Issue URL:
Issue reported by: @varshamb
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1671297275829189
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0117d024ede160bf14
Upwork Job ID:<PHONE_NUMBER>016178688
Last Price Increase: 2022-12-20

I can work on this!

PR here

Why didn't the title update on this? @eVoloshchak is due payment for the PR review, right?

Offers have been sent to you both @eVoloshchak and @varshamb on this job.

Okay settled up:
@eVoloshchak - $1k for the internal PR review
@varshamb - $250 for the bug report.
Closing, thanks! 👍


Extension: Similarity transformation for Point-to-Point Error Minimizer

Hi
I extended the Point-to-Point error minimizer by a scaling factor.
The implementation is in the branch similarity_transformation: https://github.com/bkueng/libpointmatcher/tree/similarity_transformation
It works but requires proper integration into the library (create a separate transformation class based on Transformation and add a new SimilarityErrorMinimizer, ...), but I currently don't have the time to do it.
Derivation is based on Umeyama1991 "Least-Squares Estimation of Transformation Parameters Between Two Point Patterns", which describes the minimization without weights. The following shows my derivation with weights, using the Lemma from Umeyama1991:

Great, that will give a good head start once we move forward in extending the transformation capabilities of the library.

I tested that this functionality works.
I would be willing to make a carefully prepared pull request for merging this into the main library. Is there interest in that?
I am thinking of making a new class, called say PointToPointSimilarityErrorMinimizer, analogous to PointToPointErrorMinimizer and PointToPlaneErrorMinimizer.
In principle something like this can work for PointToPlane as well, but I don't have time to go into the math. If bkueng could do the math, I could do the integration.

I don't have a current use case for it, but I see many interesting applications in Robotics. I can support the review process, but hardly the development.
A new class named PointToPointSimilarityErrorMinimizer would make sense. You will need a new Transformation class also, which might highlight some missing features in the library as we only tested with the RigidTransformation.
I've the feeling that all the points collapsing into a single location is also a valid solution. Maybe a toy example with points on spheres with two different radius would help and could be added in the unit test data.

I tested the current implementation quite a lot, before trying to integrate it. The code works, and gives correct results. Of course, just as plain point-to-point ICP, it does not converge if the two clouds have very different scales/translations/rotations. In fact, the similarity point-to-point ICP is even somewhat more sensitive than regular point-to-point ICP to how close the clouds are. But this is not news, and more likely people will use this code for clouds which are not far off to start with.
I will of course add a unit test for this and add a new transformation class.

Yes the additional DOF makes it somewhat less stable if point correspondences are mismatched. That's why I added the scaling factor.
Sorry, currently I don't have the time to further work on this.

I integrated the similarity transform. I added a new PointToPointSimilarityErrorMinimizer class, and a new SimilarityTransform class. I added a unit test which checks if the scale is what is expected.

I put in a pull request. All tests pass.

I added a blurb in the tutorial about this new fancy ability as well. This was merged. I think it is safe to close this.

Great, as soon as I've a bit of time, I'll be delighted to play with this.


Feature/race2023 page

Changed the 2023_race page information and changed the banner of the website to make it consistent

Ajust the homepage icon


Fix SRO Db reading from GO db

Describe your changes

Ensure all SRO data is prefixed with sro_ before storing

Issue or discord link

https://discord.com/channels/785153694478893126/1183809278209970217/1194004959759446016

Testing/validation

Checked the console.log output of localstorage and ensured all SRO keys were properly prefixed
Uploaded GO JSON to GO, then opened sr-frontend. Ensured charMeta was not loaded by mistake
Ensure that extra DBs are being stored properly and not overwriting GO extra DBs

Checklist before requesting a review (leave this PR as draft if any part of this list is not done.)

[ ] I have commented my code, in hard-to understand areas.
[ ] I have made corresponding changes to README or wiki.
[ ] For front-end changes, I have updated the corresponding English translations.
[ ] Ran yarn run mini-ci locally to validate format + lint.
[ ] If there were format issues, I ran nx format write to resolve them automatically.
[ ] If I have added a new library or app, I have updated the deployment scripts to ignore changes as needed

need to update unit tests for db


refactor out supervise* methods a bit + add tests

Refactor out supervise and supervise_as methods to external module to avoid duplication + add more tests for testing parameter handling (related to #506, #505, #504, #503).

@e2, thanks for the tests. Not sure if I'll merge this though because it seems to preempt #506. I will come back to this. There will be discussion about supervision I'm sure, I'll keep this branch in mind during that.

@digitalextremist - this is just further refactoring on 0.16-stable + some tests. Ideally the tests should be both for SupervisionHelper and as a shared example group in the other 2 places - but that's not worth the work if you're going to almost reimplement supervision from scratch, etc.

This is defunct as supervision is in its own gem now.


Add Content-Security-Policy in report-only mode

Who is this PR for?
Everyone!
What problem does this PR fix?
Adds Content-Security-Policy directives in report-only mode so we can see what we missed, then turn it on.  This PR updates Rollbar since it was previously served from an ambiguous cloudfront domain and now it's served from a more clearly scoped URL.
Unfortunately CSP won't particularly help in IE11: https://stackoverflow.com/a/42940115 (which manual testing verifies).  But this seems worth adding regardless.
This reports to https://github.com/studentinsights/csp-logger and the new Heroku deployment of it, logging violations in the database.  I verified that this works as expected with the app locally in development mode and submissions look like this running the command here:
 id |   substr   |               documentURI               |                                                              blockedURI                                                              | sourceFile | lineNumber
----+------------+-----------------------------------------+--------------------------------------------------------------------------------------------------------------------------------------+------------+------------
 20 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery_ujs.self-784a997f6726036b1993eb2217c9cb558e1cbb801c6da88105588c56f13b466a.js?body=1              |            |
 19 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/effect-bounce.self-bfcb9288797ecd234ddeedf4fbc5b4c9de94a4e6d8d44e6faa5b5e9836fad9fd.js?body=1 |            |
 18 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/effect-clip.self-7368e4597c32360b162d264ff8bbd0cf9fb1535eb9b4564e6eb7c1db3c774bf3.js?body=1   |            |
 17 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/effect-blind.self-ed102209d7aa0b7374f3db8478cc7dcff5f9980d3146e8bf9fbfab959c6d5d8a.js?body=1  |            |
 16 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/effect.self-3acdb29e7da09c8f0195b994d2f04d73bfef50dc539c84fd2c835d964d33e5c1.js?body=1        |            |
 15 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/dialog.self-5a6c7980dfa3f6b74445b0f4b20eed799ee65363163500ed7f032e603d6b022d.js?body=1        |            |
 14 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/droppable.self-a7a2b4c2cec8fc176f51e4e4291d92bfd3eec40b6a580b47fc54c16334d52882.js?body=1     |            |
 13 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/resizable.self-60b870565f3cd62982c1907828119bb51f42ec8228d5e23868153edc3abc685c.js?body=1     |            |
 12 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/draggable.self-debcca3d0d5d9afe2fe91a6fd1aebdb41ca0a55f2f6017352e6c02b6d8a6e64c.js?body=1     |            |
 11 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/mouse.self-c513294e8da73f31f84ba3ef11e2a1180a47faea0eb2fea4a53fc26153dd21fd.js?body=1         |            |
 10 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/datepicker.self-6267be6d4d0f0e8665be3b4d6ac07f40ba23a32aff8eaa5d1bfc00f76a9eeae7.js?body=1    |            |
  9 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/button.self-4d4f21ba77bbf8cc08ae65e2b6329a7a1b952b32c3ba7bffca8313e7e8c93d0c.js?body=1        |            |
  8 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/autocomplete.self-3befc48aba87cead090cfd014562b2771a662ac6d2c8197b24c08b2d9f3d9f2d.js?body=1  |            |
  7 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/menu.self-7abc1c9401509a7e4fc609b1e461be6fd17e8116e8fda19cc59d05ba79aefb68.js?body=1          |            |
  6 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/position.self-e693ced4ecfa1a276f0b794f32c1d30d792764b08788bf68ecfa3b388c291333.js?body=1      |            |
  5 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/widget.self-c1602241ddc51216b58391768667068867b8e15b9efc722befcd25771eda6819.js?body=1        |            |
  4 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/core.self-93be4d22eda916787802a64d8f88b52db8a9027d4ccbbc0942625324a7d12d44.js?body=1          |            |
  3 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery-ui/accordion.self-c95f320275150a94241b004a91e21eef9c5ae9965a793cb3e2475f6078a1670d.js?body=1     |            |
  2 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/jquery.self-bd7ddd393353a8d2480a622e80342adf488fb6006d667e8b42e4c0073393abee.js?body=1                  |            |
  1 | script-src | http://localhost:3000/educators/sign_in | http://localhost:3000/assets/env.self-4f6714e1ca417f5bf8dd55419d21306ff71a53b5570b9498085969ee5786d6b9.js?body=1                     |            |
(20 rows)

@alexsoble I second mention bot, want to take a look at this when you have a chance?

huh also Rollbar has also since enabled telemetry by default.  We don't want to increase our exposure on Rollbar so this opts out of DOM and log instrumentation, otherwise this would log form inputs to Rollbar.  Seems like a really bad default to opt people into.
I was planning on deploying this to demo first anyway, and then verifying, but this is another thing we'll need to manually verify when deploying.

@alexsoble awesomely noticed that this blocks things in dev when it should be report only.  Need to look at this.

@alexsoble also, can you review the Rollbar changes related to Telemetry here too?

hey @alexsoble, good question! You can see more about this in the earlier
comments about Rollbar. What initially drove it is that the version we were
using was from a generic cloudfront domain so in updating we can more
strictly scope the CSP scripts whitelist.
However, turns out that newer versions of Rollbar switch on a “telemetry”
feature that does some scary things like instruments all user interactions
and DOM events. So this is opting out of all that.
Alternately we could just add the current Rollbar cloudfront URL to the CSP
whitelist (no less secure than it is today) and ship just the CSP change
first.
On Thu, May 17, 2018 at 5:58 PM Alex Soble<EMAIL_ADDRESS>wrote:

@alexsoble commented on this pull request.
In app/views/shared/_rollbar.html.erb
https://github.com/studentinsights/studentinsights/pull/1698#discussion_r189113872
:

@@ -29,19 +29,43 @@
}
}

// Only report if env exists.
function filterTelemetry(e) {

@kevinrobinson https://github.com/kevinrobinson Why are we only
reporting telemetry is env exists?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/studentinsights/studentinsights/pull/1698#pullrequestreview-121231723,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAgvROUf56JeySxS91ELBNvlaH4tXY_ks5tzfKKgaJpZM4UBtZt
.

@alexsoble okay, split this into two PRs.  This is just CSP with current Rollbar and has to allow unsave-eval since the current version of Rollbar uses it.  https://github.com/studentinsights/studentinsights/pull/1702 updates Rollbar and makes that stricter.  That one needs a review since Rollbar's new Telemetry feature is now turned on by default and we want to opt-out.
After that we can see if anything else comes up, and then turn this on from just report-only.

🚢 and moved the Rollbar change out to https://github.com/studentinsights/studentinsights/pull/1702.

I tested this in the demo site, and we'll need to resolve a few issues before turning this on.  They're noted in https://github.com/studentinsights/studentinsights/issues/1704.  I disabled the reporting for now since we got the info we need to do the next step of work.

@kevinrobinson Great, thanks for shipping this incrementally Kevin. I think splitting the PR into two was a good move since it let you 🚢 faster.


How to apply click event on TextInput in react native

Hi I am using TextInput in my react native application. I want to open Date Dialog on clicking on TextInput. I am using TouchableOpacity to do some tricky stuff. I observed that it automatically calls onPress on loading my component every time but it doesn't when I explicitly press on it.
<TouchableOpacity onPress={console.log("Pressed")}>
  <TextInput
    style={{
      height: 40,
      borderColor: "gray",
      borderWidth: 1,
      marginTop: 8
    }}
    underlineColorAndroid="transparent"
    placeholder={strings.schedule_date}
  />
</TouchableOpacity>

Does anyone know any work around for this issue ?

@ajaysaini-sgvu This is a question better suited for stack overflow
However, to answer your question, onPress accepts a function
<TouchableOpacity onPress={() => console.log("Pressed")}>
...

The function is called when the component is pressed.
In your example you are assigning the output of console.log("Pressed") to the onPress handler, so it is evaluated when the components are rendered.

@Amwam Thanks for your reply. I already tried suggested solution.  It didn't work for me. Looks like something is broken with react native itself.
<TouchableOpacity onPress={() => console.log("Pressed")}>
  <TextInput
    style={{
      height: 40,
      borderColor: "gray",
      borderWidth: 1,
      marginTop: 8
    }}
    underlineColorAndroid="transparent"
    placeholder={strings.schedule_date}
  />
</TouchableOpacity>

@ajaysaini-sgvu I've just noticed you are using a TextInput, you should be using just Text for this, as the TextInput will likely have its own handling for press events (i.e. on the native side showing the keyboard).
If your are still having problems, a post on stack overflow will allow for better feedback.

Add pointerEvents="none" on TextInput
Example
<TouchableOpacity onPress={() => console.log("Pressed")}>
  <TextInput
   pointerEvents="none"
  />
</TouchableOpacity>

You can trigger event using onTouchStart. So TouchableOpacity is not needed. Here it is:
<TextInput
onTouchStart={()=>  alert("Hello...")}
editable={false}
placeholder="Please enter your text" />

Thx @azwar ! that really saved me. I somehow oversaw this method. Answering old posts does matter! ;)

You are welcome @Laslo89 . Be careful when you use it on Android. onTouchStart seems not working on Android.

Thank @azwar any work around for android??

@prdpsdfc The best method is using TouchableOpacity

Thanks @azwar i started with TouchableOpacity however the place holder text is showing only few characters

@prdpsdfc you can set editable InputText as false and set the text programatically rather than using placeholder, if it is not a must.

thanx @azwar It's working.!!!

@Mehran-khan-memon 🍺


geopmpolicy-whitelist-root

"geopmpolicy -w" requires root access, but output is only a
function of cpuid.

This was fixed as of 9f7dab1fcffb2eb8521b63e542df8240e575f586


bug: User can not interact with markers to the right and left of Disclosure Footer pop up until disclosure is dismissed

Bug Definition
Red disclosure/disclaimer pop up appears when the user lands on afetharita.com. While this disclaimer is visible:

User can not click and open pins right and left of the banner across the browser (y coordinates within the boundary height of the disclaimer box). You can clearly see the Disclosure box element footer class="FooterBanner_footer__pXKYQ"  extends to the edge of the browser.
User can click and open pins above and below the banner at any x coordinate as long as the y is above or below the footer banner y coordinates.

Once the disclosure is dismissed user can interact with any markers
-- Please add discord contact information
** discord username: @utkubenli#4662**
Severity
Medium
Bug environment
Describe the environment produces the bug. For example:
rc.afetharita.com
afetharita.com
Describe how you are producing the bug step by step

Go to 'afetharita.com'
Wait for the red disclaimer pop up to appear
Hover over around the disclaimer and attempt to click on markers above, at same height and below the disclaimer either to the right or left of the Disclosure box.

Expected Behaviour
User should not be obstructed from clicking around the Disclaimer banner.
Screen shots
https://www.hizliresim.com/byip8kl
https://www.hizliresim.com/802jwfy
Desktop Information

Operating System: Win10
Browser Google Chrome
Version 110.0.5481.78

Mobile Phone Information
Not tested
Additional Context
NA

Fixed in RC and Prod. 👍

Closing as fixed


vscode command to report issue has serious problems

The above is the problem that I am reporting!

Does this issue occur when all extensions are disabled?: No

VS Code Version: 1.65.2
OS Version: Windows 10

Steps to Reproduce:

ctrl-shift-p  report issue
a local window opens
click on some other window
report issue windows is gone

Problem 1
I needed to look at something else in vscode to finish filling out the form. When I clicked outside the form window, the issue window was gone. This happens if I make any other window on the computer active.
I thought that I had lost everything I had typed.
I ran the report issue command again and it opened the form that I was working on so I didn't just give up
Having the window close without the user causing it directly is very bad!
Problem 2
After I finished filling out the form I clicked on the submit to github (or whatever it was called)
I got a popup telling me it was to large to send and that I would have to copy it.
I only typed a couple paragraphs. But I guess checking the boxes to automatically include the extra information at the bottom added too much stuff.
So I copied and pasted, and I was shocked to see a bunch of nasty html show up in the window. My words were far down the page.
Problem 3
Trying to submit an issue on github tells me that I have to use the vscode action. I can't skip all the problems.
Suggested solution

the command just opens a new issue on github or
I can open an issue directly on github and
you can use a github template for the issue to get the information that you need

After thinking about it overnight, I can see two things that would make it useful to report issues from within vscode:

vscode can automatically fill in version information, extensions installed, etc. which is useful for tracking down problems
users without GitHub accounts can submit issues.

The drawback is that they won't have a chance to search and see if the issue has already been reported. However, given that vscode has over 5000 open issues I don't think anyone is going to do a thorough search anyway.
The problem is trying to submit from vscode and from github at the same time.
Submit From VScode
I think the solution would be to fill out the issue in vscode and have it automatically be sent/posted on GitHub using special credentials for the purpose, instead of having users even see that part of the process. There is no reason that it should ever be too large to transfer. Vscode is controlling the collecting of the information.
Submit from GitHub
In addition, for users with GitHub accounts that want to go to GitHub and search issues, etc. and fill submit the issue there you should allow that. Use a template to get the information you need, and add a menu option under help (show system information) to provide the information you need and they can copy and paste it.
(I really do like vscode. I just think it has some warts that need to be removed.)

It's better to report different problems in different issues, to simplify tracking different issues by developers.
Bad window behavior is already tracked in #130497
For problem 2 do yoy have example of that "nasty HTML"?
Usually user content is at the top and below is extensions and system info.
Not sure about problem 3,do you mean suggestion in issue template?

users without GitHub accounts can submit issues

This is tracked here: https://github.com/microsoft/vscode/issues/145249

I got a popup telling me it was to large to send and that I would have to copy it.

we open issues like so: https://github.com/microsoft/vscode/issues/new?body=hello
We add extra details that is super useful for us in diagnosing the problem... unfortunately, browsers have a limit to how long a query string can be... which is why we ask for the paste. It's not ideal, but we are waiting on GitHub to give us another option and nothing yet...
I think all the points outlined in this issue are tracked in other issues... so I'm going to close this. If not, can you please open a new issue per-problem you are seeing? It's way easier for us to track... (we can close issues as we fix specific problems rather than having to track a big issue)


Split file in case of limit of 5000 words

As of now it gives an error
deepl.deepl.DeepLCLIArgCheckingError: limit of script is less than 5000 chars(Now: 19078 chars).

when text with more than 5000 chars is inserted. It would great if the text can be split, processed in chunks and combined later.

To fix error:

split files with split command and run deepl separately
use deeplcom/deepl-python with API token

I live in India, it doesn't support signup from that location. I will try the first option. Thanks.


Update Dependecies ⭐

Updated Dependencies like displayrecorder
Also, is there any way to handle GraphicsCaptureItem.Closed event in Rust? I tried to set up DispatcheQueue, but I Failed

Thanks for the contribution! This looks great, although I have one request. Could you restore the launch.json file? I get that this breaks convention, but I hate having to copy and paste that file between projects every time I want to debug inside VS Code.

Sorry, didn't notice your question. Could you open an issue about the failure when setting up a DispatcherQueue? The minesweeper-rs repo has an example of setting that up, and you also have to be pumping messages on that thread for the event to properly work.

Thanks!

No problem, The Dispatcher issue is fixed now and here is a screen capture library that I wrote with Graphics Capture API for Rust If you are interested windows-capture


AXI4Slave Exception :  Write to object m00_axi_WREADY was scheduled during a read-only sync phase.

I'm writing a Vivado AXI4 master VHDL component. And to test it I'm using cocotb with amba.py driver to test AXI4.
I had this error when I simulated it :
Send raised exception: Write to object m00_axi_WREADY was scheduled during a read-only sync phase.
     File "/opt/cocotb/cocotb/decorators.py", line 115, in send
       return self._coro.send(value)
     File "/opt/cocotb/cocotb/drivers/amba.py", line 248, in _write_data
       self.bus.WREADY <= 1
     File "/opt/cocotb/cocotb/handle.py", line 385, in __le__
       self.value = value
     File "/opt/cocotb/cocotb/handle.py", line 129, in __setattr__
       return object.__setattr__(self, name, value)
     File "/opt/cocotb/cocotb/handle.py", line 401, in <lambda>
       fset=lambda self,v: self._setcachedvalue(v),
     File "/opt/cocotb/cocotb/handle.py", line 585, in _setcachedvalue
       cocotb.scheduler.save_write(self, value)
     File "/opt/cocotb/cocotb/scheduler.py", line 379, in save_write
       raise Exception("Write to object {} was scheduled during a read-only sync phase.".format(handle._name))

I manage to correct it with this commit : https://github.com/Martoni/cocotb/commit/dbcb89cd48031c8d1043a7e6aefaeb5ffd493b7f
But not sure that it's the right way to do it.

@Martoni You got the error as a user is not allowed to schedule a value chage in ReadOnly(), however this is best way to be certain that the values that are being sampled are stable. What you can do instead is do insert a yield NextTimeStep() before doing the assignment but after the samling of the condition.
         while True:
￼              while True:
￼                  self.bus.WREADY <= 0
￼                  yield ReadOnly()
￼                  if self.bus.AWVALID.value:
+                     yield NextTimeStep()
￼                      self.bus.WREADY <= 1
￼                      break
￼                  yield clock_re
￼
￼              yield ReadOnly()

Hi Stuarthodgson,
I'm using GHDL as simulator, and if I change amba.py as you recommend I got this error :
https://github.com/tgingold/ghdl/issues/281


Support decoding header independent of message

The CoAP header can now be decoded independent of the entire encoded message. Useful in situations where the entire message is only needed conditionally based on information available in the header. Usage documentation available under the README's Header section.
Introduces ByteArrayReader that wraps (and reads) from a backing ByteArray (which avoids the extra copy imposed by Okio).
The token was also changed to be non-null, as per RFC 7252 (pg. 33) 5.3.1. Token:

The Token is used to match a response with a request. The token value is a sequence of 0 to 8 bytes. (Note that every message carries a token, even if it is of zero length.) Every request carries a client-generated token that the server MUST echo (without modification) in any resulting response.

Snapshot available as:
repositories {
    maven { url "https://oss.sonatype.org/content/repositories/snapshots/" }
}

dependencies {
    implementation "com.juul.koap:koap:0.3.0-SNAPSHOT"
}

Note that every message carries a token, even if it is of zero length.

Oof. Can you help me understand how a message would carry a token if the token is of zero length? What does this mean for parsing messages? Does that mean the token byte is still populated with a zero value?

Note that every message carries a token, even if it is of zero length.

Oof. Can you help me understand how a message would carry a token if the token is of zero length? What does this mean for parsing messages? Does that mean the token byte is still populated with a zero value?

I took it to mean that a zero-length token should be assumed to be a token value of 0.
Which makes some sense, in that it describes when writing the token you should begin writing at the first non-zero byte. If your token value is 0, then you'll never write any token bytes.
Additionally, you'll never need to have a byte to represent 0, as the absence of that byte (with a value 0x00) means token value of 0.

ByteArrayReader is rad 👍
I propose moving this into tuulbox!

tuulbox is currently internal-only so I'll move it if we find a need to use it in another library; until then it's nice to leave it in this open-source library. 😄

Just a thought after attempting to integrate this into tin-can: It would be nice if the header objects could provide the raw bytes the header was built from.
At least for tin-can, I need the individual header bytes and options/payload bytes for the encryption step (since only the payload is actually encrypted and the header data is used as additional data for the encryption step). As such it would be great if I could do something like:
val message = Message.Udp(...)

val udpHeaderBytes = message.encode().headerBytes()
val payloadBytes = message.encode().payloadBytes()

Or maybe there could be a way to get the message header size and payload size from the Message object?
val message = Message.Udp(...)
val headerSize = message.headerSize // or headerSize?
val payloadSize = message.payloadSize // or payloadSize

val messageBytes = message.encode()
val udpHeaderBytes = messageBytes.copyOf(0, headerSize)
val payloadBytes = ...


Starting the background

I've seen this being discussed before, during January this year, but I'm wondering whether it was ever considered to support starting the app in the background. In my case the app will issue local notifications if it has something to say, so the user will be aware of that anyway, and can act on it.
Is there any other safe way to achieve that?
I'm not sure how the backbutton plugin could help.
Regards,
Anders

Seconded.

Still in te TODO list as mentionend in #10 . And unfortunately the latest Android versions (6.x) seems to stop long running Javascript tasks after couple of hours even you are using the de.appplant.cordova.plugin.background-mode plugin. So I think at the moment only reliable solution is to use the native Android background service coded with Java.

What might save me is that I use background geolocation. At least the app hasn't stopped after 6+ hours. I'll have it running continuously to see if it ever stops. So to me auto-start in the background is still of interest.

Hi Andres, anyway the solution would be quite easy to implemented. Just a new SharedPreference variable to indicate the triggered auto-startup and a new Javascript method to read and clear it. If you need it urgently, feel free to contribute to this plugin.
At the moment I'm quite busy with other task...

Sadly I am too (release on friday), but I will see if I understand the code enough to provide a solution.

Please help and see how you can implement it, I want to use it on dev app which i intend to make production ready (release soon).
@ToniKorin @andersborgabiro

I apologize, but I'm probably not cut out for this. Right now I don't even know how to make a Cordova app start "minimized".

When your app is noticing the auto-startup, just trigger the service of "cordova-plugin-backbutton" to minimize the app.

A fast temporal solution its add this line in the AppStarter class of the plugin:
if( !className.equals("") ){
//Log.d("Cordova AppStarter", className);
Intent serviceIntent = new Intent();
serviceIntent.setClassName(context, packageName + "." + className);
serviceIntent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
serviceIntent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP);
serviceIntent.getExtras().putBoolean("cdvStartInBackground",true);//this line
context.startActivity(serviceIntent);
}
And be sure that the cordova activity has this code:
// enable Cordova apps to be started in the background
Bundle extras = getIntent().getExtras();
if (extras != null && extras.getBoolean("cdvStartInBackground", false)) {
moveTaskToBack(true);
}

@jorantos I am trying to follow your suggestion. Where can I check And be sure that the cordova activity has this code: // enable Cordova apps to be started in the background Bundle extras = getIntent().getExtras(); if (extras != null && extras.getBoolean("cdvStartInBackground", false)) { moveTaskToBack(true); } this part?

@smchae
In the MainActivity.java.
And better than use serviceIntent.getExtras().putBoolean("cdvStartInBackground",true); in AppStarter, use  serviceIntent.putExtra("cdvStartInBackground",true) . Because getExtras can be null.

@jorantos it works

any updates on this if we can use the JS method to start in background/foreground?

@jorantos can you tell me where exactly should i put the following? Because there's no MainActivity.java file in this repo.
And be sure that the cordova activity has this code:
// enable Cordova apps to be started in the background
Bundle extras = getIntent().getExtras();
if (extras != null && extras.getBoolean("cdvStartInBackground", false)) {
moveTaskToBack(true);
}

@paredesivan @smchae were you guys able to fix this?

@kodeine I am using Ionic. MainActivity.java is created when you build. Its under platforms/src/com/whatever your package name is

@smchae oh thanks, im using ionic3 as well.

@jorantos this work fine in first time
but if reboot device again
app open in foreground mode

@jorantos please help me :(

in BootCompletedReceiver.java file
replace
appStarter.run(context, intent, PackageManager.COMPONENT_ENABLED_STATE_ENABLED);
with this
 appStarter.run(context, intent, PackageManager.COMPONENT_ENABLED_STATE_DISABLED);
work fine for me

@ali-mahmoodvand not for me, the app is still opening in foreground.

@ToniKorin
Could you give some hints on how to write a Cordova Plugin that solves this problems, and is compatible with cordova-plugin-autostart?
I have never written a Cordova Plugin

Hello, does anyone has a working solution working in javascript side? I am using Ionic and have not access to native Java

Hello, does anyone has a working solution working in javascript side? I am using Ionic and have not access to native Java

tenes que ir a la ruta src donde esta el codigo de java de android, modificarlo, quitar el plugin y agregarlo nuevamente con el codigo actualizado

Hello, does anyone has a working solution working in javascript side? I am using Ionic and have not access to native Java

tenes que ir a la ruta src donde esta el codigo de java de android, modificarlo, quitar el plugin y agregarlo nuevamente con el codigo actualizado

you mean
"you have to go to the src path where is the android java code, modify it, remove the plugin and add it again with the updated code"


Could you please advise how to add feature vectors for each word?

In my case, each word has a feature vector with length 200. Their weights also need to be learned.

At the moment arbitrary feature vectors are not possible. The model only accepts one-hot encoded sparse input, for example the line in train.dat:
2 1 2 5

represents a training example where the label is 2, and the input feature vector is [1 1 0 0 1]. (The 1st, 2d, and 5th elements of the vector is 1 and the rest zero).
Do you want to use neural-embeddings as input features? Or do you want to train the embeddings from scratch? What sort of application do you have in mind - we might be able to work something out?

Yes. I plan to use Word2Vec to generate a vector for each word in a sequence and then use HCRF to classify each sequence. Most sequences contains about 2-4 words. There are 13 categories.
So far, I just tried use the one-hot encoding as what has been implemented. The learning process is very slow.
CONSOLE OUTPUT:
79832 training examples and 20168 test examples
RUNNING THE L-BFGS-B CODE
           * * *
Machine precision = 2.220D-16
N =      3005743     M =           10
At X0         0 variables are exactly at the bounds

After about one hour, it prints the result of the first iteration:
-203084.209946
At iterate    0    f=  2.03134D+05    |proj g|=  9.99996D-02

Actually, I was just using a small subset of the data I have. Do you have any idea?

The library is currently implemented in pure (and slow) python.
I think that just porting parts of it to cython/numba will increase the speed tremendously, but I don't have time to do it at the moment unfortunately because I'm working on a more general graphical model package. I might attempt a re-write in a week or two if I have time?
If you want to try to do it, however, I'm happy to help out here and there - especially if you have a pull-request or two you want me to look at. Adding support for arbitrary features shouldn't be that difficult either.

I really have interested to improve it, since there are few good HCRF packages out there.
I am doing a project for my internship which will end in next few weeks. So I probably do not have enough time to improve the existing code. I will try it on a small set of the data and see how it works. If the result is promising, I will try numba afterwards (even my internship is over).

If there is interest then I'll put in some effort so we can have a nice HCRF package - didn't realise that there weren't many implementations.
I've started to refactor the code to be closer to the sklearn interface. See this branch. The steps that I think will be necessary from here:

Replace the get_f methods with vector inner products. This will allow dense inputs (like word embeddings), and sparse (like currently). The user should be able to choose which.
Add finite difference tests for the derivative.
Try to get rid of all the state in the object - I only learned that storing variables like T and der on the object is a Bad Idea after writing the current implementation.
Remove load_params and save_params - they're the users responsibility.
Move all the helper functions like load_data to another module.
Update setup.py.
Profile and optimise (still wondering whether numba or cython will be better.)

I'm excited to get this working nicely - hope the above is what you have in mind as well

I re-wrote (See https://github.com/dirko/pyhcrf/pull/2) most of the model to conform to something closer to the sklearn interface. Support for dense and sparse input has also been added. Let me know whether it works for you.
Things I still need to do:

update the readme.
benchmark and cythonize.
add an example notebook (let me know whether you have a dataset that we can use to test things out on)

I suspect that the initialisation of the parameters will have a large effect on the model's performance, so I'll add a way to pass initialisations in.

Could I use this library for image classification？

I‘m a novice for HCRF, I read some paper,they mention the 'undirected graph',but I cannot find  it in your code .That really confuse me.Could you give me some help please?Thanks a lot

Sure it could be used for that - think it might work for sequences of images. Then the input X to the fit method is a list of numpy arrays where each array is of shape (T_i, D), where T_i is how many images is in that example, and D is the dimensionality of each image (the number of pixels or features - so if you're using pixels this must be flattened).
In the original paper they describe two models, the one works with sequences and the other with trees. In this implementation I only implemented sequences (which is a simple chain graph - this graph structure is implicit and you won't find any reference to it in the code). I'm also interested in implementing trees but will try to do it in a more general graphical model package (I'm working on that here if you want to see how that is coming along).
Please let me know which parts I must elaborate more on! When I get time I'll also add an example of how to build an image sequence model.


feat(rds): added rds-apply-immediately field

Signed-off-by: haarchri<EMAIL_ADDRESS>Description of your changes
added new field applyImmediately & added isUptoDate & preUpdate for db-cluster
Fixes #880
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
db-instance:

create
changed instance type &  Apply Immediately true
deleted

db-cluster:

create
changed EnableIAMDatabaseAuthentication to true & Apply Immediately true
changed EnableIAMDatabaseAuthentication to false & Apply Immediately true
deleted

@muvaf did you think it's ready to merge ?

@muvaf any chance to get final review ?

@muvaf  can you check for final review ? i rebased the branch with master
thanks


APEXMALHAR-2331 #resolve #comment StateTracker#bucketAccessed should …

…add bucket to bucketAccessTimes

@chandnisingh @davidyan74 Please review


USB accelerator cannot run edgetpu model

HI guys,
Yesterday I tried to get the USB accelerator to work on my Desktop PC with Ubuntu 18.04. Unfortunately I couldn't get python examples to run with the  'mobilenet_v2_1.0_224_inat_bird_quant_edgetpu.tflite'. The  'mobilenet_v2_1.0_224_inat_bird_quant.tflite'  model is working.
It seems that the Problem is similar to #44 and #214. I think I still miss something.
I get the following error message.

In the next Image are some System Information. Also I'm using the newest TFLITE_runtime version (2.5.0). I installed the current TFLITE_runtime wheel from the Python quickstart. The TFLITE_runtime version is running in an miniconda3 environment.

I tried different Python versions in the environment and the suitable TFLITE_runtime version according to the used Python version.
Maybe someone has an idea what else I can try.
Cheers Guys

I ran into the similar issue and found some information from another thread: https://github.com/google-coral/pycoral/issues/1#issuecomment-724312390

@DemolationPeter
Hey, sorry about basically this is because tflite_runtime 2.5 doesn't match with libedgetpu 14.1. We did have it in our apt repo but needed to temporary rollback to make some changes, it should be re-release soon. For now, you can take a look at this to use the new version:
https://github.com/google-coral/pycoral/issues/1#issuecomment-724312390

@Namburger and @selrahc
Thank you for the quick response. Know everything works like a charm.


header.php and footer.php templates are required in themes

As far as I can't find any explicit declaration of requiring header.php and footer.php in documentation, it looks like that files are required. wp-signup.php and wp-activate.php (which are used for creating and activating accounts in WP Network) use old-style theming and call get_header() and get_footer().
https://github.com/WordPress/WordPress/blob/master/wp-includes/general-template.php#L22
If theme does not have header.php or footer.php, templates from /wp-includes/theme-compat/ are loaded:
https://github.com/WordPress/WordPress/tree/master/wp-includes/theme-compat
Here you can find 4 templates that are used as fallback and all generate notice like:

Notice: Theme without header.php is deprecated since version 3.0 with no alternative available. Please include a header.php template in your theme.

According to this, all templates should contain header.php, footer.php, comments.php and sidebar.php templates in main theme directory - right now none of them are available in Sage.

https://github.com/roots/sage/issues/1052#issuecomment-133672080

right now none of them are available in Sage.

and they won't be — this is a wontfix. add them if you need to for your purposes.

Seriously? What is the point of starter theme for WordPress, if it is broken by default and need special work for make it compatible with build-in WordPress features?

You can feel free to take your complaint upstream if you want, but there are already tickets open that could resolve it, one of which has been open for 3 years and another for 6 years.
Once this issue is resolved in WordPress core, I imagine it wouldn't be too much trouble for us to add partials/head.php (Sage 9) or templates/head.php (Sage 8) to get_header(), same goes for the footer and the rest.

Once this issue is resolved in WordPress core, I imagine it wouldn't be too much trouble for us to add partials/head.php (Sage 9) or templates/head.php (Sage 8) to get_header(), same goes for the footer and the rest.

And what is the trouble with creating header.php with get_template_part('templates/head');?

Well, you tell me. You're the one who opened this issue because doing that is too much "special work" for you. ¯\_(ツ)_/¯

@QWp6t Please... Now I know so header.php is required and I will fix this for my future themes, but so far I created few themes based on Sage and all of them are broken in Network installations, because I don't know that before. There is probably many people like me which don't know that and still creating themes based on Sage, and all of them does not work correct on Network installations. Don't you think so including header.php by default will create more bulletproof starter theme, which will result less broken themes?
When I search starter theme I expect some modern template, which will satisfy all WP requirements, so I can focus on creating themes, not digging WP documentation to find some nuances. I'm disappointed that you know about this issue, and just ignore it because "everyone can fix it by own". Well, every part of theme can be done by theme author, so you can throw out every single Sage feature and say "you can do this by yourself, it is not big problem".

I created few themes based on Sage and all of them are broken in Network installations

Omitting header.php doesn't break themes. It simply triggers a notice when WP_DEBUG is enabled. It's not that big of a deal, and you can suppress the notice using WP hooks.
We're not going to add header.php or footer.php to the root of Sage's theme directory. Use a workaround, add the files yourself, or take the issue upstream.
I don't know what else to tell you. As @retlehs said:

this is a wontfix.


Manufacture power shelf

The shelf that will hold the power components (motor controller, arduinos, encoder board, etc) has a cad in fusion 360. Tasks:

Cut shelf out of appropriate sheet (HDPE, acrylic, other, etc) by laser cutting or other methods
Attach corner brackets to sheet
Once gearboxes are mounted, position shelf above gearboxes, and mark holes for brackets
Drill holes, careful to not get shavings into gearbox
Complete assembly and mount the shelf into chasis

Created the dxf file

Power shelf was cut with laser cutter out of acrylic. Any further machining can be done by hand or with the dude's laser cutter.


WebSSH Widgets

Feature description
Widgets inside WebSSH can be kind of several types, each one related to a specific part of WebSSH.
Webdgets - iOS Widgets
Session Live Bar

https://apexcharts.com/angular-chart-demos/sparklines/basic/

Sparklines :

👀 https://github.com/dagronf/DSFSparkline
https://benpickles.github.io/peity/
https://omnipotent.net/jquery.sparkline/#s-about

#910

Session Stats Board
Host Dashboard

https://apexcharts.com/javascript-chart-demos/radialbar-charts/
#910
https://github.com/jordibruin/Swift-Charts-Examples
Grid layout : https://github.com/gridstack/gridstack.js

SFTP Stats

https://apexcharts.com/javascript-chart-demos/treemap-charts/

Closing in favour of #910


For denoising

Hi, I'm having some difficulties and I was wondering what's wrong with this one

Hi, are you running on a windows system?  You may refer to the closed issue "i cant run the process https://github.com/wyhuai/DDNM/issues/2"


Massive Search reindexing security awareness and count.

Documents (Sulu Document Manager) and Entities (Doctrine ORM) are indexed by the search engine, but, due to security, the number of objects reported as needing to be indexed (when running the reindex command) may differ from the actual number that are indexed.

Secure documents are filtered when they are being hydrated, but the count given by the provider does not deduct "secure" documents. This should be possible as this information IS contained with the PHPCR node.

Doctrine Entites are filtered on the user side by a repository method which has been deprecated. These user side methods for filtering should use the new method, i.e. modifying the given query builder. The count and the number of entities reported to the user should then correspond.

Given that this leaks security information (although only metadata), should we label this as critical?

Well, the only information we are leaking is the descrepency between the number of documents/entities managed by the system and the number that we report as having indexed - in this case that number represents the number of documents which are "secure".
On the other hand this information is only available to somebody who is updating the search index from the command line.

Ok, doesn't sound too critical then. If someone has access to the CLI we have different problems already.


Fix one code format in documentation

Currently, the -- renders as – in the docs, which had me confused until I read the example more carefully.

Thanks !


Add ability to go back to a thread from the plugin RHS

We want to be able to go back to a previously open thread from the plugin RHS. This is currently not possible as the back button system only supports things like search and pinned posts.
RHS code: https://github.com/mattermost/mattermost/blob/86689b7ee69b6f914f7ceacd29b2f356a6b1888c/webapp/channels/src/components/rhs_header_post/rhs_header_post.tsx#L49
Where it's used in figma designs

That works as well. If the implementation is not clean we should probably fix the system. Otherwise if its simple then we can just do that direct plugin implementation.

I think the implementation is clean enough to do it in that way, and it looks like a preliminary step to get the whole RHS working as we want as part of the AI plugin. But I think we are going to face other kind of challenges there first. So I think this should be keep on hold. We can make it work, so no action is needed here for now. Whenever we start figuring out the whole RHS probably we have a better understanding of how deep we need to go with the extensibility in mattermost.

Dropping this for now.


ES2018: tagged template literals should allow invalid escape sequences

https://github.com/tc39/proposal-template-literal-revision is stage 4. tagged template literals should be allowed to have invalid escape sequences like
latex`\unicode{blah}`

see https://github.com/tc39/test262/blob/master/test/language/expressions/tagged-template/invalid-escape-sequences.js for some more examples.

As far as I understand, there are two parts here: (1) parsing and (2) type checking. I’d like to comment on the type checking part.
Consider the latex function in @mroch’s example. Currently, the first parameter of a template-tag-function always has the type Array<string> (as far as I know). I guess that type would need to be changed to something like Array<string | void> & {raw: Array<string>}?


route with forms in ng-view getting error "Cannot set property '$validationSummary' of undefined"

I have a hiccup at work, and have been trying to reproduce it.
Interaction of routes with validation, I think it might be tied up with $destroy of scopes, but not sure.
This example is not the same behaviour I have at work but this is related.
It is similar as it is the same error message 1. below.
There is a chance I have just mucked my code and so its my bug not one in angular-validation, but I think I have squashed most of the dumb out of the example now.
http://plnkr.co/edit/c94r1xA8e0M3iXnWCe6w?p=preview
Load the demo, then click in FIeld1 and click out no error.
Load the demo, then click in Field1 and type a letter no error.
Now click on "First" next to "Choose Route" this activates the route.
Now two errors occur.

Go to field 1 and type a letter get an error in javascript console.
"TypeError: Cannot set property '$validationSummary' of undefined"

The following error appears if you click in Field1 and the click out of it to blur.
"Uncaught TypeError: Cannot read property 'isValidationCancelled' of null"

Sometimes I can get error case 1. without typing anything into field one, but its hit and miss and not frequently.
If line 37 is commented out - "var v = new validationService()", then error 1 does not happen, but 2. still does.
Cheers, Rob.

Hello Rob,
I can't reproduce any of your errors 1 or 2, but in order for me to fix these undefined or null problems, I'll have to know exactly which lines it gives the error (from the unminified versions). What I suggest you is to load the unminified versions and pass me the line number and the variable name giving the error...
Does it give you any other functionality issues, or is it simply the fact that it throws errors in the console? If it's just JS console errors, then it's easy to fix, I would simply check if variable is not null before querying it or whatever else is needed.
Let me know

It is not just console errors.
I may have removed to much from what I had earlier, that showed other stuff was not working.
I hope you can see the console errors at least.
I've put back back in the validation summary display stuff of the outer controller and field.
Have also switched over to non minified validation libraries.
I've added second route, and defaulted to First route, now a console error occurs immediately on page load [not surprising to me]. Note that if you add a char to Field1 error goes away, but the template field value of Field1 jsut above is not displayed between the [[]] markers, removing a character only restores the red box when you leave the field it does not restore the error message below Field1, And the error on clearing FIeld1 only gets applied when leaving Field1 not after timeout. So something is not running any more I think.
The case at work is stopping the route view from changing properly between views, and it is occurring when we switch between routes without any other inputs, so that both the before and after views are visible in the view. I have not been able to get that to happen in this example so far.
This is not the same error, I remember a $destroy handler in one of the stack tracers at work for the $validationSummary null message. I'll be revisisting works error monday morning again.
Rob.

Ok so here is what I found so far, you are using 3 different VMs and you expect them to be completely independent but it doesn't look like they are. I just added a couple of console.log here and there, mostly just before the error and I can see that the first formName of the element it found is test because the first one showing in the console is the index.html and the element is f1 but if we add another console log, the VM that is showing is not the first one, it's instead the one that holds the form firstForm and so they are not in sync. I think that what is happening is that the validationService might not be 3 independent services, I'll have to confirm that but so far that is what I found. Which is also the reason that if you comment out your other 2 VMs it will work, so it looks like they are overwriting each others.
Hmmm ok I think I found the problem, this problem will most probably only show up when using the controllerAs syntax, I have too many global variables in the validation-common.js file, I should use the this to make these variables really owned by each independent object. OOP inside javascript is still risky business. So the fact that they are global makes them exist outside of each VM object, so everytime you touch the Global Options (in your case you declare the controllerAs) it will overwrite previous call and is mixing up the VMs. It looks like I'll have a lot of refactoring to do and lot of potential to break, so it will probably take a few days to fix.

LOL. I just came back to this page to re read what i wrote to see if i had missed something, and saw your reply appear...
Anyway ouch that sounds like it might be a chunk of work to clean up.

Yeah OOP in javascript is still a little bit of a pain and it looks like I didn't fully understand it, I'll have to review all my code. Considering that we'll eventually have to use TypeScript for Angular 2.0, that would have probably helped in my situation :S

Angular 2 is very different beast, i suspect its a re-engineer from ground up no path of migration except for concepts. I hope to use React some time soon, I've been tinkering with it, and so much stuff is just simpler.

I changed a lot of code to make the validationService more independent but now it's seems that it's too much independent and now the Directive does not have access to the same Global Options anymore, because well it's not the same object and it's no more Global anymore... so I'm seriously thinking about rolling back.
Now the only 2 options that I see that would work, but are really not interesting:

pass the ControllerAs into the DOM input element as a new attribute
use only the validationService for validation. To me, none of these 2 choices are really interesting.

Actually thinking about it while writing, there might be another option, I could rollback everything and make it the way it was and possibly add a new Global Options being controller and if pass would then make independent controllerAs which will not collide with each other. That way, everything stays global and these global can be access by both the Directive and Service still.

It's fixed now, I ended up staying with variables that are shared with Directive & Service, going fully independent wasn't going well and was removing too many functionalities.
But you have to know that your code was actually divided in 2 problems which I fixed:
1- Multiple ControllerAs wasn't working correctly because the $validationSummary is saved into each forms inside each ControllerAs, but since you had many it was bring overwritten by the last one.
2- You use Route inside your project and you expected it to not affect the top form (the one outisde of your Route).
For the problem 2), I added a new Global Options that you need to use to fix this issue. The new option is resetGlobalOptionsOnRouteChange (with this name, you should understand the meaning of it) and you need to put it to False (by default it's set to True) on your top Controller.
So in code, you need to have your first Controller, the one outside of the Route like the following:
var vm = this;
var v1 = new validationService({ controllerAs: vm, resetGlobalOptionsOnRouteChange: false });

You can see the Global Options for this new option of resetGlobalOptionsOnRouteChange
Looks like you'll need to pay a beer when I visit again Australia :P

Had a bit of a play, looks decent, I can't use it in our problem area at the moment as we are in count down to a release, and can't afford to reset some of the testing progress.
Cheers Rob.


Build and test instructions

I would love to take a look at the code and try to make a PR.
In order to do that, I need some build instructions of the program so that I can test it on my own vita.
Maybe adding this instructions to the README.md is the best way.
The launcher is awesome, so thank's for your work.

Got any news? Stuck here.


fix: dividers

:tada: This PR is included in version 1.27.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Enhancement: Make spfx doctor compatible with SPFx v1.15.2

Update spfx doctor command so it works for SPFx v1.15.2.

Willing to work on this one.


Enable changing of permissions for log directory

Previously the permissions for node['chef_client']['log_dir'] were
hardcoded to 00750.
Sometimes it can be helpful for non-root users to view Chef logs,
and at othertimes, users may wish to set log_dir to /var/log.
In both cases it can be useful to allow the easy changing of mode
for this directory. This change permits it via an attribute.

:+1:

I'm still :-1: on this idea, as I was here https://github.com/opscode-cookbooks/chef-client/pull/257
Creating attributes for this kind of thing, to make every single bit customizable defeats the purpose of a general cookbook that fits the broadest general use case. It also increases the complexity of maintenance and testing because we have two conditions to test against.

I take your point with regard to complexity and maintenance. However equally burying hardcoded special case defaults cause pain and disruption..
If the goal was to be generic, and simple, why have the special case for the log_dir directory, in setting the mode to 0750 ?
mode 00750 if dir == 'log_dir'

Why not just rely on what has been configured for OS defaults in that case? - as that would truly leave the end result up to the user.
Hardcoding permissions like this can be disruptive, as it's somewhat non obvious to a user of such a cookbook.
We hit this because we wanted to enable log rotation for Chef logs, and run chef from cron.
Previously node['chef_client']['cron']['log_file] was set to /var/log/chef-client.log
The log rotation stuff in the chef client cookbook relies on the values of  node['chef_client']['log_dir'] and `node['chef_client']['log_file'] so we set those appropriately (i.e. log_dir to /var/log, and log_file to chef_client.log) and altered the cron log_file to be a File.join of those attributes.
The net result was /var/log being set to 00750, which was somewhat disruptive.
Of course now we have test cases in place of the permissions of /var/log but the lack of documented clarity regarding the effect of setting node['chef_client']['log_dir'] to an existing directory caused avoidable disruption.

If the goal was to be generic, and simple, why have the special case for the log_dir directory, in setting the mode to 0750 ?

That particular line has a fairly sordid history which took some digging to find. Logic around the log directory mode specifically was added in dde507750523ee86332b45edad23b6461a810d57 which is where it appears the permissions were changed to remove "other" read/execute permission per COOK-2052 as an "insecure default." If anything, I'd argue against the log dir original mode of 00755 being an insecure default, and that it should be the default hard coded value, effectively reverting that change.
My point about being simple is not about what we've done in the past, but how we maintain this in the future. This cookbook is very, clearly, absolutely, not generic and simple. It has cruft and technical debt that makes it, in my opinion, difficult to maintain going forward. It tries to do too much and provide too many knobs for the user to customize because it accommodates so many use cases and bikesheds. Adding more attributes increases complexity. More attributes means more testing to guard against regressions.

The log rotation stuff in the chef client cookbook relies on the values of node['chef_client']['log_dir'] and node['chef_client']['log_file'] so we set those appropriately (i.e. log_dir to '/var/log', and log_file to 'chef_client.log') and altered the cron log_file to be a File.join of those attributes.

IMO we shouldn't have a log_dir attribute. We should have only log_file and have a helper method that can return the log directory using File.dirname. At the very least, the README should be clear that this directory should be specifically for Chef, and not a general purpose directory. That's a common convention with non-system daemons, e.g. /var/log/apache2 or similar.
To summarize the above, rather than adding another attribute, I prefer that we:

Clarify the documentation about the node['chef_client']['log_dir'] attribute that it is for Chef's logs, and should not be set to a shared log location.
Set the mode to 00755 as it was before commit dde507750523ee86332b45edad23b6461a810d57.

To summarize the above, rather than adding another attribute, I prefer that we:

Clarify the documentation about the node['chef_client']['log_dir'] attribute that it is for Chef's logs, and >     should not be set to a shared log location.
Set the mode to 00755 as it was before commit dde5077.

+1 on that. That sounds like a good approach. Would you like me to prepare a pull request for that? - or shall I update this one with that?
Thank you also for explaining a little on the future direction of the cookbook. I've noticed various inconsistencies in there, and will be happy to help tidy and simplify it in future.

Make a new PR.

And thanks :cake:

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

We've settled on just changing the log directory's permissions 0755 in #257 so I'm closing this now.


Website doc - Ansible  Organize docs

Description of the change:

Rename Development Tips to k8s Ansible Modules since it is the subject of the doc
Create a new category Layout to centralize the flow and scaffold layout

HI @asmacdo and @fabianvf,
WDYT ?

/lgtm


Composing accents on mac os don't work

On windows the composition of an accented character doesn't send the keystroke immediately, but on OSX it appears it sends the accent, then the character, then combines them? This makes direct accent entry via composition not work because it advances as soon as the accent is entered.

(I accidentally filed this in the wrong repo.)


Fixes transaction not translating next to income/expend issue #244

Was looking at issue number #244 and noticed that transactions wasn't translating so I fixed that.
Looks like the issue with the word income and expend is that there are no translations in translations.js so those would need to be added?
Wondering where we could verify translations before adding those keys and their translations to each language! Or how to best to proceed to get those translations and then I'll add 'em!

+5


OTA script not working , no response from gateway(URL)

The OTA update script is not working and is giving JSON errors. I suspect that this is due to the fact the URL https://api.52pi.com/update is not working ( 502 bad gateway error).  (https://api.52pi.com is not working either).
error code I am getting
python3 OTA_firmware_upgrade.py
Traceback (most recent call last):
File "OTA_firmware_upgrade.py", line 32, in
r = json.loads(r.text)
File "/usr/lib/python3.7/json/init.py", line 348, in loads
return _default_decoder.decode(s)
File "/usr/lib/python3.7/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib/python3.7/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
The UPS worked fine prior to the update attempt and setting in thru method 2 (i2cset -y 1 0x17 50 127 b) in upgrade mode.
Is there a i2cset command on 0x18 to switch back to normal mode or is there an alternative URL for the upgrade?

Just to confirm that I too am seeing this same error when trying to upgrade my firmware.

There is a slight change this morning. The website api.52pi.com is up again but the  /update still gives an error.
--
Oops! An Error Occurred
The server returned a "405 Method Not Allowed".
Something is broken. Please let us know what you were doing when this error occurred. We will fix it as soon as possible. Sorry for any inconvenience caused.
It would be great to have the OTA script changed to just restore it to normal operations ( i2c  address 17)

IoT reporting in a control script also currently results in an error (copied from HA):
 <title>502 Bad Gateway</title>
502 Bad Gateway

nginx/1.18.0

IoT server appears to be up again. IoT reporting from Home Assistant is working on my UPSplus now.

Thanks ArjenR49 for the heads up, yes it is back up and is now version 10. I just download the scripts from github again and no mention of version 10 in the docs. It worries me a little that there is no offline update option for the failed update with the site being down left my UPSplus in a non functioning state.
We are planning on using these in a number of our products and not being able to reliably installed updates could be a real headache, especially if there is no way of rolling back.

Version 10 blocks i2c bus access from the pi at regular intervals.
Depending on the sampling interval setting.
That doesn't prevent operation of frtz14's control script in my experience.
I have not used Geeekpi's control script for s long time now.
Arjen
(On The Road)
Op zo 3 okt. 2021 14:59 schreef tobylaw @.***>:

Thanks ArjenR49 for the heads up, yes it is back up and is now version 10.
I just download the scripts from github again and no mention of version 10
in the docs. It worries me a little that there is no offline update option
for the failed update with the site being down left my UPSplus in a non
functioning state.
We are planning on using these in a number of our products and not being
able to reliably installed updates could be a real headache, especially if
there is no way of rolling back.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/geeekpi/upsplus/issues/81#issuecomment-932946416, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANCBKNV2WTF4JOAA7AJN4LTUFBOUNANCNFSM5FCNP33Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I can confirm the upgrade worked hence the upgrade servers is back up and running. I still think it would be good to have a bailout option in the software changing it back to operation ( (17) mode. But that would be more of a feature request.


Log proper backoff messages on git clone

Can we drop completely logic about handling wording second/seconds?  I think that everybody can handle to see 1 seconds in a log (or use second(s))

No release notes added. This was a fix for a bug introduced in this release cycle.


Add configuration for the number of transfers a worker can process concurrently

Is your feature request related to a problem? Please describe.
When more than one transfers are started in a short time Enduro will attempt to process all of them (max 1,000) simultaneously, which can cause resource starvation in the worker environment or in the preservation system (a3m or Archivematica).  I would like a way to limit the number of transfers that each preservation worker can process concurrently without changing the Enduro code.
Describe the solution you'd like
Add a preservation worker configuration option (e.g. "capacity") to limit the number of transfers a worker can concurrently work on.
Describe alternatives you've considered
None
Additional context

I believe we can use the MaxConcurrentWorkflowTaskExecutionSize worker option to limit the number of workflow executions a single worker can take on at once

The MaxConcurrentWorkflowTaskExecutionSize limits the number of Workflow tasks that can execute, not the number of Workflow executions, so that's not a viable option.
Because we are using worker sessions for the preservation workers, we can use the MaxConcurrentSessionExecutionSize option to limit the number of concurrent transfers.


add backticks to all fields and types references

See https://cloud-native.slack.com/archives/C06B8KJUU2Y/p1718824809171489

LGTM


Start jupyter with a custom port and token

Rather than letting jupyter start with some random port and random token, we should start jupyter with:

Custom (unused) ports (similar to how we do with the debugger)
Generate a token and use that

Today we rely on scraping output from active servers to get this information.
Startup speed could be improved. (e.g. if a port is in use, then jupyter tries another port, & so on).
Related to microsoft/vscode-python#9089 microsoft/vscode-python#7242

Duplicate of https://github.com/microsoft/vscode-jupyter/issues/10765


OpenAI Rust libraries

https://github.com/dongri/openai-api-rs
https://github.com/rellfy/openai
https://github.com/YanceyOfficial/rs-openai
https://github.com/openai-rs/openai-api

Star History

Added by https://github.com/hongbo-miao/hongbomiao.com/commit/ccc3610e219a1c6d163dbf85903e930193b6cc52


New note

Create a note in folder

Click Add button in drawer/Add floating button to start empty editor
Default note name "Untitled"/未命名
Save the note when editing

Auto save when editor unfocused
Auto save periodically(15s)

Shown in file tree view when created

[ ] Add
[ ] Save


[flutter_svg] [WEB] The SVG image is displayed entirely in black instead of showing only the outlined path

Imported from https://github.com/dnfield/flutter_svg/issues/1018
Original report by @Mage111 on Dec 7, 2023
[WEB] The SVG image is displayed entirely in black instead of showing only the outlined path

Flutter SVG 2.0.9
Dart SDK version: 3.2.0 (stable)
flutter: 3.16.0

Here is the .svg image (in Chrome/Edge):

The image is displayed entirely in black in the flutter_svg package:

The .svg file content:
<?xml version="1.0" encoding="utf-8"?>
<!-- Generator: Adobe Illustrator 23.0.6, SVG Export Plug-In . SVG Version: 6.00 Build 0)  -->
<svg version="1.1" id="Layer_1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" x="0px" y="0px"
	 viewBox="0 0 512 512" style="enable-background:new 0 0 512 512;" xml:space="preserve">
<style type="text/css">
	.st0{fill:none;stroke:#231F20;stroke-width:12.8598;stroke-linecap:round;stroke-linejoin:round;stroke-miterlimit:10;}
</style>
<g>
	<g>
		<g>
			<path class="st0" d="M409.9,249.5V120.9c0-7.1-5.8-12.9-12.9-12.9H114.1c-7.1,0-12.9,5.8-12.9,12.9v128.6H409.9z"/>
			<rect x="69.1" y="365.2" class="st0" width="25.7" height="38.6"/>
			<rect x="416.3" y="365.2" class="st0" width="25.7" height="38.6"/>
		</g>
		<path class="st0" d="M307,236.6c0,7.1-5.8,12.9-12.9,12.9H217c-7.1,0-12.9-5.8-12.9-12.9v-51.4c0-7.1,5.8-12.9,12.9-12.9h77.2
			c7.1,0,12.9,5.8,12.9,12.9V236.6z"/>
		<line class="st0" x1="56.3" y1="339.5" x2="454.9" y2="339.5"/>
	</g>
	<rect x="56.3" y="281.6" class="st0" width="398.7" height="83.6"/>
	<path class="st0" d="M442.1,281.6v-19.3c0-7.1-5.8-12.9-12.9-12.9H82c-7.1,0-12.9,5.8-12.9,12.9v19.3H442.1z"/>
</g>
</svg>

If I remove the fill="none" from element id=".st0", then I get the same thing in Chrome as I see in Flutter.
So, I guess flutter_svg ignoring fill="none" somehow.

Comment by @ivanluizjr on Dec 22, 2023
Estou com mesmo problema

Comment by @spydon on Mar 11, 2024
The style tag isn't supported.
If you used illustrator to export this, when you export to an svg, the dialog box that shows has a dropdown that lets you select "Inline Attributes", if this option is used the resulting svg will work with the flutter_svg package.

Comment by @divyadave1205 on Sep 12, 2024
What if i am exporting it from figma?

As commented on the issue, style tag is not supported.


Java: Move ping() commands out of baseclient

Move the ping() command out of baseclient and into the redisclient.  Javadocs updated appropriately and copied more from Python.

Add UT for ping with route for coverage

ping_with_route and ping_with_message_with_route already exist.

Add UT for ping with route for coverage

ping_with_route and ping_with_message_with_route already exist.

It is IT, not UT


Implement or test download accelerators

Just saw a comment about download accerlators. I think raising this as an enhancement/feature request will help track this performance opportunity. Checking out the commented package pySmartDL, it looks the package is not too active anymore -- maybe there are new alternatives?
The pySmartDL package and the author also refer to a couple of HTTP websites (not HTTPS). Why HTTP is no not secure is written in this article. This can be dangerous and opens the user to a number of attacks:

On-path attacks
DNS hijacking
BGP hijacking
Domain spoofing

Anyways, the concept and application of download accelerators can enhance the package. Would be an awesome feature.

download accelerators can be damaging to server infrastructure, especially free hosting sites like geofabrik. This will not be implemented. There are ideas to host filtered primary features in future versions.


How to improve embulk-input-filesplit performance

Hi,
     We have lots of 100 GB files in a directory. We need to basically read those files line by line and mask couple of columns using hash function and write it to another file. For following configuration it takes 9 hrs 10 mins to process single 100 GB file (1360,000,000 rows). Our hardware configuration 8 core, 65 GB RAM and 1 TB hard disk. Is there is any way to improve the performance?

in:
type: filesplit
path: {{ env.input_fpath }}
header_line: true
tasks: 24
parser:
charset: UTF-8
newline: CRLF
type: csv
delimiter: ','
quote: '"'
escape: '"'
trim_if_not_quoted: false
skip_header_lines: 0
allow_extra_columns: false
allow_optional_columns: false
columns:
- {name: timestamp, type: string}
- {name: src_ip, type: string}
- {name: src_port, type: long}
- {name: dest_ip, type: string}
- {name: dest_port, type: long}
- {name: user_id, type: string}
- {name: in_bytes, type: long}
- {name: out_bytes, type: long}
filters:

type: hash
columns:

{name: src_ip, algorithm: SHA-512}
{name: user_id, algorithm: SHA-512}
out:
type: file
path_prefix: {{ env.output_dpath }}/{{ env.fname }}_
sequence_format: '%03d.'
file_ext: csv
formatter: {type: csv}

Thanks,
Ben

Hi
You can use embulk-executor-mapreduce plugin and run embulk on hadoop.
It may improve performance.

Hi,
     Thanks for your reply. Getting hadoop cluster may not possible in our user case. Just want to under is there any improvements that could made to the configuration to improve the current embulk-input-filesplit performance.

Thanks,
Ben

Hi
It will not take so much time to input or output file.
I suppose that the bottleneck is calculating hash.
(I extracted the method for calculating hash from embulk-filter-hash plugin and executed in my machine. It took about 7 seconds to calculate hash for 100,000 items. Namely, about 53 hours for 2 items of 1,360,000,000 rows.)
How much is CPU usage? If it is almost 100%, and cannot change the hardware, the way to improve performance would be change the hash algorithm.
Thanks

Hi
This is example.
MD5 faster than SHA-512. (1.6x)
SHA-1 too.
It is good idea to change hash algorithm.
this is my test configuration.
in:
  type: file
  path_prefix: /path/to/file
  parser:
    type: none
    message_key: message
filters:
  - type: hash
    columns:
#      - { name: message, algorithm: SHA-512 }
      - { name: message, algorithm: MD5 }
out:
  type: "null"

Thanks.

Thanks for the suggestion. Will try the same.
Regards,
Ben

embulk v0.8.0 will have #355 which improves performance on multicore CPUs.


When I run the example. I have an RuntimeError: mat1 and mat2 shapes cannot be multiplied (18x1 and 18x256)

When I run the command
python examples/train_task.py --algo_name=mopo --exp_name=halfcheetah --task HalfCheetah-v3 --task_data_type low --task_train_num 2
It shows :
File "examples/train_task.py", line 19, in <module>
   fire.Fire(run_algo)
 File "/home/lksgcc/.pyenv/versions/anaconda3-5.0.1/envs/mujoco_py/lib/python3.8/site-packages/fire/core.py", line 141, in Fire
   component_trace = _Fire(component, args, parsed_flag_args, context, name)
 File "/home/lksgcc/.pyenv/versions/anaconda3-5.0.1/envs/mujoco_py/lib/python3.8/site-packages/fire/core.py", line 466, in _Fire
   component, remaining_args = _CallAndUpdateTrace(
 File "/home/lksgcc/.pyenv/versions/anaconda3-5.0.1/envs/mujoco_py/lib/python3.8/site-packages/fire/core.py", line 681, in _CallAndUpdateTrace
   component = fn(*varargs, **kwargs)
 File "examples/train_task.py", line 16, in run_algo
   algo_trainer.train(train_buffer, val_buffer, callback_fn=callback)
 File "/media/lksgcc/new_disk/lk_git/3_Reinforcement_Learning/3_2_Offline_Learning/OfflineRL/offlinerl/algo/modelbase/mopo.py", line 94, in train
   self.train_policy(train_buffer, val_buffer, self.transition, callback_fn)
 File "/media/lksgcc/new_disk/lk_git/3_Reinforcement_Learning/3_2_Offline_Learning/OfflineRL/offlinerl/algo/modelbase/mopo.py", line 206, in train_policy
   res = callback_fn(self.get_policy())
 File "/media/lksgcc/new_disk/lk_git/3_Reinforcement_Learning/3_2_Offline_Learning/OfflineRL/offlinerl/evaluation/__init__.py", line 80, in __call__
   eval_res.update(test_on_real_env(policy, self.env, number_of_runs=self.number_of_runs))
 File "/media/lksgcc/new_disk/lk_git/3_Reinforcement_Learning/3_2_Offline_Learning/OfflineRL/offlinerl/evaluation/neorl.py", line 54, in test_on_real_env
   results = [test_one_trail_sp_local(env, policy) for _ in range(number_of_runs)]
 File "/media/lksgcc/new_disk/lk_git/3_Reinforcement_Learning/3_2_Offline_Learning/OfflineRL/offlinerl/evaluation/neorl.py", line 54, in <listcomp>
   results = [test_one_trail_sp_local(env, policy) for _ in range(number_of_runs)]
 File "/media/lksgcc/new_disk/lk_git/3_Reinforcement_Learning/3_2_Offline_Learning/OfflineRL/offlinerl/evaluation/neorl.py", line 39, in test_one_trail_sp_local
   action = policy.get_action(state).reshape(-1, act_dim)
 File "/media/lksgcc/new_disk/lk_git/3_Reinforcement_Learning/3_2_Offline_Learning/OfflineRL/offlinerl/utils/net/common.py", line 33, in get_action
   act = to_array_as(self.policy_infer(obs_tensor), obs)
 File "/media/lksgcc/new_disk/lk_git/3_Reinforcement_Learning/3_2_Offline_Learning/OfflineRL/offlinerl/utils/net/tanhpolicy.py", line 164, in policy_infer
   return self(obs).mode
 File "/home/lksgcc/.pyenv/versions/anaconda3-5.0.1/envs/mujoco_py/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
   return forward_call(*input, **kwargs)
 File "/media/lksgcc/new_disk/lk_git/3_Reinforcement_Learning/3_2_Offline_Learning/OfflineRL/offlinerl/utils/net/tanhpolicy.py", line 147, in forward
   logits, h = self.preprocess(obs, state)
 File "/home/lksgcc/.pyenv/versions/anaconda3-5.0.1/envs/mujoco_py/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
   return forward_call(*input, **kwargs)
 File "/media/lksgcc/new_disk/lk_git/3_Reinforcement_Learning/3_2_Offline_Learning/OfflineRL/offlinerl/utils/net/common.py", line 113, in forward
   logits = self.model(s)
 File "/home/lksgcc/.pyenv/versions/anaconda3-5.0.1/envs/mujoco_py/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
   return forward_call(*input, **kwargs)
 File "/home/lksgcc/.pyenv/versions/anaconda3-5.0.1/envs/mujoco_py/lib/python3.8/site-packages/torch/nn/modules/container.py", line 141, in forward
   input = module(input)
 File "/home/lksgcc/.pyenv/versions/anaconda3-5.0.1/envs/mujoco_py/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
   return forward_call(*input, **kwargs)
 File "/home/lksgcc/.pyenv/versions/anaconda3-5.0.1/envs/mujoco_py/lib/python3.8/site-packages/torch/nn/modules/linear.py", line 103, in forward
   return F.linear(input, self.weight, self.bias)
RuntimeError: mat1 and mat2 shapes cannot be multiplied (18x1 and 18x256)

Other algos also show the same error.  Thanks for solving this problem!

Hi @lk1983823, I have faced with ur bug and I think what happens in here is that the shape of the state is not in the right way. State must has its shape like [batch_size, num_feats]. So I change a little bit in the file offlinerl/evaluation/neorl.py, from action = policy.get_action(state).reshape(-1, act_dim) to if len(state.shape) == 1:             state = state.reshape(-1, state.shape[0])         action = policy.get_action(state).reshape(-1, act_dim)         if len(action.shape) == 1:             action = action.reshape(-1, action.shape[0])
Hope it can help.


Idea: find unused Rust files?

idea: maybe check cyclic dependencies 😄

Don't have time for this, close as not planned for now 😢


[Navigation Animation] There's no way to provide no transition

Description
I'm trying to build a list/detail screen.

The list screen should just "stay as it is" - no animation.
The detail screen should slide over the list screen.

I have attached two videos.

one depicting my real layout: device-2023-01-05-110645.webm
one depicting the minimal reproducible sample: device-2023-01-05-113156.webm

In both cases you'll notice:

When navigating from list to detail the list fades out. (hard to see)
When navigating from detail to list the list fades in. (easier to see)

I now want to avoid this fade-animation:

I therefore set exitTransition = null & popEnterTransition = null.
This does not skip the animation, but rather seems to play AnimatedNavHosts default animation?
However, i can't provide null to AnimatedNavHosts default animations.

Steps to reproduce
My minimal reproducible sample looks like :
AnimatedNavHost(
    navController = rememberAnimatedNavController(),
    startDestination = infoNavGraph,
    modifier = Modifier.fillMaxSize(),
    builder = {
        infoNavGraph(navController)
    }
)

fun NavGraphBuilder.infoNavGraph(navController: NavController) {
    navigation(
        startDestination = infoMainScreen,
        route = infoNavGraph
    ) {

        composable(infoMainScreen) {
            Box(
                modifier = Modifier
                    .fillMaxSize()
                    .background(Color.Gray)
            ) {
                Text(
                    text = "List Screen. Navigate to Detail via click here ",
                    modifier = Modifier.clickable { navController.navigate(infoDetailScreen) }
                )
            }
        }

        horizontalSlide(infoDetailScreen) {
            Box(
                modifier = Modifier
                    .fillMaxSize()
                    .background(Color.LightGray)
                    .clickable { navController.popBackStack() }
            ) {
                Text(
                    text = "Detail. Navigate back to List via click here",
                    modifier = Modifier.clickable { navController.popBackStack() }
                )
            }
        }
    }
}

fun NavGraphBuilder.horizontalSlide(
    route: String,
    arguments: List<NamedNavArgument> = emptyList(),
    deepLinks: List<NavDeepLink> = emptyList(),
    content: @Composable AnimatedVisibilityScope.(NavBackStackEntry) -> Unit,
) = composable(
    route = route,
    arguments = arguments,
    deepLinks = deepLinks,
    enterTransition = { slideIntoContainer(towards = Left, animationSpec = tween(ANIMATION_SLIDE_DURATION)) },
    exitTransition = null, // <- note the null here
    popExitTransition = { slideOutOfContainer(towards = Right, animationSpec = tween(ANIMATION_SLIDE_DURATION)) },
    popEnterTransition = null, // <- note the null here
    content = content
)

Suggested solution
I don't think "provide a null to a composables animation to show no animations" is a good idea, since "provide null to a composables animation to show default animation" is a nice usecase.
I'd therefore suggest a "noop" animation for this case, but i'm not sure how this would work

Hi, you can solve this by putting { fadeIn(initialAlpha = 1f) } on enterTransition, and { fadeOut(targetAlpha = 1f) } on exitTransition.
Here is an exemple -> composable(         route = Screen.Home.route,         enterTransition = { fadeIn(initialAlpha = 1f) },         exitTransition = { fadeOut(targetAlpha = 1f) }     ) {         Home(paddingValues = paddingValues)     }
Glad to help !

As per #807, use EnterTransition.None and ExitTransition.None.


Helm addes imagePullPolicy to deployment

Helm adds an imagePullPolicy entry to each deployment. The imagePullPolicy entry is set to ifNotPresent when using version tags for the docker images. If the tag is changed back to latest the imagePullPolicy still stays in  ifNotPresent. As a result rollout restart of deployments using latest tags will not update the image.
I think helm should not manipulate/set the imagePullPolicy option and let Kubernetes decide.
Output of helm version:
version.BuildInfo{Version:"v3.6.3", GitCommit:"d506314abfb5d21419df8c7e7e68012379db2354", GitTreeState:"clean", GoVersion:"go1.16.5"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.1+k3s1", GitCommit:"75dba57f9b1de3ec0403b148c52c348e1dee2a5e", GitTreeState:"clean", BuildDate:"2021-05-21T16:12:29Z", GoVersion:"go1.16.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.1+k3s1", GitCommit:"75dba57f9b1de3ec0403b148c52c348e1dee2a5e", GitTreeState:"clean", BuildDate:"2021-05-21T16:12:29Z", GoVersion:"go1.16.4", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
K3OS

This is done automatically by Kubernetes. See https://kubernetes.io/docs/concepts/containers/images/#imagepullpolicy-defaulting.

if you omit the imagePullPolicy field, and you specify the tag for the container image that isn't :latest, the imagePullPolicy is automatically set to IfNotPresent.