Using with password-less authentication

Hi @nsarno! Thank you for open sourcing this library! I was running into a conceptual problem while using it in a password-less authentication model (tldr: user enters email, clicks on link in sent email to authenticate - all of which is already set up).
I was curious as to how best to implement the authenticate method that the model needs to support. It seems like it is being used here and the password field is being sent: https://github.com/nsarno/knock/blob/d78f2481d86a2acd139d95075c3b181024c320c4/app/controllers/knock/auth_token_controller.rb#L13
It seems that since it was authenticating the password in regular setups, in my case, it would authenticate the email-token sent to check if it is authentic or not. Is that correct? If yes, how do I send that token to the authenticate method since it seems the password is hardcoded to be sent right now.
Also, is my understanding correct that the user_token#create is just the route to authenticate, and that if I already have that happening elsewhere, I might not need that route, and instead have that controller inherit from Knock::AuthTokenController?
Thank you!

edited the question above for clarity.

I haven't tried to implement this myself so I can only guess what I would try to do myself.
For the initial token creation, I see two possible solutions:

Pass the token from the email via the auth[password] param and implement your own User.authenticate method

or

Implement your own authentication controller that does whatever you want and returns the auth token (with AuthToken.new payload: { sub: entity.id })

Also, is my understanding correct that the user_token#create is just the route to authenticate

Yes. This is a route that would be generated when you run the generator:
rails generate knock:token_controller user
If you implement your own route and controller to create the token, then you don't need this.


Is there any possible way to change AI behavior

I gave tearlaments deck to CPU but it is not using deck properly.
If possible I might teach AI how to run this deck.

No. If you want to play against an AI that can play a specific deck you'll want to code an AI for a specific deck on ygopro and use a ygopro based simulator e.g. https://github.com/IceYGO/windbot/tree/master/Game/AI/Decks


Biome deep ocean has too many ships

The biome ocean/ocean-deep (DEEP_OCEAN) spawns a lot of ships (ships/generic)

Already dropped the spawn rate on  the latest branch


Two more places to check if attempting half_closed connection logic is feasible

There were two places where closing the connection was being delayed to handle the half-closed case.  But adding delay does not make sense for HTTP/2 which does not support half-closed at the TCP level.
Found while tracking down leaking Http2ClientSessions using the zombie event in PR #3713

[approve ci autest]


chore: Optimize Dockerfile and update run script

Pull Request
Description
This PR optimises the Docker build process and updates the run script for improved execution:

Implements a multi-stage Docker build to reduce image size
Uses poetry export to generate a requirements.txt file
Installs dependencies directly from requirements.txt instead of using Poetry in the final image
Updates the run.sh script to change directories, display environment variables, and execute the analyser module directly with Python instead of using Poetry

These changes aim to streamline the build process, reduce the final image size, and simplify the execution of the analyser.
fixes #100

#99  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @JackPlowman and the rest of your teammates on  Graphite

Merge activity

Sep 30, 4:09 PM EDT: @JackPlowman merged this pull request with Graphite.


Web smoke test: install and activate vscode-smoketest-check extension

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=108012&view=logs&j=3792f238-f35e-5f82-0dbc-272432d9a0fb&t=bd1c96ef-8f6c-518c-1bd9-e87b3a72835d&l=108
1) VSCode Smoke Tests (Web)
       VSCode Smoke Tests (Web)
         Extensions
           install and activate vscode-smoketest-check extension:
     Error: Timeout: get text content 'footer[id="workbench.parts.statusbar"] .statusbar-item[title="smoke test"]' after 20 seconds.
      at poll (/Users/runner/work/1/s/test/automation/out/code.js:173:19)
      at async Code.waitForTextContent (/Users/runner/work/1/s/test/automation/out/code.js:234:16)
      at async StatusBar.waitForStatusbarText (/Users/runner/work/1/s/test/automation/out/statusbar.js:22:9)
      at async Context.<anonymous> (out/areas/extensions/extensions.test.js:22:13)

Disabled via https://github.com/microsoft/vscode/commit/9c78fa40ca7ab6b58d0ab5cd33858a39a2b7102d


Catching exceptions when Celery broker is down

Hello!
First: thank's for this package, it is really useful in our project!
I have one problem: In one of my views a signal is triggered which then calls a celery task like this:
def post_save_donation(sender, instance, created, raw, using, **kwargs):
    create_something.delay(pk)

post_save.connect(post_save_donation, sender=Donation)

The task is defined like this:
from djcelery_transactions import task

@task
def create_something(donation_pk):
    # do something

Everything works great, but:
If RabbitMQ is down when I am calling the task no error is thrown during the create_something.delay(pk) call. But the view throws this error:
[Errno 111] Connection refused

The question now is: Why is this error thrown in the view but not in the signal handler where the create_something.delay(pk) is called?
Any help is greatly appreciated

I think you're simply not trying to catch the error in the signal, so it's bubbling up to the highest level, and the Django view will catch it if you don't.
Make sure to add a try...except in your signal.
Let me know if I misunderstand the issue


Use in Canada

Hi guys
I want to fork this application and use it Canada wide what file changes are necessary after doing so Any help?

Hi Neil, You are welcome to fork and stand up your own GeoQ with no restrictions. NGA has released this code with an MIT License.  If you run into some issues with standing up a server please post them to GitHub and we will respond as time permits.  Due to legal and policy issues NGA can not stand up and/or run software for groups or individuals outside the US Government or our mission set.
Some good news is that we have some Chef Scripts and other links to stand up a GeoQ Server.

GeoQ S2I Builder Image for OpenShift https://hub.docker.com/r/kevensen/centos-geoq-s2i/

jason-callaway/geoq-postgresql

ngageoint/geoq-chef-installer

FYI, Redhat has integrated GeoQ into OpenShift.  This means that for a small cost you can contact Redhat and they have the ability with OpenShift 3 (cloud) to stand up GeoQ servers in less than 2 minutes. This server can be linked to our GitHub Repository and then the server is updated whenever we push updates.


new teams client - deep Link to file not working

Steps to reproduce
create a deep link with the following syntax in a teams chat:
https://teams.microsoft.com/l/file/<FileID>?tenantId=<TenantID>&fileType=<FileExtension>&objectUrl=<LinkToFile>
Example:
https://teams.microsoft.com/l/file/XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX?tenantId=XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXX&fileType=pdf&objectUrl=https%3A%2F%2Fcontoso.sharepoint.com%2Fsites%2FSITENAME%2FLIBRARYNAME%2FFILENAME.pdf
In my case the links were generated by a powerautomate flow and sent to a teams channel as a user.
Expected behavior
The file should open within the teams client like it did with the old teams client.
According to our users it is still working on iOS devices, but i wont be able to confirm until tomorrow
Actual behavior
The file wont open in the new teams client and teams web client, its stuck on a blank screen
Example:

Pasting the url manually into a teams chat will create a preview like this

If you click on the preview button, the file opens within the teams client.
If you click on the link the file wont open and the screen stays blank
Error details
No response

Hello @bckrma ,
A bug has already been raised for the same issue. We are actively checking the status of bug with engineering team and let you know the updates.

I saw the documentation page has been updated today, but the format for file deep links still does not seem to be working for me in either the desktop or web clients for Teams v2 ; both open to a blank frame rather than previewing the file. I've also been following #9802 , which seems to have been at least mostly resolved, but maybe the fix was specific to app deep links rather than file deep links?

Hello @bsnyder9 ,
We got an update from the engineering teams that-
Deep linking to the OneDrive/Files personal app needs to be done using the following URL:
https://teams.microsoft.com/l/entity/5af6a76b-40fc-4ba1-af29-8f49b08e44fd/OneDriveFilesApp
Deep linking to Channel Files Tab needs to be done using the following URL:
https://teams.microsoft.com/l/entity/3ed5b337-c2c9-4d5d-b7b4-84ff09a8fc1c/FileBrowserTabApp?context={channelId:
Note: The whole context object should be encoded as per our docs here:
image
All of this is based on the entity deep link docs here:
https://learn.microsoft.com/en-us/microsoftteams/platform/concepts/build-and-test/deep-link-application?tabs=teamsjs-v2#configure-deep-link-to-browse-within-your-app-manually

@Vaibhav-MSFT  , thanks for the reply! I have only looked at those options a little, but I am not sure if I see how either can open a specific file. Would using Action.OpenUrl within an adaptive card possibly be the best approach?
For context, I am trying to set up a workaround via Power Automate so that when an email is forwarded to a channel, a comment is added to the post with a link to view that uploaded EML file within Teams; since the built-in "View original email" button on the post tries to download it instead of just viewing it. If I manually paste the link into a comment, it generates an abbreviated hyperlink as well as what appears to be an adaptive card, both of which load the in-app viewer. However, Power Automate does not do the same; instead the hyperlink it posts opens via SharePoint in a web browser tab. That isn't terrible, but viewing the EML file directly in Teams would be preferable. The "Generate deep links to a file in a channel" instructions originally seemed the most relevant and seems to almost work, since I am getting a frame within Teams that looks like it tried to preview something, but the actual EML file contents and title are not showing up within that frame.

So I actually already tried both previously and had forgotten that neither worked. Also, I am not trying to enable downloading the file - that is already available from the "View original email" card that gets added to any email forwarded to a Teams channel. I am trying to enable the same previewing functionality that happens when a URL is pasted into a comment by hand. The EML file is stored within the SharePoint document library for that channel's Files tab. It seemed like a deeplink was the intended solution, and I saw this issue so I thought it was just temporarily not working with the v2 clients.
If I should be asking about this elsewhere, I am happy to take suggestions!


feat: add support for packing tokenized datasets

What does this PR do?
Fixes #1848
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[x] Was this discussed/approved via a GitHub issue? Please add a link
to it if that's the case.  #1848
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines.
[x] Did you write any new necessary tests?

Who can review?
@qgallouedec
Anyone from the community!

Anyone having bandwidth, requesting review thank you - @qgallouedec @lewtun  @kashif @lvwerra or others from community.
Discussion can be seen here - https://github.com/huggingface/trl/issues/1848

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@lewtun Thank you for your review. I have addressed the review comments and as well added the test cases. Thank you.

@lewtun @qgallouedec requesting review and escalation, thank you.

@lewtun @qgallouedec apologies for the failing tests before. Tests passing now for me locally!  Thank you.

@lewtun @qgallouedec rebased looking for escalation, thanks.

@lewtun @qgallouedec Related tests pass after rebase, looking forward to including this work soon! Appreciate it.

