Extra backspaces cause the browser to navigate back to previous page

Click on a search suggestion.
Type a letter in the search field, then backspace to erase it.
Type backspace again. The search suggestion field disappears. The larger search box is no longer in focus.
Type backspace again. This causes the browser to navigate back to the previous page. This happens to me pretty often.

Is there a way that the larger search field could stay in focus after search parameter fields disappear? This would prevent an extra backspace from navigating to the previous page.
Thank you for making this available.

Ok, also noticed this sometimes. Thanks for this bug report.

Please test the latest build

It's not working for me. I'm trying to be sure that it's not something that I've done. Have you put the new build on your demo page? I ask because I see the same behavior there. When the outer box is blue, it's focused and extra backspaces are blocked. After a search parameter has been cleared with backspaces, though, the outer box is gray and not focused. Extra backspaces still navigate back.
I'll keep trying to see if it's something that I've done, but I wanted to get back to you. Thanks.

Ah, the Demo page was not updated yesterday, but now. (Clear browser cache)

I'm still seeing the issue on the demo page, even after clearing my cache. I sent you an email yesterday with a video of my screen. This might explain more clearly what I am seeing.

I updated the build and Demo page, please try again

Demo looks great!  Thank you!
I'll try it out in my code in the next few days.


Enhancements - Interval Trigger with start date and time.

Interval Trigger
The IntervalTrigger allows you to create a trigger that repeats at a specific interval. The trigger accepts two properties, an interval and an optional timeUnit.
This trigger can be used to implement timers.
For example, to set a trigger to repeat every 30 minutes from now:
import notifee, { IntervalTrigger, TriggerType, TimeUnit } from '@notifee/react-native';
const trigger: IntervalTrigger = {
type: TriggerType.INTERVAL,
interval: 30,
timeUnit: TimeUnit.MINUTES
};
ABOVE FEATURE ALREADY EXIST IN THE LIBRARY
but what I want is to start interval after specific date and time.
for example.  I want to start drinking water every 1 hour from tomorrow 10:AM onwards
So I should receive notifications from tomorrow by 10:00 AM, 11:00AM, 12:00PM, 1:00PM ....
This is the only implementation I want from the library.
Let me know if it is possible or not.

Is there any change to have this implementation ? There is a way to specify the start date and time for the interval triggers ?

i have same case here

I also have same case. Is there any progress in that?

@ashishmangukiya @Alexzenderus @brunul
Since this issue has been marked as stale. I have created another one for this here. Can you please comment on this to keep it alive?


Feature Request: List available zig versions for installation

This feature will allow us to verify what zig versions are available before installing it:
Something like:
zvm list

or
zvm check

or
zvm versions

To me the choice of the cmd does't mather much, only to find what binaries are available to be downloaded

You can use zvm ls for this, no?

You can use zvm ls for this, no?

Actually not, because zvm ls only lists installed versions.
Also, I have trouble installing other versions of the current dev release e.g I currently can install master which gives me version  0.12.0-dev.790+ad6f8e3a5 but for another project, I would like to be able to test version 0.12.0-dev.415+5af5d87ad.

@mlaass That is an upcoming feature of ZVM. Currently, it only installs the daily release and stable, but within the month It should support all versions of Zig (starting with 0.12.0-dev-***).

This feature has been added. Thanks @felipetesc for your patience.

This feature has been added. Thanks @felipetesc for your patience.

It's you who should be patient with us when we don't use common sense to understand the nature of open source projects. I can only acknowledge by understanding that with the use of your free time you have improved the library. Library which, have as the mainly beneficiaries ourselves. Thanks!


DOC: Modified the collector README.txt to push up the Preparation/grants little above to catch the user's attention earlier

Documentation change request Details?
Change
Version
None
What oracle database version are you seeing the problem on?
None
Which sql script version ?
None
Relevant log output
No response

closed by #234


add -LineNumber switch to select-string and get-content

Hi Guys !!
with a large file ex log file its usefull to display the line number.
example: if we serach the term 'Error' within a large log file with select-string its useful to return the match string and the line number...
PS > cat /var/myverybiglog.log | sls 'Error' -LineNumber

If you let Select-String read the file, you already get line-number information - and the command will perform much better as well:
PS> 'a'..'z' > in.txt; Select-String q in.txt | Select Line, LineNumber

Line LineNumber
---- ----------
q            17

@mklement0
thanks for this info. but the pipeline slow the operation + linenumber is much better if it present as switch parameter. also cmdlet like get-content and format-table  (index property in display)  need indexing the line number

@p0W3RH311, yes, allowing Get-Content to directly allow selecting lines based on indices / line numbers would be helpful, similar to Select-Object -Index - see #3237
I don't understand how Format-Table would benefit from such a feature, however.


Native capabilities of macOS

Let's gather ideas for native capabilities here
Any ideas are welcome, feel free to add them as comments to this issue

Notifications

Adding support for the menu bar would be highly appreciated. This includes not only the apps own menu but also its own status icon if it has one. But could also include notification center and others.

https://support.apple.com/en-gb/guide/mac-help/mchlp1446/mac

Modal dialogs would also be needed. Most importantly NSAlert.
(@jBorkowska would it make sense to open single issues for each feature and link them from this one tracking issue?)


Multitline yaml config

Minimal .gitlab-ci.yml illustrating the issue
---
job:
  script:
    - echo $VAL1
    - echo $VAL2
    - echo $VAL3
    - echo $VAL4
    - echo $VAL5

# .gitlab-ci-local-variables.yml

VAL1: 'Host gitlab.com
  HostName gitlab.com
  PreferredAuthentications publickey
  IdentityFile ~/.ssh/id_autobot_ed25519'
VAL2: |
  Host gitlab.com
    HostName gitlab.com
    PreferredAuthentications publickey
    IdentityFile ~/.ssh/id_autobot_ed25519
VAL3: Host gitlab.com
  HostName gitlab.com
  PreferredAuthentications publickey
  IdentityFile ~/.ssh/id_autobot_ed25519
VAL4: "
  Host gitlab.com
    HostName gitlab.com
    PreferredAuthentications publickey
    IdentityFile ~/.ssh/id_autobot_ed25519
  "
VAL5: >2
  Host gitlab.com
    HostName gitlab.com
    PreferredAuthentications publickey
    IdentityFile ~/.ssh/id_autobot_ed25519

Expected behavior
It should echo multiline values preserving the \n character.
Actual behavior
build             $ echo $VAL1
build             > Host gitlab.com HostName gitlab.com PreferredAuthentications publickey IdentityFile ~/.ssh/id_autobot_ed25519
build             $ echo $VAL2
build             > Host gitlab.com HostName gitlab.com PreferredAuthentications publickey IdentityFile ~/.ssh/id_autobot_ed25519
build             $ echo $VAL3
build             > Host gitlab.com HostName gitlab.com PreferredAuthentications publickey IdentityFile ~/.ssh/id_autobot_ed25519
build             $ echo $VAL4
build             > Host gitlab.com HostName gitlab.com PreferredAuthentications publickey IdentityFile ~/.ssh/id_autobot_ed25519
build             $ echo $VAL5
build             > Host gitlab.com HostName gitlab.com PreferredAuthentications publickey IdentityFile ~/.ssh/id_autobot_ed25519

Host information
Ubuntu
gitlab-ci-local 4.1.33
Containerd binary
No
Additional context
I am trying to replicate the gitlab environment variables locally and I am struggling. In gitlab, my script creates .ssh/config by echoing the value of the file, which is stored in gitlab variables. The script basically just does echo:
echo $VAL1 > ~/.ssh/config

However, no matter what multiline format I use, the line endings are striped and the multilines are converted to a single string. Either I am doing this wrong, or this is a bug.
PS. gitlab-ci-local is awesome - tyvm! I'm eager to get it working.

hey, i don't think this is a bug on our side but rather quirks with shell
try surrounding your variables with double quotes

@ANGkeith, thanks for your quick reply. I tried what you suggest and got it to work, but it only works with .env. The yaml config behaves differently and only ever produces one line.
This is exactly what I am using:

Your explanation is clear. I don't know what wizardry gitlab is using, but unfortunately it requires that I change my CI to work-around this difference in behaviour, which is unfortunate.
Incidentally, I came across another separate, but related issue. I'll raise a separate ticket.

hihi, sorry but i don't really understand you.
could you share again what's the exact yaml that worked in gitlab.com but didn't work in gcl ?


Thumbnails not displaying on search results

I think this is a result of https://github.com/pulibrary/pomegranate/blob/master/app/helpers/application_helper.rb#L82
@document isn't defined in search results. I pushed up a branch to test, but I can't deploy it because of an existing issue with the shared_files mount not being able to mount ATM.

Can we make this a high priority, @tpendragon ? We need it for the exhibit, opening Sept 12, and I just got a rush request for it for the Spain 15M movement from Fernando.


Campaign | Challenges: Accept challenge invitation

Once one team has selected another team for a challenge in #308, the other team receives an invitation which they can either accept to decline. If the invitation is accepted the challenge becomes live and is shown in a different color on the teams page #180.

Prototype: https://www.figma.com/proto/7wE0Iveas3GGb8u4oL0714/App-Design-Mitglieder-App-(1)-(Copy)?node-id=2769-7270&t=b8swD5IOAFfZ0OSo-1&starting-point-node-id=2425%3A17429
Pollion reference: https://github.com/pollion/sunflower-app/issues/101

UX questions @leagerndt @JuliaFreitagBGSt :

does only the team lead get challenge invitations or all team members?


Include a theme toggler in the navbar

Having to log in and navigate to the user settings page is a long journey to change the theme. There should be a theme toggler in the Navbar (Public and Private) to make this accessible to everyone.
Example

I'd love to work on this.

Nice idea. You can work on it. Please try to reuse the existing logic from the account setting page.


Two changes to make local debugging of ILC failures easier

