Problem with importing large dump file

Try to import JCR_ALBATROS.sql (almost 10G) from r2d2 tmp/sorin into a mysql 5.6 database.
The upload counts up to 50% two times then:
This site can’t be reached
The connection was reset.
Can you please try this from your PC as well, Dani?

Yup, something definitely goes wrong. The following is logged:
2017/04/20 10:00:11 http: panic serving <IP_ADDRESS>:38270: runtime error: invalid memory address or nil pointer dereference
goroutine 115164 [running]:
net/http.(*conn).serve.func1(0xc4202c74a0)
	/usr/local/go/src/net/http/server.go:1721 +0xd0
panic(0x821c40, 0xaa2bb0)
	/usr/local/go/src/runtime/panic.go:489 +0x2cf
main.importAction(0xa7eac0, 0xc42041a7e0, 0xc4205c4700)
	/home/javdaniel/go/src/github.com/djavorszky/ddn/server/handlers.go:62 +0x2d2
net/http.HandlerFunc.ServeHTTP(0x8bafe8, 0xa7eac0, 0xc42041a7e0, 0xc4205c4700)
	/usr/local/go/src/net/http/server.go:1942 +0x44
github.com/djavorszky/ddn/common/srv.Logger.func1(0xa7eac0, 0xc42041a7e0, 0xc4205c4700)
	/home/javdaniel/go/src/github.com/djavorszky/ddn/common/srv/srv.go:14 +0x9c
net/http.HandlerFunc.ServeHTTP(0xc42012f700, 0xa7eac0, 0xc42041a7e0, 0xc4205c4700)
	/usr/local/go/src/net/http/server.go:1942 +0x44
github.com/gorilla/mux.(*Router).ServeHTTP(0xc42011a870, 0xa7eac0, 0xc42041a7e0, 0xc4205c4700)
	/home/javdaniel/go/src/github.com/gorilla/mux/mux.go:114 +0x10c
net/http.serverHandler.ServeHTTP(0xc42018c000, 0xa7eac0, 0xc42041a7e0, 0xc420327d00)
	/usr/local/go/src/net/http/server.go:2568 +0x92
net/http.(*conn).serve(0xc4202c74a0, 0xa7f240, 0xc4205178c0)
	/usr/local/go/src/net/http/server.go:1825 +0x612
created by net/http.(*Server).Serve
	/usr/local/go/src/net/http/server.go:2668 +0x2ce

I'm not entirely sure why though - the panic comes from here, but I'm not sure why - it works with smaller files.
I just tested it locally, after about a minute of upload, it is uploaded successfully, so I believe the root cause of the issue is that the upload takes too long and the web server in front drops the connection.
I'll look into what we could do - e.g. see if we can upload in an async way or something along that line - I'll also talk with the sysadmin about it.

Hi Dani,
Any news on this?

We're working on it. With an increased timeout, the upload got to 99%, then failed. Our sysadmin expressed concerns about modifying the webserver of that particular server, as it has some public facing sites as well, and he doesn't want to take chances.
As a next step, we're moving the ddn server to another server instance over which we have more authority and control, should be done later today or early tomorrow.
Note though, that the configuration files of the connectors may need to be updated to point to the new address. I'll let you know once there's more progress.

I've tested this further:
Works locally without webserver
Works locally with webserver
Does not work remote without webserver
Does not work remote without webserver
It looks like at the 30 second mark the connection is reset - most probably network configuration.

Hi Zsolt,
Could you help us in getting Péter Erdei to have a look at this? It seems
to be a problem caused by our internal network settings, so maybe he could
help.
https://github.com/djavorszky/ddn/issues/91
With this problem the CloudDB will be pretty much useless when we want to
import larger dumps, which can frequently be the case.
Regards.
On Wed, May 3, 2017 at 10:34 AM, Daniel Javorszky<EMAIL_ADDRESS>wrote:

I've tested this further:
Works locally without webserver
Works locally with webserver
Does not work remote without webserver
Does not work remote without webserver
It looks like at the 30 second mark the connection is reset - most
probably network configuration.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/djavorszky/ddn/issues/91#issuecomment-298852207, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGyJTXYGur1iQhAzvcho01DPuf4R-SOxks5r2DwMgaJpZM4NB9QC
.

Alternatively, I'm thinking about implementing #66 or some kind of async upload, if that could work.

