Few questions about this repo by 🇷🇺

I'm against the war, but I'm just a civilian, working on gaming related projects, so seeing this repo made me curious about following questions:

What makes provided sources trusted and how they gather and validate information in the era of fakes and day-to-day misinformation even in peaceful time?
What is your motivation for encouraging people to stop using Russian software?
Do you plan to provide the full list of projects, technologies and products, created or maintained by Russian people?
Do you suggest everyone else to stop using nginx, postgresql, google, etc. because Russian leaders decided to start all this?
How do you distinct average Russian people and those, responsible for current situation?
Do you also suggest to stop any interaction with any Russian living in different country?

Hi. @AdamBrianBright thanks for reaching me out.
Many of your questions are related to my personal point of view.
Regarding my personal opinion.

I'm not forcing anyone to take actions based on information in this repo. I will however, do my best to take into account and issues, PRs and comments. Russian input included.
Kharkiv, Kyiv, Bucha, Irpin (my native town), Sumy, Kherson, ... Please google.
Yes, I will do my best to provide such lists
Your army and government is your moral responsibility.
Your army and government is your moral responsibility.
Absolutely not.

@AdamBrianBright please point to the specific questions that were not answered. I'll try to elaborate

I was curious about trustee of sources, how can we know that everything there is true?
4, 5. These questions were answered as it's all our* fault regardless. As a citizen of Russia it is at least not an answer for me.

 Thank you

Regarding point n.6, I'll try to update the README.md to express this.

Sorry, I don't see any hate speech here. The discussion is productive, I think.

Long story short, Yes. Unless Russia stops invasion, withdraws troops.
Russian people did allow this. They do fund the Russian Army. They did elect Putin, didn't they?

Regarding question n.1, I'm not sure how to answer.

Regarding war in general: My own home town is burning. I did see Russian Army invading Ukraine with my naked eyes.
Regarding sources: as you can see there is just a few of them. I selected them because none of them is backed by any governments, funds, tycoons, etc.

@AdamBrianBright I hope all questions were answered to an acceptable level.

Understandable, I'm sorry to hear that.
Understandable, thank you for the answer.
We can discuss about the following to 5th point questions a lot, but imho:

We didn't allow it, we were never asked. We don't fund army, we pay taxes (because it's law, and we don't want to be in jail), which goes to federal fund, which gains money from different sources (mostly gas/oil). I don't know if we elected Putin, it's a mystery for me. And we didn't ask to be born and raised in Russia. Especially we didn't go to work with thoughts like "I wish my tax money will fund invasion". So... it's complicated I guess? And there are a lot of people protesting and being arrested for it, so it doesn't seem like everyone want war. And I don't believe* like loosing a bit of money from taxes will run those money-packed pockets dry.

This is my residential area. Many civilians have died here.

Now, I'm not calling for annihilating of Russian people. I just want them to stop paying for this war.
As long as dozens of civilians die in Ukraine from Russian missiles, your arguments do not hold water.
Please read the history on nazi and post-nazi Germany.

I understand the tragedy and hope it will end, but this conversation seems went to emotions, so I would call it an end.

Reopening for further reference. Discussion closed

But please explain to this video，fucking Nazi！
https://video.weibo.com/show?fid=1034:4742655347982440&mark_id=999_reallog_mark_ad%253A999%257CWeiboADNatural
I'm also against the war, but I can't stand the Nazi, so I'm for Russia, and please say
"u are against the wat and death" to the citizen that killed by Ukraine army

when i share the video in twitter, i was forbidden
Is this freedom of speech?
Is this the Ukrainian government you support?

@Py-debug please stop posting russian fascist fakes. Thanks in advance.

Thank you for showing me the so-called freedom of speech,
If you think what I posted is false, please show evidence
Your deletion will only make me more convinced that you are hiding something @webknjaz

@Py-debug you have enough evidence in this thread. And in Google search bar.
Sorry, this is not discussable. Should I sponsor you a visit to Ukraine?

Ok, Should I sponsor you a visit to Odessa?Donetsk?Lugantsk?
If I doubt something, do I have to do it on the ground?
You at least have internet, electricity here, and those tens of thousands of victims in Donetsk, Lugentsk, Odessa, who fell under the guns of their own government, fell under the guns of the Nazis, and are still forbidden to tell what happened to them.
you means, What is against you is not discussable?
Grateful to github, at least I can say these things.

The Odessa tragedy, also known as the Odessa bloodbath, which left 46 people dead and 214 injured, was a massacre of peaceful demonstrators protesting against the Nazis, supported, planned and led by the right-wing Ukrainian neo-Nazis, who forced peaceful elderly people, pregnant women and children into buildings and set them on fire, while those who tried to escape were strangled to death. The Ukrainian government informed that it was a fabrication of rumors and shifted the responsibility to the victims.

Russia's Investigative Committee Cloud on Wednesday launched a criminal case after a spate of civilian mass graves were found in the Donbass region of eastern Ukraine. The agency issued a statement saying, "A criminal case was opened as a result of the discovery of civilian mass graves in the Donbass region. It is based on Part 1 of Article 356 of the Russian Criminal Code - mistreatment of civilians, use of means and methods prohibited by international treaties in the context of armed conflict." The statement said, "In 2014, the bodies of at least 295 civilians who died as a result of indiscriminate shelling by the Ukrainian armed forces were exhumed. It has been learned that among the remains there were female bodies of different ages."
The young hero in the picture is named Vadim Papula, a native of Odessa, born on July 24, 1996, today is his birthday. He is honest, kind and tolerant. At the age of 16, he joined the Ukrainian Komsomol and became a delegate to the Congress of the Komsomol in Kiev in the same year. The situation in Odessa deteriorated as a result of the Russian-Ukrainian crisis, and Ukrainian nationalists planned to occupy Odessa regional government institutions. Starting with pro-Russian actions, he was involved in a number of activities such as opposition to violence in Kiev. on May 2, 2014, Vadim participated as a member of the Odessa squad in a clash with Ukrainian nationalists on the Odessa factory peyecKon square. When news came that the Ukrainian nationalists were advancing in the direction of their camp, Vadim and his comrades were forced to retreat to the trade union building. His mother remembers the moment when Vadim left the house - he dressed quickly and left, perhaps having received a phone call or seen a message on social networks. The last time his grandmother saw him, she asked him, "Where are you going?" To which he replied: "I'm going to protect you". Together with his comrades, he was forced to hide in the trade union building, among a crowd of fanatical nationalists. After jumping out of a burning building during a fire in the Odessa trade union building, in which he hid the red flag on his chest, he was caught, brutally mutilated and beaten to death by the nationalists.In October 2014, young Marxists in Lipetsk, Russia, announced a fundraising effort for a monument to Vadim in Odessa. Vadim Papula also became one of the central images in French journalist Paul Moreira's documentary "Ukraine, the Mask of Revolution.

If you ban me, it will be my honor

@AdamBrianBright It's definitely a bad time to be Russian, but it's a terrible time to be Ukrainian.
From one day to the other they lost everything that you give for granted, like having peace, home, food, family and life.
Honestly I think everyone can and should help, and I think Russian people are the first ones that can make the difference.
If Ukrainian people are willing to die for protecting their country and their rights, I really beg that Russian people go all outside and start protesting in masses for this nonsense war. Nothing is more powerful then people and it's simply impossible for the government to arrest everyone.
Please fight for what you believe, fight for letting know the truth, fight for having a fair country with a true democracy, fight for the human rights!
Everyone... together we can make the difference 💪

Note: You won't be able to guess my political views, so don't even try :D.
I decided to investigate the claims made by @Py-debug and it seems there is plenty of information about it. Those times of the revolution in Ukraine were certainly bloody indeed, especially what happened in Odessa. I will let others investigate for themselves but I would say not to take any evidence on its own, but in the whole context of what was occurring in Ukraine at that time.
Who is in power in Ukraine, what the laws are in Ukraine, these are problems for Ukraine and no one else. That seems fairly self evident.
There is censorship of vital information in Russia right now, preventing the population from being able to investigate and understand for themselves. As I understand it, this repository is an effort to help inform the Russian population that this is happening.
How to get accurate information, fact check etc? Sources, many of them. Take into context whether they have a reason to mislead, are there any examples of them misleading in the past (evidence, not rumour or conjecture)? Look to overseas sources from those unconnected to either side in this conflict, what are they reporting to their people?
But when Rupert Murdoch's global news empire, the BBC and Al Jazeera are all saying the same thing, that's a fairly strong indication it may be true.
And if you want to know how the people are responsible for their government and what they can do, I suggest reading up on Ukraine 2014.

@internalsystemerror fair enough.

@internalsystemerror
I'm sorry, I was watching videos that were too gory to videos that there could still be massacres today.
Actually, I am not originally from Ukraine and have no right to say anything. It's just that I think the BBC and CNN videos are full of bias and falsehoods, and similar content would be deleted immediately on twitter. The control over the news itself is part of the war. The superior side will win the opinion war with huge amount of information, but the truth often does not shift because of superiority.
I decided it was better not to do anything, and everything will come out after the war is over.
Sorry for the previous presumptuous behavior.

@internalsystemerror
If you are interested in this history, I am very happy to be able to share with you all the videos I have saved, including the ones that are scorned in mainstream media such as twitter and facebook. False evidence is always false, and I am sure you will be able to tell with your own ability.

@Py-debug Interesting that twitter and facebook are considered mainstream media :D. Given anyone can post anything on these platforms I consider social media the most untrustworthy and most worthy of scrutiny. You don't know the source; you don't know whether it really is where and when it suggests; I'm not a military expert able to spot a vehicle and know exactly what it is, what its for, who's it is, whether its stolen etc etc.
I would say more, but I don't want to derail this from the topic of the restriction of accurate information in Russia and by Russian news sources which is the purpose of this repository. The assumption being, and one that I share, that if the Russian people were given all the facts, then things may change (to put it politely).

Just FYI, I see that JetBrains is in the list of Russian products, but it officially condemned the war in Ukraine.

@AdamBrianBright To any russian citizen that is "againstthewar" and "werenotallthesame" I say this - you retaining russian passport and citizenship means your consent to and shared responsibility for actions of your elected government. So unless you're actively prohibiting your govt from committing unlawful acts or unless you're denouncing russian citizenship - you are an accomplice to all of the war crimes and are equally responsible for violence, destruction, and suffering caused by russian govt anywhere in the world.
Anticipating your exclamations - this situation is binary and there's no grey area for you to hide in.

