Docker file for IoTgo Docker image

@humingchun
I am interested in the docke image for IoTgo.
Where can I can the docker file used for building the image?
I have checked on the github folder and its not there
Hear from you soon!
Many thanks!
God blesses!!!
Best regards,
Sanyaade

Hi @sanyaade-g2g-repos
thank you very much for your interest. I wasn't actually using a dockerfile to build the IoTgo image, I was committing a container into an image instead.
But if you prefer a dockerfile, I will try to write one when I have time. It should come before the end of August.

Hi @humingchun,
Ok!
Although I am new to docker and learning how to use it. I can help also if you give me a brief of guideline on steps you have taken. I will try and follow IoTgo build instructions on the github and see if I could achieve the container, image and then create the dockerfile for it.
Hear from you soon!
Many thanks!
God blesses!!!
Best regards,
Sanyaade

Hi @humingchun,
I made a Dockerfile (attached).
I have tested it on my laptop, everything seems a bit working apart from the IoTgo section that is given some problems. The Dockerfile built the image and run but I was not that clear in how I should set the entry-point and the volume drive.
Also I was unable to copy the init.d/iotgo file directly but then I ssh into the container daemon and I did it but it is not committing the changes if I exit and shell back into the container. Please help!
 Many thanks!
Hear from you soon!
God blesses!!!
Best regards,
Sanyaade
#####==============================================================#######
#Experimental IoTgo Docker build file
Pull base image.
FROM ubuntu:14.04
==========================================================================
Packaged dependencies
RUN
apt-get update && \
apt-get install -y build-essential curl git wget
==========================================================================
Install Python.
RUN
apt-get update &&
apt-get install -y python python-dev python-pip python-virtualenv &&
rm -rf /var/lib/apt/lists/*
Define working directory.
WORKDIR /data
Define default command.
CMD ["bash"]
===========================================================================
Install MongoDB.
RUN
apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv 7F0CEB10 &&
echo 'deb http://downloads-distro.mongodb.org/repo/ubuntu-upstart dist 10gen' > /etc/apt/sources.list.d/mongodb.list &&
apt-get update &&
apt-get install -y mongodb-org &&
rm -rf /var/lib/apt/lists/*
Define mountable directories.
VOLUME ["/data/db"]
Define working directory.
WORKDIR /data
Define default command.
CMD ["mongod"]
Expose ports.
- 27017: process
- 28017: http
EXPOSE 27017
EXPOSE 28017
===========================================================================
Install Node.js
RUN
cd /tmp &&
wget http://nodejs.org/dist/node-latest.tar.gz &&
tar xvzf node-latest.tar.gz &&
rm -f node-latest.tar.gz &&
cd node-v* &&
./configure &&
CXX="g++ -Wno-unused-local-typedefs" make &&
CXX="g++ -Wno-unused-local-typedefs" make install &&
cd /tmp &&
rm -rf /tmp/node-v* &&
npm install -g npm &&
printf '\n# Node.js\nexport PATH="node_modules/.bin:$PATH"' >> /root/.bashrc
Define working directory.
WORKDIR /data
Define default command.
CMD ["bash"]
===========================================================================
Next setup Nodejs, Bower and Grunt

Node.js w/ Bower & Grunt Dockerfile

https://github.com/dockerfile/nodejs-bower-grunt

Pull base image.
FROM dockerfile/nodejs
===========================================================================
Install forever, Bower & Grunt
RUN npm install -g forever bower grunt-cli
Define working directory.
WORKDIR /data
Define default command.

VOLUME ["/data"]
Define working directory.
WORKDIR /data
Define default command.
CMD ["bash"]
Expose ports public/private
EXPOSE 80
EXPOSE 27017
EXPOSE 28017

Hi Everybody,
I need help! A locally installed copy of IoTgo om my Laptop gave the errors below. Please advice
Many thanks!
. . .
sanyaade@showcase:/var/www/html/iotgo/localwkdir/IoTgo$ PORT="3000" DEBUG="iotgo" ./bin/www
iotgo Express server listening on port 3000 +0ms
iotgo Connect to DB failed! +28ms
iotgo MongoError: auth fails
at Object.toError (/var/www/html/iotgo/localwkdir/IoTgo/node_modules/mongoose/node_modules/mongodb/lib/mongodb/utils.js:114:11)
at /var/www/html/iotgo/localwkdir/IoTgo/node_modules/mongoose/node_modules/mongodb/lib/mongodb/db.js:1196:31
at /var/www/html/iotgo/localwkdir/IoTgo/node_modules/mongoose/node_modules/mongodb/lib/mongodb/db.js:1905:9
at Server.Base._callHandler (/var/www/html/iotgo/localwkdir/IoTgo/node_modules/mongoose/node_modules/mongodb/lib/mongodb/connection/base.js:453:41)
at /var/www/html/iotgo/localwkdir/IoTgo/node_modules/mongoose/node_modules/mongodb/lib/mongodb/connection/server.js:488:18
at MongoReply.parseBody (/var/www/html/iotgo/localwkdir/IoTgo/node_modules/mongoose/node_modules/mongodb/lib/mongodb/responses/mongo_reply.js:68:5)
at null. (/var/www/html/iotgo/localwkdir/IoTgo/node_modules/mongoose/node_modules/mongodb/lib/mongodb/connection/server.js:446:20)
at emit (events.js:107:17)
at null. (/var/www/html/iotgo/localwkdir/IoTgo/node_modules/mongoose/node_modules/mongodb/lib/mongodb/connection/connection_pool.js:207:13)
at emit (events.js:110:17)
at Socket. (/var/www/html/iotgo/localwkdir/IoTgo/node_modules/mongoose/node_modules/mongodb/lib/mongodb/connection/connection.js:440:22)
at Socket.emit (events.js:107:17)
at readableAddChunk (_stream_readable.js:163:16)
at Socket.Readable.push (_stream_readable.js:126:10)
at TCP.onread (net.js:538:20) +2ms
' ' '

I meet the same error:
iotgo Connect to DB failed! +28ms
mongod 3.2.9
node:4.5.0

iotgo Connect to DB failed! +291ms
any help


fix: add Divyansh Raj

Changes proposed
Added my profile
Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[X] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Kindly revert the package-lock.json file.

Thanks for all the great and helpful suggestions. How can I revert the package-lock.json file, do I have to delete it?

Kindly revert the package-lock.json file.

Thanks for all the great and helpful suggestions. How can I revert the package-lock.json file, do I have to delete it?

Copy the current package-lock.json from the main branch into your branch and then replace and commit the changes.


Set Up Project Structure and Initial Files

Create the project structure, set up the server and client directories, and initialize necessary files.

Done!

Still needs back-end initial files


Multiple Column Sortable

This was posted a while ago, and I believe that it was fixed. https://github.com/esvit/ng-table/issues/160
   var data = [
     { name: 'lincoln',    birth: 1809, death: 1865 },
     { name: 'fdr',        birth: 1882, death: 1945 },
     { name: 'washington', birth: 1732, death: 1799 },
    { name: 'jfk',        birth: 1917, death: 1963 },
    { name: 'darwin',     birth: 1809, death: 1882 },
    { name: 'poe',        birth: 1809, death: 1849 }
   ];

   $scope.table = new ngTableParams({sorting: {name: 'asc'}}, {
       total: data.length,
       getData: function($defer, params) {
       $defer.resolve($filter('orderBy')(data, params.orderBy()));
   }});

    {{row.name}}
    {{row.birth}} - {{row.death}}

This would result in:
Sorting: { "birth,death": "asc" }

A small fix for this would be
if (params.sorting().hasOwnProperty('birth')) {
if (!params.sorting().hasOwnProperty('death')) {
params.sorting()['death'] = 'asc';
}
}


Display download status instead of "Locking..." during package install

Today I was trolled by this "Locking..." message and I find that it's actually a couple hundred developers who don't understand why CPU usage is 0 during install: https://stackoverflow.com/questions/56440090/pipenv-stuck-locking
As it turned out, Pipenv was actually downloading a big file. I think it would be better to display a download status when downloading a big file, so developers won't think pipenv is doing nothing.
Reproduce:
pipenv shell
pipenv install torch

I think it should be the latest release, but I can't check since I already removed it. I was very upset about pipenv not using the python version configured with pyenv.

Okay so I think the problem was I installed pipenv and then pyenv, so pipenv was already there when I installed the 3.10 python. (My global system had 3.11). Does it mean it's impossible to change python version after the first pipenv command was issued?

It does not matter the order of pipenv and pyenv install, but it is true that it will often prefer system python over pyenv or asdf -- unless you specify otherwise.
Once the virtualenv is created for a project space with a version of python you do not prefer, you have to first remove it with pipenv --rm and then re-create it.  Then check that the generated Pipfile doesn't call out the wrong version of python that you want.  If it does you can either remove this section or you could adjust the specifier there to be what you want so that you don't have to specify on the CLI the --python flag.
Pipenv uses pythonfinder to determine which python to use, and by default it will just find the first default one on the PATH I believe, which is typically the system python.   However with pyenv or asdf installed and specifying a version you don't have installed should prompt it to be installed, and this will work normally in one command.
However for pre-release python versions, it does work currently but requires issuing two commands.  For example, if I want to try latest 3.12 and have pyenv installed:
$ pipenv install requests --python 3.12.0b1
Warning: Python 3.12.0b1 was not found on your system...
Would you like us to install CPython 3.12.0b1 with Pyenv? [Y/n]: Y
Installing CPython 3.12.0b1 with C:\Users\matte\.pyenv\pyenv-win\bin\pyenv.BAT (this may take a few minutes)...
Success!
[====] Installing python...:: [Info] ::  Mirror: https://www.python.org/ftp/python
:: [Installing] ::  3.12.0b1 ...
:: [Info] :: completed! 3.12.0b1

There is an edge case right now unfortunately on the pre-release version numbers like my above example 3.12.0b1 because I had to specify 3.12.0b1 to get pyenv to install the pre-release, but then pythonfinder needs just 3.12 or 3.12.0 to find it:

Warning: The Python you just installed is not available on your PATH, apparently.

So re-issuing the command with --python 3.12.0 allows working around this pre-release specifier issue:
matte@LAPTOP-N5VSGIBD MINGW64 ~/Projects/pipenv-triage/issue-5718
$ pipenv install requests --python 3.12.0
Creating a virtualenv for this project...
Pipfile: C:\Users\matte\Projects\pipenv-triage\issue-5718\Pipfile
Using c:/users/matte/.pyenv/pyenv-win/versions/3.12.0b1/python3.exe (3.12.0) to create virtualenv...
[    ] Creating virtual environment...created virtual environment CPython3.12.0.beta.1-64 in 2858ms
  creator CPython3Windows(dest=C:\Users\matte\.virtualenvs\issue-5718-v26YOnrf, clear=False, no_vcs_ignore=False, global=False)
  seeder FromAppData(download=False, pip=bundle, via=copy, app_data_dir=C:\Users\matte\AppData\Local\pypa\virtualenv)
    added seed packages: pip==23.1.2
  activators BashActivator,BatchActivator,FishActivator,NushellActivator,PowerShellActivator,PythonActivator

Successfully created virtual environment!
Virtualenv location: C:\Users\matte\.virtualenvs\issue-5718-v26YOnrf
Creating a Pipfile for this project...
Installing requests...
Resolving requests...
Adding requests to Pipfile's [packages] ...
Installation Succeeded
Pipfile.lock not found, creating...
Locking [packages] dependencies...
Building requirements...
Resolving dependencies...
Success!
Locking [dev-packages] dependencies...
Updated Pipfile.lock (238262fc123a448f5636a8e03c5a139138a6bb719558e6a3867b751ddfee5a6b)!
Installing dependencies from Pipfile.lock (ee5a6b)...
To activate this project's virtualenv, run pipenv shell.
Alternatively, run a command inside the virtualenv with pipenv run.

I hope that helps.
I closed this as not planned currently, for a few reasons that we cannot display the download status currently:
1.) downloads would happen in requirementslib and pip -- requirementslib is under major overhaul right now.
2.) we use multi-processing in some places making it harder to display a progress -- the old pipenv  had a progress bar and it was impacting overall performance of pipenv and so it was removed not only for performance, but because its progress status was very misleading as well.
Maybe one day we can re-consider ways to add progress indicators, but it is not top of mind for the project right now and we generally advice if you want more progress indicator to run with the --verbose flag to see more output.

Can this issue be reopened as a feature request please?
Thousands of developers are confused by this "Locking..." status, when pipenv could display a progress of some sort of downloading anything has been taking more than 10 seconds.


Clarify cookie_samesite None for Sessions plugin

https://konghq.atlassian.net/browse/DOCS-1324
DOCS-1324 Document interdependency for Session Plugin cookie_samesite None and cookie_secure true
Added verbiage recommended by @aaronhmiller to description:
None requires the Secure attribute in latest browser versions.
Direct review link:
https://deploy-preview-2334--kongdocs.netlify.app/hub/kong-inc/session/
Updates:

Make live links:

Just what the "doc"tor ordered! Thank you @tlblessing this bit a few of us recently and it's GREAT to see it documented so well.

Love it. Thanks!

Thank you @aaronhmiller ! @GayleNeumann or @lena-larionova, can I please get a docs approval so I can merge?


chore(docs): add telemetry to overview page

Which issue does this PR address?
Closes #
Acceptance Criteria

[ ] Corresponds to the concept
[ ] Corresponds to the design

Definition of Done

[ ] corresponds to the design principles
[ ] corresponds to the code standards
[ ] passes Continuous Integration checks
[ ] available as feature branch on GitHub

[ ] contains the cleaned up commit history
[ ] commit messages satisfy our commit message guidelines
[ ] a single commit closes the issue via Closes #issuenr

Updating the word makes much sense. I pushed a newer commit based on your proposal.


SyntaxError: only named arguments may follow *expression when use python2

When I use python2, I'll got the error as follow:
$ pyspark
Python 2.7.5 (default, Aug  4 2017, 00:39:18)
// comment：some pyspark output are omitted
Using Python version 2.7.5 (default, Aug  4 2017 00:39:18)
SparkSession available as 'spark'.
>>> import handyspark
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python2.7/site-packages/handyspark/__init__.py", line 1, in <module>
    from handyspark.extensions.evaluation import BinaryClassificationMetrics
  File "/usr/lib/python2.7/site-packages/handyspark/extensions/__init__.py", line 2, in <module>
    from handyspark.extensions.evaluation import BinaryClassificationMetrics
  File "/usr/lib/python2.7/site-packages/handyspark/extensions/evaluation.py", line 3, in <module>
    from handyspark.plot import roc_curve, pr_curve
  File "/usr/lib/python2.7/site-packages/handyspark/plot.py", line 53
    splits = np.linspace(*sdf.agg(F.min(col), F.max(col)).rdd.map(tuple).collect()[0], n + 1)
SyntaxError: only named arguments may follow *expression

Python modules' info:
$ pip list |grep -E "spark"
handyspark (0.2.1a1)
pyspark (2.4.0)

As PEP 3132 says:

Only allow a starred expression as the last item in the exprlist. This would simplify the unpacking code a bit and allow for the starred expression to be assigned an iterator. This behavior was rejected because it would be too surprising.

This error only appear in python2.

Hi, did you solve this? If so, how did you do it? I'm trying to use this package in python 2.7 with spark 2.4.0


fix: update deprecated commands

Fix issue #63
updated
Update deprecated set-output to use the new format for multi-line strings as described in the GitHub documentation:
(https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#multiline-strings)
Also updated actions/checkout to the non-deprecated v3.
@austinpray-mixpanel Needed to update this for multi-line string handling, can you review please, thanks

thanks!


feat(DTFS2-6499): Make amendment button (Portal)

Introduction
Adding links to the start amendment journey in Portal
Acceptance Criteria

Given I am a Maker on the deal details page
When the page load
When the deal status is “Acknowledged” only
Then “Make an amendment” button must be visible under the facilities section (as per the design)
Given I am a Maker on the facility landing page
When I select on the “Make an amendment” button
Then I must be directed to “What do you need to change” page

Resolution

Added "Amend" links to each row in the Bond and Loan tables
Links only appear when:

Deal is acknowledged
User is a maker
Facility has been Issued

Clicking the link takes you to the "What do you need to change page" which is currently blank and will be filled in as part of future tickets.

Other Notes

The Codacy check is failing, but I think the errors are safe to ignore again. They are things like cy not being defined.

Closing for now, to be re-opened once portal amendment work has been re-scheduled.


recent_topics: Display compose box & enable compose hotkeys.

We move compose from being a part of message feed to
being a part of  middle column which is a common parent of recent
topics and message feed. This allows us to use a common compose
box for both the views. Fortunately, compose actions were
independent of this change so there weren't any evident
side effects.
We hide reply button in recent topics view for now. We may
bring it back with the ability to reply to selected topic / stream
after we are confident with bringing compose to recent topics.
We also enable compose hotkeys except for reply.
Fixes #17543

CodeQL is failing the build for unrelated reason.

@showell would you be up for taking point on reviewing this?
(The CodeQL thing is a fixed CI bug).

So excited about this one!

Thanks @Signior-X !

I am gonna focus on UI feedback first.  I think if you hit "New stream message" it should populate the stream and topic fields in the compose box based on the row you have selected in RT.

@showell updated!

Looks like you cannot focus on the last topic row. I am working of fixing it.

Fixed.

I think the main strategic thing we need to figure out with this PR is whether we're OK with using the message feed's bottom_whitespace behavior on recent_topics.  E.g. an alternative vision of the HTML part of this would be to:

Do a preparatory refactoring commit (or mini PR) to change the open compose box HTML to be a handlebars template (which is probably a good idea anyway, since it'd let us share code with message editing).  There's a bunch of code in static/templates/message_edit_form.hbs that this could set us up to eventually deduplicate.  (We could possibly do the same for the closed-compose section too, but I haven't thought about that).
Which might then allow us to have opening the compose box when in "Recent topics" have it shrink the size of the container for "recent topics" rather than using the bottom_whitespace approach (which is UX designed for message feeds where you might want to have new messages come in at the bottom without scrolling the view).

The second strategic question is whether we like the UX of having the "close compose box" state at the bottom of recent topics; it kinda feels like clutter to me right now, and that might get worse with the design change for close compose; not sure.

For your last comment, I will do an independent PR.

Opened #18031 to start planning on the puppeteer tests.

The second strategic question is whether we like the UX of having the "close compose box" state at the bottom of recent topics; it kinda feels like clutter to me right now, and that might get worse with the design change for close compose; not sure.

We will have to wait to get feedback on it.

Now that the compose box is a handlebars template, do you think we have a viable way to render the compose box for "Recent topics" "on top of" the main view (so that it may occlude the last few "recent topics" entries if open) rather than using the bottom_whitespace technique in that view?  If we could do that, I think it'd be a lot more clearly an improvement to merge this.

Since this is a huge change, I did a lot of manual testing and I would encourage you to test tricky cases as well.

This is not ready for review yet. I found a few more issues and I am trying to address them.

This should be ready for review now. I am working on reducing the redundant renders like for pms etc.

Deploying to chat.zulip.org now.


remove multi processing spawn process to avoid conflict with torch.dist.launch

TLDR; we should either torch multiprocessing spawn or torch distributed launch for invoking processes for each gpu
This PR will help improve the following symptoms

During dataloader initialization, the main GPU memory usage spikes. For 40 workers, I saw > 16GB being used for basically no reason. After the fix, GPU:0 take ~300MB which is for some communication related purpose I think and not dependent on num_workers
Training time per iteration is very slow. This can be seen from the screenshot shown below

Testing:

I tested the code by using cached data and feeding the same samples to each GPU. With this I verified that all GPUs use the same memory now (GPU:0 still takes ~300 MB more being the primary GPU)
The dataloader initialization is also very fast now with no spike in memory usage on GPU:0

@dmund95  Hi,
I have met a question that train get freeze at first epoch and memory causes huge cost(maybe leak, not on gpu), and have no clue, until I see this PR.
I think this should be a bug in framework, usually, if set start method with 'spawn', then should not use torch.distributed.launch,
I have changed it and no more freeze and train starts.


Adding wired-up fields to the edit view of Basics tab Customer Info

Description
This makes the fields in the customer info section editable and retrieves values from them from the db. For full effect, make your own service member record to view.
Reviewer Notes
I am pretty certain there are better Reactionary/Reduxy approaches I could have taken here. What are they? Should the country code be in there right now? It isn't in the wireframe but is in Swagger. CONUS and all.
Code Review Verification Steps

[ ] All tests pass.
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in
Querying the Database Safely
have been satisfied.
[ ] Any migrations/schema changes also update the diagram in docs/schema/dp3.sqs
[ ] There are no aXe warnings for UI.
[ ] This works in IE.
[ ] Request review from a member of a different team.
[ ] (TRIAL) Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

Screenshots

Looks better in its current incarnation too!

@kilbergr, do you still have issues with the current address saving? It's working for me, and I'd like to verify that it works elsewhere too.


Add const values for batch header.standard entry class code

#392

Codecov Report

Merging #395 into master will decrease coverage by 0.18%.
The diff coverage is 88.57%.

@@            Coverage Diff             @@
##           master     #395      +/-   ##
==========================================
- Coverage   85.17%   84.99%   -0.19%
==========================================
  Files         107      109       +2
  Lines        6785     6849      +64
==========================================
+ Hits         5779     5821      +42
- Misses        723      734      +11
- Partials      283      294      +11

Impacted Files
Coverage Δ

batchHeader.go
98.29% <ø> (ø)
:arrow_up:

test/ach-pop-write/main.go
84.21% <100%> (ø)
:arrow_up:

test/ach-ppd-write/main.go
82.85% <100%> (ø)
:arrow_up:

cmd/writeACH/main.go
58.33% <100%> (ø)
:arrow_up:

batchCOR.go
95.23% <100%> (ø)
:arrow_up:

test/ach-ccd-write/main.go
87.23% <100%> (ø)
:arrow_up:

test/ach-arc-write/main.go
83.33% <100%> (ø)
:arrow_up:

reader.go
82.96% <100%> (ø)
:arrow_up:

test/ach-adv-write/main.go
89.47% <100%> (ø)
:arrow_up:

batchCIE.go
100% <100%> (ø)
:arrow_up:

... and 47 more

Continue to review full report at Codecov.

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


Approximate nearest neighbors support

As the ANN plugin is out, it would be great to have an index and field for it

I believe https://github.com/opensearch-project/opensearch-py/issues/291 is a related request.

I believe #291 is a related request.

pinging @armenabnousi as they may be interested...
I'm down to create a PR to introduce k-NN APIs into opensearch-py. It would be tremendously helpful to get some community feedback on what features and abstractions people want to see. There's clearly demand for (k-NN) plugin integration cf. #177 #291 etc.
Per @arnavdas88's suggestion, I think it would make sense to incorporate k-NN via client.knn as below...
from opensearchpy import OpenSearch

client = OpenSearch(...)

my_vector = [1, 2, 3, ... 1024]

# approx k-NN
client.knn.approx(index="example", field="vector", vector=my_vector, size=10)

# exact k-NN
query = {
    "query": {
        "match_all": {}
    }
}
client.knn.exact(index="example", body=query, field="vector", vector=my_vector, size=10)

This would be a nice augmentation to existing search functionality for k-NN users. What APIs from k-NN plugin API would you like to see?
Perhaps methods like GET /_plugins/_knn/models/{model_id} could become client.knn.models.get(), .delete(), .train(), and so on...

One can do k-nn in the client today, see https://github.com/opensearch-project/opensearch-py/blob/main/guides/plugins/knn.md, renamed this issue to add a higher level DSL.

A mechanism for plugins was added in https://github.com/opensearch-project/opensearch-py/pull/93. Hopefully someone can pick this one up.

I will work on adding specifications for the knn-plugin and fine-tuning the generator for plugin generation.

I am closing this issue since KNN APIs are now available in opensearch-py. If you feel the issue is not resolved, please feel free to reopen it.

I can see docs at https://github.com/opensearch-project/opensearch-py/blob/main/guides/plugins/knn.md.
But it is low level API.
Are there high level DSL API for k-nn plugin?

Are there high level DSL API for k-nn plugin?

Do you have an example of what we have vs. what you'd like to write? Maybe we should reopen this.

This page explains High level Python API.
But no description on knn vector.
I want to know

how to declare field of knn_vector type.
how to search with knn_vector.

There's no higher level DSL implemented indeed. You should open a feature request for it, but that's basically my question: what would you like to write in Python that doesn't look like https://github.com/opensearch-project/opensearch-py/blob/main/samples/knn/knn_basics.py#L78. Include that in that issue. Note there are some related issues like https://github.com/opensearch-project/opensearch-py/issues/630 too.

If there is a High level API, it would be natural to try that first.
If the High level API does not support some features, it would be kind to describe it in the documentation.
It would be unfortunate to implement a new feature using the High level API, only to find out halfway through that it is not supported and have to re-do it.

I agree @kenmasumitsu. Help us out by contributing?


Get it to work with pm2

I struggle to get this to work with pm2.
I have create a new SSH key, added it as deploy key in my repo where the action runs. Added it as a secret in the repo too.
Thats the output:
[PM2] Spawning PM2 daemon with pm2_home=/home/runner/.pm2
[PM2] PM2 Successfully daemonized
--> Deploying to production environment
--> on host xxxx
fatal: HEAD does not point to a branch
  ○ deploying origin/master
  ○ executing pre-deploy-local
  ○ hook pre-deploy
Host key verification failed.
  ○ fetching updates
  ○ full fetch
Host key verification failed.

  fetch failed

The action's usage can be reviewed here
And my pm2 ecosystem file

I don't know about pm2. Does that SSH into another machine? Does Host key verification failed. happen there?

Yes it does SSH into the machine the app should be deployed on. The error happens there

Then you need to find a way how to provision your known_hosts key there. I am afraid this action cannot do this for you.

Ok, thanks for taking the time. I'll try over at pm2 repo to find some answers.

Ok I figured something out. After disabling strict host checking in my pm2 ecosytem file I got this output:
[PM2] Spawning PM2 daemon with pm2_home=/home/runner/.pm2
[PM2] PM2 Successfully daemonized
--> Deploying to production environment
--> on host MY.SERVER
fatal: HEAD does not point to a branch
  ○ deploying origin/master
  ○ executing pre-deploy-local
  ○ hook pre-deploy
Warning: Permanently added '[MY.SERVER]:36077,[<IP_ADDRESS>]:36077' (ECDSA) to the list of known hosts.

So it seems the problem was that host that I wanted deploy to (MY.SERVER) was not in the list of known_hosts on the GH build instance.
Is that something this action could also take care of? Like it does with adding the private key to the ssh-agent?

My guess would be that there is no benefit of having this in this central action. I've added a note at https://github.com/webfactory/ssh-agent/blob/master/README.md#run-ssh-keyscan-to-add-host-keys-for-additional-hosts.


TAJO-1781: Join condition is still not found when it exists in OR clause

The problem is some hard code in CommonConditionReduceRule.
I've fixed this problem and added more tests.

Thanks. I've fixed comments.

+1
The patch looks good to me, and the examples included in comments look comprehensive. Also, the unit tests are enough to verify this feature.


metrics: change resource group label name

What is changed and how it works?

Issue Number: Close #16191

What's Changed:
change metrics label name for resource-group-name from "name" to "resource_group".

To be backward compatible with old grafana panel, we add a new label name and keep the old one. We are going to deprecate the old label in v8.0.

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

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

Side effects

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

Release note

None

the grafana.json file needs to change also.

the grafana.json file needs to change also.

@bufferflies We want to keep backward compatibility with v7.x so the expression in grafana is not changed. Maybe we can change it in v8.0

/merge

/hold

/unhold

/merge

/run-cherry-picker


Fatigue damage and DEL calculation

Dear Team,
I am using pCrunch to calculate fatigue damage/DEL for tension time series. I am encountering some issues in interpreting the compute_del function in analysis.py.

In DEL calculation line the variable 'elapsed' is used in the denominator which is the total time of the simulation. It is my understanding that this should be the total number of cycles or elapsed time * delFrequency. Could you please clarify?

In the linear Endurance curve creation, based on the  example given for fatpack for generating endurance curves, to generate a curve having a given slope and y-intercept of log-log SN curve of Scin, this has to be modified as :

curve = fatpack.LinearEnduranceCurve(1) #curve.Sc = 1
curve.m = slope
curve.Nc = Scin

This seems to produce the correct curve. Could you please have a look?

Thank you for your questions and use of pCrunch.  In response:

The use of the elapsed time in seconds in the denominator at that line sets the DELs to be 1-Hz equivalent.  We should probably comment this a bit better and even make the desired output frequency equivalent a user-option.
I don't think your use of fatpack there is accurate.  Please see the code documentation here.  For a linear S-N curve, it is only necessary to define one point and the slope of the line in log-log space.  In pCrunch, we use the failure stress such that 1 cycle at that stress amplitude leads to failure.  Thus, Nc=1 and Sc=Sult.  By then passing in the stress-count histogram from the rainflow counting routine, it computes the damage accumulation.  From the suggested code that you have provided, it looks like Sc and Nc are swapped.

Thank you for the quick response!

Okay, an implicit assumption of 1Hz makes sense.

The issue was a difference in pCrunch's SN curve definition( $S=AN^{-1/m}$ ) vs an alternative definition of SN curve ($A = S^{m}N$). But I see that you are correct.

Thank you very much for the clarifications !


Feature/display cover image in maintable

Our team has made some changes to implement the required features, currently here is what we have done so far regarding issue #10120

Created a resizable SplitPane to hold the book cover.
Added functionalities to load images from local computer to the book cover, with the "cover" description being the first priority for JabRef to choose which book cover to choose from if an entry has multiple image files. (A default image is loaded if no images are provided)
Please review and run JabRef to check if the behaviours match what I have described above.

Hey team, I noticed two errors during the build:
Architecture Rule Violation In MainArchitectureTest, the method showBookCover in LibraryTab.java is calling Class.getResource(...), which violates the architecture rule. It’s recommended to use getResourceAsStream(...) instead.
Unused Images in IconsPropertiesTest The existenceOfIconImagesReferencedFromIconsProperties test failed. The test expected only red.png to be present, but it found both default_image.png and red.png. If default_image.png isn’t needed, we should remove it; otherwise, the test logic might need adjusting. Also, there’s a checkstyle violation that caused the build to fail.
These two shouldn't be a big deal. Just give them a quick check before merging :)

If I changed to getResourceAsStream(...), the code couldn't load the local images for some reasons. And yes, we did use default_image.png to load a default image for every entry in case the user doesn't provide any local images.

Besides what Naaeeen talked about, everything else looks good to me. From your explaination the fail of first test seems ok, but for The existenceOfIconImagesReferencedFromIconsProperties test you may want to change the testing logic or the behaviour of the code to pass the test. And maybe you want to double check the code style that has been reported.

It's a great progress on the feature, but there are some checkstyle violations related to the Javadoc comments. Fixing these minor formatting issues should  allow the build to pass successfully.


Default/example values when a model definition is present

We are currently implementing a new set of api's and using the swagger spec. Trying to find a quick way for FE devs to have data be mocked while the services are being built. I have found the auto mocking of default values works fine until there is a ref to a defined model.
Ideally, we should be able to define everything, and utilize default or example (ideally example) so we can quickly use the mock middleware without the need to have "temporary" things in the yaml. It's entirely possible I missed something in the documentation. Still ramping up on the spec and these tools.
Also, I've never got "example" to work, only "default."
Thank you for taking a look at my questions.
For default this works:
/pets/{petName}:
    parameters:
      - $ref: "#/parameters/petName"

    get:
      description: Returns a pet by name
      operationId: findPetByName
      responses:
        default:
          description: Returns the pet data
          schema:
            default:
              name: "Fido"
              type: "dog"
              age: 9
          headers:
            last-modified:
              type: string
              description: The date/time that the pet was last modified

but this does not:
/pets/{petName}:
    parameters:
      - $ref: "#/parameters/petName"

    get:
      description: Returns a pet by name
      operationId: findPetByName
      responses:
        default:
          description: Returns the pet data
          schema:
            $ref: "#/definitions/pet"
            default:
              name: "Fido"
              type: "dog"
              age: 9
          headers:
            last-modified:
              type: string
              description: The date/time that the pet was last modified

I think I discovered my oversight, it appears I need to put the default or example data in the model definition when there is a ref to it. It appears to be working for me now.

Ah, yes.  That's a known bug in Swagger Parser (which Swagger Express Middleware uses under the hood).  It will be fixed in the next release.

Great, thanks for the info!


Rename coreos.firstboot kernel cmdline arg to ignition.firstboot

to make options more distribution neutral.
Update dracut/README.txt to take into account recent changes.
Resolves:  #24
Depended on by: https://github.com/coreos/fedora-coreos-config/pull/30 https://github.com/coreos/coreos-assembler/pull/220

this doesn't resolve #29 I don't think so we should probably remove that from the commit message.

So umm...I know that we don't want to carry a legacy forever but there's so many git repositories and builds involved here.  And the failure case is going to be Ignition not running which is going to be painful.
How about we support both for say a month, and the follow up again and drop the old one?

How about we support both for say a month, and the follow up again and drop the old one?

I originally started to agree with you here, but I think it might be more complicated than I originally thought. What does "support both" mean to you? I think we'd have to do more than just supporting the two kernel args.

What does "support both" mean to you? I think we'd have to do more than just supporting the two kernel args.

Yeah, we'd need to assume firstboot if we see either one, and also delete both files on completion or something?

Yeah, we'd need to assume firstboot if we see either one, and also delete both files on completion or something?

talked with colin.. we're going to coordinate the change (i'll plan to do this on Monday) so that we don't have to support multiple options in multiple places. Thanks colin for discussing that with me and developing a plan together.

Hold this until v3 has been branched?

Hold this until v3 has been branched?

I don't think we need to do that. The only part that needs to wait for v3 is the ignition PR

LGTM. @bgilbert can I get a final review from you?


Critical: Volume Module on the bar can be overamplified  infinitely.

Description
Scrolling on the Volume module can infinitely increase volume percentage.
How to reproduce
Scroll on the volume module indefinitely to increase the volume to a very high percentage.
Expected behavior
If overamplification is enabled, volume should cap at 150% or if it is not enables, it should be capped at 100%
Screenshots

Desktop

Distribution: Arch
Window Manager/Desktop Environment: Hyprland
Pipewire

Additional context
The reason this is critical is because I have had instances where I have damages laptop speakers by over amplifying it to 150%.

Added a new command to adjust volume based on hyprpanel settings. Caps volume to 100% or 150% (if overamplification is enabled).
Please make sure to reset your scroll up/down settings in Configuration > Bar > Volume > Scroll Up/Down to make sure that the new command is used. Simply click the reset icon next to the text field of both of these settings.

This is perfect thanks


Luca-AlexanderThomas.AutoHome-Autostart version 1.0.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hello I dont´t now how i must create an installer for an Packege manager... where i can read how to do that, or can you explain me how in must do it...

Actually, I have not created an installer for any application till now, but I would suggest using Inno or Nullsoft for creating installers that are currently supported by WinGet. Support for portable installers and Dependencies will be coming soon.

@luca-alexander-thomas take a look at MSIX. There are also other open source or "free" tools to help building MSIX packages.


Several processes are performed and each carries a named. Each process has its own ProgressBar

Not the way the progress bars are currently handled.
As a workaround, you can simply write a text line with a name before the progress bar, like:
context.WriteLine("Total:");
var totalBar = context.WriteProgressBar();
context.WriteLine("Persistent:");
var persistentBar = context.WriteProgressBar();

That's exactly what I did. Results on the screen: first the title, and below progressBars ))))))

Just added support for named progress bars in 1.3.6. You can test it as soon as it gets indexed on NuGet.


git has an undeclared runtime dependency on ssh

Running git clone with my still-quite-new nix environment, I get:
error: cannot run ssh: No such file or directory

Installing openssh fixes the problem for me, of course. But this means that git really ought to declare a dependency on openssh, no?

Is there a recommended solution for this now?

The recommended solution is to patch SSH, IMHO. It's not hard, as @7c6f434c pointed out.

Triage: this issue does still exist on 16.03-beta:
$ nix-shell --pure -p git --command "git clone git@github.com:NixOS/nixpkgs.git"
Cloning into 'nixpkgs'...
error: cannot run ssh: No such file or directory
fatal: unable to fork

Patched in #14207

The solution to this issue is now causing a different issue on Fedora: https://github.com/NixOS/nixpkgs/issues/160527
I'm not sure if we should really hard-code a SSH binary or if we should e.g. just reference a SSH binary as a fallback.
Opinions welcome :)

I'm not sure if we should really hard-code a SSH binary or if we should e.g. just reference a SSH binary as a fallback.

FYI: https://github.com/NixOS/nixpkgs/pull/166334 is about to revert this and I currently believe that this is a good idea.


Strip spdlog dependency

Other than the horrible state of logger.{cpp,hpp}, we use but a smidge of spdlog's abilities, so we should remove this dependency and roll our own.

Closed via aa3805ef0a488a1176b4e0f17d94d5d405d3e749.


Question about Bts*

Additionally, we use an asterisk sign ‘∗’ to denote an oracle version Bts∗, where we use the ground truth depth map to
factor out the global scale.

Is it means that the mono depth network's prediction may not have the same scale as the depth map and Bts* is to estimate some parameter (e.g. mean and std in (depth-mean)/std) to adjust its scale according to the gt depth map?

Hi @07hyx06 , right, mono depth estimators such as Bts do not have accurate scale estimation. In Bts*, we use median scaling to correct the scale factor before evaluating depth error.


Set a default for trackClicks

Is it possible to set a global default for trackClicks? I'd like it to be FLASE but it looks like it's hard coded to be a default of TRUE. I know I can change this on a per-message basis, I was just wondering if there was a way to do it globally.

Nevermind, I figure I can just extend the Message class with my own class and set it in the constructor.
Thanks!

Hey David, it is now "null" so account preferences have precedence. Best,
Sven

@slot
thanks!


chore: gracefully shutdown when error encountered

Tested by throwing RuntimeException from udsink. Verified both numa and udsink containers kept getting restarted. Also tested a happy path where the simple log continues logging with no issue.

I was under the impression that shutting down the server would also shut down the event loops.
The changes look good to me.
One suggestion: We shouldn't expose the event loop-related details to all the components. The event loops are only used by the io.grpc.Server. Instead of using GrpcServerHelper, could we write a wrapper for io.grpc.Server that exposes the necessary methods to create, start, gracefully stop, and force stop the server? This way, it can be abstracted out, and inside the wrapper, you can also shut down the event loops.


Add 'Shrink Me' link to the README.md file

I added the web tool 'Shrink Me' as an additional way to use SVGO on the web.
Shrink Me:
https://shrinkme.app/

It's not obvious "shrink me" is using svgo. It's not open source right? Let me know if I'm wrong.


Turn off streaming replay

It seems that streaming API replay is enabled by default. Is there a way to disable this?

I am actually trying to make use of replay, but I haven't been able to confirm or deny that it is working.  If it does work, is there something I need to add to my code to automatically process missed messages at the time the application comes back online?

I am also looking for a way to reply events in JS and I can't seem to fins it anywhere...

I could not find anything in the docs myself, although issue #814 certainly alludes that it is possible to control replay in the streaming client.
@mattdrees You seem to know something about this? Can you confirm if replay works, can it be configured?


Knapsack01 problem code in C++

First thanks for taking time to send this pull request. At this time I do not accept C++ solution. I do not want to confuse the code based with mostly java solutions and C++ sprinkled in b/w here and there. Plus i dunno what Knapsack01.cbp does(probably another build system) and would really clutter code base with a.out files. If you wish to contribute and help improve my codebase pls contribute solutions in Java and I would be glad to accept it.

The program is in main.cpp and a.out is result of compilation.Knapsack01.cbp is acutally auto generated bu codeblock editor.Yeah I will send pull request if I would do something in java.

Thanks. Appreciate your help.


Don't blow up if we don't have data to look for sidebar analyses in.

Thanks for the check, sorry for the trouble!


Upgrade base RocksDB version to 7.7.8

we're currently on 7.7.3 and there are only Bug fixes so no reason not to upgrade

@udi-speedb  , im having trouble merging commit bbf61996a94d3281a120cef2892d9126b9083a45 which conflicts which changes you introduced in https://github.com/speedb-io/speedb/issues/132 . can you cherry-pick the commit and solve the conflicts on the 357 branch?

@erez-speedb  plz check. there should be no degradation

No degradation, small DB even shows some improvement.
Small DB
https://admin.speedb.io/performance?items=OReqh8hUYTtHjg2ev7j8&items=MTnDOa9oyhGa33CyQgci&colors=%23F06292&colors=%2300796b
Small obj
https://admin.speedb.io/performance?items=q70AYGrgS0T1oi8MsaZF&items=zmqpQ9ogatg10p1lqXkC&colors=%23F06292&colors=%2300796b
Large obj
https://admin.speedb.io/performance?items=lT4eb2RvOIOgkQtMA4eN&items=DQbr0ntUjYJDz3RLExzO&colors=%23F06292&colors=%2300796b

@Yuval-Ariel if this passed QA pls move it to merge


CM-701

Implemented CM-701. Added separate error messages via InvalidUsageError for each field that could be wrong (email and password). Reiterated password requirements when password is wrong, reiterated their provided email when email is wrong.
Tested locally on my machine by submitting bogus emails/passwords and checking that the error messages are returned.

Nice work! Looks great.


Flickering of image when calling setImage()

Issue Description
What
I have a UIImageView and set the image through kf.setImage().
It works fine, except that the ImageView is flickering once each time I call kf.setImage() again on the same Image View, even if the image is exactly the same and loaded from the Cache.
Is there a way to turn this off or avoid re-setting the image?
Thanks!
Reproduce
[The steps to reproduce this issue. What is the url you were trying to load, where did you put your code, etc.]

This is due to the image is already released from memory cache and will be loaded again from disk cache, with a nil placeholder. You could try to set images with .keepCurrentImageWhileLoading to see whether it could work:
imageView.kf.setImage(with: url, options: [.keepCurrentImageWhileLoading])

Hi @onevcat, thanks for the answer.
This fixes the "flickering", but unfortunately messes up my TableView. I have this ImageView inside a TableViewCell and set the image in cellForRow. After scrolling cells have images set that don't belong to them....

Hi @onevcat ,
I am also seeing this even with keepCurrentImageWhileLoading.
I think my issue is with the transition. Since even with that flags it tries to set to nil the placeholder. Or am I using it wrong there?
Thanks,
H.

Nevermind, @onevcat. I was not using that flag on my second call.

@hernangonzalez  👍
@renshaw1
BTW, if you are using it in a reusable cell in table view or collection, maybe the flickering comes from the cell reusing. You could also try:
cell.imageView.kf.setImage(with: url, placeholder: cell.imageView.image)

to use the current image as your loading placeholder to prevent it.

.keepCurrentImageWhileLoading seems to work.
In my situation, I got a blink because I called collectionView.reloadSections(indexSet:) on a section that contains a cell that displays an image. The problem with that is reloadSections forces the collectionView to create new cells (as in new pointers) for these sections. Since it's displaying a new cell, the imageView starts off as its backgroundColor, then fades in the image.

@hellaandrew hi buddy, did u have any solution? Ofcourse we need reload section for avoid reload other cell

If the problem happens due to lack of memory cache, another solution worths to try is adding a [.loadDiskFileSynchronously option](https://kingfisher.onevcat.com/kingfisheroptionsinfoitem/#kingfisheroptionsinfoitem.loaddiskfilesynchronously. It turns the async loading from disk to sync behavior and prevents the possible flickering.

I've also had this problem. In my case, it came because extra call DispatchQueue.main.async {} despite it being called in the main thread.


Falt Portals dont seem to work

I see in the latest commit that flat portals are possible now but using this i cant seem to activate the portal with a 5x5 obsidian portal with a flint and steel.
CustomPortalBuilder.beginPortal() 		    .frameBlock(Blocks.OBSIDIAN) 		    .destDimID(new Identifier("the_nether")) 		    .tintColor(131,66,184) 		    .flatPortal() 		    .registerPortal();

Can you try a different block? You also probably have an error in the log about registering a portal that already exists.
When registering a portal it checks for conflicting portals, but doesn't check if they are of different types. It's most likely detecting this as a conflict

I see, I'll try it with a different block. It'd be nice if it detects different types of portals for the same block types.

It works with other blocks. Thanks for the help : ) But yeah the above thing would be nice as a feature if you're looking for enhancement suggestions.

I did it to help avoid confusion for the player. They may make a flat version of a portal and expect it to go to the same place. Not sure if it's really a big deal but that was my thinking

Ah fair, well i'll close it since its working as intended.


Order of node features

I am trying to determine how removing certain node features changes the predictive performance. Are the 70 node features in the  array in the same order in which they are presented in Appendix A.1 (e.g. first 20 being PSSM features)?

Yes, the order should match, so the first 20 are the PSSM features. Let me know if something looks awry!


Adding frequentist numerical tests + Integrate StatsAPI

This PR adds frequentist numerical tests, and also integrates StatsAPI into CRRao.

@sourish-cmi
Currently this PR tests frequentist models on ordinary formulae and 0 intercept formulae. The tests just check whether the coeftables match.

Should we test on other formulae/datasets?
Should we test on more (or all) functions in src/frequentist/getter.jl?

We should test for  predict
Some possible formulas:
i) mpg~hp+hp^2 + wt + wt*hp
ii) log(mpg)~log(hp)+log(wt)
Later we can add few more datasets

We should test for  predict

Some possible formulas:
i) mpg~hp+hp^2 + wt + wt*hp
ii) log(mpg)~log(hp)+log(wt)

Later we can add few more datasets

I added predict and some formulae.
For Logistic, Negative binomial and Poisson should there be an example with log on the left?

We should test for  predict

Some possible formulas:
i) mpg~hp+hp^2 + wt + wt*hp
ii) log(mpg)~log(hp)+log(wt)
Later we can add few more datasets

I added predict and some formulae.
For Logistic, Negative binomial and Poisson should there be an example with log on the left?

No @ShouvikGhosh2048  for Logistic, Negative binomial and Poisson, we don't need a log on the left.

Perhaps we should check for AIC and BIC
> attach(mtcars)
> fit = lm(mpg ~ hp + wt + gear,mtcars)
> AIC(fit)
[1] 157.0528
> BIC(fit)
[1] 164.3815

Add it in the code and check if CRRao yields AIC = 157.0528 and BIC=164.3815

@sourish-cmi
using RDatasets, GLM, StatsBase

mtcars = dataset("datasets", "mtcars")
glm_model = lm(@formula(MPG ~ HP + HP^2 + WT + WT * HP), mtcars)
print(aic(glm_model))

gives<PHONE_NUMBER>8935536
and
attach(mtcars)
fit = lm(mpg ~ hp + hp^2 + wt + wt * hp, mtcars)
AIC(fit)

gives 145.6109.

In your R-code, there is a syntax mistake.

fit = lm(mpg ~ hp + hp^2 + wt + wt * hp, mtcars)
AIC(fit)

The correct formula would be as follows:
> fit = lm(mpg ~ hp + I(hp^2) + wt + wt * hp, mtcars)
> AIC(fit)
[1] 145.923

library("Zelig")
data(turnout)
fit = glm(vote ~ age + I(age^2) + race + income * educate, family = binomial(link = "cloglog"), data = turnout)
sprintf("%.15f", AIC(fit))

gives 2022.884413390652753,
while
using GLM, RDatasets
turnout = dataset("Zelig", "turnout")
model = glm(@formula(Vote ~ Age + Age^2 + Race + Income * Educate), turnout, Binomial(), CloglogLink())
GLM.aic(model)

gives 2022.8844886368.

library("Zelig")
library(MASS)
data(sanction)
fit = glm.nb(num ~ target + coop + ncost, data = sanction)
sprintf("%.15f", AIC(fit))

gives<PHONE_NUMBER>11806205,
while
using GLM, RDatasets
sanction = dataset("Zelig", "sanction")
model = glm(@formula(Num ~ Target + Coop + NCost), sanction, NegativeBinomial(), LogLink())
GLM.aic(model)

gives<PHONE_NUMBER>654269.

library("Zelig")
data(turnout)
fit = glm(vote ~ age + I(age^2) + race + income * educate, family = binomial(link = "cloglog"), data = turnout)
sprintf("%.15f", AIC(fit))

gives 2022.884413390652753, while
using GLM, RDatasets
turnout = dataset("Zelig", "turnout")
model = glm(@formula(Vote ~ Age + Age^2 + Race + Income * Educate), turnout, Binomial(), CloglogLink())
GLM.aic(model)

gives 2022.8844886368.
library("Zelig")
library(MASS)
data(sanction)
fit = glm.nb(num ~ target + coop + ncost, data = sanction)
sprintf("%.15f", AIC(fit))

gives<PHONE_NUMBER>11806205, while
using GLM, RDatasets
sanction = dataset("Zelig", "sanction")
model = glm(@formula(Num ~ Target + Coop + NCost), sanction, NegativeBinomial(), LogLink())
GLM.aic(model)

gives<PHONE_NUMBER>654269.

Okay, it is an issue with GLM.jl. Can you please raise an issue in GLM.jl?

@mousum-github can you please check this?

glm.nb estimates the \theta  but glm with a Negative Binomial, you have to give theta and the default value is 0.5.
negbin is the corresponding function where \theta is estimated from the data.

@mousum-github

For the Julia program in (2), what will the corresponding R program be?
Do the programs in (1) match in what they should compute? If so, is the difference significant?

@mousum-github

For the Julia program in (2), what will the corresponding R program be?
Not sure, you may try with negative binomial distribution as a family parameter in R glm function
Do the programs in (1) match in what they should compute? If so, is the AIC difference significant?
model = glm(@formula(Vote ~ Age + Age^2 + Race + Income * Educate), turnout, Binomial(), CloglogLink(); rtol=1.0E-16)
GLM.aic(model)
2022.8844131094252

@ShouvikGhosh2048 for now, we can remove this example with cloglog.  We can say on the front page that the test for cloglog link function is not completed yet. We have to move ahead with other tests.  Particularly with the Bayesian test.
In this issue can you mention the completeness of the tests? or the cases that I have not considered yet !!

@ShouvikGhosh2048 for now, we can remove this example with cloglog. We can say on the front page that the test for cloglog link function is not completed yet. We have to move ahead with other tests. Particularly with the Bayesian test.
In this issue can you mention the completeness of the tests? or the cases that I have not considered yet !!

I couldn't construct a R program for NegativeBinomial, so those tests are still incomplete.
I can start working on Bayesian tests parallelly.

@ShouvikGhosh2048 @mousum-github
As discussed in yesterday's meeting we will focus on functional testing.

@ShouvikGhosh2048  will develop the functional testing for the Bayesian module. For a given seed, the Bayesian module should reproduce the result.

@mousum-github will correct the negative binomial model with negbin instead of glm and provide the test with R output

@ShouvikGhosh2048 Mousum da added the negative binomial test, and I approved and merged the test. Perhaps you pull the latest from the main branch. Please check.

@ShouvikGhosh2048 since you created separate PR for Bayesian tests. I believe now we can merge this PR.  Please confirm.

I've updated the numerical tests for negative binomial. I haven't matched the AIC/BIC values with R - I'm not sure which values to pass to R's negbin to match Julia's negbin.
I tried:
library(aod)
library(Zelig)
data(sanction)
fit = negbin(num ~ 0 + target + coop + ncost, ~1, sanction)
AIC(fit)

This gives 344.896 compared to CRRao's<PHONE_NUMBER>7518536.
Otherwise the PR is ready to be merged.

We can merge it for now (to make sure our frequentist models are consistent), but we should keep track of the difference in Negative Binomial models.

I have filed the issue  with the title AIC value for negbin regression does not matches with R. Otherwise, this PR is fine - so I am merging the PR with the main.


znc keeps using old system dns resolvers even after theyre changed?

is that how znc works? because i was having trouble all day with my dns messing up so znc was unable to connect to anything. and then when i finally fixed the system dns, znc continued to not work until i restarted it. so it seemed like znc continued to use the old dns servers instead of getting the new system ones. not sure if this always happens or if it was a special case
<*status> ZNC 1.6.3 - http://znc.in
<*status> IPv6: yes, SSL: yes, DNS: threads, charset: no
CentOS 6.7

Try restarting znc

It just uses getaddrinfo() to resolve DNS, as the majority of other software...

it didnt seem to pick up any of my dns changes until i restarted it. only then did it start resolving hosts again. idk

well if it does what every other software does maybe something else caused it somehow. now i know i can restart it if it happens again. so you can close this if you think theres no problem with znc

had this happen again on another server. i set up unbound to manage my dns requests and every application was able to start  resolving hosts using the new resolvers but znc wouldnt. it needed to be restarted

I think /etc/resolv.conf had some option for telling everything to always read it which is said to be a bad idea and searx says that you should service network restart.

kk ill try that next time


Update for Chef 14

Update Gems.
Drop deprecated foodcritic.
Fix up cookstyle issues.
Update testing for Chef 14.
Drop unused windows cookbook.

Closes: https://github.com/windowschefcookbooks/seven_zip/issues/20
Signed-off-by: Ben Kochie<EMAIL_ADDRESS>

Ok, I think I have everything working correctly, but there's something I don't understand up with the kitchen test.

Hey @SuperQ Thanks for the PR this has now been superseded by #38


Multiple JsonSerializerSettings are sharing same CamelCasePropertyNamesContractResolver cache

Source types
var data = new Dictionary<string, object>(){
    { "camelCase", "value"},
    { "PascalCase", "Value" },
    { "snake_case", "v_lue" },
    {"YELLING", "VALUE"}
};

destination JSON
{
  "camelCase": "value",
  "pascalCase": "Value",
  "snake_case": "v_lue",
  "yelling": "VALUE"
}
// or
 {
  "camelCase": "value",
  "PascalCase": "Value",
  "snake_case": "v_lue",
  "YELLING": "VALUE"
}

Expected behavior
Contract of type is isolated between two different JsonSerializerSettings
Actual behavior
Contract of type is being shared (with a static cache) across JsonSerializerSettings and serialization
Steps to reproduce
using System.Collections;
using System.Reflection;
using Newtonsoft.Json;
using Newtonsoft.Json.Serialization;

var data = new Dictionary<string, object>(){
    { "camelCase", "value"},
    { "PascalCase", "Value" },
    { "snake_case", "v_lue" },
    {"YELLING", "VALUE"}
};

var camelSettings = new JsonSerializerSettings()
{
    Formatting = Formatting.Indented,
    ContractResolver = new CamelCasePropertyNamesContractResolver()
        { NamingStrategy = new CamelCaseNamingStrategy() { ProcessDictionaryKeys = true } }
};
var unprocessedSettings = new JsonSerializerSettings()
{
    Formatting = Formatting.Indented,
    ContractResolver = new CamelCasePropertyNamesContractResolver()
        { NamingStrategy = new CamelCaseNamingStrategy() { ProcessDictionaryKeys = false } }
};

Console.WriteLine($"Camel Settings: {JsonConvert.SerializeObject(data, camelSettings)}\n");
Console.WriteLine($"Unprocessed Settings: {JsonConvert.SerializeObject(data, unprocessedSettings)}");

((IDictionary)typeof(CamelCasePropertyNamesContractResolver).GetField("_contractCache", BindingFlags.Static | BindingFlags.NonPublic)?.GetValue(null)!).Clear();
Console.WriteLine("\n-- Forced Clearing of CamelCasePropertyNamesContractResolver internal cache --\n");

Console.WriteLine($"Unprocessed Settings: {JsonConvert.SerializeObject(data, unprocessedSettings)}\n");
Console.WriteLine($"Camel Settings: {JsonConvert.SerializeObject(data, camelSettings)}");

Camel Settings: {
  "camelCase": "value",
  "pascalCase": "Value",
  "snake_case": "v_lue",
  "yelling": "VALUE"
}

Unprocessed Settings: {
  "camelCase": "value",
  "pascalCase": "Value",
  "snake_case": "v_lue",
  "yelling": "VALUE"
}

-- Forced Clearing of CamelCasePropertyNamesContractResolver internal cache --

Unprocessed Settings: {
  "camelCase": "value",
  "PascalCase": "Value",
  "snake_case": "v_lue",
  "YELLING": "VALUE"
}

Camel Settings: {
  "camelCase": "value",
  "PascalCase": "Value",
  "snake_case": "v_lue",
  "YELLING": "VALUE"
}

We have an application that interact with multiple different HTTP APIs, each having their own set of serialization rules, and depending on which endpoint is used first (who ever serializes data for a request) ends up causing invalid data for the others due to a shared cache of contracts.

Digging into the issues here more closely, it seems this is expected behavior do to worry of performance degradation of legacy/existing applications..
Personally, I would suggest the CamelCasePropertyNamesContractResolver be marked with a deprecated message for the implementer to manage the lifetime of DefaultContractResolver instead or at the very least in its documentation describe its shared caching behavior.


Element not return smart xpath i need smart xpath

Hi @JonathanMagnan
i am explain with example :
i have open Google.com all html load load in dom
HtmlAgilityPack.HtmlDocument doc = new HtmlAgilityPack.HtmlDocument();
doc.LoadHtml(HTMLDoc);
var p = doc.DocumentNode.SelectSingleNode("//*[@id='hplogo']");
var xx = p.XPath;

Here , xx  variable has xpath :  /html/body/div/div[7]/span/center/div[1]/img
But i need //*[@id='hplogo'] smart xpath
There are chromium driver native using for smart xpath logic
https://github.com/chromium/chromium/blob/77578ccb4082ae20a9326d9e673225f1189ebb63/third_party/blink/renderer/devtools/front_end/elements/DOMPath.js#L242
so , i think you understand this issue about generate smart xpath

Hello @rajesh-smartwebtech ,
Thank you for reporting.
We will look at what the smart xpath is
Best Regards,
Jonathan

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

Hi @JonathanMagnan
Thank you
is it possible in Performance Libraries ??

Hello @rajesh-smartwebtech ,
I have checked your request but I believe we already support your "smart xpath": https://dotnetfiddle.net/cUEupp
You can already use xpath expression when searching.
Is it what you were looking for?

Hi , @JonathanMagnan
I have checked this it working well
but one element i get xpath that time not return smart xpath
"smart xpath" with find element is working best !
Thank you

I'm not sure if everything now works or you need something else?


Old cgmartini.nl forum posts?

Hi @danielpastor97 & @Lp0lp  - just thought of something: did we lose (access to) all the posts in the old cgmartini.nl forum? I suppose it would (have) be(en) good to salvage some of that material... maybe give it to ChatGPT or your favorite LLM to make a summary of that and add it to the FAQs or something.

Hey @ricalessandri . We considered finding a way of somehow archiving the forum. In reality we could not find a way (to be fair I did not dig too much into it) to do so. We did build a FAQ and I tried to update it somewhat based on the posts I saw on the forums but obviously did not dig through it all.

Gotcha. Are the posts still available somewhere? Perhaps I could help crystallize out some more FAQs.

Forum posts should still be available on the old server machine in Groningen. I think we should keep an archive copy of it somewhere. Perhaps Alex can assist us with this. @AlexHdeVriesMDGroup ?

Hello Alex! That sounds good to me. While I'm not very familiar with .sql files, I understand that they should be sufficient for extracting the information we need. Also, (from a quick Google search,) we might be able to convert the data to JSON format if needed, which could make it easier to work with.

Hello Riccardo, Daniel, and others,
Yesterday, I sat down with Bart and had a look at the .sql file. It turned out to be non-encrypted, so it can be opened in vi or another editor and you see the text. I do not know much about databases (and SQL), but learned a little bit yesterday. The .sql file itself, while readable is not a convenient format because it is full of SQL keywords and not nicely formatted.
We attempted to convert it to a .db format that can be browsed by nautilus. However, this ran into errors - the file was not recognized because it is not sqlite3, but MySQL. We then tried to install stuff to translate MySQL to sqlite3 but failed. We did manage to start up an interface using mysql (note that we used a copy on our local machines, not the webserver.). We could interact with it using mysql syntax (for example SHOW TABLES, which gives an overview of all content categorized, and SELECT * FROM j25_kunena_user_topics which contains a number of Forum discussions). I do not particularly like this interface, but there may be others. We did see there is a pandas reader for SQL but it did not work out of the box (probably need to install extra stuff) and did not have/take time to further pursue it.
Siewert would like to have recent Forum discussions (maybe from the past year) incorporated, but I think quite a few topics are already in the FAQ, so personally would not give it that much of priority if it turns out to be quite cumbersome.
Of course you can see what an LLM can do, but I would first clean up the file because it contains the entire website, including user names, encrypted passwords - these need to be handled with care (at least under European Law), tutorials and all other information. Maybe an LLM is smart enough; this is another area I know next to nothing about.
The full .sql file is about 7 Mb in size, so easy to transport over to any of you guys.
Many greetings, Alex

Hi @AlexHdeVriesMDGroup , thanks for the detailed update and for taking the time to investigate the .sql file. Feel free to email the .sql file to Daniel, Luis, and me when convenient, and we’ll take it from there. No rush, as I won’t have time to look into it until mid-October, but I plan to do so then.


Add selenium related requirements to requirements.txt

Notably, webdriver-manager and selenium-stealth seem to be required for genesis to perform selenium searches (when looking up APIs, etc.). Adding these to requirements.txt eliminates the errors encountered when using the genesis jupyter notebook.

Hey @sigmaeni, these are only used for the browsing agent, so I don't want the whole project to depend on them. Maybe we can add some checks and more descriptive error messages when initializing the browsing agent. The jupyter notebook has pip install on top


Add ConfigureAwait(false) calls to awaited tasks

References:

ConfigureAwait FAQ - .NET Blog.
C# - Iterating with Async Enumerables in C# 8 | Microsoft Learn

Tested in CTS & ENFO:

Action\Implementation
File System
In Memory
Azure Blob

CTS: Save email text file
✓
❔
✓

CTS: Export DB
✓
✓
✓

CTS: Delete old DB export
✓
❔
✓

CTS: Upload non-image attachment
✓
✓
✓

CTS: Delete non-image attachment
✓
✓
✓

CTS: Upload image attachment
✓
✓
✓

CTS: Delete Image attachment
✓
✓
✓

ENFO: Upload Orders
✓
✓
✓

ENFO: Delete Orders
✓
✓
✓

❔: These files are not read in the application so deletion could not be tested.


ScrollBar of CONTACT form

Is your feature request related to a problem? Please describe.
Hello @sah
I went through the website and found that the scroll bar of CONTACT US form is not properly styled. To make it look more attractive it requires a little fix in its UI.
Describe the solution you'd like
Current styling-

There is this resize thing which is not looking good, so we can remove it.
Describe alternatives you've considered
After Improvement -

After styling we can make look more attractive and bug free.
Kindly assign me this issue under GSSOC'23
Thankyou.
Additional context
No response
Code of Conduct

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

Hey @fantasticbuddy
I think its okay to have the resize thing, I think @SahityaRoy  can help you with what exactly to do

Actually the thing is
The purpose what it serves will remain even after it is removed. So I think it's better to remove it on the part of styling.
Yes let's wait for @SahityaRoy


Option to send name of thread to graylog

...needed that for one of our projects. Not sure whether it's useful for anybody else.
Implemented like the "useLoggerName" option...

Looks great! I bet a lot of people will need the Thread Name. Thanks so much for contributing.


chore: rename to vue-simpleicons

[x] I've read the Contributing Guidelines and the Code of Conduct

Description

Rename to vue-simpleicons due to #22.
Why is this change required?
Fixes #22

@ericcornelissen I think component name is good, we don't need to rename it

Awaiting @gabrieledarrigo's review


Replace the manual skewing code with the BALANCED sharding strategy

The BALANCED sharding strategy ensures balance among many streams on the
server-side, obviating the need for the client to mimic such behavior. Note that
I'm using an unknown field to force the BALANCED sharding strategy. We are in
the process of releasing new Java libraries. Once the new libraries are out, I
will send another pull request to use the generated code for setting the
sharding strategy.

@pmkc please take a look when you get a chance!


Wrapping component codesandbox is missing

Got this but it seems it's been deleted:

https://codesandbox.io/s/react-menu-wrapping-q0b59To

Nvm, for some reason my browser was thinking the "To" from the next line was part of the link
Actual href is working fine https://codesandbox.io/s/react-menu-wrapping-q0b59


Fix #1081. Refocus prompt when escape from search component.

Search is focused and we need to give the focus back to prompt.

Good job, @rebornix, thanks!

I just tested it, and prompt does get focus, but typing doesn't work for some reason. Is it the same for you?


Documentation translation to other languages

First of all, thanks everyone for the amazing job on the Cardano docs!
I could not find any translation branches/issues/pull-requests or the structure to help on this matter. Since the documentation is open to anyone to contribute, I decided to leave here some question on this regard:

Is the technologies published in the docs mature enough for translations?
Is there already a contribution process defined?
If they are not, is it planed only for the 'Basho' phase of the Cardano roadmap?
What is still needed for the translation process take place?

Hi @alcemirsantos thank you kindly for your feedback. This new site is better placed to handle auto-translation. I will contact our translation team and investigate if there are any translation plans and let you know! thanks.


bug(layerdb): race to create broadcast channels

We were using the wrong kind of lock a (RwLock rather than a Mutex) around the creation of broadcast channels in the activity multiplexer. When we went to create a new subscription, we would acquire a Read lock when checking for the existence of the broadcast channel, and if it didn't exist, continue on to create a new one by acquiring a Write lock later in the function.
This, of course, was racy - multiple consumers could acquire a failing Read lock condition, and then pass through to the critical section where we create a new broadcast channel - the result being multiple consumers of the underlying NATS jetstream would be created, only one of which was actually sending the messages we wanted.
I believe this is the source of the occasional deadline misses in rebase finished requests - at the very least, this code was easy to trigger with the test coverage in this PR, which leads me to believe it was also the source of our other issues.
Let's see how it goes. :)

bors merge


Any new releases??

It's been awhile since this page has been updated

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

ID: 53c02fee-38c5-649a-0faa-23bf5edbcd7f
Version Independent ID: 65ee90be-a0f2-15c0-b487-7210eddda17a
Content: Release notes - Power Automate
Content Source: articles/release-notes.md
Service: flow
GitHub Login: @stepsic-microsoft-com
Microsoft Alias: stepsic

I agree, where is the new content?

@cconrado Thanks for reporting this issue. It's now fixed. Find release plans here: https://docs.microsoft.com/dynamics365/release-plans/)

Thanks, but that does not fix the problem on the referenced Release Notes page, it would be helpful if there was a cross reference link added to this page.

@MSFTMan agree with @DeanGross


Support ActiveSupport::Cache::CacheStore#write_multi

It would be great if this library supported Rails' write_multi method.

We're not recommending :dalli_store anymore, rather use Rails's official :mem_cache_store.


checkmake --version . . . doesn't have a version

Expected behaviour
checkmake v1.0 built at 2017/12/17 by Joshua Nelson with Go v<something here>
Actual behaviour
checkmake  built at  by  with
Output of checkmake --version
checkmake  built at  by  with

Closing for inactivity. Please reopen if the problem persists.

@mrtazz, this still is happening with the following steps (taken from README.md):
$ uname -a
Linux f3929e4856f6 5.4.39-linuxkit #1 SMP Fri May 8 23:03:06 UTC 2020 x86_64 GNU/Linux

$ go version
go version go1.15.6 linux/amd64

$ go get github.com/mrtazz/checkmake && \
>     cd $GOPATH/src/github.com/mrtazz/checkmake && \
>     git config --global user.name "Marc Carré" && \
>     git config --global user.email<EMAIL_ADDRESS>&& \
>     GOOS=linux GOARCH=amd64 CGO_ENABLED=0 make binaries && \
>     make test

$ checkmake --version
checkmake  built at  by  with

Any suggestion?


vm/qemu: Improve debug output.

When running in debug mode, the number of VMs is reduced to 1.
State this in the debug output.

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

This is a small change, but useful. It took me a while to figure out why only one VM is used in debug mode... I'm mostly pushing this separately to figure out how the process works. I'm working on using syzkaller on a FreeBSD host to test FreeBSD running in qemu VMs. So there are more PRs coming...

Are you up to adding similar output to other VM types for consistency?
I asked myself the question why this is not in common code. But, yes, there are reasons: some other VM types specify number of devices by enumerating all device IDs or IP addresses.

I'm mostly pushing this separately to figure out how the process works.

Please also add yourself to AUTHORS/CONTRIBUTORS files.

I'm working on using syzkaller on a FreeBSD host to test FreeBSD running in qemu VMs. So there are more PRs coming...

Sounds great!

Are you up to adding similar output to other VM types for consistency?

Sure. Will add that in a couple of minutes. Makes sense.

I asked myself the question why this is not in common code. But, yes, there are reasons: some other VM types specify number of devices by enumerating all device IDs or IP addresses.

I'm mostly pushing this separately to figure out how the process works.

Please also add yourself to AUTHORS/CONTRIBUTORS files.

To both?

I'm working on using syzkaller on a FreeBSD host to test FreeBSD running in qemu VMs. So there are more PRs coming...

Sounds great!

Yes, to both, here is some explanation:
https://github.com/google/syzkaller/blob/master/CONTRIBUTORS#L1

Yes, to both, here is some explanation:
https://github.com/google/syzkaller/blob/master/CONTRIBUTORS#L1

Fixed by https://github.com/google/syzkaller/pull/832/commits/06b1ab0ccd6224a5cae9d2414f0db64d2ffc3c18.

Are you up to adding similar output to other VM types for consistency?

Sure. Will add that in a couple of minutes. Makes sense.

Done in https://github.com/google/syzkaller/pull/832/commits/ecf19402e093675de9ed62c03392f1fce14794f3 and https://github.com/google/syzkaller/pull/832/commits/2dc9a10d10980b12b59ab2c5f85c087d274a273a.

I asked myself the question why this is not in common code. But, yes, there are reasons: some other VM types specify number of devices by enumerating all device IDs or IP addresses.

Buildbot seems to show some true warnings:
env CGO_ENABLED=1 gometalinter.v2 ./...
vm/gvisor/gvisor.go:62::error: result of fmt.Errorf call not used (vet)
vm/isolated/isolated.go:66::error: result of fmt.Errorf call not used (vet)
vm/kvm/kvm.go:71::error: result of fmt.Errorf call not used (vet)

fmt.Errorf just returns a value of type error.

Buildbot seems to show some true warnings:
env CGO_ENABLED=1 gometalinter.v2 ./...
vm/gvisor/gvisor.go:62::error: result of fmt.Errorf call not used (vet)
vm/isolated/isolated.go:66::error: result of fmt.Errorf call not used (vet)
vm/kvm/kvm.go:71::error: result of fmt.Errorf call not used (vet)

fmt.Errorf just returns a value of type error.

Should be fixed by https://github.com/google/syzkaller/pull/832/commits/514de3c653600df1f07ef82f0158fe0b3cdcbef3.
Can only do compile time testing...


CORE-8984 Modify log4j to replicate bug

Does it have to be merged into the main branch?
Alternative approach I was thinking is to have this change in the feature branch and run CI builds as necessary via dedicated configuration to get to the bottom of the flakiness. It it also possible to create a Jenkins Cron job to trigger CI execution more frequently.

Does it have to be merged into the main branch? Alternative approach I was thinking is to have this change in the feature branch and run CI builds as necessary via dedicated configuration to get to the bottom of the flakiness. It it also possible to create a Jenkins Cron job to trigger CI execution more frequently.

I ran the test 100 times but couldn't get it to fail. If we really don't want to do this, then I can setup a cron job (Ronan told me how to do this when I was trying to replicate this bug before).

I ran the test 100 times but couldn't get it to fail. If we really don't want to do this, then I can setup a cron job (Ronan told me how to do this when I was trying to replicate this bug before).

100 times is a decent count to claim stability, possibly the test is not flaky anymore? Or possibly it is a function in which order tests are executed?

I ran the test 100 times but couldn't get it to fail. If we really don't want to do this, then I can setup a cron job (Ronan told me how to do this when I was trying to replicate this bug before).

100 times is a decent count to claim stability, possibly the test is not flaky anymore? Or possibly it is a function in which order tests are executed?

Unfortunately, it seemed to fail on the release branch the other day. I thought the same since there were a few weeks between when I tried to replicate it and when it was first reported, but it still seems to happen.

I ran the test 100 times but couldn't get it to fail. If we really don't want to do this, then I can setup a cron job (Ronan told me how to do this when I was trying to replicate this bug before).

100 times is a decent count to claim stability, possibly the test is not flaky anymore? Or possibly it is a function in which order tests are executed?

Unfortunately, it seemed to fail on the release branch the other day. I thought the same since there were a few weeks between when I tried to replicate it and when it was first reported, but it still seems to happen.

I failed on one of my PRs yesterday, so I don't think we can claim it's stable.
I'm ok with this change going on, we should remember to turn it off the next beta release branch if it hasn't been resolved by then.


Search box not working properly

On cirm-dev.isrd.isi.edu, in the Scan search view, when I enter text in the search box, the following text is displayed in the sidebar:
“{{FacetsData.searchFilter}}” found in {{sideBar.getEnabledFilters()}} attribute(s):

In the browser console, the following error is displayed:
Error: $scope.FacetsData.box[facet.table][facet.name] is undefined
showFacetMatch@https://cirm-dev.isrd.isi.edu/chaise/scripts/controller/ermrestSideBarController.js?v=47bd899826a9293d721b98246defef6a:246:1
anonymous/fn@https://cirm-dev.isrd.isi.edu/chaise/scripts/vendor/angular.js?v=ea03e809effc2b87cc7bd368e1740e84 line 14432 > Function:2:650
$RootScopeProvider/this.$get</Scope.prototype.$digest@https://cirm-dev.isrd.isi.edu/chaise/scripts/vendor/angular.js?v=ea03e809effc2b87cc7bd368e1740e84:17064:34
$RootScopeProvider/this.$get</Scope.prototype.$apply@https://cirm-dev.isrd.isi.edu/chaise/scripts/vendor/angular.js?v=ea03e809effc2b87cc7bd368e1740e84:17337:13
NgModelController</this.$$debounceViewValueCommit@https://cirm-dev.isrd.isi.edu/chaise/scripts/vendor/angular.js?v=ea03e809effc2b87cc7bd368e1740e84:26912:7
NgModelController</this.$setViewValue@https://cirm-dev.isrd.isi.edu/chaise/scripts/vendor/angular.js?v=ea03e809effc2b87cc7bd368e1740e84:26884:7
baseInputType/listener@https://cirm-dev.isrd.isi.edu/chaise/scripts/vendor/angular.js?v=ea03e809effc2b87cc7bd368e1740e84:23129:7
m.event.dispatch@https://cirm-dev.isrd.isi.edu/chaise/scripts/vendor/jquery-latest.min.js?v=8101d596b2b8fa35fe3a634ea342d7c3:3:8384
m.event.add/r.handle@https://cirm-dev.isrd.isi.edu/chaise/scripts/vendor/jquery-latest.min.js?v=8101d596b2b8fa35fe3a634ea342d7c3:3:5122

I pushed changes to master to make sure the global handleException function is not run in the search app. We previously made changes to limit when window.onerror would run but didn't make the same changes to angular's global exception handler.
The changes can be seen in commit 73476540a61b4e76ffbeb81af61265854ce96d05.


hard-xml-derive: create an upstream pull request for with attribute

ue-rs has several 3rd-party patches under vendor/hard-xml-derive to add with attributes.
Ideally the changes should be created as an upstream pull request of the upstream repo.

Created an upstream PR https://gitlab.com/kevincox/hard-xml/-/merge_requests/13.

The upstream PR https://gitlab.com/kevincox/hard-xml/-/merge_requests/13 was merged, and included in hard-xml v1.34.0. We need to also update the corresponding ue-rs code.

Should we open a new issue for the remaining task?

Right, sorry, I still have to get back to the rest work.


[x86][sse4.1] Add phminposuw & pmul* instructions

Unfortunately, pmulld is not supported by llvm.
This is a PR for https://github.com/rust-lang-nursery/stdsimd/issues/40 tracking issue.

@BurntSushi pmulld gives a misleading result when a sign bit is set during multiplication of two positives.
Can we do smth with that?

@crypto-universe Sorry, I don't understand your question. Can you unpack? Remember the goal: match the vendor interface.

Sorry, what do you mean by unpack? It is done similar to _mm_add_epi16 in sse2 in this repo.
Interface fits an Intel Intrinsics Guide.

@crypto-universe It means I'd like you to explain more. I don't understand your question. It might help to provide code examples.

I was wondering if it good to leave it as it is, when 2 positive values after multiplication gives a negative value. Of course, SIMD instruction works like this and doesn't care about signed/unsigned data types, but maybe we should care on rust level somehow?
    #[simd_test = "sse4.1"]
    unsafe fn _mm_mullo_epi32() {
        let a = i32x4::new(15, -2, **1234567**, 99999);
        let b = i32x4::new(-20, -256, **666666**, -99999);
        let r = sse41::_mm_mullo_epi32(a, b);
        // Pay attention, most significant bit in e[2] is treated
        // as a sign bit!
        // **1234567 * 666666 = -1589877210**
        let e = i32x4::new(-300, 512, **-1589877210**, -1409865409);
        assert_eq!(r, e);
    }

I was wondering if it good to leave it as it is, when 2 positive values after multiplication gives a negative value.

_mm_mullo_epi32's is specified by Intel to work on signed integers. It multiplies the pairs of 32bit integers as 64bit integers [*], and the lowest 32 to bits of that 64bit signed integer are returned.
Intel declares the intrinsic to return a signed integer, but specifies the semantics of the intrinsic as returning "just some bits".
I'd say that we should stick to returning a i32x4 for consistency with Intel intrinsics. While the behavior can be surprising, returning an u32x4 would just make things surprising in a different way.
Maybe you could improve the docs a bit to state that this instructions don't return the result of the multiplication of two signed integers, but emphasize how they work (that is, that they perform a lossless 64-bit multiply, and then return the lowest 32-bit, whatever they might be). And then give as an example that while pmulld i32x4::splat(2), i32x4::splat(2) behaves as an user would expect, pmulld i32x4::splat(i32::MAX) i32x4::splat(2) would return a negative number.
[*](avoiding signed integer overflow in C and C++ which is undefined behavior) semantics are to wrap around on overflow, so I think that is fine.

@gnzlbg if you don't mind, I'll use sentences from your comment in documentation :)

@crypto-universe np, don't forget to run rustfmt :)

@gnzlbg rustfmt has no complains about sse41.rs content (but fixed another 22 files :P)
@BurntSushi GitHub claims that "Changes requested" by you. Changes are done, but with --amend and git push --force, so GitHub didn't notice changes. Make a review again, please.

rustfmt has no complains about sse41.rs content

The build bot is broken, are you using rustfmt-nightly?

cargo install rustfmt-nightly --force
cargo fmt --all

The diff is:
-        let a = i32x4::new(15, 2/*ignored*/, 1234567, 4/*ignored*/);⏎
-        let b = i32x4::new(-20, -256/*ignored*/, 666666, 666666/*ignored*/);⏎
+        let a =⏎
+            i32x4::new(15, 2 /* ignored */, 1234567, 4 /* ignored */);⏎
+        let b = i32x4::new(⏎
+            -20,⏎
+            -256, /* ignored */⏎
+            666666,⏎
+            666666, /* ignored */⏎
+        );⏎

