Can't update progress or total when the Status is initialized with a job id string

Commit 82c062bb0863e21b788dff895817222dcf87aa38 sets the progress and total on the job itself as well as setting it in the storage object.
The Status object can either be initialized with the job argument as an ActiveJob instance or with the id (string type) of an instance. This method fails if the Status is initialized with the id since strings do not have a progress attribute/setter.
For example:
job = MyJob.perform_later

status1 = ActiveJob::Status.get(job)
status1[:progress] = 1  # this succeeds because the @job instance variable refers to the job itself

status2 = ActiveJob::Status.get(job.job_id)
status2[:progress] = 1 # this fails because the @job instance variable refers to the id of the job (a string):
                       # NoMethodError Exception: undefined method `progress' for "XXXXXXXXXXXXXXXXXXXX":String

The expected outcome is that setting the progress or total values of a Status instance should work the same no matter how the status instance is initialized (ie. the old behaviour from before this commit).

I'll take a deeper look at this issue by the end of the week

I'm curious : in what use case would you update the progress from outside the job?
The status was originally designed to be updated from inside the job and read from the outside.
The issue described above will be solved with the next commit, but there will always be a problem:
updating the progress from the outside won't update the already initialized progress object inside the job and updating progress will overwrite any progress/total set from the outside
class SampleJob < ActiveJob::Base
  include ActiveJob::Status
  self.queue_adapter = :async

  def perform
    progress
    sleep(1)
    puts "wake up, current progress: #{progress.progress} / #{progress.total}"
    progress.finish
  end
end

job = SampleJob.perform_later
ActiveJob::Status.get(job.job_id)[:progress] = 1
ActiveJob::Status.get(job.job_id)[:total] = 5
sleep(1)
puts "status ends with #{ActiveJob::Status.get(job.job_id)}"

# ==> wake up, current progress is 0 / 100
# ==> status ends with {:status=>:completed, :progress=>100, :total=>100}

@inkstak Thanks for the fix!
To answer your question....
We have some old code that determines whether or not a job is successful if the progress and total values match. So in the case where we want that code to think the job was successful even if the job finished early we set the progress to equal the total manually.
I completely understand how you've set it up though and I will refactor our code to not rely on progress == total as a check so that we can avoid this situation going forward.
Thanks again!

Ok so even before 82c062bb0863e21b788dff895817222dcf87aa38, progress.progress was not updated form the outside.
You had to use status[:progress]. It should work again :
class SampleJob < ActiveJob::Base
  include ActiveJob::Status
  self.queue_adapter = :async

  def perform
    progress
    sleep(1)
    puts "wake up, is it completed ? #{progress.progress == progress.total}"
    puts "try again, is it completed ? #{status[:progress] == status[:total]}"
  end
end

job = SampleJob.perform_later
ActiveJob::Status.get(job.job_id)[:progress] = 5
ActiveJob::Status.get(job.job_id)[:total] = 5

# ==> wake up, is it completed ? false
# ==> try again, is it completed ? true

That makes sense and yes we were using status[:progress] not status.progress.
Thanks again for the fix

New version released


refactor(@angular/cli): provide a default extract-i18n target for applications

The extract-i18n command will now use a default project target and builder name if the target entry is not explicitly defined. This allows the removal of additional configuration from an angular.json file. If the target is already present than it will take priority over any default builder behavior. Infrastructure is also now present to allow other architect commands to have similar functionality in the future.

The changes were merged into the following branches: main, 18.1.x


using i18n.js with teaspoon ?

I'm using teaspoon with qunit to test my ember.js application, which uses i18n.js for internationalization.
When I run tests I get :
  Uncaught ReferenceError: require is not defined

It is called in this file, line 6.
When I manually change file and call factory(this.I18n) directly, it works.
Any clue on this? What would be the proper way to make it work ?
Thanks!

The problem seems to be this condition .
It checks the module object from the global namespace. And actually module is used by Qunit, so it breaks the condition that should be false.
What is module ?

That's for Node.js. I think the check must/can be improved. Maybe something like this:
if (typeof module !== 'undefined' && module.exports)

On Jan 26, 2015, at 3:39 PM, Jeremy Venezia<EMAIL_ADDRESS>wrote:
The problem seems to be this condition .
It checks the module object from the global namespace. And actually module is used by Qunit, so it breaks the condition that should be false.
What is module ?
—
Reply to this email directly or view it on GitHub.

That's for Node.js. I think the check must/can be improved. Maybe something like this:
if (typeof module !== 'undefined' && module.exports) {}

Yep, it does this here .

Fixed by 9453118d2914c7cf1272be34919be01b407af595

Thank's !


Relay setup of node id fails if type's property name isn't Id

Describe the bug
When setting up a type as a node, doing descriptor.AsNode().IdField(u => u.UserID) will fail.
SchemaException: Object type User does not implement the field id from interface Node. - Type: User
Fudging on an Id property onto my User class and doing the same but passing u => u.Id now works.
Something is assuming everyone will always have their underlying type with a property name of Id.
To Reproduce
Take an working schema with a relay node type setup, refactor your Id property to MyTypeId and it should fail on startup.
Expected behavior
Should support those of us who have varying id property names.
Desktop (please complete the following information):

OS: Windows
Version: 10

Ha, I had a feeling it was this, but I was looking in one of the other classes of that folder :P. Cheers!


[Improve][SourceConnector] Unified schema parameter, update Iceberg source fields to schema

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
Unified schema parameter, update Iceberg source option fields to schema.
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Hi, @EricJoy2048  I'm new to open source. could you assign me this issue I would like to give it a try on this.


Set focus after removing an enumerator for applicant view

Description
Set focus to add button or remove button for enumerators after an enumerator entity is removed. Setting focus is important for keyboard only / screenreader users.
Release notes
Accessibility - Set focus to add button or remove button for enumerators after an enumerator entity is removed.
Checklist
General

[x] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >
[ ] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

User visible changes

[ ] Wrote browser tests using the validateAccessibility method
[ ] Manually tested at 200% size
[x] Manually evaluated tab order

New Features

[ ] Add new FeatureFlag env vars to server/conf/application.conf
[ ] Conditioned new functionality on a FeatureFlag
[ ] Wrote browser tests with the feature flag off and on, etc.

Issue(s) this completes
Fixes #3566

Codecov Report

Merging #4082 (01fe421) into main (88ec004) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 01fe421 differs from pull request most recent head 6bf6cbf. Consider uploading reports for the commit 6bf6cbf to get more accurate results

@@            Coverage Diff            @@
##               main    #4082   +/-   ##
=========================================
  Coverage     79.47%   79.47%
  Complexity     2505     2505
=========================================
  Files           308      308
  Lines         10115    10115
  Branches        676      676
=========================================
  Hits           8039     8039
  Misses         1813     1813
  Partials        263      263

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


The warning message should be correct. if the user is trying to pay using pay by PayPal Smart Button

Bagisto Version: Master
Steps To reproduce--
1. Visite the shop
2. Create an order.
3. Select the PayPal Smart Button payment method.
Expected result: The warning message should be correct. if the user is trying to pay using pay by PayPal Smart Button
Actual result: The warning message is not correct. if the user is trying to pay using pay by PayPal Smart Button Please find the screenshot Here

I am also getting the translation issue here. Please find the screenshot Here.

@jitendra-webkul
Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot for future reference Here


Update Starlette to 0.28.0

Hi, @everyone
Will this solve the problem of unexpected hangs when using await request.body(), json(), and form() methods in middleware?
I just installed Starlette 0.28.0 for testing, but it was also unexpectedly suspended, I'm just using them for logging purposes
FastAPI version: 0.95.2

Hi.
Will this solve the problem of unexpected hangs when using await request.body(), json(), and form() methods in middleware?
I just installed Starlette 0.28.0 for testing, after obtaining await request.body(), json(), and form().，await self.app(request.scope, request.receive, send) unexpectedly hung up
I'm just using them for logging purposes

Not sure what you are doing or trying but the example in https://github.com/tiangolo/fastapi/issues/5386 works on this branch so yes it does solve the problem.

📝 Docs preview for commit 288dd7ca36b6b073b95f3fb26e0c2132106bc0cb at: https://64853da1aab7f44a4d6614c3--fastapi.netlify.app

Would be great to get this merged in to pick up this change: https://github.com/encode/starlette/pull/2026

Anything in particular blocking this?

would be nice to get this approved.... poetry install of starlette currently fails with the latest fastapi release

Can this PR be merged? or even upgrade to a higher version of Starlette. :)
I am having some dependency conflicts. Currently using --no-deps flag to install requirements.

Any news about this PR ?
Is there an estimated date to merge it?

Is there an estimated date to merge it?

Starlette is now up to 0.31.1 - any reason that FastAPI can't be upgraded? This continues to be a thorn in our side.

@thornewolf merging it could be only one git commit away. It could be your git commit...

@rafalkrupinski  Perhaps I'm mislead but I think the necessary commits are already finished. We just need merge approval?

@rafalkrupinski Perhaps I'm mislead but I think the necessary commits are already finished. We just need merge approval?

From the latest comments I take it isn't finished, and even if it was, the branch is in conflict.

@rafalkrupinski Perhaps I'm mislead but I think the necessary commits are already finished. We just need merge approval?

From the latest comments I take it isn't finished, and even if it was, the branch is in conflict.

To me it seems clear from https://github.com/tiangolo/fastapi/pull/9636#discussion_r1224626560 that we’re waiting for @tiangolo to return to the topic and decide exactly how (and when) to proceed. Nothing gets merged without @tiangolo anyway. I don’t see what a random contributor could offer here.

I'm sure @musicinmybrain is right, the community is always positive, but for @tiangolo personally, he probably doesn't have enough time to deal with the issues since he works for more than just fastapi and, since the project belongs to an individual, not an organization, there's not much any contributor can do about it!

To me it seems clear from #9636 (comment) that we’re waiting for @tiangolo to return to the topic and decide exactly how (and when) to proceed. Nothing gets merged without @tiangolo anyway. I don’t see what a random contributor could offer here.

Admittedly I only skimmed thru the comments. Of course you're right, it's just waiting for @tiangolo

I had a naive approach that simply hooked the exception chain and reraised exceptions in the context of the AsyncExitStack that worked from starlette 0.27 -> 0.31.1. Looks like this may be a more full fledged/robust change.

I needed to finish this dependency refactor first: https://github.com/tiangolo/fastapi/pull/10831, and that had to be done after the Pydantic v2 migration. But now that is done, the way is free for the Starlette upgrades! 🎉