Feature is implemented (see #66) and live


BeanCreationException while launching the jenkins-codequality-3.1.0-SNAPSHOT.jar

application.properties.txt
Hi,
I built hygieia-codequality-jenkins-collector and deployed using the below command:
java -jar jenkins-codequality-3.1.0-SNAPSHOT.jar --spring.config.name=jenkins-codequality --spring.config.location=..\application.properties
I am getting the BeanCreationException.  Attached is my application.properties file and the error message.  Please help.

Thanks,
Sunandha Ramasamy

@sunabala looks like its unable to locate the properties file. Can you give full path to properties file while launching the collector

@rvema, I had provided the complete path to properties file while launching the collector... Still the issue exists.

On further debugging, found that this.settings.getCron() call in public String getCron() method of JenkinsCodeQualityCollectorTask.java is null.  What needs to be done for this to work?

the cron property is not properly formed or the property file is unable to be read. Can you check if other properties mentioned in the application.properties are read?

Yes it is reading the application.properties file.  It is able to establish the connection with MONGO DB based on the DB credentials provided in the properties file.
This is the content of the properties file:
Database Name
dbname=dashboarddb
Database HostName - default is localhost
dbhost=localhost
Database Port - default is 27017
dbport=27017
MongoDB replicaset
dbreplicaset=false
dbhostport=localhost:27017
Database Username - default is blank
dbusername=dashboarduser
Database Password - default is blank
dbpassword=dbpassword
Collector schedule (required)
jenkins-codequality.cron=0 0/1 * * * *
Collector servers (required) - can be multiple
jenkins.servers[0]=https://jenkins.company.com
Collector types (note not required, but the regex should be match only the type specified)
jenkins-codequality.artifactRegex.junit=TEST-.*\.xml
jenkins-codequality.artifactRegex.findbugs=findbugsXml.xml
jenkins-codequality.artifactRegex.pmd=pmd.xml
jenkins-codequality.artifactRegex.checkstyle=checkstyle-result.xml
jenkins-codequality.artifactRegex.jacoco=jacoco.xml
Collector job depth (required) should be set to at least 1, and more if you use folder jobs etc
jenkins-codequality.jobDepth=4


Release v0.4.2

#348 の修正のため、パッチリリースします。
藤井研の人が使うため、早めに進めたいです。

Codecov Report

Merging #368 (26e2913) into dev (8879375) will increase coverage by 0.15%.
The diff coverage is n/a.

:exclamation: Current head 26e2913 differs from pull request most recent head 6438d2e. Consider uploading reports for the commit 6438d2e to get more accurate results

@@            Coverage Diff             @@
##              dev     #368      +/-   ##
==========================================
+ Coverage   70.95%   71.11%   +0.15%
==========================================
  Files          84       84
  Lines        5971     5971
==========================================
+ Hits         4237     4246       +9
+ Misses       1734     1725       -9

Impacted Files
Coverage Δ

src/csim/stat_ops_expectation_value.cpp
78.48% <0.00%> (-17.73%)
:arrow_down:

src/csim/update_ops_matrix_diagonal_single.cpp
84.44% <0.00%> (-13.34%)
:arrow_down:

src/csim/update_ops_pauli_single.cpp
85.41% <0.00%> (-6.25%)
:arrow_down:

src/csim/update_ops_pauli_multi.cpp
65.62% <0.00%> (+1.25%)
:arrow_up:

src/csim/update_ops_matrix_dense_single.cpp
98.87% <0.00%> (+10.11%)
:arrow_up:

...rc/csim/update_ops_control_multi_target_single.cpp
97.61% <0.00%> (+16.66%)
:arrow_up:

Continue to review full report at Codecov.

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


mobfox ad load is not working

java.net.ConnectException: Failed to connect to my.mobfox.com/<IP_ADDRESS>:443

Hi,
Can you please send us a network log such as a "Charles Proxy" log?
Thanks,
Itamar

Could you able to fix it? I'm also getting this error.

was this ever resolved? I am trying to implement the Mobfox SDK and getting the same error

@nabriski


Keep local state for unread entries

Right now in the unread view, we make an entry item appear "read" if the data in the /feed/{feed_uuid}/entries state has a read_at value or if the actual entry itself has a read_at value (this is true when you view an entry for the first time).
That last bit works well enough but if the feed entries state changes for whatever reason (switching tabs and coming back, after a long enough interval, etc.) then the just-read entry disappears from the list.
We need to keep a local state in the <EntryList /> (i.e. don't use the tanstack-query state directly) component so that as long as the user is still viewing entries from that feed, we don't remove any entries unexpectedly.
This will require that we add entries to that local state if new entries are added to the tanstack-query state, and we need to keep the functionality (in one way or another) to make entries appear as read as you click through them. Navigating to another page should reset the local state (probably to whatever is in the query cache for that feed).
Something like this:
// Local state to avoid entries being removed unexpectedly
const [localEntries, setLocalEntries] = createSignal(entries.getAllEntries());

createEffect(() => {
  const currentUuids = localEntries().map(entry => entry.uuid);
  const newEntries = entries.getAllEntries().filter(entry => !currentUuids.includes(entry.uuid));

  // Don't bother updating local state if we've got nothing to add
  if (!newEntries.length) return;

  setLocalEntries([...localEntries(), ...newEntries]);
})

Reopening this because the order must be preserved when pulling in the new entries, which is a little tricky depending on the sort order...


What's missing for wasm support?

I noticed wasm support was mentiioned as a todo. I'm wondering what is missing for wasm support?

I'm currently kind of busy working on my company project for the recent couple of months, so I don't have much time to work on my side projects.

Oh sorry to imply that. I was more curious on what is needed so I could contribute :) (I'm a noob with respect to wasm)


Fix/test locale

Fixed a test broken if run with non-english locales.
Also setting travis to test with locales:
C
pl_PL
zh_CN
so any errors depending on locale should show.

excellent!


Adding new Overlay and Concept nodes to the graph.

Preparation for mk/concepts This are the cpg-core affecting changes.

LGTM. The only remarks I have is that I would prefer the edges to go from the overlay node to the underlay node, but that is more because it was always like this in my imagination. And then we would need a different edge label so it is not really something we have to change for this PR.

I can just switch the direction, if you want

LGTM. The only remarks I have is that I would prefer the edges to go from the overlay node to the underlay node, but that is more because it was always like this in my imagination. And then we would need a different edge label so it is not really something we have to change for this PR.

I can just switch the direction, if you want

AH, no that doesn't work. It needs to be outgoing from an AST node, otherwise it will not end up in the persisted graph. I will keep it as is for now.


Use generics types in ITableAdapter contract

Is this still necessary after https://github.com/evrencoskun/TableView/commit/6720a03dac8428344ae39a6faf386be3271d6998?


enforce maven-dependency-plugin; remove hamcrest

Pull Request Test Coverage Report for Build 1697

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.354%

Totals

Change from base Build 1688:
0.0%

Covered Lines:
2238

Relevant Lines:
2533

💛  - Coveralls


Added book DDD in PHP.

Added to books section book "DDD in PHP".
I'm sure this is really great book!

Done. Description already added.
PS: I know that i received message from bot,
but for a fun i replied to him!)


Priam 2.x is unable to run Apache Cassandra 2.0.17

Hi,
I got the following error message when Priam is trying to start Cassandra:
ERROR 13:43:21,820 Fatal configuration error org.apache.cassandra.exceptions.ConfigurationException: The hsha rpc_server_type is not compatible with an rpc_max_threads setting of 'unlimited'.  Please see the comments in cassandra.yaml for rpc_server_type and rpc_max_threads.         at org.apache.cassandra.config.DatabaseDescriptor.applyConfig(DatabaseDescriptor.java:339)         at org.apache.cassandra.config.DatabaseDescriptor.<clinit>(DatabaseDescriptor.java:112)         at org.apache.cassandra.service.CassandraDaemon.setup(CassandraDaemon.java:213)         at org.apache.cassandra.service.CassandraDaemon.activate(CassandraDaemon.java:554)         at org.apache.cassandra.service.CassandraDaemon.main(CassandraDaemon.java:643) The hsha rpc_server_type is not compatible with an rpc_max_threads setting of 'unlimited'.  Please see the comments in cassandra.yaml for rpc_server_type and rpc_max_threads. Fatal configuration error; unable to start. See log for stacktrace.
Using latest version of 2.x branch.
Please advise.
Regards,
Andor

Opened the following pull request:
https://github.com/Netflix/Priam/pull/491

Hi Andor, pull request #491 looks good, just one question, are these yaml parameters applicable to C* 2.1 as well?

Yes, they are. The following has been copied from cassandra.yaml in 2.1.14 release:
# Uncomment rpc_min|max_thread to set request pool size limits.
#
# Regardless of your choice of RPC server (see above), the number of maximum requests in the
# RPC thread pool dictates how many concurrent requests are possible (but if you are using the sync
# RPC server, it also dictates the number of clients that can be connected at all).
#
# The default is unlimited and thus provides no protection against clients overwhelming the server. You are
# encouraged to set a maximum that makes sense for you in production, but do keep in mind that
# rpc_max_threads represents the maximum number of client requests this server may execute concurrently.
#
# rpc_min_threads: 16
# rpc_max_threads: 2048


pull request for e2e test at 01/30/2017 07:56:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Feature: Use fzf keybindings to add additional functionality to history

Using fzf keybindings, allow for specific history items to be removed from history list in the same manner they're selected from fzf.
Example:

tap C-r
use up/down/C-k/C-j to select a history entry
use something like M-d to use fzf's execute-silent() to call a function which will delete the selected history item from the current zsh session as well as $HISTFILE
use fzf's abort() + execute() to automatically reload updated history (with removed history item removed)

This is something I will try to tackle, but need to refresh myself on advanced zsh syntax.

Thanks for all the great work, the plugin is awesome. This feature would be really great to have 👍🏼 🙏🏼


No proxy support

Installing on a vanilla Win10 VM fails when a proxy is configured which requires authentication:
[ * ] Installing Boxstarter
Exception calling "DownloadString" with "1" argument(s): "The remote server returned an error: (407) Proxy
Authentication Required."
At C:\Users\user\Desktop\commando-vm-master\install.ps1:60 char:3
+   iex ((New-Object System.Net.WebClient).DownloadString('http://boxst ...
+   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : WebException

@ST8Z6FR57ABE6A8RE9UF  Can you elaborate more on your use case, why you need to support a proxy and specifically what support do you need?

My organisation requires a proxy for Internet access which needs authentication. The install prompts for the logged on users password, but not a proxy a credential, resulting in the above error. I tried some variations on this code prior to .\install.ps1 without luck.
https://serverfault.com/questions/689949/powershell-set-default-proxy-configuration-for-all-ps-cmdlets
We also have an unauthenticated proxy, however only whitelisted URLs are accessible. Using this proxy does not give connection errors, however the module is not downloaded:
Import-Module : The specified module 'C:\ProgramData\boxstarter\boxstarter.chocolatey\boxstarter.chocolatey.psd1' was not loaded because no valid module file was found in any module directory.
At line:41 char:9
+         Import-Module "$env:ProgramData\boxstarter\boxstarter.chocola ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ResourceUnavailable: (C:\ProgramData\...chocolatey.psd1:String) [Import-Module], FileNotFoundException
    + FullyQualifiedErrorId : Modules_ModuleNotFound,Microsoft.PowerShell.Commands.ImportModuleCommand

This issue can be closed.
I manually configured the proxy settings in install.ps1 which downloaded https://boxstarter.org/bootstrapper.ps1. I even updated proxy references in those functions.
However the Chocolatey install itself makes similar calls to when running https://chocolatey.org/install.ps1
I will attempt to build when off-site, and connected without a proxy.

@ST8Z6FR57ABE6A8RE9UF Thanks I will close this issue shortly. If you would like to, feel free to detail how you manually configured proxy settings in the install.ps1 file that way others who might run into the issue can use this as reference.

I split out lines similar to this
iex ((New-Object System.Net.WebClient).DownloadString('https://boxstarter.org/bootstrapper.ps1'))

Adding proxy properties to the WebClient object
$wc = New-Object System.Net.WebClient
$wc.Proxy = [System.Net.WebProxy]::new("<proxy server>",<proxy port>)
$wc.Proxy.Credentials = (Get-Credential)
iex ($wc.DownloadString('https://boxstarter.org/bootstrapper.ps1'))

In hindsight, this would not be sustainable since basically every downloaded script (including chocolatey) makes similar calls. Updating $Global:PSDefaultParameterValues would also not work since we are referencing the .Net object directly and not using Invoke-WebRequest cmdlets.
If an authenticated proxy is needed, the only option which worked was to configure it as the system proxy in Control Panel > Internet Options, and run the install.ps1 in the context of an allowed user.


Drop support for python version that is over end-of-life date

Drops End-of-Life annouced versions as of May 2021.
https://endoflife.date/python
It will drops;

2.x
3.1.x
3.2.x
3.3.x
3.4.x
3.5.x

done


solve_triangular() not working for lower triangular matrix

The following example demonstrates that the method scipy.linalg.solve_triangular() does not work for lower triangular matrix. I intended to use this method for forward substitution. The backward substitution works fine.
Maybe I misunderstood the documentation. But I think the method should work for both cases.
Best Sebastian
import numpy as np
import scipy
# parameters
h = 1E-2
n = 3
# pertubed Laplace matrix
A  = 1/h**2 * ( np.diag( np.ones( (n-1, ) ), k=-1 ) \
    - 2.0 * np.diag( np.ones( (n,) ) ) \
    + np.diag( np.ones( (n-1, ) ), k=1 ) ) \
    + h * np.random.rand(n,n)
L  = np.tril(A)
U  = np.triu(A)
# right hand side
b = np.random.rand(n)
# solve using lower triangular matrix
x = scipy.linalg.solve(L,b)
y = scipy.linalg.solve_triangular(L,b)
print "solve lower triangular matrix"
print "||L*x-b|| = ", scipy.linalg.norm(L.dot(x)-b)
print "||L*y-b|| = ", scipy.linalg.norm(L.dot(y)-b)
# solve using upper triangular matrix
x = scipy.linalg.solve(U,b)
y = scipy.linalg.solve_triangular(U,b)
print "solve upper triangular matrix"
print "||U*x-b|| = ", scipy.linalg.norm(U.dot(x)-b)
print "||U*y-b|| = ", scipy.linalg.norm(U.dot(y)-b)

You need to use the lower parameter to tell the routine whether it's
upper or lower triangular. By default, it assumes upper triangle.

Yes you are right. This issue can be closed. I recommend to add a note to the documentation.


Temporarily revert jQuery bump

Background
@davidrans tried deploying his recent change #177 but we forgot about an unresolved rollback that happened a month or two ago.
Let's look into why the jQuery bump caused issues, but let's run the
other recent changes in the mean time.
This should allow us to use the most recent changes aside from those in https://github.com/iFixit/pulldasher/pull/173
QA
Run this branch and see if you can spot any immediately obvious problems. If there's nothing to glaring, let's merge this and let it run in the wild.

CR 🎏

Issue written when we first ran into the jQuery problem. There might be more useful info in @danielbeardsley's noggin. I'll try to update it with better details soon. https://github.com/iFixit/pulldasher/issues/174

CR :cactus:

testing
dev_block 🪂 this is testing

un_dev_block 🪂 test

QA 👍


Release build process should also push changelog to website

Tagging @n8henrie for this one.
It's my bad, but I always forget to upload the latest ChangesBare.html file to the server (final step in the process here). It'd be great if we could automate this, then we can remove it from the list!

P.S. I'll try and get round to moving the QS server this week/next week sometime.

Hmm, I didn't initially realize that this file was gitignored, had planned to just have the server download it from GH.
Will try passing via stdin. May have a few fake tag pushes to trigger.

Looks like https://github.com/quicksilver/Quicksilver/commit/390ccafde656550126377be094a91358b3dd8831 broke our build.

Along with https://github.com/quicksilver/server-scripts/commit/0d6a4df33df9f4f3b806ff65d76c79077ace8dc9 I think this should work.

Also updated the relevant GHA secrets for https://github.com/quicksilver/QSApp.com/pull/34 and https://github.com/quicksilver/manual/pull/14

Looks like https://github.com/quicksilver/Quicksilver/commit/390ccafde656550126377be094a91358b3dd8831 broke our build.
Yeah, the lang files seem to have the wrong encoding. I will fix now.

Will try passing via stdin. May have a few fake tag pushes to trigger.

I was thinking of just scp-ing it at the end of the 'Release' build process. It looks like that's what you've done though, right?

That probably would have been fine, but I already have a locked-down SSH key for GHA use, so passing the file over stdin/stdout allowed me to reuse the existing setup instead of set up a separate key for SCP (or instead of relax the restrictions on the key).

I don't really understand the difference, but if you've considered both options and picked this as the best - then all good!


python312Packages.distorm3: disable on Python 3.12

Description of changes
Still uses distutils, https://github.com/gdabah/distorm/issues/191
ZHF: https://github.com/NixOS/nixpkgs/issues/309482

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 313655 run on x86_64-linux 1

2 packages built:

python311Packages.distorm3
python311Packages.distorm3.dist


Review POST microservice API and persist data if needed

I still don't know all the details and information returned by the /microservice/data/ POST API.
Depending on what the use case will be, e.g.:

given a workflow name, return all its blocks and locations where data was subscribed, etc

we might have to persist data somewhere (central CouchDB is a good call) such that we don't need to query the data-service every time the Microservice has to handle the same query.
Thus we need to review that code and also document what it supports and what it's supposed to do.
Documentation can be written here:
https://github.com/dmwm/WMCore/wiki/ReqMgr2-MicroService-Transferor

Just an update to this ticket.
The REST APIs had a few changes and we support the following APIs for both GET and POST calls:
https://alancc7-cloud1.cern.ch/ms-transferor/data/status
https://alancc7-cloud1.cern.ch/ms-transferor/data/info
e.g.:
$ curl --cert $X509_USER_CERT --key $X509_USER_KEY -X POST -H "Content-type: application/json"  -H "Accept: application/json"  -d '{"request":"amaltaro_StepChain_DupOutMod_HG2004_Val_200331_104220_690"}' https://alancc7-cloud1.cern.ch/ms-transferor/data/info
{"result": [
 {"request": "amaltaro_StepChain_DupOutMod_HG2004_Val_200331_104220_690", "transferDoc": {"ConfigType": "TRANSFER", "lastUpdate":<PHONE_NUMBER>, "workflowName": "amaltaro_StepChain_DupOutMod_HG2004_Val_200331_104220_690", "transfers": []}}
]}

we could actually support PUT method such that we can update some information in the transfer documents. One of the items that would be useful to update is the list of transfer requests.

@amaltaro , can we close this issue?

It's more than 2 years old and there is specific issue for MSTransferor. Yes, let's close this one out as well. Thanks Valentin.


The new optical component for beam transporting

Scope
a new optical component for beam transporting
Suggestion
The existing optical components will inevitably change the direction of the beam when splitting it, which is very unfavorable for centralized management and standardized splitting of the beam. Adding new beam components that will not affect the direction of the main beam when placed in the path of the beam would allow for outputting the split beam from the side according to the size of the redstone signal.
Justification
I love this mod very much and I hope it could be better.
Impact
It will make the beam tranporting easier, but it's really convinient.

Should be fairly easy to implement. Perhaps as craftable variants of the beam splitter/siphon?

That would be great. I have spent quite a time on this mod and really love it.


Fix getIPC infinite recursion

The use of a postfix ++ operation was causing getIPC to always be recursively called with id = 0, resulting in an infinite recursion (and subsequently CPU & memory spikes). Switched to a prefix operation to fix the issue.
Might be related to #14 as well, but the RPC_CONNECTION_TIMEOUT errors are still produced.

can you change it to i + 1 thanks


GNOME 44 problems with Qt-applications

Describe the bug
Since the Gnome 44 (under wayland) update, on multiple devices reproducible errors with Qt based applications.
Steps To Reproduce

nextcloud-client: The application does not show itself after first start, it is only visible in Activities as an empty frame. You have to restart the application and it works.
keepassxc: The Drag-and-Drop functionality does not work.

This are only two errors that I found, but I think many Qt-applications are affected. The solution is to set QT_QPA_PLATFORM=wayland inside a shell and start the application. The errors vanish. Currently, my workaround is setting it global with
  environment.variables = {
    QT_QPA_PLATFORM = "wayland";
  };

Perhaps, can you add the variable as a default setting for each gnome under wayland session?
Thank you
Notify maintainers
@bobby285271
@hedning
@jtojnar
@dasj19
@maxeaubrey

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
this path will be fetched (0.01 MiB download, 0.05 MiB unpacked):
  /nix/store/pgf87rgkzdbqhhsddpligy8xiwar63w8-stdenv-linux
copying path '/nix/store/pgf87rgkzdbqhhsddpligy8xiwar63w8-stdenv-linux' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 6.3.0, NixOS, 23.05 (Stoat), 23.05pre480050.1a411f23ba2`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(root): `"nixos"`
 - channels(goetz): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Could you please try if https://github.com/NixOS/nixpkgs/pull/210077 fixes it?

Does not solve the problem.

Same issue with wireshark
QT_STYLE_OVERRIDE
https://github.com/NixOS/nixpkgs/issues/227512#issuecomment-1534686509

strawberry player is also affected. Outputs:
10:08:56.258 DEBUG unknown                          Warning: Ignoring XDG_SESSION_TYPE=wayland on Gnome. Use QT_QPA_PLATFORM=wayland to run on Wayland anyway.
When running: QT_QPA_PLATFORM=wayland strawberry drag and drop works, without that env variable drag and drop does not work.

I am also experiencing this issue, under x11. picard's drag and drop does not work, which render the program almost unusable.

#237042 has possible fix for https://gitlab.gnome.org/GNOME/mutter/-/issues/2715

Will this get backported to stable?

Already happened https://github.com/NixOS/nixpkgs/pull/237226


Remove the object manager class and use RAII to serialise everything

The current model of using the ObjectManager class to manage the objects destruction doesn't seem necessary and is error-prone. It's better to use RAII to do it instead.

Done


(SUP-3121) Agent connection to pxp broker

This Commit adds AS002 -
Determines if the pxp-agent has an established connection to a pxp broker
. This fact can also be used as a target filter for running tasks, ensuring time is not wasted sending instructions to agents not connected to a broker

Did @jarretlavallee leave a comment earlier about using Facter::Core::Execution.execute(), or am I imagining that?

I did, but I am not sure where it went. The main thing is just to rescue the exception thrown so it doesn't show a failure. Facter::Core::Execution.execute makes that easy and standardizes the shell calls.

@jarretlavallee  @m0dular  I switched to facter core, it captures the exception by default,  i didn't see any value it it returning a string, because true or false would be mis leading if its not able to determine it

Facter::Core::Execution.execute will raise a Facter::Core::Execution::ExecutionFailure on failure by default. The :on_fail option can be used, but you would have to parse the string to see that failure.


Lem hangs after M-x start-lisp-repl

I'm using latest OS version, built from sources: lem 1.5-25c95fbc (X86-64)
Same problem with Lem, installed with ros install cxxxr/lem

Interesting, tried to checkout older commits, and it does not work either.

I've checked 1.5-25c95fbc on my other laptop, and it does work!
Interesting, why does it hang on one MacBook and doesn't on another?

I have never reproduced it.
I think it's probably due to usocket or something, not a lem issue.

@cxxxr do you have an idea how to debug it?

What about print debug?
Provided the function pdebug

How should I use it?
I found that Lem hangs even when I just change current buffer's to lisp-mode.
OSX's activity monitory shows that after I call lisp-mode, Lem opens a port like localhost:54361 and connects to it localhost:51384->localhost:54361.

How about this
https://github.com/cxxxr/lem/compare/issue-411-print-debug

Thank you!

I've modified a lem a little bit such way that it does not call new-connection.
Started a separate Lisp process with an Emacs and SLY.
Loaded Lem's sources
And tried to call new-connection.
It hangs on read-return-message call after the (emacs-rex-string connection (swank:init-presentations)") on line: https://github.com/cxxxr/lem/blob/35e915a561d8faccf3802cee5dd8d4ff86b1def9/modes/lisp-mode/swank-protocol.lisp#L208

Probably using some logging library across all Lem's codebase will make debugging much easier. How about integrating a log4cl?

Also, there is no timeout in read-return-message itself:
(defun read-return-message (connection)
  "Read only ':return' message. Other messages such as ':indentation-update' are dropped."
  (loop :for waiting := (message-waiting-p connection :timeout 5)
        :with info
        :do (unless waiting (return nil))
            (setf info (read-message connection))
            (when (eq (car info) :return)
              (return info))))

that is why Lem hangs instead of giving some error like "Unable to setup SLIME connection.

Hello.
I debugged this part before.

https://github.com/cxxxr/lem/pull/436 [lisp-mode] Update swank protocol read message function
https://github.com/cxxxr/lem/pull/393 Improve slime connection on Windows v3
https://github.com/cxxxr/lem/pull/385 Improve slime connection on Windows

Sorry, I have only windows pc.
I used Wireshark-Portable and Npcap.
https://www.wireshark.org/download.html
https://nmap.org/npcap/
Npcap enabled to capture TCP packets only one pc.
(select Npcap Loopback Adapter)
(But, it seems for windows only, sorry I don't know other OS ...)
I set Wireshark's filter as follows.
tcp and ip.addr == <IP_ADDRESS>

I found the problem cause.
It fixed in this pull request: https://github.com/cxxxr/lem/pull/457
BTW, these changes were highly useful for discovering the bug: https://github.com/cxxxr/lem/pull/456 please review them.

This issue was fixed by pull https://github.com/cxxxr/lem/pull/457


Allow querying the online lemmeknow by URL

When opening the lemmeknow webpage with a URL such as this:
https://swanandx.github.io/lemmeknow-frontend/?q=search+term
It should use this as input and try to figure out what the search term could be.
Why?
Browser Search engines.
With this feature you could register lemmeknow as a search engine for your browser. You could (for example) use the alias lmk to search lemmeknow.
Then typing lmk dQw4w9WgXcQ would lead you to find out what the ID stands for.

Thanks for the suggestion, but this issue should be at lemmeknow-frontend. Can you please move it there?
https://github.com/swanandx/lemmeknow-frontend


setState While not mounted causes application to crash

Smartphone

Device: iPhone 12
OS: iOS 17.3
Library version: 8.0.4

Describe the bug
When going back to a previous page ( with some complex bloc builders ) that has map with the current location marker, the app crashes, probably because a stream listener callback  was called and it uses setState on a disposed widget.
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: This widget has been unmounted, so the State no longer has a context (and should be considered defunct).
Consider canceling any active work during "dispose" or using the "mounted" getter to determine if the State is still active.
#0      State.context.<anonymous closure> (package:flutter/src/widgets/framework.dart:951:9)
#1      State.context (package:flutter/src/widgets/framework.dart:957:6)
#2      TickerProviderStateMixin._updateTickerModeNotifier (package:flutter/src/widgets/ticker_provider.dart:328:70)
#3      TickerProviderStateMixin.createTicker (package:flutter/src/widgets/ticker_provider.dart:292:7)
#4      new AnimationController (package:flutter/src/animation/animation_controller.dart:247:21)
#5      _CurrentLocationLayerState._rotateMarker (package:flutter_map_location_marker/src/widgets/current_location_layer.dart:619:40)
#6      _CurrentLocationLayerState._subscriptHeadingStream.<anonymous closure> (package:flutter_map_location_marker/src/widgets/current_location_layer.dart:456:13)
#7      _RootZone.runUnaryGuarded (dart:async/zone.dart:1594:10)
#8      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:339:11)
.
.
.
.
.
#31 .....

Suggested Solution
Maybe wrap the setState in listeners callbacks with an if statement checking the mounted.

in my case it doesn't crash BUT the same "Unhandled Exception" errors appears in logs.

@tlserver
Still not fixed everywhere even in latest release, some places where there is a setState are stilling missing the if statement
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: This widget has been unmounted, so the State no longer has a context (and should be considered defunct).
Consider canceling any active work during “dispose” or using the “mounted” getter to determine if the State is still active.
#0      State.context.<anonymous closure> (package:flutter/src/widgets/framework.dart:951:9)
#1      State.context (package:flutter/src/widgets/framework.dart:957:6)
#2      _CurrentLocationLayerState._subscriptHeadingStream.<anonymous closure> (package:flutter_map_location_marker/src/widgets/current_location_layer.dart:443:14)
#3      _RootZone.runUnaryGuarded (dart:async/zone.dart:1594:10)
#4      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:339:11)
#5      _BufferingStreamSubscription._add (dart:async/stream_impl.dart:271:7)
#6      _ForwardingStreamSubscription._add (dart:async/stream_pipe.dart:123:11)
#7      _MapStream._handleData (dart:async/stream_pipe.dart:218:10)
#8      _ForwardingStreamSubscription._handleData (dart:async/stream_pipe.dart:153:13)
#9      _RootZone.runUnaryGuarded (dart:async/zone.dart:1594:10)
#10     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:339:11)
#11     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:271:7)
#12     _ForwardingStreamSubscription._add (dart:async/stream_pipe.dart:123:11)
#13     _WhereStream._handleData (dart:async/stream_pipe.dart:195:12)
#14     _ForwardingStreamSubscription._handleData (dart:async/stream_pipe.dart:153:13)
#15     _RootZone.runUnaryGuarded (dart:async/zone.dart:1594:10)
#16     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:339:11)
#17     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:271:7)
#18     _ForwardingStreamSubscription._add (dart:async/stream_pipe.dart:123:11)
#19     _MapStream._handleData (dart:async/stream_pipe.dart:218:10)
#20     _ForwardingStreamSubscription._handleData (dart:async/stream_pipe.dart:153:13)

brilliant, thanks for 8.0.6, that fixed the Error at least for me.


Fix Callable calls in mono module

Follow-up to #63595
The Callable::call and Callable::call_deferred methods have been renamed to Callable::callp and Callable::call_deferredp.

Thanks!


Regarding ejjaberd url option

When we used ehox plugin it had an option for entering ejjaberd url now in whixp it only giving option for connect
using jaberID password and port so could u tell where we should config the ejjaberd url

In Whixp, you can now assign the host property when configuring the instance for a connection. Additionally, anonymous authentication is supported. To use anonymous authentication, ensure that you correctly assign the host property before attempting to connect.
For example:
void main() {
  final whixp = Whixp(
    jabberID: 'vsevex@localhost/resource',
    password: 'passwd',
    logger: Log(enableWarning: true, enableError: true, includeTimestamp: true),
    internalDatabasePath: 'whixp',
    reconnectionPolicy: RandomBackoffReconnectionPolicy(1, 3),
  );

  whixp.connect();
}

This should resolve any connection issues you may encounter. Ensure that the host property is correctly configured, especially when using anonymous authentication.


Moving the 'roslyn-internal' commit pointer forward

Moving the 'roslyn-internal' commit pointer forward to consume the latest changes.

Successful Jenkins run: https://ci.dot.net/job/dotnet_roslyn-project-system/job/master/job/vsi/521/


Utility function to read snakemake rule

In GitLab by @Huite on Aug 14, 2023, 11:51
To run workflows both interactively and in a snakemake workflow, this does the trick:
def read_snakemake_rule(path, rule: str) -> "snakemake.rules.Rule":
    """
    Parameters
    ----------
    path: str, pathlib.Path
        The path to the snakefile.
    rule: str
        Name of the rule in the snakefile that runs this script.

    Returns
    -------
    rule: snakemake.rules.Rules

    Examples
    --------
    To run an example both interactively and in a workflow, e.g.:

    >>> if "snakemake" not in globals():
    >>>     snakemake = read_snakemake_rule("snakefile", rule="my_rule")
    >>> modelname = snakemake.params.modelname
    >>> template = snakemake.input["template"]
    """
    import snakemake as sm

    workflow = sm.Workflow(snakefile="snakefile")
    workflow.include(path)
    return workflow.get_rule(rule)

It's probably a good idea to add this to imod.util.

I tried this and it appears the Snakemake API has changed and this doesn't seem to work anymore from a python session. There is some internal documentation on the API, but it is very minimal: https://snakemake-api.readthedocs.io/en/stable/api_reference/snakemake_api.html
It's stated that this is for Snakemake internal developers only. The furthest I got with the most recent API is this:
def read_snakemake_workflow(path) -> "snakemake.rules.Rule":
    """
    Parameters
    ----------
    path: str, pathlib.Path
        The path to the snakefile.
    rule: str
        Name of the rule in the snakefile that runs this script.

    Returns
    -------
    rule: snakemake.rules.Rules

    Examples
    --------
    To run an example both interactively and in a workflow, e.g.:

    >>> if "snakemake" not in globals():
    >>>     snakemake = read_snakemake_rule("snakefile", rule="my_rule")
    >>> modelname = snakemake.params.modelname
    >>> template = snakemake.input["template"]
    """
    import snakemake.settings as settings
    from snakemake.api import SnakemakeApi

    with SnakemakeApi(
        settings.OutputSettings(
            verbose=True,
            show_failed_logs=True,
        ),
    ) as snakemake_api:
        workflow = snakemake_api.workflow(
            storage_settings=settings.StorageSettings(),
            resource_settings=settings.ResourceSettings(),
            snakefile=Path(path),
        )

    # get_rule method disappeared
    return workflow

# This prints rules as type `odict_values`, but the original ordinary dictionary I could not find
print(workflow._workflow.rules)

The docs refer to a --debug option in the CLI, but how to use this I can't seem to find anywhere. It didn't seem to do anything when I tried it.
I think the Snakemake API is subject to changes, and therefore keeping a utility like this alive is quite maintenance costly.
I'm closing this for now, if a good alternative pops up, we can reopen this issue again.

I think the Snakemake API is subject to changes, and therefore keeping a utility like this alive is quite maintenance costly.

Keep in mind what the alternative is here: multiple people have already come up with tricks to parse snakefiles to extract values out of them.
I agree that this is tedious to maintain: IIRC, I pieced the above implementation together from a snakemake issue.
The trade-off is here: either we do it with an hour of maintenance here and there, or there will be multiple people spending over a day on this!

I think an hour of maintenance here and there is an underestimation of the amount of work required.
FYI: I spent 4 hours on this without coming to a satisfactory replacement of the proposed function for Snakemake 8.
I just found that the announcement for the changes was in this issue here:
https://github.com/snakemake/snakemake/issues/2409
Breaking changes upon a snakemake release can stall the development team with a day easily fixing a broken pipeline, as documentation of its internals is (currently?) insufficient. Furthermore, it adds an extra layer of complexity for developers of iMOD Python to understand. Finally, Snakemake would be quite a heavy (optional) extra dependency, though the developers work on making it less heavy with a conda injection system.
I'd rather create a separate package for this feature, to separate concerns. This could also be used by other colleagues at Deltares who do not require the other features of iMOD Python. I found a similar initiative to this, but unfortunately it seems stale now for 1 year, thus not compatible with Snakemake 8.
https://github.com/teristam/snakehelper

Separate package is indeed best, because hydromt users would also benefit.
And we can indeed isolate it so it doesn't block CI.
Ideally though, this would just work through snakemake right? I searched through the snakemake issues back then, but I don't remember my conclusion.

Another thought: maybe add the snippet above to the FAQ with a note that it works on snakemake 8, but we don't guarantee it working always.

Alright, got it working again:
from pathlib import Path

def read_snakemake_rule(path, name: str) -> "snakemake.rules.Rule":
    """
    Parameters
    ----------
    path: str, pathlib.Path
        The path to the snakefile.
    name: str
        Name of the rule in the snakefile that runs this script.

    Returns
    -------
    rule: snakemake.rules.Rules

    Examples
    --------
    To run an example both interactively and in a workflow, e.g.:

    >>> if "snakemake" not in globals():
    >>>     snakemake = read_snakemake_rule("snakefile", rule="my_rule")
    >>> modelname = snakemake.params.modelname
    >>> template = snakemake.input["template"]
    """
    from snakemake.settings.types import ResourceSettings
    from snakemake.api import SnakemakeApi

    with SnakemakeApi() as snakemake_api:
        workflow = snakemake_api.workflow(
            resource_settings=ResourceSettings(),
            snakefile=Path(path),
        )
        rules = {rule.name: rule for rule in workflow._workflow.rules}
    return rules[name]


A simple typo fix.
Why?
Better developer experience? :)

Thank you


Input field not working in cell templates

Hello,
if I put cell template with input field it doesn't work. Why is that?
Example: http://codepen.io/anon/pen/gabaXm

in SelectionController, ev.preventDefault() prevents all key presses.

I saw it and I fixed it in my local script.

This is here for specific reason. Perhaps adding a case that avoids input cases?


[FEAT] All the cards have abrupt animation upon hovering

Is your feature request related to a problem? Please describe.
All the cards have abrupt animation upon hovering
Describe the solution you'd like
This can be solved by creating a transform time
Screenshots

Important Announcement: Closure of GitHub Repository Issues, Website Redesign, and Discord Channel Invitation
We apologize for the inconvenience caused, but we must inform you that all current issues in our repository will be closed. This decision aims to improve our development process and enhance organization.
Excitingly, we are announcing a complete redesign of our website, focused on improving user experience and incorporating new features aligned with our goals.
Please note that approximately half of the existing issues are redundant and will be closed accordingly.
We also invite you to join our Discord channel for project collaboration and communication. Visit GSSoC server and assign yourself Web Resource Hub project role using the self-roles feature.
Join here
Your feedback and support are invaluable to us. We look forward to hearing your thoughts on the upcoming changes.
Thank you for your understanding and continued support.
Sincerely,
Project Admin


Bugfix: update the value mapping with the correct result of graphop.

(1) Fixes SR-8298.
(2) Updates the test to run the frontend with strict deabstraction turned on and off.
Some tests did not have any CHECKs. I have left it as is for now.

I believe no_copy.swift is intended to not have CHECKs for each test -- instead its main focus is to make sure the test cases do not generate any "copy to host" / "copy to accel" warnings.


gtest.cmake adds host-directories to include path in crosscompilation build

In gtest.cmake there are two calls to find_path with NO_CMAKE_FIND_ROOT_PATH set. This causes e.g. the hosts /usr/include to be added to the INCLUDE_DIRECTORIES which will cause cross-compilation builds.

Please provide more information what exactly your are trying to do and how it fails. Please also consider providing a pull request how you think it should be changed.

I hope the comment and PR clarifies the issue.


warmupTest inside StresstestTest randomly fails

The warmupTest inside the StresstestTest fails sometimes. The test sometimes only executes 5 query instead of the expected 6.

The test was poorly designed so we disabled it.


Add an option to show diagnostic source

This adds an option to show diagnostic source at the end of the message e.g.
'count' is already declared in the upper scope on line 6 column 10. (eslint)

This is especially useful for metalinters such as golangci-lint which runs multiple linters so the source actually shows the underlining linter that triggers an issue e.g.
exported function `MyFunction` should have comment or be unexported (golint)

This is also the same behavior that neomake implements by default.

Looks good to me! Thanks!

@seblj oops, forgot to add a comma in the README.md example code

No problem, I just fixed it :)


Increase timeout to pass test.

mint test(wc-test) can pass on slow machine now.

Codecov Report

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

@@             Coverage Diff              @@
##             master    #1024      +/-   ##
============================================
- Coverage     68.38%   68.32%   -0.07%
+ Complexity     4727     4724       -3
============================================
  Files           378      378
  Lines         29907    29907
  Branches       3804     3804
============================================
- Hits          20451    20433      -18
- Misses         8428     8437       +9
- Partials       1028     1037       +9

Impacted Files
Coverage Δ
Complexity Δ

...java/com.github.ambry.store/CompactionManager.java
86.75% <0%> (-2.65%)
19% <0%> (ø)

...github.ambry.rest/AsyncRequestResponseHandler.java
88.59% <0%> (-2.29%)
23% <0%> (ø)

.../com.github.ambry.router/GetBlobInfoOperation.java
82.28% <0%> (-1.15%)
38% <0%> (-1%)

...b.ambry.network/BlockingChannelConnectionPool.java
62.6% <0%> (-0.87%)
7% <0%> (-1%)

...in/java/com.github.ambry.store/BlobStoreStats.java
70.84% <0%> (-0.65%)
102% <0%> (ø)

...ain/java/com.github.ambry.router/PutOperation.java
89.8% <0%> (-0.39%)
81% <0%> (-1%)

.../java/com.github.ambry.router/DeleteOperation.java
93.61% <0%> (+1.41%)
48% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


Adding missing real dependencies for automation package

This PR fixes two missing real dependencies that are needed for the vscode-automation package.
When publishing this package and consuming it through a feed, it will fail to compile due to those missing dependencies that the playwright driver inside this package needs.

Can you clarify where we are publishing this package? Is there an issue?

@bpasero Internally we use this package for our automated tests that run against a VS Code instance.
To do that we manually build and publish this package to a private feed and consume it through there. This allow us to simply publish a new version of this package whenever there is a UI change on a new VS Code release.
When we updated our feed with the latest version of this package we encounter this dependency issue. Seems that the package is truly depending on tree-kill and playwright packages but not declaring them as such.
Looks like it's currently consuming those packages from the root repo node modules.

@osortega

Looks like it's currently consuming those packages from the root repo node modules.

Yes, to avoid duplication.
I do not think we support to take out individual pieces from our repo as standalone thing, so I am probably going ahead and close this PR, sorry.


I said to_lower not to_upper

Why are you allowed to code after midnight? https://github.com/DenizenScript/Denizen2Core/blob/master/src/main/java/com/denizenscript/denizen2core/tags/objects/TextTag.java#L80

Let's not spam minor issues please.


F7 band locked to .600

F7 band locked to .600

Try it now, found a bug I made when changing to a simpler frequency band table yesterday.

I confirm. Cannot put  eg. 470 MHz into VFO

I can

This isn't egzumers firmware repo though

OK. He is behind you a few commits

This isn't egzumers firmware repo though

I think the reason he keeps raising issues here is Egzumer has stated on telegraph he just merges  yours and Fagci's FW together and nothing more is thats the case anything unrelated to the spectrum is related to your FW.. And I can confirm the issue.. BUT egzumer  will not be merging the commits for  at least a few hours  yet.   having said that would still be best to start the ssue with his repo first.

On the latest 1of commit bug is fixed - I confirm


File processing

Construct resource functions, filled in only to the point of processing files for filling in data. UI wiring.
pre final import PR

This change is

Review status: 0 of 5 files reviewed at latest revision, 4 unresolved discussions.

src/controller/import_resource_popup_controller.js, line 34 at r3 (raw file):

 * @author celinechoo (Celine Choo), sagev (Sage Vouse)
 */
class ImportResourcePopupController extends PopupController {

OpenProjectPopupController/View

src/controller/read_write_controller.js, line 284 at r3 (raw file):

      } else if (resource.resourceType == PREFIXES.WORKSPACE_CONFIG) {
          this.constructWorkspaceConfig(resource[platform].filepath);
      }

Log or throw error when unrecognized resource type is found.

src/view/app_view.js, line 89 at r3 (raw file):

        ['Save All', () => { this.appController.saveProject(); }],
        ['Import', [
          ['Project', () => {this.importProject()}],

Make this a top-level "Open project" menu item.
Import should be reserved for adding an existing project to the currently open project.

src/view/app_view.js, line 226 at r3 (raw file):

   */
  importLibrary() {
    this.appController.readWriteController.importResource(PREFIXES.PROJECT);

import != open project.  Leave this unimplemented for now.
Better yet, add a console warning and a GitHub issue.
Same below.

Comments from Reviewable

Minor notes on Open vs Import. Otherwise

Review status: 0 of 5 files reviewed at latest revision, 4 unresolved discussions.

Comments from Reviewable

Are the things I commented on in readwriteController just setup for another PR?  If so, a few nits then LGTM.  If not, color me confused and then please explain.

Reviewed 3 of 5 files at r2, 2 of 2 files at r3.
Review status: all files reviewed at latest revision, 9 unresolved discussions.

src/controller/import_resource_popup_controller.js, line 53 at r3 (raw file):

    /**
     * Type of resource to import.
     * @type {Array<Object>}

Should be string.

src/controller/read_write_controller.js, line 266 at r1 (raw file):

        this.constructToolbox(resource.name, resource[platform].filepath);
      } else if (resource.resourceType == PREFIXES.WORKSPACE_CONTENTS) {
          this.constructWorkspaceContents(resource.name, resource[platform].filepath);

nit: indentation.  This line and the one two lines down.

src/controller/read_write_controller.js, line 294 at r3 (raw file):

   */
  constructLibrary(path) {
    const dataString = fs.readFileSync(path, 'utf8');

This doesn't do anything.

src/controller/read_write_controller.js, line 298 at r3 (raw file):

  /**
   * Construct a block based off of its metadata, and add it to the project.

  A block seems like it needs to be constructed from the file data, not the metadata.

src/controller/read_write_controller.js, line 311 at r3 (raw file):

  constructToolbox(path) {
    const dataString = fs.readFileSync(path, 'utf8');
    let refinedString = this.processToolboxDataString(dataString);

This one and the next few also don't do anything?

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 9 unresolved discussions.

src/controller/read_write_controller.js, line 266 at r1 (raw file):
Previously, rachel-fenichel (Rachel Fenichel) wrote…
nit: indentation.  This line and the one two lines down.

Done.

src/controller/read_write_controller.js, line 284 at r3 (raw file):
Previously, AnmAtAnm (Andrew n marshall) wrote…
Log or throw error when unrecognized resource type is found.

Done.

src/controller/read_write_controller.js, line 294 at r3 (raw file):
Previously, rachel-fenichel (Rachel Fenichel) wrote…
This doesn't do anything.

Yup-- not as of this PR

src/controller/read_write_controller.js, line 298 at r3 (raw file):
Previously, rachel-fenichel (Rachel Fenichel) wrote…
  A block seems like it needs to be constructed from the file data, not the metadata.

Fixed (here and elsewhere)

src/controller/read_write_controller.js, line 311 at r3 (raw file):
Previously, rachel-fenichel (Rachel Fenichel) wrote…
This one and the next few also don't do anything?

Yup-- not as of this PR

src/view/app_view.js, line 89 at r3 (raw file):
Previously, AnmAtAnm (Andrew n marshall) wrote…
Make this a top-level "Open project" menu item.
Import should be reserved for adding an existing project to the currently open project.

Done.

src/view/app_view.js, line 226 at r3 (raw file):
Previously, AnmAtAnm (Andrew n marshall) wrote…
import != open project.  Leave this unimplemented for now.
Better yet, add a console warning and a GitHub issue.
Same below.

Done.

src/controller/import_resource_popup_controller.js, line 34 at r3 (raw file):
Previously, AnmAtAnm (Andrew n marshall) wrote…
OpenProjectPopupController/View

Done.

src/controller/import_resource_popup_controller.js, line 53 at r3 (raw file):
Previously, rachel-fenichel (Rachel Fenichel) wrote…
Should be string.

Done.

Comments from Reviewable

Reviewed 7 of 7 files at r4.
Review status: all files reviewed at latest revision, 6 unresolved discussions.

src/controller/read_write_controller.js, line 266 at r1 (raw file):
Previously, svouse wrote…
Done.

Indentation in these else statements is still off.  Should only be two spaces.

src/view/app_view.js, line 226 at r3 (raw file):
Previously, svouse wrote…
Done.

TODO(#228): ...
And console warning for the unimplemented function, please.

Comments from Reviewable

Review status: 4 of 5 files reviewed at latest revision, 6 unresolved discussions.

src/controller/read_write_controller.js, line 266 at r1 (raw file):
Previously, AnmAtAnm (Andrew n marshall) wrote…
Indentation in these else statements is still off.  Should only be two spaces.

whoops fixed

src/view/app_view.js, line 226 at r3 (raw file):
Previously, AnmAtAnm (Andrew n marshall) wrote…
TODO(#228): ...
And console warning for the unimplemented function, please.

Done.

Comments from Reviewable

Review status: 2 of 5 files reviewed at latest revision, 6 unresolved discussions.

Comments from Reviewable


Fix vsock configuration.

The xrdp connection is currently failing with new installs (with xrdp 0.9.12 - the current version available). This is due to a change in how xrdp expects vsock to be parameterized in xrdp.ini.
See issue raised against xrdp.
The parameter use_vsock=yes  in /etc/xrdp/xrdp.ini is no longer preferred (and no longer seems to work), and the use of vsock should be specified in the port parameter as:
port=vsock://-1:3389

LGTM. @cwilhit - PTAL

LGTM


[Bug] Save data when switching tabs

Version
latest
Reproduction link
https://demos.creative-tim.com/material-dashboard-react/?_ga=2.162934463.226425432.1631562653-1386804182.1631104963#/documentation/tabs
Operating System
Mac OS
Device
Macbook
Browser & Version
Chrome
Steps to reproduce

Create tabs as shown in https://demos.creative-tim.com/material-dashboard-react/?_ga=2.162934463.226425432.1631562653-1386804182.1631104963#/documentation/tabs
Input data into fields
Switch to another tab
Come back to previous tab and you will notice all data entered previously is lost.

What is expected?
I want to retain the data when switching tabs
What is actually happening?
Data is lost

Solution
I want to implement the solution proposed here https://stackoverflow.com/questions/50871406/react-recreates-my-component-on-each-material-ui-tab-switch
Additional comments

Why do you even provide an option to raise issue if you don't bother responding?

Hi there
We've just updated the product now it has a new look with a flexible strucutre.
All the best
Sajad


Show User's Rating next to ranking

In the cases where I voted for a game, could we have my vote next to the ranking results?
It would be interesting to see how many stars other people have awarded in relation to my choice.
Could be integrated in this view should it be added: https://github.com/ludumdare/ludumdare/issues/991

The node request on a game already is quite complex, if you ask me. But I'd like the same type of statistics. My suggestion would be to have a page at:
https://ldjam.com/events/ludum-dare/38/myvotes
Where you see all the games you voted on, what votes you've given them and after voting closes, how your votes compares to what people thought in general.
(I'm not sure about if that would be best URI, or if it should be under the user instead)

I didn't even know about that. There are so many cool pages and websites, there should be a list of all the cool stuff.
I'm all for having a separate page with more detailed information, like I mentioned above and the one in https://github.com/ludumdare/ludumdare/issues/991. That could be linked from the rating block on the game pages.

In the past there was a bunch of outrage over people getting 1 or 5 star ratings, so it was decided then that "too much information was a thing". I love the idea of having some way to revisit the list of games you voted for, and compare your vote vs the result.


fix: scheduler piece cost time

Signed-off-by: Gaius<EMAIL_ADDRESS>
Description

Fix scheduler piece cost time.

Related Issue

Motivation and Context

Screenshots (if appropriate)
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[x] Code compiles correctly.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #1118 (b343d1a) into main (d7ab722) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1118      +/-   ##
==========================================
- Coverage   45.93%   45.91%   -0.02%
==========================================
  Files         130      130
  Lines        9983     9985       +2
==========================================
- Hits         4586     4585       -1
- Misses       4877     4880       +3
  Partials      520      520

Flag
Coverage Δ

Object-compatibility-e2etests
?

e2etests
?

unittests
45.91% <100.00%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/util/timeutils/time_utils.go
66.66% <100.00%> (+5.12%)
:arrow_up:

scheduler/service/service.go
87.19% <100.00%> (ø)

cdn/supervisor/progress/progress.go
91.04% <0.00%> (-5.98%)
:arrow_down:

client/daemon/peer/piece_broker.go
88.88% <0.00%> (+3.70%)
:arrow_up:


Mandatory vs Non-Mandatory Event inputs

The only two options that need to be input into the events page when adding a new event should be the title and description. All other options (location, time, date, image, and link) do not need to be mandatory. This allows for virtual meetings to include just a link and for the description to allow for all information to be input to reduce repeat information. Some may not have a link or an image associated with the event.

events implemented correctly


Login popups comes when the browser is refreshed

Describe the bug

The login popup comes when the user refreshes the page.

To Reproduce
Steps to reproduce the behaviour:

Go to a page
Refresh the page
Login popup comes and goes

Expected behaviour

The popup shouldn't come when the browser refreshed

Desktop (please complete the following information):

OS: BigSur
Browser : Chrome

I think this happens because we have the conditional rendering on the login pop up to check for the user object in the context, which takes additional time to get loaded. Within that short amount of time the UI is getting rendered multiple times, so for that split second between the initial render of the UI and detection of the logged-in user object the login modal pops up.

I think we can fix this buy introducing a "isLoading" global state. And whenever it's in the loading state hold off on conditional rendering


Miscellaneous ActiveStorage preparation

Append catch-all 404 route in after_initialize block instead of
having it at the end of routes.rb (so it doesn't override routes
added by engines such as ActiveStorage)
Proxy ActiveStorage downloads through the application (so we can
cache them more easily later)
Change routes prefix to /attachments (because it looks nicer)

Review app deployed to https://teaching-vacancies-review-pr-3547.london.cloudapps.digital


Update manual entries on Load Balancing strategies

Original issue: https://charm.cs.illinois.edu/redmine/issues/1656

The section describing the built-in strategies should be more descriptive of the trade-offs in strategies and should try to guide users to strategies we think are useful. When should you use a Comm-aware LB or a distributed LB or a hybrid LB? What are the differences between the Comm-aware LBs? When should you use RandCentLB or RotateLB?

Original date: 2018-03-28 01:19:54

This would be good to have for 6.9.0

Original date: 2018-05-02 05:10:05

Bump

Original date: 2018-11-08 20:07:34

~~https://charm.cs.illinois.edu/gerrit/#/c/charm/+/4794/~~ https://github.com/UIUC-PPL/charm/commit/9d4699f5d295e027c023292592502c8ffd612d5f


CALCITE-1124: backport timestampadd / timestampdiff functionality

-1. We need to stop backporting changes onto the Calcite fork and move back to master.

Sounds reasonable. What features do we have in Drill Calcite fork that are
missing in Calcite?
On Wed, Mar 9, 2016 at 6:38 PM Jacques Nadeau<EMAIL_ADDRESS>wrote:

-1. We need to stop backporting changes onto the Calcite fork and move
back to master.
—
Reply to this email directly or view it on GitHub
https://github.com/mapr/incubator-calcite/pull/5#issuecomment-194386355.


[$1000] Web - Sometimes, after creating a new group with email and phone number as participants, only participants with email are displayed in the group chat report

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Log in to an account
Click on FAB button and select New group
In the contact list, select both email and phone number as participants
Create group and check the participants information in the group chat report

Expected Result:
All participants should be displayed in the group chat report
Actual Result:
Sometimes, the participants with phone numbers are not displayed until reloading the group chat report
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.34-1
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/d4c7fc63-f48d-4fd0-b7c2-d4784505be0c
https://github.com/Expensify/App/assets/93399543/223df2f6-9633-45aa-a4b7-de61bf08cd36
Expensify/Expensify Issue URL:
Issue reported by: @tranvantoan-qn
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688291017734929
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0169e2e0f2d4b094b8
Upwork Job ID:<PHONE_NUMBER>648274432
Last Price Increase: 2023-07-17

Can reproduce

i looked into issue defined here  and i found that "/api?command=OpenReport" api is used while creating a new group and also to get existing group details and according to that response group details are rendering. when we're calling openreport api while creating a new group, response from that api is wrong ,it is only returning group member details that are logged in with email only(for ref. look into onyxData array with "key: personalDetailsList").without changing backend code one possible solution is to call openreport after new group created one more time with same groupId(reportId)

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01576033752dd3eb7a
i already sent proposal before!!

Proposal
Problem
Web - Sometimes, after creating a new group with email and phone number as participants, only participants with email are displayed in the group chat report
Root cause
After creating a group, only particpants with email is received from backend and set as the state.
.
when the following is visibility change, it set the state including the phone number participants, Most probably it is backend issue.
Visibility.onVisibilityChange(() => {
// If the report is not fully visible (AKA on small screen devices and LHR is open) or the report is optimistic (AKA not yet created)
// we don't need to call openReport
if (!getIsReportFullyVisible(this.props.isFocused) || this.props.report.isOptimisticReport) {
return;
}
Report.openReport(this.props.report.reportID);
});
Changes
This could be handled on Client-side if we force fetch report on component mount
 componentDidMount() {
        this.unsubscribeVisibilityListener = Visibility.onVisibilityChange(() => {
            // If the report is not fully visible (AKA on small screen devices and LHR is open) or the report is optimistic (AKA not yet created)
            // we don't need to call openReport
            if (!getIsReportFullyVisible(this.props.isFocused) || this.props.report.isOptimisticReport) {
                return;
            }
            Report.openReport(this.props.report.reportID);
        });

        // this.fetchReportIfNeeded();
        Report.openReport(this.props.report.reportID);
        ComposerActions.setShouldShowComposeInput(true);
    }

Alternative
Investigate the root cause in backend.

@snwork89 has rightly pointed out the issue here.

when we're calling openreport api while creating a new group, response from that api is wrong ,it is only returning group member details that are logged in with email only

Looks like the issue exists on backend. I can create a group without any issue offline.
@anmurali Can we add an internal engineer. Looks like the issue is in api response.

@MonilBhavsar - any updates?

Sorry, no. Getting to this today

Working on the PR

This was deployed last week!

It was internal, so we need to pay the reporter and add regression tests @tranvantoan-qn
Tests can be -

Click on + button > 'Group chat'
Add two users with email and a user with phone number login
Ensure that group is created and ensure the welcome message has all three participants

@tranvantoan-qn - Offer is here

Thank you, I've accepted the offer

Issue for QA created here https://github.com/Expensify/Expensify/issues/307952

Opened issue to add tests. We can close this now


Failing to ReRoutes leads to 404 when omitting "/api/" in get url

As part of a test in order to later use Ocelot, I built a mock project using " https://www.c-sharpcorner.com/article/building-api-gateway-using-ocelot-in-asp-net-core/ ", updated it to work with NetCore 2.1/Ocelot 8 and started playing with it.
The project is a Core console project as gateway using 2 core web api (except for names, they're identical thanks to copy paste).
I have a problem with ReRouting one of the API I'm using. I can access the "customers" api using localhost:5000/customers but I have to use localhost:5000/api/products if I want to access the "products" one. Note that I can access BOTH of them (but with an unexpected route for one).
Am I missing something ?
This is the Gateway.zip project (ocelot.json included), let me know if you need anything else.
Thanks !
Expected Behavior
Free access to "products" without having to put "/api/" in the url.
Actual Behavior
404 when trying to access localhost:5000/products.
Specifications

Version: 8 (NuGet VS2017)

Ok, I just found something using another project. There's a difference between the ocelot.json I want to use and the one used in the bin folder (looks generated).
Any help on setting this up so the file used is the one I edit please ?
THanks !

Further investigation lead to a fix. Simply renamed my ocelot.json to oselot.global.json.
From the sample I've seen on github, it might be a bug. Any feedback appreciated !
Thanks !

Hi @Ehrys , I don't think, it is a bug, rather it's a feature, Please check #296  for more details, and Here is the documentation.
Thanks.

@Ehrys thanks for your interest in Ocelot.
@anktsrkr thanks for helping me!! I really appreciate the support.
You should have a ocelot.json for each environment you want to work in e.g. ocelot.development.json, ocelot.staging.json, ocelot.production.json or something.
However you should just be able to add ocelot.json and everything will work ok if you do
.AddJsonFile("ocelot.json")

Let me know if you are still having problems?

@anktsrkr as per @TomPallister I was expecting having only the ocelot.json file to work (as it's supposed to work when using .AddJsonFile but seems not to be working with AddOcelot).
@TomPallister I fixed it by renaming my ocelot.json to ocelot.global.json, anyway I was expecting the AddOcelot to work with a simple ocelot.json file but I missread the doc's regex (note that "ocelot..json" matches the doc's regex).
Thank you both for your help !

@Ehrys thanks for your interest in Ocelot.
@anktsrkr thanks for helping me!! I really appreciate the support.
You should have a ocelot.json for each environment you want to work in e.g. ocelot.development.json, ocelot.staging.json, ocelot.production.json or something.
However you should just be able to add ocelot.json and everything will work ok if you do
.AddJsonFile("ocelot.json")

Let me know if you are still having problems?

Where the heck do you put this .addJsonFile("ocelot.json") ? Everyone says "oh yeah use this", but they never say where!

However you should just be able to add ocelot.json and everything will work ok if you do
.AddJsonFile("ocelot.json")

Let me know if you are still having problems?

Where the heck do you put this .addJsonFile("ocelot.json") ? Everyone says "oh yeah use this", but they never say where!

you need to put inside the program.cs or startup.cs for example in program.cs
public static IHostBuilder CreateHostBuilder(string[] args) =>
Host.CreateDefaultBuilder(args)
.ConfigureAppConfiguration((hostingContext, config) =>
{
config
.SetBasePath(hostingContext.HostingEnvironment.ContentRootPath)
.AddJsonFile("appsettings.json", optional: true, reloadOnChange: true)
.AddJsonFile($"appsettings.{hostingContext.HostingEnvironment.EnvironmentName}.json",
optional: true, reloadOnChange: true)
.AddJsonFile($"appsettings.local.json", optional: true, reloadOnChange: true)
.AddJsonFile("ocelot.json")
.AddEnvironmentVariables();
})
.ConfigureWebHostDefaults(webBuilder =>
{
webBuilder.UseStartup();
            });


setup shutdown hooks on ServiceLifecycle::startAndWait

motivation: fix startAndWait to work as advertised
changes:

extract shutdown setup to a func and call it from both start and startAndWait
add test
fix api doc

closes #68


Typo Report in Automating image optimization page

Page Affected: https://developers.google.com/web/fundamentals/performance/optimizing-content-efficiency/automating-image-optimization/
What needs to be done?
There's a typo in the section explaining what Vector graphics are. The sentence in question is listed below:
Vector graphics use points, lines and polygons to represent images and formats using simple geometric shapes (e.g logos) offering a high-resolution and zoom like SVG handle this use case better.
The sentence should probably be split into the following two sentences:
Vector graphics use points, lines and polygons to represent images and formats using simple geometric shapes (e.g logos) offering a high-resolution and zoom. Formats like SVG handle this use case better.

I would like to work on this issue.


Decks don't actually get generated into Anki

Even tho Obsidian says the Deck has been successfully exported, the Deck never appears in Anki. Is there some way to get the deck as a file, that I can manually import into Anki?
I am using Windows 11 if that may have contributed to the error.

Will look into this!

This happened to me as well: it happens when Anki is set to a language different from English. To solve I had to manually dive into the plugin code and change the model name and the q/a fields to reflect the Anki names.
Here's how it looks in my language now (Italian):

So maybe to solve the best way is to leave the user set its own fields name?


Can someone translate this to JavaScript?

There's a GameMaker Language extension called 2D Island Generator that does this also, but I need it for JavaScript.  Have had no luck trying to translate it.

I modified a preexisting JS autotile function I found elsewhere to do what I needed.


Add catchall for router

Feature Request
Crates
tonic
Motivation
While trying to implement a gRPC proxy server (and blend it with locally-implemented services), I found that I was forced to use the NamedService trait and tie that "generic" service to a single service name due to how the router tests leading strings.  Rather than using an empty string as the name for the service, I thought it might be useful to have a means to intercept all requests at the bottom of the routing stack in case you want to forward anything you can't handle locally to another server.
Proposal
The simplest means I found was to add add_catchall_service().  This likely requires you to implement a full tower service that's able to inspect the to perform the appropriate action (and return tonic-compatible replies).
Alternatives
A more elegant solution would be to support services that can accept or decline the request (i.e., provide their own route predicate).

Sorry for the delay on this, I think this is a great idea but I want to refactor some of the transport so will consider this in the new version.

Any news on this?


How do I get the value of the selected item?

How do I get the value of the selected item?

Hello @java773863202
You can get both index and value from 'onSelect' of RadioGroup
<RadioGroup
    //...other props
    onSelect={this.onSelect}
>
   //...radio buttons
</RadioGroup>

onSelect(index, value){
    this.setState({
        selectedIndex: index
        selectedValue: value
    })
}

If this issue still a problem, please re-open issue again


feat: 🔥 Add random super hero functions

Add randSuperHero. Add randSuperHeroName
Also, added last of missing char to full-name regex
✅ Closes: #134
PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/ngneat/falso/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: 134
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

I wasn't sure if the category for randSuperhero should be entity? If so, which category should randSuperheroName be under?

randSuperhero - entity, randSuperheroName - comics?

CI is failing

@NetanelBasal - I'm fixing it now and adding a few tests to ensure it doesnt happen again. Sorry


[xmlsec] no uwp

x509.c.obj : error LNK2019: unresolved external symbol BN_print_fp referenced in function xmlSecOpenSSLX509CertDebugDump

libxmlsec1-openssl.dll : fatal error LNK1120: 1 unresolved externals

BN_print_fp is only defined by openssl when not used on uwp

This one is also on CI baseline.

Associated issue: https://github.com/microsoft/vcpkg/issues/35005


Update to heading component

What?
Updating headings to use component gem
Why?
As part of on-going work to update Frontend apps inline with component gem architecture, improves a11y, consistency & allows changes to be cascaded across all apps. (ticket)
Visuals
No visual changes, Design maintained.
Example links impacted:
https://www.gov.uk/email-signup/confirm?topic=%2Fmoney%2Fexpenses-employee-benefits
https://www.gov.uk/foreign-travel-advice/canada/email-signup
https://www.gov.uk/email/subscriptions/verify
https://www.gov.uk/email/manage
https://www.gov.uk/email/manage/authenticate
https://www.gov.uk/email/manage/unsubscribe-all
https://www.gov.uk/email/manage/address
Anything else?
README broken link updated + link to docs explaining how to receive test emails.

⚠  This repo is Continuously Deployed: make sure you follow the guidance ⚠

Hey @injms I don't have auth to merge this one, could you do the honours if happy?

Hey @injms I don't have auth to merge this one, could you do the honours if happy?


pip install seems to take a long time

Did anyone see this issue? For me, it has been going on for a long time.

Can you provide more info?

I will pull the container and specify a specific version. JacobySent from the void. On May 9, 2023, at 07:57, webyacusa @.***> wrote:
It is happening to me as well. This is what I did, running on a Mac running Ventura with the M1 chipset:
checked out the latest code from the repository
Downloaded the  Universal Sentence Encoder, extracted the files to the location specified in the instructions
Edited the line in api.py with the local reference (the self.use = hub.load('./Universal Sentence Encoder/') line)
Executed from the CLI docker-compose -f docker-compose.yaml up
As a result, the building seems to be running, but it is taking a very long time. I am already at 5000 seconds in the build. A couple of observations: first, it does not seems like it is hanged, from time to time it does seem to be downloading and running things. Second, I got this message:
=> => # INFO: pip is looking at multiple versions of pydub to determine which version is compatible wit
=> => # h other requirements. This could take a while.
=> => #   Downloading pydub-0.24.0-py2.py3-none-any.whl (30 kB)
=> => # INFO: This is taking longer than usual. You might need to provide the dependency resolver with
=> => # stricter constraints to reduce runtime. See https://pip.pypa.io/warnings/backtracking for guida
=> => # nce. If you want to abort this run, press Ctrl + C.
Am I missing something?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

Hello,
Were you able to find what needed to be changed on certain package versions in the requirements.txt?
I'm seeing the exact same problem when trying to use docker-compose -f docker-compose.yaml. I also see the same issue when trying to run docker build . as well.
I'm also using an M1 mac and when I try to run code a pip3 install -r requirements.txt in a virtual environment, it complains about being unable to find tensorflow versions:
Collecting PyMuPDF
  Downloading PyMuPDF-1.22.3-cp310-cp310-macosx_11_0_arm64.whl (12.7 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 12.7/12.7 MB 7.3 MB/s eta 0:00:00
Collecting numpy
  Using cached numpy-1.24.3-cp310-cp310-macosx_11_0_arm64.whl (13.9 MB)
Collecting scikit-learn
  Using cached scikit_learn-1.2.2-cp310-cp310-macosx_12_0_arm64.whl (8.5 MB)
ERROR: Could not find a version that satisfies the requirement tensorflow>=2.0.0 (from versions: 2.13.0rc0)
ERROR: No matching distribution found for tensorflow>=2.0.0

So this is likely a mac / tensorflow incompatility issue.
But when I try to run on a Windows11 machine, to run pip3 install -r requirements.txt on python 3.8.2 it will simply complain about being unable to find a version of langchain-serve that work with the rest of the dependencies.
ERROR: Cannot install -r requirements.txt (line 8) because these package versions have conflicting dependencies.

The conflict is caused by:
    langchain-serve 0.0.32 depends on pandasai
    langchain-serve 0.0.31 depends on pandasai
    langchain-serve 0.0.30 depends on pandasai
    langchain-serve 0.0.29 depends on pandasai
    langchain-serve 0.0.28 depends on pandasai
    langchain-serve 0.0.27 depends on pandasai
    langchain-serve 0.0.26 depends on pandasai
    langchain-serve 0.0.25 depends on pandasai
    langchain-serve 0.0.24 depends on pandasai
    langchain-serve 0.0.23 depends on pandasai

I'm hoping to be able to run this locally or in a docker container.
Let me know if I'm missing anything.
Thanks!

It’s an OS issue. It builds in Windows. I haven’t found the specific issue yet. Sent from the void. On May 21, 2023, at 21:17, Kenneth Wong @.***> wrote:
Hello,
Were you able to find what needed to be changed on certain package versions in the requirements.txt?
I'm seeing the exact same problem when trying to use docker-compose -f docker-compose.yaml. I also see the same issue when trying to run docker build . as well.
I'm also using an M1 mac and when I try to run code a pip3 install -r requirements.txt in a virtual environment, it complains about being unable to find tensorflow versions:
Collecting PyMuPDF
Downloading PyMuPDF-1.22.3-cp310-cp310-macosx_11_0_arm64.whl (12.7 MB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 12.7/12.7 MB 7.3 MB/s eta 0:00:00
Collecting numpy
Using cached numpy-1.24.3-cp310-cp310-macosx_11_0_arm64.whl (13.9 MB)
Collecting scikit-learn
Using cached scikit_learn-1.2.2-cp310-cp310-macosx_12_0_arm64.whl (8.5 MB)
ERROR: Could not find a version that satisfies the requirement tensorflow>=2.0.0 (from versions: 2.13.0rc0)
ERROR: No matching distribution found for tensorflow>=2.0.0
So this is likely a mac / tensorflow incompatility issue.
But when I try to run on a Windows11 machine, to run pip3 install -r requirements.txt on python 3.8.2 it will simply complain about being unable to find a version of langchain-serve that work with the rest of the dependencies.
ERROR: Cannot install -r requirements.txt (line 8) because these package versions have conflicting dependencies.
The conflict is caused by:
langchain-serve 0.0.32 depends on pandasai
langchain-serve 0.0.31 depends on pandasai
langchain-serve 0.0.30 depends on pandasai
langchain-serve 0.0.29 depends on pandasai
langchain-serve 0.0.28 depends on pandasai
langchain-serve 0.0.27 depends on pandasai
langchain-serve 0.0.26 depends on pandasai
langchain-serve 0.0.25 depends on pandasai
langchain-serve 0.0.24 depends on pandasai
langchain-serve 0.0.23 depends on pandasai
I'm hoping to be able to run this locally or in a docker container.
Let me know if I'm missing anything.
Thanks!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

It works perfectly fine on linux based systems. IT gives error in mac based systems cuz of tensorflow package version onflict with grpcio. I will open an issue regarding the same.

Hello, Were you able to find what needed to be changed on certain package versions in the requirements.txt? I'm seeing the exact same problem when trying to use docker-compose -f docker-compose.yaml. I also see the same issue when trying to run docker build . as well.
I'm also using an M1 mac and when I try to run code a pip3 install -r requirements.txt in a virtual environment, it complains about being unable to find tensorflow versions:
Collecting PyMuPDF
  Downloading PyMuPDF-1.22.3-cp310-cp310-macosx_11_0_arm64.whl (12.7 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 12.7/12.7 MB 7.3 MB/s eta 0:00:00
Collecting numpy
  Using cached numpy-1.24.3-cp310-cp310-macosx_11_0_arm64.whl (13.9 MB)
Collecting scikit-learn
  Using cached scikit_learn-1.2.2-cp310-cp310-macosx_12_0_arm64.whl (8.5 MB)
ERROR: Could not find a version that satisfies the requirement tensorflow>=2.0.0 (from versions: 2.13.0rc0)
ERROR: No matching distribution found for tensorflow>=2.0.0

So this is likely a mac / tensorflow incompatility issue.
But when I try to run on a Windows11 machine, to run pip3 install -r requirements.txt on python 3.8.2 it will simply complain about being unable to find a version of langchain-serve that work with the rest of the dependencies.
ERROR: Cannot install -r requirements.txt (line 8) because these package versions have conflicting dependencies.

The conflict is caused by:
    langchain-serve 0.0.32 depends on pandasai
    langchain-serve 0.0.31 depends on pandasai
    langchain-serve 0.0.30 depends on pandasai
    langchain-serve 0.0.29 depends on pandasai
    langchain-serve 0.0.28 depends on pandasai
    langchain-serve 0.0.27 depends on pandasai
    langchain-serve 0.0.26 depends on pandasai
    langchain-serve 0.0.25 depends on pandasai
    langchain-serve 0.0.24 depends on pandasai
    langchain-serve 0.0.23 depends on pandasai

I'm hoping to be able to run this locally or in a docker container. Let me know if I'm missing anything.
Thanks!

I think you can try this https://github.com/bhaskatripathi/pdfGPT/issues/10#issuecomment-1518678791


[lain-cli] the display of lain dashboard is unordered, a little mess

RT

Fixed at https://github.com/laincloud/lain-cli/commit/3bc612d9f3c6eb3a0901fc3a57421eb033f3e367.
Use -s parameter to show available apps in order, default according to appname.


Deploy VANotify to VA environment

Value Statement:
As a VANotify system owner
I want to deploy my system in VA network
So that my consumers can use the system
Acceptance Criteria
GIVEN A new commit is pushed
AND CI run passes (build & test)
THEN VANotify is automatically deployed to development environment in VAEC
AND the application works (can send email)
GIVEN A successful deployment to development environment
THEN VANotify is automatically deployed to staging environment in VAEC
AND the application works (can send email)
GIVEN VANotify system owner
WHEN they manually trigger a production deployment using prod version tag
THEN VANotify is automatically deployed to production environment in VAEC
AND the application works (can send email)
Checklist:
Assumptions:
The environments are dev, staging or prod.
Additional Info/Resources:
Out of Scope:
Open Questions:

The migrations is now running successfully.

The VAEC notification API on dev is installed and managed to create organization, user, service, template, api key and send the email.


Support for Apple Silicon with prebuilt binaries

Describe the bug
Support for Apple Silicon
To Reproduce
Steps to reproduce the behavior:

Download the executable on release page
Run swag on Apple Silicon
You will get bad CPU type in executable: ./swag
If you use the last Apple silicon support swag version, it opens some debug Gin server

./swag init -g cmd/api/main.go -o docs/swagger
[GIN-debug] [WARNING] Creating an Engine instance with the Logger and Recovery middleware already attached.

[GIN-debug] [WARNING] Running in "debug" mode. Switch to "release" mode in production.
 - using env:   export GIN_MODE=release
 - using code:  gin.SetMode(gin.ReleaseMode)

[GIN-debug] GET    /api/v1/map               --> github.com/swaggo/swag/example/object-map-example/controller.(*Controller).GetMap-fm (3 handlers)
[GIN-debug] GET    /swagger/*any             --> github.com/swaggo/gin-swagger.CustomWrapHandler.func1 (3 handlers)
[GIN-debug] [WARNING] You trusted all proxies, this is NOT safe. We recommend you to set a value.
Please check https://pkg.go.dev/github.com/gin-gonic/gin#readme-don-t-trust-all-proxies for details.
[GIN-debug] Listening and serving HTTP on :8080

Expected behavior
All MacBooks are comes with Apple Silicon these days. Better if we can download the ARM version on release page
Your swag version
e.g. 1.8.9
Your go version
e.g. 1.19
Desktop (please complete the following information):

OS: MacOS/ ARM

Hi,
We will include the darwin/arm64 version in the next release. Thanks for bringing this to our attention.


Change default action when clicking the name of a task

Hi,
By default, clicking the name of a task marks it as complete, but I personally find it a bit misleading and confusing. What about changing the default behavior from marking as complete to just editing it ?
That way, if we want to mark a task as complete we can still click the checkbox but we can also edit the task by clicking on its name.
Thanks again !

I think clicking the description should result in the same action as clicking the checkbox, but I could add support for long click/press to open the edit dialog. I've been meaning to do this to improve the mobile experience.
Thoughts? If lot's of users find this confusing, I could probably make this configurable.

The solution in v2.1 isn't working very well, particularly on mobile devices. I'm going to remove the click/tap on the label. You can still long press the label to edit, but now only the checkbox will toggle the todo.


fix: Configure local llm to work with Nginx.

I see now in the channel that this is deployed by using a helm chart. So if the purpose of this PR is to use this helm chart, you should include the instructions in our readme and then remove the nim resources.

I assume by these changes that there is a new deployment for a standalone nim service. Is it only for the llm or will also be done for the embed instance? If so where are the deployment resources to manage it?
So if you plan to manage the deployment independently, link the new resources and delete the ones in this folder instead of commenting them out.

This is only for the llm, the embed instance works good.

I see now in the channel that this is deployed by using a helm chart. So if the purpose of this PR is to use this helm chart, you should include the instructions in our readme and then remove the nim resources.

Yes, going to do that.

I think that both the llm and embed instance should be deployed in the same way and in the same namespace. I don't mind having them deployed in the nim namespace but I think it's better to have them deployed together.

We have two robust working installation processes of installing nim openai meta/llama3.1-8b LLM:

Using kserve - which requires kserve operators on the cluster - this is how we installed the model up until re-provisioning of the cluster, but now it's gone, and need to be reinstalled.
using the helm chart - which is straight forward, and requires only helm cli  and nothing more.

We can try installing the helm chart on the same namespace, but sometimes it's cumbersome and produce some metadata validation errors, and it's always best practice to install helm chart  on a brand new namespace.
If you want i can check it out anyway.


chore: TypeScript

Tried @rusackas 's approach of bringing a react-bootstrap component in Storybook just for fun.
One thought is around the fact that labels are used in controls, and that it calls for eventually having to refactor components into its own package that other packages can depend on.
TODO: emotion

I believe I've fixed last test... if this gets merged in, remind me to open another PR and use the new syledMount helper all over the place.

Codecov Report

Merging #10494 into master will decrease coverage by 7.25%.
The diff coverage is 81.48%.

@@            Coverage Diff             @@
##           master   #10494      +/-   ##
==========================================
- Coverage   70.84%   63.59%   -7.26%
==========================================
  Files         605      765     +160
  Lines       32428    36178    +3750
  Branches     3423     3441      +18
==========================================
+ Hits        22975    23008      +33
- Misses       9341    13054    +3713
- Partials      112      116       +4

Flag
Coverage Δ

#cypress
54.14% <94.44%> (-0.42%)
:arrow_down:

#javascript
60.09% <81.48%> (+0.14%)
:arrow_up:

#python
59.07% <ø> (-11.36%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...et-frontend/src/SqlLab/components/LimitControl.jsx
87.17% <ø> (ø)

...frontend/src/SqlLab/components/QueryStateLabel.jsx
100.00% <ø> (ø)

...rset-frontend/src/SqlLab/components/QueryTable.jsx
61.53% <ø> (ø)

...erset-frontend/src/SqlLab/components/SouthPane.jsx
81.81% <ø> (ø)

...erset-frontend/src/SqlLab/components/SqlEditor.jsx
52.12% <ø> (-1.22%)
:arrow_down:

superset-frontend/src/components/CachedLabel.jsx
42.10% <ø> (ø)

...et-frontend/src/components/Label/label.stories.jsx
0.00% <0.00%> (ø)

superset-frontend/src/components/TableSelector.jsx
84.96% <ø> (ø)

superset-frontend/src/components/Timer.jsx
100.00% <ø> (ø)

...erset-frontend/src/datasource/DatasourceEditor.jsx
71.00% <0.00%> (ø)

... and 206 more

Continue to review full report at Codecov.

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

Ugh... this python pre-commit hook is failing on a random text file. I'm fixing it in this in https://github.com/apache/incubator-superset/pull/10515

🏷 storybook

I was just working with labels and wanted to make some tweaks. Having this would make it easier to add other styles to the label.


Readme Improvement for pencil sketch implementation (Issue-364)

Added Readme Improvement for pencil sketch implementation (Issue-364).
File- Readme(ForPencilSketchIplementation).md

Thanks!