Previously I did it with cargo install rustfmt. Now should be good.

@BurntSushi , @gnzlbg any comments to this PR?

@crypto-universe I'm just waiting on @BurntSushi to check it out again, but this looks good to me.
I want to thank you for working on this, and please don't take the long delay here personally. Sometimes we all have a lot of time and the timezone god blesses us and things get merged very quickly, but it is normal for pull-requests to take ~1 week to get merged.
If you need this changes in a release to be able to use them it always helps to let us know about it and we try to speed things up.
I know it is hard but what I try to do is pick a different item in the mean time and have multiple parallel pull-requests. I am pretty sure your next pull-request will be merged quicker :)


devcontainer support

Added configurations files to devcontainer environment and improved README.md.

Appreciate the PR but I'm going to pass on this. Assuming folks can just deploy as a webcontainer if they want that.


Transaction aborted not working for oracle insertion: ORA error given

When insert operation done on Oracle database table, the aborted section is not invoked as expected, rather following oracle exception occurs
"error in ballerina program: uncaught error: Error{ msg : execute update failed: ORA-00904: "ID": invalid identifier
}"
Please find the attached code and the payload details
scenario1.zip

In this scenario the aborted section is executed without any issue. But the cause for the abort is an exception occurred in the update action. So that exception should be caught somewhere within the execution flow. Otherwise it will thrown as uncaught error as observed


