Tensorflow c++ memory leak - Valgrind

I am trying to run a simple tensorflow code from one of your sources https://github.com/Qualeams/Android-Face-Recognition-with-Deep-Learning-Library/blob/17136a831d1f592026982497ef75841499de593b/jni-build/jni/tensorflow_jni.cc
I am executing simple tensorflow code to create graph def as shown below
tensorflow::NewSession (options, &session)
ReadBinaryProto (tensorflow::Env::Default(), "/home/ashok/eclipseWorkspace/faceRecognition-x86_64/Data/models/optimized_facenet.pb", &graph_def));
session->Create (graph_def);

But when I run Valgrind as shown below
valgrind --leak-check=full --show-leak-kinds=all --vex-guest-max-insns=25 ./faceRecognition-x86_64 -r -i

I get below errors
==12366== 16,000 bytes in 1 blocks are still reachable in loss record 47,782 of 47,905
==12366==    at 0x4C2E19F: operator new(unsigned long) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==12366==    by 0xBF875DC: std::vector<tensorflow::CostModel::MemUsage, std::allocator<tensorflow::CostModel::MemUsage> >::reserve(unsigned long) (in /usr/lib/libtensorflow_cc.so)
==12366==    by 0xBF90128: tensorflow::CostModel::InitFromGraph(tensorflow::Graph const&) (in /usr/lib/libtensorflow_cc.so)
==12366==    by 0xBEE48D3: tensorflow::SimpleGraphExecutionState::InitBaseGraph(tensorflow::BuildGraphOptions const&) (in /usr/lib/libtensorflow_cc.so)
==12366==    by 0xBEE52CF: tensorflow::SimpleGraphExecutionState::MakeForBaseGraph(tensorflow::GraphDef*, tensorflow::SimpleGraphExecutionStateOptions const&, std::unique_ptr<tensorflow::SimpleGraphExecutionState, std::default_delete<tensorflow::SimpleGraphExecutionState> >*) (in /usr/lib/libtensorflow_cc.so)
==12366==    by 0xBE68B9D: tensorflow::DirectSession::MaybeInitializeExecutionState(tensorflow::GraphDef const&, bool*) (in /usr/lib/libtensorflow_cc.so)
==12366==    by 0xBE68CF9: tensorflow::DirectSession::ExtendLocked(tensorflow::GraphDef const&) (in /usr/lib/libtensorflow_cc.so)
==12366==    by 0xBE68FC7: tensorflow::DirectSession::Create(tensorflow::GraphDef const&) (in /usr/lib/libtensorflow_cc.so)
==12366==    by 0x26B899: TensorFlow::initializeRecognition() (in /home/ashok/eclipseWorkspace/faceRecognition-x86_64/Debug/faceRecognition-x86_64)
==12366==    by 0x24197D: RecognitionWithImages::RecognitionWithImages() (in /home/ashok/eclipseWorkspace/faceRecognition-x86_64/Debug/faceRecognition-x86_64)
==12366==    by 0x12F27C: main (in /home/ashok/eclipseWorkspace/faceRecognition-x86_64/Debug/faceRecognition-x86_64)

These type of errors are also generated when I do session -> run ()
Due to the above issues, the memory needed to run the program keeps increasing as time passes and the application crashes due to insufficient memory after  a certain point of time.

Hi @ashokbugude this seems to be a common issue with TF's C++ implementation. Check this comment and the links he provides: https://github.com/tensorflow/tensorflow/issues/5302#issuecomment-257430153 using tcmalloc seems to work

Thanks,
Can I know please know where and how could tcmalloc be implemented in  https://github.com/Qualeams/Android-Face-Recognition-with-Deep-Learning-Library/blob/17136a831d1f592026982497ef75841499de593b/jni-build/jni/tensorflow_jni.cc

As to "the memory needed to run the program keeps increasing as time passes and the application crashes due to insufficient memory after a certain point of time."
There is another reason !  I have a patch to fix it but don't know how to submit it!

Great! You can check this guide on how to create a pull request https://gist.github.com/MarcDiethelm/7303312

Hi @tomsang best would be to open 2 separate pull requests, so things don't get mixed up. I will then test the code before merging it.

Hi @sladomic ,  I will firstly submit part 2 because it's very simple.
But when I using following command,
git push --set-upstream origin memrelease
where memrelease is my local branch.
I get the following access deny error
remote: Permission to Qualeams/Android-Face-Recognition-with-Deep-Learning-Library.git denied to tomsang.
Could you give me permission?  Or is my command wrong?  This is my first time to submit patch to github.

@sladomic I've finished two pull requests for lib and test framework on part 2.

Part 1 is done. See pull request in test framework and lib , respectively


chore: move .gitignore rules to packages

Summary
I've moved some .gitignore rules into package-local .gitignore files which IMO makes keeping track of the relevant rules easier than managing them globally.
Just one question: It seems packages/documentation/static/versioned_examples/ is populated ever. Is that right? The .gitignore rule
!packages/documentation/static/versioned_examples/*/webcomponent-examples

since none of its parent directories was excluded, so there's no need to include this specific subpath again.
How did you test this change?
Nothing to test here.

@sisp It was intended to keep only one gitignore file. Can you please move the lines back to the root/.gitignore.
Just one question: It seems packages/documentation/static/versioned_examples/ is populated ever. Is that right? The .gitignore rule

!packages/documentation/static/versioned_examples/*/webcomponent-examples

