text:
Slash in title results in server error on save

Overview:
Creating a new note with a title that contains a slash (/) character, gives an error "Unable to communicate with the server".
I have not debugged the problem or checked the code, but I assume the title is used as part of a filename, and the slash messes up the path?
Suggestions:

Escape the slash character so saving is possible
Provide a more meaningful error message, such as: Invalid characters in the title. Allowed characters are [a-z][A-Z] etc...
If the title is indeed used as a file path, maybe rethink this idea by having a unique randomly generated ID associated with the note and use that instead.

Hi @dominikheinz. Your assumption is correct, this is a limitation of using the titles as filenames. This is a duplicate of #52 and there's a bit more detail on there.
I really should be validating this in the front end as well (with a friendly message as you suggest). I'll add this in the next release.

A friendly message has been added in v2.2.0.

authors: Dullage dominikheinz

repo: Dullage/flatnotes


text:
[FEATURE] Add close and re-search button

Description
Enable users to close their current anonymous chat and automatically start searching for a new random buddy
TODO

[ ] Add a button labelled with something similar to Close 'n find new buddy
[ ] Confirm close action from the user after clicking on the button
[ ] After a positive confirmation from the user, close the current chat and automatically start searching for a new random buddy

Screenshots
No response

It's great having you contribute to this project by creating an issue Make sure you are assigned to this before you work on it and you read the Contributing.md file, Thank you! Welcome to the community :nerd_face:

@Dun-sin I'm taking this one

source: gharchive/issue

authors: Dun-sin mathiasayivor

repo: Dun-sin/Whisper

license: MIT

license_type: permissive

license_source: github-api


text:
Warning: TypeError: Cannot read property 'bubblingEventTypes' of null

Describe the bug
Warning: TypeError: Cannot read property 'bubblingEventTypes' of null
Dependency versions

React Native version: 0.75.4
React version: 18.3.1
React Native Fast Image version: ^8.6.3

Note: if these are not the latest versions of each I recommend updating as extra effort will not be taken to be backwards compatible, and updating might resolving your issue.

I also encountered this issue, but reopening my emulators fixed it.Try closing and opening your simulator and android emulator.

I encountered same issue with same version,
Solution -

delete node module and reinstall it
cd android && ./gradlew clean
npm run android  <-- this is the main step

@mlcpro You've closed the issue. How have you solved the problem ?

@FelipeRaulino yes, thank you

authors: FelipeRaulino jedaydiya mlcpro satyajitX001

repo: DylanVann/react-native-fast-image

license: mit

license_source: bigquery


text:
dsDemos: error in Matlab 2014a

SIMULATING MODEL:
Solving system using /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/solve/solve_ode_demo_cluster_3.m
Changing directory to /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/solve
Saving model parameters: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/solve/params.mat

Running simulation 1/3 (solver='rk4', dt=0.01, tspan=[0 100]) ...
Elapsed time: 0.100735 seconds.
	Data saved to /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/data/study_sim1_data.mat
	Executing post-processing function: dsPlot
		Elapsed time: 0.097045 sec
Error: Invalid or deleted object.
	 in dsAnalyze (line 237)
	 in dsSimulate/tryFn (line 1057)
	 in dsSimulate (line 751)
	 in dsDemos (line 295)
created temporary lock file for this process: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/.lock_20171002112757544_1
loading study file: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/studyinfo.mat
deleting temporary lock file for this process: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/.lock_20171002112757544_1
setting status='failed' where sim_id=1
created temporary lock file for this process: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/.lock_20171002112757627_1
updating simulation metadata in study file: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/studyinfo.mat
deleting temporary lock file for this process: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/.lock_20171002112757627_1
Changing directory to /home/jsardid/Documents/DynaSimData/demos
Error using handle.handle/set
Invalid or deleted object.

Error in dsAnalyze (line 237)
        set(thisResult, 'PaperPositionMode','auto');

Error in dsSimulate/tryFn (line 1057)
            tmpresult=dsAnalyze(tmpdata,siminfo.result_functions{f},'result_file',siminfo.result_files{f},'save_data_flag',1,'save_results_flag',1,siminfo.result_options{f}{:});

Error in dsSimulate (line 751)
    tryFn(nargout)