I just merged master into this branch, and because of the recent changes in the dependencies PR, now this one was a clean pure Starlette upgrade. 😎
Thanks for all the discussion and work here @adriangb! 🚀
This will be available in FastAPI 0.107.0 in the next hours. 🎉
I'll try to make several consecutive releases upgrading each recent version of Starlette (if everything works out well). 🤞

Awesome stuff @tiangolo !

Hello team! @tiangolo and @adriangb I'm not sure if anyone noticed, but I started getting this error when using fastapi==0.107.0 and fastapi==0.108.0 and I see this the PR related to the first of the mentioned releases.
I am using as well uvicorn==0.24.0.post1 but also got the same error with uvicorn==0.25.0
I had to rollback to using fastapi==0.106.0 in the mean time and my issue was resolved.
2023-12-26 17:54:01 Traceback (most recent call last):
2023-12-26 17:54:01   File "/usr/local/lib/python3.10/site-packages/starlette/middleware/base.py", line 192, in __call__
2023-12-26 17:54:01     await response(scope, wrapped_receive, send)
2023-12-26 17:54:01   File "/usr/local/lib/python3.10/site-packages/starlette/responses.py", line 259, in __call__
2023-12-26 17:54:01     await wrap(partial(self.listen_for_disconnect, receive))
2023-12-26 17:54:01   File "/usr/local/lib/python3.10/site-packages/starlette/responses.py", line 255, in wrap
2023-12-26 17:54:01     await func()
2023-12-26 17:54:01   File "/usr/local/lib/python3.10/site-packages/starlette/responses.py", line 232, in listen_for_disconnect
2023-12-26 17:54:01     message = await receive()
2023-12-26 17:54:01   File "/usr/local/lib/python3.10/site-packages/starlette/middleware/base.py", line 52, in wrapped_receive
2023-12-26 17:54:01     msg = await self.receive()
2023-12-26 17:54:01   File "/usr/local/lib/python3.10/site-packages/sentry_sdk/integrations/starlette.py", line 136, in _sentry_receive
2023-12-26 17:54:01     return await receive(*args, **kwargs)
2023-12-26 17:54:01   File "/usr/local/lib/python3.10/site-packages/starlette/middleware/base.py", line 128, in receive_or_disconnect
2023-12-26 17:54:01     message = await wrap(wrapped_receive)
2023-12-26 17:54:01   File "/usr/local/lib/python3.10/site-packages/starlette/middleware/base.py", line 123, in wrap
2023-12-26 17:54:01     result = await func()
2023-12-26 17:54:01   File "/usr/local/lib/python3.10/site-packages/starlette/middleware/base.py", line 56, in wrapped_receive
2023-12-26 17:54:01     raise RuntimeError(f"Unexpected message received: {msg['type']}")
2023-12-26 17:54:01 RuntimeError: Unexpected message received: http.request

I don’t recall seeing that before. Can you post a new issue with a minimal reproducible example and link it here? Thanks

I don’t recall seeing that before. Can you post a new issue with a minimal reproducible example and link it here? Thanks

I have solved the issue with the help of another user. This happens when using a middleware and trying to awaiting the request.body from it. There used to be a workaround that we were using from this github discussion. But now, the workaround seems to be much simpler with a higher version of Starlette. Here's my comment with an MRE.

I am still having RuntimeError: Unexpected message received: http.request with latest fastapi 0.108.0 and startlette 0.32.0.post1, is there any open issue for that @tiangolo ?

Read this: https://github.com/tiangolo/fastapi/discussions/8187#discussioncomment-7962881

@Kludex also not clear for me this comment, so this is a backward incompatible change? Why existing middlewares stopped working? I also saw another upgrade PR for Starlette 0.35 (https://github.com/tiangolo/fastapi/pull/10798/files), does it maybe keep current middlewares working?


Possible to use a Scatter Plot's Zoom as a Filter for other charts?

Question
I have several bar and line graphs on a dashboard, and a scatter plot.  The Y axis of the other graphs are plotted as X/Y on the scatterplot but then organized via different bins or groups.  With the filter interaction I can click on a point in the scatter plot and can filter the other graphs on based on that one clicked point, but what I would like to do is have either the Selection or the Zoom for the scatter plot set the filter ranges for the other graphs.  Is that possible?
Code/Examples
No response
Which package?
vizro
Code of Conduct

[X] I agree to follow the Code of Conduct.

Here is what I have as a starting point:
I have a scatter plot defined like this:
score_scatter = px.scatter(score_data,
                           x="Top Cat 2 Score",
                           y="Top Cat 1 Score",
                           color="Top Cat 1 Name",
                           custom_data=["Top Cat 1 Score", "Top Cat 2 Score"])
score_scatter.update_layout(clickmode='event+select')

The update_layout lets me use dash callbacks for when the selection changes.  I have these filters defined:
score_1_filter = vm.Filter(column="Top Cat 1 Score")
score_2_filter = vm.Filter(column="Top Cat 2 Score")

So the idea would be to have the selection from the score_scatter adjust the two filters.  I have the following Dash callback so far:
@callback(
    Output('holder', 'children'),
    Input('score_scatter', 'selectedData'))
def display_selected_data(selectedData):
    selection = "No Selection"
    print(selectedData)
    if selectedData is not None:
        score_1_range = selectedData['range']['y']
        score_2_range = selectedData['range']['x']

        selection = json.dumps(selectedData['range'])

        # score_1_filter.value = score_1_range
        # score_2_filter.value = score_2_range

    return selection

This responds to the selection and gets the range in a usable format.  But I don't know how to change the filters (the commented code doesn't work - no 'value' attribute).


[DNM][RFC] Renode sanity check

This pull requests adds Renode support for sanitycheck.
The tests on Renode are enabled for m2gl025_miv platform.
To run tests on Renode, you need to have it installed from a package.
Please download Renode from the GitHub repository and follow the build instructions.
To start tests, run ./scripts/sanitycheck --platform m2gl025_miv -v
It is still work in progress. Renode hangs on failed tests, we need to solve closing the framework on timeout.
We are very interested in knowing if this approach (via cmake files and console analysis) is right and how should we move forward with enabling more platforms.

The general approach wrt. the build system looks aligned with qemu, and is therefore OK. There are some code-technical improvements needed, but we can discuss those when this is no longer an RFC.

One question we'll need to address is how to enable Renode for platforms that already have an emulator - like hifive1, which can be started on QEMU (it has EMU_PLATFORM variable already set).

Codecov Report

Merging #11503 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #11503   +/-   ##
=======================================
  Coverage   48.37%   48.37%
=======================================
  Files         270      270
  Lines       42080    42080
  Branches    10132    10132
=======================================
  Hits        20358    20358
  Misses      17645    17645
  Partials     4077     4077

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 03c7d9b...ddf408a. Read the comment docs.

One question we'll need to address is how to enable Renode for platforms that already have an emulator - like hifive1, which can be started on QEMU (it has EMU_PLATFORM variable already set).

Hmm, perhaps we could make EMU_PLATFORM into a list. And use some prefixes in the custom target names to allow running either emulator. Or we could have the user choose an emulator through Kconfig/CMake.

any instructions on building renode on fedora 29?

seems to work fine, you need to fix your commit messages for this to go in.


Fixup BUILD.gn file

TBR= @s-perron will commit once the builders pass to fixup the Chromium breakage.


Update use cases for DG

added use cases for

task addition
deadline addition
task deletion
modules left for graduation
view all active tasks and deadlines

Codecov Report

Merging #25 (99b8bda) into master (d001114) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #25   +/-   ##
=========================================
  Coverage     72.10%   72.10%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1233     1233
  Branches        125      125
=========================================
  Hits            889      889
  Misses          312      312
  Partials         32       32

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


can't get my png

Hi,
thanks for sharing your library
I am testing your example on ESP32 to get pgn on my SD card
I found a little mistake in your code at line 85 :         rc = rc = zip.openZIP(name, myOpen, myClose, myRead, mySeek);
I think it should be  rc = zip.openZIP(name, myOpen, myClose, myRead, mySeek);
Right ?
Now I place a chitubox zip at root on the SD card (I check there is PNG there inside)
the serial give me
File: Carriage.ZIP
[869141][E][vfs_api.cpp:29] open(): Carriage.ZIP does not start with /
this error message seem to come from library, can you help to solve the pb ?
Kind regards

hi,
maybe replace this:
    const char *name = entry.name();

by this:
    const char *name = entry.path();


Not possible to use database/table names with dot in it

Laravel Version: relevant to all versions 4.0 and above
PHP Version: not relevant
Database Driver & Version:
mysql version is not relavent.
However i haven't tested all drivers yet.

Description:
While working on a legacy project i came across an issue with how the query grammer formats the tablenames/databasenames and column names for the query.
The issue here is that i have a database with a . in it's name.
the query grammer would format this as `database/table`.`name`.`columnname` instead of the correct value of `tablename/database`.`columnname`
However i'm not sure if this a bug or a limitation to the database component.
Steps To Reproduce:
Create a database with a . in it's name.
Run a query against it using the query builder.

I am not sure this will be fixed. Will cause a major change in query structure

will this ever be solved? I am not a fan of dot in table name. But my client has this database and I need to reproduce a product based on his already made tables. It works while using plain PHP why not in Laravel?

https://github.com/laravel/framework/issues/28307#issuecomment-1622328456 - this solution worked for me


chore(package.json): npm version dependency >= 10.2.4

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


File /lib/libml/dmlc/xgboost4j.so was not found inside JAR.