Put the .ilc.rsp file to the $(IntermediateOutputPath), not
into $(NativeIntermediateOutputPath) folder as the latter one
gets deleted after each run. The response file is typically
less then 1 K in size and so it should pose no threat to
the lab disk space.

Opportunistically create the output directory in the
ILCompiler so that, once we copy & paste the @(sometest.ilc.rsp)
file from the test log into the ILCompiler VS project,
we can directly run the compiler without manually recreating
the previously deleted "native" subdirectory.

Thanks
Tomas

I can easily live with this change in a private branch if we believe it's somehow harmful to the compiler eco-system or design. As for not deleting intermediates, my understanding from Simon is that in the lab we must always delete the intermediates no matter what as otherwise we'd start immediately overflowing the disk space.

I think it would be better to address the problem by changing the test script to delete the files only and not delete the .rsp file. It does not sound right to be changing everything else to compensate for the test script behavior.


Does having an actual date for sequencing open the potential for subject identification?

My understanding is one technique to avoid subject identification is by replacing dates with relative times from an arbitrary t0 time point and that's what we have done from most of the AIRR standards. However I noticed sequencing_date is an actual date. Does it make sense to convert this to a relative time as well?

sequencing_date is when you turn on the NextSeq, right? That could be arbitrarily long after sampling or even after the conclusion of the study. So I'm not sure relative time would make sense...

Yeah, that's my understanding, though it should be before the conclusion of the study, if the study involves analyzing AIRR-seq then you need to sequence first... But yes, it could be arbitrary long after sampling or very soon.
I don't have a clear use case in my mind on what might happen. I was mainly wondering, if you knew the sequencing date then maybe there was the potential to backtrack and roughly determine other dates. As always you need other information if trying to do identification.
Thinking about it in another way. By having a date, you don't really know when the sequencing was done in comparison to the sampling, did the sample sit in the freezer for 20 years? Maybe that's a useful tidbit?


geoparquet coordinate reference system default / format / extension

There are a lot of options for how we approach coordinate reference systems.

GeoJSON only allows 4326 data. They started with more options, but then narrowed it down.
Simple Features for SQL defines an 'SRID table' where you are supposed to map number id's to crs well known text. PostGIS uses the same srid's as epsg, but oracle doesn't.
WKT doesn't include projection info, but that's seen as a weakness, and one main reason why ewkt came. I believe ewkt just assumes epsg / postgis default srid table, so it's not fully interoperable with Oracle.
STAC uses epsg, but you can set it to null and provide projjson or crs-wkt
OGC API - Features core specifies just WGS-84 (long, lat), using a URI like http://www.opengis.net/def/crs/OGC/1.3/CRS84, see crs info

And there's obviously more.
My general take is that we should have a default, and expect most things to use that. But should specify it in a way that it could be an extension in the future. So we shouldn't just say 'everything is 4326' just in the spec, but should have a field that says this field is always 4326 for the core spec, but in the future that field could have other values.
So I think we do the first version with just 4326, and then when people ask for more we can have an extension.
One thing I'm not sure about is whether we should use 'epsg' as the field. EPSG covers most projections people want, but not all. In geopackage they just create a whole srid table to then refer to, so the SRID's used are defined. Usually the full epsg database is included, but then users can add other options.
One potential option would be to follow ogc api - features and use URI's. I'm not sure how widely accepted that approach is, like if the full epsg database is already referenced online. So instead of 'epsg' as the field we'd have 'crs', and it's a string URI.

Hi @cholmes, I'm a data engineer at the Department of Environment, Food, and Rural Affairs in the UK. We're currently in the process of moving our spatial data processing and analysis into the cloud, where we're hoping that Spark will help us to exploit data sources that were previously beyond the capabilities of our aging IT infrastructure. As such we've been watching the work that @jorisvandenbossche and the GeoPandas' team have been doing on geoparquet with anticipation, so we're really pleased that you're all working together on this. Is there anyway we can help or contribute?

@EFT-Defra - I'm super sorry for the super slow response on your query, I somehow didn't see it until a couple of days ago. It would be great to help out! I think the easiest way to help out would be to make some data available for others in any version of a geoparquet spec. So testing out the tools to help it, and give others test data. And then of course contributing to any actual spec.
At my company (Planet) we're getting some more resources to start on this, so I'm hoping to gather a group on it soon.

@paleolimbot - I think I'm good with OGC:CRS84 - you make good points about different standards (I hadn't heard of that specifically, but I'd believe it since lat long ordering is a mess - which are the ones that do it differently?)
I do agree that using a suitable projected coordinate system will improve speed and accuracy when used right. But I am not sure it'll always be used right / selected properly. There are a lot of parquet users who don't know anything about geospatial, so I think we should make some recommendation for what they should use. I'm not at all attached to which one. But I don't think the core spec should make people understand projections.

It seems like the main thing is to keep users from putting longitude, latitude (which users are often handed in some way) into a parquet file without labelling it as such via a CRS.

Fully agree.

I think that may be easier to solve at the implementation level rather than the spec level though.

Definitely. Though I think it'd be good to have an early 'validator' that will raise errors if it encounters geo data that is not labeled with a CRS. That could help guide implementations, instead of us trying to 'catch' every possible implementation.

If there was no CRS present, a parquet/Arrow writer should probably error by default with a message that tells somebody what the CRS is for lon/lat since that's probably what they have?

Yeah, I like this. Could give them some hints to help verify that their data is lon/lat before they naively stick it in. Maybe it's not in the spec itself, but a 'best practice' recommending implementations do this.

A note that you can write a prototype Parquet file for any GDAL-readable data source via the R geoarrow package

Oh awesome!

Thanks to everybody for the work on moving this forward.  I agree with the overall idea, and solving the default CRS at the implementation level rather than the spec level though.
In my experience with cloud vendors, I think would be better to include always a CRS. Today it's a little bit messy because some magic appears due to the absence of a CRS and with the mix between geographies and geometries. BigQuery only supports spherical coordinates, if you upload data in WKT it assumes the data uses WGS84 coordinates with spherical edges, but if you load data in GeoJSON format it assumes it's in planar edges and it makes a conversion. On the other hand, Snowflake treats GeoJSON as JSON-formatted WKT with spherical semantics. I've seen lots of issues around this problem during the last year. If the data includes the CRS, the conversion from planar edges to spherical edges will be implicit and lots of issues will disappear.
I think those warehouses will include support for geometry types beyond their current geography and it'll become more important for them.
I agree with @jorisvandenbossche that it can also be a boost in performance since knowing the CRS can save projections operations.

I think the easiest way to help out would be to make some data available for others in any version of a geoparquet spec.

Hi @cholmes, the problem we have there is that it's not possible to write geoparquet directly from pyspark as pyspark doesn't allow you manipulate schema-level metadata, only column-level metadata. We have a work around that involves encoding geometries as WKB, writing to vanilla parquet, grabbing the first part of the partitioned dataset with pyarrow, updating the metadata, then re-writing the first part.
TBH, this is such a faff that we tend to just convert all datasets to British National Gird (EPSG:27700), as most of the major ones we use (such as OS MasterMap) already are in that projection, convert all geometries to WKB, and keep everything in vanilla parquet.

@EFT-Defra - very interesting. Is that common that tools only let you manipulate schema-level metadata? Is the same thing true for other spark tools? It seems like the thing to do would be to improve pyspark to manipulate schema-level metadata. I'm also curious if there's other tools you could potentially use to write out a geoparquet from spark? That might be easier to improve to write out schema-level metadata.
cc @echeipesh @pomadchin - do you all have experience with pyspark? And thoughts on how to get it to play nicely with geoparquet?
Also, we're gathering a group to co-work on GeoParquet to hopefully get to a 0.1 release next tuesday at 15:00 UTC. Send me an email at cholmes at planet dot com if you are interested in attending, all are welcome.

Notes from the call:
What format to allow for coordinate system? Just wkt? Authority codes?

CRS field required or not (allow nulls)

Yes required, no nulls allowed

Does it have a default or not? Or recommended?

Recommended  - be sure to say for interoperability. Why is it is recommended - for most wide interoperability. And if you’re going to support one just do this. Expect some readers will only do this.

What is the recommended?

OGC CRS 84

WKT2 or more?

Just WKT2

Coordinate order

Stored in fixed xy / lonlat and it doesn’t depend on the crs? Or interpreted according to CRS? -> Fixed order
Use geopackage wording: "The axis order in WKB stored in a GeoPackage follows the de facto standard for axis order in WKB and is therefore always (x,y{,z}{,m}) where x is easting or longitude, y is northing or latitude, z is optional elevation, and m is optional measure. This ordering explicitly overrides the axis order as specified in the SRS metadata"

Closing after merge https://github.com/opengeospatial/geoparquet/pull/25


AssertionError: read=3, expected=10

Excuse me, i have this error trying to open some psd files.
This file open correctly in any other program but i cant with the lates version 1.9.11
Traceback (most recent call last):
File "./test.py", line 18, in
psd = PSDImage.open("./test2.psd")
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/api/psd_image.py", line 104, in open
self = cls(PSD.read(f, **kwargs))
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/psd/init.py", line 72, in read
LayerAndMaskInformation.read(fp, encoding, header.version),
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/psd/layer_and_mask.py", line 58, in read
self = cls._read_body(fp, end_pos, encoding, version)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/psd/layer_and_mask.py", line 65, in _read_body
layer_info = LayerInfo.read(fp, encoding, version)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/psd/layer_and_mask.py", line 131, in read
self = cls._read_body(fp, encoding, version)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/psd/layer_and_mask.py", line 140, in _read_body
layer_records = LayerRecords.read(fp, layer_count, encoding, version)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/psd/layer_and_mask.py", line 339, in read
items.append(LayerRecord.read(fp, encoding, version))
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/psd/layer_and_mask.py", line 444, in read
*cls._read_extra(f, encoding, version)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/psd/layer_and_mask.py", line 458, in _read_extra
tagged_blocks = TaggedBlocks.read(fp, version=version, padding=1)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/psd/tagged_blocks.py", line 158, in read
block = TaggedBlock.read(fp, version, padding)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/psd/tagged_blocks.py", line 256, in read
data = kls.frombytes(raw_data, version=version)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/psd/base.py", line 68, in frombytes
return self.read(f, *args, **kwargs)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/psd/adjustments.py", line 192, in read
extra = CurvesExtraMarker.read(fp, is_map=is_map)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/psd/adjustments.py", line 225, in read
signature, version, count = read_fmt('4sHI', fp)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/psd_tools/utils.py", line 37, in read_fmt
) == fmt_size, 'read=%d, expected=%d' % (len(data), fmt_size)
AssertionError: read=3, expected=10
test2.psd.zip