Yes we use the docusaurus versioning. (https://docusaurus.io/docs/versioning - https://github.com/siemens/ix/blob/main/packages/documentation/src/components/Demo.tsx#L42)

I'll just close the PR if you prefer a single .gitignore file at the root of the project; nothing to move back.


Custom_Components

I was attempting to create a custom_component for a simple chat feature.   I have code in the javascript section that should only be run server side, not on the client.  All of the javascript ends up in the /Client folder so the server side javascript never executes.  Where is the best place to put that code?  In the API,  there is a place for server startup code but I don't see a way to specify javascript that should only be run on the server.  It would also be ideal to package that with the custom_component,  maybe a server side javascript and client javascript area in custom_components?

Hi, components are only for client side. You can create .js with any content inside and do something like this in your json input file:
{
  "copy_files": [
    { " source": "path/to/your/file/relative/to/this/json/input/file", "dest": "SERVER_DIR/your_file.js" }
  ]
}

SERVER_DIR is one of possible constants, please see api reference, file_pair for other constants.
that is if you are doing directly from json (without GUI). If you are using kitchen gui, therecis no possibility to do something like that except server_startup_code

server_startup_code will 100% execute server-side only because it is located in /server dir (see meteor docs about "structuring your app"). For code that resides in /server or /client you don't need if(Meteor.isServer) or if(Meteor.isClient).

That makes sense and dropping my server side code in the server_startup_code worked perfectly.  Thank you!

:+1:


Two new nodes

Hey, I have been busy. I have created two new nodes for your plugin for your consideration

Location node - this node gets events that are globally fired by Hubitat. You find them in HE under "System Events". An example is the "systemStart" event that notifies that the hub rebooted.
HSM node - this is a node that can get the HSM state, trigger when there is an alarm and also send commands like arming and disarming to HSM.

Both of them work as usual via the MakerAPI interface.
Let me know if you have questions

Hey, the callbacks with the fake device ids for mode, hsm and location drove my OCD through the roof. So I added a second change to this PR where the interaction is event based instead of registering callbacks.

I'm still not at home for now, but would it be possible to separate your PR into 3 PR (HSM, Location, ring node) :pray:
I will be able to check/merge it easier and if one feature take more time to review, it will not block other features to be released

I am closing this one and you got a few new once


no /dev/kvm

Can I not use/dev/kvm? Because I want to enable Docker Windows and also use virtualbox

是的，你可以设置
environment:
  KVM: "N"

消耗KVM即可运行。但是也意味着机器会慢动作，除非您因此希望所有的东西都慢动作，否则就没有用了。

Yes you can set
environment:
  KVM: "N"

to run without KVM. But it will also mean the machine will be much slower, so it's not useful to do that unless you want everything to be in slow-motion.

I've been stuck on this interface ever since I set it up like this

Virtualbox

If I don't disable KVM, I can't use Virtualbox. But it seems like there will be issues with disabling KVM for me

我不知道为什么......它不会真正建议禁用KVM。如果你想同时使用Virtualbox应该不是问题，因为多个VM可以同时使用/dev/kvm。它不仅仅可以只有1个程序使用。

I'll try setting the GPU to Y


clone gophercon workshops

Hey Sara here is the PR I opened with the clones files and you can edit them here how you want. For each new URL you can see it by the name of the folder they are in.
Checklist:

[ ] I have reviewed the style guide.
[ ] If blogging, I have reviewed the blogging guide.
[ ] I have manually confirmed that all new links work.
[ ] I added aliases (i.e., redirects) for all filename changes.
[ ] If making css changes, I rebuilt the bundle.

@sean1588 @cnunciato the events are at 5 pm... does it use the 24-hour format? I'm not sure how to indicate it otherwise.


The type or namespace name 'agora_gaming_rtc' could not be found

I want to playing demo but a lot of not found error is glad to me.
how can I solve it?

Please make sure you download and import the Agora Video SDK.

Please make sure you download and import the Agora Video SDK.

thx


Dictionary issue

Hi! I just downloaded your aligner to test it out, and I've come across a few dictionary issues.
The first problem I encountered is that it appears many basic words are missing from the default dictionary, such as AIN'T, THAT'S, and YOU'LL.
I guess the default dictionary is for basic tests, so I tried the dictionary from the dictionary repository. Sadly these words are still missing.
I tried another dictionary, this time from CMUSphinx. This one has a lot more words, but it gave me a few errors because of some # comments in the file.
I removed these comments, but then it gave me another error of the word 's' being out of order. I tried to sort it with sort.py, but this made no difference.
I'd really appreciate finding a working dictionary with more words in it, including acronyms such as the ones mentioned above. Trying to fix the CMUSphinx dictionary would also be fantastic.
Thank you for this software and any help with this!

The dictionary distributed here is just a trivial fork of the classic CMU
one. We're not really in the pronunciation dictionary business. Why don't
you just add those words to the dictionary locally?
IMO 'S should be tokenized separately, as a single word, since it can
attach to literally any category of word in English:
[Peter]'s mother
[The Queen of England]'s corgis
[The woman I saw yesterday]'s new hat
[The man you like]'s best friend
etc.
If you don't do it this way you will (slowly, inefficiently) have to add an
"'S" form of the entire lexicon.
You may want to make sure you don't have "smart" apostrophes---I'm not sure
the underlying HTK library takes kindly to those with respect to sort order.
On Thu, Feb 8, 2018 at 7:01 AM, MysteryPancake<EMAIL_ADDRESS>wrote:

Hi! I just downloaded your aligner to test it out, and I've come across a
few dictionary issues.
The first problem I encountered is that it appears many basic words are
missing from the default dictionary, such as AIN'T and THAT'S.
I guess the default dictionary is just for basic tests, so I tried the
dictionary from the dictionary repository
https://github.com/prosodylab/prosodylab.dictionaries/blob/master/en.dict.
These words are still missing.
So, I tried another dictionary from CMUSphinx
https://github.com/cmusphinx/cmudict/blob/master/cmudict.dict. This one
has a lot more words, but it gave me a few errors because of some comments
in the dict file.
I removed these comments, but then it gave me another error of the word
's' being out of order. I tried to reorder it with sort.py, but this made
no difference.
I'd really appreciate finding a working dictionary with more words in it,
including acronyms such as the ones mentioned above. Even fixing the
CMUSphinx dictionary would help.
Thank you for this software!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prosodylab/Prosodylab-Aligner/issues/68, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAJuOZtRxa3fM1SA4DezDRCpUlYCAgrmks5tSuIYgaJpZM4R-PSv
.

Acronyms are a whole open class, so they're not usually done in the
pronunciation dictionary.
They can be read as words ("NATO"), letter sequences ("USA"), specialzied
versions thereof ("NAACP") and occasionally as the full form (if I see
"SRY" I might read it as "sorry"). There is a bunch of literature on how to
determine which but the state of the art gives us relatively low recall,
and feels out of scope for this project.
On Thu, Feb 8, 2018 at 9:28 AM, Kyle Gorman<EMAIL_ADDRESS>wrote:

The dictionary distributed here is just a trivial fork of the classic CMU
one. We're not really in the pronunciation dictionary business. Why don't
you just add those words to the dictionary locally?
IMO 'S should be tokenized separately, as a single word, since it can
attach to literally any category of word in English:
[Peter]'s mother
[The Queen of England]'s corgis
[The woman I saw yesterday]'s new hat
[The man you like]'s best friend
etc.
If you don't do it this way you will (slowly, inefficiently) have to add
an "'S" form of the entire lexicon.
You may want to make sure you don't have "smart" apostrophes---I'm not
sure the underlying HTK library takes kindly to those with respect to sort
order.
On Thu, Feb 8, 2018 at 7:01 AM, MysteryPancake<EMAIL_ADDRESS>wrote:

Hi! I just downloaded your aligner to test it out, and I've come across a
few dictionary issues.
The first problem I encountered is that it appears many basic words are
missing from the default dictionary, such as AIN'T and THAT'S.
I guess the default dictionary is just for basic tests, so I tried the
dictionary from the dictionary repository
https://github.com/prosodylab/prosodylab.dictionaries/blob/master/en.dict.
These words are still missing.
So, I tried another dictionary from CMUSphinx
https://github.com/cmusphinx/cmudict/blob/master/cmudict.dict. This
one has a lot more words, but it gave me a few errors because of some
comments in the dict file.
I removed these comments, but then it gave me another error of the word
's' being out of order. I tried to reorder it with sort.py, but this made
no difference.
I'd really appreciate finding a working dictionary with more words in it,
including acronyms such as the ones mentioned above. Even fixing the
CMUSphinx dictionary would help.
Thank you for this software!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prosodylab/Prosodylab-Aligner/issues/68, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAJuOZtRxa3fM1SA4DezDRCpUlYCAgrmks5tSuIYgaJpZM4R-PSv
.

Ok, thanks for the information. I didn't want to add the words myself because those are only a few examples out of the many, many words I would like to recognize. It'd take me longer than my lifetime to add the rest of the English language to the dictionary without using a model.


Update README.md

Small text changes

Apart from README.md changes, does your PR conributes anything to the codebase?


test: add test cases

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

Please upload report for BASE (dev@be864f5). Learn more about missing BASE report.

Additional details and impacted files

@@          Coverage Diff           @@
##             dev       #3   +/-   ##
======================================
  Coverage       ?   45.44%
======================================
  Files          ?       92
  Lines          ?     5813
  Branches       ?      134
======================================
  Hits           ?     2642
  Misses         ?     3171
  Partials       ?        0


Simulation with Navigation Tutorial - Odometry and Relative Position of the Robot

ISSUE TEMPLATE ver. 0.3.0

Which TurtleBot3 you have?

[x] Burger Simulation
[ ] Waffle
[ ] Waffle Pi

Which ROS is working with TurtleBot3?

[ ] ROS 1 Kinetic Kame
[x] ROS 1 Melodic Morenia
[ ] ROS 2 Dashing Diademata
[ ] ROS 2 Eloquent Elusor
[ ] etc (PLEASE, WRITE DOWN YOUR ROS VERSION HERE)

Which SBC(Single Board Computer) is working on TurtleBot3?

[ ] Raspberry Pi 3
[ ] Intel Joule 570x
[ ] etc (PLEASE, WRITE DOWN YOUR SBC HERE)

Which OS you installed in SBC?

[ ] Raspbian
[ ] Ubuntu MATE 16.04 or later
[ ] Ubuntu Server 18.04 or later
[ ] etc (PLEASE, WRITE DOWN YOUR OS)

Which OS you installed in Remote PC?

[ ] Ubuntu 16.04 LTS (Xenial Xerus)
[x] Ubuntu 18.04 LTS (Bionic Beaver)
[ ] Linux Mint 18.x
[ ] Linux Mint 19.x
[ ] etc (PLEASE, WRITE DOWN YOUR OS)

Write down software version and firmware version

Software version: [x.x.x]
Firmware version: [x.x.x]

Write down the commands you used in order

roslaunch turtlebot3_gazebo turtlebot3_world.launch
roslaunch turtlebot3_navigation turtlebot3_navigation.launch map_file:=$HOME/map.yaml

Copy and Paste your error message on terminal

[ WARN] [1587301388.239094507]: Request for map failed; trying again...
[ WARN] [1587301388.408921807,<PHONE_NUMBER>00]: Timed out waiting for transform from base_footprint to map to become available before running costmap, tf error: canTransform: target_frame map does not exist.. canTransform returned after 331.467 timeout was 0.1.
[ WARN] [1587301389.071870437,<PHONE_NUMBER>00]: global_costmap: Pre-Hydro parameter "static_map" unused since "plugins" is provided
[ WARN] [1587301389.072218122,<PHONE_NUMBER>00]: global_costmap: Pre-Hydro parameter "map_type" unused since "plugins" is provided
[ INFO] [1587301389.072792353,<PHONE_NUMBER>00]: global_costmap: Using plugin "static_layer"
[ INFO] [1587301389.077796208,<PHONE_NUMBER>00]: Requesting the map...
[ INFO] [1587301389.280235713,<PHONE_NUMBER>00]: Resizing costmap to 384 X 384 at 0.050000 m/pix
[ INFO] [1587301389.380003368,<PHONE_NUMBER>00]: Received a 384 X 384 map at 0.050000 m/pix
[ INFO] [1587301389.382628392,<PHONE_NUMBER>00]: global_costmap: Using plugin "obstacle_layer"
[ INFO] [1587301389.385791493,<PHONE_NUMBER>00]:     Subscribed to Topics: scan
[ INFO] [1587301389.397858572,<PHONE_NUMBER>00]: global_costmap: Using plugin "inflation_layer"
[ WARN] [1587301389.429684765,<PHONE_NUMBER>00]: local_costmap: Pre-Hydro parameter "static_map" unused since "plugins" is provided
[ WARN] [1587301389.429928199,<PHONE_NUMBER>00]: local_costmap: Pre-Hydro parameter "map_type" unused since "plugins" is provided

Please, describe in detail what difficulty you are in

What I am trying to do: Running SLAM and Navigation tutorials successfully with turtlebot3_simulations package using Gazebo and turtlebot3_world
I successfully created the map by moving teleop_keyboard and saved it.
When I run the the navigation task with turtlebot3_navigation.launch, I get the warning messages I copy-pasted above.
When rviz shows up after running turtlebot3_navigation.launch, I initially get this.

At this point, I would like to understand whether it's normal or not. Is the robot suppose to appear in a random point after the initial launch or should the laser scan and the loaded map should be more or less matching? Am I getting this weird match because of the errors displayed while launching the node?

When I select 2D Pose Estimate and click on the place where the robot actually is on Gazebo simulation,  and moving/rotating it around a little to let it localize itself, I get the following result, which actually looks accurate.

At this point, I would like to understand what should the correct /odom publish values be. Since this it the inital position that we estimated, should it be 0 0 0? In my case, the map looks correct but my odom publishes something like position.x = -2 and position.y = -0.5 which is exactly the distances between the position of the robot from my first image above, where the robot initially appears after running the navigation node for the first time.

header:
seq: 31440
stamp:
secs: 1048
nsecs: 427000000
frame_id: "odom"
child_frame_id: "base_footprint"
pose:
pose:
position:
x: -2.00012834414
y: -0.499377554506
z: -0.00100139400472
orientation:
x: -2.59687525772e-05
y: 0.00385302931369
z: 0.00490783327682
w: 0.999980533142

I have read through similar issues with the same warning messages, that's why I created this issue to understand if something is wrong at all about my results. Thanks for the help in advance.

@ozangungortuhh
Hi, I'm sorry for the late response. Your navigation window seems okay. In order to properly run the Navigation, you should set the actual pose of the robot in the map using 2D Pose Estimate button. When setting the robot pose, it will help amcl algorithm to estimate the current robot location which is described as small green arrows around the robot. As the robot travels further, these arrows will getting closer to the actual location of the robot with better accuracy.
When setting the 2D Pose Estimate, you should click at the actual robot location and drag to the direction where the robot is facing.
Thank you.


falco-0-26-2.md file translation Chinese

falco-0-26-2.md file translation Chinese
Signed-off-by: tianying<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: cytianying
To complete the pull request process, please assign jasondellaluce after the PR has been reviewed.
You can assign the PR to them by writing /assign @jasondellaluce in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 4bcc602388ab909ccb514f5d96bc8ec1707a6553


Alter database option

What problem does the new feature solve?
Since v0.8, GreptimeDB supports database TTL just like:
CREATE DATABASE test with(ttl='7d');

which means all the tables in this database will inherit this option if they don't have their own TTL setting.
It does not support alerting the options after they are created.
What does the feature do?
Alter the database options:
Alter database test set ttl = '14d`;

Implementation challenges
No response

I'm interested in this issue. I guess we can implement this feature with the following high level steps.

Allow parser to handle sql like ALTER DATABASE test SET ttl = '14d'.
Implement AlterDatabaseProcedure. (I guess that greptimedb doesn't support this yet.) This procedure should alter the ttl for every inherited table(region) in a database, and then also alter the database ttl.

Cool. @NiwakaDev

Allow parser to handle sql like ALTER DATABASE test SET ttl = '14d'.

I recommend designing the statement to use set OPTIONS(ttl='14d') since we might need to SET other things in the future. Introducing the OPTIONS keyword is more beneficial. The use of parentheses () allows for batch setting of options, for example:
alter database test options ('ttl'='14d', 'compaction.time_window'='1d');

Implement AlterDatabaseProcedure. (I guess that greptimedb doesn't support this yet.) This procedure should alter the ttl > for every inherited table(region) in a database, and then also alter the database ttl.

Yes, you have to implement a procedure, which needs to change the database metadata. I am not quite sure if we need to alter all the inherited table options too, because:

We don't have a value to represent if the option is inherited or defined by table DDL.
It needs the table supports alter table set options, but it doesn't #1084

What do you think?  @evenyag @fengjiachun

alter database test options ('ttl'='14d', 'compaction.time_window'='1d');

Looks good.

It requires the table to support alter table set options, but it doesn't

Should table options have higher priority than database options?

It requires the table to support alter table set options, but it doesn't

I think alter database relying on table options might not be a good idea. Because synchronizing and modifying all table options when executing alter database options could involve a large number of table modifications.
My idea is that alter database options should only update the database metadata. When the region GC worker operates, it retrieves the TTL of the database from DatabaseMetadataManager, merges it with the region's TTL to get the minimum value, and then continues its task.
Right now, we don't have a DatabaseMetadataManager, but that's not a big issue. We just need to wrap the KvBackend, and then we can get the TTL value of database from metasrv.

@fengjiachun

When the region GC worker operates, it retrieves the TTL of the database from DatabaseMetadataManager, merges it with the region's TTL to get the minimum value, and then continues its task.

Aren't there any situations where we want to prioritize the table ttl even if the database ttl is smaller?

@fengjiachun

When the region GC worker operates, it retrieves the TTL of the database from DatabaseMetadataManager, merges it with the region's TTL to get the minimum value, and then continues its task.

Aren't there any situations where we want to prioritize the table ttl even if the database ttl is smaller?

I think the region's TTL option should have higher priority. It overwrites the database's TTL if we set the table TTL.
@fengjiachun What do you think?

Of course, the table's custom TTL must be prioritized over the database TTL.
However, the table TTL is set to the database TTL when creating a table without the ttl table option.
https://github.com/GreptimeTeam/greptimedb/blob/b81d3a28e6223fb3cdcc53f8fed91dc73f99a335/src/operator/src/statement/ddl.rs#L1501
We don't know if a table is inherited or custom the TTL at all. There is one solution is that we can add a flag to figure out that the table TTL is merged from database TTL, and then while altering database TTL, we can change these tables(regions) TTL too.
Another choice is as @fengjiachun said, let's do not change the region's TTL but just the database options.

Another choice is as @fengjiachun said, let's not change the region's TTL but just the database options. I prefer the second solution. It's much simpler, and altering many tables is expensive.

Even if we choose the choice, I think we still need a flag indicates whether or not the table is inherited. I guess that adding the flag isn't expensive.
When the region GC works, if the flag is set, we prioritize the table TTL. Otherwise, we select the database TTL from the DatabaseMetadataManager.

Another choice is as @fengjiachun said, let's not change the region's TTL but just the database options. I prefer the second solution. It's much simpler, and altering many tables is expensive.

Even if we choose the choice, I think we still need a flag indicates whether or not the table is inherited. I guess that adding the flag isn't expensive.

Agree, let's add a flag to indicate whether or not the table option is inherited. Maybe it's an option set like inherited_options.

When the region GC works, if the flag is set, we prioritize the table TTL. Otherwise, we select the database TTL from the DatabaseMetadataManager.

Sound good. Then it doesn't need to alter all the table options, but just retrieving the database TTL while gc.

I'll implement this feature with the following high level steps:

[ ] Implement DatabaseMetadataManager.
[ ] Implement AlterDatabaseProcedure.
[ ] Allow parser to handle sql like alter database set options ('ttl'='14d', ~);.

Sorry for the delay in implementing this. I'll submit a first PR next week.


Chroma installation problems diffuse to personids-lite dev version

Docker-chroma is being built for start-dev.sh
However, at this moment, the chroma release is broken - see https://github.com/chroma-core/chroma/issues/797.
This causes personids-lite not to function since when chroma does not work the plugin has an exception and the product is not working.
Moreover, the Dockerfile-chroma uses

RUN git clone https://github.com/chroma-core/chroma.git
so it takes the version that exists at the time of building, even if it is not the official version, simply the latest.

The production version takes it's image from personoids/chroma which is more stable, maybe the dev version should do the same since the dev version is probably for the personids-lite  development and not for the chroma version. So taking it from the fixed version seems to avoid transient issues in the chroma development.

AFAIU the problem was diffused to personoids/chroma image as well...


Fixed error when using custom error codes

Fixed error when using custom error codes > 800. The default color at index 0 is used by default.
Fixes #72

Is there a plan to merge this? :)

3.2.1


feat(ecs): Add RepositoryCredentials support

Implements #1737

Pull Request Checklist

[ ] Testing

Unit test added (prefer not to modify an existing test, otherwise, it's probably a breaking change)
CLI change?: coordinate update of integration tests with team
cdk-init template change?: coordinated update of integration tests with team

[ ] Docs

jsdocs: All public APIs documented
README: README and/or documentation topic updated

[ ] Title and Description

Change type: title prefixed with fix, feat will appear in changelog
Title: use lower-case and doesn't end with a period
Breaking?: last paragraph: "BREAKING CHANGE: <describe what changed + link for details>"
Issues: Indicate issues fixed via: "Fixes #xxx" or "Closes #xxx"

[ ] Sensitive Modules (requires 2 PR approvers)

IAM Policy Document (in @aws-cdk/aws-iam)
EC2 Security Groups and ACLs (in @aws-cdk/aws-ec2)
Grant APIs (only if not based on official documentation with a reference)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

@SoManyHs & @rix0rrr - posting this for initial feedback. Suggested changes welcome. Thanks!

Feel free to just post these PRs directly to the aws-cdk repository. It's actually easier for me to go over them there.

BTW, is it still possible to merge this PR into https://github.com/awslabs/aws-cdk/pull/1737 ?

@rix0rrr sure! I'll address the above feedback and add it to  awslabs#1737

was merged as https://github.com/awslabs/aws-cdk/pull/1737


Please enable WITH_VAAPI=ON in nightly builds

Is there any reason to keep it at WITH_VAAPI=OFF? It would be nice to be able to use GPU acceleration.
Though it would be nice to have a configuration option to enable or disable it at runtime in case you expect some instability with it.

pull requests welcome, but I doubt you will be able to implement something that is not works on my machine for a tiny subset of connection parameters

It can be for example some environment variable if you prefer. Something like FREERDP_USE_VAAPI=true if  you prefer that over connection paramters.
Other than that, I just tested it (on AMD GPU) and it works fine with VAAPI. So it's sort of  waste not to have an option to use it if it's available. I see even Debian package isn't enabling it and I suspect for the same reason - there is no way to make it configurable at runtime. I can take a look at where this can be made conditional.

OK, I took a look. It has a whole bunch of conditionals with
#ifdef WITH_VAAPI
...

I can add a boolean flag that would also be checked in addition to ifdefs (for parts that actually do some logic), and that flag would be set depending on some env variable like above or if may be command line parameter like /vaapi or something like that.
What do you think?

On a side note - I've been using  a VAAPI enabled rebuilt Debian package without any issues so far.

@shmerl yes, it works for certain hardware combinations and connection settings (that is the reason we have it in there) but not for others and there is no easy way to detect which ones work and which do not.
so, we´re happy about pull requests to make this a runtime option, but definitely not to be enabled by default.

Btw, do you prefer to remove VAAPI related ifdefs and simply have a runtime check, or you want to keep compile time exclusion with ifdef and add a runtime check on top for case when compile time was enabled? It feels somewhat messy to have all those ifdefs with runtime checks on top.

@shmerl keep the #ifdef for now (we don´t want to enable stuff not tested at all in stable releases) but if found stable we can remove these (so runtime checks on top)

I made an initial PR, just enabling the /vaapi switch in the initial commit. It's not being used yet (just in case I got something wrong before proceeding further):
!10181.

How can I get access to settings from h264_ffmpeg.c module? Not that familiar with the code, but other modules have something called context for that?

@shmerl depends on what you want to implement.
but basically you need to use h264_context_set_option  to pass it to the subsystem. (you might need to add a new value to the H264_CONTEXT_OPTION enum)

Also, rdpShadowEncoder* encoder object there has access to  encode->client->context.settings and  encode->server->settings. Which one should I use to pull the info from?

@shmerl since that is a server side configuration option that is independent of the client settings encoder->server->settings.
encoder->client->context.settings has the configuration for the connected peer (graphics mode, ...) which contains the settings that both, the server and client support. (e.g. if the server supports GFX-AVC444 and the client only GFX-AVC420 that setting is disabled)

OK, I made a change and tried testing it, but I don't see it kicking in even with /gdi:hw set. Must be I'm missing something or I made some mistake. Let me open a PR for review anyway and I'll try to debug it a bit later when I have a chance.

Now I'm getting something confusing. Tried building without my patches simply with VAAPI_ENABLED=ON.
And I still get:
VCN: Powered down

Reported by amdgpu. So something isn't working regardless of my changes even.

@shmerl better join our matrix chat for such support questions ;)
anyway, can´t really tell without seeing your code, you got a fork with the patches somewhere?

Yeah, posted the PR from it: https://github.com/FreeRDP/FreeRDP/pull/10191
See: https://github.com/shmerl/FreeRDP/tree/runtime_vaapi_2

I updated the PR, but as above, I don't see actual VCN turning on according to amdgpu. I can join matrix a bit later to dig more into this.

Digging into it in the debugger, somehow it skips over this part:
#ifdef WITH_VAAPI
		if (h264->HardwareAccelerated != 0)
		{
			if (!sys->hwctx)
			{
...

Checking why h264->HardwareAccelerated wasn't set.

Hmm, shadow_encoder_init_h264 isn't even called in my case.

Going up the call chain, neither is static DWORD WINAPI shadow_client_thread(LPVOID arg) called anywhere.

@shmerl what are you talking about?
the init function is called on first frame sent, the other is a thread function passed to CreateThread


Saved model to file

Hi, I have made changes so that the model will be saved (after training) in a .sav file named 'trained_model.sav' (defined in config.yaml). Let me know if any other changes are required. (Issue #3)

Hey thanks, this is pretty nice!
Rather than import pickle, though, torch has a built-in function torch.save https://pytorch.org/docs/stable/generated/torch.save.html
And I think the typical file extension is trained_model.pt.
If you update these two things then I'll merge your PR!

Sure, will do that. Thanks!

Looks good, I'll go ahead and merge it! Thanks for the help!


Do you have the document detection in real time while using the camera to get the photo through MediaCapture API ?

Do you have the document detection in real time while using the camera to get the photo through MediaCapture API ?
thank you !

Have you tried the one on the README? It's here

Yes, sorry, I should have said it ... I tried it but it seems is broken since I see my webcam, and two divs (canvas and result) with just a zoomed in part of the camera (the top left part)

The problem is that the width and height of both canvas are not set. Get the canvas elements and assign the mentioned properties. After that it should work.
Like: canvas.width = video.videoWidth.

It's because the canvas width and height. By default is always 300 x 150

Interesting. I haven't taken a look at the specifics yet, but have you tried setting the dimensions of result to that of resultCanvas?

Yes, I did it that way forcing the width/height (setting them by javascript). No problem in that.
Thank you !


Fix arrow for Select

Closes #371
Inspired by: https://codepen.io/herrfischer/pen/aNpWwy

Codecov Report

Merging #374 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #374   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          62     62
  Lines         316    316
=====================================
  Hits          316    316

Impacted Files
Coverage Δ

src/Select.js
100% <100%> (ø)
:arrow_up:

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 debd594...5004aa0. Read the comment docs.

ping to @jxnblk !

Thanks! I think snapshots need to be updated with npm t -- -u

@jxnblk done! sorry for the delay
What do you need for releasing v2.x officially? 🙂

Awesome thanks! For v2, just a couple small issues and mostly docs for the official release


Toogle button for Calibrate CJC has a different shape

To Reproduce
open Chrome browser on windows PC
Expected behaviour
all toggle buttons are the same shape and size
Actual behaviour
Only the Calibrate toggle button is strange shape. it looks like instead of a white ball, it is vertical and partially cut off
Screenshots

ah, windows edge cases. i'll tweak the switches style a little

fixed now  in version Source Code  PineSAM-1.5.7 I just tested.


Swiper Pagination Bug

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 4.3.3

Platform/Target and Browser Versions: macOS, Windows, Chrome, Safari, Edge, Firefox

Live Link or JSFiddle/Codepen or website with issue: https://codepen.io/deskmaster/pen/dqeQbB

What you did
Created a new swiper and set loop to true.
Expected Behavior
Everytime I click the pagination bullet it should go to the correct slide.
Actual Behavior
If you click pagination bullets enough eventually it goes to the wrong slide.
I have checked the closed and open issues and not seen one where this bug is actually resolved.

It works correctly, if you click too fast, then you can do a swipe instead of click


Riscv smp mmu

Summary

This commit adds SMP support for BUILD_KERNEL

Impact

RISC-V: BUILD_KERNEL + SMP only

Testing

Tested with QEMU 6.2

See the original discussion in https://github.com/apache/incubator-nuttx/pull/7247

@xiaoxiang781216
I think this PR is much simpler than #7247


Cannot find module 'nan' when setting up FluidHelloWorld

When installing the dependencies for the FluidHelloWorld app, users might encounter an issue with the module 'nan' not being available. I was able to fix this by deleting the package-lock.json file and running npm install after.

To Reproduce
Steps to reproduce the behavior:

git clone https://github.com/microsoft/FluidHelloWorld.git
cd FluidHelloWorld
npm install
See error

Expected behavior
Dependencies should be installed with no issues.

Logs

Thank you for the report.  I have tried your steps, and wasn't able to reproduce the error.
Looking at the package-lock.json, the project don't seem to haven't an dependency to the nan package.
Can you try npm ci?

Thanks for looking into it. It is working for me now after removing the package-lock file.

Excellent! I'm going to resolve this issue then.


Hi👌✌✌👍😍 @piascikj

🙏🚶👉🤟🎥

Thailand customer📸📸

Siam city😁😁😁

🎥📸🎥 Thailand


The intro doesnt work when the user is using an adblocker

I was doing some testing and apparently the intro doesnt work at all when you use an adblocker, the browser im using is firefox and the adblock im using is uBlock Origin paired with nanodefender to allow me to block ads on website that would give you a popup saying that you have to disable your adblocker.
I did some testing including disabling all extensions and even installing another adblock because ublock origin might be not working correctly and the only way the intro properly loaded was when i uninstalled or disabled the adblocker.

Adblock is blocking ip trackers, I can't do anything with this.
Thanks for posting.


Adding conda to get Cookiecutter

Description
Rationale
Use case(s) / visualization(s)

pip is not the only way to get Cookiecutter, we can also use  conda

Correct, it's not the only way, there is also (at least):

Homebrew
pipx
pipenv

Cookiecutter Django is a tempalate, I don't think we should list all of the ways to install the cookiecutter CLI, but rather document the preferred way and then link to the CLI documentation for other ways. Something along the lines of this maybe?

First, get Cookiecutter via pip (or via another installation method)

Thanks for the effort in trying to get this better documented, but it's not in a state to be accepted here and this is quite well documented in the CLI documentation.


tfchain DAO: why paramterizing in ProposalIndex

https://github.com/threefoldtech/tfchain/blob/development/substrate-node/pallets/pallet-dao/src/proposal.rs
While this should be fine
pub type ProposalIndex = u32;

#[derive(PartialEq, Eq, PartialOrd, Ord, Clone, Debug)]
pub struct DaoProposal{
    pub index: ProposalIndex,
    pub description: Vec<u8>,
    pub link: Vec<u8>,
}

fn main() {
  let d = DaoProposal{index:5, description: vec![], link:vec![]};
  println!("{:#?}", d);
}

Probably a code style error that slipped in, can be removed


Erroneous next fragment term offset computation

This is mostly a nitpick, and I'm not sure my analysis below is correct, so please correct me if I'm wrong.
In FragmentAssembler#handleFragment, the term offset of the next fragment is computed as
BitUtil.align(offset + length + HEADER_LENGTH, FRAME_ALIGNMENT)

offset and length are (most likely) provided by TermReader#read so their values are frameOffset + HEADER_LENGTH and frameLength - HEADER_LENGTH respectively, and therefore offset + length is equal to frameOffset + frameLength. Aligning that value to the FRAME_ALIGNMENT to obtain the offset of the next frame makes sense. But adding + HEADER_LENGTH before doing the alignment doesn't.
It just so happens that HEADER_LENGTH is equal to FRAME_ALIGNMENT and so the operations x -> BitUtil.align(x, FRAME_ALIGNMENT) and x -> x + HEADER_LENGTH commute. But if HEADER_LENGTH was not a multiple of FRAME_ALIGNMENT then this code would break. So for cleanliness and to avoid a bad surprise if the transport protocol ever changes (unlikely I guess), it would be best to instead write BitUtil.align(offset + length, FRAME_ALIGNMENT) + HEADER_LENGTH.

Fixed in https://github.com/real-logic/aeron/commit/037f0c4bed0d4d9ca0b43d30b30314684ea684fe.


Append trailing slash in Django

This commit https://github.com/omab/python-social-auth/commit/f57e0caf4ec06f3b9a2367d9ff3897e2a3a71608 made trailing slashes optional.
The side effect of that is that the redirect_uri is created without a trailing slash.
With this change the standard Django APPEND_SLASH is checked when generating the redirect url.

Thanks!


potential.total_energy barfs for ndim>2

Should return an array of energies.

Fixed in branch break-api


2.0.2 version import error

When impoting responder 2.0.0 to 2.0.2, import error was raised:

Traceback (most recent call last):
File "", line 1, in
File "D:\Envs\pp-ng\lib\site-packages\responder_init_.py", line 1, in
from .core import *
File "D:\Envs\pp-ng\lib\site-packages\responder\core.py", line 1, in
from .api import API
File "D:\Envs\pp-ng\lib\site-packages\responder\api.py", line 28, in
from .templates import Templates
ImportError: cannot import name 'Templates' from 'responder.templates' (D:\Envs\pp-ng\lib\site-packages\responder\templates_init_.py)
ImportError: cannot import name 'Templates' from 'responder.templates'

Version 1.3.2 imported just fine.

Also when I test in a new venv. starlette also raised some error.

ModuleNotFoundError: No module named 'itsdangerous'

I can confirmed both errors:
 test_responder pipenv --python 3.7
Creating a virtualenv for this project…
Pipfile: /home/iancleary/dev/test_responder/Pipfile
Using /usr/bin/python3.7 (3.7.5rc1) to create virtualenv…
⠧ Creating virtual environment...Already using interpreter /usr/bin/python3.7
Using base prefix '/usr'
New python executable in /home/iancleary/.local/share/virtualenvs/test_responder-RWmZuL56/bin/python3.7
Also creating executable in /home/iancleary/.local/share/virtualenvs/test_responder-RWmZuL56/bin/python
Installing setuptools, pip, wheel...
done.
Running virtualenv with interpreter /usr/bin/python3.7

✔ Successfully created virtual environment!
Virtualenv location: /home/iancleary/.local/share/virtualenvs/test_responder-RWmZuL56
Creating a Pipfile for this project…
➜  test_responder ls
Pipfile
➜  test_responder pipenv shell
Launching subshell in virtual environment…
 . /home/iancleary/.local/share/virtualenvs/test_responder-RWmZuL56/bin/activate
➜  test_responder  . /home/iancleary/.local/share/virtualenvs/test_responder-RWmZuL56/bin/activate
(test_responder) ➜  test_responder pipenv install responder
Installing responder…
Adding responder to Pipfile's [packages]…
✔ Installation Succeeded
Pipfile.lock not found, creating…
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
✔ Success!
Updated Pipfile.lock (42ca41)!
Installing dependencies from Pipfile.lock (42ca41)…
  🐍   ▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉ 34/34 — 00:00:03
(test_responder) ➜  test_responder python
Python 3.7.5rc1 (default, Oct  8 2019, 16:47:45)
[GCC 9.2.1 20191008] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import responder
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/iancleary/.local/share/virtualenvs/test_responder-RWmZuL56/lib/python3.7/site-packages/responder/__init__.py", line 1, in <module>
    from .core import *
  File "/home/iancleary/.local/share/virtualenvs/test_responder-RWmZuL56/lib/python3.7/site-packages/responder/core.py", line 1, in <module>
    from .api import API
  File "/home/iancleary/.local/share/virtualenvs/test_responder-RWmZuL56/lib/python3.7/site-packages/responder/api.py", line 15, in <module>
    from starlette.middleware.sessions import SessionMiddleware
  File "/home/iancleary/.local/share/virtualenvs/test_responder-RWmZuL56/lib/python3.7/site-packages/starlette/middleware/sessions.py", line 5, in <module>
    import itsdangerous
ModuleNotFoundError: No module named 'itsdangerous'
>>> exit()
(test_responder) ➜  test_responder pipenv install itsdangerous
Installing itsdangerous…
Adding itsdangerous to Pipfile's [packages]…
✔ Installation Succeeded
Pipfile.lock (6e2a34) out of date, updating to (42ca41)…
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
✔ Success!
Updated Pipfile.lock (6e2a34)!
Installing dependencies from Pipfile.lock (6e2a34)…
  🐍   ▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉ 35/35 — 00:00:02
(test_responder) ➜  test_responder python
Python 3.7.5rc1 (default, Oct  8 2019, 16:47:45)
[GCC 9.2.1 20191008] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import responder
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/iancleary/.local/share/virtualenvs/test_responder-RWmZuL56/lib/python3.7/site-packages/responder/__init__.py", line 1, in <module>
    from .core import *
  File "/home/iancleary/.local/share/virtualenvs/test_responder-RWmZuL56/lib/python3.7/site-packages/responder/core.py", line 1, in <module>
    from .api import API
  File "/home/iancleary/.local/share/virtualenvs/test_responder-RWmZuL56/lib/python3.7/site-packages/responder/api.py", line 28, in <module>
    from .templates import Templates
ImportError: cannot import name 'Templates' from 'responder.templates' (/home/iancleary/.local/share/virtualenvs/test_responder-RWmZuL56/lib/python3.7/site-packages/responder/templates/__init__.py)

We could add itsdangerous to the Pipfile if it's no longer installed by a dependency.

I can confirm same issue on either v2.0.0 or v2.0.2, on v1.3.2 works well.

Thanks guys, can you check with v2.0.3 ?

@taoufik07 it is fixed on v2.0.3, awesome, thanks!


Guanyu/feature/spring and lock

Good news! Now everything works well in simulation. Thank you!
I am not sure why we need to update pointmassfirstorder. it seems that if we consider it as a low pass filter, the dc gain will contain the stiffness, and the time constant is a combination of both the damping and stiffness.
$X = \frac{Force/k}{1+\frac{d}{k}s}$
where k is the stiffness, d is damping, and $\frac{d}{k}$ is the time constant.

Hi Jianwei,
I tried to create a workflow in the actions section for the unit test. Do you have any recommendations of those workflows like c/c++ with make or MSbased projects?

Ah, you don't need to worry about any of that. Please follow the instructions at the Contributing section (near the end) of the README.md.

I cmake with the prefix path to mujoco and osqp. It can find osqp config files. However, for mujoco, it shows that "CMake Warning at CMakeLists.txt:46 (find_package):
Could not find a configuration file for package "mujoco" that is compatible
with requested version "2.3.1".
The following configuration files were considered but not accepted:
E:/PhD_Research/Two_arm_exo/exoul8/lib/mujoco-2.3.1/build/mujocoConfig.cmake, version: 2.3.1 (32bit)". The version is 2.3.1 as shown here, but it said it cannot find the package that is compatible with 2.3.1.

It seems like after I broadcast the affine terms, the barrier, rectangular, and circular constraints tests cannot pass. The mujoco-related tests can neither pass even in the main branch no matter whether there is the affine term.


fsdfdsfd

Was this intentional?

Reverted this change.

Yes, it was unintentional (thought I was merging to my fork's main). Thank you @awvwgk.


Tells users to pipe to shell

This looks like a really cool project and I look forward to trying it.  :)
However, it's a terrible anti-pattern to tell people to pipe from curl or wget to a shell.  Please do not do this.

Even though it's generally (seen as) a bad practice, it's so useful for:

people who already inspected install.sh and found no malicious code;
me.

Anyway I provide a couple of alternatives.
But out of curiosity... how does the pipe approach is different from:
wget -qO- https://raw.githubusercontent.com/cyrus-and/trace/master/install.sh
sh install.sh

or even a "Click here to download the installer then run sh install.sh", or even running any other piece of code on your computer?
The only real concern may be the partial execution of the shell file due to an abrupt wget termination, but honestly it seems to me such an unlikely event that a proper solution would be overkill, especially for a small utility like this.

Another attack vector is for a malicious server to return different content based on the user-agent, so people who examine the script in their browser could see a harmless script, but then receive a malicious one when they curl or wget it.
Frankly, I think the early-termination issue is nothing to sneeze at, and is enough reason by itself to never recommend this practice.
Then there are other issues, such as updating the software.  If you clone a repo with git, the subsequent pulls are more difficult to tamper with, and auditable.  Plus, if you use signed git tags, it provides even more security for updates.
The bottom line is that it's just a very, very bad idea, and one that should be avoided at all costs, even on principle alone.
But, hey, it's your repo... :/


Thoroughly document all exported symbols with JSDoc

Also, remove removeComments TSConfig option now that we have some
relevant comments.

Codecov Report

Merging #11 (0b1a575) into master (f21a14b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #11   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           58        58
  Branches         9         9
=========================================
  Hits            58        58

Impacted Files
Coverage Δ

packages/core/src/index.ts
100.00% <ø> (ø)

packages/express/src/index.ts
100.00% <ø> (ø)

packages/next/src/index.ts
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Locking to Main Thread

Hello, is it possible to lock the Main Actor to the main OS thread? When writing GUI applications on Mac, many functions can only be called from the main thread

No, it isn't possible at this time.
If you would like to follow up on this, please join the Zulip.
For all questions, we ask that you use the Zulip.


Good to have a prereq() function which mention the prerequistes of this tool.

Its good to have a function which mention the prerequisites we need to have for the gk-deploy to run.  This helps the storage admin wrt to the deployment of this solution.

Mention or check?... @humblec
I think these would be 2 different issues.

@obnoxxx ok.. lets have this issue for "mention" and will open another one for "check" later. :)

@humblec ACK, thanks.


TransportQueryError when using GraphQL tool

System Info
When running the following code:
from langchain import OpenAI
from langchain.agents import load_tools, initialize_agent, AgentType
from langchain.utilities import GraphQLAPIWrapper
from langchain.memory import ConversationBufferMemory

llm = OpenAI(temperature=0, openai_api_key=openai_api_key)
token = "..."
tools = load_tools(
    ["graphql"],
    custom_headers={"Authorization": token, "Content-Type": "application/json"},
    graphql_endpoint="...",
    llm=llm
)

memory = ConversationBufferMemory(memory_key="chat_history")
agent = initialize_agent(
    tools, llm, agent=AgentType.CONVERSATIONAL_REACT_DESCRIPTION, verbose=True, memory=memory
)

graphql_fields = """query getCompanies {get_companies}"""

suffix = "Call the API with schema "

agent.run(f"{suffix} {graphql_fields}")

Im getting the error:
TransportQueryError: Error while fetching schema: {'errorType': 'UnauthorizedException', 'message': 'You are not authorized to make this call.'}
If you don't need the schema, you can try with: "fetch_schema_from_transport=False"
It doesn't matter what value is provided under custom_headers, or if it is passed as a parameter at all. The error is always the same. Playground code from https://python.langchain.com/docs/modules/agents/tools/integrations/graphql worked as intended.
Any idea of what the problem is?
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[X] Agents / Agent Executors
[X] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
from langchain import OpenAI
from langchain.agents import load_tools, initialize_agent, AgentType
from langchain.utilities import GraphQLAPIWrapper
from langchain.memory import ConversationBufferMemory

llm = OpenAI(temperature=0, openai_api_key=openai_api_key)
token = "..."
tools = load_tools(
    ["graphql"],
    custom_headers={"Authorization": token, "Content-Type": "application/json"},
    graphql_endpoint="...",
    llm=llm
)

memory = ConversationBufferMemory(memory_key="chat_history")
agent = initialize_agent(
    tools, llm, agent=AgentType.CONVERSATIONAL_REACT_DESCRIPTION, verbose=True, memory=memory
)

graphql_fields = """query getCompanies {get_companies}"""

suffix = "Call the API with schema "

agent.run(f"{suffix} {graphql_fields}")

TransportQueryError: Error while fetching schema: {'errorType': 'UnauthorizedException', 'message': 'You are not authorized to make this call.'}
If you don't need the schema, you can try with: "fetch_schema_from_transport=False"

Expected behavior
An allowed API call that doesn't cause authentication issues

This is fixed in this PR here: https://github.com/langchain-ai/langchain/pull/8187

Hi,
I have the same issue.
Graphql endpoint is public so
I don't know why I should authenticate ?
Will the pull request solve this ?
Thanks :)


Add projects argument under browse-osp

Used urllib to extract easily the project names in every URL provided by the  section of the packages.
In this way we can parse the Git URL (extracted with the net_loc) and receive the projecs (Which is the path) following the URL Syntactic Components.

Hmm, after a few thoughts, I have a doubt it should be done in this place.
The ospinfo itself does not provide projects per se, so why to add it, with another custom set of commands, while what we do is we process the packages information and so we can simply take basenames of znoyder browse-osp packages --output osp-patches?
Can we discuss tomorrow again what's the goal and how we want to use this?
I would vote more for adding filter on osp-patches to get_packages() if we really need it and not introduce additional command like that here.

Hmm, after a few thoughts, I have a doubt it should be done in this place. The ospinfo itself does not provide projects per se, so why to add it, with another custom set of commands, while what we do is we process the packages information and so we can simply take basenames of znoyder browse-osp packages --output osp-patches? Can we discuss tomorrow again what's the goal and how we want to use this?
I would vote more for adding filter on osp-patches to get_packages() if we really need it and not introduce additional command like that here.

Yes, let discuss it tomorrow.


In Fluent Meshing, recording python journal when save mesh file instead saves fmd file

🔍 Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

🐞 Description of the bug
Recording journal of the action of saving an msh.gz file records the following command:
PartManagement.SaveFmdFile(FilePath=r'E:/test_workflow_files\test.fmd')
It should instead save it as an msh.gz file.
📝 Steps to reproduce
Record a journal including the operation of saving a mesh file.
💻 Which operating system are you using?

Sounds like a Fluent server issue and not PyFluent?
The new Fluent meshing datamodel exposure for file operations might help with this, depending on what exactly the issue is. Will see if I can reproduce this locally (a script would certainly help).

This is indeed a server-side issue, going to debug this as I can find the time

This is indeed a server-side issue, going to debug this as I can find the time

@raph-luc Does that mean you have a script to reproduce?

@seanpearsonuk sorry, I should've written more details down.
No script necessary as this is a server-side issue and not related to PyFluent: just start up Fluent (can be 2023 R2 as reported here or newer), either load an existing mesh or e.g. initialize a WTM workflow and at least import a geometry, then start recording a Python journal, and use the file menu to write a mesh file. Finally, check the journal file and see that the wrong command was recorded.
Tested just now with 2024 R2 and it seems to save workflow.SaveWorkflow(FilePath=r'C:/Tests/mesh_workflow_files\mesh.wft') instead which is still not what we want.
I am thinking we should close this issue here in the PyFluent repo, and create a Fluent issue tracker for this. Should I go ahead and do that?

create a Fluent issue tracker for this. Should I go ahead and do that?

Yes, please. That will save me working through it and duplicating your effort. Thanks very much.

I am thinking we should close this issue here in the PyFluent repo

Generally we keep the PyFluent issues open until the child issue is closed. I normally write the Fluent ID in a comment. I realise that it's heavyweight but it has advantages too.

Fluent issue tracker 1035421


Plugin configuration

Developers should be able to configure their plugin via multiple ways:

YAML (plugin.yaml)
JSON (plugin.json)
PHP (inside Plugin.php)

Plugin configuration should include:

Identifier
Name
Description
Homepage
Version
Author (or Authors)

Name
Email
Homepage

Dependencies

Identifier
Version

Base configuration will only be possible via the plugin.yaml configuration. Any further required customization can be done through the plugin's main class. Included fields are subject to change.


SPR-136 Dependency Workflow Scanning

Description
This PR includes the following proposed change(s):

Added workflows that check for npm dependency updates, courtesy of @BradyMitch

Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change
[ ] Dependencies added/removed

Aspect(s) of Project Affected

[ ] Frontend
[ ] API
[ ] Database
[x] Workflows
[ ] Documentation

Best Practices Checklist

[x] I have performed a self-review of my own code
[ ] My code follows the Airbnb React Style Guidelines
[ ] Documentation has been updated to reflect my changes
[ ] New and existing tests pass locally with my changes

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Testing live in GH Actions.

Pending @BradyMitch 's request changes, this looks good to me.


Sorry to interupt, when will the new version be released?

Hello,
Sorry for the delay, the new 2.9.0 version is now live.


ArrayIndexOutOfBoundsException from DependencyUseMapNotation

Discovered using SAAS to run DependencyUseMapNotation on Default repositories.
The recipe can throw an index out-of-bounds exception.

Since this is on the basic string side, it may be easier to utilize the DependencyStringNotationConverter utility clas for parsing the string. This would then make it easier to put the appropriate pieces into their map entries and also help to complete the picture for the remaining string notations (ie. Classifier and extension).


HsImport: Return codeactions importing from Internal modules last

Fixes #1702

Thank you, nice addition!

Thanks :)
On Sun, Mar 15, 2020, 6:47 PM fendor<EMAIL_ADDRESS>wrote:

Thank you, nice addition!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/haskell/haskell-ide-engine/pull/1703#issuecomment-599193802,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGRJXABEAKF6IYTLMNCF6DRHSW3LANCNFSM4LIQAQAA
.


Update API versions of Intel and K8s DRA

Update apiVersion in the examples to align with current iplementation:

gpu.dra.intel.com/v1alpha changed to gpu.resource.intel.com/v1alpha2
resource.k8s.io/v1alpha1 to resource.k8s.io/v1alpha2

Thanks for the contribution!

I noticed that the examples aren't aligned with the current implementation. I.e. when I deploy the DRA plugin from main branch, the examples don't work because of the wrong (old) apiVersion of both K8s DRA API and Intel resource API. I'm wondering if there's a reason for that that I'm missing.

An oversight, there's more examples in deployments/tests that are conformant with latest API version, and they are example workloads as well, even though the directory is named "tests". We'll need to cleanup either the old examples directory structure that is split into directories or the "tests".

Oh, I just noticed, we didn't update the deployments/tests either, you updated it too, thanks!


--

There's multiple, you should specify which one instead of saying it doesn't work

https://drive.google.com/file/d/1tWh_JElu5XI2LWRpmcES8DWJNqXSZ7p9/view?usp=sharing
see

Like I said, you should specify which one you're using before simply saying it isn't working.

i using this Agile.NET-Deobfuscator
i am using this
https://github.com/ribthegreat99OrN0P/Agile.NET-Deobfuscator/issues
after decrypt

Make an issue there then, not here

@developerss this repo is only for listing unpackers/deobfuscators not for support with the programs listed in the readme.

posted it in [on Jun 24, 2021] now 2023 why people still bump up this , thanks for the close and i have updated the issue ,

bump

posted it in [on Jun 24, 2021] now 2023 why people still bump up this , thanks for the close and i have updated the issue ,
@OnlyIslam lol

posted it in [on Jun 24, 2021] now 2023 why people still bump up this , thanks for the close and i have updated the issue ,

posted it in [on Jun 24, 2021] now 2023 why people still bump up this , thanks for the close and i have updated the issue ,

@remotemachine1 you bumped this 2 days ago tho?


[WIP] Support blocksize and aggregate_files options in ReadParquetPyarrowFS

Proposed Changes
Adds optimization-time ("tune up") support for blocksize and aggregate_files in ReadParquetPyarrowFS.
I like how dask-expr currently "squashes" small files together at optimization time. This PR simply expands the functionality of _tune_up to:

Split (rather than fuse) oversized parquet files (when blocksize is provided; Default is None for now).
Enable/disable small-file fusion with the aggregate_files argument (default to True when filesystem="arrow").

Note on (1): This is definitely a much higher priority than (2).
Note on (2): ReadParquetPyarrowFS already "aggregates" files by default to avoid small partitions. It seems pretty natural to extend this functionality to target a specific blocksize value when one is specified. I do think aggregate_files is  an intuitive way to enable/disable the "squashing" of small files at optimization time, but I'm okay with leaving this out in favor of the "dataframe.parquet.minimum-partition-size" config. With that said, I do think blocksize should take precedence over that config value.
Background
The legacy read_parquet infrastructure is obviously a mess, and I'd like to avoid the need to keep maintaining it (in both dask and rapids). The logic in ReadParquetPyarrowFS is slightly arrow-specific, but is already very close to what I was already planning to do in rapids. The only missing feature that is preventing it from supporting real-world use cases is the lack of support for splitting oversized files (a need we definitely run into a lot in the wild).

I do have a pretty strong preference on not adding keywords that interact with other things counterintuitively. One of the main complexity drivers of the old read_parquet implementation is that there is a plethora of options that disable each other, so I don't want to repeat this here

I do have a pretty strong preference on not adding keywords that interact with other things counterintuitively.

Yeah, that makes sense. What do you think is the best way to deal with oversized files? I don't think the aggregate_files argument is necessary, but the lack of support for "splitting" large files is a pretty serious blocker right now. Would a "dataframe.parquet.maximum-partition-size" config be more palatable?

Update: I moved away from using blocksize and aggregate_files in favor of "dataframe.parquet.minimum-partition-size" (already used) and "dataframe.parquet.maximum-partition-size" (new) configs.


[test] Remove FUSD contract referrences

Description
The FUSD contract was removed from Flow Emulator, so I have updated the flow-emulator and flow-go versions and fixed the relevant breaking changes that occurred.

[x] Targeted PR against master branch
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[x] Code follows the standards mentioned here
[x] Updated relevant documentation
[x] Re-reviewed Files changed in the Github PR explorer
[x] Added appropriate labels

@sideninja After this is merged, we'll need a release for cadence-tools/test, and I will follow up with a PR on flow-cli to bump the version.

@m-Peter should we also release LS?

I am not sure if there are unreleased changes there, but if there are, it's a good time to release them both :pray:

I meant because this PR changes LS go deps too

Oh, yes, I totally forgot about this :sweat_smile:

The build would fail: https://github.com/onflow/cadence-tools/actions/runs/5623624068/job/15238707883, that's why I had to update the deps in the entire repo.


Found Issue with Build - fatal error: hardware_interface/handle.hpp

Hey Josh,
Having an issue when building the diffdrive_arduino package (on the humble branch):
--- stderr: diffdrive_arduino                                                                     In file included from /home/ralph/r7_ws/src/diffdrive_arduino/hardware/diffbot_system.cpp:15: /home/ralph/r7_ws/src/diffdrive_arduino/hardware/include/diffdrive_arduino/diffbot_system.hpp:22:10: fatal error: hardware_interface/handle.hpp: No such file or directory    22 | #include "hardware_interface/handle.hpp"       |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ compilation terminated.
It seems there is a missing: filehandle.hpp
I am currently going through the configuration to establish if this is an issue with the build and will keep this thread updated as to how i get on.
Any help here would be superb, Love the series!

Further to the above,

I have tested the following fork : https://github.com/joshnewans/diffdrive_arduino/pull/4
I have also tested with the latest Rolling and Humble ros2 control packages

Things to note - WS is fairly unchanged from the tutorial


Fix RECURSIVE over SQL adapter

How does using it multiple times work?

In SQL, recursive keyword does not apply to a specific CTE, but it applies to the whole query. So any/all CTEs can be recursive when the keyword is present.
Location of the flag in our AST is the questionable part here. We should probably has a separate ast node With { ctes: List[CommonTableExpression], recursive: bool }.

(... but my code does not take that into account properly)

Remember to label for backport. too


Save draft somewhere when sending fails

I tried to send an email, but it failed.
❯ EDITOR=nvim h write
(s)end, (d)raft, (e)dit or (q)uit? s
Sending…
internal client error: timed out waiting for connection: Connection error: Could not connect

The email text is no longer accessible to me, but himalaya should at least print the path to the email file, and maybe also allow me to retry it (eg: h write --input=email-draft.txt).

Duplicate https://github.com/soywod/himalaya/issues/59.


JSON schema for I-V curve files

@cwhanse I decided to use "Issues" to provide feedback on this effort. You might consider opening up the "Discussions" section for this repo, which might be more appropriate for these items, which are really up for discussion :). Thanks to you and your team for taking on this project.
My first idea is to provide a JSON schema for the I-V curve files. The one that I am familiar with is https://json-schema.org/learn/getting-started-step-by-step, which has at least one Python implementation: https://python-jsonschema.readthedocs.io/en/stable/. Among other things, this should allow users to check if their files are formatted correctly.

One related issue here: The units of the Voltage and Current could be made more explicit in the JSON files. When working with 2" reference cells, mA is not uncommon for the current (sometimes also mV for voltage), so it would be convenient (but by no means necessary) to allow other units. Either way, it seems like documenting the units makes the JSON files more self contained and foolproof.

Seems like including the series cells per string and parallel strings per device would make the I-V curve files more self-contained.
If these files are eventually used for shading studies, then more circuit-topology info could be useful, such as bypass diodes, but that gets complicated and probably could be left for v2.

Metadata around the irradiance and temperature seem warranted. For example, back of module average temperature vs. an embedded temperature probe directly touching the backside of a reference cell. Also, what is the spectrum of the irradiance?

One more suggestion: The I-V curves are naturally ordered/indexed as members of the "IV Curves" JSON array. Thus, I would rename the  "Index" key to "Label" and allow users to provide custom label strings for each curve.
This could satisfy, for example, a typical use case where each curve is read in from a separate file, and the user could naturally assign the filename as the label of each curve. Such labels could likewise refer to the nominal temperature and irradiance combination in the IEC 61853-1 matrix.


CRN-1052 News Data Provider - GraphQL

rebase to master to solve code conflict

rebase to master to solve conflict


inquiry: Possibility of selected sql fields

I'm currently exploring graphql and stumbled upon this project, and it's a great tool for my current project. I noticed though that by default, the execution of the select command fetches all columns. Would it be possible to have a feature to only select fields that were included in the original graphql query? Thanks!

It is possible to exclude columns for graphql (select would fetch all and I can think of a way to have this possibility as well) but until that's not available you can use exclude option.
https://github.com/almost-full-stack/graphcraft/blob/feature/refactor/src/index.js#L105
you can exclude via Model.graphql.attributes.exclude = ['id', 'name'] and so on and these columns won't be available in any of the type generated by package.

Can we just pass the query fields to the attributes array so only those are returned and queried against the db https://sequelize.org/master/manual/model-querying-basics.html#specifying-attributes-for-select-queries


Unable to get Instance ID token

After allowing notification permission, I wasn't able to see Instance Token ID on Chrome 54.0.2840.59 (64-bit) version
Screenshot: http://prnt.sc/cwkozz

http://prnt.sc/cwkozz

If the IID token generation fails, it should be logged in the console, are you seeing any error message there?

no

I got error Error "retrieving Instance ID token."

but on MAC OS it just saying "loading..." with not erro in console.

I think I'm missing something...
http://www.solidarityfarm.tk/quickstart-js-master/messaging/index.html
This below is the only error I'm getting with Firefox 49.0.2 :
An error occurred while retrieving token.  Object { code: "messaging/unsupported-browser", message: "Messaging: This browser doesn't sup…", name: "FirebaseError", stack: "", 1 more

With above URL I am getting "Only secure origins are allowed (see: https://goo.gl/Y0ZkNV)." error in console.

Thank you for trying it inautus :) I appreciate that.
Should I ask my hosting service to put my URL in an https:// in order for this messaging to work? Or is there another way to make it work?

i have eror too , like @inautus  error:
browserErrorMessage
:
"Failed to register a ServiceWorker: ServiceWorker script evaluation failed"
code
:
"messaging/failed-serviceworker-registration"
message
:
"Messaging: We are unable to register the default service worker. Failed to register a ServiceWorker: ServiceWorker script evaluation failed (messaging/failed-serviceworker-registration)
runing in localhost/firebase/quickstart/messaging
anyone can hlep  ?

The "script evaluation failed" normally means there was a problem with the
javascript in the service worker file.
Does it link to a line number? Also are you starting the server in the
messaging folder?
Matt
On Mon, 31 Oct 2016, 23:53 alfat osk<EMAIL_ADDRESS>wrote:

i have eror too , like @inautus https://github.com/inautus error:
browserErrorMessage : "Failed to register a ServiceWorker: ServiceWorker
script evaluation failed" code :
"messaging/failed-serviceworker-registration" message : "Messaging: We are
unable to register the default service worker. Failed to register a
ServiceWorker: ServiceWorker script evaluation failed
(messaging/failed-serviceworker-registration)
runing in localhost/firebase/quickstart/messaging
anyone can hlep ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/firebase/quickstart-js/issues/69#issuecomment-257500829,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIh8P1KH5mK1m8eOJgPu3qpVv7SNoIbks5q5uH9gaJpZM4Kb1bu
.

yess i got it , the problem is , script read server worker in http://localhost/firebase-messaging-sw.js , not in my app folder http://localhost/firebase/quickstart/messaging , so i must add file firebase-messaging-sw.js in root of folder localhost (/var/ww/html)  , i dont know but its work now

The 'firebase-messaging-sw.js' file needs to be at the root of your project.
If you can't do that you can use the useServiceWorker(registration) method which would look something like this:
navigator.serviceworker.register('/app/path/example/sw.js')
.then((registration) => {
    const messaging = firebase.messaging();
    messaging.useServiceWorker(registration);
    return messaging.getToken();
})
.then((token) => {
    ....
});

I am getting the similar error as following:
An error occurred while retrieving token.  V {code: "messaging/failed-serviceworker-registration", message: "Messaging: We are unable to register the default s…t. (messaging/failed-serviceworker-registration).", browserErrorMessage: "Failed to register a ServiceWorker: A bad HTTP res…code (404) was received when fetching the script.", stack: "FirebaseError: Messaging: We are unable to registe….gstatic.com/firebasejs/3.5.1/firebase.js:537:225"}

Where do you get firebase-messaging-sw.js code and how do you add that to the root of folder localhost? @riddbengkok

And It is working fine and generating token on the chrome's localhost server but not working on my hosting server.
Please Help! on issue.

I got same problem with all browser, When I am trying with my hosting server.

+1

I have the same problem. Analysing the code I saw the worker is registered as '/firebase-messaging-sw.js', so, putting 'firebase-messaging-sw.js' on website root, it works correctly.
It seems to me it's a bug.

Yeah, it seems that you need to set up an SSL certificate in your domain. In localhost it will run fine, but, in order for a service worker to start, it must come from a secure origin meaning it has to come through the "https" protocol

solved this problem by changing the file firebase.js
on line :
return navigator.serviceWorker.register("/firebase-messaging-sw.js",{scope:"/firebase-cloud-messaging-push-scope"})
change :
return navigator.serviceWorker.register("firebase-messaging-sw.js",{scope:"firebase-cloud-messaging-push-scope"})
and I made a folder firebase-cloud-messaging-push-scope with firebase.js files in it

@karsa87 The better solution is to do as you are and call useServiceWorker() to use a service worker if the location of the service worker file is a problem: https://firebase.google.com/docs/reference/js/firebase.messaging.Messaging#useServiceWorker

@gauntface thank you, for your suggestion. it worked well.

Hello i'm having the same issue
FirebaseError: Messaging: We are unable to register the default service worker. Failed to register a ServiceWorker: A bad HTTP response code (404) was received when fetching the script. (messaging/failed-serviceworker-registration).
at http://localhost:5000/__/firebase/3.8.0/firebase.js:544:225
I have my firebase-messaging-sw.js file in a root directory and it still giving me a error, what is the problem?
Please help

@MiloslavKrejza Please continue discussing this on the other issue on this repo that you've commented on.

i am unable to generate the token in http localhost, what could be the reason? I don't see any error in console

I prefer to use useServiceWorker using the following syntax
firebase.messaging().useServiceWorker(registration)
what should be the value of registration object that we pass in here?
It should point to my firebase_ServiceWorker.js file in same directory. which contains:
//start of firebase_ServiceWorker
firebase.initializeApp({
'messagingSenderId': '103953800507'
});
const messaging = firebase.messaging();
//END of firebase_ServiceWorker
After this is done, I expect to get FCM token on web.


PSI Library: Implement Join (Transform) (Rust)

OpenAPI Definition

[ ] Write definition
[ ] Write comments/docs

Rust Runtime

[ ] Write runtime
[ ] Implement tests
[ ] Write comments/docs

Rust Validator

[ ] Write validator
[ ] Implement tests
[ ] Write comments/docs

Python bindings

[ ] Create bindings
[ ] Implement tests
[ ] Write docs

R bindings

[ ] Create bindings
[ ] Implement tests
[ ] Write docs

Design needed.  Which implementation type, etc


thread 'tokio-runtime-worker' panicked at 'Paths must start with a /',

Upon running trunk serve I receive this error
Full Error

2023-01-11T03:53:08.421068Z  INFO ✅ success
thread 'tokio-runtime-worker' panicked at 'Paths must start with a /', /home/guest/.cargo/registry/src/github.com-1ecc6299db9ec823/axum-0.5.17/src/routing/mod.rs:134:17
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
Aborted (core dumped)

Modifying Trunk.toml to

[build]
public_url = "/"

corrects the issue.
I'm unaware of the implications of such a change. A PR will follow

#5 addresses the issue. Unsure of possible unintended consequences. Builds on my machine (ubuntu-linux)


adds uri encoding to edit comment view

https://looker.atlassian.net/browse/MRKTI-442

Hey TJ, noticed I left out the uri encode/decode logic from when a comment is being edited. Small PR here, thanks


Inverter struct and reference names

To do: for v0.4.
Discussion on proper names for:

[ ] Inverter components' struct
[ ] Reference names for Inner Vars and Symbol names of Inverter

Specific changes for PowerSystems:

[x] Rename VirtualInertiaQdroop (outerloop) control to a generic one (OuterLoopControl?) since the changes are different models are in the active and reactive power parts.
[ ] Discussion on symbol names: xi and gamma are not standard names for PI states of voltage and current controller respectively. I suggest to adopt that convention (used by D'Arco).
[x] Rename VSControl abstract to InnerControl.
[ ] Rename all ports that uses VSM to OLC (outer loop control), e.g. theta_vsm or omega_vsm.
[ ] CurrentControlVirtualZ and SimpleCurrentControl

Rename inner vars. Proposed list:

md_var
mq_var
Vdc_var
Vdo_var => Vd_filter_var (behind last L filter, probably transformer)
Vqo_var => Vq_filter_var (behind last L filter, probably transformer)
omega_freq_estimator_var
v_control_var => V_oc_var (Voltage reference provided by the outer loop to the inner loop)
omega_control_var => omega_oc (simply the virtual frequency in the outer loop control)
delta_dqRI_var => theta_oc
VR_inv_var
VI_inv_var
Vdcnv_var => Vd_cnv_var
Vqcnv_var => Vq_cnv_var

Standarize symbols for Inverters. Proposed list:

For OuterLoop stick with omega and theta, and avoid delta_theta y delta_omega. This change will require to modify the D'Arco VSM model.
For the capacitor voltage we are using :vd_cap and :vq_cap, that should be the same that the proposed inner vars Vd_out_var and Vq_out_var. The key challenge here is that if a RL filter is used (without capacitor) Vd_out_var should simply be VR_inv_var and VI_inv_var before the dq transformation. A proposed change is simply go with :Vd_out and :Vq_out.
Modify :id_o and :iq_o to :Id_out and :Id_out, and modify :id_c and :iq_c to :Id_cnv and :Iq_cnv

Some specific changes must be done in PowerSystems.

On the Inverter Struct, the vscontrol must be renamed to innercontrol
On the Inverter Struct, the getter function must be renamed from get_vscontrol to get_vscontrol. This change will require to modify the inner control model in LITS.
On the Inverter Struct, All VC acronyms must be replaced by IC (inner control).
On all changes, documentation must be updated.
The AvgCnvFixedDC must be eliminated (AverageConverter already added).


Blazor component structure

Hello!
Is it possible to build an Blazor component with separating C# component stuff (@functions {}), html and css files?
Something similar we have in Angular framework.
Structure:

Usage from .ts:

@SteveSandersonMS nice, thanks!


@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/r-tensora-feedstock#3.


[Bug]: Pop-up of calendar date picker goes underneath other panes, should be above

Describe the bug
I have my todo list set in the left sidebar as a page that's always open - using the calendar date picker when the line is close to the next pane (my central one) results in the pop-up being half underneath the next pane, making it impossible to select a date.
Expected Behavior
Calendar date picker should be displayed above any other pane.
Steps to reproduce
Create a note, put it on the left sidebar (not just on left side of the current note, but in the side-bar you can collapse), write some text to get close to the edge of the main pane, hit (@ to summon the calendar date picker, notice its underneath the main pane.
Operating system
Windows

Thank you for the report.  I fixed that problem in v1.1.1.
Before fix

After fix

Please review the item mentioned below at the same time since it seems to be another issue with the Calendar pop-up!
https://github.com/uphy/obsidian-reminder/issues/34#issue-1033793124

Fixed.  Also the calendar popup works on mobile now.
So I close this issue.


Problem: Job Scheduling via Cron Notation complicated

Description
Quantum currently allows configuration via Cron Notation. This works very well for statically recurring events, but doesn't work very well for dynamic ones.
If you for example want to run a dynamic job once every day for 7 days until it stops, that's very complicated to do.
Implementation
Generic way of configuring such jobs should be provided. Any Ideas how to do that?
PR
We'd be happy to receive a PR (after the implementation details are cleared). If this is a popular request please leave a 👍 so that we see that this is important to our users and implement it ourselves.

I strongly agree this is important functionality that I'm missing. I have some ideas on how this could be implemented. For very simplistic tasks (like no exeution guarantees) I wrote a home-grown library (that is nowhere near community standards, but allows me to do my stuff) that kind of tracks execution period and transitions to inactive state when the period expires. Probably, will look into it when other activities are drained.

I would like to implement a generic scheduling interface for v3.0.0, that could be extended with other providers in the future.


⚠ Bofrost.fr has degraded performance

In 7d993fa, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1783 ms

Resolved: Bofrost.fr performance has improved in 5b451d2 after 10 minutes.


⚠ GreatPosterWall has degraded performance

In 12ce0e4, GreatPosterWall ($GPW) experienced degraded performance:

HTTP code: 200
Response time: 1741 ms

Resolved: GreatPosterWall performance has improved in d498ead.


Error while loading a FasterRCNN model in C++

Hello,
did you try as well to load a FasterRCNN model in C++ ? a simple code for models.py would be:
import torch
import torchvision
from torchvision import models
from torchvision.models.detection.faster_rcnn import FastRCNNPredictor
# An instance of your model.
#model = torchvision.models.resnet18()
model = models.detection.fasterrcnn_resnet50_fpn(pretrained=True)
script_model = torch.jit.script(model)
script_model.save("model.pt")

This does save the model.pt. But upon loading the model in C++,
#include <torch/script.h> // One-stop header.
#include <iostream>
#include <memory>
int main(int argc, const char* argv[]) {
  if (argc != 2) {
    std::cerr << "usage: example-app <path-to-exported-script-module>\n";
    return -1;
  }
    torch::jit::script::Module module;
  try {
    // Deserialize the ScriptModule from a file using torch::jit::load().
    module = torch::jit::load(argv[1]);
  }
  catch (const c10::Error& e) {
    std::cerr << "error loading the model\n";
    return -1;
  }
  std::cout << "ok\n";
}

There is an error which says:
 ./example-app ../model.pt
terminate called after throwing an instance of 'torch::jit::ErrorReport'
  what():
Unknown builtin op: torchvision::nms.
Could not find any similar ops to torchvision::nms. This op may not exist or may not be currently supported in TorchScript.
:
  File "/home/ubuntu/Documents/python-virtual-environments/env/lib/python3.8/site-packages/torchvision/ops/boxes.py", line 35
    """
    _assert_has_ops()
    return torch.ops.torchvision.nms(boxes, scores, iou_threshold)
           ~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
Serialized   File "code/__torch__/torchvision/ops/boxes.py", line 148
  _61 = __torch__.torchvision.extension._assert_has_ops
  _62 = _61()
  _63 = ops.torchvision.nms(boxes, scores, iou_threshold)
        ~~~~~~~~~~~~~~~~~~~ <--- HERE
  return _63
'nms' is being compiled since it was called from '_batched_nms_vanilla'
  File "/home/ubuntu/Documents/python-virtual-environments/env/lib/python3.8/site-packages/torchvision/ops/boxes.py", line 102
    for class_id in torch.unique(idxs):
        curr_indices = torch.where(idxs == class_id)[0]
        curr_keep_indices = nms(boxes[curr_indices], scores[curr_indices], iou_threshold)
                            ~~~ <--- HERE
        keep_mask[curr_indices[curr_keep_indices]] = True
    keep_indices = torch.where(keep_mask)[0]
Serialized   File "code/__torch__/torchvision/ops/boxes.py", line 77
    _28 = torch.index(boxes, _27)
    _29 = annotate(List[Optional[Tensor]], [curr_indices])
    curr_keep_indices = __torch__.torchvision.ops.boxes.nms(_28, torch.index(scores, _29), iou_threshold, )
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
    _30 = annotate(List[Optional[Tensor]], [curr_keep_indices])
    _31 = torch.index(curr_indices, _30)
'_batched_nms_vanilla' is being compiled since it was called from 'batched_nms'
Serialized   File "code/__torch__/torchvision/ops/boxes.py", line 35
    idxs: Tensor,
    iou_threshold: float) -> Tensor:
  _9 = __torch__.torchvision.ops.boxes._batched_nms_vanilla
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
  _10 = __torch__.torchvision.ops.boxes._batched_nms_coordinate_trick
  if torch.gt(torch.numel(boxes), 4000):
'batched_nms' is being compiled since it was called from 'RegionProposalNetwork.filter_proposals'
Serialized   File "code/__torch__/torchvision/models/detection/rpn.py", line 72
    _11 = __torch__.torchvision.ops.boxes.clip_boxes_to_image
    _12 = __torch__.torchvision.ops.boxes.remove_small_boxes
    _13 = __torch__.torchvision.ops.boxes.batched_nms
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
    num_images = (torch.size(proposals))[0]
    device = ops.prim.device(proposals)
'RegionProposalNetwork.filter_proposals' is being compiled since it was called from 'RegionProposalNetwork.forward'
  File "/home/ubuntu/Documents/python-virtual-environments/env/lib/python3.8/site-packages/torchvision/models/detection/rpn.py", line 356
        proposals = self.box_coder.decode(pred_bbox_deltas.detach(), anchors)
        proposals = proposals.view(num_images, -1, 4)
        boxes, scores = self.filter_proposals(proposals, objectness, images.image_sizes, num_anchors_per_level)
                        ~~~~~~~~~~~~~~~~~~~~~ <--- HERE

        losses = {}
Serialized   File "code/__torch__/torchvision/models/detection/rpn.py", line 43
    proposals0 = torch.view(proposals, [num_images, -1, 4])
    image_sizes = images.image_sizes
    _8 = (self).filter_proposals(proposals0, objectness0, image_sizes, num_anchors_per_level, )
                                                                       ~~~~~~~~~~~~~~~~~~~~~ <--- HERE
    boxes, scores, = _8
    losses = annotate(Dict[str, Tensor], {})

Aborted (core dumped)

There was a similar error reported here:
https://github.com/pytorch/pytorch/issues/17054
One of the solution was to use the Nightly build. I tried that as well but no difference.
Could you please help me out on this ?

I solved this. Sorry I did not build the torchvision libraries. The example below worked.after linking the torchvision libraries.
https://github.com/pytorch/vision/tree/master/test/tracing/frcnn


Address bar icons lose their color highlights

This is a Bug Report
Description
I posted to this thread  https://community.brave.com/t/brave-portable-version/381456 and bookmarked the page.
Every time I re-post, the address bar icon color highlighting disappears, and I must repeat the bookmark process, even tho the bookmark remains in it's assigned folder.
Is this due to their website configuration or something else.

Version of the add-on: 2-2.0.106
Firefox edition: Firefox (regular edition)
Firefox version:  100
Firefox language: English
Platform/OS: Windows 10
Platform/OS version: 10

Happening here as well https://community.bitwarden.com/t/another-request-for-prompt-to-save-password-clarification/41089
Both websites definitely look and function like they are designed by the same person

and here https://portableapps.com/node/66327#comment-249961.  Btw, works normally for me om github.


.NetStandart Load with proxy problem

In a Task.Run statement when we use HtmlWeb.Load() with IWebProxy and ICredentials parameters Task.Run statement never hits.

Hello @cacheoff ,
Thank you for reporting,
We will look at this issue very soon.
Best Regards,
Jonathan

Hello @cacheoff ,
A new version has been released v1.5.2-beta4,
We removed some code that was calling await for nothing and make sure to add ConfigureAwait(false) everywhere.
Let me know if we fixed deadlock caused by the library.
Best Regards,
Jonathan

Hello Jonathan,
After hours of testing, i have noticed that the bug is not about your code.
"Task.Run" statement never hits even when it includes only a line such as
"var m = new HttpMethod("GET")". Therefore this is not a deadlock. I think
this is a bug of System.Net.Http library in .NETStandart framework.
2017-08-02 22:57 GMT+03:00 Jonathan Magnan<EMAIL_ADDRESS>
Hello @cacheoff https://github.com/cacheoff ,
A new version has been released v1.5.2-beta4,
We removed some code that was calling await for nothing and make sure to
add ConfigureAwait(false) everywhere.
Let me know if we fixed deadlock caused by the library.
Best Regards,
Jonathan
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zzzprojects/html-agility-pack/issues/59#issuecomment-319780857,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFmRlTC6B-diuBHLGAWZJvXqUlClhnVVks5sUNSqgaJpZM4OqUcQ
.

Which version of .NET Standard are you using?
I don't remember the link, but I have heard of such issue in .NET Standard 1.3
Best Regards,
Jonathan

I was using .NetStandard 1.5. But because of that issue I converted my project to .NetFramework 4.6. Now i have no problem anymore. Everything works properly.

Yup, that's why I released a version for .NET Standard 1.6
When I have asked to MysticMind (https://github.com/mysticmind/mysticmind-htmlagilitypack) which is the one that made almost all the code to support .NET Standard, he answering me this:

When I tried with NetStandard 1.3, it resulted in several errors pertaining to System.Net.Http and others. Then I read some material online and found out that NetStandard 1.6 has better support covering more surface area of libraries, it which readily worked with few tweaks/changes.

I still released a version for 1.3 since some people wanted to only use it without loading the HTML directly from the web.
I will close this issue then since you moved back to .NET Framework 4.6 and anyway, I don't think there is something I can really do.
Best Regards,
Jonathan