@lewtun @qgallouedec any updates on this would be helpful!

@lewtun @qgallouedec it will be of great help pushing this forward before it gets into stale state for being open for long time after having a positive discussion over the issue https://github.com/huggingface/trl/issues/1848 with @qgallouedec
If you would like to tag anyone else from the community who have bandwidth basing on our discussion over the issue (https://github.com/huggingface/trl/issues/1848) would be nice!

Sorry for the delay. I'm trying to keep track of all these issues but it's not easy.
Lgtm overall, thanks. Just making a few comments

@qgallouedec Thank you for circling back, I have addressed your comments.


Multiple consumer groups in same process

In some situation you may want to subscribe to a topic with multiple consumer groups, to perform different parallel and independent tasks. The only way to do this at the moment is to subscribe to the IInboundMessage<T> and perform different operations according to the value of the Endpoint.Configuration.GroupId property.
It would be nice to have a cleaner way to filter incoming messages. For example using an attribute.
[KafkaGroupId("my-group-1")]
public void OnOrderReceived1(OrderCreatedEvent @event)
{
}

[KafkaGroupId("my-group-2")]
public void OnOrderReceived2(OrderCreatedEvent @event)
{
}

The filters attribute mechanism can be generic enough, to allow the creation of other filters in the future.

The proposed filter attribute has been implemented and is already available in the pre-release 2.0.0-rc1.
See https://beagle1984.github.io/silverback/docs/kafka/multiple-consumer-groups.


chore: Add tooling to detect missing specification tests

This PR
This adds a tool so we know which spec tests are missing. It is nearly identical to https://github.com/open-feature/java-sdk/blob/main/spec_finder.py, save the regexps.
How to test
$ python spec_finder.py
In the spec, but not in our tests:
  1.1.1: The API, and any state it maintains SHOULD exist as a global singleton, even in cases wherein multiple versions of the API are present at runtime.
  <IP_ADDRESS>: The API MUST define a provider mutator, a function to set the default provider, which accepts an API-conformant provider implementation.
  <IP_ADDRESS>: The provider mutator function MUST invoke the initialize function on the newly registered provider before using it to resolve flag values.
  <IP_ADDRESS>: The provider mutator function MUST invoke the shutdown function on the previously registered provider once it's no longer being used to resolve flag values.
  1.1.3: The API MUST provide a function to bind a given provider to one or more client names. If the client-name already has a bound provider, it is overwritten with the new mapping.
  1.1.4: The API MUST provide a function to add hooks which accepts one or more API-conformant hooks, and appends them to the collection of any previously added hooks. When new hooks are added, previously added hooks are not removed.
  1.1.5: The API MUST provide a function for retrieving the metadata field of the configured provider.
  1.1.6: The API MUST provide a function for creating a client which accepts the following options:  - name (optional): A logical string identifier for the client.
  1.1.7: The client creation function MUST NOT throw, or otherwise abnormally terminate.
  1.1.8: The API SHOULD provide functions to set a provider and
....a bunch more....

Just one question: How do you handle the Conditions in the spec? :-)

@sheepduke https://github.com/open-feature/java-sdk/blob/main/src/test/java/dev/openfeature/sdk/FlagEvaluationSpecTest.java#L318
All specification items need to be accounted for. For some, I annotate a bare test w/ a name that explains why this doesn't apply to us.

@sheepduke https://github.com/open-feature/java-sdk/blob/main/src/test/java/dev/openfeature/sdk/FlagEvaluationSpecTest.java#L318-L325
All specification items need to be accounted for. For some, I annotate a bare test w/ a name that explains why this doesn't apply to us.

That makes sense. Thank you!


System unavailable: Several mac machines

Please put the system name in the title of this issue.

Anything not showing as online at https://ci.adoptopenjdk.net/label/mac/

Please describe the issue:

Given the costs associated with the mac servers we should ensure that they are available, decommissioned or reassigned as appropriate. There are currently machines at macstadium, macincloud and nine which are offline. The Nine ones are a known issue where they periodically lose their network address, but the others are currently unknown. Suggest doing an audit to ensure that the machines are still available to us, and if so ensure they are live in jenkins.
Related: https://github.com/adoptium/infrastructure/issues/2378

Hmmm

test-macstadium-macos1014-x64-2 - Down in jenkins (launch fails), can ssh into, pingable

test-macstadium-macos1014-x64-2:~ administrator$ sudo su - jenkins
-bash: fork: Resource temporarily unavailable

I was getting permission denied when trying to ls the jenkins home directories as administrator
Try switching from root to jenkins, rather than administrator to jenkins

I was getting permission denied when trying to ls the jenkins home directories as administrator which I found odd. Try switching from root to jenkins, rather than administrator to jenkins

Looks ok to me:
test-macstadium-macos1014-x64-2:~ administrator$ ls /Users/jenkins
Desktop			freemarker.jar		kcpass
Documents		javasharedresources	kcpassdecode
Downloads		jdk-11.0.13+8		remoting
Library			jdk11u			remoting.jar
caches			jtreg			workspace
test-macstadium-macos1014-x64-2:~ administrator$

Hmmm it looks like several of the online ones aren't marked as build or test systems. @smlambert are you aware of any issue preventing us from adding the tags onto those machines?
We have https://github.com/adoptium/infrastructure/issues/1085 and https://github.com/adoptium/infrastructure/issues/1922 relating to two of the macincloud systems but we should start working towards enabling these machines (and decomissioning any we don't need!)

Hmmm we seem to have three macos/x64 build machines. Given the cost of these we should considker reducing that by converting at least one to be a test machine, and possibly two, keeping one of them as a "backup" build machine too if required.
https://ci.adoptopenjdk.net/label/mac&&!ci.role.test/

re: https://github.com/adoptium/infrastructure/issues/2467#issuecomment-1041542350 +1 to that, especially given we are at a ratio of "for every hour of build time, we typically need 17 hrs of test time".

Added ci.role.test hw.arch.x86 sw.os.osx and removed build from build-macstadium-macos1014-x64-3 to alleviate this backlog:

MacStadium machines now back up, Nine machines back online after restarting the RA service for the [TEST] group.


PYIC-4013: Added reprove_identity analytics parameter to IPV_JOURNEY_…

…START event

Proposed changes
What changed
Added reprove_identity analytics parameter to IPV_JOURNEY_START event
Why did it change
To facilitate the generation of reports that track how many identity proofing journeys are initiated as a result of account interventions.
Issue tracking

PYIC-4013

Checklists
Environment variables or secrets

[ ] No environment variables or secrets were added or changed

This looks good to me, are there any issues/considerations with deploying it in sync with code that will understand/report on the new data?


Not correct typechecking for files

Magic number approach doesn't work with empty files and files with malformed content

You can depend on filename extension and show parsed type as additional info
Example for file with tar content:
<LASER_COLOR_MEDIA>tarFileExample.jpeg</LASER_COLOR_MEDIA> (tar content)

For file viewer filename must be more important than content.

Possible solution:
Check filename extension (maybe with another env variable)
if it fails , check content for magic number.

Sounds like a grate idea! Though I don't really understand what you mean if it fails, if the file has no extension then we check the magic numbers?

Sounds like a grate idea! Though I don't really understand what you mean if it fails, if the file has no extension then we check the magic numbers?

Yep

Maybe something like plugins will be a good idea , plugin for file coloring , info and etc. With dlopen from plugin dir for example.

https://github.com/jmattaa/laser/pull/28


C++ Linter cannot add linker flags

The issue is that g++ requires flags such as the -lncurses flag to occur at the end of the command, but the variable
g:ale_cpp_gcc_options places all command line arguments in the same place.
current attempt:
let g:ale_cpp_gcc_options = '-std=c++14 -Wall -lncurses'
Is there a  way to do this?

I take it -l arguments have to come after -? Are there any other types of arguments which need to appear at the end? Maybe we'll just have to add another global variable.

I assume. I think any flags for the linter occur at the end.

I don't think you need -l parameters for linting. They're for linking surely?

Well my issue was that without -l ncurses the linter won't recognize any ncurses commands. Is there a way to include that library without the -l flag?

I think @esran might be right. I think -l is just for linking with a library, separate from reading header files. I can't remember if there's a "don't link" option for gcc, but I think there might be. That might be an answer.

Yeah, the gcc command should be doing that already. The -S flag is being used, and that means no assembling or linking should be done. https://gcc.gnu.org/onlinedocs/gcc/Overall-Options.html#index-S

What errors do you receive?

I suspect that when linting it is being stricter than the compiler. Those first four lines (noe shown) would be key. Maybe a missing include for the curses header file?

@esran Yes, I think that makes sense. Maybe GCC is letting you get away without including headers if you link against curses, like an automatic header include. After some quick reading, this does appear to be how GCC works.
I wonder if it might make sense then to add an option for setting linker flags. It might be the easiest way to fix problems like these for some users. It would be nice to somehow configure these settings per-project, or automatically read project settings and set the flags for you.

I can't repeat the issue on my machine and I don't know what to do about it. If anyone has any ideas, open a pull request.

Almost all C/C++ issues with flags should now be discussed in #3276, so a comprehensive solution can be found for most people, for automatically detecting flags.


Import error, can't find files: /index.html /index.ts

Describe the bug
I am using CRA with typescript.  the index.html is in ./public and the index.ts is in ./src which i thinks is pretty standard for create react app.  If i move these files to the root it is fine. I have played around with a lot of different config. also there does not seem to be any way to get more error reporting. from web search it seems to have cropped up a few times, but do not see any resolution. i am using your embed api to make the project
Link to the blitz that caused the error
https://ygdtqq--run.stackblitz.io
Steps to reproduce
use embed api to load a standard create react app in typescript into the editor.
preview
Expected behavior
not get the error, the project to run
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

I tried downloading the project from stackblitz and building it locally and it had some problems, so this is what caused the problem. however not seeing any errors apart from the spurious one above makes it very hard to know what was going on.


Cuts in Ex and theta_cm

After updating my version of the ISSsort code I am experiencing strange Ex and theta_cm histograms. Doesn't matter if I enable the butler algorithm or not. (currently on d80bf0a)

(this one without butler)
Also it looks like theta_cm is wrong (too low values)
@lpgaff Any idea what might have caused this?

Almost certainly my attemp to put the preprocessor flags in to the code. I will have a look at this this afternoon. Sorry.
Can you confirm that if you roll back to beaceea that it works?