@AdamBrianBright To any russian citizen that is "againstthewar" and "werenotallthesame" I say this - you retaining russian passport and citizenship means your consent to and shared responsibility for actions of your elected government. So unless you're actively prohibiting your govt from committing unlawful acts or unless you're denouncing russian citizenship - you are an accomplice to all of the war crimes and are equally responsible for violence, destruction, and suffering caused by russian govt anywhere in the world. Anticipating your exclamations - this situation is binary and there's no grey area for you to hide in.

Let's use that same logic with America and Israel.

@internalsystemerror Because I suddenly think there is no point in playing to the bull, I live in a happy and stable country and I will always support my country. So I don't want to meddle in this matter anymore, just wait for the final result. I believe the Nazi war mongers will be judged.

@kstupak I agree, given all the facts this is one of the most obvious examples of right and wrong that I've seen in my life time. But based on the above results, I'm not sure if they're purposely trying to spread disinformation, or just refuse to investigate the facts for themselves. Either way, I fear for humanity.

Sorry, I had to clean up the thread. With @Py-debug and @AdamBrianBright  removing their comments randomly, it was really hard to follow. I need to restrict it again. Hope for understanding.


internal use of stings as keys is buggy

Internally we wrap keys as strings. This takes a copy of the array that is passed. Unfortunately not all byte values in an arbitrary byte array are mapped to unique characters of a string.

fixed in 1.0.0-RC3


🧪 Create invalid sign up and login tests

Describe the screen, component, or feature that needs to be tested
Logging in with no info, invalid info. Same for sign up.
Additional context
N/A

Closed by #236


Add WebTorrent Desktop

Stream any user provided torrent file or magnet link sequentially (Audio, Video)
https://github.com/webtorrent/webtorrent-desktop
Deb Available here
https://github.com/webtorrent/webtorrent-desktop/releases

There seem to be upstream issues of viability and direction that preclude including this at this time.
https://github.com/webtorrent/webtorrent/issues/2609
https://github.com/webtorrent/webtorrent/issues/2498
https://github.com/webtorrent/webtorrent-desktop/issues/2309


digikam: add video support through libqtav

Motivation for this change
Issue #24426
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@the-kenny

Sorry I was on vacation around easter. LGTM, build is running right now on my machine. Will merge if it succeeds & works.

thanks


[To discuss] Network - Prevent duplicate projects/organizations

In FO, the fact that duplicates are not detected is troublesome. Is it possible to suggest existing results based on X similar characters?

JIRA #ORC-501


Find fails on YAML sequences

Some pandas args take lists (which become sequences in YAML). build.py:find did not work for this case; it would descend the children of the sequence and then blow up because, e.g. in test case below, the children (ints) are not iterable.
subgroup:
   # transform and skiprows apply to all children, unless overidden
   transform: tsv
   skiprows: [0]

will rebase and resub


Used some of your code

Hello,
I've used some of your code in the following library: https://github.com/rjgtorres/oo_hdf
Thank you, it really accelerated my project.
If you wish to use it or steal some parts be my guest! ;)
Best regards,
Ricardo

Ricardo,
I am happy you liked my code and that it helped you advance your project. I've recently looked into OO Fortran, but I haven't quite figured it out and if it could help my codes (I am also a physicist, studying the atmospheric structure of planets and exoplanets). I will look through your code and it might help me understand Fortran 2003+.
The aim of my project is to hide as many HDF5 types as I can (I still need to keep HID_T for file_id and group_id). My current goals are to better handle error checking, returning on error while closing any open objects, and including an optional, intent(out) hdf_error. Also I am looking into a elegant way to access subsets of the dataset. My current needs are for single column (already implemented) and 2d array along the leading dimensions (next). But hyperslabs could be useful. And there is much more in HDF5, like I see you have chunked datasets, and there is encripted/zipped dataset, all interesting.
Regards,
Justin


Is it possible to automatically overflow crossover PEQ?

I'm looking to use the MSO strategy of loading input PEQs into the output channel PEQ and output channel specific PEQ into the crossover slots.
The files exported don't account for the two groups of four crossovers.  Is there a way to enable automatic overflow of the first crossover group into the second?  The error printed is:
Warning: Some filters were not imported because they didn't fit (try usingall)
so some user convenience and potentially dropping the need to specify groups would be great.
I'm really hoping to load a set of 8 PEQ into the crossover slots all at once.  This is possible with copy paste through the GUI, so I'm hoping we can make it work here.

To stay backwards compatible, I added an all value for the group parameter which allows you to use import across both groups simultaneously.
Implemented in def74a7
$ minidsp output 0 crossover all all import /home/mrene/dev/minidsp-rs/minidsp/test_fixtures/rew-filters.txt
Xover 0.0: Applied imported filter: biquad1
Xover 0.1: Applied imported filter: biquad2
Xover 0.2: Applied imported filter: biquad3
Xover 0.3: Applied imported filter: biquad4
Xover 1.0: Applied imported filter: biquad5
Xover 1.1: Applied imported filter: biquad6
Xover 1.2: Applied imported filter: biquad7
Xover 1.3: Applied imported filter: biquad8

Amazing!  This should really simplify how to use these filters since they are so critical with BEQ.


Provide extra info of the received signal for re-apply from reset, and others

Is your feature request related to a problem? Please describe.
After reset, the signal reapply is great but sometimes I need to partially reapply to the new run. For example, some signals are from human that are important to keep. Some are not from human and it’s intended to drop so that workflow can behave differently.
This is impossible to do today because there is no way to know if the signal is before or after reset point for reapply.
Additionally, it would be nice to provide info like the time of the signal. This is recorded in history event but not exposed in SDK.
Describe the solution you'd like
It is possible to get this info from history— we have a reset event and then comes with a batch of history to reapply in that batch. So it would be nice if SDK can provide a flag to tell whether or not the signal is from reapply.
The new api may be something like :
type SignalInfo struct{
    FromReapply bool
    SendingTime time.Time
}

info, ok := channel.ReceiveAsyncWithInfo(&val)

Describe alternatives you've considered
Another idea is to let server provide a feature to filter some signals when reapply — maybe based on signal names. But I feel like an SDK feature is better — more flexible and easier to implement
Additional context
Slack thread with Temporal team(Maxim) https://temporalio.slack.com/archives/CTDTU3J4T/p1686762087116689?thread_ts=1686722741.716739&cid=CTDTU3J4T

I understand this generically, but if you have different signals that need to behave differently (e.g. some from human and some not), might it be best to provide that info in the signal argument?
I am a bit scared of encouraging people to write the workflow to do different things based on whether it was reset. Having said that, we have the unsafe/discouraged-for-use workflow.IsReplaying(ctx) bool. So maybe we can have a similar workflow.IsReplayingBeforeResetPoint(ctx) bool.

It’s actually for the signals after the reset point which are considered as “reapply”. The one before reset point shouldn’t be treated differently in workflow code.
There is no way for workflow to know by argument.

So is !workflow.IsReplayingBeforeResetPoint(ctx) enough to know that?

No , it’s different than that…maybe I’ll make up a history as an example later.
The signals that I need to process differently are going to be re-processed again after reset, without any replay at all.

This requires a server-side change. The service must mark reapplied signals to distinguish them from the newly received signals.

Transferring to the server repo...


DOC: add citations

Depends on https://github.com/qiime2/qiime2/pull/387
Bibtex was generated via Google Scholar (super easy!).
Citations is iterable, but you could also have created a list like: [citation['vazquez2013emperor'], citation['vazquez2017bringing']].

Looks like CITATIONS.bib can't be in the root while being a non-egg conda-package. As cool as it would be to see that in the root of the repo, it's probably easier to just stick it inside q2_emperor/ rather than rejigger the dark magic that is conda package installation and site-packages.


Dev

Added traffic light detection program and model files.

Looks like an accidental push to origin master :) Make sure to push to your team's repo and not the Udacity one.


KAFKA-14879: Update system tests to use latest versions

Update the system tests to default to running the latest versions:

2.6.3
2.7.2
3.3.2

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Did you mean to make this a draft or is it ready for review @kirktrue ?

It's meant to be a draft for now. I first want to run through the system tests on a build server to make sure they behave.

Test failures are Kraft-related and do not appear to be caused by this change.

@jolshan Can you take a look at this when you have some time? Thanks

This appears to be taken up by someone else, but they're doing it in a more automated/dynamic manner, which is better anyway.