I'm experimenting with several learning machines, and I encountered XGBoost as one. Trying to run it as a regressor in my own code, it crashes, stating that the native file is not found. Full stack trace:
Aug 16, 2016 9:56:31 AM ml.dmlc.xgboost4j.java.NativeLibLoader smartLoad
SEVERE: failed to load library from both native path and jar
Aug 16, 2016 9:56:31 AM ml.dmlc.xgboost4j.java.DMatrix
SEVERE: load native library failed.
Aug 16, 2016 9:56:31 AM ml.dmlc.xgboost4j.java.DMatrix
SEVERE: java.io.FileNotFoundException: File /lib/libml/dmlc/xgboost4j.so was not found inside JAR.
Exception in thread "main" java.lang.UnsatisfiedLinkError: ml.dmlc.xgboost4j.java.XGBoostJNI.XGDMatrixCreateFromMat([FIIF[J)I
at ml.dmlc.xgboost4j.java.XGBoostJNI.XGDMatrixCreateFromMat(Native Method)
at ml.dmlc.xgboost4j.java.DMatrix.(DMatrix.java:117)
at com.experty.gboost.GradientBoosting.predict(GradientBoosting.java:40)
But in the downloaded directory, I cannot find this file either. Is there somewhere where I can find this file. My Google Fu has resulted in nothing.
Thanks in advance.

I also got this problem. How do you solve it?

I solved it by first downloading the dmlc-core and rabit packages and placing them in the corresponding folders in the xgboost folder, as they do not come with the download of xgboost itself. Then, I used the "make" command in the jvm-package folder. That gives a jar-file, which I added to the project and there we go: done.
Hope it helps


removed db-helper, replaced db-helper in service type, statuses, serv…

…ice entries and recipients and left skeleton functions in respective models.
Description
Summary of your changes
removed db-helper, replaced db-helper in service type, statuses, service entries and recipients and left skeleton functions in respective models. Also commented anywhere I found db-helper out.
What work was done?
removed db-helper, replaced db-helper in service type, statuses, service entries and recipients and left skeleton functions in respective models.
Details of your changes
Why was this work done?
to prepare for database changes.
Why?
Fixing models and routers for database and removing db-helper
Type of change

New feature (non-breaking change which adds functionality)?
Bug Fix?

Change Status

Completed, ready to review and merge?
In progress?
etc.

Checklist

[] Endpoint tested with Postman, or Unit test.
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] There are no merge conflicts

Michael's recent pull requests supersede these updates. No need to push them any longer.


Can no longer install on Julia 1.5: "no known versions"

(@v1.5) pkg> update
   Updating registry at `C:\Users\sternlab\.julia\registries\General`
   Updating git-repo `https://github.com/JuliaRegistries/General.git`
   Updating git-repo `https://github.com/JuliaLang/TOML.jl.git`
ERROR: UUID `fa267f1f-6049-4f14-aa54-33bafae1ed76` given by project file `C:\Users\sternlab\AppData\Local\Temp\jl_pC8oli\Project.toml` does not match given UUID `191fdcea-f9f2-43e0-b922-d33f71e2abc3`

(@v1.5) pkg> status TOML
Status `C:\Users\sternlab\.julia\environments\v1.5\Project.toml`
  [191fdcea] TOML v0.4.0 `https://github.com/JuliaLang/TOML.jl.git#master`

(@v1.5) pkg> remove TOML
Updating `C:\Users\sternlab\.julia\environments\v1.5\Project.toml`
  [191fdcea] - TOML v0.4.0 `https://github.com/JuliaLang/TOML.jl.git#master`
No Changes to `C:\Users\sternlab\.julia\environments\v1.5\Manifest.toml`

(@v1.5) pkg> update
   Updating registry at `C:\Users\sternlab\.julia\registries\General`
   Updating git-repo `https://github.com/JuliaRegistries/General.git`
ERROR: Unsatisfiable requirements detected for package TOML [191fdcea]:
 TOML [191fdcea] log:
 ├─TOML [191fdcea] has no known versions!
 └─restricted to versions * by Elegans [27ab0b70] — no versions left
   └─Elegans [27ab0b70] log:
     ├─possible versions are: 0.2.0 or uninstalled
     └─Elegans [27ab0b70] is fixed to version 0.2.0

(@v1.5) pkg> add https://github.com/JuliaLang/TOML.jl.git
   Updating git-repo `https://github.com/JuliaLang/TOML.jl.git`
  Resolving package versions...
ERROR: Unsatisfiable requirements detected for package TOML [191fdcea]:
 TOML [191fdcea] log:
 ├─TOML [191fdcea] has no known versions!
 └─restricted to versions * by Elegans [27ab0b70] — no versions left
   └─Elegans [27ab0b70] log:
     ├─possible versions are: 0.2.0 or uninstalled
     └─Elegans [27ab0b70] is fixed to version 0.2.0

The old TOML parser was moved to https://github.com/JuliaAttic/TOML.jl. You can use that one or change to this one which supports the 1.0 version of the TOML spec.

I see. Perhaps there should be a notice in this parser's README? The error message from Pkg was quite confusing without the background knowledge.
Anyway, removing and re-adding TOML in the environment that had the dependency fixed the problem.

Yeah, a notice could be added.


Hostedcluster controller: Don't store log on Reconciler

The log is specific to each Reconcile() invocation, so we can not store
it persistently, as multiple concurrent Reconciles will stomp each
others logger.
This happens to not be a problem right now, because we run with only one
worker, but that is something we likely want to change to improve
throughput.
/cc @csrwng

Thank you, this makes sense.
/lgtm


Delivery items animation

Describe the Feature you want to end

i want to add this Delivery items animation
Expected behavior

Screenshots (optional)

https://github.com/pranjay-poddar/Dev-Geeks/assets/120590200/ec14b9ea-7d11-42d3-82b6-0e915b800098
Additional context (optional )

@MohitGupta121 @TusharKesarwani please assign this issue to me

@MohitGupta121 please assign this to me please

@MohitGupta121 please assign this to me


Rankine model performance is insensitive to boiler pressure

Problem:
Results from SAM models using the Rankine cycle are insensitive to the user input of boiler pressure. This behavior is by design because SAM has no internal relationships between cycle boiler pressure, cycle design performance, and cycle cost. SAM relies on users to update these values base on the system they are trying to model.
Solution:
In the current state, this model behavior is not clear to the user. Specifically, the purpose and/or behavior of boiler pressure specification is not well documented. This issue could be handle by either of the following methods:

Update documentation to clearly state the purpose of the boiler pressure input and that the user should update design efficiency and power cycle cost assumptions.
Remove boiler pressure for the SAM GUI, provide models with a nominal boiler pressure, and update documentation.

Both of these solutions would require a characterization of the purpose of boiler pressure within the current SAM Rankine cycle model.
Quick Parametric Results (CSP Tower Molten Salt - Rankine Cycle)
LCOE (real):

Capacity Factor:

Note: The versions of the Rankine cycle model are used across multiple SAM models.

See https://sam.nrel.gov/component/kunena/forum-general/3725.
If we go with Option 1, it would be good to have some guidance on the how to change the design efficiency, perhaps a table of suggested values or a specific example.


Parallelise get_traceback_path

This can be done using 'concurrent.futures'.

The above function can be used as below.
def test(a):
    return(a[0] + a[1])

c = threaded_map(test, [(1, 2), (3, 4)], num_workers=2)

The second argument could take a list of numpy.ndarray, which contain sample paths.

It is actually already pretty fast per call, when precision is not too high (< 26).


Change minimum RU/s from 2,500 to 1,000.

In practice we're able to create an unlimited container from the portal with 1,000 RU/s as the minimum.

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

See set-throughput.md for details on limits.


Chat Service

Will progress #100

Chat Service will first ask User Service to retrieve this User's ChatInstances from Azure AI Search DB.
User Service will return an empty or non-empty list of instances: Chat Service loads this into Chat Instances locally.
Any time Chat Service is asked to make a change to it's local chat instance(s), Chat Service must send the updated local chat instance(s) to User Service so it can be updated on Azure AI Search.
this is the basic flow of maintaining user preferences' chat instances

Chat Service will first ask User Service to retrieve this User's ChatInstances from Azure AI Search DB.
User Service will return an empty or non-empty list of instances: Chat Service loads this into Chat Instances locally.
Any time Chat Service is asked to make a change to it's local chat instance(s), Chat Service must send the updated local chat instance(s) to User Service so it can be updated on Azure AI Search.
this is the basic flow of maintaining user preferences' chat instances

in the future when #106 is completed, we will also allow User Service to be subscribed to alerts from Azure AI Search. IF a change is made on azure ai search, websocket will send out alerts to only specific user(s) that was changed. User Service gets the alert and asks Chat Service to update.
Some notes about this: This will require websocket but also make sure that every query is handled by a minimalapi of ours and that the minimalapi is the one delivering this alert to clientapp(s). Currently, ClientApp apiClient is making these requests on its own. This should be refactored later.


Module 'stream' does not exist in the Haste module map?

This issue appear then I have updated on new react native version 0.59.9. How can I fix this issue? I stuck a day and can not find exact solution.

Infomation when I run react-native info:
Binaries:
Node: 11.4.0 - /usr/local/bin/node
Yarn: 1.12.3 - /usr/local/bin/yarn
npm: 6.4.1 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
Android SDK:
API Levels: 23, 24, 25, 26, 27, 28
Build Tools: 23.0.1, 23.0.2, 23.0.3, 25.0.0, 25.0.1, 25.0.2, 25.0.3, 26.0.1, 26.0.3, 27.0.3, 28.0.1, 28.0.3
System Images: android-23 | Intel x86 Atom_64, android-23 | Google APIs Intel x86 Atom, android-25 | Intel x86 Atom_64, android-25 | Google APIs Intel x86 Atom, android-26 | Intel x86 Atom_64, android-26 | Google APIs Intel x86 Atom, android-27 | Intel x86 Atom_64, android-27 | Google APIs Intel x86 Atom, android-28 | Intel x86 Atom_64, android-28 | Google APIs Intel x86 Atom
IDEs:
Android Studio: 3.2 AI-181.55<IP_ADDRESS>56338
Xcode: 10.1/10B61 - /usr/bin/xcodebuild
npmPackages:
react: 16.8.3 => 16.8.3
react-native: 0.59.9 => 0.59.9
npmGlobalPackages:
react-native-cli: 2.0.1

I fixed this issue by removed lib not use.

I have some kind of the same issue. Can you be more clear how you resolved this @lelup ?
Thanks :-)

@mbrehme  If you have just installed a module and you are getting errors try and manually link this module.


Add method to intercept the request

This change adds a method to intercept the http request before sending
it to the handler under test

Super. Please document in README and add an example


Fix invalid encoding

Pull Request
Description
It seems that some insurances (test case "Techniker Krankenkasse") despite specifying an encoding inside the billing report use invalid characters inside the response xml.
The "Techniker Krankenkasse" was made aware of this problem and they are going to check and fix the issue on their end eventually.
To circumvent the issue we can read the encoding specified inside the xml and use this to re-encode the data.
Closes
#49

released in 1.1.3 @gtuk


stdlib.h compile error

Hi,
I'm on linuxmint 18.
I've cloned the repository, installed golang with linuxbrew, go-sqlite3 with go getand launched make binary-linux-x64
I get the following output
# github.com/mattn/go-sqlite3
../../../.go/src/github.com/mattn/go-sqlite3/backup.go:14:20: fatal error: stdlib.h: No such file or directory
compilation terminated.