The zip option includes too many unnecessary files

Hi, @YosfanEilay
Right now, the ZIP option includes all files into the "ForensicMiner" folder:

but I think it would be cleaner and more efficient to include only the folders marked with the red box, with all their contents.
That way, the analyst only gets the files he/she needs, leaving out the rest that will not be part of the analysis.
Regards!

Hello @Nostramo-DFIR !!
You are right about that one, working on it right now ^_^

Hi @Nostramo-DFIR :)
Done with the improvement of the ZIP option, can you please check it our and say what you think about it ?
It's now in v1.2 you can download from the main repo.

Thank

Hi, @YosfanEilay
I've tried it. The feature is perfect right now ;)
Thanks a lot.


Faster updates from GitHub app repos

For repos that are apps, we should be getting updates faster.  For lockers exposed on a public url, we should be able to use http://developer.github.com/v3/repos/hooks/.
Could also use faster polling from GitHub when present in the dashboard or something like that, but IMO the repos push API is very straightforward and we need to address push soon anyways.
Total alternative: manual sync button(s) on publish page (and possibly a few others)

I did the work in #771, going to re-milestone this now.

We can either do use the GH push API or add more sync now actions (i vote for the former), but many people are still having trouble getting things in a reasonable amount of time, so I'm moving this to be high priority