Sorry I must have missed this ping. :( My bad.

Sorry I must have missed this ping. :( My bad.

No worries at all 👍


James 1651

CI is building, see : https://github.com/linagora/james-project/pull/75 result

:+1:

:+1:

:+1:

merged.

Late but :+1:


schedule: fix scheduler removes region leader (#1462)

What problem does this PR solve?
Normally, PD avoids removing leader directly when scheduling. When the leader needs to be removed, the PD inserts a transfer leader step. However, when the number of replica is set to 1, this scheme does not attempt to transfer the leader to the peer that will be created, resulting in the direct remove of the leader.
What is changed and how it works?
Constructs a list based on the current peers and the peer to be added, then looks for the target of the transferLeader in the list. If target is not found, it returns an error to cancel the schedule process.
Check List
Tests

Unit test

Related changes

Need to cherry-pick to the release branch
Need to be included in the release notes

/run-all-tests tidb=release-2.1 tikv=release-2.1 tidb-test=release-2.1


Force immediate shutdown

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the latest JDA version.
[x] I have checked the branches or the maintainers' PRs for upcoming features.

Feature Request

So, i asked about this "feature" (more like an addition or fix if you wish) in the JDA guild already and i was advised to rather create an issue instead of creating a PR, so here it is. I saw many people having "issue" with JDA not shutting down immediately even when using shutdownNow() method. This could be fixed by using two methods provided in the http client.
Example Use-Case

These are the methods that could result in an immediate shutdown.
httpClient.connectionPool().evictAll();
httpClient.dispatcher().executorService().shutdown(); //assuming httpClient is really an http client variable, of course

Those methods could be implemented in 2 ways.

Implement them into the shutdownNow() method
Create a separate method for it, for example shutdownForcely(), which would call shutdownNow() and those 2 methods

Example

shutdownNow()

    @Override
    public synchronized void shutdownNow()
    {
        shutdown();
        threadConfig.shutdownNow();
        this.getHttpClient().connectionPool().evictAll();
        this.getHttpClient().dispatcher().executorService().shutdown();
    }

shutdownForcely() (?)

    @Override
    public synchronized void shutdownForcely()
    {
        shutdownNow();
        this.getHttpClient().connectionPool().evictAll();
        this.getHttpClient().dispatcher().executorService().shutdown();
    }

Calling such methods would make JDA completely unusable for further use, obviously.

A few things to consider here:

We have to be careful to not close clients that JDA didn't create, but were passed into the JDABuilder
Being double-careful that those connection pools are per-client and not global OkHttp pools, in which case they would also violate first point
can really only be applied to the shutdownNow or a new one like suggested above, since this will abort any ongoing or queued RestActions
IMO this is a bug of OkHttp and not sure if adding (possibly temporary) workarounds into JDA is the right call
disabling HTTP2 in Okhttp might be the better option, if possible

* can really only be applied to the shutdown**Now** or a new one like suggested above, since this will abort any ongoing or queued RestActions

Well, it's obvious that an immediate shutdown is really immediate, so queued RestActions won't get executed so people have to count on it.

* IMO this is a bug of OkHttp and not sure if adding (possibly temporary) workarounds into JDA is the right call

If this is a bug, then it's a really old one, as there were few releases which didn't fix it (yet?). But i agree with other points.

as there were few releases which didn't fix it (yet?).

The same applies to JDA. I have known about this behavior for months if not years and decided not to change our handling. I intentionally did not add any special shutdown behavior for this and I don't intend to do.

and decided not to change our handling. I intentionally did not add any special shutdown behavior for this and I don't intend to do.
May i please know why?

I consider this bad behavior and a bug in okhttp that we should not need to workaround. I try not to implement workarounds if possible. And in this case its definitely not needed as the thread times out either way.

Just throwing in my two cents but I would like a blocking shutdown method that only returns once pending RestActions have completed, after that I don't really care about the okhtttp connection pool and could just exit manually. Even with http 1.1 the okhttp still seems to take 1 minute to shut down (considerably better than 5 minutes with http 2 but still).

I consider this bad behavior and a bug in okhttp that we should not need to workaround. I try not to implement workarounds if possible.

I mean, the method as shutdownForcely() for example could be useful for people that really want to shut JDA down immediately after calling the method. They might/might not have enough experience to find out the solution by themself.

I think adding a section in the troubleshooting page of the wiki where the workarounds are explained and a corresponding tag for the Discord servers Spectra bot would be a good option to consider in the case that no changes JDAs code are made.

I mean, the method as shutdownForcely() for example could be useful for people that really want to shut JDA down immediately after calling the method. They might/might not have enough experience to find the solution out by themself.

From my experience, most people figure out how to use System.exit(0); on their own. I don't really see this as a good argument and adding a bunch of shutdown methods is not useful. Again this would be trying to work around an issue with okhttp which I don't want to do.

adding a section in the troubleshooting page of the wiki where the issue and workarounds are explained

I think that is reasonable.

I have added an entry for this in the troubleshooting guide: Shutdown but the process doesn't exit

I have added an entry for this in the troubleshooting guide: Shutdown but the process doesn't exit-Troubleshooting#shutdown-but-the-process-doesnt-exit)

I consider this as a reason to close this issue. Thanks for your time.


metrics stopped updating

I was looking around for games with just below 15 ratings to bump them into the next category ('pre-Danger'?) I've rated two such games around 2 hours ago, and commented on one of them (see screen-shot), but they're still stuck below 15, and my own metrics haven't updated either.
The two games are

Conservation Island, from which a screenshot.
Fat Puss

Interestingly, 'events/ludum-dare/40/mygrades' has updated, with the two games mentioned!
I'm not the only one with this problem, I think: see here and here.

If /events/ludum-dare/40/mygrades has updated you can be sure that the new grade took and is in the DB.
The bug,  was another symptom of the issue that coused #1533 and is resolved by #1534

Yes, that and that the grades (stars) did not disappear when I left a game-page suggested that it must've been saved at least somewhere.
This seems back in working order in any case :-)

Feel free to close the issues then :)


fix: Fix segment index assertions with DAI

When DAI adds ad segments faster than real-time, the availability
windows gets messed up, and some assertions in SegmentIndex break.
This locks down the presentation start time after the initial manifest
is parsed, so that the availability window is stable throughout
playback.
b/233075535

I feel like this should probably be mentioned in the manifest parser tutorial. I guess in the "Full Manifest Parser Example" section?

I don't see a good place for it.  In fact, that tutorial doesn't even mention notifySegments(), which this new method is linked to.


⚠ deces.matchid.io search API has degraded performance

In 7fbc4dc, deces.matchid.io search API (https://deces.matchid.io/deces/api/v1/search?q=pompidou) experienced degraded performance:

HTTP code: 200
Response time: 2113 ms

Resolved: deces.matchid.io search API performance has improved in 017a451.


Setup repo for independent Viz

Probably at github.com/ncsa/phylodiver-viz
Need source code and 'build' method from Chad/Matt.
To start, will have a builds/prebuilt/ directory with our current static files.
Then populate that repo's issues with tasks from https://docs.google.com/presentation/d/1DLfKwCaQFD9i7OP2gMv1xGVlr0DStuNztCU7Zb_czhQ/edit#slide=id.g126c2999b5a_0_14

It's at: https://github.com/ncsa/phylodiver-viz/


cannot read from public GCS bucket if non logged in

Environment
Delta-rs version: deltalake==0.19.2
Binding: Python
Environment:

Cloud provider: GCS for storage, but running locally
OS: MacOS
Other:

Bug
What happened:
I have a public GCS bucket with a bunch of Delta Lake tables. the bucket has viewer access for allUsers, meaning unauthenticated users can access it. you can easily test this with pandas or other libraries (I hit this with Ibis):
[ins] In [1]: import gcsfs

[ins] In [2]: import pandas as pd

[ins] In [3]: from deltalake import DeltaTable

[nav] In [4]: pd.read_parquet("gs://ibis-analytics/penguins.parquet", storage_options={"token": "anon"})
Out[4]:
       species     island  bill_length_mm  bill_depth_mm  flipper_length_mm  body_mass_g     sex  year
0       Adelie  Torgersen            39.1           18.7              181.0       3750.0    male  2007
1       Adelie  Torgersen            39.5           17.4              186.0       3800.0  female  2007
2       Adelie  Torgersen            40.3           18.0              195.0       3250.0  female  2007
3       Adelie  Torgersen             NaN            NaN                NaN          NaN    None  2007
4       Adelie  Torgersen            36.7           19.3              193.0       3450.0  female  2007
..         ...        ...             ...            ...                ...          ...     ...   ...
339  Chinstrap      Dream            55.8           19.8              207.0       4000.0    male  2009
340  Chinstrap      Dream            43.5           18.1              202.0       3400.0  female  2009
341  Chinstrap      Dream            49.6           18.2              193.0       3775.0    male  2009
342  Chinstrap      Dream            50.8           19.0              210.0       4100.0    male  2009
343  Chinstrap      Dream            50.2           18.7              198.0       3775.0  female  2009

[344 rows x 8 columns]

[nav] In [5]: pd.read_csv("gs://ibis-analytics/penguins.csv", storage_options={"token": "anon"})
Out[5]:
       species     island  bill_length_mm  bill_depth_mm  flipper_length_mm  body_mass_g     sex  year
0       Adelie  Torgersen            39.1           18.7              181.0       3750.0    male  2007
1       Adelie  Torgersen            39.5           17.4              186.0       3800.0  female  2007
2       Adelie  Torgersen            40.3           18.0              195.0       3250.0  female  2007
3       Adelie  Torgersen             NaN            NaN                NaN          NaN     NaN  2007
4       Adelie  Torgersen            36.7           19.3              193.0       3450.0  female  2007
..         ...        ...             ...            ...                ...          ...     ...   ...
339  Chinstrap      Dream            55.8           19.8              207.0       4000.0    male  2009
340  Chinstrap      Dream            43.5           18.1              202.0       3400.0  female  2009
341  Chinstrap      Dream            49.6           18.2              193.0       3775.0    male  2009
342  Chinstrap      Dream            50.8           19.0              210.0       4100.0    male  2009
343  Chinstrap      Dream            50.2           18.7              198.0       3775.0  female  2009

[344 rows x 8 columns]

but trying to read a Delta Lake table in the same place -- if not authenticated with GCP it seems -- results in an error:
[ins] In [6]: DeltaTable("gs://ibis-analytics/penguins.delta", storage_options={"token": "anon"})
---------------------------------------------------------------------------
OSError                                   Traceback (most recent call last)
Cell In[6], line 1
----> 1 DeltaTable("gs://ibis-analytics/penguins.delta", storage_options={"token": "anon"})

File ~/repos/ibis-analytics/.venv/lib/python3.12/site-packages/deltalake/table.py:380, in DeltaTable.__init__(self, table_uri, version, storage_options, without_files, log_buffer_size)
    360 """
    361 Create the Delta Table from a path with an optional version.
    362 Multiple StorageBackends are currently supported: AWS S3, Azure Data Lake Storage Gen2, Google Cloud Storage (GCS) and local URI.
   (...)
    377
    378 """
    379 self._storage_options = storage_options
--> 380 self._table = RawDeltaTable(
    381     str(table_uri),
    382     version=version,
    383     storage_options=storage_options,
    384     without_files=without_files,
    385     log_buffer_size=log_buffer_size,
    386 )

OSError: Generic GCS error: Error performing token request: Error after 10 retries in 8.200125916s, max_retries:10, retry_timeout:180s, source:error sending request for url (http://<IP_ADDRESS>/computeMetadata/v1/instance/service-accounts/default/token?audience=https%3A%2F%2Fwww.googleapis.com%2Foauth2%2Fv4%2Ftoken)

there's not a lot in the stacktrace to go on
What you expected to happen:
above works
How to reproduce it:
You can try it out on the bucket noted above: gs://ibis-analytics has penguins.csv, penguins.parquet, and penguins.delta in it
More details:
this was reproduced by others as well

We simply use the object store crate in Rust, if it's not working then it's because anon is not a supported config, you should try asking this in arrow-rs where object store belongs

is there any documentation on what is supported in storage_options?


No command 'postgrest' found

I downloaded postgrest-<IP_ADDRESS>.tar.xz to my ubuntu 14.04 server. then i run the following command to install it.
postgrest  --db-host localhost  --db-port 5432
--db-name my_db      --db-user postgres
--db-pass foobar     --db-pool 200
--anonymous postgres --secure
--port 3000
I got the following error message.  what did I do wrong?
No command 'postgrest' found, did you mean:
Command 'postgres' from package 'postgres-xc' (universe)

Is postgrest in your PATH?
Perhaps cd into the path where you extracted the binary and try ./postgrest instead

Also have you decompressed the binary from the tar file?
tar xf postgrest-<IP_ADDRESS>.tar.xz

I decompressed the file.  got this new error message. how do i fix it?
-bash: ./postgrest-<IP_ADDRESS>: cannot execute binary file: Exec format error

another quick question. what is --anonymous postgres and why we need it?  we already provide username and password --db-user postgres --db-pass foobar

Are you on Mac or Linux? Make sure you downloaded the binary to match your operating system.
If you like you can pick anonymous to be the same role as db-user. The db-user role is what is used to check passwords and switch roles if someone tries to authenticate. If the client does not try to authenticate (does not send HTTP Basic auth headers) then Postgrest executes their queries as the anonymous role.

I am using ubuntu 14.04. and download the right version.  Here's what I did step by step.  Dont know why still couldn't get it to work.
vagrant@vagrant-ubuntu-trusty-32:~$ wget http://bin.begriffs.com/dbapi/heroku/postgrest-<IP_ADDRESS>.tar.xz
--2015-01-13 01:37:51--  http://bin.begriffs.com/dbapi/heroku/postgrest-<IP_ADDRESS>.tar.xz
Resolving bin.begriffs.com (bin.begriffs.com)... <IP_ADDRESS>
Connecting to bin.begriffs.com (bin.begriffs.com)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 4284312 (4.1M) [binary/octet-stream]
Saving to: ‘postgrest-<IP_ADDRESS>.tar.xz’
100%[=====================================================================================================================>] 4,284,312   1.68MB/s   in 2.4s
2015-01-13 01:37:59 (1.68 MB/s) - ‘postgrest-<IP_ADDRESS>.tar.xz’ saved [4284312/4284312]
vagrant@vagrant-ubuntu-trusty-32:~$ ls
db  postgrest-<IP_ADDRESS>.tar.xz
vagrant@vagrant-ubuntu-trusty-32:~$ tar xf postgrest-<IP_ADDRESS>.tar.xz
vagrant@vagrant-ubuntu-trusty-32:~$ ls
db  postgrest-<IP_ADDRESS>  postgrest-<IP_ADDRESS>.tar.xz
vagrant@vagrant-ubuntu-trusty-32:~$ postgest
postgest: command not found
vagrant@vagrant-ubuntu-trusty-32:~$ ./postgrest-<IP_ADDRESS>
-bash: ./postgrest-<IP_ADDRESS>: cannot execute binary file: Exec format error
vagrant@vagrant-ubuntu-trusty-32:~$ ./postgrest-<IP_ADDRESS> --db-host localhost --db-port 5432 --db-name appmonkey --db-user appmonkey --db-pass foo123456 --db-pool 200 --anonymous appmonkey --secure --port 3000
-bash: ./postgrest-<IP_ADDRESS>: cannot execute binary file: Exec format error

What's your CPU architecture? The binary on the readme was built for x86 64-bit.

got it. i am running ubuntu trusty 32. no wonder it wont work. let me try it on 64 bit. thanks for  your help

begriffs, thanks alot for your help. I got it working , but still have one more issue.
$ ./postgrest-<IP_ADDRESS>
--db-host localhost
--db-port 5432
--db-name appmonkey
--db-user appmonkey
--db-pass Abcd1234567
--db-pool 200
--anonymous postgres
--port 3000
WARNING, running in insecure mode, auth will be in plaintext
when I test it using postman, i got following error message.  postgres is the default superuser, what permission does it need? btw, I have another user called "appmonkey" and its the owner of "appmonkey" database.
{
"status": "Fatal error",
"hint": null,
"code": "42501",
"message": "permission denied to set role "postgres"
}

What's happening is that you've chosen the user appmonkey to do the authentication. A request arrives without credentials and postgres tries to switch to the anonymous role which you've specified as postgres. It attempts to do a set role command as appmonkey, but that account does not have the permissions that postgres does, so the database forbids it to switch roles to a more powerful account.
What you need to do is choose a --db-user which is at least as powerful as the anonymous role. In your case you could set both db-user and anonymous to be postgres.

finally got it to work, thank you


Cannot enter channel, error message "cannot cast..."

Description
When trying to enter one of the channels on our RC instance, I get the message org.commonmark.node.Orderedlist cannot be cast to org.commonmark.node.Paragraph
Devices and Versions

Your Rocket.Chat.Android version: 2.5.0 2034
Your Rocket.Chat Server version: don't know.

Mobile device model and OS version: Huawei P9 Lite - Android 7.0
Steps to reproduce

Open app, go into the channel, which has 11 unread messages, error message appears, page stays blank. Other channels are fine.
Logs

Hi! Duplicate #1532 ?

Closed by #1558


⚠ Cioos Pacific CKAN Sitemap has degraded performance

In dfddb71, Cioos Pacific CKAN Sitemap (https://catalogue.cioospacific.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 9643 ms

Resolved: Cioos Pacific CKAN Sitemap performance has improved in 3f43323 after 17 minutes.


fix: Podfile.lock changed everytime when installed on different machines

This PR is related to issue #1707, our team encountered this issue, after search around, we found this line of code can solve our problem, I'm not sure this is the root cause, hoping this can provide any insights on this issue

Are you hitting this when using React Native? Note that it has its own (very similar) podspec, different from the one in this repo: https://github.com/facebook/react-native/blob/main/packages/react-native/ReactCommon/yoga/Yoga.podspec
But the underlying cause here is that different Ruby versions sort things differently

Are you hitting this when using React Native? Note that it has its own (very similar) podspec, different from the one in this repo: https://github.com/facebook/react-native/blob/main/packages/react-native/ReactCommon/yoga/Yoga.podspec
But the underlying cause here is that different Ruby versions sort things differently, so they may create different results depending on machine if you aren't using consistent versions.

Aha, I thought this is the underlying yoga of RN, thanks for point me out
By the way, the ruby version between out team member already keep consistent with each other (ruby 3.2.3), so I also wondering what's the root cause


visit(url) times out w/ EventSource

My findings: https://gist.github.com/obfusk/7803381.
PhantomJS issue: ariya/phantomjs#11816.
- Felix

Is this still an issue with phantomjs 2+ and poltergeist 1.8.0+ ?

Closing due to lack of response


Improve bookmarks

The new bookmarks system is very hidden from the user, at the very end of an already bloated menu, and no way of having a overall view of the bookmarks, and to go directly to them. This commit fixes that:

This gives the bookmarks its own accessible menu in script editor. Not only that, it shows all bookmarks of the current script, and clicking them takes you directly to the line.

CC @KoBeWi
What's that unknown character in the screenshot? A tab character?

@akien-mga I actually use strip_edges(), but for some reasons those tabs aren't removed.

This is what I did originally with bookmarks, but @Paulb23 said that it should be an Edit sub-menu. Although with the bookmark list, it probably makes sense to have a completely separate menu for them.

@KoBeWi Made the same improvements to the text and shader editors. The shader editor however doesn't save bookmarks, as it doesn't save the shader state at all.

@Paulb23

Yeah it should stay as a Edit sub-menu.
This doesn't address the issues I made. Bookmarks are supposed to be something easy to localize and reach, and where they are now, I'm quite people won't event notice that the feature exists. Every IDE I've used has its own menu for bookmarks, for a reason.

I took my time to test both options with multiple bookmarks. Here's comparison:
Old one

New one

IMO the new one is better for navigating the bookmark list. But it has a bug - shortcut buttons don't work until you open the menu for the first time.

@KoBeWi Fixed.

@YeldhamDev Can't we have some shortcuts to jump to bookmarks like Ctrl/Cmd+Shift+1,..+2,..+3 and so on.
It would work for first 9 similar to tab switching in browsers.

Can't we have some shortcuts to jump to bookmarks like Ctrl/Cmd+Shift+1,..+2,..+3 and so on.

This would work for tabs (and would be nice feature to have actually), but for bookmarks... not really. You can set bookmarks to arbitrary lines, they aren't labelled in any way, you rarely see all of them at once and they tend to shift a lot (I change bookmarked lines very often). Thus binding numbers to bookmarks is just impractical.

The visibility issue also applies to breakpoints, as there is currently no easy way to view all of them, so a similar approach will need to be applied to them eventually.
I've seen three general approaches.
Firstly, The approach here is used by Sublime text, with a list of bookmarks under goto -> bookmarks.
Secondly, I've also seen a separate panel under the script eg. in the bottom bar containing a tree of bookmarks.
Alternatively, as a switchable panel in place of the function overview.
Finally, last time something was added to the main bar it cased issues with smaller screens (#18078).

@Paulb23 I could see breakpoints and bookmarks sharing a menu, "Goto", as you pointed out that Sublime does. That's still more accessible than putting it in the very end of the "Edit" menu.
And about the screen size, we have to find an equilibrium on things not taking too much space and easy localization. People which smaller screen may have to settle on lowering scale/font size on their end. And I speak as someone which a small screen myself (1366x768)

I could see breakpoints and bookmarks sharing a menu, "Goto", as you pointed out that Sublime does.

If we do that, I think we should spell it correctly as "Go to", otherwise it will be a headache for internationalization (and there's no real reason to spell it "Goto" as it's not really related to the programming goto statement).

Sure, but while breakpoints aren't navigable (this should be done in a different commit), having the menu be "Bookmarks", and after they are, they can be united in a "Go To" menu.

So what's the status in the end? Can we merge as is or should it be changed?

I'll merge as is for now to reduce the backlog, it can be further improved in another PR.

Thanks!


Firebase Auth with Google Cloud Functions

Thanks to @jhalborg for contributing this example: https://gist.github.com/jhalborg/5042201f558034243376cffb55111d85
Would love to see a step-by-step guide for this, so if anyone wants to get their hands dirty, feel free  to create a PR 😄 I'd be happy to review it

Might write it up later, but essentially what's happening is:

Login/sign up the user on the client against Firebase using the Auth SDK to receive a Firebase token
Send the auth/ID token as a param to your schema with the new query, getCustomToken
In your cloud function, use the Firebase Admin SDK to validate the given Firebase ID token.
If it is valid, look for the firebase userID (can be extracted from the verified/decoded token) in your Graphcool users.
4a. If one is found, get the Graphcool user ID
4b. If the user does not yet exist on Graphcool, create one and get the ID of the new user
Generate a Graphcool access token from the users ID using the graphcool lib.
Embed the generated Graphcool token as a 'claim' in a newly minted Firebase auth custom token and return it
On the client, re-sign in the user using the signInWithCustomToken(token)

You can now access Firebase with the created token, and Graphcool with the firebasetoken.claims.graphcool token - happy day! :-) If you don't like having the tokens embedded, you can skip 6/7 and just return the Graphcool token and keep the two separate on the client if that's what you prefer.
The Firebase docs and jwt.io is your friend when building this stuff ;-)

@jhalborg Thank you very much for example and explanation. I just want to clarify point number 3. By cloud function, you mean a function deployed to eg. AWS or is that something that can be done with GraphCool Functions?
I am not sure how those Schema Extension functions can be created. There are only Server-Side subscription and Request Pipeline. Either one of them doesn't feel like the right thing since it requires some trigger. From your snippet of schema-extension.sdl  I get the impression that this custom mutation should be actual trigger.

here is one https://github.com/Gregor1971/firebase-graphcool-authentication

@mkozhukharenko Looks good, but I used Firebase Functions instead of serverless. There is great advantage that you don't need to worry about service key. Just receive token, verify it and return it's payload. In Graphcool I can then just take care of finding/creating user without any Firebase interference.
const functions = require('firebase-functions')
const admin = require('firebase-admin')

admin.initializeApp(functions.config().firebase)

exports.verifyToken = functions.https.onRequest((req, res) => {
  const { idToken } = req.query
  if (!idToken) {
    res.json({ error: 'Missing idToken in query' })
    return
  }
  admin.auth().verifyIdToken(idToken)
    .then(authUser => res.json(authUser))
    .catch(() => res.json({ error: 'Could not verify id token with Firebase' }))
})

Thanks so much @Gregor1971, I just merged your PR: https://github.com/graphcool-examples/functions/pull/66. 🎉


fix: temporarily raise page size for testing

Goal
Temporarily raise page size for iOS testing.
Let's wait til Monday to push this.

Oops! Didn't see some checks were still failing

Looks like circleci failed to download the rover image? or something similar. I'm re-running it


Model selection

Hi,
For each sub-dataset (davis-2017 validation/test, davis-2016/youtube-vos), do you use only one model or several models selected for different sub-datasets? Could you tell me the detail about the model selection, thank you!

A single model is used for all datasets. We simply pick the last saved model.

Thank you!


Do some GAMM/RF exploration with the combined lake levels

Figure out which lakes influence the others the most, and use the smooths to figure out in what ways, etc.

Got some variable importance plots back from RF. Looking at those, then it will have to be some AIC stepwise fun with the GAMMs.

Used the RF results as a starting place, tried some different versions of the models using AIC stepwise, found that the starting models with the RF variables were still the best ones.


Attribution for caleb

Provide attribution for caleb in the readme regarding design and an initial java prototype

@miapolis shoudn't you have closed this?


Enable slower-than-realtime drift setups to achieve full accuracy in SITL

@dagar @priseborough @bresch @Stifael @MaEtUgR @tumbili I wanted to do this forever but only now got around to it: We had already a lockstep interface that would pause the simulation if the simulator paused, however, we did not have the ability to deal with constant drift (simulator running constantly at a slower rate). This fixes the simulation engine to properly deal with this and makes the whole simulation environment a lot easier to use.

I've tested this on gazebo and jmavsim and the ekf2 performance has improved. I did notice when running on gazebo that the GPS data appears to have a 200msec arrival  jitter which is not reflected in the time stamp which is evidenced as speed dependant noise on the position innovations and the timestamp field in vehicle_gps_position being regular. Here's a plot of the EKF's GPS position innovation and vehicle velocity*0.2


WIP: feat/tls: Support optional TLS for helm / tiller

Tasks/Todos:

[x]  Add pkg/tlsutil/cfg.go to support constructing *tls.Config for helm / tiller
[x]  Add TLS flags to tiller and support defaults extracted from OS environment.
[x]  Add helm option to support enabling TLS on client calls.
[ ]  Add flags to helm init to support deployment of tiller with TLS enabled:
* --tls-tiller-enable
* --tls-tiller-verify
* --tls-tiller-cert
* --tls-tiller-key
* --tls-ca-cert
[ ]  Add kubernetes secret and mount as volume to tiller container. Update tiller deployment.
[ ]  Add TLS flags to helm and support defaults extracted from OS environment.

Per the new developer workflow guidelines, I have removed the wip label and have prefaced the issue with  "WIP." Please remove the "WIP" preface when it is ready for review.

To create a test cert: https://gist.github.com/adamreese/c78b9d2f072c29a493f8110348c6b101


Windows self-rebuild does not run the new executable

On windows, a self-rebuild will succeed, but the new executable is not run after the compilation finishes --- instead the old one is executed.
Only after re-running it, then the new executable is actually executed.
Not a problem on unix.

Closed in 45112a1


Gson UserHook Args/Result serialization causes OutOfMemoryError

UserHooks parseParams causes an OutOfMemoryError when serializing a non-trivial, nested Object to JSON.
UserHooks#parseParam
I observed the issue when playing around with Inspeckage and DIVA. In DIVA, many methods are passed a View or another class that contains multiple references to other objects.
I did not find a trivial way to limit Gson parsing of nested objects (especially if it should support arbitrary objects, and not objects that are defined with Gson de/serialization in mind). In my tests, simply using toString() instead of gson.toJson() avoided the problem at the cost of the output format's "beauty".

I will look that.


Add support for complex helm charts with variables

When using  a Tiltfile with
k8s_yaml(helm("./charts/nginx-ingress"))
and the nginx-ingress is the chart from https://kubernetes-charts.storage.googleapis.com
when running tilt up I get this response..
Any Ideas?.
I've tried using k8s_yaml(local("helm template -n dodgy-release ./charts/nginx-ingress")
and this will generate valid information, but the specified release-name does not match the generated one, so the linkages are broken.
RELEASE-NAME-nginx-ingress-controller (ConfigMap)
      RELEASE-NAME-nginx-ingress (ServiceAccount)
      RELEASE-NAME-nginx-ingress (ClusterRole)
      RELEASE-NAME-nginx-ingress (ClusterRoleBinding)
      RELEASE-NAME-nginx-ingress (Role)
      RELEASE-NAME-nginx-ingress (RoleBinding)
      RELEASE-NAME-nginx-ingress-controller (Service)
      RELEASE-NAME-nginx-ingress-default-backend (Service)
      RELEASE-NAME-nginx-ingress-controller (Deployment)
      RELEASE-NAME-nginx-ingress-default-backend (Deployment)
    ERROR:
       Error: Error upserting global_yaml: kubectl apply:
       stderr: Error from server (Invalid): error when creating "STDIN": ConfigMap "RELEASE-NAME-nginx-ingress-controller" is invalid: metadata.name: Invalid value: "RELEASE-NAME-nginx-ingress-controller": a DNS-1123 subdomain must
       consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is
       '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')
       Error from server (Invalid): error when creating "STDIN": ServiceAccount "RELEASE-NAME-nginx-ingress" is invalid: metadata.name: Invalid value: "RELEASE-NAME-nginx-ingress": a DNS-1123 subdomain must consist of lower case
       alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')
       Error from server (Invalid): error when creating "STDIN": ClusterRoleBinding.rbac.authorization.k8s.io "RELEASE-NAME-nginx-ingress" is invalid: subjects[0].name: Invalid value: "RELEASE-NAME-nginx-ingress": a DNS-1123 subdomain
       must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is
       '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')
       Error from server (Invalid): error when creating "STDIN": RoleBinding.rbac.authorization.k8s.io "RELEASE-NAME-nginx-ingress" is invalid: subjects[0].name: Invalid value: "RELEASE-NAME-nginx-ingress": a DNS-1123 subdomain must
       consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is
       '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')
       Error from server (Invalid): error when creating "STDIN": Service "RELEASE-NAME-nginx-ingress-controller" is invalid: metadata.name: Invalid value: "RELEASE-NAME-nginx-ingress-controller": a DNS-1035 label must consist of lower
       case alphanumeric characters or '-', start with an alphabetic character, and end with an alphanumeric character (e.g. 'my-name',  or 'abc-123', regex used for validation is '[a-z]([-a-z0-9]*[a-z0-9])?')
       Error from server (Invalid): error when creating "STDIN": Service "RELEASE-NAME-nginx-ingress-default-backend" is invalid: metadata.name: Invalid value: "RELEASE-NAME-nginx-ingress-default-backend": a DNS-1035 label must consist
        of lower case alphanumeric characters or '-', start with an alphabetic character, and end with an alphanumeric character (e.g. 'my-name',  or 'abc-123', regex used for validation is '[a-z]([-a-z0-9]*[a-z0-9])?')
       Error from server (Invalid): error when creating "STDIN": Deployment.apps "RELEASE-NAME-nginx-ingress-controller" is invalid: [metadata.name: Invalid value: "RELEASE-NAME-nginx-ingress-controller": a DNS-1123 subdomain must
       consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is
       '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*'), spec.template.spec.serviceAccountName: Invalid value: "RELEASE-NAME-nginx-ingress": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-'
        or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')]
       Error from server (Invalid): error when creating "STDIN": Deployment.apps "RELEASE-NAME-nginx-ingress-default-backend" is invalid: metadata.name: Invalid value: "RELEASE-NAME-nginx-ingress-default-backend": a DNS-1123 subdomain
       must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is
       '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')