Yes, works on https://github.com/ISOLDESolenoidalSpectrometer/ISSSort/commit/beaceea1436a53973e357188d829d36bb92b6430
Analysed just a few run files to check:

I'm going to close this, as I think it is fixed in the latest versions, at least from commit d07cf4c74ffc17ef4d40a6eaed727da271cc4d85

I'm going to close this, as I think it is fixed in the latest versions, at least from commit d07cf4c74ffc17ef4d40a6eaed727da271cc4d85


Logical operators question

Hi all, just checking in to see how you wrote up this piece from the lesson this week. Not sure I did it right...
everyone that scored over 20 on the strength test
these.strength <- dat$strength > 20
dat[ these.strength , "name"]
My results came back like this:
[1] frank wanda nancy
Levels: frank nancy sanjay wanda

That looks correct: the code these.strength <- dat$strength > 20 will give you a vector of TRUE and FALSE values that correspond to people who had strength scores over 20. The second line gets the names of the people who scored over 20 - which was frank wanda and nancy.


WIP: Add license and contributing info to team page. See #404

[ ] Wait for #402 to be merged,
[ ] change link to CONTRIBUTING.md (see FIXME in team.Rnw), and then
[ ] remove "WIP"

@HeidiSeibold #402 is merged, merge this yourself if done


DDCE-4530: Changes as part of sm2 upgrade

When running test:clean; test:compile in the sbt shell, there are a few of these deprecation warnings that could be address with this PR... Auto-application to () is deprecated.
These can be addressed as followed:
From this...
val countryOptions: Seq[InputOption] = app.injector.instanceOf[CountryOptionsNonUK].options
To this...
val countryOptions: Seq[InputOption] = app.injector.instanceOf[CountryOptionsNonUK].options()

There is 1 failing test in the local AT's that I'm seeing on my side.

I got one failing test for this which looks like it is being addressed as part of this PR
https://github.com/hmrc/trusts-acceptance-tests/pull/774/files


Skip documentation formatting if configured in AlignSettingsSection

Closes #622

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

Comparison is base (1404f35) 96.79% compared to head (8f8720f) 94.47%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #627      +/-   ##
==========================================
- Coverage   96.79%   94.47%   -2.32%
==========================================
  Files          50       50
  Lines        4993     4999       +6
==========================================
- Hits         4833     4723     -110
- Misses        160      276     +116

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Saving Multi-GPU

Thanks for this repo!
It seems like the code doesn't save the models correctly when using multiple GPUs right? Like if I train on multiple GPUs but load on a single CPU, it won't work I think.
You have this
    def save(self, path):
        states = {
            'G': self.G.state_dict(),
            'D': self.D.state_dict(),
            'optim_G': self.optim_G.state_dict(),
            'optim_D': self.optim_D.state_dict()
        }
        torch.save(states, path)

But I think you need self.G.module.state_dict() ?

Hello @EoinKenny , it could be the case when the model is trained on multiple GPUs. I trained the model on single GPU, so I didn't encounter any error. Please refine the code if you see it. Also, feel free to create a pull request!


[HYP-66]Revise course page

Jira: HYP-66

active nav bar highlight color
responsive block
detail of course

– [x] Artifacts Added

LGTM

@iionayne @iionayne25 Please fix this one.


make public, rio pansharpen

Is there any reason to keep this repo private? I propose we make it a proper rio pansharpen tool and spend a few hours cleaning it up for a release.

Made pansharpening public! Let's think more about renaming the repo.


Truncate overly long suggest fields before persisting them

The constraint field from the organization_package_link table is limited to 255 chars.
Currently, synchronization fails for packages that use suggest fields longer than 255 chars, and it becomes impossible to manage them from the UI (even if only to remove them).
Note that I am open to suggestions on how we might otherwise handle these cases.

Truncated description is not a problem ?
We can maybe change type in database in order to allow longest text ? And manage length in UI ? (css, ellipsis or something else)

agree with @alamirault, you'd better change the field type to text :)


[Bug]: Clicking a disabled delete icon triggers an error message.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When we click a disabled delete icon of a home page on the Page Properties UI, an error message comes up "Cannot Delete the Home Page". This message seems redundant.
Expected behavior - on clicking a disabled icon, no action should be triggered.
Steps To Reproduce

On the Page Properties page, for a Page that is set as Home Page (say Page1), hover over the Delete icon on the right side
Note that the delete is disabled. This is correct. We should not be able to delete a Home Page
However, on clicking it, an error message comes up "Cannot Delete the Home Page"

This is redundant. If the icon is disabled and not clickable, it should not generate any message.
Public Sample App
No response
Issue video log
https://ddpa7mfuvv.vmaker.com/record/fT2LVXy655GSvSdE
Version
Production

This has been resolved and is no longer an issue. The delete button no longer appears for the home page


Segmentation fault with OpenCV when enabling color stream

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
Linux (Ubuntu 16)

Kernel Version (Linux Only)
4.13.0-43-generic

Platform
PC

SDK Version
2.10.3

Language
C++

Issue Description
I've been struggling with this for a while. I don't know if this is expected behavior or not, so I thought I'd share it.
I was playing with the imshow example. I replaced every "depth" in the code by "color" and rs2::frame to rs2::video_frame. Then I decided to use a configuration and just added cfg.enable_stream(RS2_STREAM_COLOR)
The resulting code is as follows:

#include <librealsense2/rs.hpp> // Include RealSense Cross Platform API
#include <opencv2/opencv.hpp>   // Include OpenCV API
int main(int argc, char * argv[])
{

    rs2::config cfg;

    //Add desired streams to configuration
    cfg.enable_stream(RS2_STREAM_COLOR);

    rs2::pipeline pipe;

    // Start streaming with default recommended configuration
    pipe.start(cfg);

    using namespace cv;
    const auto window_name = "Display Image";

    while (true)
    {
        rs2::frameset data = pipe.wait_for_frames(); // Wait for next set of frames from the camera
        rs2::video_frame color = data.get_color_frame();

        // Query frame size (width and height)
        const int w = color.get_width();
        const int h = color.get_height();

        // Create OpenCV matrix of size (w,h) from the colorized color data
        Mat image(Size(w, h), CV_8UC3, (void*)color.get_data(), Mat::AUTO_STEP);
        // Update the window with new data
        imshow(window_name, image);

        char c = (char) cv::waitKey(25);
        if(c==27) break;
    }

    return EXIT_SUCCESS;
}

This code as is rises a Segmentation fault (core dumped) when reaching the imshow line. This does not happen with the original example when adding RS2_STREAM_DEPTH nor if no configuration is used. In order to fix this I had to add a second parameter: cfg.enable_stream(RS2_STREAM_COLOR,RS2_FORMAT_BGR8);.
My guess is that when only asking for the Color Stream the remaining default parameters are such that imshow is unable to handle the resulting frame, but I'm not quite sure why. Also, as I said above, I don't really know if this is actually expected behavior or not, but it got me nuts for a while.
Hope this helps somebody!

[Realsense Customer Engineering Team Comment]
@javierselva
since you only enable stream with type specified but not the format, so it will return the default value for you which is "RS2_FORMAT_RAW16" that's the reason why you encounter the problem. To fix the problem is simple, just specify which format you need, so change your line of "cfg.enable_stream(RS2_STREAM_COLOR)" to "cfg.enable_stream(RS2_STREAM_COLOR, RS2_FORMAT_BGR)" then everything works well. You can also get back the activated profile to check which format is being configured, I attache the code snippet below for your reference.
 rs2::config cfg;
 //cfg.enable_stream(RS2_STREAM_COLOR); //this will cause seg fault in Mat(...)
 cfg.enable_stream(RS2_STREAM_COLOR, RS2_FORMAT_BGR8);
 rs2::pipeline_profile pp = pipe.start(cfg);
 rs2::stream_profile sp = pp.get_stream(RS2_STREAM_COLOR);
``

I see! So it is just that the default format is not a valid frame for OpenCV.
Thanks a lot, the solution works!


Component inside nuxt-link is rendered twice

Nuxt version : "nuxt": "^1.0.0-rc11"
Env: dev
I recently encountered a strange bug, more exactly if you have a component inside a nuxt-link the component will display twice. I have no errors and I tried everything
Ex:
The result will be a link with the Logo component rendered twice.
The content of Logo.vue can be anything, it doesnt seem to mather.

Could you have a on nuxt@next ?

@clarkdo Sorry but I'm kind of new to nuxt. What do you mean exactly?

Change your dependecy from  "nuxt": "^1.0.0-rc11" to "nuxt": "next"  to have a try 😸 , but be aware that it's the unstable channel until 1.0 is hit.

Update: I searched through nuxt releases and encountered this "npm install --save nuxt@next" on Release "v1.0.0-rc3". After running it my nuxt version changed to "^1.0.0-gh-887ebd5" and the bug dissapeared.
Thanks for the help!


completed work

completed navbar @sailaja-adapa akka

good work @abhi200446

work on fixing donation section and also remove unnecessary text on navbar which is appearing in home page i.e -->

ok akka

Try to fix Navbar on all pages with good allignment and include all options remove icons

@sailaja-adapa akka clarity ga cheppava em changes cheyalo donate section lo


Including an "En Dash" in an axis title results in a 500 error on retrieving via getImage

Bug Description
Adding the En Dash character (–, not a dash) to an axis title or a figure title will prevent the figure from being retrieved via getImage.
Repo Steps:

Create a figure
Add the En Dash character: – to the title and/or axis title
Retrieve the figure via the Node.js SDK
Observe the following error output:

{ Error: Bad response status code 500
    at ClientRequest.handleResponse (/.../node_modules/plotly/index.js:237:25)
    at Object.onceWrapper (events.js:324:30)
    at emitOne (events.js:125:13)
    at ClientRequest.emit (events.js:221:7)
    at HTTPParser.parserOnIncomingClient [as onIncoming] (_http_client.js:550:21)
    at HTTPParser.parserOnHeadersComplete (_http_common.js:117:23)
    at TLSSocket.socketOnData (_http_client.js:439:20)
    at emitOne (events.js:125:13)
    at TLSSocket.emit (events.js:221:7)
    at addChunk (_stream_readable.js:265:12) msg: undefined }

@SpencerKaiser Thanks for the report. Unfortunately this library has not seen much use by our customers so we're not able to spend significant resources maintaining it. However it is an open source project and we welcome a fix from the community for this issue.
Note that the node.js library is using currently performing image generation using a legacy endpoint (/apigenimage). Switching to the new endpoint, available and documented here, is likely to fix this issue as well as #55. The new endpoint takes the same JSON body but authentication has changed somewhat.