Seems the file is not generated by Adobe Photoshop, and contains bad structure around Curves. Next release implements workaround.

 Thanks friend!!


appsre: disable aarch64 AMI creation temporarily

This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

temporarily until #2961 is resolved.

Just a change to disable one packer build, we need to have this in main asap, otherwise the jenkins pipeline keeps failing, so I'm admin-merging this.

btw, @diaasami, I think we prefer # for comments in HCL, but // of course works too. No hard feelings though. ;)


Make inheriting from simulation world possible

This just allows users to be able to inherit from the simulation world to build their own simulation worlds :rocket:

Can one of the admins verify this patch?

@alexmillane Hi Alex, would it be possible to merge this?

Can one of the admins get this tested please :) @mfehr @helenol

test this please

Since you turned the sim world member into a pointer, you should check if it is a nullptr everywhere it is dereferenced, or at least once per function that is using it.

@mfehr Done! Thanks for the review Marius!

ok to test

Yes! Although I think I did switch to CHECK(unique_ptr) in Kimera as well... bcs earlier versions of glog had this problem.
It's really up to you, I can change that to be consistent with voxblox, but, in principle, new users should not see any issues.

Friendly ping @mfehr :)


[BACKLOG-32776] Fixed relative drivers location on Windows Pentaho Se…

…rver

:white_check_mark: Build finished in 20m 22s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,engine

:ok_hand: All tests passed!
Tests run: 4737, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 19m 52s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,engine

:ok_hand: All tests passed!
Tests run: 4737, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message


Unnecessary white space on the left of the cells

There's a lot of white space on the left of the cells.
Enough space for a whole vertical scrollbar on the far left.

Let's use 8px for the left margin.

Woo hooo...Validated


Move examples to another repo

Current main repo is around 100 MB when download those examples and the history files. It is better to move those unnecessary files to another repo to keep this one cleaner, and easy to clone.

Yes, some examples need to be moved elsewhere, such as ppocr and YOLOX, as their models cannot be generated online. Meanwhile, the corresponding links in the online documentation also need to be updated.

Maybe put all the models on Huggingface. No need to update those many links.


Skip Auth header if Anonymous-Options is given to request message

In some cases we might wish to call an API before we are authenticated (health endpoints, kodeverk, etc..).
To make the HttpAuthHandler not add authentication headers to a single request you can add an Option
to the request with the key name "Anonymous"

The README file needs to be updated to reflect this new option.

Would be good to add a unit test for this method (SendAsync) now.

Otherwise the code LGTM.

The README file does not contain readme-instructions. Should I just add a new section, or update the readme in fhi-azdev?

I have updated the wiki with the new functionallity:
https://github.com/folkehelseinstituttet/fhi.helseid/wiki/step-by-step-guide


Make inline code underlined in man pages

This is the promised second change in the man pages. The documentation in the ruby file says it all.
Don't be confused, I configured my urxvt to show underline in blue text coloe. So here's a before (top) and after (bottom):

Yes, it's a bit sad, that if something contains spaces (see the <config-command> … in the middle of the pages) the spaces themselves are not underlined. But the result is still better in my opinion. Maybe something for the future.

Coverage remained the same at 59.168% when pulling 3f1a4c05e092bc77695183b822a80a1a95c246e4 on der-lyse:prettify-manpages into f87c6424ae49de4dfcdb0c5605fe3df9fba4e3be on newsboat:master.

This looks cool! I want to see if I can do anything about the spaces not getting underscored, so I'll hold this for now. Also, this doesn't work with older versions of Asciidoctor (see Ubuntu 18.04 logs), but I didn't check how hard it'll be to fix.

Ah, crap! I will try to see what can be done about it on older Asciidoctor versions.

Ubuntu 18.04 ships Asciidoctor 1.5.5. In the previous major version things were a little differently named and organized. So this code it's quite a bit messy. But it works. Let's see what CI will find with potentially even more different versions. I hope nothing will pop up. Fingers crossed.

Whoops, left some debugging stuff in there. It's now removed.

Meanwhile, I satiated my curiosity about underlining; apparently that's just how terminals chose to display it, and there's no macro in groff that can make them behave differently. Oh well. This is still way better than no formatting, so I'm merging. Thanks for the work!


Bug Fixture Remapping VCWidgets

Related to this forum post : https://www.qlcplus.org/forum/viewtopic.php?f=20&t=15864
Somehow some VCWidgets are listed twice (or more ?) with getVCChildren. The remapping happens then more than once, making confusion in fixture IDs at the second remapping.
Just removing duplicate VCWidget from the list using a Set.

How is it possible that a widget is listed twice?

I haven't debugged that far. I had a conditional break point on the id of the slider widget that I knew was problematic. That break point was hit twice. That's how I found out there were duplicates in the list.

getVCChildren is recursive. Hitting a breakpoint twice doesn't mean there are duplicates.
Is this patch actually solving the issue you describe in the forum?
Debugging this issue with 65 fixture like in your project is impossible for me.

Yes, this patch solves the issue I described in the forum.
The break point I set was in the accept function in the for loop after the call to getVCChildren like this :  ( line 831 )
if (slider->sliderMode() == VCSlider::Level)             {                 if (widget->id() == 19) {  // ID of the widget I knew was problematic.                     QDebug() << "19"; // break point here                 }
At that point the list shouldn't have any duplicates so that's why I thought about using a QSet here.
I also noticed that one to one remapping didn't keep the HTP/LTP settings on the fixture. That's solved the third commit of this pull request ( probably should have addressed this in a new PR ).

Ok. The HTP/LTP patch is legit.
Right now I'm adding fixture import code as you suggested.
I really would like to understand this ID duplication but I'd like to have a simplified project to reproduce the issue

I have pushed the import code and your patch for HTP/LTP.
I'm gonna analyze the ID duplication later

Great thanks.
In the meantime I have come with a simplified project that reproduce the issues. It uses uses 2 fixtures, 2 sliders and 3 frames. Each slider is in its own frame and those two frames are in another frame. Although the remapping ( cloning ) has no issues, the sliders are listed twice. Seems like something's wrong with the nested frames.
As I can't attach a .qxw file here, I'll upload it in the forum post.

Finally got it! See 5f2bf86868b5c714c5fbbbcf0d2a5c59d2d82be3
Thanks for the simplified project. It helped indeed.
Basically QObject::findChildren is already recursive unless explicitely specified!
Thanks for reporting :+1:


Hide tabbar on push is not working.

Hide tabbar on push is not working.
TabbarController -> KYDrawerController - > MyUITableViewController -> push to detail view
I want to hide the UITabBar on the detail View. It does work.

same for me

Perhaps Hide tabbar on push does not work only in the navigation controller to manage the tab bar controller.

If you wish to achieve it, you need to configuration as shown in FIG.

This looks like an iOS development question rather than something that specifically relates to KyDrawerController, thus closing.


appendDots is overriding className declarations

with this configuration
appendDots: dots =>

{dots}

,
div.myClass is reasonable to expect for,
div.slick-dots is resulting tough.

className is over-written here
https://github.com/akiran/react-slick/blob/master/src/dots.js#L71
Feel free to open a PR fixing this issue

Has this already been worked on? Or is it still up for grabs? I'd like to work on fixing it if it's still open. :)

sure, go ahead.

Hey @sinharaksh1t , are you still on this issue ?

Pull request ready for review
https://github.com/akiran/react-slick/pull/1666

Hey @sinharaksh1t , are you still on this issue ?

Sorry @arnaudmanaranche, I haven't been able to work on this. Got a bit too busy with IRL stuff. Thanks for getting this done @VincentCharpentier 🙌

Hi @akiran ,
Do you think it is possible to have a review for the pull request and maybe an update relatively soon ?
That would be awesome, thanks :)

Up, PR waiting for review

Hi! While we waiting the approval, the react fragment works well for this issue:
appendDots: dots => <>

{dots}

</>


xy_19

Closes #297
NCL Plots: https://www.ncl.ucar.edu/Applications/Images/xy_19_1_lg.png https://www.ncl.ucar.edu/Applications/Images/xy_19_2_lg.png
https://www.ncl.ucar.edu/Applications/Images/xy_19_3_lg.png
Python Plots:

I don't think that there will be a status reported for this pull request. I may be wrong, but I've been seeing no status reports from github actions on pull requests from forked repos.

I don't think that there will be a status reported for this pull request. I may be wrong, but I've been seeing no status reports from github actions on pull requests from forked repos.

I don't think that there will be a status reported for this pull request. I may be wrong, but I've been seeing no status reports from github actions on pull requests from forked repos.

Right, I'm not sure why that is. Possibly because before the only checks being run on forked branches were the ones that got removed last week? To check the forked PRs, I've been cloning the branch and letting the actions run (see test_xy_19_again) and activating the cloned branch on RTD.

I don't think that there will be a status reported for this pull request. I may be wrong, but I've been seeing no status reports from github actions on pull requests from forked repos.