hmmm... yes, you're on the right track! Doing
k8s_yaml(local("helm template -n dodgy-release ./charts/nginx-ingress"))

is a perfectly fine way to do more complex things with helm when the built-in doesn't have what you want.
Can you elaborate a bit on what you mean when you say "the specified release-name does not match the generated one, so the linkages are broken"? What do you want the release name to be? What linkages are broken?
I know very little about this chart, but I ran the local command above and it seemed to deploy ok for me.

hmmm...I'm going to close this ticket since I haven't heard a response. Happy to reopen if you're still having issues!


Find the page source of potential broken URLs

Over and above the awesome work that our build/test tooling does, I was curious to see if any Fermyon Developer Home pages have broken URLs that are pointing to other Fermyon content. For example, URLs that are pointing to our GitHub repos, our website and our blog posts. (Content that we can control update/change). The build/test automation gives a list of broken links but this creates a mapping of where those links are (which pages we can quickly go and update).
The following Python script generates a JSON object that has the page in Fermyon Developer Home (that needs updating) as the key and any broken URLs as the value (a list of URLs in the markdown that we need to update).
# python3.11 -m pip install --upgrade pip
# python3.11 -m pip install beautifulsoup4
# python3.11 link-check.py
import json
import requests
import xml.dom.minidom as minidom
from bs4 import BeautifulSoup