@quartzjer Here's a mock showing a "sync Github" button in the UI:


About set requires_grad_ problem

your code:
one_hot = torch.from_numpy(one_hot).requires_grad_(True)
when i change this code follow:
one_hot = torch.from_numpy(one_hot)
,finally ,i get same grad val。。 useful about requires_grad set??

office docment explain:

generally speaking，the param default is False,then my grad not to comput,but in this code,my grad val still have output

i seem find this answer，about backward() exlplain:

right？


Assertion for number of objects

In spatial_transform.py there is an assertion assert n_obj <= z_wheres.shape[1] in function add_bounding_boxes. Shoudln't the index here be 0 because the z_where is (max_objects, features)?

Hi, I'm terribly sorry for the very late reply.
In principle z_wheres could have shape (1, max_n_objects, 3) or (max_n_objects, 3). I think the right thing to do is to change the index to 0 as you suggest, and move this assertion after this block:
if len(z_wheres.shape) == 3:
    assert z_wheres.shape[0] == 1
    z_wheres = z_wheres[0]

such that the shape of z_where is always (max_n_objects, 3).
Feel free to update that and open a PR :) Thank you!


feat: refactor inventory / resolve plugins

Which issue(s) this PR fixes

Fixes #650
Additional comments

In this pr:

Rename inventory plugin to tdp_vars
Tdp vars plugin now cacheable
Tdp vars now configurable in ansible.cfg
Now longer need to put tdp vars in inventory variable

Example of new configuration:
[inventory]
cache = true # Cache not mandatory, but faster, especially with tdp-lib usage
cache_plugin = jsonfile
cache_timeout = 7200
cache_connection = .cache

[tdp]
vars = tdp_vars # path relative to ansible working directory or absolute

What's next
This PR lays the ground work needed to be able to load tdp vars from defaults.
Next: Load tdp vars from collections defaults and project's tdp vars to allow having only what's changed in project's tdp vars
Agreements

[X] I hereby declare this contribution to be licensed under the Apache License Version 2.0, January 2004.
[X] I hereby agree to grant TOSIT a copyright license to use my contributions.

Force push is editing wrong import path

De we need to move tdp_vars outside of inventory directory ?

We don't have to, but they're not a resource managed by ansible anymore.
That's why in the example I moved them outside of the inventory folder.