@scjody thanks for the update. I used Plot.ly as part of my company's CI/CD solution along with Artillery.io to generate performance test graphs for some of our new services. If we scale things up and get a license, I may try to update the SDK for that new endpoint!


Plan to finalise the MS (12 Sept 2023)

@Mike-Garratt and @itchyshin had a meeting and discussed what is left to do

[ ] @itchyshin checks the ones with >150% extension
[x] Make Fig 1 nicer - get @mlagisz'e help
[x] Create Fig 4 like Fig 1 @itchyshin + @mlagisz  (make sure we combine all types of sterilizations)
[ ] Get @mlagisz to check Fig 1 and 4
[ ] Create an HTML (a separate one from the meta-analysis one, which is completed already) for the Zoo analyses, including all figs and tables - pre-post (but leave Gamma stuff for now as we do not discuss this in the main text).
[ ] Add stats to the MS - @itchyshin and then get @Mike-Garratt to check (@itchyshin - adding to @Jeff-Lemaitre's version)
[ ] Get the final list of authors - @Mike-Garratt
[ ] Probably @Mike-Garratt and @itchyshin have a meeting again once @itchyshin finishes his part
[ ] @Mike-Garratt send enquires to Science/Nature editors

@Mike-Garratt if I forget something please do add (If I have questions, I will email you and double-check)

Fig 1

Fig 4

@Mike-Garratt - I realise that I have to redo the HTML as fig numbers etc changed, I have some issues with old code not running (very common issues with R), it may take a while for me to resolve). Also, this is last 2 weeks in Okinawa and they are jam-pakced with meetings and workshops.
Now I am adding stats etc but I realise to be consistent we need to discuss what to do with suppl. figs etc (reporting of heterogeneity - zoo data has I2 etc as well). So once I do what I can do for now I will set a meeting.

Thanks Shinichi, these figures look awesome!  I think as long we have the made statistics in the manuscript we should be good and we can match up the stats between the two types of analysis at a later point.
Have you already started working on a copy of the manuscript? I was going to address Jeff's comments but don't want to start a conflicted version.
Hope you can enjoy your last couple of weeks in Okinawa, sounds like a great experience.
Mike

@Mike-Garratt - thanks for your reply - I will email you the current version


[TF 2.0] Unsupported op node error messages in latest tf-nightly (8-27-19)

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): tf-nightly-gpu-2.0-preview==2.0.0.dev20190827
Python version: 3.7.4
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 10.0/7.6.2
GPU model and memory: Titan Xp 12 gb

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
When running a model that previously yielded no errors, I'm getting errors of the form
2019-08-27 11:35:15.004095: E tensorflow/compiler/jit/compilability_check_util.cc:346] unsupported op node : {name:'policy/fn__call__/build_one_time_effects_or_recurrence/get_effect
s/get_phiC/add_collisions/while/enter/_15' id:223 op device:{/job:localhost/replica:0/task:0/device:GPU:0} def:{{{node policy/fn__call__/build_one_time_effects_or_recurrence/get_eff
ects/get_phiC/add_collisions/while/enter/_15}} = Enter[T=DT_INT32, frame_name="policy/fn_...ions/while", is_constant=false, parallel_iterations=10, _device="/job:localhost/replica:0
/task:0/device:GPU:0"](policy/apply_gauss_force/ExpandDims/dim)}}
2019-08-27 11:35:15.004669: E tensorflow/compiler/jit/compilability_check_util.cc:346] unsupported op node : {name:'policy/fn__call__/build_one_time_effects_or_recurrence/get_effect
s/get_phiC/add_collisions/while/enter/_9' id:224 op device:{/job:localhost/replica:0/task:0/device:GPU:0} def:{{{node policy/fn__call__/build_one_time_effects_or_recurrence/get_effe
cts/get_phiC/add_collisions/while/enter/_9}} = Enter[T=DT_INT32, frame_name="policy/fn_...ions/while", is_constant=false, parallel_iterations=10, _device="/job:localhost/replica:0/t
ask:0/device:GPU:0"](policy/postprocess_policy_inputs/Sum/reduction_indices)}}
2019-08-27 11:35:15.005043: E tensorflow/compiler/jit/compilability_check_util.cc:346] unsupported op node : {name:'policy/fn__call__/build_one_time_effects_or_recurrence/get_effect
s/get_phiC/add_collisions/while/enter/_4' id:227 op device:{/job:localhost/replica:0/task:0/device:GPU:0} def:{{{node policy/fn__call__/build_one_time_effects_or_recurrence/get_effe
cts/get_phiC/add_collisions/while/enter/_4}} = Enter[T=DT_INT32, frame_name="policy/fn_...ions/while", is_constant=false, parallel_iterations=10, _device="/job:localhost/replica:0/t
ask:0/device:GPU:0"](policy/postprocess_policy_inputs/get_collision/strided_slice_3/stack_1/0)}}
2019-08-27 11:35:15.005378: E tensorflow/compiler/jit/compilability_check_util.cc:346] unsupported op node : {name:'policy/fn__call__/build_one_time_effects_or_recurrence/get_effect
s/get_phiC/add_collisions/while/enter/_2' id:228 op device:{/job:localhost/replica:0/task:0/device:GPU:0} def:{{{node policy/fn__call__/build_one_time_effects_or_recurrence/get_effe
cts/get_phiC/add_collisions/while/enter/_2}} = Enter[T=DT_INT32, frame_name="policy/fn_...ions/while", is_constant=false, parallel_iterations=10, _device="/job:localhost/replica:0/t
ask:0/device:GPU:0"](policy/postprocess_policy_inputs/get_collision/strided_slice_3/stack_1/0)}}
2019-08-27 11:35:15.005796: E tensorflow/compiler/jit/compilability_check_util.cc:346] unsupported op node : {name:'policy/fn__call__/build_one_time_effects_or_recurrence/get_effect
s/get_phiC/add_collisions/while/enter/_11' id:242 op device:{/job:localhost/replica:0/task:0/device:GPU:0} def:{{{node policy/fn__call__/build_one_time_effects_or_recurrence/get_eff
ects/get_phiC/add_collisions/while/enter/_11}} = Enter[T=DT_BOOL, frame_name="policy/fn_...ions/while", is_constant=false, parallel_iterations=10, _device="/job:localhost/replica:0/
task:0/device:GPU:0"](policy/postprocess_policy_inputs/Tile_7)}}
2019-08-27 11:35:15.006245: E tensorflow/compiler/jit/compilability_check_util.cc:346] unsupported op node : {name:'policy/fn__call__/build_one_time_effects_or_recurrence/get_effect
s/get_phiC/add_collisions/while/merge/_29' id:255 op device:{/job:localhost/replica:0/task:0/device:GPU:0} def:{{{node policy/fn__call__/build_one_time_effects_or_recurrence/get_eff
ects/get_phiC/add_collisions/while/merge/_29}} = Merge[N=2, T=DT_INT32, _device="/job:localhost/replica:0/task:0/device:GPU:0"](policy/fn__call__/build_one_time_effects_or_recurrenc
e/get_effects/get_phiC/add_collisions/while/enter/_15, policy/fn__call__/build_one_time_effects_or_recurrence/get_effects/get_phiC/add_collisions/while/next_iteration/_74)}}
2019-08-27 11:35:15.006623: E tensorflow/compiler/jit/compilability_check_util.cc:346] unsupported op node : {name:'policy/fn__call__/build_one_time_effects_or_recurrence/get_effect
s/get_phiC/add_collisions/while/merge/_23' id:256 op device:{/job:localhost/replica:0/task:0/device:GPU:0} def:{{{node policy/fn__call__/build_one_time_effects_or_recurrence/get_eff
ects/get_phiC/add_collisions/while/merge/_23}} = Merge[N=2, T=DT_INT32, _device="/job:localhost/replica:0/task:0/device:GPU:0"](policy/fn__call__/build_one_time_effects_or_recurrenc
e/get_effects/get_phiC/add_collisions/while/enter/_9, policy/fn__call__/build_one_time_effects_or_recurrence/get_effects/get_phiC/add_collisions/while/next_iteration/_68)}}
2019-08-27 11:35:15.007007: E tensorflow/compiler/jit/compilability_check_util.cc:346] unsupported op node : {name:'policy/fn__call__/build_one_time_effects_or_recurrence/get_effect
s/get_phiC/add_collisions/while/merge/_18' id:259 op device:{/job:localhost/replica:0/task:0/device:GPU:0} def:{{{node policy/fn__call__/build_one_time_effects_or_recurrence/get_eff
ects/get_phiC/add_collisions/while/merge/_18}} = Merge[N=2, T=DT_INT32, _device="/job:localhost/replica:0/task:0/device:GPU:0"](policy/fn__call__/build_one_time_effects_or_recurrenc
e/get_effects/get_phiC/add_collisions/while/enter/_4, policy/fn__call__/build_one_time_effects_or_recurrence/get_effects/get_phiC/add_collisions/while/next_iteration/_63)}}
2019-08-27 11:35:15.007369: E tensorflow/compiler/jit/compilability_check_util.cc:346] unsupported op node : {name:'policy/fn__call__/build_one_time_effects_or_recurrence/get_effect
s/get_phiC/add_collisions/while/merge/_16' id:260 op device:{/job:localhost/replica:0/task:0/device:GPU:0} def:{{{node policy/fn__call__/build_one_time_effects_or_recurrence/get_eff
ects/get_phiC/add_collisions/while/merge/_16}} = Merge[N=2, T=DT_INT32, _device="/job:localhost/replica:0/task:0/device:GPU:0"](policy/fn__call__/build_one_time_effects_or_recurrenc
e/get_effects/get_phiC/add_collisions/while/enter/_2, policy/fn__call__/build_one_time_effects_or_recurrence/get_effects/get_phiC/add_collisions/while/next_iteration/_61)}}

when using autograph with the tf.function decorator.
Describe the expected behavior
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
Other info / logs
I'm just curious how we are supposed to interpret these logs; so far it's non blocking, but I'm wondering if this is indicating autograph is failing to convert parts of my model.

@mjlbach, In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!

I countered the same issue. Here is my code that can be used to reproduce it.
It works fine in tensorflow 2.0 without GPU, but omit errors with  tf-nightly-gpu-2.0-preview 2.0.0.dev20190827
import numpy as np
import tensorflow as tf