# Fetch sitemap's text toggle using comments
## Developer
website_sitemap = requests.get('https://developer.fermyon.com/sitemap.xml', allow_redirects=True).text
## Website
# website_sitemap = requests.get('https://www.fermyon.com//sitemap.xml', allow_redirects=True).text

# Parse the sitemap's text to obtain the list of pages
parsed_website_sitemap_document = minidom.parseString(website_sitemap)

# Cherry pick just the loc elements from the XML
website_sitemap_loc_elements = parsed_website_sitemap_document.getElementsByTagName('loc')

# Declare blank lists of pages for the website being indexed
website_page_urls = []

# Iterate over loc elements (of the sitemap) and add to the site's list of pages
for website_sitemap_loc_element in website_sitemap_loc_elements:
    website_page_urls.append(website_sitemap_loc_element.toxml().removesuffix("</loc>").removeprefix("<loc>"))

# This is the list of pages that will be indexed
print("Number of page to process is {}\n First page to process is {} and the last page to process is {}".format(len(website_page_urls), website_page_urls[0], website_page_urls[len(website_page_urls) - 1]))

# Loop through the array of urls

# Create a dict with each page we are reading as the string/key and each broken link in a list (the value)
page_url_dict = {}
for one_page in website_page_urls:
    print(f"Processing {one_page}")
    page_url_dict[one_page] = []
    response = requests.get(one_page)
    if response.status_code == 200:
        # Parse the content of the page
        soup = BeautifulSoup(response.content, 'html.parser')

        # Find all the <a> tags
        links = soup.find_all('a')

        # Extract the href attribute from each link
        urls = [link.get('href') for link in links]

        # loop through URLs to check
        # print(urls)
        for single_url in urls:
            if single_url is not None:
                if single_url.startswith("http"):
                    # only check fermyon related urls
                    substring = "fermyon"
                    if substring in single_url:
                        print(f"Processing {single_url}")
                        single_url_response = requests.get(single_url)
                        if response.status_code == 200:
                            print(f"Success {single_url}.")
                        else:
                            page_url_dict[one_page].append(single_url)
    else:
        print(f"Failed to retrieve content from {one_page}, status code: {response.status_code}")