Right, I'm not sure why that is. Possibly because before the only checks being run on forked branches were the ones that got removed last week? To check the forked PRs, I've been cloning the branch and letting the actions run (see test_xy_19_again) and activating the cloned branch on RTD.


error: openlogin-sdk.init or openlogin-sdk.redirectUrl is not set

This bug is reproduces only for the first attempt of autorisation with Google provider. I use v3.5.1. What's wrong and how to fix that?

@chaitanyapotti I get this same issue in ios devices only at first time of login. Even after uninstalling and reinstalling I am unable to reproduce this. Please check this issue.

@chaitanyapotti I am currently experiencing this issue, any particular fix pls?

Fixed in 6.0.0. Pls upgrade


Add support for self-hosting

Does this mean if I clone the repo, I can't actually run it?  (it will only run on the deploy target?)

You can already run it on shuttle.rs with make dev, but some people want to run it outside of shuttle apparently.
https://news.ycombinator.com/item?id=33110791


Update gnn_citations.py

Fixing a simple issue
Traceback (most recent call last):
File "/Users/xxxxxx/Desktop/keras-io/examples/graph/gnn_citations.py", line 251, in
x_train = train_data[feature_names].to_numpy()
~~~~~~~~~~^^^^^^^^^^^^^^^
File "/opt/homebrew/lib/python3.11/site-packages/pandas/core/frame.py", line 3857, in getitem
check_dict_or_set_indexers(key)
File "/opt/homebrew/lib/python3.11/site-packages/pandas/core/indexing.py", line 2687, in check_dict_or_set_indexers
raise TypeError(
TypeError: Passing a set as an indexer is not supported. Use a list instead.
Convert to list

Added those files too. Thanks a lot Francois!


New task needed - Soft_validation - summary page

I would like to get a soft_validation overview for entire dataset. At the moment we have soft_validation at least on TaxonName, TaxonNameRelationship, TaxonNameClassification, Source. We need to find a way to soft_validate entire dataset and summarize the problem sorted by categories. Preferably, if we could utilize parallel processes and do validation in reasonable time. Alternatively, soft_validation messages could be cached on each record in the database.
Bonus, restrict validation to a particular brunch of the tree.

Think this is a duplicate?

Closing as duplicate/related of #553.


[Question] How to reload plugin with new wasm file?

How to reload plugin with new wasm file?

@fangzhengjin - which SDK are you using?
Also, what are you hoping to preserve by reloading a plug-in? If there's nothing about the previous plug-in needed, you should just create a new plug-in with the new wasm.
depending on the SDK you may also want to free the previous plug-in.

@fangzhengjin - which SDK are you using?
Also, what are you hoping to preserve by reloading a plug-in? If there's nothing about the previous plug-in needed, you should just create a new plug-in with the new wasm.
depending on the SDK you may also want to free the previous plug-in.

I use Rust SDK.
I want to upgrade the plug -in without ending the main process and replace the WASM file. For upper users, they are still the same plug -in.
I saw that Rust SDK provided some running methods, but they were all marked as unsafe, and lack of detailed explanation and use examples. Should I use them?

Maybe extism can provide a way to upgrade the plugin?
Block new plug-in call requests during the upgrade process, clean up old plug-in resources and resume plug-in call execution after the upgrade.

I want to upgrade the plugin without ending the main process and replace the WASM file. For upper users, they are still the same plugin.

Can you elaborate on that please?

I want to upgrade the plugin without ending the main process and replace the WASM file. For upper users, they are still the same plugin.

Can you elaborate on that please?

I plan to plug-in some system functions and dynamically load them into the system. At this time, a single plug-in will be upgraded, but the main service will not be shut down.
Just like the Jenkins plug-in system, I can modularly update system functions online without restarting the service. For users, they have no perception. The only place where lags may occur is the blocking of requests during plug-in upgrades. New call requests and uninstalling old plug-ins, loading new plug-ins, and resuming blocked requests. If the operation goes smoothly, the user should not notice anything.

@fangzhengjin you should just be able to instantiate a new Extism Plugin and replace it with the old one as @nilslice suggested above. There shouldn't be a need to restart your service to do this.
Can you share a bit more about your architecture? (e.g., is this a multi-threaded app where different threads are sharing the Plugin object)

Closing due to inactivity


fix(tests): do not attempt to read empty ssh keys

How to categorize this PR?

/kind bug
/area os
/os garden-linux
What this PR does / why we need it:
With the changes introduced in PR #730, the ssh-key generator for the test framework will run into an error if the ssh-key file exists but is of size zero (which is the case for the platform tests). Therefore, extending the check if a keyfile is already present by checking on its file size too.
Release note:

fix(tests): do not attempt to read empty ssh keys

/invite @chrinorse

/ready

lgtm


Configuration support

Commandline options
Many commandline option strings have been changed. The base names, types etc. are centrally defined in ConfigurationBaseOptions.h (using X macros).
Configuration system
Introduced Configuration base class that encapsulates pass options.

Commandline configuration is based on llvm::cl::opt values.
FileConfiguration reads yaml file with option configuration.

These base classes are used in TypeARTConfiguration, which combines both (and prioritizes set) commandline options.
In addition, the compiler wrapper supports flags/env. variables to pass a file configuration to the pass.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

407 of 424 (95.99%)  changed or added relevant lines in 10 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.9%) to 90.719%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/passes/Commandline.cpp
119
120
99.17%

lib/passes/analysis/MemInstFinder.cpp
28
29
96.55%

lib/passes/TypeARTPass.cpp
51
53
96.23%

lib/support/Configuration.h
31
33
93.94%

lib/support/FileConfiguration.cpp
140
145
96.55%

lib/passes/TypeARTConfiguration.cpp
34
40
85.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.9%

Covered Lines:
3509

Relevant Lines:
3868

💛  - Coveralls


Add docs option to CMake

Add CMake option for docs target, so docs aren't added to the main lib target unless the option is set.

Before merging, must add BUILD_DOCS option for pages job in CI


Update path, change TestConfig to TKG

Signed-off-by: Renfei Wang<EMAIL_ADDRESS>

reviewer: @smlambert


Add batteryinfo capabilities to charger

healthd already has sysfs_batteryinfo caps and charger should use them them as well.
This is needed for accessing /sys/devices/platform/soc/75b9000.i2c/i2c-11/11-0025/power_supply/typec/type on tone for which I have specified sysfs_batteryinfo in https://github.com/sonyxperiadev/device-sony-tone/pull/150.
The wake_alarm capability may not be needed in normal operation, I have only seen it requested in conjunction with fpc crashes. Please discuss.

please remove wake_alarm from this commit

Updated per @jerpelea's request

the same shoukd be needed by all platforms


Add gRPC support for ibc-transfer/getEscrowAddress

Summary
From this issue #189  came the escrow-address cli function. It would be most useful for end users to have a grpc query route as well.
Problem Definition

As an end-user, I'd like to know what the escrow address is for a particular channel. Using cli isn't feasible when a web app wants to query via gRPC. I have a function that mimics how the escrow address is created, but that won't withstand future changes to the address creation.
Proposal
Add a gRPC route to get the escrow address for a channel, using the same inputs as the CLI.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

Add definition for gRPC escrow-address method with route:

rpc EscrowAddress(QueryEscrowAddressRequest) returns (QueryEscrowAddressResponse) {
  option (google.api.http).get = "/ibc/apps/transfer/v1/escrow_address/port/{port_id}/channel/{channel_id}";
}

with request/response messages:
message QueryEscrowAddressRequest {}

message QueryEscrowAddressResponse {
  string escrow_address = 1;
}

Implement method in transfer's grpc_query.go

This is now available in v3.1.0, v2.3.0 and v1.5.0.


Failed requests for URLs containing

Expected behaviour
No requests for URLs containing <no source>
Actual behaviour
Logs show failed requests to /s/css/census/themes/census/<no%20source>
Steps to reproduce the behaviour
Unknown, these errors are from the server logs

Also blocked by the WAF as a XSS pattern


Fix for Issue 176