Any help would be appreciated

please show me your go env

Hi !
Sorry for the delay:
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/leonard/.go"
GORACE=""
GOROOT="/home/linuxbrew/.linuxbrew/Cellar/go/1.9.2/libexec"
GOTOOLDIR="/home/linuxbrew/.linuxbrew/Cellar/go/1.9.2/libexec/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc-5"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build399343442=/tmp/go-build -gno-record-gcc-switches"
CXX="g++-5"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"

Maybe you don't install C compiler collectory?

Hi ! Thanks for your response !
Actually, I tracked it down to an issue with my LinuxBrew installation and the version of gcc it uses. After disabling it, it did built ok !

I am on Mac and I can't build the code. My output for go env is the following
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/mayukhsarkar/Library/Caches/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/mayukhsarkar/Documents/Golang"
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/h4/5m8_frlx3y568n11bpfx6xz40000gn/T/go-build854985387=/tmp/go-build -gno-record-gcc-switches -fno-common"

and the error messages it throws
WORK=/var/folders/h4/5m8_frlx3y568n11bpfx6xz40000gn/T/go-build042584498
github.com/mattn/go-sqlite3
mkdir -p $WORK/b001/
cd /Users/mayukhsarkar/Documents/Golang/src/github.com/mattn/go-sqlite3
CGO_LDFLAGS='"-g" "-O2"' /usr/local/go/pkg/tool/darwin_amd64/cgo -objdir $WORK/b001/ -importpath github.com/mattn/go-sqlite3 -- -I $WORK/b001/ -g -O2 -std=gnu99 -DSQLITE_ENABLE_RTREE -DSQLITE_THREADSAFE=1 -DHAVE_USLEEP=1 -DSQLITE_ENABLE_FTS3 -DSQLITE_ENABLE_FTS3_PARENTHESIS -DSQLITE_ENABLE_FTS4_UNICODE61 -DSQLITE_TRACE_SIZE_LIMIT=15 -DSQLITE_OMIT_DEPRECATED -DSQLITE_DISABLE_INTRINSIC -DSQLITE_DEFAULT_WAL_SYNCHRONOUS=1 -DSQLITE_ENABLE_UPDATE_DELETE_LIMIT -Wno-deprecated-declarations -I/Users/mayukhsarkar/Documents/Golang/src/github.com/mattn/go-sqlite3 ./backup.go ./callback.go ./error.go ./sqlite3.go ./sqlite3_context.go ./sqlite3_load_extension.go ./sqlite3_opt_userauth_omit.go ./sqlite3_other.go ./sqlite3_type.go
# github.com/mattn/go-sqlite3
../github.com/mattn/go-sqlite3/backup.go:14:10: fatal error: 'stdlib.h' file not found
#include <stdlib.h>
         ^~~~~~~~~~
1 error generated.

I have also tried the following.
CC=clang
go get -v -x github.com/mattn/go-sqlite3

Please help. I am on Mac.

Never mind, I installed it. Turns out that my gcc installation was messed up. Fixed it using the following.
$ xcode-select --install

@MayukhSobo Only gcc should be used, even on Mac. The following works for my mac.
 export CGO_ENABLED=1; export CC=gcc; go get -v -x github.com/mattn/go-sqlite3

Besides this comment, there is another solution for Mac users.

The root reason is due to a missing /usr/include directory.
Installing command-line tools (xcode-select --install), sometimes, will not automatically add it.
After installing command-line tools Install the package at:
# run this in Terminal
open /Library/Developer/CommandLineTools/Packages/macOS_SDK_headers_for_macOS_10.14.pkg

After that,
you should have a directory /usr/include with header files

From StackOverflow Answer.

talha131 solved my problem

Solution from @how2 works good. When i run my go app it shows this warning:
# command-line-arguments
clang: warning: argument unused during compilation: '-nopie' [-Wunused-command-line-argument]

How to hide the warning?

It happened after upgraded from Mojave to OSX Catalina. This does the trick to solve the warning before go get the package:

export SDKROOT=xcrun --show-sdk-path


fix(compiler-core): defineProps infer TSParenthesizedType

fix #4146

Note: this alone does not fix #4146


Auth APIs for users

Create a user signup, login, logout, delete profile, updateProfile  etc controllers.

Done


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1730 ms

Resolved: Bofrost.lu performance has improved in d0d047a after 1 hour, 30 minutes.


[Feature]: Tasks dataview compatibility

Goal or desired outcome of this feature
When using the dataview method for the updated Tasks plugin it won't show any reminders, and when using the default reminder format it won't update when repeating
Describe the feature
When using the Tasks dataview method for making tasks, being able to read/write to their due tag as the reminder area, or a seperate reminder field for notifications
Screenshots, mockups, or videos
No response

Would this feature add the task tag as [remind:: ] or similar?    This would be handy and allow for more consistent syntax with dataview users who need to be reminded.

I was thinking just reading the [due:: ] tag as the reminder, but adding a [remind:: ] tag would be great as well if not better

The ideal behavior would be to read a tag called 'reminder,' as well as to extract dates from all date-related tags that tasks can have, such as 'due,' 'scheduled,' and 'start.' Users would be able to select which tags they want to be read in the settings. For instance, they could opt to receive reminders for 'due' and 'scheduled' but not 'start,' or for 'start' and 'due' but not 'scheduled'... You know.


Failed assertion on expr_tuple

actor Main
  new create(env: Env) =>
    ("", foo([""]))

  fun foo(x: Array[USize]) => None

Version:
0.14.0-f96682fa0 [debug]
compiled with: llvm 3.9.1 -- clang version 3.9.1-5ubuntu1 (tags/RELEASE_391/rc2)

Output:
$ ~/src/ponyc/build/debug/ponyc
Building builtin -> /home/theodus/src/ponyc/packages/builtin
Building . -> /home/theodus/dev/wat
src/libponyc/expr/reference.c:707: expr_tuple: Assertion `c_type != NULL` failed.

Backtrace:
  ../ponyc/build/debug/ponyc(ponyint_assert_fail+0xb1) [0x77ee87]
  ../ponyc/build/debug/ponyc(expr_tuple+0xd0) [0x77885d]
  ../ponyc/build/debug/ponyc(pass_expr+0x331) [0x759846]
  ../ponyc/build/debug/ponyc(ast_visit+0x252) [0x6db768]
  ../ponyc/build/debug/ponyc(ast_visit+0x1a4) [0x6db6ba]
  ../ponyc/build/debug/ponyc(ast_visit+0x1a4) [0x6db6ba]
  ../ponyc/build/debug/ponyc(ast_visit+0x1a4) [0x6db6ba]
  ../ponyc/build/debug/ponyc(ast_visit+0x1a4) [0x6db6ba]
  ../ponyc/build/debug/ponyc(ast_visit+0x1a4) [0x6db6ba]
  ../ponyc/build/debug/ponyc(ast_visit+0x1a4) [0x6db6ba]
  ../ponyc/build/debug/ponyc(ast_visit+0x1a4) [0x6db6ba]
  ../ponyc/build/debug/ponyc() [0x6daecc]
  ../ponyc/build/debug/ponyc() [0x6db21f]
  ../ponyc/build/debug/ponyc(ast_passes_program+0x28) [0x6db308]
  ../ponyc/build/debug/ponyc(program_load+0xbf) [0x6f3edd]
  ../ponyc/build/debug/ponyc() [0x6c0cb3]
  ../ponyc/build/debug/ponyc(main+0x46c) [0x6c119c]
  /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf5) [0x7f7056f91f45]
  ../ponyc/build/debug/ponyc() [0x6c0b5c]
Aborted (core dumped)

This only occurs when the array of the wrong type is passed to foo inside of a tuple.


Improve performance and reduce memory usage of /openapi/tree

Overview
This PR changes the way the /openapi/tree action returns the response payload to minimize memory usage.
Notes
This change takes advantage of the fact that the ASP.NET Response object provides access to a BodyWriter instance that can be passed to the constructor of Urtf8JsonWriter.  This enables the server to construct the JSON response and have it written directly to the network stream without having to have the entire document buffered in memory.
A further enhancement could be made to cache the entire UrlNodeTree rather than reconstructing on every request.
One downside to this approach is that if an exception occurs whilst writing the tree, the HTTP response will already have returned a 200 response.
Testing Instructions
Calling https://localhost:5001/openapi/tree?graphVersions=v1.0 from multiple concurrent threads shows reduced memory usage on the server and shorter time to first byte.

Thank you @darrelmiller for this fix. From my local tests the tree doesn't render on the browser. The expectation is to be able to render the tree on the browser as well, as is currently in prod.

I do get the tree to show up once I fixed the launch configuration and made sure an exception pause wasn't being raised (I don't have the connection strings to the storage account)


*: use histogram coarse timer

Histogram coarse timer is much faster than "normal" timer on linux platform, it improves performance.

please fix the conflict

PTAL @BusyJay

LGTM


Initial support for glTF

glTF is the standard web transmission format for 3d graphics. As discussed in #2696, an API should be provided that accepts a glTF model and produces a RenderGraphic suitable for use in decorations.
@itwin/core-frontend contains a GltfReader class used internally to deserialize tile graphics but provides no public API for it. The reader was implemented specifically for the kinds of glTF we encounter in our tiles (whether iModel, terrain, or reality data) - not as a general-purpose glTF reader.
Initial attempt to import the camera model supplied by Evercam revealed significant deficiencies in our implementation:

Inspection of the code revealed many deviations and omissions from the glTF specs. The code was quite difficult to read because it is entirely untyped.
Ultimately we intend to provide a complete implementation of the glTF 2.0 spec (with the exception of currently unsupported features like skinning and PBR materials).
This PR constrains itself to the first step toward that goal: provide a readGltfGraphics API that correctly loads the Evercam model.
Subsequent PR(s) will flesh out the implementation and test it against the wide variety of sample models supplied by Khronos. Many of those sample models already load and display correctly. Some exhibit missing normals or textures; some fail to load at all.
To make it easier to implement and document the spec (and to understand the implementation), this PR also defines TypeScript interfaces for the glTF data types according to the glTF schema and updates GltfReader to use them.
This required removing the inheritance relationship between ImdlReader and GltfReader, which never made much sense anyway - they shared almost no code.
For testing purposes, display-test-app's dta gltf keyin will allow the user to select a glTF model from the file system to be displayed as a decoration at the center of the project extents.
The Evercam model in display-test-app:

Are you looking for review focused on ImdlReader here, since GltfReader will be ironed out in a subsequent PR?