json_string = json.dumps(page_url_dict, indent=4)
print(json_string)

All links now resolve; this can be used in future (as a diagnostic tool) if required.


[WIP][#163084103] Exclude rdsadmin queries from max_tx_age metric, create new max_system_tx_age metric

What
Excludes queries run by the rdsadmin user from the data used to create the max_tx_age metric. Introduces a new max_system_tx_age which includes only queries run by the rdsadmin user.
How to review
Code review
Run it down the pipeline using the paas-cf branch and check that the max_tx_age metric is still being published, and that the new max_system_tx_age metric is available.
1. cf install-plugin log-cache
2. cf target -o admin -s billing
3. cf tail $(cf service billing-db --guid) -f
Paired with @tnwhitwell and verified that the queries work and produce the intended results. The tests also assert that. Probably no need for the reviewer to check again.
Who can review
Not @AP-Hunt or @tnwhitwell

Integration tests will probably keep failing until paas-cf#1745 is merged and deployed, because they rely on it.

Blocked by https://github.com/alphagov/paas-rds-broker/pull/97


fix members decorator check with transformers

Fixes #17384

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Hey @shlomiassaf, thanks for the PR! @rbuckton can you review this? Do we have any specific API tests for something like this?

@DanielRosenwasser @rbuckton
I didn't find a previous test case for this, but its probably there.
I think it should be a conformance test right?

@DanielRosenwasser
It looks like there are a lot of logic that does not take into account applying transformers, in my case I see that when applying transformations on decorated nodes.
For example, in you run this code it will transpile a module with a simple transformer that remove all decorators for all class parameters
It will error with:
/node_modules/typescript/lib/typescript.js:27549
                        var moduleExports = getSymbolOfNode(location).exports;
                                                                     ^
TypeError: Cannot read property 'exports' of undefined

This happens since resolveNameHelper in checker.ts calls getSymbolOfNode:
        function getSymbolOfNode(node) {
            return getMergedSymbol(node.symbol);
        }

        function getMergedSymbol(symbol) {
            var merged;
            return symbol && symbol.mergeId && (merged = mergedSymbols[symbol.mergeId]) ? merged : symbol;
        }

Under the hood since we removed decorators, typescript updates the ModuleDeclaration (ts.updatesXXXX).
So, the node is synthetic which means it has no symbol, the symbol is in the original node.

Here's another example
This time, instead of removing the decorators of class parameters I replaces them with NotEmittedStatement.
  function visitNode(node: ts.Node) {
    if (node.kind === ts.SyntaxKind.Decorator && node.parent.kind === 146) {
      return ts.createNotEmittedStatement(node);
    } else {
      return node;
    }
  }

The error now:
node_modules/typescript/lib/typescript.js:15388
        while (node.kind === 297 /* PartiallyEmittedExpression */) {
                   ^
TypeError: Cannot read property 'kind' of undefined

Another thing I run into, when removing/replacing some of the decorators the output for    __metadata("design:paramtypes", [..... TYPES HERE ....]
Will always be [Object, Object....]
This happens because of resolver.getTypeReferenceSerializationKind(node.typeName, currentScope)
The currentScope is the synthetic node which can't resolve the type properly.
Maybe I'm doing it all wrong maybe these are real issues, dont know... didn't want to spam the issue list so I put all of it here

After some more digging it looks like most issues are related to bindings and the type serializers in the typescript transformer (ts.ts)
More precisely, synthetic nodes not having bindings set (Symbol, locals etc...).
Since each factory update (e.g. ts.updateParameter, ts.updateProperty...) produces a synthetic node it comes up.
For now the workaround I'v used is to manually assign locals and symbols to a visited (and replaced) symbol which is also a scope (usually SourceFile)
if (sourceFile['origin']) {
  sourceFile['symbol'] =  sourceFile['original']['symbol'];
  sourceFile['locals'] = sourceFile['original']['locals'];
}

This is a shady hack that seems to work at the moment but i'm not comfortable with it, who knows what logic in the native transformers might conflict with it.
That fact it works gives a clear direction on the problem...

Another one in the type checker
        function checkGrammarDisallowedModifiersOnObjectLiteralExpressionMethod(node: MethodDeclaration) {
            // We only disallow modifier on a method declaration if it is a property of object-literal-expression
            if (node.modifiers && node.parent.kind === SyntaxKind.ObjectLiteralExpression) {
                if (getFunctionFlags(node) & FunctionFlags.Async) {
                    if (node.modifiers.length > 1) {
                        return grammarErrorOnFirstToken(node, Diagnostics.Modifiers_cannot_appear_here);
                    }
                }
                else {
                    return grammarErrorOnFirstToken(node, Diagnostics.Modifiers_cannot_appear_here);
                }
            }
        }

Will fail upon running diagnostics.
node.modifiers && node.parent.kind

All of the built-in transformers with the exception of ts.ts were written to avoid using parent pointers. However, ts.ts has not been fully updated to avoid referencing .parent.

@rbuckton @DanielRosenwasser note that there are also some in the checker....


Wrap match-cases in parens

This fixes #584 and makes match a quite powerful pattern-matching tool.
Test included.
Compiles:
match 1, 3, 3
| 1, 1, 2 or 3 => ok 0
| 1, 2 or 3, 3 => ok 1
| _            => ok 0

To:
// Generated by LiveScript 1.4.0
(function(){
  var ref$;
  switch (ref$ = [1, 3, 3], false) {
  case !(1 === ref$[0] && 1 === ref$[1] && (2 === ref$[2] || 3 === ref$[2])):
    ok(0);
    break;
  case !(1 === ref$[0] && (2 === ref$[1] || 3 === ref$[1]) && 3 === ref$[2]):
    ok(1);
    break;
  default:
    ok(0);
  }
}).call(this);

Seems like something is fishy somewhere else, because it should already be wrapped in parens...

Looks like they get stripped out. A stand-alone case wouldn't need it. The context of being part in a greater expression seems lost there.

Any reason to close this PR? It might be the right fix, but I need to dig deeper

Woops. That was Incidental from me moving the fix to another branch on my fork. I guess i have to create a new one.


Passing ServerlessRestApi to function variables creates circular dependencies

Description:
I'm trying to pass the generated implicit API url to a function, but it fails with circular dependencies error:
code sample:
  Clients:
    Type: "AWS::Serverless::Function"
    CodeUri: ./lambdas/clients/build/main
    Properties:
      Handler: ./lambdas/clients/build/main
      Policies:
        - DynamoDBCrudPolicy:
            TableName: !Ref ClientsTable
      Environment:
        Variables:
          TABLE_NAME: !Ref ClientsTable
          BASE_URL: !Sub "https://${ServerlessRestApi}.execute-api.${AWS::Region}.amazonaws.com/Prod/"

I tried adding a dependency to the implicit resource, but it didn't help.
DependsOn: "ServerlessRestApi"

cloudformation output:
Waiting for changeset to be created..
Failed to create the changeset: Waiter ChangeSetCreateComplete failed: Waiter encountered a terminal failure state
Status: FAILED.
Reason: Circular dependency between resources: [ClientsDeleteclientsPermissionTest, ClientsDeleteclientsPermissionProd, VehiclesGetVehiclesPermissionProd, Clients, ClientsGetclientsPermissionProd, VehiclesGetVehiclesPermissionTest, ServerlessRestApiDeployment9836ce6440, Vehicles, ClientsPutclientsPermissionProd, ClientsPostclientsPermissionTest, ServerlessRestApi, ClientsPutclientsPermissionTest, ClientsGetclientsPermissionTest, ClientsPostclientsPermissionProd, ServerlessRestApiProdStage]

Tried explicitly defining API, it didn't work either :disappointed:
Currently testing a workaround, it looks stupid but the template does deploy!
Two lambda functions, same code or different up to you!

If it is an incoming request, the user already knows the API URL.
Otherwise, if it is an outgoing request triggered by an AWS event for example, the outgoing function has the URL in the environment variable that you can send over to your "first time" user maybe via e-mail or a webhook that points back to the "incoming" lambda function.

  Incoming:
    Type: 'AWS::Serverless::Function'
    Properties:
      FunctionName: Incoming
      Description: Incoming
      Runtime: nodejs8.10
      Handler: index.handler
      MemorySize: 256
      Timeout: 30
      AutoPublishAlias: prod
      Events:
        Trigger:
          Type: Api
          Properties:
            Path: /
            Method: post

  Outgoing:
    Type: 'AWS::Serverless::Function'
    Properties:
      FunctionName: Outgoing
      Description: Outgoing
      Runtime: nodejs8.10
      Handler: index.handler
      MemorySize: 256
      Timeout: 30
      AutoPublishAlias: prod
      Environment:
        Variables:
          URL: !Sub "https://${ServerlessRestApi}.execute-api.${AWS::Region}.amazonaws.com/Prod/"

This is a CloudFormation circular dependency issue. I'm not aware of a way to solve this, though you should be able to construct the URL based on the event your function receives.


Latin chars

The VS Code do not respect latin chars.

VSCode Version: Last
OS Version: Win 10 x64

Steps to Reproduce:

Open and type ç, ã, á

@jssmttgit can you please provide more details. May be a screen cast about what you see. I have no problems entering latin characters like èüéöäà

Hi,
when I do open the file it do not goes to ISO-8859-1.

Would be greate to set a default char codification.

You can: the setting is "files.encoding" and it defaults to utf8. What does the status line say about the encoding of the file

Thanks for your reply.
Yes, I do change, but it does not save this option for the next time, I look out for change settings but do not found where do I set default encoding.

You can change this globally or per workspace in settings (F1 > open global settings, F1 > open workspace settings). there are other issues opened to allow to persist this setting per file.


feat(crypto)!: modular transaction hashing

Closes https://github.com/cometbft/cometbft/issues/3383
#go-api-breaking
Removes unused HashFromByteSlicesIterative and SumMany in favor of simple implementation.

PR checklist

[x] Tests written/updated
[x] Changelog entry added in .changelog (we use unclog to manage our changelog)
[x] Updated relevant documentation (docs/ or spec/) and code comments
[x] Title follows the Conventional Commits spec

Are the changes under crypto/merkle a necessary part of this PR, or just a further contribution?

Are the changes under crypto/merkle a necessary part of this PR, or just a further contribution?

further contribution

further contribution

Would you mind to create another issue with those contributions? I can help with, if you want.

A general comment: doesn't package txhash render tmhash.New() and tmhash.Sum(...) obsolete? Moreover, should we be migrating all calls to tmhash.New() and tmhash.Sum(...) to txhash?

No, because if we change consensus params or Merkle tree hashing, we lose IBC compatibility. The destination chain needs to know the hashing algorithm to verify hashes. So, we must keep tmhash until we have a field in the header specifying the hashing algorithm.

@fedekunze @itsdevbear, does this satisfy your requirements? Would you please test this PR and see if it helps with your use case?

There is no feedback here.


Change golang versioning type to docker

Google's golang download link doesn't include the patch version if it's zero. To deal with this, we switched the version of the golang dependency in on-safe-to-test-label.yml to docker versioning (major.minor), which will allow us to always contact a working go download link.
Signed-off-by: Brady Siegel<EMAIL_ADDRESS>

Skipping the test label for this because:

Its a testing configuration change
The testing will not work until this testing configuration is in place.


Use GpuAlias when handling Empty2Null in GpuOptimisticTransaction

When handling empty-to-null processing with the GPU version of Delta Lake constraints, it builds an explicit GpuProjectExec with an alias.  It should be a GpuAlias which is fixed here.

build

I assume that there is a test coming that will exercise this?

Yes, this code path is exercised by some of the Delta Merge tests being added as part of the fix for #7007.


update AddImplementation

Codecov Report

Merging #329 (f38e634) into master (454ee74) will increase coverage by 1.40%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #329      +/-   ##
============================================
+ Coverage     62.45%   63.85%   +1.40%
- Complexity      459      475      +16
============================================
  Files            90       90
  Lines          1779     1779
  Branches        189      189
============================================
+ Hits           1111     1136      +25
+ Misses          617      592      -25
  Partials         51       51

Impacted Files
Coverage Δ

src/main/java/seedu/unify/model/task/State.java
91.66% <0.00%> (+12.50%)
:arrow_up:

src/main/java/seedu/unify/model/task/Task.java
96.82% <0.00%> (+14.28%)
:arrow_up:

src/main/java/seedu/unify/model/task/Priority.java
79.31% <0.00%> (+27.58%)
:arrow_up:

src/main/java/seedu/unify/model/task/Time.java
81.25% <0.00%> (+31.25%)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM


DEQ - UIC Inventory production release

Benefit

Final app issues and tasks to bring it closer to prod-ready functionality
Other tasks to-be-determined to facilitate a production deployment

Acceptance Criteria

[ ] Inventory Review and Approval milestone tasks
[ ] Prod-ready milestone tasks, and others that arise to necessitate a production release

Notes
Risks
Accept:

client feedback
unforeseen production release blocks
Mitigate:
client expectations

Sprint 3

Production release tasks
Final feature/function validation

Sprint 4

https://github.com/uic-utah/uic-inventory/issues/143
https://github.com/uic-utah/uic-inventory/issues/272
https://github.com/uic-utah/uic-inventory/issues/313

Sprint Final

https://github.com/uic-utah/uic-inventory/issues/313


java.net.URL serialization depends on hashCode which creates cache misses

Expected Behavior
URL is normalized so that to URL with the same toString() are considered identical inputs like it is done for the configuration cache
Current Behavior
URL is not normalized. That's because URL does DNS lookup and all kinds of fun stuff in its equal implementation. OkHttp has a nice explanation about it.
Context (optional)
This is a follow up from this Gradle slack thread
Steps to Reproduce
Reproducer: https://github.com/martinbonnin/test-url
$ git clone https://github.com/test-url
$ cd test-url
$ ./gradlew test

BUILD SUCCESSFUL in 304ms
1 actionable task: 1 executed

# Same thing but calling hashCode() on the URL
$ CALL_HASHCODE=1 ./gradlew test

BUILD SUCCESSFUL in 304ms
1 actionable task: 1 executed

The task should be up-to-date no matter if we call hashCode() or not. Expected output:
$ ./gradlew test

BUILD SUCCESSFUL in 304ms
1 actionable task: 1 executed

# Same thing but calling hashCode() on the URL
$ CALL_HASHCODE=1 ./gradlew test

BUILD SUCCESSFUL in 304ms
1 actionable task: 1 up-to-date

Gradle version
8.1.1
Build scan URL (optional)
No response
Your Environment (optional)
No response

some additional context: the java.net.URL constructor will be deprecated in Java 20, the recommended replacement is URI
https://download.java.net/java/early_access/jdk20/docs/api/java.base/java/net/URL.html#constructor-deprecation

Thank you for providing a valid reproducer.
The issue is in the backlog of the relevant team, but this area of Gradle is currently not a focus one, so it might take a while before a fix is made.

@jbartok would you accept a pull request?

Yes, sure.

Bumping @aSemy pull request here: https://github.com/gradle/gradle/pull/24987
Many thanks for looking into this @aSemy ❤ !

Is there a reason you need to use URL specifically? Would using URI (or even a String) work instead? Those types don't exhibit this problematic behavior.

Late to the party but to answer that question:

Is there a reason you need to use URL specifically?

There's not really but URL is used in tools that I don't own (Dokka here)

This is a bug in how URL is serialized by the Java serialization. Since the bug is unlikely to be fixed in the JDK

Agreed but Gradle is also in a position to fix and/or warn about this. Would be nice to have a warning if someone is doing so.

Would a warning be okay? We do warn about potentially unsafe situations. We could add a warning about URL being used as an input. Doing so would mean that we'd disable execution optimizations (up-to-date checks, caching, parallel execution) for the task, too.

A warning would help smooth off this rough edge. It's only because @martinbonnin thoroughly investigated a Build Scan that this issue was discovered, I imagine that many other plugin authors will not be so fortunate to have such a dedicated tester!
I would also suggest the warning links to a brief section in the docs. The docs section wouldn't specifically be about URL, but a brief general section about the importance of hashcode/equals in https://docs.gradle.org/current/userguide/build_cache_debugging.html, e.g.

Cache misses can be caused by classes with inconsistent implementations of hashcode()/equals(), a promeninent example being java.net.URL + $otherExamples. It is recommended to use an alternative class instead - java.net.URL can be replaced with java.net.URI.

(Maybe this section exists already but I couldn't find it).
Do types with invalid hashcodes/equals cause Configuration Cache issues, or only Build Cache issues?

OK, let's add a warning then.

I would also suggest the warning links to a brief section in the docs. The docs section wouldn't specifically be about URL, but a brief general section about the importance of hashcode/equals in docs.gradle.org/current/userguide/build_cache_debugging.html, e.g.

Is there's a general problem with hashCode() and equals()? Isn't this issue specifically about how URL's serialized form changes based on whether or not you have called hashCode() yet? Gradle doesn't use equals() to compare serialized inputs (not since Gradle 8.0, see https://github.com/gradle/gradle/issues/17324).

Is there's a general problem with hashCode() and equals()? Isn't this issue specifically about how URL's serialized form changes based on whether or not you have called hashCode() yet? Gradle doesn't use equals() to compare serialized inputs (not since Gradle 8.0, see #17324).

I try to avoid digging too much into Java/Gradle internals so I will defer to your experience :), but in my mind hashCode() and equals() are inextricably linked. Also, even if neither is used directly by Gradle they are used by Java Serialization, which Gradle does use. That's just my layman's understanding, which might not be accurate.

+1 for the warning, that will discourage plugins authors to use URL and hopefully encourage the already existing usages to migrate away.

Isn't this issue specifically about how URL's serialized form changes based on whether or not you have called hashCode() yet?

My superficial understanding is that URL tries to make 2 urls that resolve to the same host equals (I know, that's 😱, OkHttp has a nice documentation page about it here).  Calling hashCode() initializes a field that is then serialized.
So I think it's fair to say it's a serialization issue. If the serialization code called hashCode() to force initialize the field then it would be fine I guess. hashCode is maybe correct but the serialization code certainly not. Or maybe it is by design that if no DNS has happened the URL is serialized differently (😱 again).


DevDependencies

Hi, is there a reason why all the packages are installed as dependencies but not as devDepensencies? I I think the babel, loaders, eslint all the test related packages could be installed using --save-dev

If you won't publish the project to npm as a library then it doesn't matter whether to use devDependencies or dependencies.

It would be nice to understand which dependencies are only required for local development and testing (devDependencies) ... especially, with all the audit vulnerabilities that result from a new app created from rekit.  If all the vulnerabilities were for devDependencies, I would still consider using rekit, but since I don't know, I'm leaning towards create-react-app.

Rekit is based on create-react-app which just also put all deps in dependencies. Reason is just as I mentioned above:

If you won't publish the project to npm as a library then it doesn't matter whether to use devDependencies or dependencies.


"yarn dev" fails (fedora 36, node version 16.13.0)

yarn run v1.22.17
$ next dev
ready - started server on <IP_ADDRESS>:3000, url: http://localhost:3000
info  - Loaded env from /home/ekayalar/candytest2/next-candy-machine/.env.local
info  - Using webpack 5. Reason: Enabled by default https://nextjs.org/docs/messages/webpack5
info  - automatically enabled Fast Refresh for 1 custom loader
Error: error:0308010C:digital envelope routines::unsupported
at new Hash (node:internal/crypto/hash:67:19)
at Object.createHash (node:crypto:130:10)
at BulkUpdateDecorator.hashFactory (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:138971:18)
at BulkUpdateDecorator.update (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:138872:50)
at OriginalSource.updateHash (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack-sources3/index.js:1:10264)
at NormalModule._initBuildHash (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68468:17)
at handleParseResult (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68534:10)
at /home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68628:4
at processResult (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68343:11)
at /home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68407:5
Error: error:0308010C:digital envelope routines::unsupported
at new Hash (node:internal/crypto/hash:67:19)
at Object.createHash (node:crypto:130:10)
at BulkUpdateDecorator.hashFactory (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:138971:18)
at BulkUpdateDecorator.update (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:138872:50)
at OriginalSource.updateHash (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack-sources3/index.js:1:10264)
at NormalModule._initBuildHash (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68468:17)
at handleParseResult (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68534:10)
at /home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68628:4
at processResult (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68343:11)
at /home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68407:5
node:internal/crypto/hash:67
this[kHandle] = new _Hash(algorithm, xofLen);
^
Error: error:0308010C:digital envelope routines::unsupported
at new Hash (node:internal/crypto/hash:67:19)
at Object.createHash (node:crypto:130:10)
at BulkUpdateDecorator.hashFactory (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:138971:18)
at BulkUpdateDecorator.update (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:138872:50)
at OriginalSource.updateHash (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack-sources3/index.js:1:10264)
at NormalModule._initBuildHash (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68468:17)
at handleParseResult (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68534:10)
at /home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68628:4
at processResult (/home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68343:11)
at /home/ekayalar/candytest2/next-candy-machine/node_modules/next/dist/compiled/webpack/bundle5.js:68407:5 {
opensslErrorStack: [ 'error:03000086:digital envelope routines::initialization error' ],
library: 'digital envelope routines',
reason: 'unsupported',
code: 'ERR_OSSL_EVP_UNSUPPORTED'
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Only extra thing I did apart from your guide: Installed yarn into metaplex/js/packages/cli because uploading assets failed, but it was fixed when I installed the yarn into cli.

hi there - i belive you are using the wrong version of node
'node -v' to confrim
try 'nvm use 14' to correct
gl


Typo "apsdoc<EMAIL_ADDRESS>

Bold with escape characters
https://docs.microsoft.com/en-us/sql/t-sql/statements/permissions-grant-deny-revoke-azure-sql-data-warehouse-parallel-data-warehouse?view=aps-pdw-2016-au7

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Verified the update; merging.


[FEATURE] hints section of features to use on your profile

Description
Many people only add links to their profile because they are unaware of the other features like milestones, testimonials and events etc
When the PR for forms is completed, there will be a sub navigation, so this can have its own page of "profile suggestions" (arrow in screenshot below)

Screenshots
No response
Additional information
No response

Hi, I would like to work on this issue if someone can guide me a bit.

This is a very good idea. Would love to see this implemented.

Hello, I would be glad to work on this. can you please assign this to me?

I added a list of missing sections to the progress bar, maybe links could be added to the docs for each section

I added a list of missing sections to the progress bar, maybe links could be added to the docs for each section

Great Idea!
We could to update the profileSections array to have the title and the links for each
  let profileSections = [
    "bio",
    "links",
    "milestones",
    "tags",
    "socials",
    "location",
    "testimonials",
  ];

Something like this :
  let profileSections = [
    {title:"bio",
    link:"actual link",
    },
  ];

then update the filter to
progress.missing = profileSections.filter(
    (property) => !Object.keys(profile).includes(property.title)
  );

Sounds great 👍

Hi, I would like to work on this issue if someone can guide me a bit.

I have put the basic overview of what could be done. I cannot assign you but you can take the issue. Let me know if you need any help

@Landouda would you still like the issue? if not I will assign to @MAYANKpandey14

@eddiejaoude  yes please, I would still like to work on this issue.

Sure, I have assigned it to you @Landouda

I almost am done with the implementation of the hint feature. I created a hint component and extended the profileSection object to include the URL, the description and the targeted mdx code scippets.
here is a glimpse of the UI, is there any feedback before I push the code?

@Landouda it is better to make a draft PR, so we can discuss the implementation before you get to far ahead

@eddiejaoude, thank you for this information. I have just created a PR and linked it this issue as a draft proposal.

This issue has been unassigned due to lack of activity.

can i work on this issue

can i work on this issue

@KushagraVarshney05 please familiarise yourself with our Contributing Guide which explains why this issue cannot be assigned to you.  Namely:

You have already been assigned an Issue and we only assign one Issue per person
The label on this Issue is status: blocked so it is not ready to be assigned to anyone.

As mentioned in the Description of this Issue this is also dependent on #6998 being merged before it can be worked on.

Closing this as we now have a progress dashboard (and we will have a full on wizard coming soon)


Fix _enterprise_repo include in enterprise_dashboard.

Description
Removed an invalid trailing .rb from sensu::enterprise_dashboard's inclusion of sensu::_enterprise_repo
Motivation and Context
sensu::enterprise_dashboard didn't work. Now it does.
Fixes #448
How Has This Been Tested?
Kitchen
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

@jrb thanks for the fix! 👍


About the training with the 91 image dataset and fine-tuning with the General 100 dataset

One last question Saafke,
when we train a network from scratch we train it with the 91-images datasets.When we have to fine-tune it we have to train the network with general 100 dataset and with 91 images dataset or only with general-100 dataset?

In the original paper and my code, we only use the general-100 dataset for finetuning. During the finetuning stage, the learning-rate is halfed (compared to the training stage).
But you raise a valid point, you could actually use both datasets during finetuning! Why don't you try out, and see which strategy works better? :)


Added methods to allow the parsing of legacy files using streams

Currently if you want to parse legacy files you seem to only be able to use parser.ParseString(string xml, bool namespaces), which just converts that string to a stream under the hood. I've already got a stream and I just wish to be able to ignore the headers

By "Legacy Files" I mean files without the correct namespace - as referenced by https://github.com/samcragg/sharpkml/blob/master/docs/LegacyFiles.md

Thanks so much for that, much appreciated. I'll create a release now so it'll be on NuGet shortly.


Cannot trigger animation on iOS device

<TouchableOpacity onPress={this.greenHeartPress}>
  <Animation
      ref={green => {this.green = green}}
      style={{marginTop: 40, left: 150, height: 100, width: 100}}
      source={require('./bouncing_heart.json')}
   />
</TouchableOpacity>

This snippet behaves great on the simulator but I can only get the animation to trigger once (if lucky) on physical device. Has anyone seen this issue?

resolved this issue by removing absolute positioning


wasm-ld: not found

When trying to build to wasm, I get an error that wasm-ld is not found.
I believe this could be solved by adding llvm-<version>/bin to the PATH.
In my own script I was doing this before and it did the trick. Although it may only be the case on ubuntu.
echo "/usr/lib/llvm-14/bin" >>$GITHUB_PATH
export PATH="/usr/lib/llvm-14/bin:$PATH"

Curiously that is what should be done already: https://github.com/laytan/setup-odin/blob/fbed82c2fbba4012331d14d89eae76953a4bcb0c/src/index.js#L181 I will have to see where it is going wrong.

It does not add it when you are pulling a pre-compiled release (cause llvm isn't needed with releases), is that what you are doing?

yes, just this on ubuntu-latest
- uses: laytan/setup-odin@v2
  with:
    token: ${{ secrets.GITHUB_TOKEN }}
    release: latest

Aha, then I don't really know how to fix this, the release just bundles llvm and not any of the wasm-ld or other binaries. And if I make it still install llvm alongside the release it kinda defeats the purpose.
I think the ubuntu runners come with llvm 14 installed, so maybe I can just add it to path in that case but idk.

I think the ubuntu runners come with llvm 14 installed, so maybe I can just add it to path in that case but idk.

This is what I was I'm naively doing in my own script.
If Odin releases would switch to building with LLVM 17, will it be a problem if wasm-ld from 14 is used?

I don't think that is a problem, you can in general use older linkers with newer code (as long as the version difference isn't enormous). The next Odin release will be LLVM 17 btw.
I think adding the preinstalled llvm to path is ultimately the best thing here.