Error in dsDemos (line 295)
dsSimulate(eqns, 'save_data_flag',1, 'study_dir','demo_cluster_3',...

Sorry for the noise, this came from me deleting a figure (trying to clean up my desktop from multiple panels, quite a lot of them opened in fullscreen) that was called afterwards.
Btw, couldn't we avoid matlab figures to be opened fullscreen? they are very annoying, at least to me...

did the sizing get fixed?

I don't think so

authors: erik-roberts kupiqu

repo: DynaSim/DynaSim


text:
Add type casting

It is a preliminary refactoring.
Introduce separate mechanisms to:

dumpt field to DynamoDB type
undump it to Dynamoid field type
type cast user value to Dynamoid field type

As a side effect custom types dumping was repaired. See https://github.com/Dynamoid/Dynamoid/pull/275/commits/64c4e2db6e3fbcbd9bf3052d873748d96f348948
The issue was - dumping convertion is used to dump and type cast values.
The goal of the next PRs is to improve type casting and add type casting and dumping to finders

@pboling @richardhsu
Could you please take a look at the PR?
There are two issues to discuss:

what type casting should be added (here)
do we still need following finders find_by_composite_key, find_all_by_composite_key and find_all_by_secondary_index? Looks like they duplicate functionality of find/find_by_id and querying with where chain

@pboling

Also should we deprecate first, for the next release, and then remove for the following release?

Yep. Makes sense.
TBH I wonder if find_by_composite_key, find_all_by_composite_key and find_all_by_secondary_index are duplicated as far as I didn't use them at all and there may be some unique features I didn't noticed

👍 Awesome.

source: gharchive/pull-request

authors: andrykonchin pboling

repo: Dynamoid/Dynamoid


text:
Work around occasionally missing webhook reconciliations

There is an issue on https://github.com/kubernetes-sigs/controller-runtime/issues/942, where pre-existing elements on the source.Channel's channel used by the webhook's bootstrapper may not be processed.
As a workaround, I'm here delaying a bit before we generate the initial request, and artificially generating periodic reconciliations rather than leaving it to the ReconcileAfter (since, if the first request is not processed, then there would be no follow-ups.)

@lrgar  hi, is it lose events not only in webhook, and recocilers too??

authors: lrgar scuzhanglei

repo: Dynatrace/dynatrace-oneagent-operator


text:
add dashboard to view all answers & questions

Option 1:

Add button to show modal
Add modal to show all questions
Create js code to show all questions
Collapse answers on click

Option 2:

Add bootstrap
Add navbar
Add navigation between 2 pages
Add new page to show all questions
Create js code to show all questions
Add responsive design to navbar
Add responsive design to new page

Prefer option 2

I would also use Bootstrap but I would say that we use a custom compiled bootstrap repo. We just need to fork and adjust the bootstrap repo and keep our current Design.


text:
Add QREFHT and U10 comparison with ERA5 variables

Closes #686

[x] Update processing script to include d2m: 2m dewpoint temperature, sp: surface pressure, 10m wind speed
[x] Add new variable and derivation formula, QREFHT(d2m, sp)
[x] Include new datasets in LCRC data server

Hi Chris @golaz I have added new ERA5 data, pre-processing script and e3sm_diags codes for surface specific humidity and 10 meter wind comparison. Here is an example output, https://acme-viewer.llnl.gov/zhang40/tests/ERA5_ext_try1/viewer/lat_lon/index.html. Please review and let me know if you'd like any changes. Thank you.


text:
tags based on new release format

Based on our discussion with Eugene we are going with the date format version so I would suggest you release tag in same format and delete the v1.1 tag https://github.com/E4S-Project/e4s/releases/tag/v1.1 because this doesn't make sense.

Yes, I think date format is the way to go. The v1.1 tag has been removed. I can create the new tag as soon as we are ready to make this prototype release "official." When should we do this?

We are trying to better specify what is meant by an E4S release. I think the consensus among facility team representatives and Ryan Adamson was that we need to simultaneously specify:

a Spack commit
a set of versioned packages built with the aformentioned Spack commit

This repository had previously had a release tag of v1.1 before we moved in this direction. We have been talking about doing date based releases. I think that is what @shahzebsiddiqui was referencing. At least that's the way I took it, and why I responded the way I did.
What do you think @maherou ?

I agree that semantic versioning such as X.Y does not make sense for E4S at this time, since there is no concept of minor version or backward compatibility that a minor release would imply.
At the same time, I am not a fan of date-based release numbers for software products.  It is too much information for most people who need to ID a particular version.  Can we simply use natural numbers?  We could call the coming release Version 2.
The release number would be tied to both the Spack commit and the set of versioned packages, which satisfies the requirements without forcing everyone who communicates about E4S to remember a long date tag.

At the same time, I am not a fan of date-based release numbers for software products. It is too much information for most people who need to ID a particular version. Can we simply use natural numbers? We could call the coming release Version 2.

100% defer to the judgement of you and your colleagues. Happy to go whichever way you decide. One of the main reasons I put this up was to preview what the new release scheme might look like and elicit these types of discussions and suggestions.

Unless there is a strong practical reason to use a date ID, I like to keep things as simple as possible.  Let's plan on a natural number versioning scheme, unless someone else speaks up.
We are using the same versioning strategy for the E4S Community Policies.  Version 1 of the policies is almost through final approval.  BTW, the policy versioning does not have to sync with the E4S releases.

semver doesn't make sense for e4s that's what we all thought initially. The date format would be useful if E4S plans to have regular release if we have commitment let's say once per quarter. If you want numbers like e4s-1 e4s-2 it doesn't mean much overtime when we have many versions. The date version scheme is used by intel, schedmd

The schedmd approach could work.  Do you know if there is a de facto standard along these lines?  I would like to tap into an approach that is commonly used.  In the end, I want to do the right thing.  Thanks for the pointer.

Here is a defacto standard based on calendar versioning.  @shahzebsiddiqui pointed me to Ubuntu's release naming strategy.  It's based on the YY.MM in which it was released.  It seems that larger projects with broad scope fit better into a calendar version release scheme since clear features are harder to plan out when rolling major versions.  Personally, I think that our strategy should be a clue for users as to how we release tagged versions -- if we are planning on releasing quarterly, we should stick with calver.

I am fine with the YY.MM approach, given the precedents.

authors: eugeneswalker maherou rmadamson shahzebsiddiqui


text:
Database locations and download

Currently, the different databases used by VIRify are auto-downloaded and need to be already available. However, from a user perspective, it would be good to mention the database URLs in the README to also allow a manual download.

Indeed, I'll add the URLs to the README file.

authors: hoelzer mberacochea

repo: EBI-Metagenomics/emg-viral-pipeline


text:
Merge DEV

This PR has:

CWL code, moved to an dev version in order to use conditionals.
Changes for https://github.com/EBI-Metagenomics/emg-viral-pipeline/issues/13
Changes for https://github.com/EBI-Metagenomics/emg-viral-pipeline/issues/12
Fix for lenght filtering script, memory problem reported by Alex A.
Mashmap optional step in CWL and Nextflow
Travis build for CWL and python scripts. Not sure how to write tests for Nextflow, at least a validation of the code (I'll dig into this).

I've tested this locally and it's working, I'll move the code to the cluster to run some tests.

Okay, that was annoying. In the mashmap module I changed
-s ${length}

to
-s ${params.mashmap_len}

because you can directly use parameters in modules. But it was not working (it was always using the default of 2000).
I figured out that the problem seems to be if you are using
shell:
"""
-s ${params.mashmap_len}
"""

or
script:
"""
-s ${params.mashmap_len}
"""

In a script: environment params.mashmap_len is updated correctly when I change --mashmap_len 4500 but not in the shell: environment!
I will also check for other modules where shell: instead of script: is used and update this.
I don't know if this is intended or a super annoying bug.

tldr:
either use
shell:
'''
'''

or
script:
"""
"""

it seems that
shell:
"""
"""

behaves strange


text:
Please add The Rat Genome Database to OLS

Hi, I am a contractor for Boehringer, we have colleagues who wish to use RDO via OLS, attached is a spreadsheet with details as per the request page.  Thanks
Kenny

New OLS ontology request RDO.xlsx

Your spreadsheet is empty - but I have never heard of RDO in the context of rats (only in the context of Radiology); as far as I know RGD (the Rat Genome Database) uses DO for diseases, CMO for measurements and vto for traits, MP for phenotypes - so not sure what Rat Genome Database exactly stands for?

Hi, apologies for the spreadsheet, it should now be attached correctly.
New OLS ontology request RDO.xlsx
The RGD does use DO for diseases, but extends them with concepts which are not in DO
Thanks

Oh my... This will be so confusing. I never ever heard of this effort of RGD: https://github.com/rat-genome-database/RDO-Disease-Ontology
It is using DOID identifiers. It is enormous with more than 8000K terms added that have not been reviewed by the DO team, or at least not published. It is republishing DO terms. I dont even get what the purpose/scope of RDO is. Initially I thought "animal diseases", but then, there are hundreds if not thousand of classes in that "extension" that are clearly not animal and are sourced from something like Mesh..
I would very strongly suggest that RDO does not belong in OLS in its current state - it needs to go through OBO first. Ideally, RGD needs to work with DO on some way to integrate the two ontologies.
Adding another disease ontology, one with DOID identifiers, is a big mistake IMO. RDO, should work on on a harmonised release product with DO, with a clear scope (rat disease or whatever) and go through OBO.

Just to be clear: I am not in any position to say yay or nay on anything related to any of the EBI resources - I am sure the SPOT team will add the ontology if they think it is the right thing to do regardless of my opinion. I was told by my friends and colleagues that my tone above was a bit too negative here, and I really did not mean to say anything negative (I am a bit passionate sometimes, huge sorry about that).  So, what I would like to offer here is the following concern:
RDO

is not yet an OBO-reviewed resource (most other RGD products actually are, so I don't know why this one is not)
uses identifiers that belong to a different ontology, which may be confusing to people (curator will find a DOID identifier searching OLS and using it thinking they are using DO, but are not).
does not have a clear enough scope at first glance - in particular, it seems (to my unprofessional eyes) to contain plenty of human diseases as well.

I also am v passionate about ontologies, they are wonderful, wonderful things.   I did not take anything negative from your response, which was very helpful, you have nothing to apologise for as far as I am concerned.

Thank you @kennybaird

@kennybaird
In general OLS is slightly more lenient in accepting ontologies into OLS than OBO. However, in its current state, we cannot accept this ontology for the following reasons.

You did not provide a purl from where we can download the .owl file for the ontology. OLS4 only support .owl files.
as @matentzn has pointed out, you are reusing purls to existing terms to identify terms in your ontology. This goes against the basic principles of the semantic web. As mentioned in an earlier comment you made, you are using terms from DOID from which you extend new terms that are not in DOID. This you can do, but then you have to import those terms from DOID and declare them in your own ontology.

I strongly you reach out to the OBO community to help you in creating a pipeline to achieve 1 and 2.

authors: henrietteharmse kennybaird matentzn


text:
API use cases

As a bioinformaticians trying to find data relevant to planned analysis, I want to retrieve pre-generated, CxG standard h5ad files based on any combination of - publication DOI, sample tissue, sample developmental stage, assay, cell type... using ontology closure to return more results. The results should take the form of a CSV table with metadata (Citation, tissue(s), stage(s) etc) plus link to h5ad file for download

the query should support combinations at the sample/annotation level not merely at the dataset level e.g.  a query for cell_type: fibroblast; tissue: Kidney should return datasets with cells annotated to types of fibroblast and parts of kidney, not just datasets that have these cell types and tissues, but not combined.

Draft tech spec:
KG - already has a queryable neo4j graph of cell sets linked to ontology terms and dataset nodes & a SOLR endpoint with all nodes indexed.
Query strategy: SOLR instance is sufficiently denormalised that above use case can be fulfilled with 1-3 queries.  Denormalizations needed: ontology closures; dataset metadata + file link?  Precise schema TBD.

Knowledge Graph use cases:  More discussion needed of what extended KG content will look like.   Will we include an extended Graph with GO.  Will we fold in GO annotations?  Analysis of cell set transcriptomes => predicted GO BP an CC?

Query for datasets containing types of T cell
MATCH (c)-[:SUBCLASSOF*0..]->(d) WHERE d.label = 'T cell'
MATCH p=(ds)-[:has_source]-(n:Cell_cluster)-[:composed_primarily_of]->(c:Class:Cell)
RETURN distinct  n.label as author_annotion, c.label as CL_annotation, ds.download_link[0], ds.title[0], ds.publication[0]

Find all datasets that use tissue from the lung
MATCH (n:Cell_cluster)-[r:tissue]->(t)-[:part_of*0..]->({label: 'lung'})
MATCH (ds)-[:has_source]-(n:Cell_cluster)-[:composed_primarily_of]->(c:Class:Cell)
RETURN distinct  n.label as author_annotion, t.label as tissue, ds.download_link[0], ds.title[0], ds.publication[0]

We could do the same for stage, disease and organism.
To do combinatorial queries. we can make the ontology restriction clauses optional and combine them.


text:
feat: json functions

JSON functions to instatiate PrmsParameters object from JSON and to save or load parameter dictionary as JSON.
Needs tests

Codecov Report

Merging #128 (ef35579) into main (c0d6882) will decrease coverage by 0.40%.
The diff coverage is 32.00%.

@@            Coverage Diff             @@
##             main     #128      +/-   ##
==========================================
- Coverage   81.90%   81.51%   -0.40%
==========================================
  Files          40       40
  Lines        4940     4965      +25
==========================================
+ Hits         4046     4047       +1
- Misses        894      918      +24

Impacted Files
Coverage Δ

pynhm/utils/parameters.py
79.59% <32.00%> (-9.76%)
:arrow_down:

pynhm/hydrology/PRMSGroundwater.py
94.28% <0.00%> (-2.86%)
:arrow_down:

pynhm/hydrology/PRMSChannel.py
96.13% <0.00%> (-1.66%)
:arrow_down:

pynhm/hydrology/PRMSCanopy.py
79.31% <0.00%> (-1.15%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I merged a lingering PR into to upstream:main and then merged upstream:main into your fork:branch. the RTD build succeeds now.
I wont merge without a test. I think the test is simple read a myparams.param file (a single domain should suffice) to a parameter object, write it to json, read the json back in and check that the read json gives an identical parameter object. You'll have to check the parts or define equality for the class.

kewl @jmccreight - that test idea was what I was planning. I just wasn't certain there was a reliable location for a params file to work with. I see this domain object in the test_parameter_read.py stuff but couldn't sort out what that is. If you can point me to a params file in the testing suite I'll dial in the test.

now that I've thought about it, let me propose something. I'll push it to your fork:branch

I'm about to push my rough draft, cause I got to put it down for now. It's close but there are checks that are not passing and it's not obvious to me why. Here's where I get to

oy - that's going to be a little tricky since I relied on the constructor to make those dimensions. But, I'll take a look

oh - actually I see the problem. I will push up a fix later today

good, cause it's not obvious to me! LMK if there's anything i can help on.
The only additional thing I'd say is that it'd be good to test update_parameters_from_json as well. Am I clear on the intent:? given an existing parameter object, update_parameters_from_json would slurp up a json and update the values in the parameter object with those found in the json, which may only be a subset. Will you actually use that since you have to interact through files anyway?

BTW, we can go over conceptually what's going on here sometime. the domain is just a parameterized pytest fixture at the conftest level, it gives all the necessary inputs to run a given model domain for multiple model domains. Now it's ALL clear!

I had to fix the default discovery of param dimension names which wasnt getting exercised. I enhanced Joe's early wild guess code for param dimension names with the metadata module that since became available. This was a long time coming and it works with only the hitch that 4 previously unidentified parameters needed added to the static/metadata/parameters.yaml, i think 3 were for SNTMP and the other was a precip threshold, none appeared in the existing code base.

authors: codecov-commenter jmccreight mnfienen

repo: EC-USGS/pynhm


text:
Support standalone running at JPL with HTTPS

In order for this software to run at JPL, it needs to be able to start without interactivity. It also needs to support HTTPS as per the latest NASA security requirements. This commit supports these ends, and also makes some progress towards streamlining.
Specifically:

Removed Redis from the labcas-ml-serve image

Redis runs as its own service in a Docker composition

Added a Docker composition

Runs the ML Serve, Redis, and the TLS termination as separate services
Added an entrypoint to the labcas-ml-serve image so it can run in a composition without indicating failure

Removed pyenv

Separate Pythons should be in separate images
For now, we support just Python 3.9, but there is build-arg support for parameterized Pythons
For multiple environments besides Python, we'll need to add even more build-arg support, but that's TBD

Removed compiled files that got committed for bizarre reason

Added .gitignore to help prevent inadvertent commits of compiled objects

Added .dockerignore to speed up image builds
Removed archive code that kept showing up in project greps
Changed TODOs.py into a Markdown text file
Added environment variable support for locating Redis, controlling Docker composition
Reduced image size
Metadata and comments in lots more places

@asitang let's hold off merging this until the changes you're making on edrn-labcas to support LabCAS integration are pushed to this branch.

repo: EDRN/labcas-ml-serve


text:
Would it be possible to use kernl to speed up Stable Diffusion?

I don't know deeply what would be involved in this, but I was wondering if kernl would have application in speeding up StableDiffusion at all?
Crossposted:

https://github.com/AUTOMATIC1111/stable-diffusion-webui/issues/4096
https://github.com/huggingface/diffusers/issues/1094

We are currently investigating Whisper, StableDiffusion is the next :)

We won't support stable diffusion in the near future.

authors: 0xdevalias gaetansnl jonathlela

repo: ELS-RD/kernl


text:
Can this feature/limitation of web assembly be exploited in EOS?

https://www.fastly.com/blog/hijacking-control-flow-webassembly-program

Sorry I raised above query using my official github account and hence deleted it. I am interested in knowing from you if there exists vulnerabilities by hijacking control flow in EOS or if that is not possible?

I don't think this will happen on EOS, as by default it uses interpretation to execute contracts.

EOS.IO is not vulnerable to system call hijacking.  Each contract gets its own copies of the eosiolib calls it uses.  Contracts do not call into some common library on the chain.  Contracts can not modify contracts on other accounts without an explicit authorization.  Providing your own version of some other contract's functions does not interact with that other contract at all and does not somehow inherit or hijack that contract's permissions.

authors: PravinBezwada ghost jgiszczak taokayan

repo: EOSIO/eos


text:
Expand content on resolving conflicts

The current section on resolving conflicts only links to contributor covenant, which is specific to code of conduct violations. It would be helpful to add more information on other types of conflicts (e.g., design decisions) and how it relates to governance models and roles from the contributor guide.

Laxman will research this issue.
There may be some documentation from NASA TOPS MOOC (under open science 101), linked to original

repo: ESIPFed/esds-community-guide


text:
Update core team info

This PR updates the contact info and members of the Core Team. The correspondin PR for ESMValCore is https://github.com/ESMValGroup/ESMValCore/pull/942

[x] CODE_OF_CONDUCT.md
[x] NOTICE - might need more revisions
[x] introduction.rst

@ESMValGroup/esmvaltool-coreteam The file NOTICE contains a list of recipes and institutes haveing the copyright. Do we want to keep this list? If so, this list needs some updates to also include more recent recipes.

@ESMValGroup/esmvaltool-coreteam The file NOTICE contains a list of recipes and institutes haveing the copyright. Do we want to keep this list? If so, this list needs some updates to also include more recent recipes.

repo: ESMValGroup/ESMValTool


text:
Fast version deangelis for test

Description
Adding recipe/testing/recipe_deangelis15nat_fig1_fast.yml,  short test recipe to reproduce figure 1 from recipe_deangelis15nat.yml using only one model. It should run around 1 minute on one CPU (Levante). The original recipe failed several times for new releases due to its derrived variables. This short version did run for ESMValCore v2.5 and the current release candidate v2.6.0rc3.

Contributes to close #1636
Link to documentation: https://docs.esmvaltool.org/en/latest/recipes/recipe_deangelis15nat.html
This is the documentation page to the original recipe recipe_deangelis15nat.yml, do the test recipes need an own docu?

Before you get started

[x] ☝ Create an issue to discuss what you are going to do

Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

[x] 🛠 This pull request has a descriptive title
[x] 🛠 Code is written according to the code quality guidelines
[ ] 🛠 Documentation is available
[x] 🛠 Tests run successfully
[ ] 🛠 The list of authors is up to date
[ ] 🛠 Any changed dependencies have been added or removed correctly
[ ] 🛠 All checks below this pull request were successful

New or updated recipe/diagnostic

[x] 🧪 Recipe runs successfully
[ ] 🧪 Recipe is well documented
[x] 🧪 Figure(s) and data look as expected from literature
[ ] 🛠 Provenance information has been added

New or updated data reformatting script

[ ] 🛠 Documentation is available
[ ] 🛠 The dataset has been added to the CMOR check recipe
[ ] 🧪 Numbers and units of the data look physically meaningful

To help with the number of pull requests:

🙏 We kindly ask you to review two other open pull requests in this repository

@esmvalbot Please run recipe/testing/recipe_deangelis15nat_fig1_fast.yml

@esmvalbot Please run testing/recipe_deangelis15nat_fig1_fast.yml

Thanks! I noticed that there was no provenance for the figure, so I added it for the diagnostic used in this recipe (it looks like it's also missing for the other diagnostic scripts in the same folder though, maybe this would be something to look into).

This is the documentation page to the original recipe recipe_deangelis15nat.yml, do the test recipes need an own docu?

Yes, I think so. No need for scientific documentation, but it would be good to list the testing recipes and their options (or link to the original recipe page) and for each recipe why it is considered a good test. Could you add a documentation page for the testing recipes? A single documentation page for all recipes in the testing directory would probably be most convenient.

I added a docu page, where also the other test recipes could be added: doc/sphinx/source/recipes/recipes_testing.rst
and put it under "Others", is it ok like this? (Does the link to the docu for recipe_deangelis15nat.yml version work?)

because even such a small thing like shortening an existing diagnistic for testing purposes creates a lot of extra work

What extra work are you are talking about? 3 lines of code needed to be added to fix the provenance for this recipe and I did that for you, so where is the extra work for you?

This can easily result in 50-100 plots, if there are 6 additional provenance files that are really too many (and from the code I'd assume the provenance files in work and plot are identical?).

In the case of this diagnostic yes, but not all diagnostics create a plot and NetCDF file with the same data. Some only create plots or even other file formats. In any case, the feature that creates the 2 extra citation info files was added over 2 years ago in ESMValGroup/ESMValCore#402, so it is a bit late to complain about the number of output files now, but you're welcome to open an issue if you have ideas about how this could be improved. I do agree that it are many files and it would look a bit messy if you try to open this with a file browser. If you view the results by opening the index.html file in your browser the presentation is nicer.

Does the link to the docu for recipe_deangelis15nat.yml version work?

I fixed it for you, you needed to add backticks around the link name. You can preview the documentation by clicking on the docs/readthedocs.org:esmvaltool check below.

Thanks! This gives me a good start to implement some automated recipe testing.

because even such a small thing like shortening an existing diagnistic for testing purposes creates a lot of extra work

What extra work are you are talking about? 3 lines of code needed to be added to fix the provenance for this recipe and I did that for you, so where is the extra work for you?

This can easily result in 50-100 plots, if there are 6 additional provenance files that are really too many (and from the code I'd assume the provenance files in work and plot are identical?).

In the case of this diagnostic yes, but not all diagnostics create a plot and NetCDF file with the same data. Some only create plots or even other file formats. In any case, the feature that creates the 2 extra citation info files was added over 2 years ago in ESMValGroup/ESMValCore#402, so it is a bit late to complain about the number of output files now, but you're welcome to open an issue if you have ideas about how this could be improved. I do agree that it are many files and it would look a bit messy if you try to open this with a file browser. If you view the results by opening the index.html file in your browser the presentation is nicer.

Does the link to the docu for recipe_deangelis15nat.yml version work?

I fixed it for you, you needed to add backticks around the link name. You can preview the documentation by clicking on the docs/readthedocs.org:esmvaltool check below.

Thanks for fixing the link and merging!
With extra work I meat first understanding the issue, but also, that I now should ideally start new PRs for my more then 2 year old recipes to fix that  - leading back to the endless discussion about who has to adapt existing recipes to new technical requirements.

Adding provenance is not a new technical requirement, it has been around since January 2019, long before there was a stable release and the promise of backward compatibility. Anyway, let's not argue on GitHub, it would probably be better to discuss this in person at the next workshop.

Adding provenance is not a new technical requirement, it has been around since January 2019, long before there was a stable release and the promise of backward compatibility. Anyway, let's not argue on GitHub, it would probably be better to discuss this in person at the next workshop.

The provenace was there, only not in the correct folder (although it is not really clear, which is the correct folder follwing the doku text). I really see the point to require provanance in a standardtised way, but from the doku/ PR you linked I don't really understand, that the standard should be 6 provenance files in two different directories for every plot...

Yes, maybe we could look at the documentation together at the next workshop and make it more clear?

In principle yes, but I'm afraid any change would lead either to the request to change something in many diagnostics or there will continue to be a lot of different solutions depending on who wrote a diagnostic when.

authors: bouweandela katjaweigel


text:
EVerest crash issue observed when using v0.19.1 framework

Describe the bug
We are currently using the latest commit of Everest Core (6e669fd867afaaca3ab45deaefea1f2a13998998) in our repository, and we are experiencing a crash. After debugging the issue, we found that the crash occurs when using Everest-Framework v0.19.1 with the latest Everest Core commit (6e669fd867afaaca3ab45deaefea1f2a13998998). However, when we use Everest-Framework v0.18.1, everything works as expected, and no crash is observed.
I suspect that there might have been a change in the framework that is causing the crash immediately after Everest starts. One observation we made is that the crash happens only when the OCPP module is loaded. If the OCPP module is not loaded, the system runs fine without any issues.
We are using security level 3 for OCPP server connection. This whole OCPP server connection is working fine with October EVerest core release.
We are compiling and creating the Everest binaries on Ubuntu 22.04.03 with the stable release up to 2024.10.0. The crash issue appears to have been introduced with the following commit:
Commit ID: 50d8bacaab06dea0b5a58f5efd212ff1d25de239
Commit Message: "Bump Everest-Framework to 0.19.1 (#987)"

There are several changes between Everest-Framework v0.18.1 and v0.19.1 that we have been unable to debug to identify the root cause. Could someone from your team assist with investigating this crash issue?
Please let me know if you need any further information from my side.
EVerest Domain
OCPP, OCPP1.6, Other
Affected EVerest Module
Everest-framework v0.19.1
To Reproduce
Compile and run the Everest on Ubuntu 22.04.03 version
Connect to OCPP using Security level 3
Anything else?
No response

EVerest_Crash.txt

Manager-config.txt

@hikinggrass @Pietfried @AssemblyJohn @andistorm Please help me in the issue

This is resolve in everest framework repository

repo: EVerest/everest-core


text:
Add dev server script.

This readds the dev server capability that watches the filesystem for changes.
Use it with make dev-server.
This PR depends on #18, so it will appear to have extra changes until #18 is merged.

@DrewWeth get yo review on, I want to pull this in before reviewing #20 :dragon:

@ryan-endacott make dev-server errors with make: *** No rule to make target 'dev-server' did you forget something in the makefile?

I built from scratch and it worked well. Merging this guy in.

authors: Felttrip ryan-endacott

repo: EaseApp/web-backend


text:
Tries to load dev bundle

Hello,
I am trying to deploy my 3.4 app, and during the Preparing app stage, I have this error :
The command "ssh -A petitecolo '(export APP_ENV=prod; cd /home/www/beta/releases/20180628172057 && /home/www/bin/composer install --no-dev --prefer-dist --no-interaction --quiet)'" failed.

  Exit Code: 1(General error)

  Working directory: /media/jon/Projets/Projets/LeCoinEcolo.fr/symfony

  Output:
  ================

  Error Output:
  ================
  Script Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::clearCache handling the symfony-scripts event terminated with an exception

    [RuntimeException]

    An error occurred when executing the "'cache:clear --no-warmup'" command:

    Fatal error: Uncaught Symfony\Component\Debug\Exception\ClassNotFoundException: Attempted to load class "DoctrineFixturesBundle" from namespace "Doctrine\Bundle\FixturesBundle".

    Did you forget a "use" statement for another namespace? in /home/www/beta/releases/20180628172057/app/AppKernel.php:45

    Stack trace:

    #0 /home/www/beta/releases/20180628172057/vendor/symfony/symfony/src/Symfony/Component/HttpKernel/Kernel.php(492): AppKernel->registerBundles()

    #1 /home/www/beta/releases/20180628172057/vendor/symfony/symfony/src/Symfony/Component/HttpKernel/Kernel.php(132): Symfony\Component\HttpKernel\Kernel->initializeBundles()

    #2 /home/www/beta/releases/20180628172057/vendor/symfony/symfony/src/Symfony/Bundle/FrameworkBundle/Console/Application.php(64): Symfony\Component\HttpKernel\Kernel->boot()

    #3 /home/www/beta/releases/20180628172057/vendor/symfony/symfony/src/Symfony/Component/Console/Application.php(148): Symfony\Bundle\FrameworkBundle\Console\Application->doRun(Object(Symfony\Component\Conso
  le\Input\ArgvInput), Object(Symfo in /home/www/beta/releases/20180628172057/app/AppKernel.php on line 45

  install [--prefer-source] [--prefer-dist] [--dry-run] [--dev] [--no-dev] [--no-custom-installers] [--no-autoloader] [--no-scripts] [--no-progress] [--no-suggest] [-v|vv|vvv|--verbose] [-o|--optimize-autoload
  er] [-a|--classmap-authoritative] [--apcu-autoloader] [--ignore-platform-reqs] [--] [<packages>]...

It uses APP_ENV, but I think it is a SF4 env variable (SYMFONY_ENV for SF 3.4), maybe that is why it tries to load a dev bundle (DoctrineFixturesBundle).
I tried to add in beforeStartingDeploy() a $this->runRemote('export SYMFONY_ENV=prod'); but I do have the error anyway.
Any clue ?
Thanks all !

I have the same problem. As a temporary workaround I ended up overriding $this->_symfonyEnvironmentEnvVarName (line 385) in DefaultConfiguration.php, but as @wyllyjon mentioned SF3.4 always seems to be handled like a flex app.
Is there any configuration option we are not aware of?

@DavidFeller, had you overridden it directly in the vendor folder or is there a way to override it properly ?

I've overridden it only for testing purposes, but haven't had time to work on a proper solution, yet

Yes, this is a problem if you use the Symfony 3 directory structure with 3.4 as the bundle assumes Symfony 4 directory structure and env variable name, which can't be changed from outside. See #66, too.

The issue is caused by those lines:
// vendor/easycorp/easy-deploy-bundle/src/Configuration/DefaultConfiguration.php:377

        } elseif (3 === $symfonyMajorVersion && 4 < $symfonyMinorVersion) {
            $this->_symfonyEnvironmentEnvVarName = 'SYMFONY_ENV';
            $this->setDirs('bin', 'app/config', 'var/cache', 'var/logs', 'src', 'app/Resources/views', 'web');
            $this->controllersToRemove(['web/app_*.php']);
            $this->sharedFiles = ['app/config/parameters.yml'];
            $this->sharedDirs = ['var/logs'];
            $this->writableDirs = ['var/cache/', 'var/logs/'];
        } elseif (4 === $symfonyMajorVersion || (3 === $symfonyMajorVersion && 4 >= $symfonyMinorVersion)) {
            $this->_symfonyEnvironmentEnvVarName = 'APP_ENV';
            $this->setDirs('bin', 'config', 'var/cache', 'var/log', 'src', 'templates', 'public');
            $this->controllersToRemove([]);
            $this->sharedDirs = ['var/log'];
            $this->writableDirs = ['var/cache/', 'var/log/'];
        }

It assumes that Symfony 3.4 is using Symfony's 4 directory structure which is wrong!
This is what fixed it for me:
    } elseif (3 === $symfonyMajorVersion) {
          $this->_symfonyEnvironmentEnvVarName = 'SYMFONY_ENV';
          $this->setDirs('bin', 'app/config', 'var/cache', 'var/logs', 'src', 'app/Resources/views', 'web');
          $this->controllersToRemove(['web/app_*.php']);
          $this->sharedFiles = ['app/config/parameters.yml'];
          $this->sharedDirs = ['var/logs'];
          $this->writableDirs = ['var/cache/', 'var/logs/'];
      } elseif (4 === $symfonyMajorVersion) {
          $this->_symfonyEnvironmentEnvVarName = 'APP_ENV';
          $this->setDirs('bin', 'config', 'var/cache', 'var/log', 'src', 'templates', 'public');
          $this->controllersToRemove([]);
          $this->sharedDirs = ['var/log'];
          $this->writableDirs = ['var/cache/', 'var/log/'];
      }

@numediaweb Take a look at my PR. It enables you to set the directory layout from the outside without patching the DefaultConfiguration class. Because you could the 4 directory layout already with 3.4 and your change wouldn't work then.

Hello YetiCGN !
How do you set the directory layout from outside ? I have looked at your commit, but I don't understand how it can be made ?
tks !

Ok !
Thanks for the info !

Can the PR from @YetiCGN  be merged please?

Seems this project is unmaintained / abandoned. No commit or merge for almost 6 months. If you add this block to your composer.json it will pull my forked version:
"repositories": [
    {
        "type": "vcs",
        "url": "https://github.com/YetiCGN/easy-deploy-bundle"
    }
],

Use at your own risk. Should this project be picked up again I can't guarantee to pull all changes to my fork.

@YetiCGN it may look abandoned ... but it's not. I use this bundle every day to deploy multiple personal projects. It's working fine to me :)

Hey, Javier! Thanks for stopping by. I am using it (almost) every day as well to deploy a project, but I have to use my fork since it's a Symfony 3.4 project.
By abandoned I meant that development has stopped almost half a year ago with no comments on 9 open pull requests from contributors. It's great to see you're still commenting here, but frankly I've decided to switch to Deployer for new projects although easy-deploy-bundle looked promising.

Still having this issue when using Symfony v3.4.29
@javiereguiluz can you please merge the PR?

@numediaweb Switch to https://github.com/deployphp/deployer/, it's much more flexible and readily maintained.
I did and never regretted it.

@numediaweb: I agree with @YetiCGN. I did the same.

authors: DavidFeller YetiCGN javiereguiluz numediaweb wyllyjon

repo: EasyCorp/easy-deploy-bundle


text:
Add book Design Patterns via C#

What does this PR do?
Add Resource book Design Patterns via C#
For resources
Description
Design patterns with C# examples. Based on GoF Design Patterns.
Why is this valuable (or not)
Design Patterns of GoF has examples in Smalltalk, it hard to understand for modern developers, so, here is a great book with examples in C#.
How do we know it's really free?
It's free but you need to have registration
For book lists, is it a book?
yes
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

looks like the book requires email. please see /CONTRIBUTING.md

@eshellman yes, it is, but I see many books require the same. Should I add (account required) note ?

Thanks, and welcome to free-programming-books!

authors: Marusyk eshellman

repo: EbookFoundation/free-programming-books


text:
(No. 17 C) PAD IDs in report map and layer style toggling [ Medium ]

Feedback, pulled from #93 :

[ ] Include PAD IDs in the pop-up within the map and allow users to turn on/off barriers to treat vs. not to treat in case someone wants to only see the solution in a print out of the solution.

Can we break this up into:

[ ] Include PAD ID in the pop-up in the map, and
[ ] allow users to turn on/off barriers to treat vs. not to treat in case someone wants to only see the solution in a print out of the solution.

Seems like the former is a smaller task than the latter.

2nd: addressed in overhaul of report page?
Breaking up report by budget tics, easier and more performant, but lose single table and single map output.

These have not been addressed as asked, but there are workarounds for both:

PAD IDs  can be pulled from table and from search box when clicked.
Can't turn off treat/don't treat, but can sort table, and there is a filtered CSV option.

authors: jtutak rhodges

repo: Ecotrust/FishPass


text:
[ODS-6434]Rebuild Database Templates build -Skip already successful package creation from working branch

Below is a structured breakdown of each use case:
Use Case 1:
Scenario:
A new commit occurs in one of the three repositories (Ed-Fi-ODS, Ed-Fi-ODS-Implementation, or Ed-Fi-Extensions).
The workflow begins by updating the CodeGen package but fails before completing.
Expected Behavior:
In the first run, the workflow updates the CodeGen package version. However, the workflow fails after this update, preventing the completion of other package updates.
Example Workflow Run:
Failed after updating CodeGen package.
Use Case 2:
Scenario:
The workflow is re-triggered after the failure in Use Case 1.
It should skip the CodeGen package update since it was already updated and continue updating the Extensions packages (TPDM, Sample, Homograph).
Expected Behavior:
The workflow should resume from the Extensions package update, bypassing the CodeGen package. However, it fails again after updating the Extensions packages.
Example Workflow Run:
Failed after updating Extensions packages.
Use Case 3:
Scenario:
The workflow is re-triggered after the failure in Use Case 2.
It should skip both the CodeGen and Extensions package updates, resuming with updates to the EdFi.Ods.Minimal.Template, EdFi.Ods.Minimal.Template.PostgreSQL, EdFi.Ods.Populated.Template, and EdFi.Ods.Populated.Template.PostgreSQL packages.
Expected Behavior:
The workflow should skip the already updated CodeGen and Extensions packages, but it fails after attempting to update the Minimal and Populated templates.
Example Workflow Run:
Failed after updating Minimal and Populated templates.
Use Case 4:
Scenario:
The workflow is re-triggered after the failure in Use Case 3.
It should skip the CodeGen, Extensions, and Minimal/Populated template updates and continue with the updates for EdFi.Ods.Minimal.Template.TPDM, EdFi.Ods.Minimal.Template.TPDM.PostgreSQL, EdFi.Ods.Populated.Template.TPDM, and EdFi.Ods.Populated.Template.TPDM.PostgreSQL packages.
Expected Behavior:
The workflow successfully completes by updating the TPDM-related templates without any failures.
Example Workflow Run:
Successful completion after TPDM package updates.
Validation Steps:
Branch Consistency Across Repositories:
Ensure the working branch exists in all three repositories (Ed-Fi-ODS, Ed-Fi-ODS-Implementation, Ed-Fi-Extensions).
Check for Previous Workflow Failures:
Before starting the process, check if there was a failure in the last run. If so, determine whether any packages were updated successfully.
Handling New Commits:
If new commits are detected after a failed workflow, ensure the workflow resumes from where it left off by skipping already updated packages.

@vimayya  All review comments are fixed . Ready for Review again . Thanks


text:
Remove unnecessary scrolling

Description
if there is no requirement then we should remove unnecessary scrolling in website
Screenshots
https://user-images.githubusercontent.com/91718608/221353199-48cb80e8-af1e-42dd-af4c-cf2a2f074119.mp4
Additional information
the video is just for my profile, but i think in each page, if not needed then remove scrolling

if maintainers are ready then I would like to work on this

if maintainers are ready then I would like to work on this

@abhijain2003 I have assigned this to you

sure sarajaoudo

sorry I am failed to raise pr as finding difficulty in setting mongodb for this repo locally

sorry I am failed to raise pr as finding difficulty in setting mongodb for this repo locally

Hey @abhijain2003 you can try using Gitpod, it is very good alternative for situations like you are facing.

do you have any documentation for this attached to the repo?

sorry I am failed to raise pr as finding difficulty in setting mongodb for this repo locally

Could you elaborate on the problem you are facing?

Unable to replicate so I will be closing this Issue for now.

repo: EddieHubCommunity/LinkFree


text:
[REVIEW] Resume Review

Resume link:
Can you review my resume? thank you

Few grammatical changes and semantic changes which I think makes it sound more professional
I'm contributing to open-source on a daily basis.
During my internship, I worked on the dashboard ~side of the~ for Radiolize. I used Vue.js and Bootstrap in this project. ~On the other hand,~ I also ~was~ worked ~in~ upon the settings and setup of IoT devices.
Also you should explain what Radiolize is

Thank you for your contribution. I changed it :smiley:

Thank you for sharing, we will review on a live stream soon 👍 . Great to see you are already getting community feedback 💥

Try to use the space in Left
Can avoid using I in Who I am  so many times !!
You can try changing some font size so can be diffrentiated

As a UX designer, I think it looks better to add link color just to distinguish other text from the hyperlinks.

Reviewed on live stream https://www.youtube.com/watch?v=8SmgE7X9yHg
Looks good, some of our thoughts

Large ST looks great! Suggest maybe having a profile photo there
Maybe group the sills together (databases, frontend ...)
Really looks nice and clean

Thank you guys reviewed my old resume ❤
I am gonna update and share it again.

authors: eddiejaoude rohan-kulkarni-25 shubhendumadhukar tansionline theriyazo

repo: EddieHubCommunity/support


text:
[REVIEW] eebbann

GitHub Profile Link
https://github.com/eebbann

Consider adding an Avatar/Profile Picture for yourself
A website or portfolio would be nice too 😄
Made PR https://github.com/eebbann/eebbann/pull/2 to improve your profile by adding stats. More PRs will come soon 🚀

Great start to your profile, looking forward to seeing where you take it.

Thanks guys i really appreciate @krishguptadev and @weeyin83

Great profile, reviewed on live stream https://www.youtube.com/watch?v=JQ1wZRjint8 ... some suggestions...

I prefer a real picture of the person, I think it is more personal
the widgets take up too much space, I suggest removing them or have them in 1 row (side by side)

authors: eddiejaoude eebbann krishguptadev weeyin83


text:
Please invite me to the GitHub Community Organisation

Invitation will be sent for the GitHub Organisation soon. We look forward to having you part of our community :nerd:
Don't forget after accepting to make it public so it appears on your github profile for everyone else to see, you can do this by finding your name in the github organisation list and change the dropdown to public https://github.com/orgs/EddieJaoudeCommunity/people
Tips for practicing:

Customise your GitHub profile, here is a live stream on it https://www.youtube.com/watch?v=cT6GXCuS0Zo
Practice repo, instructions how to add your name to the README in the README https://github.com/EddieJaoudeCommunity/hacktoberfest-practice
Remember contributing to open source is not just about code, its about collaboration, communication and adding value

I hope that helps

Invitation sent for the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see, you can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/EddieJaoudeCommunity/people
Tips for practising:
Customise your GitHub profile, here is a live stream on it https://www.youtube.com/watch?v=cT6GXCuS0Zo
Practice repo, instructions on how to add your name to the README in the README https://github.com/EddieJaoudeCommunity/hacktoberfest-practice
Remember contributing to open source is not just about code, its about collaboration, communication and adding value
I hope that helps

authors: divinenaman eddiejaoude

repo: EddieJaoudeCommunity/support


text:
Sign In to iCloud

When this plugin is added to a Cordova project and run on iOS8, the iPhone/iPad will throw up a prompt on app startup asking for the Apple ID password.
"
Sign In to iCloud
Enter the Apple ID password for
(your account)
[password]
Cancel | OK
"
Removing the plugin stops this occurring, adding the plugin back in makes it happen.
"cordova plugin add cordova-plugin-x-socialsharing"

And what is the problem?

authors: jmail krisrandall


text:
🛑 Portal Vicerrectoria Administracion is down

In 3d712e2, Portal Vicerrectoria Administracion (https://www.vadm.una.ac.cr/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Portal Vicerrectoria Administracion is back up in 6948d16 after 11 minutes.

repo: EdgardoRamosRoque/status


text:
it can not run on the new dotnet sdk-2.1.200-win-x64

i had update the sdk to the new version "dotnet-sdk-2.1.200-win-x64",but i can't run anymore
i find the issue

It's running normally.
Try to change global.json with the correct version of SDK.
Post some error message.

repo: EduardoPires/EquinoxProject


text:
MQTT with Bluemix MQTT iothub

Hello Robin,
I am trying to use your MQTT module to connect my MQTT server in the bluemix. I got some initial problem with username and password. Why it should be less than 12 simbols ? I removed that limit but anyway have
[2017-03-07 01:09:58.106] [I] [core] [MQTT-12] Trying to reconnect (NaN)
Could you help me to solve the problem ? I created MQTT for your testing and sent all credential in the email.

Why it should be less than 12 simbols ?
Good question! The MQTT 3.1 spec used by the MQTT client we're using (the only client known to me which is plain, non-websocket, JavaScript and therefor works within the limited JS of ZWay) recommends to keep the username and password within 12 chars, see: http://public.dhe.ibm.com/software/dw/webservices/ws-mqtt/mqtt-v3r1.html#connect

As there are quite some calculations going on while composing the messages I can imagine there are some that assume a username/password within 12 chars. Are you able to try the broker with a shorter password? If that works we can dive into this.

Based on spec is is not mandatory. I know a lot of project where MQTT pass is 256 symb. Unfortunately I have no option to change this password. Did you recieve my email with test credentials. I also tried another MQTT client and got good respons from MQTT server.

Based on spec is is not mandatory.

I know, but as this MQTT client is a bit older and explicitly checks for the length, I'm afraid that somewhere they handle username/passwords longer than 12 chars incorrectly. Unfortunately the JS available to Zway apps is very limited, so this was the only MQTT client I could find that actually works.

Unfortunately I have no option to change this password. Did you recieve my email with test credentials. I also tried another MQTT client and got good respons from MQTT server

That's a pity. Yes I checked and confirmed it worked with another client. I didn't have the opportunity yet to test within Zway.

authors: Edubits xapundel

repo: Edubits/Zway-MQTT


text:
Improving Navbar for phones or smaller devices

Describe the bug
as soon we hit the hamburger menu the navbar shows up but it is very difficult to read it's content which makes up a bad user
experience. I would like to fix this issue.
Expected behaviour
The content in the navbar should be visible for user friendly experience.
Screenshots

Additional context
I would like to work on this issue
Code of Conduct

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

@SahityaRoy I would like to work on this issue , could you please assign it to me.


text:
build new Integrations screen

final design: https://www.figma.com/design/ItxXKIvUZRpdabGhMAlmPs/PR-Designs-(2024)?node-id=1101-21087&t=fq3M7syJdTLIpY49-4

access point in the app Settings: https://www.figma.com/design/ItxXKIvUZRpdabGhMAlmPs/PR-Designs-(2024)?node-id=874-15376&t=fq3M7syJdTLIpY49-4

during the Restore process, access to payment integrations has to be disabled because we cannot verify the wallet's balance:

final design: https://www.figma.com/design/ItxXKIvUZRpdabGhMAlmPs/PR-Designs-(2024)?node-id=1153-21113&t=fq3M7syJdTLIpY49-4

Resolved in #1362

authors: LukasKorba true-jared

repo: Electric-Coin-Company/zashi-ios


text:
added option for custom package.json

added option for custom package.json in electronize start comman
similar like manifest was added

All checks have failed - please fix it - Then a simple copy is not enough for everything else to work. Please have a look at Electron.NET HostHook feature. Here you can create and use your own package.json.

Electron.NET is not compatible with lower versions of Electron. It all depends on the Electron.NET versioning. As an example the next version will be: 8.31.1 - 8 stands for the supported electron version. The 31 for .NET Core 3.1 - and the 1 for our release from the new Electron version.
Compatibility for an electron / node version can be introduced with the electron-rebuild super package.
Alternatively, you can manage the Electron HostHook manually different package.json files yourself.


text:
定义列表未加粗

自定义 mainfont 之后，定义列表没有加粗效果
\documentclass[cn]{elegantbook}
\setmainfont[]{Source Sans Pro}

\begin{document}
\begin{description}
\item[Linux]
Linux
\end{description}
\end{document}

去掉 \setmainfont[]{Source Sans Pro}之后，正常的效果：

用模板就不要改模板，如果要改，说明你可能需要的不是 ElegantBook
\documentclass{ctexart}
\usepackage{sourceserifpro}
\begin{document}
\begin{description}
\item[Linux]
Linux
\end{description}
\end{document}

还有，请分清什么叫无衬线，什么叫衬线，\setmainfont[]{Source Sans Pro} 完全没有起到作用，除非你修改 description 的定义

注释
https://github.com/ElegantLaTeX/ElegantBook/blob/5d02e85f01096c8e1819c9d18c3bd2df388e3709/elegantbook.cls#L189
然后按你的代码即可，后果自负

authors: annProg sikouhjw

repo: ElegantLaTeX/ElegantBook


text:
cooking watch指令报错 没解决

执行 cooking watch指令 显示
[cooking] - 插件加载成功: vue
[cooking] - 插件加载成功: lint

[cooking] - server 启动中...
[cooking] - Listening at http://localhost:8080
Child cooking.conf.js:

    ERROR in Cannot find module './lib/source-map-generator'
     @ multi app

    ERROR in /Users/kevinli/~/webpack-dev-server/~/sockjs-client/~/json3/lib/json3.js
    Module parse failed: /Users/kevinli/node_modules/webpack-dev-server/node_modules/sockjs-client/node_modules/json3/lib/json3.js Line 778: Unexpected end of input
    You may need an appropriate loader to handle this file type.
    |               // Parses a JSON object, returning a new JavaScript object.
    |               results = {};
    |
     @ /Users/kevinli/~/webpack-dev-server/~/sockjs-client/lib/main.js 7:12-28
    Child html-webpack-plugin for "index.html":

webpack: bundle is now VALID.

为什么会在用户目录下有个 node_modules 😂

确保 cooking 安装在全局
把 /Users/kevinli/node_modules 这个 node_modules 删掉（应该会在这个目录有个项目吧？）

谢谢 解决了~

authors: LeonDaily QingWei-Li

repo: ElemeFE/cooking


text:
How [Feature Request] correctly sets the boder attribute of el-table

Existing Component
是
Component Name
el-table
Description
我自己尝试了修改了一下，不知道是否是正确方法：
https://jsfiddle.net/t75obz8w/2/
另外请教一下如果正确修改全局的el-table的border？
.el-table--border td,.el-table--border th,.el-table td, .el-table th.is-leaf{
border-color: #f50003;
}
.el-table--border{
border-color: #f50003;
border-bottom: 1px solid #f50003;
border-right: 1px solid #f50003;
}
这样修改是否是正确方法？

https://jsfiddle.net/t75obz8w/6/

sililar issue ,but I need border bottom none ,then solve issue by use
.el-table--border::after, .el-table--group::after, .el-table::before {
    content: '';
    position: absolute;
    background-color: #ebeef5;
    z-index: 1;
}

repo: ElemeFE/element


text:
[Bug Report] When using tree components, it encountered the wrong problem of dragging level

Element UI version
2.5.4
OS/Browsers version
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3626.109 Safari/537.36
Vue version
2.5.21
Reproduction Link
https://jsfiddle.net/h2vux1e5/1/
Steps to reproduce
节点拖拽操作即可。
What is Expected?
拖拽后的层级该是多少是多少
What is actually happening?
例如1层级拖拽的2层架，结果node-drop捕获到的值都还是1，实际上一个是1一个是2了。

没人回答呀

level 层级在拖拽完成后才会改变。


text:
[Bug Report] el-select will show value not label after deleting an item if don't use reserve-keyword

Element UI version
2.11.1
OS/Browsers version
Chrome / 75
Vue version
2.5.2
Reproduction Link
https://codepen.io/bitjjj/pen/qejpyx
Steps to reproduce

select some items
delete an item

What is Expected?
select should still show item's label not value
What is actually happening?
select show item's value after deleting an item

I also encountered, the original selection data will be cleared

authors: bitjjj durongzhang


text:
[Feature Request] 穿梭框什么时候有树形结构数据

Existing Component
是
Component Name
el-transfer
Description
el-transfer穿梭框什么时候能传入带有树形结构数据

同希望能支持树形表格穿梭

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

authors: Janet-wj bebezy


text:
[Feature Request] Select component add minCollapseTagsNumber prop and more slot

Existing Component
Yes
Component Name
Select
Description
// add minCollapseTagsNumber  prop and more slot:

prop: {
      minCollapseTagsNumber: {
        type: Number,
        default: 1
      }
}

// slot
 replace
  {{ selected.length - 1 }}
with
<slot name="more" :count="selected.length-minCollapseTagsNumber">  {{ selected.length - minCollapseTagsNumber }}</slot>

For example minCollapseTagsNumber = 3, the input collapsetagsnumber should be displayed：双皮奶 蚵仔煎 龙须面 +1


text:
[Bug Report] 表单 数字类型验证 不准确

Element UI version
1.2.8
OS/Browsers version
Win10 Chrome 59
Vue version
2.2.6
Reproduction Link
https://jsfiddle.net/没有。官方文档例子就可以重现了
Steps to reproduce
http://element.eleme.io/#/zh-CN/component/form
数字类型验证
输入 1000asdasdas,没有检测到不是数字
v-model.number 转换的时候会把'1000asdasdas' 转成1000
What is Expected?
检测更完善
What is actually happening?
输入 1000asdasdas,没有检测到不是数字

Vue 的行为就是如此：https://jsfiddle.net/leopoldthecuber/6hvk2qLx/


text:
[Feature Request] Trigger click on NavMenu collapse

Existing Component
Yes
Component Name
NavMenu
Description
Please add trigger="click" in NavMenu when collapse

very important issue

Sorry, we have no plan to support this.

authors: Leopoldthecoder dustinhbs wyqsess


text:
[Feature Request] add draggable attribute in Tree Component

Existing Component
Yes
Component Name
Tree
Description
The Tree Component can drag.
like https://fe-driver.github.io/vue-beauty/#/components/tree

Sorry, we have no plan to support this.

please ,help me .@Leopoldthecoder  BUG

authors: Leopoldthecoder wjtGitHub


text:
The label of [Bug Report] radio is an initialization state exception at number

Element UI version
2.0.7
OS/Browsers version
Ubuntu/Chrome62.0.3202.94
Vue version
2.5.9
Reproduction Link
https://jsfiddle.net/usey9u3d/
Steps to reproduce
进入https://jsfiddle.net/usey9u3d/
就可以看到，未激活，没有被选中。
What is Expected?
未激活被选中。
What is actually happening?
未激活没有被选中。

https://jsfiddle.net/usey9u3d/1/

authors: Leopoldthecoder baifan


text:
fix multiple input chat tempalte

For multiple input tasks the context = [], which made the chat template raise an exception. This PR applies the chat template to the choices as we use them as context in case of multiple input.
footgun: The llama tokenizer adds a system message automatically to all prompts, and if using few-shots the fewshot samples and the ctx are formatted separately which leads to two system messages in the prompt.

Worked for me 👍

repo: EleutherAI/lm-evaluation-harness


text:
Research Company - Valve

Company name:
Location:
Mission statement:
IT roles:
Glassdoor rating:
Benefits:
Link to jobs/careers page.

Company name: Valve
Location: Bellevue
Mission statement: (No official mission statement)
IT roles:

Game Development Software Engineer
Electrical Engineer
Industrial Designer
Software Engineer for HW
Hardware Sourcing and Supply Chain
Mechanical Engineer
Steam Database Administrator
Glassdoor rating: https://www.glassdoor.com/Reviews/Valve-Corporation-Reviews-E24849.htm
Benefits:
Valve is somewhat of a secretive company and doesn't list their official benefits on their website. However they are known to be one of the best game design companies to work at that offer a very flexible work schedule and the ability to work on whatever project the employee desires. Because of this, they keep their number of employees low at about 360 at the time of writing. This is incredibly small compared to other large game companies which have employee numbers in the thousands.
Link to jobs/careers page: https://www.valvesoftware.com/en/

authors: Elias-Carr heyhaylan


text:
Update create_rebabel_convert.js

The script no longer resets the virtual environment every time it is run. Instead, it checks if a virtual environment already exists, and activates it to create the executable. If it doesn't find a virtual environment, it will create one.
Also, the script console logging is improved. Fixes the issue where every output from pyinstaller was being treated as an error.

I get this error, "Failed to grant permissions: Command failed: chmod +x "

repo: ElizabethThorner/FrontEndRebabel


text:
Create a zombie infection simulation

Simple virus spreading simulation:

Create a number of people
Create very few infected
Infected move faster than people
When an infected touches a person that person becomes infected
When infection starts it takes some time to be infected (should change colour)

I'd be interested in taking this one if no one else is on it.  Should I make a new repo for it and then transfer ownership?

Yeah you can take it. Yeah I think that would be the best approach (after transfer you should keep contributor rights encase you wish to maintain/enhance it).
What are your thoughts on the implementation?

I was thinking it could be a perfect application for #101.
Basically, when a person is infected, they change to a transitional type, and the EntityListener schedules them for an action that gradually changes their color and eventually changes their type to Infected.
When they change type to Infected, they could undergo whatever changes they need to undergo, e.g. increase their speed, make them move toward people instead of away from zombies, etc.

I think you're right about #101 (so I updated that issue to contain what I thought it should be like - can you check it over?).
This is definitely the right way to go about the logic! Possible additions: Infected will die after X time/steps without infecting another and you can maybe add some sliders for amount of starting Infected, total population (not infected) and maybe population density.
The above additions will probably depend on how complex the model is. Sounds like you have it covered!

As of release v1.0.6 (on central) there is EntityTypeListener  and I've updated the wiki here
Should be good to go!

I think this is basically ready for a look: https://github.com/wlmiller/JALSE-Zombies
It's kind of fun to play with the different sliders.

Just gave it a look and a run up it's awesome! Definitely the coolest example we have right now.
How was it to make? Easy, challenging, frustrating? Do you think the core is missing anything to make life a bit easier?
Really impressed by this (and others who watch the repo have said so too)!

Thanks a lot!  Honestly it all came together pretty easily.
There were a couple of things that the entity type model made things more frustrating than I expected:

It's not easy to get the type(s) an entity is marked as.  When I was experimenting with it, the best way I found was to have a type Attribute, and use the entity type listener to set the attribute, which works, but was a little cumbersome.
It would be nice to have some type of unmarkAsAllTypes() method on entities, but this wasn't a big deal.
I would have like to have put the method that chooses which direction a person will move as a default method on the respective types - so e.g. Person.direction, Healthy.direction, Infected.direction, but every way I tried to make this work (I think I tried using Entity.asType() as well as setting an attribute as mentioned above) was really, really slow, so I ended up just throwing those methods together and giving the entities a Method attribute pointing to the appropriate one.

But overall the listeners and actions made it pretty easy to just sort of wire everything up and let it go.

Great that it went pretty smoothly!

So what would you recommend? I added getMarkedTypes() for the entity listener changes.
I think that is a good method to add and I will also rename getMarkedTypes() to getMarkedAsTypes()!
I have run into this issue before. The unreflectSpecial of default methods on proxies is incredibly slow and it's not something we can fix ourselves (we should raise with JDK really). I think that if we addressed 1 (or used another Action to generate the Direction and put it on the Entity in a chain MultiAction) we could get around this.

Let me know your thoughts

https://github.com/Ellzord/JALSE/pull/136

Or of course alot of ifs or a switch in a  for loop.

I have done some research here see this gist (I know its not properly benchmarked). It looks if we cache the method handle and not the we cut out over half the time but it's still much slower.
I was playing with MethodHandle and found I could get it to be quicker than regular reflection if I knew upfront what I wanted to do
unflectSpecial 0ms (42299ns) ignore 0ms (311ns) invokeExact 0ms (622ns) Total 0ms (43232ns)
unflectSpecial 0ms (43232ns) ignore 0ms (311ns) invokeExact 0ms (622ns) Total 0ms (44165ns)

The 622ns can be optimised further with caching the handle (JIT)!
NOTE: it took ~2000ns for each regular Method.invoke() on an each impl.
So it looks like there are some real optimisation to be done for the EntityProxyFactory system we currently use - I think I will will jump on this.

For 1 and 2, I think the things you suggested are good fixes, and shouldn't be hard to implement.  For 3, it was more of an observation - the type system doesn't lend itself to that type of inheritance, so my mental model of how I thought I would implement it didn't work in practice.

Cool - 1 and 2 I pushed a changed for already (including streamMarkedAsTypes()) so I will do a release shortly but you can use 1.0.8-SNAPSHOT until then. The MultiActionBuilder should help with generating the direction up front then they're uniformly processed via an action chain.
As for the performance concerns I have actually met these before and I think they're quite important really. Otherwise we're forcing our users to entirely use bean-like structures because JALSE can't handle any logic on a type (and requires a controller/helper/action class per type). I do think the action chain is a good approach for the Zombies simulation though. So I will do some changes around default methods (especially as private defaults are coming in Java 9).
Probably shouldn't be in this ticket but I'd like to come up with a roadmap for JALSE (what we should do next, focus on, move towards in the future, etc). Do you think you could gitter/email me your thoughts?

@wlmiller Great work!

authors: Ellzord wlmiller

repo: Ellzord/JALSE


text:
in multimodule mode, rewrite entry name to "main"

Feel free to close it if this is not desired.

This definitely can make loading modules earlier. It might be better as a config option though?

I see how this can make sense. But yeah, it does need to be an option, so please make it one and we'll be open to merging this.

Personally, I don't want to make it an option because it seems to be feature bloat to me.
I have the assumption that multi-module feature is just for compatibility reasons, so the entry rewrite seems always desired (or at least I don't see when it causes inconvenience). But I am just very new to graphics programming, my assumption might be wrong.

Your assumption is unfortunately wrong.. It turns out that multimodule is pretty much required today, as there are too many ecosystem bugs with keeping multiple shaders in a single SPIR-V binary. But we still like to use more specific entry point names than main. So merging this as-is would break our code, which we are not willing to do.

authors: expenses hrydgard molikto

repo: EmbarkStudios/rust-gpu


text:
Can't open more than three ports on win10

SerialPort version: 4.0.1
NodeJS Version: 4.4.7 and 6.3.1
Operating System and Hardware: Win10 64bit
Are you using Electron/Atom/NwJS?: no
(Run node -v && npm -v && python --version to get version numbers)
v4.4.7
2.15.8
Python 2.7.10
Summary of Problem
Cannot create more then 3 serialports on Win10.
Expected Behavior
Be open more ports.
Actual Behavior
If I create 4 or more it cannot be closed.
It also crashed my express webserver.
Steps and Code to Reproduce the Issue
If you put more than 3 portname to ports, the test will stuck after 'closing'.
If less then 4, everything is ok.
As I see the problem maybe at UV_THREADPOOL_SIZE
http://docs.libuv.org/en/v1.x/threadpool.html#
serialTest.txt

I found a workaround for this.
http://stackoverflow.com/questions/30983672/node-serialport-on-windows-with-multiple-devices-hangs
Set UV_THREADPOOL_SIZE enviroment variable larger than count of serial ports BEFORE you start your app.

You're welcome. :)
2016-09-02 5:12 GMT+02:00 Francis Gulotta notifications@github.com:

I'm glad you found this. Sorry it's an issue. I'm going to keep this open
pending the 5x release which might be able to address this issue on windows.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EmergingTechnologyAdvisors/node-serialport/issues/914#issuecomment-244272589,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGnNYwnGID2_3u8dv31aUej_qUZ6nhOKks5ql5QagaJpZM4JrVQ-
.

Would be great if 5.0 release could include a fix for this rather than setting environmental variables outside the program.

Ok thanks for confirming. I'll factor the environment variable into the setup for my app.

To work with many ports on windows one should use IO completion ports (IOCP) API.

Create completion port: CreateIoCompletionPort
Open port with overlapped flag, set port params. Add port fd to IOCP.
Issue ReadFile for port fd.
In separate thread do GetQueuedCompletionStatus in loop and get i/o state, grab data, emit events.

AFAIK libuv didn't have IOCP support, at least 1 year ago.

Hi, I'm having this same problem and would really like to have 4+ ports open simultaneously.
I understand that I need to change an environment variable before the app is setup, but I don't understand how to do that. I think I'm supposed to paste this line of code somewhere:
process.env.UV_THREADPOOL_SIZE = 6;
But I don't understand where to put that code. Can someone point me in the right direction? I pasted at the very top of my main app, but it's not working.
Thanks

Perfect thanks. I have it working now. I'm running my code from the command prompt, and it worked when I entered this into the prompt before running my app:
set UV_THREADPOOL_SIZE=8

I've same problem. Please mention this limitaion and env variable in release notes so people know about this..  Unfortunately it takes long time before I saw this issue and possible workaround fix ..

authors: Zensey deandob jaredrolson jupe konyee


text:
Crash on Startup, RadeonPro.

Just found a bug with program crashing on startup when Radeonpro is started first.
Minor issue.
Can be avoided with opening skill tree planner first, then Radeonpro.

Since no more info have been provided and no other reports, I close this.

repo: EmmittJ/PoESkillTree


text:
How is this different from the other EmmyLua plugin?

Hi, I have enjoyed using the EmmyLua plugin for Rider and wanted to know if I should be upgrading to the EmmyLua2 plugin? Maybe an updated readme.txt would help.
Thanks!

You can try it out first; it surpasses the original EmmyLua in every aspect

You can try it out first; it surpasses the original EmmyLua in every aspect

Great job, the plugin works really well and is much faster!

authors: BrianOstrander CppCXY SirMishaa


text:
Docs: Need to remove Ansible uninstall instructions from Kubernetes guide

Since Ansible installation is not an option for Kubernetes, need to conditionalize docs so that the uninstall using Ansible procedure is not included in the Kubernetes guide.

PR: https://github.com/EnMasseProject/enmasse/pull/3038

PR merged, so closing this issue

repo: EnMasseProject/enmasse


text:
Move results to the right side panel.

It would be better to put the query result to the right side, in a panel, just like the Google does. Like this

Hi @Explorare
Thank you so much for your suggestion! I'm unsatisfied with the current UIUX and I'm thinking of how to improve it.
My current js implement is too hacky, and I'm thinking of refactoring it. After that, I will be able to adopt a better UI

@Explorare Sorry for my late update. After some refactor work, I'm planning to add an experimental layout (facilitate the right side of Google).
However, the DOM tree of google.com is quite confusing to me, and I have no idea of how to insert a DOM element into that area now.
Do you know if there is any open-source browser extension or GreaseMonkey Script which is utilizing that area?
Thank you

I didn't find a script to modify the right side panel, but found this
搜索引擎切换器 / Search Engine Switcher
It adds a left panel.
Thanks for your effort on this plugin.

I didn't find a script to modify the right side panel, but found this 搜索引擎切换器 / Search Engine Switcher
It adds a left panel.
Thanks for your effort on this plugin.

Thank you. I'll have a look

I added an experimental layout

One can check the Experimental Layout in extension setting

0.0.13 version has been approved by Mozilla  https://addons.mozilla.org/en-CA/firefox/addon/fireseqsearch/versions/

In extension setting, check Experimental Layout, then refresh the page, the fireSeqSearch box should appear at right
I'll close this issue now, although I'll definitely keep working on the layout. I'm not satisfied with its aesthetics.

authors: Endle Explorare

repo: Endle/fireSeqSearch


text:
setting up Buck2 cpp RE EngFlow example

Add an example of RE with Buck2 for a simple cpp project.
See readme for more details.

Key open Qs I have about the setup are related to the platforms:

I had to create two different targets for platforms, not really sure this is the best / recommended path to get this working
Seems the exec_compatible_with is expecting a ConfigurationInfo provider but the default_target_platform is expecting a PlatformInfo provider.
Seems a bit confusing but it's working with this setup.
Any advice as to how to make this a bit easier to parse is appreciated

repo: EngFlow/example


text:
Fix package name in README.md for hud demo

Fix for README file that wasn't updated to reflect the new package changes

Thank you for spotting this!

authors: EtienneBruines jzs

repo: EngoEngine/engo


text:
Changes to Prometheus

Fixed #254
Fixing #253

@EnigmaCurry, this works insofar as it seems to correctly format the comma-separated list of targets from the .env file into the correct syntax to be templated into prometheus.yml. I can shell into the Prometheus container and cat /etc/prometheus/prometheus.yml and the values are there.
However, Prometheus isn't activating those targets - they're not available in the UI. In the past, I had manually edited /etc/prometheus/prometheus.yml to add these targets, and Prometheus recognized them. Now that d.ry is automatically adding them, Prometheus isn't recognizing them. I'm not sure why.


text:
Display and allow user to select PASSED_ON status for jobs

This adds PASSED_ON to both the display filters and selection dropdown for each job in the jobs tab. Will need to be merged up to db_version/91 after acceptance.

Addresses ENSCORESW-3281

repo: Ensembl/guiHive


text:
🛑 EnsembleTravel.ca is down

In 674a7d7, EnsembleTravel.ca (https://www.ensembletravel.ca) was down:

HTTP code: 0
Response time: 0 ms

Resolved: EnsembleTravel.ca is back up in adf5865.


text:
Seems to be slower than original model.

It seems to be a bit slower than using the original phi3-vision model.
I even tried ISQ and its still slower.
The original takes about 5 to 6 seconds and, when using mistral.rs it takes about 9 to 11seconds.
Is there something I'm missing?

Thanks for reporting this. Are you building for CUDA, Metal, or some CPU accelerator?

@EricLBuehler I'm building for CUDA.

@2U1 can you please let me know the prompt and completion/decoding T/s metrics for the Python and mistral.rs implementations?

Just merged #580 which adds +20% T/s to prompt processing speed of Phi 3V (in the CLIP model).

@2U1 closing now as I am measuring a 24% throughput improvement over Transformers including image processing, prompt, and decoding steps.


text:
Memory leak and channel closure issues when reusing/dropping Model

Describe the bug
When initializing and dropping the Model repeatedly:

Memory usage continuously increases as GGUF models aren't properly cleaned up
Channel is erroneously closed after the first iteration

Steps to Reproduce

Create a service that initializes and drops the model multiple times
Run the following code:

use anyhow::Result;
use mistralrs::{GgufModelBuilder, PagedAttentionMetaBuilder, TextMessageRole, TextMessages};
use std::time::Duration;
use tokio::time::sleep;

struct ChatService {
    model: Option<mistralrs::Model>,
}

impl ChatService {
    async fn new() -> Result<Self> {
        Ok(Self { model: None })
    }

    async fn initialize_model(&mut self) -> Result<()> {
        self.model = Some(
            GgufModelBuilder::new(
                "gguf_models/mistral_v0.1/",
                vec!["mistral-7b-instruct-v0.1.Q4_K_M.gguf"],
            )
            .with_chat_template("chat_templates/mistral.json")
            .with_paged_attn(|| PagedAttentionMetaBuilder::default().build())?
            .build()
            .await?,
        );
        Ok(())
    }

    async fn chat(&self, prompt: &str) -> Result<String> {
        let messages = TextMessages::new().add_message(TextMessageRole::User, prompt);

        let response = self
            .model
            .as_ref()
            .unwrap()
            .send_chat_request(messages)
            .await?;

        Ok(response.choices[0]
            .message
            .content
            .clone()
            .unwrap_or_default())
    }
}

#[tokio::main]
async fn main() -> Result<()> {
    for i in 0..3 {
        println!("Iteration {}", i);

        let mut service = ChatService::new().await?;
        service.initialize_model().await?;

        let response = service.chat("Write a short greeting").await?;
        println!("Response: {}", response);

        // Model is dropped here, but GGUF remains in memory
        drop(service);

        // Wait to make memory usage observable
        sleep(Duration::from_secs(5)).await;
    }

    Ok(())
}

Cargo.toml is here:
[package]
name = "memory_bug_mistral"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
tokio = { version = "1", features = ["full"] }
anyhow = "1.0"
mistralrs = { git = "https://github.com/EricLBuehler/mistral.rs.git", branch = "master", features = [
    "metal",
] }
regex="1.10.6"

Observed Behavior
Memory usage increases with each iteration even after explicit drop
After first iteration, receiving error:
Error: Channel was erroneously closed!

Expected Behavior

Memory should be properly freed when model is dropped
Channel should remain functional for subsequent iterations

Latest commit or version

mistralrs version: latest master: 32e894510696e9aa3c11db79268ee031a3ecefa6
Mac: M2
OS: Sonoma 14.7
Rust version: 1.80.1

@EricLBuehler
I'm wondering if you have any plans to address this memory management issue in the library?
While I could work around it using a web server or child processes for now, I'd like to understand your timeline for implementing a native solution. This would help me decide whether to proceed with a temporary workaround or wait for an official fix. Could you share your thoughts on this?


text:
Added option of using a different index per model

For our use-case we'd like to use different indexes for the models.  Laravel's Scout documentation has searchableAs as the function to override to set the index name.

Typically, this is the plural form of the model name; however, you are free to customize the model's index by overriding the searchableAs method on the model:

This pull request provides a configuration option within config/scout.php to allow an index per model.  It is backward compliant so it will behave the same as before if the option isn't set.
// config/scout.php
    'elasticsearch' => [
        'index' => env('ELASTICSEARCH_INDEX', 'laravel'),
        'hosts' => [
            env('ELASTICSEARCH_HOST', 'http://localhost'),
        ],
        'perModelIndex' => true,
    ],

Plus one on this feature. It would be good to have this option

Neat.
Elasticsearch 6.0 has several breaking changes, specifically in Laravel Scouts case, your only allowed one index. This is the first iteration to completely remove the _type support, which will be dropped by Elasticsearch 9.0.

A think that this PR can be very useful in various scenarios, including dealing with applications that exploit multiple database connections. For instance, given that two tables with the same name foo  but different fields exist in two separate databases (db1, db2), using an index per model approach can help to bypass the problem of storing two models with the same name (that is _type) under the same index.
In addition to that it provides a backward-compatible alternative to deprecated mapping types by exploiting an index per document type

For those who are looking for easy drop-in replacement for this package which works:
composer require prialgauskaslt/laravel-scout-elastic
I have added per-model indicing and plan on maintaining my fork for the upcoming years to come.
Ping me at my fork with issue linking PR in this repo which you think I should merge in new fork.
Cheers!

Now by default, the index name will be the model table

authors: ErickTamayo hoersten iraklisg prialgauskaslt princeton255 zanematthew

repo: ErickTamayo/laravel-scout-elastic


text:
Got no video when remaking calls

Everything is almost the same as what is in the NativeDemo, the only difference is that when I  make a video call, I present the video call VC and dismiss it when the video call is done.
The problem only happens after I successfully made a call. If I make another call, the call will go through and the audio is fine, but the videos do not show up(both self and remote) with an error message:** (:533): CRITICAL **: _owr_window_registry_register_renderer: assertion '!data->renderer' failed.
Should I do some additional work to avoid this error?

@songzhe1988 Hello,
Did you find a solution for your problem?
I am facing the same issue, and cannot solve it. Can you help please?

repo: EricssonResearch/openwebrtc-ios-sdk


text:
Тайлы Космоснимков

Предлагаю добавить русские тайлы Космоснимков по умолчанию (ну или хотя бы на выбор), потому что там более привычная раскраска карты, особенно транков, и более качественно проставлены названия дорог и более адекватно подобрана плотность населённых пунктов.

+1 на выбор, оставив возможность выбрать слой osm.org (всё равно переключатель слоёв уже сделали)

А что не так с лицензией osm.by?

добавил

СканЭкс не против, но опечатку в линке на kosmosnimki.ru поправьте :3

authors: AMDmi3 ErshKUS Ilis Komzpa Zverik


text:
Update v.next with main

PR to update the v.next branch with main.
Reasons for the PR

The main branch contains substantial stability changes
Browse build floors sample was missing from v.next but is present in the main branch
version.gradle changes <- This could be the reason for the daily toolkit builds failing.

I can't seem to make changes to resolve the merge conflicts, since v.next is a protected branch.  @TADraeseke
Conflicts:
kotlin/query-with-cql-filters/src/main/AndroidManifest.xml
kotlin/show-popup/build.gradle
version.gradle

To resolve: Use the v.next (head) in all cases.

repo: Esri/arcgis-runtime-samples-android


text:
Updated MAUI Sample Viewer to NET 7

Description
Updated MAUI Sample Viewer to NET 7 and fixed issues encountered during the update.
Type of change

[ ] Bug fix
[ ] New sample implementation
[x] Sample viewer enhancement
[ ] Other enhancement

Platforms tested on:

[x] MAUI Android
[x] MAUI Windows
[ ] MAUI Mac Catalyst
[ ] MAUI iOS

Checklist

[x] Runs and compiles on all active platforms
[x] Branch is up to date with the latest main/v.next
[x] All merge conflicts have been resolved
[x] Self-review of changes
[x] There are no warnings related to changes
[x] No unrelated changes have been made to any other code or project files
[ ] sample_sync.py runs without making changes
[x] Codemaid and XAML styler extensions have been run on every changed file
[x] Code is commented with correct formatting
[x] All variable and method names are good and make sense
[x] There is no leftover commented code

We should change any mentions of NET 6 to NET 7 i.e. reference doc tag on the readme page, any links, etc.

We should change any mentions of NET 6 to NET 7 i.e. reference doc tag on the readme page, any links, etc. Looks good to me otherwise

I've searched for net6 + net 6 and haven't come back with any hits on the MAUI viewer so I think it's all good to go.

repo: Esri/arcgis-runtime-samples-dotnet


text:
Android Time Slider: Rename .targets file to match package ID

Changes the name of the Esri.ArcGISRuntime.Android.Toolkit.targets file to Esri.ArcGISRuntime.Toolkit.targets to match the NuGet package's ID.  This is a requirement for the targets file to be added when the NuGet package is referenced.  The file has also been moved into a directory named build so it's not at the root level of the project.

Should it be moved to an Android subfolder below \build\ then?

Sure, would make sense for it to be in case we ever want to add one for another platform.  Done.

repo: Esri/arcgis-toolkit-dotnet


text:
Slider Component - slider-thumb styling issue in Mozilla

Expected Behavior
The slider thumb should be "circle-shaped" in all the browsers

Current Behavior
The slider thumb is "square-shaped" in Mozilla Firefox

Your Environment

Version used: 0.39.0
Browser Name and version: Mozilla Firefox Developer Edition - 69.0b8
Operating System and version (desktop or mobile): MacOS High Sierra - 10.13.6

Resolved in #278

authors: ChandanMahapatra codylawson

repo: Esri/calcite-react


text:
Support expanding multiple accordions when same link is in both accordions

We have cases where a sample in the JavaScript SDK https://js.arcgis.com will exist in the "Latest Samples" accordion and also exist in another section.
Here was the original question:

Could we figure out a way to open the sample folder entry point that was clicked?

Hi @lheberlie - doing some issue cleanup, apologies for the radio silence. It looks like maybe the link you provided has changed as I don't see an accordion there?
We have a new component library - Calcite Components - that includes an updated accordion: https://esri.github.io/calcite-components/?path=/story/accordion--simple
The updated component may resolve your issue, and if not, feel free to file a new issue at the Calcite Components repo: https://github.com/Esri/calcite-components

authors: lheberlie macandcheese

repo: Esri/calcite-web


text:
Last update to Dashboard render configuration pane blank

Hi,
I have used these samples with great success in the past, but after the last update of the Dashboard app (windows) both my own widgets and the samples in this repo are not functioning correctly. I'm not seeing the configuration page of any widgets, just a blank page. The debug console does not provide any information either. Is this behaviour known? It may be a user error of course. Thanks.

Hi @steffenph - did you see this issue only for configuration of the widget or everything about the widget? Does the widget itself load correctly?

It seems like this issue is both for the configuration and for the display of the widget. Using the URLs from the debug console directly in a web browser shows the page. More importantly, is this isolated to my dashboard install? Or will the new update break all the widgets in this repo for everyone.
Any tips for how to troubleshoot this?

hmm, this looks like could possibly related to a known issue that we have a KB article. Do you happen to have dual graphic cards in your computer? If you do, please refer to the solution described at http://support.esri.com/technical-article/000012611

Thanks a lot, @jaychen15! Reconfiguring my dual graphics card setup did the trick. All of the custom widgets now work as expected.
PS: Looking forward to the new version of Operations Dashboard:)

repo: Esri/dashboard-samples


text:
Hosted service CA_Measles_17APR2015/FeatureServer does not have a service definition item.

The hosted service "California Measles Outbreak" item exists id:406fb20818ca4b1081b6cfa7fdc93305 but does not have an associated service definition portal item. Without the service definition item the service won't be published to any user conference machines.
NOTE: this hosted feature service was published from a .csv file and not from ArcGIS Desktop or ArcGIS Pro, which means that this service will not be published to any of the UC servers.

I am building a demonstration for "Collaboration and Reporting Using Portal for ArcGIS" that shows Smart Mapping and non-address geocoding using a CSV file, and how it can be used in building web maps and apps.
Should I be using a different server?

Should you use a different server? Not necessarily. By using a CSV file and publishing a hosted service from it means that the service won't be automatically published during the Ops Server installation. This service would have to be manually published from the csv item that it on portal.

The first step in the demo is me showing the audience how to publish a CSV to Portal as a hosted service. Afterwards I use the hosted feature service in other maps and apps.

This will be okay.

authors: elinz mfunk

repo: Esri/ops-server-config


text:
HLZ Touchdown Points has validation error with Input Airframe Specification Table

HLZ Touchdown Points has validation error with Input Airframe Specification Table. If you open the tool in Pro (Final) the default Airframe selection comes from some previous run of the tool with an input Aircraft_Specification table (which doesn't exist).

But if we add a custom table the Input Airframe Specification Table has a validation error:

The error is misleading because the table DOES exist (just browsed to it), and why it is not supported is unknown. The problem might have something to do with the validation in one of the component tools in the HLZ model.
This is a problem for FedGC demo and is highest priority. Assigning to myself for now, but notifying @nfeuerstein

Testing this one out in ArcGIS Desktop 10.3.1 and ArcGIS Pro 1.2.

Created table in folder: .\Test_SGT_Issue_309\test_issue_309.gdb\test_issue_309.gdb
Open HLZ Touchdown Points in ArcMap.
point to double-named folder/gdb and it fails.

Created second table: .\Test_SGT_Issue_309\test_issue_309\test_issue_309.gdb
Open table in tool and it works.

OK. so the problem here is that there is a file geodatabase in a folder with the same name, including the ".gdb". Let's try it in a system tool, say Copy Rows:
6. Try to open the "bad" path, and it fails too.

But the "good" path works fine:

So in the end it seems that this is a data error with the path. Make sure you don't have a folder with a *.gdb extension.

repo: Esri/solutions-geoprocessing-toolbox