Closes #176
Closes #146
Please let me know if there's any thoughts here. It is my understanding that $ParentVHD and ($Database, $LatestImage) are mutually exclusive. However, the codebase requires $Database to be set, so when using $ParentVHD it wont actually work (since there are no databases set). This fix basically looks up the database name if supplied a ParentVHD
I also added a new parameter, $ImageID, to be able to create a clone based on an image ID.
Also, I'm not sure what utility $LatestImage has. The code looks like this:
 foreach ($db in $Database) {

            if (-not $ParentVhd) {

                if ($LatestImage) {
                    $result = Get-DcnImage -Database $db | Sort-Object -Descending CreatedOn | Select-Object -First 1
                }

                # Check the results
                if ($null -eq $result) {
                    Stop-PSFFunction -Message "No image could be found for database $db" -Target $pdcSqlInstance -Continue
                }
                else {
                    $ParentVhd = $result.ImageLocation
                }
            }

$LatestImage seems to be required, otherwise $result is always unset. Is there a purpose for this? I propose it gets removed.

The thought about -LatestImage was to make it easier for the user to use the latest image and didn't have to look it up.
Removing it would probably not be a good idea

@sanderstad That makes sense, but it's not implemented in that way. There isn't actually a way to say "I want this image instead" besides giving it a $ParentVhd, but by doing so it makes the $LatestImage redundant
if (-not $ParentVhd) {
                if ($LatestImage) {

So if you provide a ParentVhd, it uses that regardless of what LatestImage is set to. If you don't use it VHD, then LatestImage MUST be set for $result to be populated. If $result isn't populated, then the function stops. There's no way in current master to be able to declare Database and select an image that isn't the latest one
This PR fixes that tho, by #1 allowing the script to work with ParentVhd and without a supplied Database value (as the example in the script shows), and #2 allowing you to set the image ID that you want to clone. I would propose, as another PR once this is merged, making either $ImageId or $LatestImage required, if $Database is set. I don't know how that would play with specifying multiple databases to clone (I would assume LatestImage would be required if specifying multiple databases, as there isn't an easy way to tie those databases with specific images).
Please let me know if I'm off base or not understanding the purpose behind something, but as it stands now (in master) there is no way of getting a specific image and LatestImage is simply redundant. That may change int he future tho


DNS Query Forward not appearing in UI

The opnsense_unbound_forward (Resource) correctly configures a query forward to an internal DNS server, but the record does not appear anywhere in the UI:
opnsense_unbound_forward.query: Creating...
opnsense_unbound_forward.query: Creation complete after 2s [id=5a8fc7f3-5040-482e-86e8-8cfa3ac3b274]

Hi there @tomjones1977, thanks for opening this issue!
Firstly, if you re-run terraform apply, does it want to recreate the resource? (i.e. did the provider correctly create the resource). If it does try to recreate it, let me know because there might be a bug with the mutex locking.
However, it's probably due to the fact that the provider creates forwards using Unbound's DNS over TLS section:

This is because the DoT section is a superset of the old Query Forwarding section. To create a regular DNS Query Forward:
// Query Forward
resource "opnsense_unbound_forward" "query" {
  domain = "example.lan"
  server_ip = "<IP_ADDRESS>"
  server_port = 53 # set this to 53 (which forwards as expected, or omit the parameter to default to 53)
}

I suspect you've already configured everything correctly, but just aren't looking under the /ui/unbound/dot section of the UI. This is probably a fault of the documentation and I will update it to make it more clear.


Fixed failure to preview rounds in Wagtail Admin

Closes #3559.
Updated the app's urls.py to include the url_patterns specified in the user app's url.py.

TBH, I believe this has something to do with the internals of the preview generation view. The view responsible for these forms is very custom. The view responsible for the previews might have changed internally b/w the wagtail version upgrades, causing this issue.
I don't fully understanding the inner working to two different urls. But you look at logs, the preview view should pick up the hypha.apply.urls and not hypha.urls.
That said, another way to resolve this could that once the public part/site is remove we have only only set of ROOT_URLCONF.
You might also be able to get away by hand-picking the urls and adding them the hypha.urls

Digging into more around the url conf issue. This is where the magic happens, and in my opinion we should get rid of this middleware or to fix this now update it.
https://github.com/HyphaApp/hypha/blob/0a29676e6dcd3a9484c0ba070c872243c66edadd/hypha/apply/middleware.py#L31-L45

@theskumar This was super helpful! Thanks a ton. Do you think it would be best to wait for the public site to be removed for these changes to be made? It sounds like there's a decent amount is reliant on that.

@wes-otf After discussing with @fourthletter and @frjo we believe it would best to wait till public is removed.
For now, you check is resolved get resolved if you update ROOT_URLCONF to hypha.apply.urls instead of hypha.urls.
I'm not sure what will be the side-effect is we switch out the default root url conf site-wide. If it doesn't break things, it might be wise to do the switch now as part of depreciating the public site.

@theskumar all sounds good! I updated the ROOT_URLCONF as suggested and that also seems to work pretty well. I did a bit of manual testing and didn't seem to see any issues.

Looks like the tests might need a fix after that though. I'll look into that now.

Pulling this thread a little more it looks like the main failures from this are coming from the newsletter functionality (or tests, at least) being reliant on the public URL side of things. Is the newsletter something that will be ripped out with the public site?

We can discuss it more in Monday's standup but I think there's bigger database implications for changing the ROOT_URLCONF that I haven't fully wrapped my head around yet. I originally thought it was just the tests that were failing (as functionality seemed unchanged) but after doing a full python3 manage.py reset_db I'm getting the same error that the CI is showing with:
django.db.utils.ProgrammingError: relation "django_content_type" does not exist
LINE 1: ..."."app_label", "django_content_type"."model" FROM "django_co...

Going to keep digging around

Closing this PR to start a new one to fix this issue and remove the public site aspect.


Optimize binary size in release builds

Change-type: patch
Signed-off-by: Zahari Petkov<EMAIL_ADDRESS>

@balena-ci I self-certify!


Revision of the french traduction

Orthography, grammar corrections, and some senseless traductions updated to better fit the french language.

oops just read that I sould not change the dummy

fixed

Thank you for the contributions!


Printing the "complete" agenda (regie book)

There should be a posibility to print all agenda items  inkl. internal/privat items, the estimated time and the comments, in a PDF. The layout style of the paper should make easy to distinguish between a "real" agenda item, and procedural instructions for the members of the board (ger. "Präsidium").

Please set milestone as you like.

[x] Agenda as PDF
[ ] est. Time in PDF


Make Debugger Public to allow disabling Debug print messages.

Hi, is it possible to make Debugger public in DebugSwift, so that I can disable printing messages into the console?

Yes, of course we can, I will resolve soon as possible :)

I have forked and created a PR to make Debugger public since we need the setting urgently. Feel free to review and merge if you think it is okay. PR link: https://github.com/DebugSwift/DebugSwift/pull/43


facets-overview crashed!

https://raw.githubusercontent.com/PAIR-code/facets/1.0.0/facets-dist/facets-jupyter.html
cant import this depends

it cant display the result because "https://raw.githubusercontent.com/PAIR-code/facets/1.0.0/facets-dist/facets-jupyter.html" cant open
i knew the jupyter extension ,but open html file is more convenient for me,
can you tell me how to import an online url like "https://raw.githubusercontent.com/PAIR-code/facets/1.0.0/facets-dist/facets-jupyter.html" rather than use  "href="/nbextensions/facets-dist/facets-jupyter.html" "

i push the file "facets-jupyter.html" on my github project and use the raw url but it doesnt work

What error do you see in the browser's developers console when you attempt to link import the github-hosted facets-jupyter.html from your own html page?

What error do you see in the browser's developers console when you attempt to link import the github-hosted facets-jupyter.html from your own html page?

i m in China ,it seems that the site https://raw.githubusercontent.com cant open, i add "<IP_ADDRESS>  raw.githubusercontent.com" in my pc hosts and the facets can show

What error do you see in the browser's developers console when you attempt to link import the github-hosted facets-jupyter.html from your own html page?

when I attempt to link import the github-hosted facets-jupyter.html from your own html page
it cant show anything

So if you copy that file to your machine or another local one and link it from there, does it work as expected?

So if you copy that file to your machine or another local one and link it from there, does it work as expected?

yes, i seems find the reason why it doesnt work, maybe my link needs gitlab account verified,
i plan to use tomcat to build a local web service and try, wait please, thank you!

So if you copy that file to your machine or another local one and link it from there, does it work as expected?

from origin 'null' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.

@jimbojw any advice for getting access to this file without CORS issues?

OK thanks, I think I understand the problem. To use the <facets-overview> tag in a page, the code that implements that tag has to load and run. The code is in the file called facets-jupyter.html. The problem is that loading data from different origins (different domains) is prohibited by the browser. There are good security reasons why this the case.
If a given server wants to allow its content to be accessible across domains, the server can supply an Access-Control-Allow-Origin HTTP header. The domain raw.githubusercontent.com does not set this header. I suspect that gitlab.vmic.xyz does not either.
If you control the server gitlab.vmic.xyz, you could possibly configure it to supply an Access-Control-Allow-Origin value that includes the host where you're trying to show <facets-overview>. Exactly how to set this header depends on what kind of software is doing the hosting. You mentioned Tomcat. I know there's a way to configure that. This page describes what the header needs to look like https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS#The_HTTP_response_headers
You could also host the facets-jupyter.html from the same domain as where your containing page runs. If the two files have the same origin, then you don't need to set up an access control origin header.

The domain raw.githubusercontent.com does not set this header.
thank you for your reply. you said "The domain raw.githubusercontent.com does not set this header.", however,

the link can work well without  CORS issues, why?

Yeah, it's a subtle distinction. If you open a link directly, your browser will have no trouble downloading and displaying the content. This is normal.
It's even possible to load code and images across domains in some cases. For example, your code fragment loads webcomponents-lite.js from the domain cdnjs.cloudflare.com in a <script> tag. Browsers allow script tags and <img> tags to load content from different domains.
But the problem is when you want load content from a different domain AND access that data programmatically in JavaScript. To load data dynamically, pages often use XMLHttpRequest. If the data is coming from the same domain, everything works fine. But if the data comes from another domain, by default the browser will reject the attempt.
This is a security precaution for your safety. It helps to prevent cross-site scripting (XSS) attacks. If XMLHttpRequest allowed arbitrary requests across domains, it would be trivial to steal visitors' credentials and other data from other sites they were logged into.
But rather than always preventing all cross-origin requests, the browser will allow such requests IF the server specifically allows those requests. The server signals that it's willing to serve to another domain by supplying the Access-Control-Allow-Origin HTTP header.

Yeah, it's a subtle distinction. If you open a link directly, your browser will have no trouble downloading and displaying the content. This is normal.
It's even possible to load code and images across domains in some cases. For example, your code fragment loads webcomponents-lite.js from the domain cdnjs.cloudflare.com in a <script> tag. Browsers allow script tags and <img> tags to load content from different domains.
But the problem is when you want load content from a different domain AND access that data programmatically in JavaScript. To load data dynamically, pages often use XMLHttpRequest. If the data is coming from the same domain, everything works fine. But if the data comes from another domain, by default the browser will reject the attempt.
This is a security precaution for your safety. It helps to prevent cross-site scripting (XSS) attacks. If XMLHttpRequest allowed arbitrary requests across domains, it would be trivial to steal visitors' credentials and other data from other sites they were logged into.
But rather than always preventing all cross-origin requests, the browser will allow such requests IF the server specifically allows those requests. The server signals that it's willing to serve to another domain by supplying the Access-Control-Allow-Origin HTTP header.

Can you provided an offline mode that i save the "facets-jupyter.html" in my PC folder path, for example "D:/facets/facets-dist/facets-jupyter.html" and add a line such as "
" in an offline HTML file, when i double click it, it can show the result.
I knew the facets support  jupyter extension , but open html file is more convenient for me, please, thank you!

You can save that file locally from our github repo on your own machine and point your jupyter notebook's code to load it from your local copy, or have your own html file point to that local version. But in that case you will need to write the html code to correctly load the visualization and set the data on it, much in the way that our homepage code does (https://pair-code.github.io/facets/ is the homepage, with the code to get it work being https://github.com/PAIR-code/facets/blob/gh-pages/index.html


[Bug] 'audit' reports critical/high vulnerabilities

Describe the bug
auditing project using this library gives two critical and four high vulnerabilities, see here for example:

Steps to reproduce the behavior

Install library
Run npm/yarn/pnpm audit

Expected behavior
No reported vulnerabilities
Environment

OS: macOS 13.3 22E252 arm64
Host: Mac14,2
Node: v16.19.0

Hi @CHE1RON
What version of the addon are you using? I wasn't able to reproduce this with the latest version, 1.0.6

Hey there @Integrayshaun, same as you (latest version, as far as I can see) 🤔 strange ..


include PACIFICA_2020 in late-model behavior

These models need to turn off gone_fast_yet bit at -3 to avoid LKAS faults.
This was likely an accidental omission from PR #1297
To avoid future similar mistakes, we could define a list of these cars in values.py instead of having the list in multiple places.

Merging in this fix, but a list in values.py sounds good.


:sparkles: Add link to npm on version badge

Hi @Errorname,
Thanks for this PR 💪
Could you apply the same logic on the default-no-html template ?
Also could you run this command at the root of the project to add you as contributor :
npx all-contributors add errorname code

Thanks !

Done! 😄


Consider to restrict DBAPI only for Presto engine

As @takuti mentioned at https://github.com/treasure-data/pytd/pull/30#issuecomment-514980178, connect(pytd.Client(database='sample_datasets', default_engine='hive')) might introduces some possibility of failure to use cursor. On this issue, we need to clarify what kind of issue will occur and should decide whether we have to limit using of Hive engine for DBAPI.

A bit more clarification - in fact, it's not "failure," but may introduce confusion for users.
Importantly, as our README states, we assume cursor.fetchone() fetches table records one-by-one; this is a more efficient option to retrieve large volume of data. However, when we call cursor with Hive, it internally uses td-client-python's "synthetic" cursor implementation, which actually fetches all records at once from job result.
Thus, I originally thought it might be better to restrict DBAPI only for Presto so that it always behaves as we expect from the DBAPI spec.

But now, I changed my mind because active Hive users may be confused if we stop providing DBAPI for Hive. Therefore, in order to keep the compatibility, adding a warning message for the point that I mentioned above should be sufficient for now: https://github.com/treasure-data/pytd/blob/b1176b37e7841c0278d3c7c14525c91ed18e62c4/pytd/query_engine.py#L197-L204


Is it slaveReadEnable not work When cluster deployed on DLedger mode?

When I set slaveReadEnable=true and isSuggestPullingFromSlave is true, I noticed consumer still pull messages from leader node. The variable whichBrokerWhenConsumeSlowly=1 seems not work?

Issues are generally used for bug reports and feature requests, so if you have any questions about     Apache  RocketMQ, it is recommended to discuss it on the mailing list first, the mailing list [address]    (https://rocketmq.apache.org/about/contact). I will close the Issue first, but please feel free to     reopen it if you have any other issues.

ok. I've found the solution


feat: cdn

This PR should be merged after the publish of v0.46.0.

If it is easy fix, you can commit into this branch directly


Working on option for PNG drawings - Autocrop Content

Was able to obtain bytes to determine the boundaries of the image - would like more testing for this feature
Able to crop using Linux command - temporary solution - probably should use node js library or try to crop using the bytes decoding and encoding that I have already been working on

TODO:

[ ] Differentiate between PNG and PDF xopp files
[ ] Create hotkey to create png drawing on obsidian - have setting to either auto open or not
[ ] When right click image on md file that has an associated xopp -> have option to open in xournal
[ ] Integrate autocropping features and ensure xournal exports as png after editing the xournal file

Did not finish due to render content feature replacing it


add Table Control

adding a table control with customization would be very helpful rather than creating a bunch of rows and columns

See https://github.com/flet-dev/flet/issues/136, looks like it is around the corner


Feedback for “Metadata”

https://langfuse.com/docs/tracing-features/metadata
How can I update this meta data after a trace?
I am using the openai js library.
When i try to update the metadata it seems to wipe the old metadata, as well as the system and user commands

When i try to update the metadata it seems to wipe the old metadata, as well as the system and user commands

This is not the expected behavior as metadata is merged with previous metadata of an object in Langfuse.
Option 1: Add metadata directly
const res = await observeOpenAI(new OpenAI(), {
  metadata: { someMetadataKey: "someValue" },
}).chat.completions.create({
  messages: [{ role: "system", content: "Tell me a story about a dog." }],
  model: "gpt-3.5-turbo",
  max_tokens: 300,
});

Option 2: Create trace and update metadata later
More on this here: https://langfuse.com/docs/integrations/openai/js/get-started#nested-traces
import Langfuse, { observeOpenAI } from "langfuse";

// Initialize SDKs
const langfuse = new Langfuse();
const openai = new OpenAI();

// Create trace and add params
const trace = langfuse.trace({ name: "my-trace" });

// Link openai generation to trace
const res = await observeOpenAI(new OpenAI(), {
  parent: trace,
}).chat.completions.create({
  messages: [{ role: "system", content: "Tell me a story about a dog." }],
  model: "gpt-3.5-turbo",
  max_tokens: 300,
});

// update metadata
trace.update({metadata: "hi there"});

moving to main repo to merge with your other issue which is the same as this one


[safety-rules] Implement standalone docker container

This does it all!

Config builder to support instantiating an appropriate backend config and libra node config
Docker container with scripts to start the service
SafetyRules client code to allow consensus to start without any safety rules
A little bit of cleanup to make building the dynamic-validator docker a little faster

I'm looking forward to the next adventure of getting this integrated into much more complex frameworks.
Note: this is built on top of this PR: https://github.com/libra/libra/pull/2459

Docker changes look fine, I'll let someone else review the Rust changes.

For transition period, can we still run safety-rules old fashion, without spinning up separate container?

For transition period, can we still run safety-rules old fashion, without spinning up separate container?

SafetyRules is highly configurable. We can go from one extreme (running internally with no real persistent history) to another (running in a separate container talking to a secure, persistent backend). So we can build that into existing systems when folks have appropriate cycles. We can also continue to test without it.

Thats great :)
For measuring perf impact of this I would suggest creating separate workspace when terraform or k8s configs are ready for separate safety rules service. And then run cti -W <workplace> --perf-run
cc @ankushagarwal

I'll do a test run for this on the k8s setup with 100 nodes

@bors-libra r=zekun000

:pushpin: Commit fbcfd0f has been approved by zekun000

:hourglass: Testing commit fbcfd0f3b6864d6b73574aedeffe21f1a227be6d with merge 5f7601e55a9a1cb3a2d0dffe66bb186dfc9d3684...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: zekun000
Pushing 5f7601e55a9a1cb3a2d0dffe66bb186dfc9d3684 to master...


README Typo

In the README, which was originally copied from your other repo, you forgot to change the name of the pod from VAProgressCircle to MMParallaxPresenter.
P.S.: Both controls are really nice. Great job. Thanks

XemaCobra,
Firstly, thank you for your kind words. I'm glad you like them. (:
Good catch on the typo. I'm working on a re-write per the issue submitted by kierancrown to convert the MMParallaxPresenter from a UIScrollView to a UITableView so I'm going to have to redo the README after that is done. I'll update the typo and close this issue once the new version is posted.


Password Strength Analyzer

This is probably a boondoggle of epic proportions.  There is such nerd rage around this.  BUT, screw it. It's all about defending against an attack. It's about password complexity and the statistical attack.

And here is a place to start: https://jsfiddle.net/r3qeh7vv/


Create Exporter component and story

ref: #83
All:

Date range:

@Fajfa is the last screenshot from the storybook or from compose?

Everything is on storybook. For compose implementation we will have to wait for API.
I can plug it into where it will be if you want.

All good, I just wanted to double-check because of duplicated dropdown icons (which I know are the issue on storybook).


fixed typos and punctuation

Identified and fixed a typo and issues with punctuation.

Thanks for the proofreading @bbezerra82 !


Use https://github.com/Atlantic18/DoctrineExtensions

Is it still possible to use the bundle with the latest version of https://github.com/Atlantic18/DoctrineExtensions and Symfony 2.7 ?
I'm trying to do so but seems like I'm missing the $translations property

Up ?


🐛 [firebase_analytics_web] screen_view/page_view event is not being shown in Firebase console

Referenced issue: https://github.com/firebase/flutterfire/issues/4910
When running the Flutter web app and enabling the "GA Debug", Screen View events are being shown in the browser console. But they are not being shown in the Firebase console.
When the web page is refreshed or opened then only the page_view event is shown in the Firebase console. After that, if you navigate to a new Route via Flutter code then only the browser console shows the screen events.
Browser console screenshot

Analytics DebugView

Except for this screen_view event, the rest of the events are working fine.

Closing this as duplicate of https://github.com/firebase/flutterfire/issues/8403


A problem occurred evaluating project ':app'. local.properties (No such file or directory)

Steps to reproduce
git clone git@github.com:commons-app/apps-android-commons.git
cd apps-android-commons
./gradlew installProdDebug

Error
Expected: Project builds.
Actually: Error below:
FAILURE: Build failed with an exception.

* Where:
Build file '/[...]/app/build.gradle' line: 350

* What went wrong:
A problem occurred evaluating project ':app'.
> /[...]/local.properties (No such file or directory)

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.9/userguide/command_line_interface.html#sec:command_line_warnings

BUILD FAILED in 1s

Workaround
touch local.properties

then build again. This time it works.

Worked for me even without the touch command. 😕

Interesting! Might depend on the environment.
No need to worry about it, I mostly put it here for the workaround in case someone encounters the same issue. 🙂

That file is automatically included in the project when opened in Android Studio. However, as you said we need to manually add this file when directly cloning and building from the terminal.

I faced a similar issue after cloning & the workaround helped. Thanks!


remove some gems that do not make sense for mondo, and add some that do

Thoughts?
@mrjman @jmyers0022 @mhfen @justinhillsjohnson

Cool. Just need some builder methods for the new gems :)

I'm curious as to whether or not it's worth it to maintain this to keep up with updates to rails and the other gems though.

@jmyers0022: good call on the version numbers.  It's a boilerplate, so it would make a lot more sense to make sure it's running the most recent version of all gems.

I think this can be fairly opinionated with gems with possibly some ways to skip portions through options. But also most of these are simple enough to just remove the gem once the project is setup if you dont need it.


Chat - User B can not put expense on hold when it has "Duplicate" status

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

Version Number: 9.0.7-8
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/45325
Action Performed:
Pre-requisite: as User B, create a Workspace and invite user A

As user A, submit 3 expenses to the Workspace with the same amount and merchant, so it appear as duplicate
As user B, go to the workspace chat and select one of the duplicate expenses
Click on the header

Expected Result:
Hold option should be available when the expense has "Duplicate" status
Actual Result:
Hold option is not shown until the user first resolve the duplicates
Workaround:
Unknown
Platforms:

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

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/eb8df01c-7589-479c-90f8-c009ab395dd1
View all open jobs on GitHub

@jliexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #wave-collect - Release 1

Hi @pecanoro @DylanDylann - it looks like this might be related to https://github.com/Expensify/App/pull/45325, if you could have a look?

From https://github.com/Expensify/App/issues/44892, if it's duplicate, we don't want to show the Unhold option. (Unhold option is previously shown for duplicate expense)

If the expense is a duplicate, it is already on HOLD so there is no point in holding it again. I think this is working as expected.


Justin reynolds mm patch 1

Added Performance Alerting Guide.

@lfbrock For screenshot review.

I missed that - @JustinReynolds-MM can you help re-upload the images via docs -> source -> images. You can ping me for any questions

@JustinReynolds-MM Can you also help remove the images not in the images folder?

I'll try and see if I can get less blurry screenshots for the first few in the doc.

Thanks @jasonblais - they look better now, a couple still look mildly blurry but I think it's just the font rendering of some of the fonts on Windows.


Safepoints

Add safepoint generation to the compiler and rework VM signal handers to be resumable in order to lay groundwork for a sampling profiler and application-level signal handling.

Rebased and merged.


Unable to bower install headroom.js

Attempting to bower install headroom.js gives the following error:
bower ENOTFOUND     Package headroom.js not found

See #214 & #215. Instructions in README https://github.com/WickyNilliams/headroom.js#install-with-bower

Sorry, thanks!


feat: add GET command (cutekit's simple package manager)

How to use it ?
It's easy as
mumu@cutekit:~$ cutekit G flags-gen

It uses a new manifest format install.json example.
it also supports partial cloning (like we do for the init command)

Wait, it's weird that the manifest file format doesn't include the plugin version.
Nor does it include the plugin 'name' ?   :thinking:

Wait, it's weird that the manifest file format doesn't include the plugin version. Nor does it include the plugin 'name' ? 🤔

Yeah we might need a version number, for the name however it's not needed. I don't see any use cases that justify the usage of a presence of the name in the manifest (correct me if I'm wrong)

Closing this one, it's too fare behind and doesn't use the new way we do externs


ExecuteModule method of a Source Text Module Record should call DisposeResources

To ensure we correctly dispose of resources introduced via using and await using at the top of a module, we must call DisposeResources() at the end of the ExecuteModule method of a Source Text Module Record.

My bad, this already exists and was only missing from my reference implementation in engine262.


[Roadmap] Frontend UI framework

Need to choose a frontend UI framework to be used.
Some examples are:

Bootstrap
Bulma
Materialize
Custom framework using things like Flexbox. See comment below

Since this is a desktop app perhaps material design might be nice to use. I would go for materialise

I think this will ultimately be a less important decision and based more around whatever UI design is mocked up.

From a designing standpoint, Bootstrap is the most widely used but from reading about them. Bulma and Materalize seem to fit the bill a little better for ATL.

Another option is not using a UI framework and instead building our own simplified one.
I've discovered the joys of Flexbox and think it may be ideal in order to keep thing lightweight and minimal

It would probably be more work, but there would be more control over the design

I agree with @jamierocks. Flexbox might be the way to go

I do like Bulma a lot, think about releasing react bindings for it if you chose it.

Styleguide approach is preferred for more control, but I don't see it happening unfortunately with the current resources, so using an existing framework may be the way to go.

Will be using our own styleguide with a react implementation of the components available for ease of use


bug in at_least_one test code logic

Describe the bug
From v1.1.0 to v1.1.1 there was a change to at_least_one test´s code.
I think this change actually broke the way the test should work. The bug is in the logic for pruned_rows CTE.
Steps to reproduce
Let´s look at a simple table, say demo_table:
| col_1 | col_2 |
+-------+-------+
| A     | 10    |
+-------+-------+
| B     | null  |
+-------+-------+

If we were to configure the at_least_one test for this table, using group_by_columns parameter,  we would have the following:
select_gb_cols = 'col_1'
groupby_gb_cols  = 'group by col_1'
select_pruned_col = 'col_1, col_2'
Then, in pruned_rows CTE:
select
  col_1, col_2
from demo_table
where col_2 is not null
limit 1

The output of this CTE would be:
| col_1 | col_2 |
+-------+-------+
| A     | 10    |
+-------+-------+

Continuing following the test's code, the following piece of logic would now be applied:
select
   col_1,
   count(col_2) as filler_column
from pruned_rows
group by col_1
having count(col_2) = 0

The output would be empty, as our only record in pruned_rows does have a non-null value for col_2.
Expected results
I would expect this test to return the record with col_1=B, since for this value there are no non-null values in col_2.
Actual results
Explained above.
System information
dbt-labs/dbt_utils version 1.1.1
dbt version 1.6.3
bigquery plugin version 1.6.4
Which database are you using dbt with?

[ ] postgres
[ ] redshift
[x] bigquery
[ ] snowflake
[ ] other (specify: ____________)

Are you interested in contributing the fix?
Can definitely provide more context if needed, but not interested in developing the code.

We are also experiencing this issue with the at_least tests, preventing us from upgrading to 1.1.1.

Still a valid issue.


[KYUUBI #5122][DOC] Hive KDF usage

Why are the changes needed?

close #5122
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Thanks, merged to master.


improve logging

WHAT is this pull request doing?
It strip the protocol string from newlines, so we get cleaner logs if users send in bad input.
HOW can this pull request be tested?
send in a header with a bad protocol, with newlines, and see that the log is not devided.

This should probably be done in all places where we log user input, but i'll put it up like this so we can discuss.


Add example blueprint for CSI snapshot functions

Change Overview
This PR consists of Blueprint example and usage of CSI Snapshot Functions to backup/restore MySQL DB
Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[ ] :bug: Bugfix
[ ] :sunflower: Feature
[x] :world_map: Documentation
[ ] :robot: Test

Issues

#1126

Test Plan

[ ] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

As discussed offline, let's change this to use a different application like timelogger.

@pavannd1 This PR is ready to be reviewed. Please have a look.


[0010] Feature availability check

Which proposal does this relate to?
0010-vk-buffer-ref.md
Describe the issue or outstanding question.
Since this is a retroactive feature, we should document and describe a feature availability check using the clang __has_feature infrastructure in DXC. We haven't historically used this for features, but it is much nicer than forcing users to key off specific compiler versions and I've locally tested that it is fully functional in DXC.
Additional context
See initial review discussion.

I propose the following syntax: _has_feature(hlsl_vk_buffer_pointer)


Feature request: icon param for notifications

Outside of the file or attachment parameters, it would be nice to have an icon parameter. This would show up on the android client to the side as a quick visual reference to the alert. I also have another use case where I would use it with windows 10 toast notifications.
Thanks!

@poblabs Would you want something like this:
curl -d hi -H "Icon: https://example.com/myicon.png" ntfy.sh/mytopic

Or (with myicon being configured in the app):
curl -d hi -H "Icon: myicon" ntfy.sh/mytopic

Also, would #155 be enough?

@binwiederhier I think per-notification icon would be a nice touch. That way I only have to sub to 1 topic, but use it for multiple apps. The icon would identify the app within the sub. I suppose I'm not picky though as I could make a new topic for each app - just more overhead to remember it all

Alright! The code is just about done!

Done, next release


Fix: windowType not checked in InAppBrowser show function macOS

Connection with issue(s)
Resolve issue #1602

That check isn't necessary, as it should always fall back to the CHILD window type.
I tested TABBED window type, and it works as expected, so I don't understand what is the problem here.

I personally execute the show() function tied to a button, making it unlikely that I am invoking it early. In my current scenario, I maintain multiple tabs. If the configuration is set for tabbed windows, it should logically prevent the possibility of entering the logic for a child window opening. The suggested modification should effectively address this issue.


TypeScript Scheme

Is there any future plan to include some TypeScript scheme for Notepad2-mod?

It's not available upstream, and it doesn't look like they're interested in implementing it themselves.
http://sourceforge.net/p/scintilla/feature-requests/958/
You could try submitting a patch upstream.

Oh well. Dunno how to make schemes. At least they're already aware of that. Thx!


fix: marshal of user info address

:tada: This PR is included in version 0.88.1 :tada:
The release is available on:

GitHub release
v0.88.1

Your semantic-release bot :package::rocket:


Wrong table is choosen when editing table in oracle

If I try to edit a table by pressing F4, if the table I'm trying to edit exists in another schema, then it happens that I edit the one not on my schema.
Example:

I want to edit the table BTARTICLES
I'm on MY_SCHEMA_DEV
I write SELECT * FROM BTARTICLES and I select the table name and click F4
I end up editing the table BTARTICLES of schema PRODUCTX_DEV

If I right-click the table in the tree at the left side everything works correctly

It is a bug.
Object chooser (F4) uses the same algorithm as hyperlink detector and selects the first object in the hyperlink list (to see the list press CTRL and hover over identifier).
Tables in the active schema must be in the beginning.
Thanks for report.

 I use dbeaver often at work, I'm glad to participate and make it better


Comment a script tag in html

Does this bug fixed https://github.com/Microsoft/vscode/issues/11081?
How can i configure?  i have no idea

Sorry, it's unclear what the bug is. Can you add an example and the steps to reproduce?

i use cmd+/ to comment the code,  it works perfect in html tag, but don't works in inline <script> tag

I can't reproduce

Is the editor in HTML mode (see lower right corner)
If you can send me your sample, that would be great.


Please style the Listing Data Changed  pop in message.

When on your store and listing data changes, a pop in message appears urging you to refresh. The message is not super visible right now and is easily missed.
It's currently set with fixed positioning. The fixed positioning is important because the user may be scrolled very far down the store when we need to show the message, so having it be fixed will allow it to be at the top of their view no matter what scroll position they're at. Hopefully, the fixed positioning can stay. All else though is probably fair game.

Will do


Update template/release versions in all examples in the docs

A lot of examples in the docs have old/inconsistent versions of templates and 2a release. Most have 0.0.3 which is the previous one

Stalled and was already done. Closing


When building the compile GraphQL step seems to run multiple times?

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
yarn
If applicable, what version of Node.js are you using?
No response
Amplify CLI Version
v10.0.0
What operating system are you using?
Windows
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
Nope
Amplify Categories
api
Amplify Commands
codegen, push
Describe the bug
Not sure this is a bug exactly but noticed something off in the logs of our CI and when building locally.
Seems the same warning about our authentication in our schema is written out multiple times which points to the chance that its compiling it multiple times.
The warning text is:
Be careful when using @auth directives on a field in a root type. @auth directives on field definitions use the source object to perform authorization logic and the source will be an empty object for fields on root types. Static group authorization should perform as expected.
Might not be a biggie but it can take a while (15-20 seconds or so) for it to compile on my machine (with fans screaming) and it renders this message out twice. However if its rendering the double message out multiple times that points that it might be rerunning it? That could mean that compile time could be reduced both locally and on the CI so could be a dev cycle performance win. It could also mean that multiple different scripts are running the same test "just to be sure" and causing the duplicate warnings.
The error message itself is just a warning, that is not what this issue is about, its the fact that it appears twice and then multiple times per build.
Locally if I run amplify api gql-compile I get two copies of the warning written out, again this doesn't worry me so much here but might be worth a look:

CI Log output shows double message again but it multiple written out times (3 in total in this run):

Expected behavior
One warn me about this issue once rather than multiple times.
Reproduction steps
We are still running the V1 of the schema transformers so I believe you will need an older style schema with the older @auth directive on it, something like so:
  @model
  @auth(
    rules: [
      { allow: groups, groups: ["admin"] }
      { allow: public, provider: apiKey }
    ]
  ) {
id: ID
name: String
}```

Then try running `amplify api gql-compile` and you should get the double warning. If you push it up to CI you should then get it running multiple times (but that could obvs just me on our setup, as usual happy to collaborate).

### GraphQL schema(s)

# Not the full schema but should give you something to play with, ask if you need the full one.
type Invoice
  @model
  @auth(
    rules: [
      { allow: groups, groups: ["admin"] }
      { allow: public, provider: apiKey }
    ]
  )
  @key(
    name: "byCustomerID"
    fields: ["customerID"]
    queryField: "invoicesByCustomerID"
  )
  @key(
    name: "byXeroInvoiceID"
    fields: ["xeroGuidID"]
    queryField: "invoicesByXeroInvoiceID"
  )
  @key(
    name: "byStatusAndDueDate"
    fields: ["status", "dueDate"]
    queryField: "invoicesByStatusAndDueDate"
  ) {
  id: ID!
  addressID: ID
  status: InvoiceStatus
  paidDate: AWSDateTime
  sentDate: AWSDateTime
  dueDate: AWSDateTime
  visitID: ID
  xeroDisplayID: String
  xeroGuidID: String
  customerID: ID
}

### Project Identifier

tried it but it failed to upload the zip:

<img width="666" alt="image" src="https://user-images.githubusercontent.com/1674590/188877730-214cdf43-a2d3-432f-93ed-7aefddc144c2.png">

### Log output

<details>

Put your logs below this line

</details>

### Additional information

_No response_

Hey @PeteDuncanson :wave: thanks for raising this! I am familiar with these prompts and will mark this as a bug pending feedback from the team. I believe the GraphQL schema compilation also occurs 4 times during the push process: twice before the status prompt and twice after, which can lead to additional redundant warnings


Feature: Environment filters for an app

🔖 Feature description
User can create filters for and environment for group of apps.
🎤 Pitch
allow users to create groups, user can only see the apps mapped in environment grouping, it can be modified by super admin any time.
✌ Solution
allow environment grouping handling.
🔄 Alternative
No response
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

AB#5859

duplicate https://github.com/devtron-labs/devtron/issues/3992


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/90003

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/90542

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/90550

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/90608

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/94193

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/94400

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/94610


socialLinks.sanitize misformats valid Youtube urls (inserts @ in the url)

Hey all, thanks much for this great utility.
In using socialLinks.sanitize for our implementation:

Given a url of https://www.youtube.com/channel/UCBTKd-zHSdr8POH-Y7LyPWg
socialLinks.sanitize will format the url as https://www.youtube.com/channel/@UCBTKd-zHSdr8POH-Y7LyPWg
This behavior can be replicated in the demo site: https://gkucmierz.github.io/social-links-app/

Demo site preview

This was ultimately not a dealbreaker for us, we were able to use an alternative strategy in our particular implementation. However I wanted to raise this with y'all for visibility.
Appreciate the efforts, all the best~

You are trying to sanitize channel link.
I think channels are different things than profiles. And these two things should be served separately.
Right now there is no profile for channels. If you need it you can do PR with profile 'youtubeChannels'.
I will merge it.


Npml NPM package has "src" production dependency

Looks like Simple Redis Cache isn't a runtime dependency of Numl. If so, it would be better to move under devDependencies
See https://github.com/tenphi/numl/blob/master/package.json#L17

Thanks! I don't know how it got there. I will remove it soon.

Remove in v0.11.27.


Continious CI is required

Continious CI is required to test booting

rootfs on LVM on DASD
rootfs on multipath
rootfs on zfcp / scsi
rootfs on qemu-kvm virtio scsi

It seems like -f flag got dropped, without fixing zipl helper. Causing inability to boot since at at least v1.39.

I agree that CI tests would be definitely nice to have for all s390-tools. But it is a major effort with having all tools maintainers involved, choosing the right framework and somehow providing the required hardware infrastructure, e.g. DASDs or FCP attached SCSI devices.
At the moment I don't see how this is going to happen.

@michael-holzheu
The LinuxOne Community Cloud is maybe able to offer the resources you're looking for:

https://developer.ibm.com/linuxone/
https://linuxone20.cloud.marist.edu/cloud/#/login

I haven't yet used it personally though.


Java 8 time API support

Currently, Phantom uses Joda time. Would be nice to have java.time API support as an additional module targeting Java 8, and also make Joda time deps optional.
@alexflav23, what do you think?

Hi @Tvaroh ,
This will likely be a lot of boilerplate for no reason and since not everyone is on Java 8 yet this is sightly dangerous. You can add a trivial conversion between Joda and Java dates and I highly doubt there would be a noticeable performance penalty in doing so.
It doesn't sound like this will be worth the trouble just yet.
Regards.

I see, thanks.

joda.time could be set as deprecated and java.time support added only for java 1.8 users. It's a good change for a 2.0 version.

From a few versions there is a phantom-jdk8 package with java.time columns.

Thanks


Inconsistent behavior with invalid semver

You can package and index a chart with an invalid semver, yet the index will then claim it doesn't include that chart:
package main

import (
	"fmt"

	"k8s.io/helm/pkg/chartutil"
	"k8s.io/helm/pkg/proto/hapi/chart"
	"k8s.io/helm/pkg/repo"
)

func main() {
	c := &chart.Chart{
		Metadata: &chart.Metadata{
			Name:    "foo",
			Version: "<IP_ADDRESS>", // invalid semver
		},
	}

	_, err := chartutil.Save(c, ".")
	if err != nil {
		panic(err)
	}

	idx, err := repo.IndexDirectory(".", "http://example.com")
	if err != nil {
		panic(err)
	}

	for _, versions := range idx.Entries {
		for _, ver := range versions {
			fmt.Println(ver.Name, ver.Version) // prints foochart <IP_ADDRESS>
		}
	}

	fmt.Println(idx.Has("foochart", "<IP_ADDRESS>")) // prints false
}

I think one of the following should be true:

index.Has can handle invalid semvers
helm repo index omits invalid semvers
helm package fails if the version is invalid

I realize that helm lint will call this out, but that doesn't prevent people from creating charts that will behave inconsistently. If the index functions won't work properly with such a chart, then it shouldn't be allowed to enter the index in the first place.
Library version: v2.8.0-rc.1-285-gd32a6ef6

/remove-lifecycle stale

@rcorre do you feel like working on a fix for this?

I'll take a look. Any preference between the 3 options I presented above?

closing as inactive.