y = tf.cast(np.random.randn(1, 100, 100), tf.float32)
z = tf.cast(np.random.randn(1, 100), tf.float32)

layer = tf.keras.layers.GRU(100)

@tf.function
def fun(layer, x):
    return layer(x)

with tf.GradientTape() as tape:
    loss = tf.norm(fun(layer, y) - z)

I think the example above is sufficient, I'll retest on nightly with https://github.com/tensorflow/addons/pull/459

cc @alextp Hi, Alexandre, can you take a look at this or redirect to someone else who can? Thanks.

cc @asimshankar

I can't reproduce it on 20190830's nightly, so I think this is a bug that was fixed.


Status of this library/gem

Hi,
I was wondering what the status of this library is, given the fact that the last commit is almost one year old, and there doesn't seem to be any activity, with native AQL Ruby support still missing.
Is this good to use in an actual project or is it abandonded or deprecrated?

I'm not working for ArangoDB since April 2015. I'm not working on Guacamole, Ashikawa::Core or the AQL gem Brazil since summer of 2014 and have no plans to do so. I'm more than happy to pass the maintainership on to someone else. As the graph API changed quite significantly since then this gem is no longer supporting the current version of ArangoDB.

Dang, ok. Thanks for the update!
Maybe put that information in the README, too? It looks like it's good to use on first sight and the ArangoDB docs have a tutorial for Guacamole that doesn't say anything about incompatibility or deprecation.

Done! Thanks for the feedback!


Empty Gantt view

Hello,
This issue is possibly duplicate of #11, #3454, #1532... but none of these helped me to solve my problem. Or maybe I misunderstand solution.
Thanks for your help.
Actual behaviour
Gantt view only shows top menu ("Sort by...") and no task appears under it.
Expected behaviour
Having tasks shown with Gantt organisation
Server Configuration

Plugin version: 1.0.2
Other plugins: Slack 1.0.6, Milestone 1.1.1, Gantt 1.0.2, Calendar 1.1.0, RocketChat 1.0.5, Self-Registration 1.0.6
Kanboard version: 1.0.48
PHP version: 7.0.20-1~dotdeb+8.1
OS version: Linux 4.9.0-4-amd64 (VPS)
Database version: 10.1.26-MariaDB-0+deb9u1
Browser: Mozilla/5.0 (X11; Linux x86_64; rv:57.0) Gecko/20100101 Firefox/57.0

Server logs
Nothing relevant.
Other infos
From #11, I checked user's role:

2 users as Project Manager
2 users as Project Member
1 group as Project Member
1 group as Project Viewer

From #1532, I tried to remove all .* in plugin directories. No success.

I notice this in apache logs:

[Mon Nov 20 10:00:08.708490 2017] [:error] [pid 5346] [client <IP_ADDRESS>:64192] PHP Warning:  filemtime(): stat failed for plugins/Gantt/Assets/gantt.css in /data/www/kanboard/app/Helper/AssetHelper.php on line 37
[Mon Nov 20 10:00:08.708503 2017] [:error] [pid 5346] [client <IP_ADDRESS>:64192] PHP Warning:  filemtime(): stat failed for plugins/Calendar/Assets/fullcalendar.min.css in /data/www/kanboard/app/Helper/AssetHelper.php on line 37
[Mon Nov 20 10:00:08.708517 2017] [:error] [pid 5346] [client <IP_ADDRESS>:64192] PHP Warning:  filemtime(): stat failed for plugins/Gantt/Assets/chart.js in /data/www/kanboard/app/Helper/AssetHelper.php on line 24
[Mon Nov 20 10:00:08.708527 2017] [:error] [pid 5346] [client <IP_ADDRESS>:64192] PHP Warning:  filemtime(): stat failed for plugins/Gantt/Assets/gantt.js in /data/www/kanboard/app/Helper/AssetHelper.php on line 24
[Mon Nov 20 10:00:08.708537 2017] [:error] [pid 5346] [client <IP_ADDRESS>:64192] PHP Warning:  filemtime(): stat failed for plugins/Calendar/Assets/moment.min.js in /data/www/kanboard/app/Helper/AssetHelper.php on line 24
[Mon Nov 20 10:00:08.708547 2017] [:error] [pid 5346] [client <IP_ADDRESS>:64192] PHP Warning:  filemtime(): stat failed for plugins/Calendar/Assets/fullcalendar.min.js in /data/www/kanboard/app/Helper/AssetHelper.php on line 24
[Mon Nov 20 10:00:08.708558 2017] [:error] [pid 5346] [client <IP_ADDRESS>:64192] PHP Warning:  filemtime(): stat failed for plugins/Calendar/Assets/locale-all.js in /data/www/kanboard/app/Helper/AssetHelper.php on line 24
[Mon Nov 20 10:00:08.708568 2017] [:error] [pid 5346] [client <IP_ADDRESS>:64192] PHP Warning:  filemtime(): stat failed for plugins/Calendar/Assets/calendar.js in /data/www/kanboard/app/Helper/AssetHelper.php on line 24

Looking #3531, I tried moving plugins dir to origins, ie not into data dir. And this solve the problem.
So first, I tried to set define('PLUGINS_DIR', DATA_DIR.DIRECTORY_SEPARATOR.'plugins'); in config.php but this is not working. Replacing plugins dir in www and define('PLUGINS_DIR', 'plugins'); in config.php solve the problem.


Improvement ideas

Thanks for building this library. I've been working with it a bit and have implemented a few improvements in my fork:

Remove features that break the build:
https://github.com/joelburget/nibble/commit/5e76623226af3b3f9d98a2d3e40ba751350a0fd5
Add to_byte_vec to NibVec:
https://github.com/joelburget/nibble/commit/2a18b917ae4a362dc49a055cf1f6ea8f3ee59222
Fix a few bugs: https://github.com/joelburget/nibble/commit/872c4754059a2c1cb73f4c236fa86bb937254bed
Quickcheck a few of the things I fixed: https://github.com/joelburget/nibble/commit/12eb90a617f0efa89c32428be37c3b43fb7d1091

Are you open to accepting changes like this?
I'd also like to start a conversation about a few things I'd like:

I'd like to concatenate two NibVecs. Should I add append()? Should the Extend trait be implemented?
Why does this library provide five types of Nibble containers? Some documentation differentiating them would be welcome. Less types would ease the implementation burden.

cc @bts

I like these ideas!
I will try and look at these within the next week or so. Until then, note that I'll eventually be moving this repository to https://git.charr.xyz/experiments/nibble and so you may not see these changes on this repo.


Add Konga in community tools

Just a simple addition to README.md

Hi,
Thanks for the addition; unfortunately we won't be adding new UI tools in there anymore. In the future, this section will be revamped. Sorry.

@thibaultcha IMHO Konga is one of the best third party tools supporting Kong at the moment, and it gets better it is open source. It is complete admin tool with ability to take snapshots and restore this snapshot to any node of your choosing. Also person can import/export this snapshot as JSON file. New version adds health checks to nodes and apis with email notifications or endpoints of your choosing. It has more features than of couple of tools out there combined together like Kong Dashboard, Kongfig etc and does better job than them in each area , sort of Master of ALL. and it adds health check to top it all. All of this packaged in best GUI which is very clean and intuitive. @pantsel  Kong's readme already mentions Konga, your request was duplicating entry.

@thibaultcha I want to add...several of those tools mentioned in community section have not been updated in last two years. Konga is active project supporting latest 0.10.x and @pantsel responds to all requests immediately and actively looks for  feedback, if anything Konga should be prominently placed in the community section.

@thibaultcha nevermind I realized Konga tool that's mentioned is different tool so pantsel's Konga definitely needs to be on the list

@thibaultcha glad to hear that it is getting a revamp, does that mean the documentation is going to be reviewed and cleared up in the not too distant future? In the meantime, would it possible to include Konga? It is a great management interface, and a little exposure on Kong's repo would help encourage development I'm sure...


readthedocs vs docs.microsoft.com

Why are the asp.net core docs powered by the read-the-docs engine, when there is a new docs engine the docs.microsoft.com? There is a misslead in the docs.microsoft.com site that the asp.net core docs is powered by the new engine. Is there any plan to migrate this to the new one?

We do plan to eventually move to docs.ms.com. At the time we started  ASP.NET Core, Read The Docs was the best platform.

Thank you! I got answers on twitter too. :)

@Rick-Anderson One other quick ?: docs.microsoft.com doesn't seem to be searchable. I mean I don't see a search box anywhere on the pages for .NET Core docs, and that's been a pain point (for me anyway) finding things over there. Is the preferred method to use search operators? ... e.g. for Bing or Google ...
<search term | "search phrase"> site:docs.microsoft.com

@GuardRex  They're working on it.


record errors derefenceable?

Should operators on errors work the same was as unions, where we automatically access the "under()" value.  While this could cause confusing behavior, it also has some nice properties... e.g., creating timestamped errors where ts is the pool key...
switch (
  ...
  default => yield error({ts:now(), ...})
)

If the result error record allowed the ts field to be derefenced, then this would just work as a pool value alongside other values sorted by ts.
Right now you get a missing error when you do this...
echo 'error({x:1})' | zq "yield {x}" -
{x:error("missing")}

That said, we shouldn't magically make errors into non-errors inside expressions, so maybe we just let this record deref case be the important expcetion?

@mccanne just bumped into what looks like a variation on this one. Repro is with Zed commit b6d853f.
In his own words:

$ zq -version
Version: v1.17.0-44-gb6d853fe

$ echo '{a:error({b:2})}' | zq 'yield a' -
error({b:2})

$ echo '{a:error({b:2})}' | zq 'yield a.b' -
error("missing")

The yield of a.b should give 2.

We had another group chat about this one. There's consensus that the current behavior is actually ok, since when it's an error, if the underlying value was transparently returned, the user might miss out on seeing there ever was an error, and that seems bad. At the same time, the user would still surely want a way to get at the underlying values when they need to, such as if they're intentionally going in to find and fix errors. Therefore our thinking is now that there should be some kind of explicit "de-error" operator/function. Naming is TBD.
This was estimated as something we might want to have working for some demos in the October, 2024 timeframe.


Multiversion

Original pull request by Giampiero Gabbiani (Bitbucket: ggabbiani, GitHub: ggabbiani).

The idea was to consolidate the current status of Coin v4 CMake branch in order make it possible the generation of usable rpms under Linux , able to coexist with the legacy v3 on the same system.
In detail:

added versioned installation path for includes and docs:

