remove generated code from the repo (apart from the server)

provide simple makefile which will regen all those (in docker = no user deps needed)

maybe server does not have to be the exception

npm requires absolute path to "link" the generated client into the website, which makes it impossible to do it inside the docker :(

I do not understand what that means. What is the essential part docker environment is missing to be able to do this?

visualization has two pieces of generated code:

/web/js/client/* which is generated in docker by visualization-gen
/web/js/bundle.js which is generated by npm from index.js and the client

generating the bundle requires full path to the client, but it uses the index.js, which is modified outside the docker during development..

it might be possible to make npm run inside the docker, but it would need to do so even during development.
furthermore, my intention with committing the bundle.js was that you/anyone can run the website without any dependencies..
i am conflicted about this..

i am using the same generated client for my js bot too. but that would hopefully still work..

in the end - i think that it would be ok to remove generated code for the server, and keep the code for python and javascript.
reasoning: the server is always run inside the docker, which can easily generate the code for itself. but javascript and python scripts should be runnable even without the need to generate any code (aka without installing docker).

Conclusion:

move example bot into separate github repository for easy forking

include generated code

keep generated bundle.js for web, do not commit generated client.js (no changes here)
remove generated code for go server, it can generate it before compilation


Raises Error: multiple bump targets

@molt/action In github action in Denops.vim (eb9ef6a):
https://github.com/vim-denops/denops.vim/actions/runs/10436394656/job/28901411214#step:3:480
deno run --allow-env --allow-read --allow-write --allow-net --allow-run=deno,git --no-prompt --unstable-kv jsr<EMAIL_ADDRESS>...
Error: multiple bump targets for jsr:@std/async

Also, when running the @molt/cli, an AssertionError without message.
> deno task update
Task update deno run --allow-env --allow-read --allow-write --allow-run=git,deno --allow-net=jsr.io,registry.npmjs.org jsr:@molt/cli **/*.ts
error: Uncaught (in promise) AssertionError
    throw new AssertionError(msg);
          ^
    at assert (https://jsr.io/@std/assert/1.0.2/assert.ts:21:11)
    at new Update (https://jsr.io/@molt/core/0.19.6/mod.ts:189:5)
    at Dependency.check (https://jsr.io/@molt/core/0.19.6/mod.ts:137:14)
    at eventLoopTick (ext:core/01_core.js:168:7)
    at async Promise.all (index 0)
    at async Command.actionHandler (https://jsr.io/@molt/cli/0.19.6/main.ts:71:20)
    at async Command.execute (https://jsr.io/@cliffy/command/1.0.0-rc.5/command.ts:1940:7)
    at async Command.parseCommand (https://jsr.io/@cliffy/command/1.0.0-rc.5/command.ts:1772:14)
    at async https://jsr.io/@molt/cli/0.19.6/main.ts:113:1

I'm guessing the cause is probably the same.
How should I deal with this?

I tried the following and it worked fine in this simple case:
File: a.ts
import * as _ from "jsr<EMAIL_ADDRESS>
File: b.ts
```ts
import * as _ from "jsr<EMAIL_ADDRESS>
Run:
deno run -A jsr:@molt/cli a.ts b.ts
� @std/async 0.224.2 → 1.0.3 (^1.0.0, ^0.224.0 → ^1.0.0)

In denops.vim, a non-existent version of @std/async was defined as a test fixture for testing the dependencies cache. This was the direct cause of the error.
File: tests/denops/testdata/no_check/cli_constraint_error_on_issue_401.ts
import * as _ from "jsr<EMAIL_ADDRESS>
Does molt/cli have a function to exclude files? I couldn't use --ignore because it ignores dependencies.
I want it like this:
deno run -A jsr:@molt/cli --exclude no_check **/*.ts

It would also be good if the same could be done with molt-action:
steps:
  - uses: hasundue/molt-action@v1
    with:
      branch: automation/update-dependencies
      config: false
      labels: automation
      token: ${{ secrets.PA_TOKEN }}
      source: "**/*.ts"
      exclude: "no_check"

@Milly Sorry for my late response, and appreciate your effort for investigating the problem :pray:

[ ] Panicking on a non-existent version is a bug. Should be fixed to just warn it gracefully.

Does molt/cli have a function to exclude files?

No, unfortunately. You may utilize the glob of your shell though.

[ ] I'm positive to add exclude option and input to the CLI and the action respectively.

I'm preparing for the movement on the weekend these days, so I'm afraid but it is unlikely for these tasks done before that.


windows: Benchmark performance

Right now the Windows implementation is not up to par in terms of performance with Unix, as there are many opportunities for improvement. The implementation should be benchmarked in terms of actual I/O throughput numbers and there should be a better handle on where exactly performance needs to be improved.

@carllerche had the excellent idea of benchmarking against libuv, would be good to get at least some comparison numbers!

This would still be nice, but I"m going to close this due to inactivity. Maybe we could have a "wish list" wikipage to track these longer term wants?


Jupyter notebooks can't run using insiders version

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
Updated my VSC-insiders this morning and now I can't run jupyter notebooks, it seems to be having trouble starting the kernel. I was using this notebook fine before the update, and it will work in the regular VSC, but something must have broke in the latest insders update.  I have tried starting different kernels using different python versions, but none of them work.
VS Code Version
1.66.0-insider
Jupyter Extension Version
v2022.4.1000891004
Jupyter logs
Info 18:06:42: Execute Cells request 0
Info 18:06:42: Execute Cell 0 ~/projects/BB-499/BB-499_analysis.ipynb
Info 18:06:42: Starting Notebook in kernel.ts id = .jvsc74a57bd0e5083375c393fea27a2cbfaf1c5ebe587d437e1d86df363a30887befcfea78b6./home/kyaki/v_envi/general/python./home/kyaki/v_envi/general/python.-m#ipykernel_launcher for ~/projects/BB-499/BB-499_analysis.ipynb
Info 18:06:42: Checking for server existence.
Info 18:06:42: Checking for server usability.
Info 18:06:42: Starting notebook server.
Info 18:06:42: Connecting to server
Info 18:06:42: Launching server
Info 18:06:42: Starting Notebook
Info 18:06:43: Generating custom default config at /tmp/c0d959a1-287a-4f23-83ae-dc2e00a1eda2/jupyter_notebook_config.py
Info 18:06:43: Starting Jupyter Notebook
Starting Jupyter from ~/.pyenv/versions/3.8.6/bin/python with command line --no-browser --notebook-dir="/home/kyaki/projects/BB-499" --KernelManager.autorestart=False --config=/tmp/c0d959a1-287a-4f23-83ae-dc2e00a1eda2/jupyter_notebook_config.py --NotebookApp.iopub_data_rate_limit=10000000000.0
Info 18:06:43: Process Execution: > ~/.pyenv/versions/3.8.6/bin/python -m jupyter notebook --no-browser --notebook-dir="/home/kyaki/projects/BB-499" --KernelManager.autorestart=False --config=/tmp/c0d959a1-287a-4f23-83ae-dc2e00a1eda2/jupyter_notebook_config.py --NotebookApp.iopub_data_rate_limit=10000000000.0
> ~/.pyenv/versions/3.8.6/bin/python -m jupyter notebook --no-browser --notebook-dir="/home/kyaki/projects/BB-499" --KernelManager.autorestart=False --config=/tmp/c0d959a1-287a-4f23-83ae-dc2e00a1eda2/jupyter_notebook_config.py --NotebookApp.iopub_data_rate_limit=10000000000.0
Info 18:06:43: Waiting for Jupyter Notebook
pyenv: jupyter-notebook: command not found
Info 18:06:44: pyenv: jupyter-notebook: command not found

The `jupyter-notebook' command exists in these Python versions:
Info 18:06:44: pyenv: jupyter-notebook: command not found

The `jupyter-notebook' command exists in these Python versions:

  3.8.6

Note: See 'pyenv help global' for tips on allowing both
      python2 and python3 to be found.
Info 18:06:44: pyenv: jupyter-notebook: command not found

The `jupyter-notebook' command exists in these Python versions:
  3.8.6

Note: See 'pyenv help global' for tips on allowing both
      python2 and python3 to be found.

Error 18:06:44: failed to create INotebook in kernel, UI Disabled = false [i [Error]: Jupyter server crashed. Unable to connect.
Error code from jupyter: 127
pyenv: jupyter-notebook: command not found

The `jupyter-notebook' command exists in these Python versions:

  3.8.6

Note: See 'pyenv help global' for tips on allowing both
      python2 and python3 to be found.

	at t.JupyterConnectionWaiter.rejectStartPromise (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:357343)
	at t._complete (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:358006)
	at t.n (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:3281174)
	at t.__tryOrUnsub (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:3281416)
	at t.complete (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:3281309)
	at t._complete (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:3279789)
	at t.complete (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:3279488)
	at ChildProcess.<anonymous> (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:821596)
	at Object.onceWrapper (node:events:510:26)
	at ChildProcess.emit (node:events:402:35)
	at Process.ChildProcess._handle.onexit (node:internal/child_process:290:12)] {
  category: 'jupyterconnection'
}
Warn 18:06:44: Kernel Error [i [Error]: Jupyter server crashed. Unable to connect.
Error code from jupyter: 127
pyenv: jupyter-notebook: command not found

The `jupyter-notebook' command exists in these Python versions:

  3.8.6

Note: See 'pyenv help global' for tips on allowing both
      python2 and python3 to be found.

	at t.JupyterConnectionWaiter.rejectStartPromise (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:357343)
	at t._complete (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:358006)
	at t.n (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:3281174)
	at t.__tryOrUnsub (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:3281416)
	at t.complete (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:3281309)
	at t._complete (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:3279789)
	at t.complete (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:3279488)
	at ChildProcess.<anonymous> (/home/kyaki/.vscode-server-insiders/extensions/ms-toolsai.jupyter-2022.4.1000891004/out/extension.js:2:821596)
	at Object.onceWrapper (node:events:510:26)
	at ChildProcess.emit (node:events:402:35)
	at Process.ChildProcess._handle.onexit (node:internal/child_process:290:12)] {
  category: 'jupyterconnection'
}
Info 18:06:44: Process Execution: > ~/v_envi/general/bin/python -c "import ipykernel"
> ~/v_envi/general/bin/python -c "import ipykernel"
Info 18:06:44: Dispose kernel file:///home/kyaki/projects/BB-499/BB-499_analysis.ipynb

Coding Language and Runtime Version
No response
Language Extension Version (if applicable)
No response
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Remote

Thanks for filing thi s issue, I believe I know what's caused this issue


Don't error out if the substate does not exist

In a clean install, and a newly generated component, the following error would be thrown:

Uncaught (in promise) TypeError: Cannot read property 'toJS' of undefined

This is because there is no substate since the redux store does not contain anything yet.
This fix returns an empty object if the substate does not exists.

Coverage remained the same at 98.953% when pulling 818c0e44171152590b7894c59c03dff1791220b0 on roderik:patch-1 into 8693af3708e31f25fc0883256da605b3ccc03c9a on mxstbr:master.

Indeed!

The error is happening because the selector can't find the state. The state need to be passed as argument when creating the selector on mapStateToProps.

I'm even more confused now. Do we need this or not?

The error happens because the generators create all the files for the container but don't add the container's reducer to the reducers file, so the container's selector can't get the piece of the state related to it because it doesn't exist.
I believe we need to make the reason clear so people can quickly understand it and solve it accordingly. A couple of comments should do the trick.

Coverage remained the same at 98.77% when pulling 40d7b7cc0655c44b5081cf681ece43e41ed002e2 on roderik:patch-1 into 6c637d5924ba4e6d9cb230bae1e313925b532109 on mxstbr:master.

Is this PR still relevant? V3.4 has a lot of changes.

@KarandikarMihir It may be relevant, but it first must be rebased against dev I just did that, and updated against current dev, did that too.
If the code survives those actions, someone should test to confirm that the OP condition still exists and that the proposed changes correct the issues.
Assuming the above pass, the community will need to decide if the error should be suppressed as in the OP solution, or documented, or both.
I vote for both, documentation in the form of a code comment.

The error happens because the generators create all the files for the container but don't add the container's reducer to the reducers file, so the container's selector can't get the piece of the state related to it because it doesn't exist.

I think #1523 is addressing the same problem, but in a more informative manner.
Unless I'm mistaken, I vote for #1523.

I agree with @gihrig #1523 looks to have this covered. It looks like no one objects, so closing this PR in favor of #1523 Thank you for raising the issue @roderik and getting the ball rolling ✨


#242 Event Time Selector - Addressed Changes Requested

Things Completed:

Fixed the formatting for event time handling
Fixed how this data is passed to the DB
Fixed the HTML logic for time formatting (%I - %H)  -> %H represents 24 hour time
Fixed Issue #242

I think you broke something when you were fixing it up for firefox. It's now expecting the 12 hour time string, instead of a date time.
When I created a new event:
ValueError: time data '15:58' does not match format '%I:%M %p'
File "/home/brian/dev/berea/celts/app/controllers/admin/routes.py", line 105, in createEvent
preprocessEventData(eventData)
File "/home/brian/dev/berea/celts/app/logic/events.py", line 281, in preprocessEventData
eventData['timeStart'] = format24HourTime(eventData['timeStart'])
File "/home/brian/dev/berea/celts/app/logic/utils.py", line 53, in format24HourTime
time = datetime.strptime(timeStr, "%I:%M %p").strftime("%H:%M") # Converts string to datetime and formats correctly


Change repo/branch name to links in Create Pull Request view

Make it easier for people to jump to GitHub from the Create Pull Request View by changing the repository/branch name to links
Via @egamma...
"My suggestion [...] is to change the repository/branch name into links:"

"This can be done by having a drop down only in front of the repo and branch name and by showing the currently selected repo/branch as a link label (that is not part of the drop down). In ASCII…:"
| v |  burkeholland/cloud-developer
| v |  master

@burkeholland I'm trying to understand the use case for this - are you just trying to continue the create on GitHub, or do you actually need to get to those repos?
I think it would be clearer to have an action in the header that just says "Create on GitHub" or something like that that could copy whatever data you've entered and open GitHub

I think it's just to make it easier for me to jump between Codespaces and GitHub. I might want to do that because I want to open the PR on GitHub, or perhaps I go to GitHub to check something - like a secret. The overall idea is to just make it easier for people to move between Codespaces and GitHub if they want to.


Add ability to filter orders by status

This adds the ability to filter orders by status and refactors the tests to use @pytest.mark.parametrize instead of single test methods.

Codecov Report

Merging #499 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #499   +/-   ##
========================================
  Coverage    96.49%   96.49%
========================================
  Files          115      115
  Lines         3647     3647
  Branches       335      335
========================================
  Hits          3519     3519
  Misses          87       87
  Partials        41       41

Impacted Files
Coverage Δ

datahub/search/omis/views.py
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


adding user rahuljain

Sorry, this needs a rebase now. Additionally, please get approval/comment here from your team mate/manager for approving this.(they can also merge if its all good).

Rebase required. Please rebase and submit the patch again.

did Rebase. Can you verify it?

Looks like rebase is not clean. Please do clean rebase and add you changes only. Currently it shows so many changes.


on open or close project/vscode run task/script

when working on several projects it would be really good if we could have a task or script that could be launched when opening or closing the project or the editor
so as not to have an artifact that continues to live
it will be useful with

docker/docker-compose
services useful only for the duration of the project

This exists on open of a folder with:
"runOptions": {"runOn": "folderOpen"},

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

@meganrogge
and on open and close of a project ?

We won't do this on close of vscode as the processes are no longer around


[BACKEND] Prevent duplicate alias creation

Prevent create creation of duplicate alias creating.
Approach :-

For /minify/add :- Check database if auto generated alias if present or not. If present regenerate.
For /minify/add/custom :- Add an additional endpoint /minify/alias/:alias/status
Here check in database if already present, if yes return {"success": "false"} else  {"success": "true"}

@Nanzzie What's your progress?

@Nanzzie Any progress?

Or shall I assign?

Yess sir u can unassign it
On Thu, 10 Mar 2022, 16:42 Jyotirmoy Bandyopadhyaya [Bravo68], <
@.***> wrote:

Or shall I assign?
—
Reply to this email directly, view it on GitHub
https://github.com/BRAVO68WEB/url-minify/issues/84#issuecomment-1063940752,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUTIMWCPMLTJPCDVLFDB6YLU7HKKPANCNFSM5PV52XMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

is it still unresolved? If yes, then i would like to try.

@Danko-chan PR under review. Lets See
You may create Issues if you want to contribute

@Danko-chan Go ahead


[Windows]Datasource name is not unique

Steps to reproduce:

Create a Data Service project and right click on the project and add new Data Service.
Click to add a new data source, insert the below information and click on save.

Data Source type: RDBMS
Datasource ID: 100
URL: jdbc:mysql://localhost:3306/Employees
Provide DB credentials

3, Now go to source view and copy paste the same config and save.

Expected: It should not allow to use the same datasource ID.

Related to the issue - https://github.com/wso2/devstudio-tooling-ei/issues/819


Bugfix: Removed animate and transition directives

Before submitting the PR:

[x] Does your PR reference an issue? Fixes: #1317
[x] Does your branch follow our naming convention? If not, please amend the branch name using `branch -m new-branch-name

What does your PR address?
Poor performance with Autocomplete on large datasets
Please briefly describe your changes here.
Removed animate and transition directives for now.

Fantastic point. I didn't even think of this, noted to make that change when I get home tonight. Thanks!

@JustBarnt saw the change come through. Looks good to me, unless you have anything else I'll aim to merge this tomorrow. Super appreciate the help here sir!

@JustBarnt thanks for duration change. I think with that this is good to merge. You should be a go on: https://github.com/skeletonlabs/skeleton/issues/1322


Convert sender to polymorphic and add User as sender

Closes #1456
This PR changes the sender of a Message to be a polymorphic association between Contributor and User. This seems like the most semantic way to deal with it, but not the least complex. I considered adding another attribute, like user_id, but I think the logic is not so clear. We need to look at the implementation to see that we save the user_id when a user sends out a Request/ChatMessage. Then we would still have sender_id: nil in a lot of cases. We could rename the sender_id to contributor_id and then have:
contributor_id
user_id
creator_id

We can still consider this if we like the simplicity of it vs it being less semantic. I don't really think that we would have other types of senders, in the future so maybe polymorphic associations are a bit overkill? One thing I still don't really understand, or keep forgetting is how we manually_create a Message. I know that when we do so we add the creator_id. I can see that the MessagesController#update and I can see we have a test that checks that if one sends a PATCH request to this method, it adds a creator_id, but I don't see where we actually do this from inside the application.
Looking at the data, it doesn't seem to be a very-used feature.

@tillprochaska I don't know that you will have time, but it would be nice to get your perspective on this since you made a comment when talking about the activity_notifications that you didn't think it was worth using polymorphic associations, at least for that case, because of their complexity.


ETL giving up after one error

OrientDB Version, operating system, or hardware.

v2.0 SNAPSHOT[ ] - .18[ ] .17[ ] .16[ ] .15[ ] .14[ ] .13[ ] .12[ ] .11[ ] .10[ ] .9[ ] .8[ ] .7[ ] .6[ ] .5[ ] .4[ ] .3[ ] .2[ ] .1[ ] .0[ ]
v2.1 SNAPSHOT[ ] - .16[ ] .15[ ] .14[ ] .13[ ] .12[ ] .11[ ] .10[ ] .9[ ] .8[ ] .7[ ] .6[ ] .5[ ] .4[ ] .3[ ] .2[ ] .1[ ] .0[ ]
v2.2 SNAPSHOT[ 12] - .rc1[ ] .beta2[ ] .beta1[ ]

Operating System

[x ] Linux
[ ] MacOSX
[ ] Windows
[ ] Other Unix
[ ] Other, name?

Error in Pipeline execution: com.orientechnologies.orient.core.exception.OCommandExecutionException: No edge has been created because no target vertices
Why give up(see below log)? Can't this process just report the problem and continue? With millions of edges in question this is behavior is inefficient and wasting lots of time. In the other ETL transformers you can specify "unresolvedLinkAction". Why not add this(or similar features) to the Command Transformer also?
Expected behavior and actual behavior
From a three column csv-file(arg1,arg2,arg3) arguments are taken to create edges with one property.
The essence of the actual ETL json config is this:
"command" :
create edge from (select from ClassX where $input.arg1..) to (select from ClassX where $input.arg2..) set something=$input.arg3
Here is what happens if one target vertice is not found:
excerpt from logging:

extracted 7,996 rows (5 rows/sec) - 7,996 rows -> loaded 7,493 vertices (5 vertices/sec) Total time: 1662154ms [0 warnings, 0 errors]
extracted 8,000 rows (4 rows/sec) - 8,000 rows -> loaded 7,497 vertices (4 vertices/sec) Total time: 1663154ms [0 warnings, 0 errors]
extracted 8,004 rows (4 rows/sec) - 8,004 rows -> loaded 7,501 vertices (4 vertices/sec) Total time: 1664154ms [0 warnings, 0 errors]
extracted 8,008 rows (4 rows/sec) - 8,008 rows -> loaded 7,505 vertices (4 vertices/sec) Total time: 1665154ms [0 warnings, 0 errors]
Error in Pipeline execution: com.orientechnologies.orient.core.exception.OCommandExecutionException: No edge has been created because no target vertices
DB name="XXXX"
DB name="XXXX"
extracted 8,010 rows (1 rows/sec) - 8,010 rows -> loaded 7,507 vertices (1 vertices/sec) Total time: 1666155ms [0 warnings, 1 errors]
extracted 8,010 rows (0 rows/sec) - 8,010 rows -> loaded 7,507 vertices (0 vertices/sec) Total time: 1667155ms [0 warnings, 1 errors]
extracted 8,010 rows (0 rows/sec) - 8,010 rows -> loaded 7,507 vertices (0 vertices/sec) Total time: 1668155ms [0 warnings, 1 errors]
extracted 8,010 rows (0 rows/sec) - 8,010 rows -> loaded 7,507 vertices (0 vertices/sec) Total time: 1669155ms [0 warnings, 1 errors]
extracted 8,010 rows (0 rows/sec) - 8,010 rows -> loaded 7,507 vertices (0 vertices/sec) Total time: 1670156ms [0 warnings, 1 errors]

Steps to reproduce the problem
You need a class containing some 1000' records and a list of the vertices to be linked (see above).

Can you provide me the configuration and sample data?

robfrank,
Linking items from the same class.
You need to edit filename in json and make your own for the inital data to be linked.
The linking will fail once it hits a col2-item that does not exist in the initially loaded data.
You need to edit/create your own json to load the classfile.
etlsamples.tar.gz

Sorry, but I don't understand how to use your samples. The config json seems truncated.
Could you please write down the entire procedure and provide a working configuration?

The json is deliberately truncated. I expect you to fill in the remaining stuff yourself, but here it is, passwords removed.

run ETL to load the classfile - master data if you like.
{
"config": {
"parallel": false,
"log": "INFO"
},
"source": { "file": { "path": "/path-to-/" } },
"extractor": { "csv": {"separator": ","} },
"transformers": [
{ "vertex": { "class": "HumanProtein" } }
],
"loader": {
"orientdb": {
"dbType": "graph",
"dbURL": "remote:localhost/XXXXX",
"dbUser": "admin",
"dbPassword": "admin",
"serverUser": "root",
"serverPassword": "********",
"dbAutoCreate": false,
"wal": false,
"dbAutoCreateProperties": true,
"classes": [
{name: 'HumanProtein', extends:"V"}
],
"tx": false,
"batchCommit": 1600
}
}
}

run ETL with the below json.

{
"config": {
"parallel": false,
"log": "DEBUG"
},
"source": { "file": { "path": "/path-to-/" }
},
"extractor": {
"row": {}
},
"transformers": [
{
"csv": {
"separator": ","
}
},
{
"command" : {
"command" : "create edge InteractWith from (select from HumanProtein where STRINGproteinId= '${input.protein1}') to (select from HumanProtein where STRINGproteinId= '${input.protein2}') set Score=${input.combined_score}",
"output"  : "edge"
}
}
],
"loader": {
"orientdb": {
"dbType": "graph",
"dbURL": "remote:localhost/XXXXX",
"dbUser": "admin",
"dbPassword": "admin",
"serverUser": "root",
"serverPassword": "*****",
"dbAutoCreate": false,
"wal": false,
"dbAutoCreateProperties": true,
"classes": [
{name: 'HumanProtein', extends:"V"},
{name: 'InteractWith', extends:"E"}
],
"tx": false
}
}
}

edit the link data to make sure one of the items in column 2 does not exist in the HumanProtein class you loaded in 1 by changing one of ENSPxxxx-code, i.e. in col 2.

Hope this helps.

just to be sure and don't waste my time:

first run using classfile.txt as input
second run using  linkfile.txt ad input BUT I need to modify the file

That's correct. The idea is to make sure the linking fails. Do you
understand the ETL concept?

nov. 2016 14.34 skrev "Roberto Franchini"<EMAIL_ADDRESS>

just to be sure and don't waste my time:

first run using classfile.txt as input
second run using linkfile.txt ad input BUT I need to modify the file

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/orientechnologies/orientdb/issues/6872#issuecomment-258142931,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASkWeGJb8Ist_f0dRNMSfaSzQQHTVuGDks5q6eKggaJpZM4KnF2r
.

The ETL processes fails the same way, even with other transformers. I believe ODB should consider
changing ETL to include a "dryrun".
'Dryrun' should just do basic checking while logging bad data-rows with line numbers.
Facing a situation with jobs loading numerous CSV files, receiving data glitches is inevitable. Cleaning up after a failed ETL is not an easy task.
{
"config": {
"log": "dryrun",
Thank you.

Hi, I'll take care of this issues next week.  I did not use it and don't know if it works, so I have to test it. The question is
whether it gives enough sensible info in the log to make any sense. Running ETL with debug leaves Gb of
useless repeated data when loading millions of rows.
So, in essence I maintain my request for a 'dry run'.

haltOnError set to false is unfortunately not any helpful option.
Neither INFO nor ERROR helped me locating the error in question. That only leaves DEBUG(which makes
to much noise).
By the way, why do you repeat the name of the database twice when things go wrong?

Logging in case of errors is improved with input data, executed command and exception message:
[445:command] ERROR exception=No edge has been created because no target vertices
	DB name="graph" - input={protein1:ENSP00000000233,protein2:ENSP00000270115,combined_score:204} - command=sql.create edge InteractWith from (select from HumanProtein where STRINGproteinId= 'ENSP00000000233') to (select from HumanProtein where STRINGproteinId='ENSP00000270115') set Score=204
Error in Pipeline execution: com.orientechnologies.orient.core.exception.OCommandExecutionException: No edge has been created because no target vertices
	DB name="graph"

WDYT?
I don't have any idea in how to implement a dry run. How to do a "dry run " interaction with the database? My suggestion is to use the plocal or even  memory  connection to test the ETL process.

This makes perfect sense to me. The notion about 'dry run' was just a way
to express an idea.
Are this messages shown when log is ERROR or when any error is detected?
On 28 December 2016 at 19:15, Roberto Franchini<EMAIL_ADDRESS>wrote:

Logging in case of errors is improved with input data, executed command
and exception message:
[445:command] ERROR exception=No edge has been created because no target vertices
DB name="graph" - input={protein1:ENSP00000000233,protein2:ENSP00000270115,combined_score:204} - command=sql.create edge InteractWith from (select from HumanProtein where STRINGproteinId= 'ENSP00000000233') to (select from HumanProtein where STRINGproteinId='ENSP00000270115') set Score=204
Error in Pipeline execution: com.orientechnologies.orient.core.exception.OCommandExecutionException: No edge has been created because no target vertices
DB name="graph"
WDYT?
I don't have any idea in how to implement a dry run. How to do a "dry run
" interaction with the database? My suggestion is to use the plocal or even
memory connection to test the ETL process.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/orientechnologies/orientdb/issues/6872#issuecomment-269516840,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASkWeOGgkVcbKNMa48ql0Jl_LtI3_TRIks5rMqc8gaJpZM4KnF2r
.

--
Tore Austrått
phone +47 90657231

When the error is detected.  This fix will be part of 2.2.15, but you can download a snapshot version  of etl and use it.


Connect-PartnerCenter help should be updated to current definition

Steps to reproduce
Get-Help Connect-PartnerCenter -Examples
NAME
Connect-PartnerCenter
SYNOPSIS
Connects to Partner Center with an authenticated account for use with cmdlet requests.
-------------------------- Example 1 --------------------------

PS C:\> Connect-PartnerCenter -ApplicationId '<AppId>'

Connect to Partner Center using the specified application identifier during authentication.
-------------------------- Example 2 --------------------------

PS C:\> $credential = Get-Credential
PS C:\> Connect-PartnerCenter -ApplicationId '<AppId>' -Credential $credential

Connect to Partner Center using the specified application identifier during authentication.
-------------------------- Example 3 --------------------------

PS C:\> $credential = Get-Credential
PS C:\> Connect-PartnerCenter -Credential $credential -ServicePrincipal -TenantId '<AppId>'

Connects to Partner Center using app only authentication. When prompted for credential specify the application
identifier for the username and the application secret for the password.

What steps can reproduce the defect?
Please share the setup, commandline for vstest.console, sample project, target
framework etc.

Expected behavior
The -ServicePrincipal switch has been removed in version 1.5.1902.1:
Definition          :
Connect-PartnerCenter -ApplicationId  [-Environment ] [-TenantId
] []
                  Connect-PartnerCenter -AccessToken <string> -ApplicationId <string> [-Environment
                  <EnvironmentName>] [-TenantId <string>] [<CommonParameters>]

                  Connect-PartnerCenter -Credential <pscredential> -TenantId <string> [-ApplicationId <string>]
                  [-Environment <EnvironmentName>] [<CommonParameters>]

Share the expected output

Actual behavior

What is the behavior observed?

Diagnostic logs

Please share test platform diagnostics logs.
The logs may contain test assembly paths, kindly review and mask those before sharing.

Environment

Please share additional details about your environment.
Version

Hi @Agazoth,
Thank you for bringing this to my attention. The necessary changes have been made with the release of version 1.5.1902.2. Please let us know if you find any other issues.


bug: OAuthCallbackError: OAuth Provider returned an error

Provide environment information
System:
OS: Linux 5.15 Ubuntu 20.04.6 LTS (Focal Fossa)
CPU: (16) x64 AMD Ryzen 7 3700X 8-Core Processor
Memory: 5.78 GB / 9.72 GB
Container: Yes
Shell: 5.8 - /usr/bin/zsh
Binaries:
Node: 20.11.0 - ~/.nvm/versions/node/v20.11.0/bin/node
npm: 10.2.4 - ~/.nvm/versions/node/v20.11.0/bin/npm
pnpm: 9.0.0-alpha.1 - ~/.local/share/pnpm/pnpm
bun: 1.0.14 - ~/.local/share/pnpm/bun
Watchman: 2<IP_ADDRESS> - /home/linuxbrew/.linuxbrew/bin/watchman
Describe the bug
https://github.com/nextauthjs/next-auth/issues/9966
Link to reproduction
https://github.com/t3-oss/create-t3-turbo
To reproduce
click 'Sign in with Discord'. click the cancel button on the Discord signin page. see error.
Additional information
it should send back to something like https://turbo.t3.gg/api/auth/signin?error=OAuthCallbackError? instead it is this http://localhost:3000/api/auth/callback/discord?error=access_denied&error_description=The+resource+owner+or+authorization+server+denied+the+request

Make sure you're on beta.9, then remove the AUTH_URL env var, it is not needed and causes some issues still...

Make sure you're on beta.9, then remove the AUTH_URL env var, it is not needed and causes some issues still...

I did, still getting the error

gonna need a different reproduction then since https://turbo.t3.gg works and signs you in properly..

gonna need a different reproduction then since https://turbo.t3.gg works and signs you in properly..

right. works in production, but not in development for me. didn't realize when I first made the issue. sorry about that.

still gonna need some more info cause repo works fine for me. what envs do you have (obv don't give me secrets), can you verify the version is beta.9 in pnpm-lock? cleared node_modules etc (run pnpn clean:workspaces)?
DB_HOST='aws.connect.psdb.cloud'
DB_NAME='personal'
DB_USERNAME=''
DB_PASSWORD='pscale_pw_'

AUTH_SECRET=""
AUTH_DISCORD_ID=""
AUTH_DISCORD_SECRET=""

https://github.com/t3-oss/create-t3-turbo/assets/51714798/33f05eb0-3c8c-48bd-9ba9-3629aaba95f2

also remove the proxy url if you have that set. not sure why that was in the example env in the first place as it's strictly to get preview envs working and not needed in dev 😅

On the Discord page does it error when you click the cancel button instead of authorize? login works, but the cancel errors on development for me. I already removed the proxy secret.

https://www.loom.com/share/7d8d95668bc8422b99c9f28b45197ec2?sid=f68db43b-4004-45d3-83b0-0b3cc403ab1b

export const runtime = "edge"; in [...nextauth]/route.ts is causing this error

Cant see how that's related to the update. What database do you use? Planetscale is edge compat so it's only if you've changed the db provider

I am using Planetscale.  Then we can close this issue as the template doesnt do that and the issue is tracked upstream.

Are you returning false in your signin? Then we can close this issue as the template doesnt do that and the issue is tracked upstream.

I haven't changed any code other than adding my env vars. All I know is when I cancel the Discord sign-in flow in development, with edge set, I get the error with a fresh clone of this template.

I see - let's keep it open then until it's fixed upstream (probably beta.10). I dont think there's anything wrong with our code here though

I should also mention that my issue (https://github.com/nextauthjs/next-auth/issues/9869) originated with t3 turbo but i reproduce it with a normal next app. Basically any time the auth flow is not successful for any reason on the edge runtime it throws a type error that I cannot catch/handle no matter what I try. Same behaviour but not edge just redirects to the default error page of next auth or you cant catch/handle normally. Nothing much to do with t3 turbo.

Related but different - why is AUTH_URL no longer needed? Is that specific to this deployment pattern or the underlying libraries? For context, I'm deploying through a docker container (GCP Cloud Run) + while my redirect_uri is properly set, I get routed to https://<IP_ADDRESS>:3000/api/auth/error?error=CallbackRouteError after I sign in with the provider (Google, in my case).
Any thoughts?


Injecting UserManager<> in class library

Hi All,
I have defined a class library that is used by other components of the application. The library contains my model classes and all of them implement a persist method that create a database record or update an existing record in the database.   My User class extends the IdentityUser class the persist class add new users correctly but if the user already exist everything becomes trickier. I though of using the UpdateAsync Method of the UserManager class to update the user but I can't figure out a way to inject/ correctly create a UserManager inside my User c;lass.
Here is my services configuration `implementation:
`public void ConfigureServices(IServiceCollection services)
        {
            // Add framework services.
            services.AddApplicationInsightsTelemetry(Configuration);

            services.AddDbContext<ApplicationDbContext>(options =>
                options.UseSqlServer(Configuration.GetConnectionString("DefaultConnection")));

            services.AddIdentity<User, IdentityRole>(config =>
            {
                config.User.RequireUniqueEmail = true;
            } ).AddEntityFrameworkStores<ApplicationDbContext>()
                .AddDefaultTokenProviders();

            var builder = services.AddMvc();
            builder.AddMvcOptions(o => o.Filters.Add(new GlobalExceptionHandling()));
            // Add application services.
            services.AddTransient<IEmailSender, AuthMessageSender>();
            services.AddTransient<ISmsSender, AuthMessageSender>();
            services.AddScoped<IUserManager, CustomUserManager>();
            services.AddTransient<UserManager<User>>();
            services.AddTransient<User>();
            services.Configure<AuthMessageSenderOptions>(Configuration);
        }

I defined a constructor to help inject a user manager as I'll do within a controller class by that does not seem to work. Please do you guys have any points how i could handle this?

@HaoK

You are trying to have a User ctor that takes a UserManager?
User(UserManager<TUser> manager)?  Why do you need the manager in your user?

I need a UserManager<User> so I can update(via the UpdateAsync method) an existing user in the database. context.User(this).State = Microsoft.EntityFrameworkCore.EntityState.Modified; followed by context.SaveChanges() threw an exception.
I tried to inject the UserManager<User> via the User ctor but the constructor with one argument was never called I guess because I am doing something like User user = new User(). before calling my persist method.

@evindj Encapsulating persistence functionality within a model class is not a good approach. You'll find your application is much easier to maintain if you keep your models as simple as possible, and use a separate Manager (or just DbContext) to handle persistence.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


feat(Sources): Add Sources component

[x] Review with fresh eyes - make any changes
[x] What name do we want?
[x] Ask about header/text treatments - Kayla's header is not a link and the text is a non-standard non-tokened color

Preview: https://virtual-assistant-pr-virtual-assistant-191.surge.sh

Designed approved here (adjusted icons per Andrew's recommendation): https://redhat-internal.slack.com/archives/C073EUCDDQC/p1728419470642589


Dark mode white flash

How can we avoid that white flash when browsing in dark mode?
I'm not sure how the dark mode is activated, but it looks like each request renders the pages in white mode and then it's switched to dark, which is quite confusing.

@jaimeiniesta ill take a look at this too. I was able to eliminate the white flash on initial implementation, but it seems to be back again about half the time the page is reloaded.

Closed by #89


service/header: implements HeaderSub

This PR implements a structure for HeaderSub, exposing an interface for any service / component that needs access to new ExtendedHeaders to be able to subscribe to new ExtendedHeader events.
Resolves #164.
Related to #24.

This PR depends on #148 being merged first.

Waiting on @liamsi for conversation resolution


issue fixed #774 (adding import file option in sketch page)

resolved #774
An import file option has been added in the sketch page on the horizontal bar. Only file type of jpeg/jpg/png/webp can be imported . for invalid file format an alert box will open .

Nice work @anshika208

here the gif that shows that d2ps loading feature was broken by your changes
It also shows that you have overwritten the d2ps loading code to convert it to your image loading code

This is the error that comes up (when d2ps file loading fails)


Changing the default 60 second scoring timeout

ML web services (at least those deployed to ACI or AKS) apparently have a time limit of 60 seconds for the processing of a scoring request. Is this configurable?
(although AksWebservice.deploy_configuration() accepts a scoring_timeout_ms parameter, the 60 second limit is still in effect even if this parameter is raised higher.)

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

ID: 38e267fb-efa2-dab5-aa83-4a4addc39ee8
Version Independent ID: e1d506d0-c6bf-47ed-a12c-7bb50416c865
Content: Deploy models as web services - Azure Machine Learning service
Content Source: articles/machine-learning/service/how-to-deploy-and-where.md
Service: machine-learning
GitHub Login: @aashishb
Microsoft Alias: aashishb

@jberezanski-mdg Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@aashishb Hi, I couldn't find anything in our document about the configuration, is there any other reference we are using or can we add more details about it? Thank you!

As mentioned in the post above scoring_timeout_ms parameter under AksWebservice.deploy_configuration() should increase the timeout. I will log a bug if this parameter is not working as expected

Is there a reason you want to set the timeout higher than 60 secs for a real time service?

@jberezanski-mdg
The scoring_timeout_ms parameter still has a maximum time of 60 seconds for scoring. This setting is meant only to lower that timeout so that a customer can have a request fail much quicker.

@YutongTie-MSFT - #please-close  we will update the SDK documentation to be more specific for this parameter.

@j-martens Thank you for the clarification.
@aashishb  A higher timeout would give us the ability to switch between models at runtime, which would make our dev/test cycle quicker (building and deploying a new image takes 15-20 minutes). Our models take 60-80 seconds to load (on a NC6 VM) and take over the entire GPU, so only one model can be loaded at once. We accept that the first request which needs to load a new model would be slow.

@j-martens Is it safe to update the scoring timeout ms by updating the environment variable SCORING_TIMEOUT_MS when using AciWebservice.deploy_configuration?

Yes, @maxvfischer that is safe

@j-martens Is it safe to update the scoring timeout ms by updating the environment variable SCORING_TIMEOUT_MS when using AciWebservice.deploy_configuration?

I know it is a long time ago, but did this work? When I try it now it doesn't change the environment variable in the Container

There's no SCORING_TIMEOUT_MS in AciWebservice, it's ony in Aks.
This issue is still unresolved !


Fix leaking handle in edge cases.

Found via static analysis - file gets leaked in a couple of edge cases.

Nice catch, thank you!


Spec: fix string.join example

string is not iterable, thus it cannot be used as an argument to
string.join.

Changed.


Fix Bazaar card padding

If businesses.name is only one line they all look the same.

Closes #1346

But if you want businesses.name to show all of them. We can also do it like this.
But at the end of the other cards there will be a very ugly gap.


Hint user to use sudo if running build as unprivileged user

Convention or laziness, I'll let you decide... Thanks for this!


Harbor  configuration preheating, error reported "500 Internal Server Error {"message":"READONLY You can't write against a read only replica."}"

Bug report:
P2P provider is health; Error reported when performing preheating operation

Environment:

Dragonfly version: CHART VERSIONS 1.1.67
OS: ubuntu 20.04
Kernel (e.g. uname -a): Linux harbor.xnwl.com 5.4.0-171-generic #189-Ubuntu SMP Fri Jan 5 14:23:02 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Others: containerd containerd.io 1.6.33 d2d58213f83a351ca8f528a95fbd145f5654e957

harbor log

habor log

Normal after replacing Dragonfly version: CHART VERSIONS 1.1.66

@monkey-19890212 You can try the latest version of the charts.


More extensible way of doing validations

Any thoughts or plans or making custom validations more powerful such as allowing sql queries to be executed during the validation?
For example this is how to check if a record is unique in the database (using a ozzo-validation before c5ea90):
// UniqueRecord checks that a value is unique value in the database.
func UniqueRecord(statement string, bindvars ...interface{}) *uniqueRecord {
	return &uniqueRecord{
		message:   "Value is not unique.",
		statement: statement,
		bindvars:  bindvars,
	}
}

type uniqueRecord struct {
	message   string
	statement string
	bindvars  []interface{}
}

func (v *uniqueRecord) Validate(value interface{}, context interface{}) error {
	var exists bool
	statement := "SELECT EXISTS(" + v.statement + ")"
	err := database.SQL.Get(&exists, statement, v.bindvars...)
	if err != nil || exists {
		return errors.New(v.message)
	}
	return nil
}

To validate:
Add("Email", validation.Required, is.Email, 	UniqueRecord(
  "SELECT 1 FROM email WHERE email=$1", invitation.Email)
)

This is no longer possible with the updates.
It would be great being able to more than data-type validations and handle SQL queries validation and more complex struct validation such as check if two struct keys are not equal.

Actually never mind, I just realized that we don't depend on context anymore to do the SQL validation.


color key labels not right-aligned with PyFerret fonts

Reported by andrew wittenberg on 10 Jun 2016 15:50 UTC
With the Hershey font, the color key labels are nicely right-aligned.  But after SET TEXT/FONT, labels with different numbers of digits, or with a negative sign in front, are not aligned -- making things look a little messy.  Plot attached.
 	NOAA/PMEL TMAP
 	PyFERRET v7 (beta/opt)
 	Linux 2.6.32-573.22.1.el6.x86_64 - 06/08/16
 	10-Jun-16 11:43

yes? use monthly_navy_winds
yes? set view ll; shade/l=1/lev=(-30,30,2) uwnd
yes? set text/font=arial/bold
yes? set view lr; shade/l=1/lev=(-30,30,2) uwnd

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2440

Comment by ansley.b.manke on 10 Jun 2016 17:37 UTC
The color key labels are right-justified using the number of characters in the label, which is not accurate for these proportional fonts. Instead we need to use a string-width computation for locating them.

Comment by ansley.b.manke on 13 Jun 2016 23:25 UTC
This fix also allows me to check for colorbar labels that run off the page, and correct for that.
I'll attach the output of this script:
yes? use monthly_navy_winds

yes? set view ul; shade/l=1/title=Hershey uwnd

yes? set text/font=arial/bold
yes? set view ur; shade/l=1/title="Ariel Bold" uwnd

yes? set text/font=tahoma/italic/bold
yes? set view ll; shade/l=1/title="Tahome Italic Bold" uwnd

yes? set text/font=courier/bold
yes? set view lr; shade/l=1/title="Courier Bold" uwnd

Comment by andrew.wittenberg on 9 Aug 2016 02:59 UTC
In addition, the new code causes problems when using PPL SHAKEY to set the attributes of the color key.  PyFerret puts the labels on the wrong side of the key!
 	NOAA/PMEL TMAP
 	FERRET v7.01 (beta/optim)
 	Linux 2.6.32-642.1.1.el6.x86_64 64-bit - 07/29/16
 	 8-Aug-16 22:49

yes? use coads_climatology
yes? def view/x=.2:.8/y=.2:.8 myview; set view myview; go magnify
yes? shade/l=1/set sst
yes? ppl shakey 1 1 0.1 0 3 9 `1.05*($vp_width)` `1.1*($vp_width)` 0 `($vp_height)`
yes? ppl shade
yes? frame/file=ferret2.gif

 	NOAA/PMEL TMAP
 	PyFERRET v7.01 (beta/optim)
 	Linux 2.6.32-642.1.1.el6.x86_64 - 07/29/16
 	 8-Aug-16 22:48

yes? use coads_climatology
yes? def view/x=.2:.8/y=.2:.8 myview; set view myview; go magnify
yes? shade/l=1/set sst
yes? ppl shakey 1 1 0.1 0 3 9 `1.05*($vp_width)` `1.1*($vp_width)` 0 `($vp_height)`
yes? ppl shade
yes? frame/file=pyferret2.png

Modified by andrew.wittenberg on 9 Aug 2016 03:00 UTC

Comment by ansley.b.manke on 9 Aug 2016 16:14 UTC
The same code is used at the plot-plus level, locating the text on the page, for both PyFerret and classic Ferret, but the drawing of text is different.  It looks like this will need two paths in the plot-plus code, restoring the previous stuff for classic Ferret, and keeping the recent changes for PyFerret.

Comment by ansley.b.manke on 9 Aug 2016 19:46 UTC
In fact the code is not the same in this case, because of how the graphics packages do scaling to put items on the page.
So, fixes are needed for both PyFerret and Ferret.
Both of these are fixed.  I'll attach the results of the script below which shows both issues in two viewports.
In Ferret, the location of the labels was correct, but they were not right-justified; both panels.
In PyFerret, the labels were justified, but when PPL SHAKEY customizes the key, they were not located correctly; upper panel.  Using a negative label-size argument to put the key labels on the other side of the colorbar is also working. (One could argue that those should be left-justified, but I have not done that.)
set win/asp=1
use coads_climatology

def view/x=.2:.8/y=0:0.5 mylo; set view mylo
shade/l=1 sst

def view/x=.2:.8/y=.6:.9 myhi; set view myhi; go magnify
shade/l=1/set sst
ppl shakey 1 1 0.1 2 3 9 `1.05*($vp_width)` `1.1*($vp_width)` 0 `($vp_height)`
ppl shade; frame/file=($program_name)_keylab.gif

Attachment from andrew.wittenberg on 10 Jun 2016 15:50 UTC
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_2440_GIT_1712/plot.png

Attachment from ansley.b.manke on 13 Jun 2016 23:26 UTC
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_2440_GIT_1712/fix2440.png

Attachment from andrew.wittenberg on 9 Aug 2016 02:11 UTC
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_2440_GIT_1712/ferret.gif

Attachment from andrew.wittenberg on 9 Aug 2016 02:11 UTC
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_2440_GIT_1712/pyferret.png

Attachment from andrew.wittenberg on 9 Aug 2016 03:00 UTC
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_2440_GIT_1712/ferret2.gif

Attachment from andrew.wittenberg on 9 Aug 2016 03:00 UTC
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_2440_GIT_1712/pyferret2.png

Attachment from ansley.b.manke on 9 Aug 2016 19:48 UTC
2-panel plot, comment 7
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_2440_GIT_1712/Ferret_keylab.gif

Attachment from ansley.b.manke on 9 Aug 2016 19:48 UTC
2-panel plot, PyFerret, for comment 7
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_2440_GIT_1712/PyFerret_keylab.png


Verification function for the convert measures rake task

A validation function was created to verify the measure converter rake task properly converted the measures into CQM format. All attributes were verified in detail except for "cql_libraries" which was only validated based on the number of elements, and "population_sets" which wasn't verified at all.
After running the rake task with the validator on the database collection of CQL measures, there were two validation notes that came up due to value sets. Here are the messages:
Conversion Difference
Measure CMS645v2: Bone density evaluation for patients with prostate cancer and receiving androgen deprivation therapy with _id 5a97f802b848467fb6a92e3c in account<EMAIL_ADDRESS>--- value_sets --- Is different from CQL measure
Conversion Difference
Measure CMSv0: mesg12 with _id 5bfe3673b84846165cd1d1fe in account<EMAIL_ADDRESS>--- value_sets --- Is different from CQL measure
Pull requests into Bonnie require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Submitter:

[x] This pull request describes why these changes were made.
[x] This PR is into the correct branch.
[x] JIRA ticket for this PR: https://jira.mitre.org/browse/BONNIE-1837
[x] JIRA ticket links to this PR
[ ] Code diff has been done and been reviewed (it does not contain: additional white space, not applicable code changes, debug statements, etc.)
[x] If UI changes have been made, google WAVE plug-in has been executed to ensure no 508 issues were introduced. - N/A
[x] Tests are included and test edge cases - N/A
[ ] Tests have been run locally and pass (remember to update Gemfile when applicable)
[x] Test fixtures updated and documented as necessary ( see internal wiki ) - N/A
[x] Code coverage has not gone down and all code touched or added is covered.

In rare situations, this may not be possible or applicable to a PR. In those situations:

Note why this could not be done or is not applicable here:
Add TODOs in the code noting that it requires a test
Add a JIRA task to add the test and link it here:

[x] Automated regression test(s) pass - N/A

Reviewer 1:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

If JIRA tests were used to supplement or replace automated tests:

[ ] JIRA tests have been run and pass
[ ] You agree with the justification for use of JIRA tests or have provided input on why you disagree

Reviewer 2:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

If JIRA tests were used to supplement or replace automated tests:

[ ] JIRA tests have been run and pass
[ ] You agree with the justification for use of JIRA tests or have provided input on why you disagree

If you want to run the rake task and only convert one of the measures listed in the description, temporarily replace line 131 in cql.rake with one of the following:
bonnie_cql_measures = CqlMeasure.where(_id: BSON::ObjectId("5a97f802b848467fb6a92e3c"))
bonnie_cql_measures = CqlMeasure.where(_id: BSON::ObjectId("5bfe3673b84846165cd1d1fe"))


[@uform/antd] time field will throw an error while clear button was clicked

Help us to manage our issues by answering the following:

How would you tag this issue?

Bug

Describe your issue:

What is the current behavior?

Create a form with a date or daterange field
Pick a date or daterange, then click clear button
Throw errors as below

Warning: Failed prop type: Invalid prop `value` of type `string` supplied to `Picker`, expected `object`.

Warning: Failed prop type: Invalid prop `value` of type `string` supplied to `Panel`, expected `object`.

What is the expected behavior?
No errors
Which versions of UForm?
0.4.3
Did this work in previous versions of UForm?
No

If your issue is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via Bug Report Template on CodeSandbox.

https://codesandbox.io/s/uform-lizi-8kcuc

Fixed in #372


open-vm-tools: 12.2.5 -> 12.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/vmware/open-vm-tools/releases.
meta.description for open-vm-tools is: Set of tools for VMWare guests to improve host-guest interaction
meta.homepage for open-vm-tools is: https://github.com/vmware/open-vm-tools
meta.changelog for open-vm-tools is: https://github.com/vmware/open-vm-tools/releases/tag/stable-12.3.0
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 12.3.0 with grep in /nix/store/fz981m14bn1x8w75bcpxrpqfcmd1nv9z-open-vm-tools-12.3.0
found 12.3.0 in filename of file in /nix/store/fz981m14bn1x8w75bcpxrpqfcmd1nv9z-open-vm-tools-12.3.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
open-vm-tools
open-vm-tools-headless

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fz981m14bn1x8w75bcpxrpqfcmd1nv9z-open-vm-tools-12.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A open-vm-tools https://github.com/r-ryantm/nixpkgs/archive/63e43aaa9f8a1b370130495c3f4c2d1c8428b391.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fz981m14bn1x8w75bcpxrpqfcmd1nv9z-open-vm-tools-12.3.0
ls -la /nix/store/fz981m14bn1x8w75bcpxrpqfcmd1nv9z-open-vm-tools-12.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

open-vm-tools
open-vm-tools-headless

Maintainer pings
cc @joamaki @kjeremy for testing.

(arriving here because the CVE related to the replaced version is throwing results in my scanner)
@kjeremy based on the release notes it looks like this is primarily a fix release and I couldn't find anything breaking in the release notes. They deprecated something to do with xml-security-c but it doesn't appear anywhere in the package here so methinks it's all good :+1:


Investigate and remove useless pointers

Investigate use of pointer in ZetaClient specifically and remove when not necessary
Example: https://github.com/zeta-chain/node/pull/2032#discussion_r1568866883

Closing, as we're revamping ZetaClient


Hotfix for #4382

Fixes a few minor nits with PR #4382, listed below.

Corrected cache to update ApprovedLanguages rather than languages. (Languages being the language filter for reviews, ApprovedLanguages being the preferred approval languages)
Changed approvers to PreferredApprovers to avoid confusion with existing Approvers list (People you can request/approve vs those who have approved)
Default behavior if no languages selected changed from never showing to always showing (old behavior)

Functionality unchanged otherwise

@praveenkuttappan Please review. The main change here is to introduce a new default behavior for approvers who have not specified any 'review languages' in their profile. Previously, it meant that approvers could not approve any libraries. This has now changed so that approvers with no 'review languages' retain the existing behavior of being able to approve all libraries for any language.


Added 2 figures to visually explain how the decoder inputs are embedded

*title correction: ENCODER, not decoder. sorry
When working through the implementation, I found it was really difficult to understand the model without being able to visualize the tensors we're feeding to the encoder and decoder. This hopefully makes it easier for users to understand the encoder input at a glance.

Hi @jalammar,
Sorry for my late reply. We do appreciate a lot when you spend time drawing the figures for us. Unfortunately, I can't accept this PR as I'd like to maintain consistent style across figures and also need to make more thorough updates to the text. We'll take note of these figures if we update the tutorial.
Thanks again! Close this for now.

Many thanks for the response and consideration Dr. Luong. I completely understand. I'd be ecstatic to contribute to visualizing any of the concepts involved in the future. Just let me know. Cheers!


Enable use of pre-commit for linting

pre-commit is framework used to orchestrating execution of many small
utilities (usually linters) which is also used by tox itself and should
not be seen as a replacement for it.
Despite its name pre-commit does not require installation of a git hook
in order to be used, this being an optional feature of the tool.
Among benefits we can count:

lower disk footprint as cached linters are reused across repos
must faster (even on CI due to ~/.cache/pre-commit)
avoids conflicts between linters, each of them being isolated
can run only changed files (w/o --all-files flag)
pinned dependencies by design
one line bumping support pre-commit autoupdate
optional git hook (pre-commit install)
allows user to run a single linter if they want

This change does not require any workflow changes for developers, as
tox -e lint will still run the linters.
Signed-off-by: Sorin Sbarnea<EMAIL_ADDRESS>PR Type

Feature Pull Request

I know next to nothing about this tool.

I hope you'll like it as we do :)


undefined symbols when linking new c++ lib

OS
Ubuntu 24.04
Programming Language
C++
VPK Version
vpk 0.0.840
Library Version
Nuget 0.0.840
What happened?
Linking with cmake the velopack_libc_linux_x64_gnu.so and using the Velopack.h and I'm getting undefined symbols
Relevant log output
ld.lld: error: undefined symbol: vpkc_set_log
>>> referenced by CLIApp.cpp:75 (/home/fran6co/code/apg/libraries/AppFramework/apg/CLIApp.cpp:75)
>>>               CLIApp.cpp.o:(apg::cliAppMain(int, char**, std::shared_ptr<apg::CLIApp> const&)) in archive libraries/AppFramework/libAppFramework.a

ld.lld: error: undefined symbol: vpkc_app_run
>>> referenced by Velopack.h:504 (/opt/apg/cmake_deps/velopack-0.0.840-src/include/Velopack.h:504)
>>>               CLIApp.cpp.o:(apg::cliAppMain(int, char**, std::shared_ptr<apg::CLIApp> const&)) in archive libraries/AppFramework/libAppFramework.a

ld.lld: error: undefined symbol: vpkc_new_update_manager
>>> referenced by Velopack.h:525 (/opt/apg/cmake_deps/velopack-0.0.840-src/include/Velopack.h:525)
>>>               CLIApp.cpp.o:(apg::cliAppMain(int, char**, std::shared_ptr<apg::CLIApp> const&)) in archive libraries/AppFramework/libAppFramework.a

ld.lld: error: undefined symbol: vpkc_download_updates
>>> referenced by Velopack.h:574 (/opt/apg/cmake_deps/velopack-0.0.840-src/include/Velopack.h:574)
>>>               CLIApp.cpp.o:(apg::cliAppMain(int, char**, std::shared_ptr<apg::CLIApp> const&)) in archive libraries/AppFramework/libAppFramework.a

ld.lld: error: undefined symbol: vpkc_free_update_manager
>>> referenced by Velopack.h:530 (/opt/apg/cmake_deps/velopack-0.0.840-src/include/Velopack.h:530)
>>>               CLIApp.cpp.o:(apg::cliAppMain(int, char**, std::shared_ptr<apg::CLIApp> const&)) in archive libraries/AppFramework/libAppFramework.a
>>> referenced by Velopack.h:530 (/opt/apg/cmake_deps/velopack-0.0.840-src/include/Velopack.h:530)
>>>               CLIApp.cpp.o:(apg::cliAppMain(int, char**, std::shared_ptr<apg::CLIApp> const&)) in archive libraries/AppFramework/libAppFramework.a

ld.lld: error: undefined symbol: vpkc_get_last_error
>>> referenced by Velopack.h:218 (/opt/apg/cmake_deps/velopack-0.0.840-src/include/Velopack.h:218)
>>>               CLIApp.cpp.o:(Velopack::throw_last_error()) in archive libraries/AppFramework/libAppFramework.a
>>> referenced by Velopack.h:220 (/opt/apg/cmake_deps/velopack-0.0.840-src/include/Velopack.h:220)
>>>               CLIApp.cpp.o:(Velopack::throw_last_error()) in archive libraries/AppFramework/libAppFramework.a

ld.lld: error: undefined symbol: vpkc_check_for_updates
>>> referenced by Velopack.h:558 (/opt/apg/cmake_deps/velopack-0.0.840-src/include/Velopack.h:558)
>>>               CLIApp.cpp.o:(Velopack::UpdateManager::CheckForUpdates()) in archive libraries/AppFramework/libAppFramework.a

ld.lld: error: undefined symbol: vpkc_free_update_info
>>> referenced by Velopack.h:567 (/opt/apg/cmake_deps/velopack-0.0.840-src/include/Velopack.h:567)
>>>               CLIApp.cpp.o:(Velopack::UpdateManager::CheckForUpdates()) in archive libraries/AppFramework/libAppFramework.a

ld.lld: error: undefined symbol: vpkc_wait_exit_then_apply_update
>>> referenced by Velopack.h:581 (/opt/apg/cmake_deps/velopack-0.0.840-src/include/Velopack.h:581)
>>>               CLIApp.cpp.o:(Velopack::UpdateManager::WaitExitThenApplyUpdate(Velopack::VelopackAsset const&, bool, bool, std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>>>)) in archive libraries/AppFramework/libAppFramework.a
clang++: error: linker command failed with exit code 1 (use -v to see invocation)
ninja: build stopped: subcommand failed.

I have looked for those symbols in the shared lib and I can't find them nm -D velopack_libc_linux_x64_gnu.so | grep vpkc_set_log comes out empty

Thank you for reporting this. The binary is compiled by Rust and I am fairly new to creating / producing C libraries in general. I am also a Windows user so the only working sample currently is on Windows :) Do you have any thoughts on why the symbols are not available on Linux? I saw a suggestion that I may be missing a link arg --export-dynamic.
For reference, the symbols are defined here: bridge.cc


Consent_date is sometimes 0000-00-00 00:00:00

This issue is imported from pivotal - Originaly created at Aug 21, 2018 by Thijs Kinkhorst
In the consent database table, there are about 50.000 entries (0.8%) that have a value of "0000-00-00 00:00:00".
We noticed because Profile throws a critical error when you have one such consent in your consents:
request.CRITICAL: Uncaught PHP Exception OpenConext\Profile\Exception\AssertionFailedException: "Consent given on date must be formatted according to the ISO8601 standard, got "-0001-11-30T00:00:00+00:19"" at /opt/openconext/OpenConext-profile-1.1.2/vendor/beberlei/assert/lib/Assert/Assertion.php line 260

What could make that consents of 0000-00-00 00:00:00 are added to the consent table?
Salient detail is that the Profile error logs only started to appear on 14 Aug. This would suggest that the invalid dates have started to appear then.
On 14 aug from 5.30-6.30 we ran with EB 5.7 which was then rolled back to 5.6. I cannot explain how that is relevant.

See: [367446527cc4560bb2a5c92649f00141b29c12c3](https://github.com/OpenConext/OpenConext-engineblock/commit/367446527cc4560bb2a5c92649f00141b29c12c3)

Could it be that the 5.6 rollback was performed successfully, but the database schema was not updated? Migration Version20180221133212 down should be run to ensure the current date is set on the consent.consent_date field. Is it possible to verify this?
In v 5.6 the schema should look like:
MariaDB [eb]> DESCRIBE consent;
+----------------+--------------+------+-----+-------------------+-------+
| Field          | Type         | Null | Key | Default           | Extra |
+----------------+--------------+------+-----+-------------------+-------+
| consent_date   | datetime     | NO   |     | CURRENT_TIMESTAMP |       |
| hashed_user_id | varchar(80)  | NO   | PRI | NULL              |       |
| service_id     | varchar(255) | NO   | PRI | NULL              |       |
| attribute      | varchar(80)  | NO   |     | NULL              |       |
| consent_type   | varchar(20)  | YES  |     | explicit          |       |
+----------------+--------------+------+-----+-------------------+-------+
5 rows in set (0.00 sec)

And in 5.7:
+----------------+--------------+------+-----+----------+-------+
| Field          | Type         | Null | Key | Default  | Extra |
+----------------+--------------+------+-----+----------+-------+
| consent_date   | datetime     | NO   |     | NULL     |       |
| hashed_user_id | varchar(80)  | NO   | PRI | NULL     |       |
| service_id     | varchar(255) | NO   | PRI | NULL     |       |
| attribute      | varchar(80)  | NO   |     | NULL     |       |
| consent_type   | varchar(20)  | YES  |     | explicit |       |
+----------------+--------------+------+-----+----------+-------+
``` (Michiel Kodde - Aug 21, 2018)

Was an incomplete DB migration on rollback. Not a bug in EB. (Thijs Kinkhorst - Sep 4, 2018)


tests(swamp): correct fraud syncing test

Previously it succeeded, but the FP was actually transmitted through PubSub and not fraud syncing.
It also makes the test discovery independent.

Codecov Report

Merging #1555 (73c0006) into main (7e4f27a) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1555      +/-   ##
==========================================
+ Coverage   55.09%   55.11%   +0.02%
==========================================
  Files         211      211
  Lines       12953    12953
==========================================
+ Hits         7136     7139       +3
+ Misses       5091     5089       -2
+ Partials      726      725       -1

Impacted Files
Coverage Δ

share/ipld/get.go
93.41% <0.00%> (+1.23%)
:arrow_up:

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

flaky in CI

@distractedm1nd, https://github.com/celestiaorg/celestia-node/pull/1554 fixes one of the flaky sources


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1669 ms

Resolved: Bofrost.lu performance has improved in b07d87c after 8 minutes.


[test] Use default credentials if none specified

Codecov Report

Merging #357 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #357      +/-   ##
============================================
- Coverage     46.78%   46.76%   -0.02%
  Complexity     2639     2639
============================================
  Files           630      630
  Lines         16806    16806
  Branches       1243     1243
============================================
- Hits           7862     7860       -2
- Misses         8480     8481       +1
- Partials        464      465       +1

Impacted Files
Coverage Δ
Complexity Δ

...m/spotify/heroic/aggregation/simple/MaxBucket.java
42.85% <0%> (-9.53%)
4% <0%> (-1%)

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 baedda3...b2e05c0. Read the comment docs.


Scenery Image should have nonzero dimensions when created with mipmap

@aaronsamuel137 and I saw this issue where the width and height of a homescreen icon (loaded with mipmap) were (0,0).  When dividing by zero, this causes NaN and the Frame which creates a linear gradient causes runtime error on iPad and osx Safari.  This is a flaky problem and doesn't occur every time.  But when it does occur, it crashes the application launching (even the built version).

We addressed this on a call. But I'll leave this open for you to review and close @jonathanolson.

Looks good, thanks!


Datetime comparison with SQLite driver

Describe the bug
Querying Date columns using the SQLite driver does not work as expected.
To Reproduce
@Entity()
class Something {
  constructor(public init?: Date) {
    this.updatedAt = init;
  }

  @PrimaryKey()
  id!: number;

  @Property()
  updatedAt?: Date;
}

const orm = await MikroORM.init({
  entities: [Something],
  name: 'sqlite_test',
  dbName: `test_sqlite_dates`,
  type: 'sqlite',
  debug: ['query', 'query-params'],
  forceUtcTimezone: true,
});

const generator = orm.getSchemaGenerator();
await generator.dropSchema();
await generator.createSchema();

const s1 = new Something(new Date(Date.UTC(2020, 10, 15)));
const s2 = new Something(new Date(Date.UTC(2020, 10, 20)));

await orm.em.persistAndFlush([s1, s2]);

const earlier = await orm.em.getRepository(Something).find({ updatedAt: { $lt: new Date(Date.UTC(2020, 10, 18)) } });
console.log(earlier.length); // expected: 1; actual: 2
const later = await orm.em.getRepository(Something).find({ updatedAt: { $gt: new Date(Date.UTC(2020, 10, 18)) } });
console.log(later.length); // expected 1; actual: 0

Generated SQL:
create table `something` (`id` integer not null primary key autoincrement, `updated_at` datetime null);
insert into `something` (`updated_at`) values ('2020-11-14 16:00:00.000')
insert into `something` (`updated_at`) values ('2020-11-19 16:00:00.000')
select `e0`.* from `something` as `e0` where `e0`.`updated_at` < '2020-11-17 16:00:00.000'

Additional context
We're using the sqlite driver to run our test suites, replicating the production mysql schema. Some of our custom repositories do include date filtering and a function that may work properly on mysql would fail in a spec if we have a date column in the where clause.
From what I collected googling around, this is a well known limitation of sqlite, however it's possible to workaround the issue by storing dates as timestamps as opposed to strings. Dates comparison then becomes integers comparison and obviously greater than and less than would work there. It sounds like a data transformation that the orm can handle.

Your reproduction is passing on my end, both calls are returning 1 item for me. What version are you on?
The problem with sqlite is that it does store JS Date objects to timestamps (number) by default, but you might end up with a string date instead if you use SQL functions for the value, e.g. now(). Then the values are no longer comparable.
Another issue is with knex - the query you see in the console is actually not what gets sent to the db, its just a way knex will serialize the Date object, but under the hood it will pass it as it to sqlite3 which will convert it to number. If you check the db, you should see that. I am currently thinking about using different way to handle the values and pass already interpolated query to the underlying driver (doing escaping at the ORM level instead of via prepared statements), that would help with this.
You can always use a custom type to handle this in a way you want.

We're still on mikro-orm 3.6, haven't been able to jump to v4 yet. Do you think that might be the reason?
We're not using any now(), we manipulate javascript dates similar to the example I provided (except we use moment().toDate()).
Custom types would work if we didn't have to handle 2 different drivers. Sure we don't care that much about what's stored in testing (sqlite) but we still want actual dates in production (mysql).

I get actual formatted string when I query the db directly
sqlite> select * from `something`;
1|Sat Nov 14 2020 16:00:00 GMT-0800 (Pacific Standard Time)
2|Thu Nov 19 2020 16:00:00 GMT-0800 (Pacific Standard Time)

What version of sqlite3 are you using? Sounds like you upgraded to v5, but they introduced one breaking change with dates, they are no longer stored numbers. In v4 the version is fixed to 4.2.

✗ sqlite3 --version
3.28.0 2019-04-15 14:49:49 378230ae7f4b721c8b8d83c8ceb891449685cd23b1702a57841f1be40b5daapl

I'm not sure about the node-sqlite3 version but I think you might be right 🤔

Is there a way to force sqlite version 4.2 from mikro-orm 3.6?

Just remove the ^ from package.json and fix the version.
https://github.com/mikro-orm/mikro-orm/blob/master/packages/sqlite/package.json#L52

ah yeah I was looking at sqlite cli client 🤦
Switching back to node-sqlite3 v4.2 fixed the issue. Thanks for the assistance.


ENH: override halfnorm fit

Reference issue
Part of #17832 and #11782
What does this implement/fix?
Adds an analytical fitting method for the halfnormal distribution. This one is a low hanging fruit compared to most other distributions. The default fit method works but is slow compared to the simple analytical result.
Additional information
Math
$$ p(x|\mu,\sigma)=\frac{\sqrt{2}}{\sigma\sqrt{\pi}}\exp\left(-\frac{(x-\mu)^2}{2\sigma^2}\right) $$
The log likelihood for the parameters $\mu$, $\sigma$ given data ($x_1,...,x_n$) is then
$$ \ln p(\mu,\sigma|x_1,...x_n)=n(1/2\ln(2)-1/2\ln(\pi)-\ln(\sigma))-\sum_i^n\frac{(x_i-\mu)^2}{2\sigma^2}$$
As $x>0$, this is is a monotonically increasing function for $\mu$. The maximum likelihood is reached for $\mu=x_{min}$ as for a higher $\mu$, the minimal value $x_{min}$ would be outside of the support of the distribution (similar to the halflogistic distribution in #18695).
Setting the gradient to zero yields the regular variance which depends on the location:
$$
\begin{align}
\frac{\partial\ln p}{\partial \sigma} &= -\frac{n}{\sigma}+\sum_{i=1}^n\frac{(x_i-\mu)^2}{\sigma^3}=0 \
\sigma & = \sqrt{\frac{\sum_i(x_i-\mu)^2}{n}}
\end{align}
$$
Fit results for free parameters
Details

import numpy as np
from scipy import stats
from time import perf_counter
import matplotlib.pyplot as plt
import os
rng = np.random.default_rng(2433006235492611347)
qrng = stats.qmc.Halton(6, seed=rng)
N = 1000
times_pr = np.full(N, np.nan)
times_super = np.full(N, np.nan)
nllfs_super = np.full(N, np.nan)
nllfs_pr = np.full(N, np.nan)
distribution = stats.halfnorm
for i in range(N):
print(i)
sample = qrng.random()
sample = stats.qmc.scale(sample, [-3, -3, -3, 1, -3, -1], [2, 3, 3, 5, 2, 1])[0]
_, loc, scale, size, floc = 10**sample[:5]
size = int(size)
dist = distribution(loc=loc, scale=scale)
rvs = dist.rvs(size=size, random_state=rng)

try:
    tic = perf_counter()
    loc_fit, scale_fit = distribution.fit(rvs)
    toc = perf_counter()
    nllf_pr = distribution.nnlf((loc_fit, scale_fit), rvs)
    nllfs_pr[i] = nllf_pr
    times_pr[i] = toc-tic

except Exception as e:
    print(f"Failure: {e}")

try:
    tic = perf_counter()
    loc_fit, scale_fit = distribution.fit(rvs, superfit=True)
    toc = perf_counter()
    nllf_super = distribution.nnlf((loc_fit, scale_fit), rvs)
    nllfs_super[i] = nllf_super
    times_super[i] = toc-tic

except Exception as e:
    print(f"Failure: {e}")

nllf_diff = (nllfs_pr - nllfs_super)/np.abs(nllfs_super)
nllf_diff = nllf_diff[np.isfinite(nllf_diff)]
nllf_diff_good = -nllf_diff[nllf_diff < 0]
nllf_diff_bad = nllf_diff[nllf_diff > 0]
bins = np.arange(-6, 1.5, 0.25)
plt.hist(np.log10(times_pr), bins=bins, alpha=0.5)
plt.hist(np.log10(times_super), bins=bins, alpha=0.5)
plt.legend(('PR', 'main'))
plt.xlabel('log10 of fit execution time (s)')
plt.ylabel('Frequency')
plt.title('Histogram of log10 of fit execution time (s), free')
plt.savefig("Time_free.png")
plt.clf()
plt.hist(np.log10(nllf_diff_good), label="good", alpha=0.5)
plt.hist(np.log10(nllf_diff_bad), label="bad", alpha=0.5)
plt.xlabel('log10 of NLLF differences')
plt.ylabel('Frequency')
plt.title('Histogram of log10 of NLLF differences, free')
plt.legend()
plt.savefig("NNLF_Differences_free.png")

Fit results for fixed location

Fit results for fixed scale

I haven't run this locally yet, but the math is good. What happens when floc is greater than the minimum of the data?

Good point. The minimal sample would be outside of the support of the distribution. The likelihood to observe that data point would be 0. Should we throw an error?

I think so. In pareto.fit, for example:
https://github.com/scipy/scipy/blob/2e16ec5bc62b7b7db21b37b838fc771c7d232006/scipy/stats/_continuous_distns.py#L7412-L7415


Question regarding proposal feature

I have a question regarding proposal feature.
In DETR paper, reshaped feature map (HW x C) is given as input to transformer encoder to learn correlation between each pixels.
However, in your paper, you use C size vector (named 'prop_feats') instead of reshaped feature map.
How does this C size vector learn the correlation among each pixels?
In my understanding this does not contain the feature information for each pixel position.
I saw your reply in one of the previous issues where you replied
'don't understand dynamic head as Q,K,V'. How should i understand this concept then??
Thank you in advance!

so roi feature can be regarded as Query and proposal features as Key?

I guess Query is proposal features [100 x C],  roi feature  is Key [100 x (7 x 7 x C)] .
Think about DETR, object query is 100 x C,  reshaped image feature map is 100 x (HW x C), where each  (HW x C) is the same.

@PeizeSun
Isn't that Q and K must have the same hidden dimension to process matrix multiplication, like in DETR Q is [100 x C] and K is [HW x C] instead of [100 x (HWC)]?


added scatter with colors

Thanks for contributing!
I just merged this, but I had to rename the function to scatter_colored() because it had the same signature as the scatter() for 3d points.


Evaluate javascript ?

Hello,
Is there a way to evaluate javascript code ? It's a fairly common task among all browser automation tools I used, but I didn't find it in chromedp (and the lack of docs didn't help either).
Thanks,
Howe

Yes, use the chromedp.Evaluate() action in conjuction with the chromedp/runner/Runner. Or use chromedp/cdp/runtime.Evaluate() with the frame handler.

Oh, thanks. I've missed that.
Howe

Thank you. After giving a thought for some time, I got it.


[RIP-21] submodule common & client

Make sure set the target branch to develop
What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.06%) to 53.869% when pulling 71cd0fbe856bd2c315cecf15d8416dbfae7f0772 on ayanamist:feature-logicalqueue into df1d93fc8859377b92ba87c6947911281656f355 on apache:5.0.0-preview.

Coverage increased (+0.06%) to 53.869% when pulling 71cd0fbe856bd2c315cecf15d8416dbfae7f0772 on ayanamist:feature-logicalqueue into df1d93fc8859377b92ba87c6947911281656f355 on apache:5.0.0-preview.

Codecov Report

Merging #3018 (71cd0fb) into 5.0.0-preview (df1d93f) will decrease coverage by 0.60%.
The diff coverage is 36.05%.

@@                 Coverage Diff                 @@
##             5.0.0-preview    #3018      +/-   ##
===================================================
- Coverage            47.97%   47.37%   -0.61%
- Complexity            4569     4609      +40
===================================================
  Files                  552      572      +20
  Lines                36628    37553     +925
  Branches              4844     5005     +161
===================================================
+ Hits                 17572    17789     +217
- Misses               16832    17457     +625
- Partials              2224     2307      +83

Impacted Files
Coverage Δ

...a/org/apache/rocketmq/client/impl/MQAdminImpl.java
3.72% <0.00%> (-1.39%)
:arrow_down:

...pache/rocketmq/client/latency/MQFaultStrategy.java
17.39% <ø> (ø)

...rg/apache/rocketmq/client/producer/SendResult.java
69.38% <0.00%> (-4.53%)
:arrow_down:

...java/org/apache/rocketmq/common/ConfigManager.java
59.45% <0.00%> (-7.21%)
:arrow_down:

...n/java/org/apache/rocketmq/common/TopicConfig.java
39.17% <0.00%> (-4.01%)
:arrow_down:

.../java/org/apache/rocketmq/common/TopicQueueId.java
0.00% <0.00%> (ø)

...rg/apache/rocketmq/common/constant/LoggerName.java
0.00% <ø> (ø)

...g/apache/rocketmq/common/message/MessageConst.java
96.96% <ø> (ø)

...g/apache/rocketmq/common/message/MessageQueue.java
45.45% <0.00%> (-4.55%)
:arrow_down:

...g/apache/rocketmq/common/protocol/RequestCode.java
0.00% <ø> (ø)

... and 97 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 df1d93f...71cd0fb. Read the comment docs.

Wow, it's happy to see RIP-22 get the response from a master.

Wow, it's happy to see RIP-22 get the response from a master.

Sorry for my mistake, this PR is for RIP-21 Logical Queue support, i have fixed this.

Got it.
发自我的iPhone
------------------ Original ------------------
From: ayanamist @.>
Date: Mon,Jun 21,2021 3:07 PM
To: apache/rocketmq @.>
Cc: dragon-zhang @.>, Comment @.>
Subject: Re: [apache/rocketmq] [RIP-21] submodule common & client (#3018)
Wow, it's happy to see RIP-22 get the response from a master.
Sorry for my mistake, this PR is for RIP-21 Logical Queue support, i have fixed this.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


ng2 completer

Hello,
I am working with the  angular 2. I used ng2 completer for the country names adn is working as i expected,but now i want to write the validations for it-
can anyone please help me to validate the ng2 completer - to accept only the local data not the fantasy names.

@pradeep403 take a look at https://angular.io/guide/form-validation
If you still have issues it's better to ask this question on stack overflow.


Optimize loading of meditor on dandisets with large metadata

When trying to click the "Edit Metadata" button on the Dandiset Landing Page, there's quite a delay in switching to the new view, when there's a lot of metadata on that dandiset (likely due to the large amount of component rendering / data validation that occurs). A good example of this is dandiset 000004.
We should look into ways that this can be optimized, or at the very least, display something to indicate that it is loading. The latter might not be possible, as I believe the browser is mostly running JavaScript (as opposed to drawing/rendering) during that time. More investigation is needed.

is this still a valid issue? not sure how to check it without written access to this big dataset

I think this can be closed for now.


Hack to avoid Django generating broken migration for waffle

Needed for vault 1.7 compatibility. We should also release this package to our pypi mirror.

Description of the problem. Waffle has a foreign key to settings.CUSTOM_USER_MODEL, but the migrations of waffle has a foreign key to settings.AUTH_USER_MODEL. When waffle is in installed apps and one runs makemigrations command, then Django detects the foreign key has changed, and tries to repoint the foreign key to CUSTOM_USER_MODEL. This creates a new migration for waffle.
While it isn't good to create migrations for 3rd party apps, the real problem is that the generated migration is broken. For some reason it doesn't contain dependency on core's initial migration. End result is that running migrate fails.
I've added the correct migration manually to waffle. This is bad, as now waffle essentially depends on vault. The only solution which doesn't require dependency on vault seems to be switching to using AUTH_USER_MODEL in vault.


Queries related to explicit-edge analysis

Hi. I have some queries related to results from explicit edge analysis on BAP and CFG generated from BAP.

There are some values shown along with function after vsa / explicit-edge analysis? Are they addresses of the function or offset values? (Highlighted in picture below)

For a small binary explicit -edge fails/crash for main function where jumptable is . Any pointers to debug?

Also for explicit-edge analysis to specify for a particular function I use the command bap ./switch -p explicit-edge --explicit-edge-sub main  but it still performs and prints to terminal all outputs for all functions with crashing before main function. Is there some error in my command specification? (sample output listed below)

I used this small switch binary to see jumps. After manually extracting and visualizing as xdot .. we could see that there is no edge for jump as shown below . Should it not have an edge for jump?

 I'm going to try to crawl through the code and documentation, but it might take a bit.

Thank you for taking your time on this. No problem.

I'll chime in with answers to the first 3.

The numbers displayed there are the TIDs. https://github.com/draperlaboratory/cbat_tools/blob/aa89797ae6c114a0b11e222e86f837165b10a6a8/value_set/plugin/value_set.ml#L45

Here the VSA is intentionally stopping, because (as the message says) it's trying to do a left bit shift, and one of the value-sets has a maximum element that's bigger than the width of the other value-set. We didn't implement handling of such cases, so the analysis simply stops and reports this. To get past this, try running the plugin with the --value-set-keep-trying flag. That will tell the plugin to simply use top as the result of the bitshift operation, rather than stopping.

Good question. This can be a bit hard to spot in the README for the explicit-edge plugin, but it notes that --explicit-edge-sub is different from --value-set-sub. So you need to include both flags in your command if you want the value-set to only run on main, and then you also want the explicit-edge to run on main. Like this: bap ./switch -p explicit-edge --value-set-sub main --explicit-edge-sub main

Thank you @jtpaasch for your inputs. I will work with these changes and update if I have any queries.


New package: jumanji-20140622.

Also includes webkitgtk 2.4.6 -> 2.6.0 (cf #4433) and some clean-up.

Merged the core part. The rest seems mostly obsoleted since.


Error while searching

No chrome.exe and installed all reqs
Message: element click intercepted: Element fenerbahce maci hangi kanalda is not clickable at point (929, 34). Other element would receive the click:
(Session info: chrome=127.0.6533.73)
Stacktrace:
GetHandleVerifier [0x00CBF653+11395]
(No symbol) [0x00C9B7A4]
(No symbol) [0x00BE1A43]
(No symbol) [0x00C2B5C7]
(No symbol) [0x00C299D9]
(No symbol) [0x00C278CB]
(No symbol) [0x00C26EAF]
(No symbol) [0x00C1BFAF]
(No symbol) [0x00C4719C]
(No symbol) [0x00C1BA45]
(No symbol) [0x00C47434]
(No symbol) [0x00C603DC]
(No symbol) [0x00C46F36]
(No symbol) [0x00C1A276]
(No symbol) [0x00C1AECD]
GetHandleVerifier [0x00DC2403+1071667]
GetHandleVerifier [0x00DCC9C4+1114100]
GetHandleVerifier [0x00DC54B2+1084130]
GetHandleVerifier [0x00CE013B+145259]
(No symbol) [0x00CA1693]
(No symbol) [0x00C9EFB8]
(No symbol) [0x00C9F157]
(No symbol) [0x00C9842E]
BaseThreadInitThunk [0x75946709+25]
RtlGetFullPathName_UEx [0x76F57D1D+173]
RtlGetFullPathName_UEx [0x76F57CEB+123]
<traceback object at 0x0000020C8334C5C0>

Getting a similar error today
⚠ Error executing ********, please check the log
Message: element click intercepted: Element triathlon olympia 2024 is not clickable at point (442, 46). Other element would receive the click:
(Session info: chrome=126.0.6478.182)
Stacktrace:
#0 0x55901ca23c5a
#1 0x55901c706e2c
#2 0x55901c75a406
#3 0x55901c758246
#4 0x55901c755c27
#5 0x55901c754a60
#6 0x55901c74815f
#7 0x55901c7765c2
#8 0x55901c747ad9
#9 0x55901c77678e
#10 0x55901c795303
#11 0x55901c776363
#12 0x55901c746247
#13 0x55901c746b9e
#14 0x55901c9ea22b
#15 0x55901c9ee2d1
#16 0x55901c9d5ade
#17 0x55901c9eee32
#18 0x55901c9ba77f
#19 0x55901ca13618
#20 0x55901ca137f0
#21 0x55901ca22d8c
#22 0x7ff3bedccac3
<traceback object at 0x7fa9121ff100>

Happens to me as well

Same error here, I think it's the "accept cookies" dialog that's screwing up the script

Do you have a full stack traceback? Look for something in the logs that starts with src. around the time you encounter the issue
Do you have a screenshot of the browser when the error happens?

Do you have a full stack traceback? Look for something in the logs that starts with src. around the time you encounter the issue
Do you have a screenshot of the browser when the error happens?

nothing with .src just this:
Traceback (most recent call last):
File "/home/deux/configs/MS-Rewards-Farmer/main.py", line 39, in main
earned_points = executeBot(currentAccount, args)
File "/home/deux/configs/MS-Rewards-Farmer/main.py", line 248, in executeBot
searches.bingSearches()
File "/home/deux/configs/MS-Rewards-Farmer/src/searches.py", line 136, in bingSearches
self.bingSearch()
File "/home/deux/configs/MS-Rewards-Farmer/src/searches.py", line 173, in bingSearch
searchbar.click()
File "/home/deux/.local/lib/python3.10/site-packages/selenium/webdriver/remote/webelement.py", line 94, in click
self._execute(Command.CLICK_ELEMENT)
File "/home/deux/.local/lib/python3.10/site-packages/selenium/webdriver/remote/webelement.py", line 395, in _execute
return self._parent.execute(command, params)
File "/home/deux/.local/lib/python3.10/site-packages/selenium/webdriver/remote/webdriver.py", line 354, in execute
self.error_handler.check_response(response)
File "/home/deux/.local/lib/python3.10/site-packages/selenium/webdriver/remote/errorhandler.py", line 229, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.ElementClickInterceptedException: Message: element click intercepted: Element pommel horse sport review is not clickable at point (442, 46). Other element would receive the click:
(Session info: chrome=127.0.6533.88)
Stacktrace:
#0 0x55c4d502b71a
#1 0x55c4d4cfc640
#2 0x55c4d4d52926
#3 0x55c4d4d50836
#4 0x55c4d4d4e2b7
#5 0x55c4d4d4d15c
#6 0x55c4d4d40c26
#7 0x55c4d4d6e8e2
#8 0x55c4d4d405d9
#9 0x55c4d4d6eaae
#10 0x55c4d4d8d08a
#11 0x55c4d4d6e683
#12 0x55c4d4d3ed71
#13 0x55c4d4d3f7de
#14 0x55c4d4ff32ab
#15 0x55c4d4ff7242
#16 0x55c4d4fe0665
#17 0x55c4d4ff7dd2
#18 0x55c4d4fc52af
#19 0x55c4d501aeb8
#20 0x55c4d501b090
#21 0x55c4d502a4ec
#22 0x7f042e120ac3

Closing as duplicate of #169 since @SteveeeeeS has added some more troubleshooting there