Are you looking for review focused on ImdlReader here, since GltfReader will be ironed out in a subsequent PR?

Feedback on all of the changes is welcome.

This would be easier to review (and easier to use as documentation) if it was properly documented. I'll add that.

Awaiting ImageTests results to confirm no regressions in reality models due to GltfReader changes.

I successfully built + ran this branch.  I used the keyin to open a sample from Khronos - Box.glb - and it rendered.
If a sample fails to parse (I know this is expected for some), we get "Keyin failed to run", right?


How bind html to Middleabel

Is it possible to bind a react component to middle label?

Duplicated: https://github.com/anseki/leader-line/issues/367


Reproducing RoBERTa STS-B results

Hi, I am trying to use the following code to evaluate the saved model.
I found that some of the value of the output is larger than 5, which is not allowed for STS-B task.
predictions = 5.0 * roberta.model.classification_heads'sentence_classification_head'
Why do we need to multiply 5.0 to the output of the model?
The same issue can also be found here.

from fairseq.models.roberta import RobertaModel
from scipy.stats import pearsonr

roberta = RobertaModel.from_pretrained(
    'checkpoints/',
    checkpoint_file='checkpoint_best.pt',
    data_name_or_path='STS-B-bin'
)

roberta.cuda()
roberta.eval()
gold, pred = [], []
with open('glue_data/STS-B/dev.tsv') as fin:
    fin.readline()
    for index, line in enumerate(fin):
        tokens = line.strip().split('\t')
        sent1, sent2, target = tokens[7], tokens[8], float(tokens[9])
        tokens = roberta.encode(sent1, sent2)
        features = roberta.extract_features(tokens)
        predictions = 5.0 * roberta.model.classification_heads['sentence_classification_head'](features)
        gold.append(target)
        pred.append(predictions.item())

print('| Pearson: ', pearsonr(gold, pred))

Hello!
In the original STS problem, the lables range from 0 to 5.
In the examples/roberta/preprocess_GLUE_tasks.sh(see https://github.com/pytorch/fairseq/blob/main/examples/roberta/preprocess_GLUE_tasks.sh#L182), the label is reranged to 0 to 1.
So in evaluation step, we need to  multiply 5.0 to the output of the model to compute the Pearson with original labels correctly.

Hi, I am trying to use the following code to evaluate the saved model.
I found that some of the value of the output is larger than 5, which is not allowed for STS-B task.
predictions = 5.0 * roberta.model.classification_heads'sentence_classification_head'
Why do we need to multiply 5.0 to the output of the model?
The same issue can also be found here.

from fairseq.models.roberta import RobertaModel
from scipy.stats import pearsonr

roberta = RobertaModel.from_pretrained(
    'checkpoints/',
    checkpoint_file='checkpoint_best.pt',
    data_name_or_path='STS-B-bin'
)

roberta.cuda()
roberta.eval()
gold, pred = [], []
with open('glue_data/STS-B/dev.tsv') as fin:
    fin.readline()
    for index, line in enumerate(fin):
        tokens = line.strip().split('\t')
        sent1, sent2, target = tokens[7], tokens[8], float(tokens[9])
        tokens = roberta.encode(sent1, sent2)
        features = roberta.extract_features(tokens)
        predictions = 5.0 * roberta.model.classification_heads['sentence_classification_head'](features)
        gold.append(target)
        pred.append(predictions.item())

print('| Pearson: ', pearsonr(gold, pred))

Hello!
In the original STS problem, the lables range from 0 to 5.
In the examples/roberta/preprocess_GLUE_tasks.sh(see Line 182 183) the label is reranged to 0 to 1.
So in evaluation step, we need to  multiply 5.0 to the output of the model to compute the Pearson with original labels correctly.


center loss doesn't seem to decay

Hi, I've tried the center loss but it seems it is always equal to 0.001953125. Although, it do has the effect to the loss.
Here's the code:
def center_loss(features, label, alfa, nrof_classes):
    with tf.variable_scope("center_loss"):
        nrof_features = features.get_shape()[1]
        centers = tf.get_variable('centers', [nrof_classes, nrof_features], dtype=tf.float32,
            initializer=tf.constant_initializer(0), trainable=False)
        label = tf.reshape(label, [-1])
        centers_batch = tf.gather(centers, label)
        diff = (1 - alfa) * (centers_batch - features)
        centers = tf.scatter_sub(centers, label, diff)
        loss = tf.reduce_mean(tf.square(features - centers_batch))
        return loss, centers
embeddings = tf.nn.l2_normalize(end_points['feat'], 1, 1e-10, name='embeddings')
center_loss, _ = center_loss(embeddings, labels, 0.95, classes)
loss = tf.add_n([l2loss, center_loss, cross_entropy_mean])

I think it's a bug. There is no "centers"  update source node in the defined graph.
In your example as well, the centers return to "_" (null), so the centers begins from the default value at each batch round.
In this case, the centers doesn't update at all, so you need to add control dependency to update centers.


Socket.framework compilation version issues

Hello,
I'm trying to compile but having issues with the pre-compile module Socket, where can I get the to compile with the newest version of Swift 4.1 please?
Yeelight.swift:10:8: Module compiled with Swift 4.0.3 cannot be imported in Swift 4.1: /Users/USER/YeelightController/Socket.framework/Modules/Socket.swiftmodule/x86_64.swiftmodule
Thanks.

Ok, I have fix it using carthage, I think it could be nice integrate carthage on the repo itself :) thx!


[FA] Add site for the installer lab

What does this PR do?
Add a --site for the installer lab
Which scenarios this will impact?
Installer scenario
Motivation
Additional Notes

What if instead you add a more generic "extraInstallerConfig" passing a yaml string, that you can merge with the existing one?


TypeScript compile error because of incompatible override of Atomic.unpack by its subclasses

I am a TypeScript user with strict mode turned on, and in that configuration this library doesn't compile without manual modifications. I would like to address that issue for fellow strict TypeScript users, to allow them access to this otherwise great library.
❗  Behavior
When trying to install and thus compile the library with TypeScript, the following error occurs:
node_modules/osc-js/lib/osc.d.ts:200:5 - error TS2416: Property 'unpack' in type 'AtomicTimetag' is not assignable to the same property in base type 'Atomic'.
  Type '(dataView: DataView, initialOffset?: number | undefined) => number' is not assignable to type '(dataView: DataView, method: string, byteLength: number, initialOffset?: number | undefined) => number'.
    Types of parameters 'initialOffset' and 'method' are incompatible.
      Type 'string' is not assignable to type 'number'.

❔ Cause
AtomicTimetag is overriding its parent method unpack in an incompatible way. Parent method Atomic.unpack is defined as
unpack(dataView: DataView, method: string, byteLength: number, initialOffset?: number): number;

While AtomicTimetag.unpack is defined as
unpack(dataView: DataView, initialOffset?: number): number;

By omitting the method and byteLength property, it renders itself incompatible with its base class, preventing TypeScript from compiling.
✔ How to fix
By changing the child signature into
unpack(dataView: DataView, initialOffset?: number|string): number;

it overlaps with the parent signature enough to compile. However, this is kinda hacky and the child classes should not override their base methods incompatibly.