CMAKE_INSTALL_DOCDIR                   from 'share/doc/Coin' to  'share/doc/Coin4'
CMAKE_INSTALL_FULL_DOCDIR         from '/share/doc/Coin' to '/share/doc/Coin4'
CMAKE_INSTALL_INCLUDEDIR           from 'include' to 'include/Coin4'
CMAKE_INSTALL_FULL_INCLUDEDIR from '/include' to '/include/Coin4'

fixed cpack generation through RPM (solving some issues when building rpms)

assigned the cmake config to development component

Other minor changes:

Implemented Qt Help generation management for supporting IDEs like QtCreator or KDevelop

fixed a warn on string constant conversion

added option controlled helper for cmake variable content debug during config

Regards
Giampiero

Original comment by Bastiaan Veelo (Bitbucket: veelo, GitHub: veelo).

This looks good to me. Thanks!


Craft 4 Install

We just recently updated our Craft CMS to v4 and are now trying to install QRCode but are all having the same errors.

We also installed several plugins after updating to v4 and they all went smoothly except for this one.

any news on this? we are running into the same problem.


[8.x] Fix array keys from cached routes

Introduced by https://github.com/laravel/framework/pull/35714.
Removed the slash to match the same array key as in
src/Illuminate/Routing/RouteCollection.php:60
We noticed a different result when we used route caching for in this code piece:
$url = parse_url($requestUri, PHP_URL_PATH);

 if (!is_string($url)) {
        return true;
 }

$routeKey = '{tenant}.' . trim(config('app.domain')) . trim($url, '/');

$routes = Route::getRoutes()->get($requestMethod);

return array_key_exists($routeKey, $routes);

Only routes with a specified domain where affected and a the result array of "->get('GET')" would not have the same array keys. Check next example with uri "test" and domain "{tenant}.laravel.test", this would give these two different results if you use cached router or not:
"{tenant}.laravel.test/test" => cached
"{tenant}.laravel.testtest" => not cached
The check for the existing of a domain is not necessary because the "->getDomain" function can return null as well and wil result in empty string (just like it behaves in the non compiled RouteCollection).
If approved & merged, I can cherry-pick it for 9.x release as well.

I don't think this is valid? Seems like existing tests are breaking because of this PR.

It indeed seems that this is the correct fix. We'll need to adjust the tests and also version the routes file anew to make sure existing cached route files aren't conflicting with this change.

I'll try to find some time this weekend to check which tested needs an update

Can we get a test on this?

There was just a test that needed the fix as well to match the changed code. If merged, I can cherry-pick for 9.x

@taylorotwell there were tests for this, they've now been changed to the new behavior 👍


permission denied to create extension "vectors"

I think this is just a database role permissions thing, but if there's anything that can be done to automatically load this from the container image side that would be great.

After creating the extension manually, immich-server errors with must be owner of extension vectors

Can we give Immich user the perms to do it?

It seems like pgvecto.rs operates on a pretty low level, and thus the extension is essentially "a database superuser on steroids" (to quote supabase. As such the security model is set up so that only superusers can use it, it seems. You could argue that since the db admin has installed and vetted the extension, they should be able to assign permissions to a role to use that extension, but I don't see any way to do that.

It seems like at least a part of this is caused because the Immich migration starts by running DROP EXTENSION IF EXISTS vectors. That immediately fails due to not being a superuser. We should be able to remove that line and just use CREATE EXTENSION IF NOT EXISTS vectors (seems like in the past pgvecto.rs did not support that).
With that, some manual action will unfortunately still be needed to create the extension ahead of time, but it'll be as straightforward as just connecting to the database with a superuser and running CREATE EXTENSION vectors.

Why is there that in the first place?

The pgvecto.rs docs suggest it, and apparently in the past IF NOT EXISTS was not supported by pgvecto.rs?

I tried doing the following:

Initialize a db cluster for Immich
Connect to it manually and run

ALTER ROLE immich WITH SUPERUSER;
SET ROLE immich;
CREATE EXTENSION vectors;
RESET ROLE;
ALTER ROLE immich WITH NOSUPERUSER;

Start the Immich containers and let them run migrations.

This didn't work because during the migrations they still call CREATE EXTENSION vectors and fail with a permission issue.
Next I tried giving SUPERUSER to the immich role only during the migrations, and then taking it away after everything started up. That seems to work OK, with the machine-learning features all still working correctly after taking away the permission, including alterations to the vector tables when updating the CLIP model. Restarting the containers also didn't cause any issues.

Step to achieve this.

ALTER USER "postgresql-immich-pgsql" WITH SUPERUSER;
Run Immich or use helm.
Wait for Up.
ALTER USER "postgresql-immich-pgsql" WITH NOSUPERUSER;


Translate concepts/cluster-administration/cloud-providers/ into Japanese

What this PR does / why we need it:
This page has not been translated.
Original file is as follows:
https://github.com/kubernetes/website/blob/master/content/en/docs/concepts/cluster-administration/cloud-providers.md
Which issue(s) this PR fixes:
Close #23120 .
Special notes for your reviewer:
None.

/assign @inductor

ご指摘ありがとうございます

ご対応ありがとうございます🙏
/lgtm

/approve


Adds support for Jaccard bag/multiset semantics

This commit adds support for calculating Jaccard similarity using
bag/multiset semantics, as described on pgs. 76-77 in chapter 3 of
Mining of Massive Datasets (MMDS).
MMDS uses the example of movie ratings:

If ratings are 1-to-5-stars, put a movie in a customer's set n times
if they rated the movie n-stars. Then, use Jaccard similarity for bags
when measuring the similarity of customers. The Jaccard similarity for
bags B and C is defined by counting an element n times in the
intersection if n is the minimum of the number of times the element
appears in B and C. In the union, we count the element the sum of the
number of times it appears in B and C.

To fit a model using bag semantics, the user:

Instantiates an LSH model with the variable repeatedItems set to
true. This is an optional variable that is false by default.
Passes their data into the model as a List or RDD of SparseVectors,
where the indices of each SparseVector correspond to the distinct
items in the set, and the values of each correspond to the number of
times each corresponding item is repeated in the set.

The only difference in running the model and getting output is that the
Jaccard similarity between two sets with bag semantics has a maximum of
0.5 rather than 1.0.

Also: I just added a couple lines in the README about the repeatedItems option, but I'm happy to also add an example if needed.


Add contact form validation

Pull Request Template
Description
I've made use of state hooks and conditional checks to validate the contact form. This system takes into account the validation in real-time, meaning the user need not click on submit explicitly to find out if the form is valid or not. Also, since the contact page component doesn't render any children components, there'd be no re-renders of children components unnecessarily(even if it did, we could make use of React.memo to prevent unnecessary re-renders). Additionally, I've also added this feature where the submit button is enabled only if all the inputs are valid, this would make sure that the user enters valid inputs in all the fields. Kindly refer to the attached video to get a clearer insight.
Fixes #123
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).

How Has This Been Tested?
Manual testing has been performed in various browsers to make sure the behavior is consistent across.
Additional Context (Please include any Screenshots/gifs if relevant)
https://user-images.githubusercontent.com/22348265/162789461-1ffd0d25-6a7f-4e84-a70b-10e240888eef.mp4
...
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.

@padmajabhol @yochannah @ikayz, Kindly review this PR and let me know if there are any more changes to be made. Thank you.

@padmajabhol , The current implementation uses instant validation only. In the video, I'm just showing that the submit button will not be enabled until all the inputs are valid. There is no need to click on the submit button to get an error message. You can verify the same in the code, where I've hooked validation functions to the onChange prop of every input field, meaning validation is checked on every keystroke. Regarding the email format, when we use input type to be email the HTML itself takes care of the correct formatting, hence I didn't write an explicit function to validate the same.  But overall, the current implementation uses instant validation only. Kindly verify the same and let me know if there are any changes or updates to be made.

@padmajabhol , The current implementation uses instant validation only. In the video, I'm just showing that the submit button will not be enabled until all the inputs are valid. There is no need to click on the submit button to get an error message. You can verify the same in the code, where I've hooked validation functions to the onChange prop of every input field, meaning validation is checked on every keystroke. Regarding the email format, when we use input type to be email the HTML itself takes care of the correct formatting, hence I didn't write an explicit function to validate the same. But overall, the current implementation uses instant validation only. Kindly verify the same and let me know if there are any changes or updates to be made.

From instant validation, I meant their could be a warning right after the user leaves the input field. Maybe you could check this out https://stackoverflow.com/questions/38791919/call-a-function-after-leaving-input-field
Otherwise I think this is good to go and we can modify it later too.

I believe that has been added too. I've hooked the functions to check for validation right after the user leaves the input field. This is accomplished by using the onBlur prop, I verified the same by looking through the answers posted in the Stackoverflow link that you've shared. The behavior is also reflected on the UI front. :)


fix cache path in windows system

windows cache path #191

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Genesis should not bail when encountering dev versions of supporting utilities.

While #150/#152 fixes the abiguous error message, the behaviour is still at odds with the way genesis v1 behaved.  In v1, if you were using a dev version, it would warn you, but allow you to continue.  I think that behaviour is necessary in order to vet new versions of supporting utilities.

The GENESIS_DEV_MODE env var can be set to force using dev-binaries.

On Jul 31, 2017, at 11:31 AM, Dennis J. Bell<EMAIL_ADDRESS>wrote:
While #150 https://github.com/starkandwayne/genesis/issues/150 fixes the abiguous error message, the behaviour is still at odds with the way genesis v1 behaved. In v1, if you were using a dev version, it would warn you, but allow you to continue. I think that behaviour is necessary in order to vet new versions of supporting utilities.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/starkandwayne/genesis/issues/154, or mute the thread https://github.com/notifications/unsubscribe-auth/AG2SUlTA0UfdRHbsJWe61jAHshkmamVGks5sTfNmgaJpZM4Ooj9H.

In that case, the test for development build is lacking:
		unless(envset('GENESIS_DEV_MODE') && $version =~ /development build/) {

Spruce did not report development build, it reported (development)

Fixed on master


Add potential with additional site wise predictions

Since CHGNet also predicts site-wise (magmoms), we need a Potential that allows that.

This is a suggestion to minimize repeated code.
Please let me know what you think @kenko911  and @shyuep

Looks good to me. Though maybe we can just have one Potential class that includes a kwarg indicating if there is a site-wise prediction?

Looks good to me. Though maybe we can just have one Potential class that includes a kwarg indicating if there is a site-wise prediction?

Ok, that sounds good and is even simpler!

Just a quick note that right now, the Calculator and Relaxer are called M3GNetCalculator. If there is no good reason why the calculator needs to be tied to a specific potential, we can rename it simply as MGLCalculator instead. Then the Calculator and Relaxer classes are agnostic to whether it is M3gnet, CHGNet or whatever Net that comes after.
But if you do change the names of the calculators, pls still add in a M3GNetCalculator(MGLCalculator) for backwards compatibility for now. We can remove it at a later date.

I updated the code to have a single Potential class and simply check the return value from it's model.

@lbluque Sorry for the late reply. The code looks fine to me. However, I am not sure if the def forward in Potential class should output other atomwise properties like charges and spins. What do you think @shyuep and @lbluque since now the . Would it possible to create another function in the Potential class to output atomwise properties instead of breaking the energy, force, stress, hessian calculations into two parts?

I think this is fine. The forward method in a Module is supposed to output predictions. Things like forces and stresses are in fact site wise predictions. Magmom is just another site-wise prediction. I will merge and do a bit of cleanup.


Docs/guide tests

Proposed changes
Implement guide test base class and the first test for the quickstart guide.
Code blocks in the documentation have been moved to external .txt files so they are easier to organize and parse without conflating different guides or codeblocks in the same file that should be run separately. Optionally, this could be achieved by parsing the markdown files but hardcoding block separation when needed.
These tests collect all the commands in the txt files, run those commands in order and check that they finish succesfully.
Pending implementation:

[ ] Command output check
[ ] Simultaneous commands (for example, docker logs check)
[ ] Skip comments in txt files

Fixes
n/a
Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an x in the box that applies

[x] Non-breaking fix (non-breaking change which fixes an issue)
[ ] Breaking fix (breaking change which fixes an issue)
[ ] Non-breaking feature (non-breaking change which adds functionality)
[ ] Breaking feature (breaking change which adds functionality)
[ ] Refactor (non-breaking change which changes implementation)
[ ] Messy (mixture of the above - requires explanation!)

Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING doc
[x] I am making a pull request against the main branch (left side). Also you should start your branch off our main.
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have locally run services that could be impacted and they do not present failures derived from my changes

I think the am of this PR is important, as it is true that documentation comands can get outdated easily. However, I feel the current approach of the PR will make somewhat harder to maintain documentation. Here is my proposal on how to tackle this issue, which, of course, can be discarded/overriden by @DavidMinarsch.
Instead of writing the commands in separate .txt files, wirte a "magic comment" that will be ignored by the Markdown parser, but that our script will interpret and "virtually generate" these .txt file contents on the fly. For example:
Please follow this instructions to fetch and deploy the Hello world service:

<--- test_this_code_magic_comment -->
'''bash
autonomy fetch bafibey123451....
cd hello_world
cp ../keys.json .
autonomy deploy
'''

The script will

search for occurrences of the magic comment <--- test_this_code_magic_comment --> (change approppriately)
collect the commands between the first opening ''' and the immediate ''' following (I think it's called greedy match),
possibly remove the language 'bash' and do any other required sanitizations.
do whatever checks it is doing now in the separate .txt files.

This has the advantage of not "dismembering" the page, enhancing manteinability, with little effort (just adding one comment on top of each code that needs to be tested).
It should be decided: (1) what is the best name for this magic comment, and (2) what is the best way to use comments in markdown. The one presented works, but it leaves the comment in the output HTML file (i.e., an user can see the comment if he views the source code of the page). See https://stackoverflow.com/questions/4823468/comments-in-markdown  for how to avoid this situation.

Alternatively, it can be enforced that every code block comment in the documentation has either a magic comment to ignore it OR a magic comment to verify the block. For example:
With this comment, the linter should parse and verify the code block:
<!-- LINTER_VERIFY -->
'''bash
cd <directory>
ls
autonomy fetch ...
'''

With this comment, the linter should ignore the code block:
<!-- LINTER_IGNORE -->
'''bash
cd <directory>
ls
autonomy fetch ...
'''

If no magic comment on a code block --> linter should fail (instead of ignore). This way we ensure that all ignored code blocks are indeed intended to be ignored.
'''bash
cd <directory>
ls
autonomy fetch ...
'''

We will take a different approach as suggested by @jmoreira-valory


MultipleSpaceClean and utrim method was added into Str helper.

multipleSpaceClean method was added into Str helper.
The inserted method clears multiple whitespace characters in a string.
Example:
$input = 'a b  c   d    e';
$output = Str:multipleSpaceClean($input);
echo $output; // Result: a b c d e
trim method was added into Str helper.
Trim that additionally removes slashes.
$input = "\hello world \n";
$output = Str:utrim($input);
echo $output; // Result: hello world

I want my contribution, if not accepted, I will publish it as a package :).
Thanks in advance.


Interpolate [name] in loader query param

Would be really great if ExtractTextPlugin.extract() had the ability to interpolate [name] so I could so something like:
...
    entry: {
        thing1: 'some path',
        thing2: 'a different path'
    },
    thing1: {
           // special loader config for thing1
    },
    thing2: {
           // special loader config for thing2
    },
    module: {
        loaders: [
            {
                test: /\.scss$/,
                loader: ExtractTextPlugin.extract(['css', 'postcss', 'sass?config=[name]'])
            }, ...
        ]
    }

Is this technically possible within the  ExtractTextPlugin.extract() (Loader) and I wonder if this would be the best way to achieve this? I mean e.g sass-loader could interpolate the [name] aswell since it's an option passed to it?

@track0x1 can your describe use case for interpolate [name] in loader query param?

Closing due to inactivity. Please test with latest version and feel free to reopen if still regressions. Thanks!


cannot run ImageDemo.py

Hi,
when I run ImageDemo.py ,some mistakes always happen.
eg:ValueError: mismatch: parameter has shape (256, 2) but value to set has shape (256, 3136)
Thanks for your reply!

Can you give more context, for example the full text of the exception with trace and line number?
Thanks,
Marek

HI,
How do I run imageDemo.py without encountering error.
The error message is;
Traceback (most recent call last):
File "C:\Users\xpert\Desktop\Data Analysis\Python Programming\Coding Files\C8Ex8 - ImageDemo.py", line 42, in
imageDemo()  # Create GUI
^^^^^^^^^^^
File "C:\Users\xpert\Desktop\Data Analysis\Python Programming\Coding Files\C8Ex8 - ImageDemo.py", line 10, in init
caImage = PhotoImage(file="image/ca.gif")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Program Files\Python311\Lib\tkinter_init_.py", line 4130, in init
Image.init(self, 'photo', name, cnf, master, **kw)
File "C:\Program Files\Python311\Lib\tkinter_init_.py", line 4075, in init
self.tk.call(('image', 'create', imgtype, name,) + options)
_tkinter.TclError: couldn't recognize data in image file "image/ca.gif"


Implement plugin wallet endpoints

new endpoints where emplemented here: https://github.com/massalabs/thyra-plugin-wallet/blob/main/api/walletApi-V0.yml
We should update our code to match those endpoints.
This Issue will be blocked until a new release of the plugin wallet (to be synch with @SlnPons for the release and with @Thykof for how to test the code using the main branche of tha repo)

to play with the wallet API you can use the postman collection.

follow the readme to install dependencies
go generate ./...
STANDALONE=1 wails dev


Support for existential preconditions

Add support for existential preconditions.
I still need to adjust the evaluate function:
https://github.com/PlanSys2/ros2_planning_system/blob/a7971965431fc5c837074e6a15059650f75645b8/plansys2_problem_expert/src/plansys2_problem_expert/Utils.cpp#L29
I am thinking about solving this by getting all instances with the problem client and evaluating the exists tree for all possible instance combinations until one is satisfied.
related to #317

I included the evaluation for the exists clause. It checks if any combination of instances fulfills the exists condition. It is not optimal, but it is the only solution I could think of. Only works when use_state is false
https://github.com/PlanSys2/ros2_planning_system/blob/bf54092e9eccdaf34aae1237bd7c56536a0b1809/plansys2_problem_expert/src/plansys2_problem_expert/Utils.cpp#L305-L344

Hi @Rezenders
I think everything is ok. Thanks!!
Merging!!! 🚀


fix: test quick install

Summary of changes

Changes introduced in this pull request:
Only used to reproduce and explain the CI failure
Reference issue to close (if applicable)

Closes
Other information and links

Change checklist

[ ] I have performed a self-review of my own code,
[ ] I have made corresponding changes to the documentation,
[ ] I have added tests that prove my fix is effective or that my feature works (if possible),
[ ] I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

Let's close it @hanabi1224?


Bring Prometheus/Grafana back

Restore the Prometheus/Grafana dashboard functionality. It was lost during move to oats2 and new docker-compose strategy.

@abalmos What still needs to be done to close this issue/

Nothing. This work requires no PRs to isoblue-avena. One day, we should have some example compose files for container-maintainer and then we could include node-exporter, cadvisor, etc.
Setting up prometheus is not dependent on Avena. We could show some example configs and a dashboards though?


Fix #20

See the issue for more information.
This fixes a layer-8 bug done at programming time.

I ran into this bug today. cc @Urucas

cc @vrunoa and @pamepros, in case they're not watching notifications for the Urucas org. :)

@vrunoa: Done! Tell me if I did what you asked me correctly.

v0.0.9 Published
https://www.npmjs.com/package/electron-packager-interactive

Thank you!

Nice. Looking forward to giving this project another try.


Minor typing changes

Resolved typing issues from generic type Observable and TemplateRef
Both required 1 type argument, so set both to any. Observable and
TemplateRef. It worked/compiled without type arguments, just added them
to be proper. If possible to make type more specific I'd like to know.

@shannonjin , something is wrong with your PR (too many changes and conflicts). I will close it. Please recreate.


Not getting views

First try with only links:

Second try with only search

screen resolution is the problem here ..can you check the VM's screen resolution? It must be at least 1024x768

And also, how can I see the made view count on database.db?

I can set the upper one windows to 1024x768 but linux (the second screen) is only server, no desktop. I connect to it via ssh
Here is full screen resolution

use xvfb for linux but imo stick with windows if you don't have GUI on linux.. with xvfb it seems views get deleted too much
to access the database in config.json set http_api enabled true.. then if you're on RDP get the public IP of that and browse that PUBLIC_IP:5000 ..from the ss of yours the public IP is <IP_ADDRESS>.. before that make sure to open the port 5000 like from networking tab in azure

Ok ill try. And what kind of view source should I use to make easiest and fastest views? Cause my pc crashed when I started with search feature, and when I used with link, on Duck Duck Go referrer youtube blocks it saying "too many requests from your ip" (or sth similar).
You see, I just want to try to make some views, as an experiment, I don't care about view source. When I succeed to make my first 100 views then I will try to experiment with other, more advanced features.

use both for better results.. also lower the thread numbers to avoid the crush and use premium proxies to avoid blocked messages as well as fast performance..

Use both you mean fill in both files, or fill in just search, than another time just url? And also is my processor enough good for this?  (i7-6700 8M Cache, up to 4.00 GHz, 4 Cores, 8 Total Threads)

I mean fill in both urls.txt and search.txt.. yes your processor is good enough... if you use premium proxies every thread will be working so say 2 thread will be enough.. unlike with free proxies we need to use a large number of threads because sometimes it hits multiple good proxies sometimes it don't at all also views don't stay with free proxies... so I'm guessing higher thread is the reason of your system crush..


PHP7 Support

I think that this needs a lot of updates with the new pthreads version.

@PEMapModder is correct. I am not going to merge these API update PRs, without reasonable assurance that the API version is supported.


Error when converting file to coco

Hi I'm new to this (github and machine learning) and I'm sorry if I didn't follow any guidelines etc. before posting and issue.
So I was running this (MMDet_Tutorial.ipynb) tutorial and there's an error when I tried to run the following codes in the picture below;

I'm simply just running the entire tutorial without doing any modification and I was planning on doing it later using different dataset. Did I skip a few steps here or I simply done a typo ? I'm sorry for the trouble.

I managed to solve this by changing the previous code cell at line 9
for idx, v in enumerate(mmengine.track_iter_progress(data_infos.values())):
to this
for idx, v in enumerate(mmengine.track_iter_progress(list(data_infos.values()))):


Dye the windows of the facility

Hello,
Is it possible to color the windows of the office and the bedroom in facility.lua? This possibility is active on GTA Online?
Thank you very much.

Feature added in the current version (2.0.8).


Close drawer feature

Working on #76
Not quite working yet. Added a "close button" to the sheet temporarily for debugging purposes. Close feature is working there but not in the code submission flow.

Will revisit later


gRPC service that returns Exception cannot restart

What version of gRPC and what language are you using?
Python, grpc
Name: grpcio
Version: 1.47.5
What operating system (Linux, Windows,...) and version?
Linux
What runtime / compiler are you using (e.g. python version or version of gcc)
Python 3.10
What did you do?
I have an example repo in https://github.com/JoanFM/grpc-error-show where u can reproduce the issue in an venv.
You can call run.py and u will see.
I start in a for loop grpc servers in a for loop, these servers will return an Exception, and these servers will be started in a subprocess.
Once the server starts, a request will be sent that raises an Exception.
The problem I see is that from the 2nd server, it does not succeed to receive calls.
What did you expect to see?
The servers should start smoothly and work fine and receive requests
What did you see instead?
Blocking behavior. Also is worth noticing that there always appear this message:
E1118 11:44:10.225386085   25592 fork_posix.cc:76]           Other threads are currently calling into gRPC, skipping fork() handlers