Thank you @soimon for bringing this up and sorry for the delayed response (have been busy with a larger long-term open source project I'm contributing to).
I think I have a fix ready at https://github.com/adzialocha/osc-js/pull/78. Maybe you can double-check that this really did the job? Thank you!
I've decided to not change the child signature but rename the base unpack method, as I think TypeScript rightly complains that they're just of different nature and they shouldn't be seen as the same.


Current af_packet code does not have SetBPFFilter similar to pf_ring/…

Current af_packet does not have SetBPFFilter similar to pf_ring/pcap. SetBPF in af_packet needs us to pass compiled BPF program. SetBPFFilter uses pcap's compile functions to do the compilation for us. This code is originally commited to packetbeat as part of this commit. https://github.com/packetb-old/gopacket/commit/44554145590f42204f5f78cb2aeb45ba08cd2c2f

I do not understand what travis build is reporting. Can someone please help?

I looked into this a while back and found this article [0] that talks about the limitations of the pcap-filter language and it mentions pflua [1] as an libpcap-independent way to compile pcap-filters. It would be great to have something like pflua written in Go.
[0] https://www.snellman.net/blog/archive/2015-05-18-whats-wrong-with-pcap-filters/
[1] https://github.com/Igalia/pflua


fix: add width, height and crossOrigin props on Image type declaration file

Summary
According this Issue #36182, the typescript compiler doesn't know the width, height and crossOrigin props for the Image component although they are taken in account in the code, docs and the flow types.
Changelog
[GENERAL] [FIXED] - Fix missing height, width, crossOrigin props on Typescript Image.d.ts file

Image.d.ts is also missing objectFit, referrerPolicy, src, srcSet, tintColor which are found in the docs. I'd say either submit a PR for all of the missing props or just submit a PR targeting height and width only. Otherwise crossOrigin is just random for this PR.

@alvessteve Looks ok to me, but please do take into account the @carlosalmonte04 comment.

TL;DR: It's more organized having PR's be associated to a specific task.
I guess it's not a big deal around here. crossOrigin deviated from the issue and it didn't fix the other problem fully either, which was adding all the props from the docs.

thank you for the comments @carlosalmonte04 and @rshest ! As it was no big deal here, I put them together but next time I will split it.

Thank you. Please do. Otherwise it brings down repo morale and reputation.


Add examples for all devices

A brief usage example for each device should be provided in the example folder.
This, currently, includes

[x] ML600
[ ] Elite11
[x] HuberChiller
[ ] KnauerPump
[ ] KnauerValve
[ ] Spinsolve
[ ] PowerSupply
[ ] FlowIR
[ ] PressureSensor
[ ] R4Heater (even though the module itself is not open sourced for NDA reasons)

Examples of config in docs are more valuable, then openAPI is self-explanatory.


Windows Build or Instructions.

I've been looking for this to help decrypt some Wii DLC I have and would like to know if there could be a Windows build for it or some instructions on how to build it for Windows.
Thank you.

it should build and work just fine under Windows Subsystem for Linux, though I think there are some tools out there that are better designed for Windows. can't remember off the top of my head, though


Revert "Add Fairy Ring AIQ to Mudskipper Point Clue"

This reverts commit 6d060ff68d9c317f139301fadfd9ac7700fe63bd.
It broke the clue

This reverts #5538


It takes too long to create "isolate"

When I execute the following code, I find that waiting for success takes about 1 second or more.
Also, when I need to create "isolation" multiple times in my application, I usually wait a long time.
When waiting for a long time, I use "CALL STACK" in vscode and use pause> Continue to operate a single "isolate",
Then "isolate" will execute correctly and give a return value, otherwise it will wait forever.
int begin = DateTime.now().millisecondsSinceEpoch;
    Isolate isolate = await Isolate.spawn(command.params['method'], argparams,
        debugName: command.params['token'],
        errorsAreFatal: true,
        onError: _onError);
    print(
        "Isolate.spawn ${command.params['token']} complete:cost time:${DateTime.now().millisecondsSinceEpoch - begin}");

I/flutter (29746): Isolate.spawn parsecore complete:cost time:5882
I/flutter (29746): Isolate.spawn hubCore complete:cost time:5911
I/flutter (29746): Isolate.spawn preparse complete:cost time:2071
I/flutter (29746): Isolate.spawn preparse complete:cost time:2549

What can I do to get the return value of "isolate.spwan" quickly?
[✓] Flutter (Channel stable, v1.12.13+hotfix.8, on Mac OS X 10.15.2 19C57, locale zh-Hans-CN)
    • Flutter version 1.12.13+hotfix.8 at /Users/XXXX/development/flutter
    • Framework revision 0b8abb4724 (7 weeks ago), 2020-02-11 11:44:36 -0800
    • Engine revision e1e6ced81d
    • Dart version 2.7.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/XXXX/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.1, Build version 11A1027
    • CocoaPods version 1.9.0

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 43.0.2
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.43.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.8.1

[✓] Connected device (2 available)
    • MI PAD 4                     • 69e3c4be      • android-arm64 • Android 8.1.0 (API 27)
    • Android SDK built for x86 64 • emulator-5554 • android-x64   • Android 10 (API 29) (emulator)

Can you provide a way to reproduce the issue, e.g. the code and step by step guide? It should not take seconds to spawn an isolate.

Do you see this sort of delays when you run your application outside of the IDE (e.g. when you just use flutter run from command line)
Do you see this sort of delays in release builds of your application? (flutter run --release).

/cc @mkustermann @aam

Hi@mraleph :
Thank you for your reply. When I tried to use the two suggestions you gave, currently testing it can give a response within 30 milliseconds quickly, and there is no long-term unresponsive situation under the IDE.
Reproduced in the IDE, as shown in the following code:
Github
This is the result of running in vscode
I/flutter (30706): create isolate finish cost time 7801
I/flutter (30706): create isolate finish cost time 7834
I/flutter (30706): create isolate finish cost time 7754
I/flutter (30706): create isolate finish cost time 9757
I/flutter (30706): create isolate finish cost time 9685
I/flutter (30706): create isolate finish cost time 9907

This is the result of running in Andoid Studio
I/flutter (30843): create isolate finish cost time 3024
I/flutter (30843): create isolate finish cost time 3013
I/flutter (30843): create isolate finish cost time 2962
I/flutter (30843): create isolate finish cost time 3000
I/flutter (30843): create isolate finish cost time 3029
I/flutter (30843): create isolate finish cost time 3020
I/flutter (30843): create isolate finish cost time 3023
I/flutter (30843): create isolate finish cost time 3031

Sounds like this is caused by IDE debugger being attached.
@rmacnak-google @bkonyi do you maybe recognize the issue?


Knockback

Discussed in https://github.com/dmdorman/hero6e-foundryvtt/discussions/802

Originally posted by Kamarathin March  5, 2024
How do I go about working out damage from a knockback in Foundry? Is this automated in some way that I can't see?

I think we can add a button which shows a dialog box.  That box can have fields for distance and hardness of item the actor was knocked backed into.  Once those values are entered we can roll damage, and then follow normal apply damage workflow.

Working on a KB button:


Refine cancel for read thread stream

What problem does this PR solve?
Issue Number: ref #8505
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/cc @gengliqi @windtalker @JinheLin


Cannot read properties of undefined (reading 'inputs')

I am using the latest version of ComfyUI. After installing ComfyUI Custom Scripts, I am unable to select display nodes when using text to input.
ComfyUI: 21404ee9aa

If you click on main.js what file is that

No update


Hermes context specific help option

Crate

Summary
add context specific help
 hermes  start --help

currently it shows
error: unrecognized option `--help`

instead , we can display help for "start" command

Problem Definition

Proposal

Acceptance Criteria

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

The Hermes help menu is currently implemented as a global option. This means that the help argument must come before any specific command. So to obtain the help for command start, we would write hermes help start.
$ hermes help start
hermes 0.3.2
Informal Systems<EMAIL_ADDRESS>Hermes is an IBC Relayer written in Rust.

USAGE:
    hermes start <OPTIONS>

DESCRIPTION:
    Start the relayer in multi-chain mode. Handles packet relaying across all open channels between all chains in the config.

The same goes for any other command, including sub-commands, e.g., hermes help create or hermes help create client.
Would this solve your issue?
Related open: https://github.com/informalsystems/ibc-rs/issues/134

global option works, but when user enters command like this
hermes create client
error: missing required free argument

hermes 0.3.2
Informal Systems<EMAIL_ADDRESS>Hermes is an IBC Relayer written in Rust.

FLAGS:
    -c, --config CONFIG       path to configuration file
    -h, --help                print help message
    -j, --json                enable JSON output

Yes, that's an unfortunate limitation of the gumdrop crate that we depend on via abscissa, the CLI framework  used to build Hermes. When Abscissa eventually moves to Clap, we'll get a much better command-line UX.

yes, that would work

Yes, that's an unfortunate limitation of the gumdrop crate that we depend on via abscissa, the CLI framework used to build Hermes. When Abscissa eventually moves to Clap, we'll get a much better command-line UX.

I did a quick experiment (#1012) with an alternative fix of modifying the help message, so that Hermes explicitly mentions the help command (instead of mentioning the -h flag, which does not work).
@leejw51crypto @romac could you please have a look over #1012 ?


Generating textured meshes.

How difficult would it be to generate a textured mesh with Atlas?
Thanks!

There is a color head implemented which regresses RGB color: https://github.com/magicleap/Atlas/blob/bee3ddbdb0935cf9aa9fc810e829115c6ab74dc5/atlas/heads3d.py#L211
The results are OK, but quite blurry (unfortunately we do not provide a pretrained model for this configuration).
More work on hyperparameter tuning and network architecture could help with this, along with additional losses like a GAN loss. The simple averaging of feature volumes is also a probable culprit for the network having a hard time just copying the color from the input images without blurring.
If you are willing to pass through the image sequence twice, you can first reconstruct the mesh with Atlas and then in a second pass use existing mesh texturing algorithms to backproject the images onto the mesh.

Sorry, can you help. I got this :
Traceback (most recent call last):
File "inference.py", line 149, in
main()
File "inference.py", line 146, in main
process(info_file, model, args.num_frames, save_path, i, len(info_files))
File "inference.py", line 91, in process
tsdf_pred = model.postprocess(outputs)[0]
File "/root/Atlas/atlas/model.py", line 310, in postprocess
color = batch[key+'_color'][i]
KeyError: 'vol_04_color'
all dict keys in batch is dict_keys(['vol_16_tsdf', 'vol_08_tsdf', 'vol_04_tsdf', 'vol_16_semseg', 'vol_08_semseg', 'vol_04_semseg'])
I did something wrong?


docker & docker-compose are not in caskroom

Minor, but caused an issue on setup. We installed docker & docker-compose instead using plain brew.

I think the danger of this is that those tools no longer get updated in tandem with the docker daemon, no?
I think I installed docker from a dmg, which presumably is what caskroom taps from, and when it prompts me for an update my CLI tools auto-update along with it.
Having said that, I still know very little of docker, maybe these CLI tools are very thin wrappers that call into the docker daemon and thus don’t really need to be updated in tandem?

I'm not sure @alloy - in our case we just got a message that docker wasn't in the caskroom. Is it possible we should just install docker for mac?

I use Docker for Mac, personally

I see. Yeah I use Docker for Mac as well, I thought that was what cask would install too 🤷♂

I see. Yeah I use Docker for Mac as well, I thought that was what cask would install too

Ditto.
I just successfully reinstalled docker and docker-compose using brew cask install.  One note is that we don't need to separately install docker-compose as it comes bundled with Docker for Mac.
isacpetruzzi@Artsy-Isac gravity (master) $ brew cask install docker
==> Satisfying dependencies
==> Downloading https://download.docker.com/mac/stable/26764/Docker.dmg
######################################################################## 100.0%
==> Verifying checksum for Cask docker
==> Installing Cask docker
==> Moving App 'Docker.app' to '/Applications/Docker.app'.
🍺  docker was successfully installed!
isacpetruzzi@Artsy-Isac gravity (master) $ which docker
/usr/local/bin/docker
isacpetruzzi@Artsy-Isac gravity (master) $ docker --version
Docker version 18.06.1-ce, build e68fc7a
isacpetruzzi@Artsy-Isac gravity (master) $ which docker-compose
/usr/local/bin/docker-compose
isacpetruzzi@Artsy-Isac gravity (master) $ docker-compose --version
docker-compose version 1.22.0, build f46880f

@erikdstock did you get Docker for Mac using plain brew install?  What versions do you have?  I am on:

MacOS 10.12.6
Homebrew 1.7.3
Homebrew-Cask 1.7.3

I feel like there is a miscommunication happening here- I'm referring specifically to the setup instructions in this PR, which I was following with Ashley:
brew install git && brew tap caskroom/cask && brew cask install docker docker-compose
=> Error: Cask 'docker-compose' is unavailable: No Cask with this name exists.

I believe brew cask is about to be deprecated anyways (the ~cask~ part)... but docker-compose is technically in reguarly brew. I don't know if they would work without docker for mac or docker machine though.

@erikdstock i see - the problem is that the brew install docker command gives you the Docker binary but you need to run the Docker daemon yourself, with docker-machine and VirtualBox - @bhoggard and I were using this method in the past and it was painful.
brew cask install docker gives you the "Docker for Mac" app, integrates with MacOS quite painlessly (has a UI and a taskbar icon for the daemon) and is what we are generally recommending.
I'd like to keep these quickstart instructions to use brew cask for this reason, and otherwise instruct folks to download the .dmg and install Docker for Mac themselves.
Will open another PR.

Ach accidentally just committed via the UI editing window rather than opening a PR - but here it is https://github.com/artsy/README/commit/633a198af192fbc7c508121500325c096a2145d8

cc @ashleyjelks - would recommend to uninstall the brew installed docker / docker-compose and use brew cask install docker or download the Docker for Mac .dmg, which contains both, bundled in an app.  Sorry for the confusion!


[Feature Request] Support for loading/saving from file for CVS sync

Other tools like Paw for Mac allow the user to save the workspace to a single file, which can be then added to a CVS system like git. A feature like this allows developers to rely more confidently on this tool because it's not bound to their system or a cloud service which may disappear at any time, but the data is saved in an easy to access shared space.
This way it becomes easier to work with the requests, and the app can become a long term investment for developer.
Also when working with several computers and different projects it's very convenient to just open a file instead of having to login to a specific account or anything like that. Using online accounts or cloud services is just clumsy and not useful. Also working with APIs under NDA requires having local storage, despite all the end-to-end encryption that the app can have, local is just better.
Just dumping and importing a database or a marshalled object could be enough.

Ah yes, this is quite a common ask, however, it's difficult to replace the current local database system with something file-based.
You can, however, already dump/import a workspace from the preferences menu.

I tried that, it's clumsy. As it's so quick to export data, why not making it a menu item, and making the app able to load its files? While you rightfully say it's hard to replace the database with a file-based solution, effectively it's already there, it just needs a few UX changes.
What else do you propose to store the data automatically in the CVS without having to do ten clicks every time the user writes a change in the UI to save the file?

Perfect way of doing this would be with a plugin which exports on save and imports the file it is newer than the current workspace. I think I have worked on something like that a while ago but the Insomnia API stopped me at some point, maybe you @j0hnwhyte have some time to look deeper into that and/or create a PR to expand the Insomnia API at some point?

In the meantime you can use the solution mentioned here, also related to this issue:
https://github.com/Kong/insomnia/issues/1638


Can't install in remote ssh mode

I usually use the remote ssh mode, but this plug-in cannot be installed in this mode. I hope it can be supported

download .vsix file from vscode marketplace and local install to solved!
https://marketplace.visualstudio.com/items?itemName=aminer.codegeex&ssr=false#version-history


Explorer Find: inflated match count on collapsed folder

Type: Bug

Watch how the promised counts of folder matches in this repo's .vscode filder and then its .vscode/extensions repo turn out to be inflated (maybe doubled?)
VS Code version: Code - Insiders 1.96.0-insider (0c9dd26823dc38dd59d6240e1b6698add43d942c, 2024-11-18T05:05:45.838Z)
OS version: Windows_NT x64 10.0.22631
Modes:

/assign @benibenj

I'm actually seeing the 8 entries in the .vscode folder. I think I might not be taking the files.exclude setting into account. Have you by chance modified the setting to exclude the out folder?

I have this as a user-level setting that isn't overridden at workspace or folder level for this repo:
"explorer.excludeGitIgnore": true,

It looks like your counting isn't respecting this.

I'm still seeing the issue on the latest Insiders

Multiple problems here:

SearchService.fileSearch returns paths with lower-case drives on windows (typically should be upper case, SearchService.textSearch returns paths with upper case drives) and the IgnoreFile utility uses case-sensitive string comparison on all platforms. #235419 #235478
FilesFilter.isIgnored was using IgnoreFile.isPathIncludedInTraversal which made sense for the explorer to use, but not to expose outwards.

Explanation:
.gitIgnore:

**/out/

Using isPathIncludedInTraversal

isIgnored('src/out') => true
isIgnored('vscode/dist/out/') => true
isIgnored('src/out/file.txt') => false
isIgnored('vscode/dist/out/file.txt') => false

Using isArbitraryPathIgnored

isIgnored('src/out') => true
isIgnored('vscode/dist/out/') => true
isIgnored('src/out/file.txt') => true
isIgnored('vscode/dist/out/file.txt') => true

I've got a very easy fix which would solve this as a candidate: https://github.com/microsoft/vscode/pull/235502
I will also push it to the main branch, but there will be a bit more required for a proper proper fix.


dbeaver/dbeaver#22248 General -> Network Connections settings is gone…

Network connection preferences page returned back

verified


typo: Double word "to"

Summary
Describe your changes here.
Fixes #Issue_Number (if available)

Seems like this stalled out. Should I break this PR up?

@nschonni It would be nice to break this up. It would make it easier to merge the obvious changes while still facilitating discussions.

I think we can get this one finished. Let's just do the blockquote on that error message and file an issue to address inconsistencies on the other article and let's merge it!


Measured characteristic change

Change in measured characteristic: alignment with Figma

@ZarkoPernar As liberty is out can you add the finishing touches here please


feat(mode): support overwriteMode

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

14 of 14 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
93

Relevant Lines:
93

💛  - Coveralls


Univariate Time Series using Facebook Prophet

Article checklist
Attention
Our Peer Reviewer and Content Moderator teams do NOT provide any revisions services. All revisions and edits should be completed on your own forked repo (as to not take up room in the queue) in order for our team to review them all in a timely manner.
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.

As a rule of thumb - please be sure to only submit articles (pull requests) that are fully polished and ready to be published. Be sure to go through our resources documents for extra and 3rd party (vetted) resources to help improve overall technical writing.

Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #6552

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

Plagiarism check done ✅

@francis966 Please perform a Grammarly check on your PR

Hello, @wilkistermumbi, I have used Used Grammarly. I have also used Hemingway Editor for readability.
Thanks.

@francis966 Hi. This is a nice read. Please work on the minor corrections before we can move on to final review.

Hello @wilkistermumbi, I have worked on the minor corrections.
Thanks

@francis966 Please check my final commits. Do not confuse df.tail() with df.head().
Otherwise, this PR will proceed for final review.

jumping on this one


Portainer with reverse proxy (caddy) can't connect to edge agent but gets heartbeats

Since a while I try now to connect my portainer instance at home to the home server at my mothers house. For that I red, the safest way to use is edge agent.
At home I run portainer behind a caddy reverse proxy on a raspberry pi 4.
Caddy is configured to route https://portainer.mydomain.de:443 to <IP_ADDRESS>:9233 and tcp://portainer.mydomain.de:8000 to port <IP_ADDRESS>:8111. In Docker I configured, 8111:8000 and 9233:9000 in the portainer compose file on my server.
I opened up port 8000 and 443 in my router (tcp and udp). Also in ufw I allowed port 443 and 8000.
Portainer is working well so far.
On my mothers server I opened up port 9001 in the router. Ufw is also configured so far.
Now I tried to set up edge agent. I used:
sudo docker run -d \
  -v /var/run/docker.sock:/var/run/docker.sock \
  -v /var/lib/docker/volumes:/var/lib/docker/volumes \
  -v /:/host \
  -v portainer_agent_data:/data \
  --restart always \
  -e EDGE=1 \
  -e EDGE_ID=----------------------------------- \
  -e EDGE_KEY=-------------------------------------------------------------- \
  -e EDGE_INSECURE_POLL=1 \
  --name portainer_edge_agent \
  portainer/agent:2.15.0

I now can see a heartbeat under Environments but if I try to connect it says Failed loading environment Environment is unreachable.
The portainer logs say:
time="2022-09-15T22:18:25+02:00" level=info msg="2022/09/15 22:18:25 [DEBUG] [chisel,monitoring] [endpoint_id: 8] [status: ACTIVE] [status_time_seconds: 7.157875] [message: environment tunnel monitoring]"
time="2022-09-15T22:18:28+02:00" level=info msg="2022/09/15 22:18:28 http: proxy error: dial tcp <IP_ADDRESS>:51018: connect: connection refused"
time="2022-09-15T22:23:30+02:00" level=info msg="2022/09/15 22:23:30 http error: Unable to find the container (err=Error: No such container: 3bfdd889277c8539ed7f13f4df61339c6821c53ad3a5a404730793545eab88c6) (code=404)"
time="2022-09-15T22:23:30+02:00" level=info msg="2022/09/15 22:23:30 http error: Unable to find the container (err=Error: No such container: dae984b1b0af5e2ab7d8a7d4a8f4d04f8d278091412641c87250d3700a5d10dd) (code=404)"
time="2022-09-15T22:34:45+02:00" level=info msg="2022/09/15 22:34:45 [DEBUG] [chisel,monitoring] [endpoint_id: 8] [status: ACTIVE] [status_time_seconds: 3.817940] [message: environment tunnel monitoring]"
time="2022-09-15T22:34:50+02:00" level=info msg="2022/09/15 22:34:50 http: proxy error: dial tcp <IP_ADDRESS>:64692: connect: connection refused"
time="2022-09-15T22:37:04+02:00" level=info msg="2022/09/15 22:37:04 [DEBUG] [chisel,monitoring] [endpoint_id: 8] [status: ACTIVE] [status_time_seconds: 8.822090] [message: environment tunnel monitoring]"
time="2022-09-15T22:37:05+02:00" level=info msg="2022/09/15 22:37:05 http: proxy error: dial tcp <IP_ADDRESS>:55147: connect: connection refused"
time="2022-09-15T22:41:24+02:00" level=info msg="2022/09/15 22:41:24 [DEBUG] [chisel,monitoring] [endpoint_id: 8] [status: REQUIRED] [status_time_seconds: 0.182232] [message: environment tunnel monitoring]"
time="2022-09-15T22:41:34+02:00" level=info msg="2022/09/15 22:41:34 http: proxy error: dial tcp <IP_ADDRESS>:65013: connect: connection refused"
time="2022-09-15T23:12:44+02:00" level=info msg="2022/09/15 23:12:44 [DEBUG] [chisel,monitoring] [endpoint_id: 8] [status: REQUIRED] [status_time_seconds: 1.361693] [message: environment tunnel monitoring]"
time="2022-09-15T23:12:53+02:00" level=info msg="2022/09/15 23:12:53 http: proxy error: dial tcp <IP_ADDRESS>:60140: connect: connection refused"
time="2022-09-15T23:13:34+02:00" level=info msg="2022/09/15 23:13:34 [DEBUG] [chisel,monitoring] [endpoint_id: 8] [status: ACTIVE] [status_time_seconds: 2.269864] [message: environment tunnel monitoring]"
time="2022-09-15T23:13:41+02:00" level=info msg="2022/09/15 23:13:41 http: proxy error: dial tcp <IP_ADDRESS>:61949: connect: connection refused"
time="2022-09-15T23:14:34+02:00" level=info msg="2022/09/15 23:14:34 [DEBUG] [chisel,monitoring] [endpoint_id: 8] [status: ACTIVE] [status_time_seconds: 2.271884] [message: environment tunnel monitoring]"
time="2022-09-15T23:14:41+02:00" level=info msg="2022/09/15 23:14:41 http: proxy error: dial tcp <IP_ADDRESS>:60159: connect: connection refused"

The Agent logs:
2022/09/16 08:59:53 [INFO] [main] [message: Agent running on Docker platform]
2022/09/16 08:59:53 [INFO] [edge] [message: Edge key loaded from options]
2022/09/16 08:59:53 [INFO] [edge,registry] [message: Starting registry credential server]
2022/09/16 08:59:53 [INFO] [http] [server_addr: <IP_ADDRESS>] [server_port: 9001] [use_tls: false] [api_version: 2.15.0] [message: Starting Agent API server]
2022/09/16 09:00:38 client: Connecting to ws://portainer.mydomain.de:8000
2022/09/16 09:00:38 client: Connection error: websocket: bad handshake
2022/09/16 09:00:38 client: Give up
2022/09/16 09:01:38 client: Connecting to ws://portainer.mydomain.de:8000
2022/09/16 09:01:38 client: Connection error: websocket: bad handshake
2022/09/16 09:01:38 client: Give up
2022/09/16 09:02:38 client: Connecting to ws://portainer.mydomain.de:8000
2022/09/16 09:02:38 client: Connection error: websocket: bad handshake
2022/09/16 09:02:38 client: Give up
2022/09/16 09:03:38 client: Connecting to ws://portainer.mydomain.de:8000
2022/09/16 09:03:38 client: Connection error: websocket: bad handshake

I googled a lot, tried a lot but nothing changes something in a better way.
Maybe it could be a problem with caddy? Also here I tried a lot. That's my config at the moment:
portainer.{$DOMAIN}:443 {
  tls {$EMAIL}
  reverse_proxy <IP_ADDRESS>:9233
}

tcp://portainer.{$DOMAIN}:8000 {
  tls {$EMAIL}
  reverse_proxy <IP_ADDRESS>:8111
}

And for the sake of completeness here also my portainer docker-compose.yml:
version: '3'

networks:
  caddy:
    external: true

services:
  portainer:
    image: portainer/portainer-ce:latest
    command: -H unix:///var/run/docker.sock
    container_name: portainer
    restart: unless-stopped
    security_opt:
      - no-new-privileges:true
    volumes:
      - /etc/localtime:/etc/localtime:ro
      - /var/run/docker.sock:/var/run/docker.sock:ro
      - ./portainer-data:/data
    ports:
      - 9233:9000
      - 8111:8000
    networks:
      caddy:
        ipv4_address: <IP_ADDRESS>
        ipv6_address: 2001:ab12::8

Im not sure if it is a problem with the agent, portainer or caddy but I hope some of you has an idea!

You need to change the the port that Portainer expects for tunnel-port. Also, try using our HTTPS service on port 9443.
version: '3'

networks:
  caddy:
    external: true

services:
  portainer:
    image: portainer/portainer-ce:latest
    command: --tunnel-port 8111 -H unix:///var/run/docker.sock
    container_name: portainer
    restart: unless-stopped
    security_opt:
      - no-new-privileges:true
    volumes:
      - /etc/localtime:/etc/localtime:ro
      - /var/run/docker.sock:/var/run/docker.sock:ro
      - ./portainer-data:/data
    ports:
      - 9233:9000
      - 8111:8111
      - 9443:9443
    networks:
      caddy:
        ipv4_address: <IP_ADDRESS>
        ipv6_address: 2001:ab12::8


[Kernel 5.4.51] Check for valid  tpm_class_ops before calling tpm_chip_start()

During device shutdown, we observe kernel panic due to
NULL pointer dereference with the following call trace:
[ 1586.593561][ T8669] tpm2_del_space+0x28/0x73
[ 1586.598718][ T8669] tpmrm_release+0x27/0x33
[ 1586.603774][ T8669] __fput+0x109/0x1d
[ 1586.608380][ T8669] task_work_run+0x7c/0x90
[ 1586.613414][ T8669] prepare_exit_to_usermode+0xb8/0x128
[ 1586.619522][ T8669] entry_SYSCALL_64_after_hwframe+0x44/0xa9
[ 1586.626068][ T8669] RIP: 0033:0x4cb4bb
This happens when driver has already unregistered its tpm_class_ops handlers
in tpm_class_shutdown(), and we are still trying to invoke the tpm_class_ops
handlers through TPM device handle closure. Putting in a check to proceed
with cleanup only when we have proper tpm_class_ops handlers populated.
Signed-off-by: Hariharasubramanian C S<EMAIL_ADDRESS>

The root cause of the issue is de-registration of chip->ops while there are still open files. I'm putting together a change which would eliminate this condition.

The root cause of the issue is de-registration of chip->ops while there are still open files. I'm putting together a change which would eliminate this condition.

ok, thanks @temerkhanov. Please let me know the PR#, I can dup this to that.

Dup'ing this to https://github.com/lf-edge/eve/pull/1702


User should not see broken image links for repos not yet in Circle CI

Circle CI now sends 404s for non-existent repos... handle more gracefully in table.

seems to have been fixed by Circle

Reopening, as we see broken image links again...

Intermittent bug.  Saved the "no builds" svg to:
https://github.com/ipfs/project-repos/blob/31e9c8b7c2463507373ce58bfbd4c07457133400/public/images/circle-ci-no-builds.svg
so we can use that if we get a 404.

I looked into this; it's hard to tell a priori if we're going to get a 404 due to cross browser issues. How would you do this? I am curious.

I suppose we'd make an ajax call for the image; if we got the image, insert an image tag pointing to it, if 404, then insert an image tag pointing to our images/circle-ci-no-builds.svg

Found a more elegant solution. See PR.


F-Droid listing

First congrats to your listing at F-Droid! At this point, apps are usually removed from my repo (especially when they did not establish reproducible builds) – but your Readme still links to my repo instead of F-Droid's. Will you replace those links?

Done, thanks @IzzySoft !

Thank you! Will be gone from my repo with the next sync then. All the best!


Stop modifying the ideascube config for nginx.

This change should have been made into ideascube's package
(extras/nginx/ideascube) and is now commited there.
Fixes #30.
dpkg now thinks the vhost has been manually modified, and won't install
its new version, leaving a .dpkg-dist file behind. We'll have to add a
file removal into ideascube's postinst script for some time :p

I agree with that point, but Ideascube is still listening on 443 port (see https://github.com/ideascube/ideascube/blob/188bec56c1400ee947b9530767e691d855f0e79e/extras/nginx/ideascube). Because we are using SSLH we could not have Ideascube and SSLH listening on the same port.
How are we going to handle that without modifying the file ?

The nginx vhost is shipped by ideascube.deb, in the future the change has to occur there, not in ansiblecube.
For the time being, we already have ansiblecube modify this file, so the ideascube.deb upgrade will leave a .dpkg-dist file behind anyway. This would have to be dealt with into ansiblecube, using some cleanup/rollingrelease way (which still has to be designed/stabilized, btw).

The PR #102 is taking care of these change adapted to oneUpdateFile branch


JetBrains: Make the plugin more convenient to debug

The first step here is to think about what low-hanging fruits we have to make plugin development more convenient for ourselves
Ideas:

Add logging
Enable tracing messages to the LLM (will be solved by the Agent)
Improve docs

One of the goals is to easily compare our communication with the LLM to what other clients are doing.

Closing since we can improve debugging on demand. The situation is somewhat improved with the agent through the build/sourcegraph/cody-agent-trace.json file.


physipy integration for units

Two way this can be accomplished
either
I: with another attribute of the custom Symbol class which is a Quantity object from physipy
or
II: use the dtype integration that they were working on and incorporate it with our ufloat dtype integration.
Cons of each method:
I: a lot of boilerplate code and testing involved
II: a lot of tinkering with dtypes basically no documentation meaning reading a lot of code
Pros of each method:
I: easy
II: it seams like there is less boilerplate i think and it is more elegant
Units:
I: Are tied to the abstract symbol
II: Are tied to the actual data and thus be loaded in with the csv
Things to read:
I: datamodel of python, our Symbol class, Quantity, Inheritence in Python
II: physipandas implementation of dtypes you can learn a lot,our dtypes, Inheritence in Python

Hi, I noticed you were considering integration of physipy in your project. If you are still interested in doing so, let me know ! I'd be happy to help


Fix: transpile default catalog name for default DuckDB test connection

SQLMesh currently assumes the default DuckDB test connection can execute queries containing the main connection's default catalog name, which is not always true due to different identifier quotes across engines.
This PR transpiles the default catalog name from the main connection dialect to the DuckDB dialect so DuckDB can accommodate default catalogs from all engines.
Background

A SQLMesh project's main connection may specify a default catalog. If the catalog name is quoted, it will use the identifier quotes of the connection's own SQL dialect.
By default, SQLMesh executes model unit tests on a local DuckDB connection.
The catalog name is part of a model's fully qualified name, which must be usable by DuckDB when it runs the test queries.

lgtm


How to set dummy IP via telemetry processor?

I'm trying to disable geolocation data collection and I've stumbled upon this git issue - #364 and it seem I can achieve it by predefining a tag on context param received in telemetry processor. And the question following:
How exactly should I add dummy IP address as context tag in addTelemetryProcessor function?
My attempts so far (none gave desired result):
client.addTelemetryProcessor((envelope, context) => {
    context.locationIp = "<IP_ADDRESS>";
    // i've also tried - context.tags.locationIp = "<IP_ADDRESS>";
    return true;
});

@markwolff and it works :^).
I guess i've been screwing up somewhere in my modules building sequence.
Appreciate the support, thanks.


Fix querying by commit integration test more

The additional matching CVE has now been rejected, and the text of the description has changed. I suspect we haven't seen the last of this yet, as a rejected CVE should be resulting in a withdrawn OSV record...
(I'm investigating why that isn't the case, but want to unbreak the test)

I confused disputed CVEs with rejected ones.
https://github.com/google/osv.dev/blob/aad1acbb33fc9d54fa6366681d50ccc9da552117/vulnfeeds/cmd/combine-to-osv/main.go#L119-L128 withdraws disputed CVEs.
I think we also need to be reflecting rejected ones as withdrawn. I'll file an issue for this.


Remove orange badge from mintupdate icon

On first boot (and until the user clicks "OK" on the welcome page), the
mintupdate-launcher icon has an orange badge on it. Since this tool
handles installing updates, it's showing an orange badge just to have
users press "OK" and see no updates.

(Another option would instead by to set hide-systray to true but for that we might want to only do it in OEM)