Anything else we should know about your project / environment?
The repo I propose is a simplification of what is done in Jina project, and this is affecting our testing because we start a lot of grpc services constantly.

Hi, thanks for the reproduction code, we'll try reproduce it locally.
I also noticed you're using 1.47.5, does this issue still exist in recent versions?

This does not happen with newest, but I see SegFaults that are hard to reprodue, I wonder if the source may be the same.

This does not happen with newest, but I see SegFaults that are hard to reprodue

Glad to hear that this particular issue has been resolved. If you're able to provide any info about segfaults you've encountered, please create a new issue with the information on it.


Connection reset by peer when trying to download library

Description
When try to run go mod tidy or go get, getting a connection reset by peer

Steps to reproduce the issue:
go mod tidy
(and)
go get github.com/xuri/excelize/v2

Describe the results you received:
github.com/xuri/excelize/v2<EMAIL_ADDRESS>read<EMAIL_ADDRESS>read tcp
:46292-><IP_ADDRESS>:443: read: connection reset by peer
nslookup <IP_ADDRESS>
<IP_ADDRESS>.in-addr.arpa    name = sfo03s25-in-f17.1e100.net.
Describe the results you expected:
Successful download of excelize package just like other go packages from github.
Output of go version:
go version go1.20.5 linux/amd64

Excelize version or commit ID:
<EMAIL_ADDRESS>
Environment details (OS, Microsoft Excel™ version, physical, etc.):
RHEL 7 running go 1.20.5 downloaded from https://go.dev

I tried to modify the GOPROXY using
go env -w GOPROXY=direct

and
go get  github.com/xuri/excelize/v2

go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>invalid version: git fetch --unshallow -f origin in /root/go/pkg/mod/cache/vcs/d7307bee4c4ea6392fc96a77e51c2a492fb1cac611e6d73d5197c5d523473656: exit status 128:
        fatal: git fetch-pack: expected shallow list

Thanks for your issue. Please check your network or try to use another goproxy service, and you can also download the project template from here in vendor mode and use it offline:
go run -mod=vendor main.go

I've closed this issue. If you have any questions, please let me know, and reopen this anytime.


Fix ms-rest-nodeauth interactiveLogin

Seeing the following stack trace when attempting to use ms-rest-nodeauth's interactiveLogin method:
Error: Entry not found in cache.
    at Logger.createError (/Users/rigibson/src/ms-rest-nodeauth/node_modules/adal-node/lib/log.js:216:13)
    at TokenRequest.<anonymous> (/Users/rigibson/src/ms-rest-nodeauth/node_modules/adal-node/lib/token-request.js:526:40)
    at /Users/rigibson/src/ms-rest-nodeauth/node_modules/adal-node/lib/token-request.js:148:20
    at /Users/rigibson/src/ms-rest-nodeauth/node_modules/adal-node/lib/cache-driver.js:361:7
    at /Users/rigibson/src/ms-rest-nodeauth/node_modules/adal-node/lib/cache-driver.js:231:5
    at /Users/rigibson/src/ms-rest-nodeauth/node_modules/adal-node/lib/cache-driver.js:163:5
    at MemoryCache.find (/Users/rigibson/src/ms-rest-nodeauth/node_modules/adal-node/lib/memory-cache.js:89:3)
    at CacheDriver._find (/Users/rigibson/src/ms-rest-nodeauth/node_modules/adal-node/lib/cache-driver.js:140:15)
    at CacheDriver._getPotentialEntries (/Users/rigibson/src/ms-rest-nodeauth/node_modules/adal-node/lib/cache-driver.js:161:8)
    at CacheDriver._loadSingleEntryFromCache (/Users/rigibson/src/ms-rest-nodeauth/node_modules/adal-node/lib/cache-driver.js:194:8)

Depends on https://github.com/Azure/ms-rest-nodeauth/issues/5.

@kpajdzik - Please take a look at this issue as well.

Fixed by https://github.com/Azure/ms-rest-nodeauth/pull/8

This is still an issue as of 4/23/2021. Using interactive login with ms-rest-nodeauth v3.0.9 -- it appears to only happen whenever the response returns a non 2xx status. In my case one 404 would result in this error for the next call.
It does not happen with service principals.

@sadasant is this an issue in ms-rest-nodeauth or ms-rest-js? If former, could we transfer the issue to ms-rest-nodeauth repo?


Add edit button to profile page

So far editing one's personal profile is done via accessing 'settings' in the three dot menu in the upper right (desktop view) or the three bar menu in the upper left (mobile view).
It would be much more intuitive to have an edit button directly on the profile page. The green pencil button, like the one we use in the store view, would be the logical choice.

We would like to give this a try

Great! Go for it! :smile_cat:

Hello @djahnie, I am working on this issue but I am unable to find the green pencil button.Can you give a picture or the font awesome icon name or the exact place where it is used in store view.

@agarwalpooja Try out Vue Devtools, it's a browser extension that gives you nice overview and insight. I'm sure you can find the button with it :)

Thank you.
On Feb 24, 2018 4:51 PM, "Tilmann Becker"<EMAIL_ADDRESS>wrote:

@agarwalpooja https://github.com/agarwalpooja Try out Vue Devtools
https://github.com/vuejs/vue-devtools, it's a browser extension that
gives you nice overview and insight. I'm sure you can find the button with
it :)
[image: vuejsdevtool]
https://user-images.githubusercontent.com/31616/36394399-6e364c92-15b4-11e8-8901-b0f988a3d973.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yunity/karrot-frontend/issues/890#issuecomment-368221466,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdNovTztco6qcc67lHokehjWjvFA_wriks5tX_ClgaJpZM4SLyKz
.

I have added an edit button which on click is redirecting to settings page. Is this button icon fine or its alignment, color needs to be changed.

Great that you made it work! I guess we can discuss a bit more on the design, could you open a Pull Request for it?

Yup, sure.
On Feb 25, 2018 3:28 PM, "Tilmann Becker"<EMAIL_ADDRESS>wrote:
Great that you made it work! I guess we can discuss a bit more on the
design, could you open a Pull Request for it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yunity/karrot-frontend/issues/890#issuecomment-368296998,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdNovSfTqYb77nflqZ4evABzYHVGWo4sks5tYS7IgaJpZM4SLyKz
.

I have changed the icon and tried to add it at various positions. Which position is appropriate or the position still needs to be changed.

Great! I think I like the second one you posted best, but I would put the button on the left hand side of the map.

Okay, I will change the position.

Is this fine?

Is this fine?

Looks good to me! Thanks! :slightly_smiling_face:

Welcome

Welcome.


CPU usage really high 15%+

Without vtop running:

With vtop

Running Ubuntu 16.04, Node v7.2.0

One of the issues I found is in the draw interval: https://github.com/MrRio/vtop/blob/master/app.js#L635
It is set to redraw the screen every 100 milliseconds, which in my opinion is too high. My environment shows that vtop consumes 25% of CPU, after changing that to 1000 milliseconds it decreased do 9.5%
It's possible add an options to the cli to set this or even a shortcut in vtop, but honestly I don't know why would someone need 100 milliseconds of visualization of these resources if poll time for each plugin is set to 200 milliseconds
So given the description above, we have 2 options:

Make the redraw time to be configured by the user either in the cli or in a shortcut
Hard-code redraw time to 1000 milliseconds since no one would need this visualization updated in in short period of time

I vote for number 2.