support for dotnet 5 latest version

Description
New tool, Bug fixing, or Improvement?
Please include a summary of the change and which issue is fixed. Also include relevant motivation and context.
For new tools, please provide total size and installation time.
Improvement: For dotnet 5 readiness on day0, adding support for latest version of dotnet 5, if it will not already be installed(this will happen when it becomes ga)

Related issue:
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

@tejasd1990 , could you please split this PR to three separate PRs? (Windows / Linux / MacOS)?
It will simplify merging later.

Hi @maxim-lobanov , can you update once you have made the changes? Thanks

@tejasd1990 , JFYI:

https://github.com/actions/virtual-environments/pull/1995
https://github.com/actions/virtual-environments/pull/1994


[Bug] Convert LatLng to Point correctly

@scolsen @JSunde  PTAL?

Codecov Report

Merging #1307 (eeba52f) into master (057b9fe) will increase coverage by 0.00%.
The diff coverage is n/a.

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


upgrade spring boot version and devon4j

refer https://github.com/devonfw/devon4j/pull/233

@vapadwal thanks for this PR. Looking very good. 👍
I have one concern that I commented on the code. Could you have a look at improve this or explain when this is not required? Then we can approve and merge.

Basically this code for batch is going to be deleted in PR https://github.com/devonfw/my-thai-star/pull/321 - when this PR get merged. So i have just made it compiler error free


Apple Silicon support

liblibvhacd.dylib is compiled for Intel only.
I'd contribute a PR but this package seems to use a fork of the original -- or at least the entry points aren't in the current master at https://github.com/kmammou/v-hacd.git

Hi, thanks for pointing that out--this repository is currently undergoing some changes; please hold as we sort it out! I've filed an internal ticket for further investigation and follow-up.
[Ticket#: AIRO-1610]

@benoithudson as a question in the interim as the repository gets reorganized, have you made (or do you plan to make) any changes to the source code in order to recompile the library for Apple Silicon support?
The source code is currently at jasonmeisel/v-hacd-unity, a fork of the source you mentioned.

No changes at all seem to be required to support M1, just recompiling works fine.
I had to add extern "C" entry points to the C++ code by hand to match the [DllImport] you use in C#. I can't see it in jasonmeisel's fork either.

@benoithudson, this repository will be archived in favor of Unity-Technologies/VHACD, which includes the source code for the compiled libraries. I am moving your issue over to the new repo here: https://github.com/Unity-Technologies/VHACD/issues/5.


Unexpected keyword argument 'numThreads' when reading Java Zarr with Python

Hi,
When creating Blosc compressor with
Compressor compressor = CompressorFactory.create("blosc", "cname", "zstd", "clevel", 9, "blocksize", 0, "shuffle", 1);

A new optional parameter for Blosc "nthreads" is added into the .zarray metadata file as
"numThreads" : 1

And this is what I got when reading this Zarr with the Python API
  File "numcodecs\blosc.pyx", line 548, in numcodecs.blosc.Blosc.__init__
TypeError: __init__() got an unexpected keyword argument 'numThreads'

Since 'numThreads' is in metadata files it's not possible anymore to read the resulting Zarr using Python
It's technically a Python error, but since JZarr data has to be compatible with the Python API I think it's more of a Java issue (or did I just do something wrong ?)

Thanks for the issue, @ZigzagAwaka. #15 should solve this issue once released. (If it doesn't, please let us know)


ci: adds piccolo to doc build

Pull Request Checklist
This PR fixes a documentation build error where piccolo is expected to be found but is not installed.

[ ] New code has 100% test coverage
[ ] (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
[ ] (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR

Description
Close Issue(s)

@cofin Did you forget to include some commits here? :upside_down_face:

@cofin this should be fixed in #1993. Piccolo had been removed as a dev dependency in #1963 which was causing this issue, so adding it back in should solve it.


Cannot parse ado in without any statements

The formatter is not happy about a file we have in our work codebase.
The file compiles fine with the latest compiler, but purs-tidy fails to parse it, and the error is just:
Unexpected '{'

The file is 4k lines long so it's hard to pinpoint exactly where the issue is, and it would be great if we could get a line number or some indication of where the parsing fails.

As an aside, @MonoidMusician found the culprit for the error, it's because of an empty ado:
ado
in { someField }

The CST parser doesn't seem to think that should be valid (and neither do I)

I agree, and I think an empty ado/in is pretty silly, but the compiler allows it and it desugars to pure.

Minimal repro:
module MuchAdoAboutNothing where

much = ado
  in Nothing

data Maybe a = Just a | Nothing

pure a = a

gives Unexpected end of indented block but compiles fine with purs compile
To see the specific error from OP, you need this:
a = much where
  much {} = ado
    in Nothing

In particular, it needs to be in a where and the error is targeting the function argument (much y = … gives Unexpected identifier y, nothing else is valid there). But that's just funky errors getting pushed backwards due to backtracking :)
I can put up a PR soon

This was addressed in the 0.9.3 release (along with printing locations for errors).


HTTP Basic Auth via IIS

I understand the security problems around HTTP basic authentication.
From the README notes:
"ASP.NET Security will not include Basic Authentication middleware due to its potential insecurity and performance problems. If you host under IIS you can enable it via IIS configuration."
What does this mean? I enabled HTTP Basic in IIS and provided a Realm.  IIS is authenticating the requests and sending them to Kestrel, but Kestrel returns a 401. What needs to be done on the .NET Core side to make this work?
I have an Intranet app using Windows Authentication which I am trying to migrate to .NET Core 2.1
95% of my traffic is intranet users, however I have a few automated processes using (linux) clients that do not support Windows Authentication. I would like to use Basic to accommodate this. As I said this is intranet only and using SSL/TLS.

Do you already have the site working with Windows Auth? The instructions for IIS Basic Auth aren't any different, it's only a matter of checking the Basic Auth checkboxes when installing and configuring it.

Yes, Windows Auth is working.
I couldn't find any documentation that says IIS Basic Auth uses the same instructions. Are you saying that IIS passes the MS-ASPNETCORE-WINAUTHTOKEN header to Kestrel even when using IIS basic auth?

Correct, IIS processes the Basic credentials, converts it to a windows identity, and passes that to ASP.NET via MS-ASPNETCORE-WINAUTHTOKEN. Digest auth works the same way.

Thank you, that is good info. Is there a way to debug to see if this is working?
Here are some of the relevant bits:
`services.AddAuthentication(IISDefaults.AuthenticationScheme);
services.AddMvc(options =>
{
options.Filters.Add(new ResponseCacheAttribute { NoStore = true, Location = ResponseCacheLocation.None });
})
.AddJsonOptions(options => options.SerializerSettings.ContractResolver = new DefaultContractResolver())
.SetCompatibilityVersion(CompatibilityVersion.Version_2_1);
services.AddAuthorization(options =>
{
options.AddPolicy("Authenticated", policyBuilder => policyBuilder.RequireAuthenticatedUser());
});`
On my controller I have:
[ApiController, Authorize(Policy = "Authenticated")]
Works with Windows Auth but not with IIS basic.

I created some middleware to log the headers on the request coming from IIS.
MS-ASPNETCORE-WINAUTHTOKEN is not present on a request coming from IIS authenticated with basic auth.
Connection:Keep-Alive Content-Type:application/json Authorization:Basic <REDACTED> Host:redacted.com User-Agent:Mozilla/5.0 (Windows NT; Windows NT 6.1; en-US) WindowsPowerShell/5.0.10586.117 MS-ASPNETCORE-TOKEN:a9b04e04-fc85-480e-9134-723d16689f5d X-Original-Proto:http X-Original-For:<IP_ADDRESS>:58064
With Windows Auth, the header MS-ASPNETCORE-WINAUTHTOKEN and everything works fine.

I think this tracks back to this line:
https://github.com/aspnet/IISIntegration/blob/b6c311c14d3c3cf89b1b28d52efa0088a580213e/src/AspNetCoreModuleV2/OutOfProcessRequestHandler/forwardinghandler.cpp#L895

@pakrym can you confirm this for V1 and V2? I thought this worked for Basic as well. Does it work for in-proc?

Assigning to @pakrym pending an answer to the above question.

I don't know the answer from the top of my head, we would need to verify.
Filed: https://github.com/aspnet/IISIntegration/issues/1429

Closing this issue here because as long as a lower level component such as AspNetCoreModule flows the token, the components in this repo should automatically light up.


Accept multiple labels for affinity

Some operators deploy multiple components and use the additional selector label (ComponentSelector) to distinguish components.

do we need to pass multiple labels, or wouldn't it be enough that the caller uses a component selector instead of a global service selector, like glance-api-external or glance-api-internal?

Some operators are already using ComponentSelector but these put only component name (eg. api) instead of the full process name(eg. heat-api).
We can adjust that but I tend to follow that pattern because having full service name along with ServiceSelector sounds redundant.

lgtm


Atom crashes with unsaved file under MacOS

Description
Atom recently starts to crash randomly when working on unsaved files. This way I often loose work.
Steps to Reproduce

Open a new file
Take some notes or write a snipped that you don't want to save
See Atom crash and loose part of your work

Expected behavior: [What you expect to happen]
It should not crash.
Actual behavior: [What actually happens]
It crashes.
Reproduces how often: [What percentage of the time does it reproduce?]
Recently almost every time when working on an unsaved file. 80%+
Versions
Atom    : 1.33.0
Electron: 2.0.11
Chrome  : 61.0.3163.100
Node    : 8.9.3
apm  2.1.2
npm  6.2.0
node 8.9.3 x64
atom 1.33.0
python 2.7.15
git 2.17.2
macOS Mojave 10.14.1

Have you reproduced this in Safe Mode? I notice you chopped off the part of the template that relates to that. I use unsaved files all the time on macOS 10.14.1 and do not experience this problem, so it could be a package that you're running that is causing the problem.
Additionally, if you are able to reproduce this in Safe Mode, please follow the instructions for collecting crash logs and either attach them here or, if you'd prefer to share them privately, email them to<EMAIL_ADDRESS>

@lee-dohm I'm getting this issue. The app freezes and I'm not able to create an issue as well.

@sidhyatikku Please open a new issue and completely fill out the issue template so we can be sure that we have all the information we need.

@lee-dohm The app is frozen and I’m not able to send an issue. I’m just a novice coder and I’ll not be able to provide detailed inputs.

@sidhyatikku You can click the "New Issue" button in the upper right of this web page though 😀

If you need help writing a bug report (something that anyone can do, no coding experience required), you can follow this guide that I wrote to help.

@lee-dohm Yes, the editor also just crashed in safe mode.
This is the crash log when in safe mode: https://gist.github.com/ospaarmann/1bc96c38dbbfac1075ed0a1f04bf1c2a#file-crash_log_in_safe_mode-log
This is the crash log when not in safe mode: https://gist.github.com/ospaarmann/1bc96c38dbbfac1075ed0a1f04bf1c2a#file-crash_log_not_in_safe_mode-log
If you need any more information please let me know.

Thanks @ospaarmann - do you happen to have multiple locales set in the spell-check packages settings?

Hey @rsese - yes. I have set the locales to en-US, de-DE.

Ahh ok, thanks for confirming - since you're using multiple locales and since your crash log looks similar, we'll close this as a duplicate of https://github.com/atom/atom/issues/15912 and you can subscribe there for updates if you'd like.


🐛  (radix-ui) do not import flex directly [b]

Something off with the order of operations it looks like. But I am not sure.
This will probably impact tree-shaking obviously, but testing to see if it actually fixes the full page re-render.
Closes #2339

This pull request has been deployed to Vercel.

Latest commit:
45b20b3

✅ Preview:
https://jeromefitzgerald-f63d4kms4-jerome5.vercel.app

🔍 Inspect:
https://vercel.com/jerome/jeromefitzgerald/FzyciGKT4ZpeHPDr6kzybQQfTWMo

View Workflow Logs

https://github.com/radix-ui/themes/pull/496


Redesign the log

Describe the feature
This is a low priority, and I found that when there is too much source code, the log needs to have the following functions to make it easy to quickly locate the source code with a mouse click
Relevant code
No response

It is your responsibility to write log messages that are clear and sufficiently convey the location; I am always able to find log messages just by searching for the message. We are not going to force people to have information they do not want in their log messages; you can easily write a helper function that does this if you want that behavior.

It is your responsibility to write log messages that are clear and sufficiently convey the location; I am always able to find log messages just by searching for the message. We are not going to force people to have information they do not want in their log messages; you can easily write a helper function that does this if you want that behavior.

ok


BUG: optimize.least_squares: Raise an exception if the initial guess x0 is complex

See #6506
Should I also add similar checks to other solvers?
Edit: Sorry, wrong issue number.

On the other hand, least_squares seems to be the only one relevant for this case; other functions are either written in Fortran/C or do not require an initial guess.

LGTM.


Update EASEY Design System Documentation

Update EASEY Design System Documentation as needed

Design System Updated - 2/23/23: https://cvpcorp.zeroheight.com/styleguide/s/59398/p/81a343-easey-design-system


Details Popup Window

Added functionality to the project details buttons to display the popup with details about the project when the user clicks on it.
Added functionality to the close (X) button of the popup that removes the popup when the user clicks on it.
Updated the main page of the projects section dynamically.

Hi John,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that was worked on by an individual instead of a paired team. Please ensure to work on this project with a partner, and kindly ask for another review when completed.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Thanks😊


[EventRequest] Codeunit 12 "Gen. Jnl.-Post Line" OnAfterApplyCustLedgerEntry

Please add the following event
[IntegrationEvent(false, false)]
local procedure OnAfterApplyCustLedgerEntry(var GenJournalLine: record "Gen. Journal Line"; var OldCustLedgEntry: record "Cust. Ledger Entry"; var NewCVLedgEntryBuf: record "CV Ledger Entry Buffer")
begin
end;

to function ApplyCustLedgEntry
        TempOldCustLedgEntry.Delete;

        // Event
        OnAfterApplyCustLedgerEntry(GenJnlLine, OldCustLedgEntry, NewCVLedgEntryBuf);
        // Event

        Completed := FindNextOldCustLedgEntryToApply(GenJnlLine, TempOldCustLedgEntry, NewCVLedgEntryBuf);
    until Completed;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Updating mbed-os to mbed-os-5.3.6

Please test/merge this PR and then tag Master with mbed-os-5.3.6

Looks good 👍


Pass rate and other settings to parent classes

See https://github.com/tensorflow/tensorflow/issues/10845

Can one of the admins verify this patch?

I signed it!

@tensorflow-jenkins test this please


[New game]: Poker Game

🎮 Game Request
Game logic and basic description:-A card game that combines strategy, skill, and luck.
Point down the features
Game points:-
1.The structure includes sections for two players and the community cards with a deal button.
2. Basic styling to center elements and style the cards.
3.Creates and shuffles a deck of cards.
4.Deals two cards to each player and five community cards.
5.Renders the dealt cards to the screen.
6.Uses setTimeout to simulate the stages of dealing the community cards (flop, turn, river).
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Swarnav-Kumar !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

assign

/assign

/assign

Hello @Swarnav-Kumar, Time's up!⏰
Sorry for closing your issue!
But it's more than a week since we haven't received anything from your side 😢 .
Come up with new ideas, create a new issue and make sure you finish it within a week! 🔥
All the best! 🚀
Happy Hacking! 💗

Hey @Swarnav-Kumar ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Report a Problem on Visual Studio setup

How do you report a problem when visual studio fail to install or fail to open or in general if Report A Problem menu option integrated in Visual studio cannot be reached?
As Far as I know a bug or problem is not a suggestion which I think visual studio developers also think same as they immediately click close on suggestions that report bugs!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7254052d-2d37-509e-e6c8-d5674ea775ca
Version Independent ID: a2acc423-07d3-409c-7de1-21dd88e3cdcd
Content: Report a problem - Visual Studio
Content Source: docs/ide/report-a-problem.md
Product: visual-studio-windows
GitHub Login: @seaniyer
Microsoft Alias: seiyer

Thank you for presenting this scenario to us! This is something that we are aware of and trying to find a way to mitigate, while still keeping the actionability of reports that we get with the Feedback Tool.
As a workaround, you can also report a problem from the Visual Studio Installer.
Please also see https://developercommunity.visualstudio.com/idea/398744/problems-cannot-be-reported-when-the-ide-doesnt-wo.html
As a side note, please log suggestions and problems about the product through Developer Community or the Feedback Tool. This is a medium that is used for problems about docs, and it's harder to track and make the connection.
Maria Ghiondea
Visual Studio Feedback System

As a workaround, you can also report a problem from the Visual Studio Installer if that one is working.

Installer fail to install, how to report a problem?

I'd suggest first searching the Installation knowledgebase https://visualstudio.microsoft.com/vs/support/?c=Download+and+Installation.  If you are not able to find an existing solution, file a support request by following the options at the bottom of that page.


[Proposal] BOS should be ignored by default in the activation store for SAEs not trained on the bos (and future SAEs trained in SAE Lens)

Proposal
Ensure that the activation_store.from_sae method creates a store that will remove BOS activations before adding them to the buffer. Setting default seqpos slice to ignore the first token on all SAEs seems like a reasonable way to do this.
Motivation
Some SAEs aren't trained on BOS tokens (as is the case with GemmaScope) and it's probably the case that this should be the default in SAE training (and may increase performance)
Checklist

[x] I have checked that there is no similar issue in the repo (required)

@chanind @curt-tigges let me know if that doesn't make sense.

I think the seqpos solution will only work for cases where every sequence is the same length, which is the case for OthelloGPT but isn't the case for most LLM datasets. For instance, when batching, we might end up with a BOS in the middle of the input rather than only at the front. I think we need to look for BOS tokens and mask them somehow, since there's also no guarantee that there's the same number of BOS tokens in a given batch. This is also somewhat problematic for pre-cached datasets since we don't currently save the special token indices. We also may want to exclude other tokens depending on how the training is set up, e.g. if the dataset includes EOS tokens or SEP tokens we probably want to exclude those too (or at least want the option to)
IMO the most robust solution is to return both activations AND a mask from the activations store / cached activations and have the SAE training exclude masked activations from loss calculation. In the mask we could define an int value for each type of special character (e.g. 1 = BOS, 2 = SEP, 3 = EOS) and then the config can specify which of these to exclude from training / eval. Sometimes tokenizers have the same token for multiple purposes (e.g. BOS = EOS = SEP) - we could have a scheme for this where, e.g. 12 = BOS + SEP, 123 = BOS + EOS + SEP, 13 = BOS + SEP, etc... Maybe I'm overthinking this, but I don't see an easy way to handle this without keep track of a token mask of some sort.


Add the wasm entrypoint defined in Crystal

Currently Crystal does not define an entrypoint for wasm programs and it is required to add the crt1.o file from wasi-libc on the link flags, which is odd and non-standard. This PR removes the need of including crt1.o and now wasm programs can be linked with: wasm-ld hello.wasm -o hello-final -L wasm32-wasi -lc -lpcre -lclang_rt.builtins-wasm32.
It also opens the door to customize the entrypoint for "reactor"-style programs in the future.
Original implementation: https://github.com/WebAssembly/wasi-libc/blob/659ff414560721b1660a19685110e484a081c3d4/libc-bottom-half/crt/crt1.c

Having some explanatory comments like in the original imp would be nice.

Does wasm always expect _start and not main? How do fun _start and fun _main interact? I think these should be addressed in source comments too because this is the first time we ever have _start.

I added some comments explaining it. Please let me know if still unclear.


Avoid boolean-trap rules for ConfigParser get() methods

See https://docs.python.org/3/library/configparser.html#fallback-values.
Fallback value can be used both as a positional argument and as a keyword argument. But it seems to me that using it as a positional argument is more readable.

Thanks! Yeah this seems fine to me.


Pulling in LivewireCalendar breaks Jetstream Livewire components

Hi there,
This is a hobby for me and infact, the first issue i've ever submitted on any public repo before, although i have tested this back and forth a few times.
System Versions (all at time of writing)
OS: MacOS using Valet
Laravel: Latest
Livewire: Latest
Replication

Brand new laravel app with --jet flag using latest laravel installer
No additional features added to jetstream in test app
Compile assets and migrate database
Load the "Profile" page from the dropdown to prove it works
Pull in livewire calendar as require from composer following readme
Try to reload the profile page

Thoughts
The error displayed in my app was different, it couldn't locate the declared Livewire Components from the jetstream package, although an error is still thrown with a fresh install
Although i'm not confident enough to make a Pull Request here, i have dismantled the package and made it local (out of the vendor DIR and with no ServiceProvider etc. and it all plays nicely.
Screenshot

Hope this is of some help to you!

same issue with a fresh install

Same here. After installing on fresh Laravel 8 I get this:
Livewire\Exceptions\ComponentNotFoundException
Unable to find component: [navigation-menu]

Same here. After installing on fresh Laravel 8 I get this:
Livewire\Exceptions\ComponentNotFoundException
Unable to find component: [navigation-menu]

This should be fixed in 2.1.0.
Thanks for your interest in the package!

This should be fixed in 2.1.0.
Thanks for your interest in the package!


Wiki Tutorial of Freezing Columns

looking for just such a thing right now - can't seem to find anything yet

ok - looks like it's setFixedColumnCount on the behaviors thing:
https://fin-hypergrid.github.io/core/2.1.15/doc/Behavior.html
https://github.com/fin-hypergrid/core/blob/master/src/behaviors/Behavior.js#L916


sanitycheck: support requirements on env variables

Some boards depend on environment variables so we want to make sure we do
not attempt to build boards requiring additional setup.
Add the section below into the board YAML file, sanitycheck will check the
environment and will only run tests on that board if the variables are
defined.
env:
 - VAR1
 - VAR2

Codecov Report

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

@@            Coverage Diff             @@
##           master   #10695      +/-   ##
==========================================
- Coverage   53.24%   52.89%   -0.36%
==========================================
  Files         214      224      +10
  Lines       25828    26409     +581
  Branches     5695     5779      +84
==========================================
+ Hits        13753    13968     +215
- Misses       9760    10099     +339
- Partials     2315     2342      +27

Impacted Files
Coverage Δ

include/posix/posix_sched.h
0% <0%> (-100%)
:arrow_down:

lib/posix/pthread_barrier.c
0% <0%> (-100%)
:arrow_down:

lib/posix/pthread_cond.c
62.5% <0%> (-20.84%)
:arrow_down:

lib/posix/pthread.c
68.03% <0%> (-12.33%)
:arrow_down:

lib/posix/pthread_mutex.c
76.38% <0%> (-1.39%)
:arrow_down:

include/sys_clock.h
100% <0%> (ø)
:arrow_up:

boards/posix/nrf52_bsim/time_machine.c
40.98% <0%> (ø)

boards/posix/nrf52_bsim/main.c
85.36% <0%> (ø)

drivers/clock_control/nrf5_power_clock.c
14.65% <0%> (ø)

boards/posix/nrf52_bsim/k_busy_wait.c
0% <0%> (ø)

... and 7 more

Continue to review full report at Codecov.

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


Auto-refresh experiment data

It would be nice if the (enrolling) experiments can be auto-updated.
Currently, in order to see the updates on enrollment, the user needs to either refresh the page or click on another tab (e.g. Feature Flags) and then click on the "Experiments" tab.

Also, in order to see the updates on the experiment detail page, the user needs to click on the "Experiments" tab and click on the experiment again to see the change.

I'm not sure what would be the best way to implement this in UpGrade, but I believe this will improve the user experience.

This could be accomplished by refetching/polling for new data on every x seconds while the component page is open. Should I pick this up @amurphy-cl ?

That'd be great!

Restating it in 'user story' language helps me out, I'm fully brainwashed in this part of agile. is this what we are saying:
As a user on the details page,
I want to see /enrollment and /details data updated automatically,
So that I don't have to click refresh.
As a user on the home page,
I want to see /enrollment data updated automatically,
So that I don't have to click refresh.
@amurphy-cl @zackcl

I'm pretty sure we have some internal code in cli-services that shares requests to avoid re-sending them.  @kschaefe can probably point you to it.

Yes, the @UseInFlightCache[1] decorator in TypeScript will prevent creating a new call while the current one is active.  It is currently limited to calls that take no parameters, but we really need it to work will all idempotent calls.  If you have a need for us to hash and cache calls with params, just let us know.
[1] https://github.dev.carnegielearning.com/CarnegieLearning/cli-services/blob/master/libs/cli-services/src/lib/rest/common/use-in-flight-cache.ts

Thanks for sharing that, I think the use-case for these particular calls is just to give up and cancel the previous request instead of sharing the response from the previous, as typically we're wanting the latest data. Swapping switchMap in place of mergeMap seems to be working for me on this where needed.
@kschaefe can you show some examples of where some of these custom typescript decorators are being used?

We currently have only one use.  We haven't gone through to attach this in all of the places we could.
https://github.dev.carnegielearning.com/CarnegieLearning/cli-services/blob/52a4f9f685e18c76d189e7bdd25ce6a00742310e/libs/cli-services/src/lib/rest/FIDO-services/authentication.service.ts#L28


chore: fix global runtime

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (b497e57) 70.45% compared to head (651acb1) 70.46%.

Files
Patch %
Lines

taos-query/src/lib.rs
85.71%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #305   +/-   ##
=======================================
  Coverage   70.45%   70.46%
=======================================
  Files         114      114
  Lines       34777    34777
=======================================
+ Hits        24502    24504    +2
+ Misses      10275    10273    -2

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

Close #278


Feature/presigned urls gcp

Codecov Report

Merging #220 (4bab45f) into develop (4f47809) will increase coverage by 0.35%.
The diff coverage is 97.36%.

@@             Coverage Diff             @@
##           develop     #220      +/-   ##
===========================================
+ Coverage    54.49%   54.85%   +0.35%
===========================================
  Files           43       43
  Lines         4485     4525      +40
===========================================
+ Hits          2444     2482      +38
- Misses        1891     1894       +3
+ Partials       150      149       -1

Flag
Coverage Δ

unittests
54.85% <97.36%> (+0.35%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/plugins/storage/storage/storage.go
73.09% <97.36%> (+7.91%)
:arrow_up:

pkg/worker/subscription_worker.go
50.00% <0.00%> (-8.34%)
:arrow_down:

pkg/worker/schedule_worker.go
0.00% <0.00%> (ø)

pkg/plugins/gateway/lambda/lambda.go
69.72% <0.00%> (+0.54%)
: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 4f47809...4bab45f. Read the comment docs.


Target Info fails to fetch data

The feature of Target info fails to operate and fetch the information requested.

@soumyashaw May be you messed up with your scripts by applying minizing browser feature. :)

Working Now


Confusing use of "Validation Set" in beginner example

Thank you for submitting a TensorFlow documentation issue. Per our GitHub
policy, we only address code/doc bugs, performance issues, feature requests, and
build/installation issues on GitHub.
The TensorFlow docs are open source! To get involved, read the documentation
contributor guide: https://www.tensorflow.org/community/contribute/docs
URL(s) with the issue:
https://www.tensorflow.org/tutorials/quickstart/beginner
Description of issue (what needs changing):
Tutorial makes use of model.evaluate(), and the documentation says that this is usually done on a "Validation Set". Everything else I read (glossary, docs for model.fit()... including validation set parameters) points to this relating to a "Test Set" since it occurs after the training phase, and the parameters passed are "x_test" and "y_test". The confusion is unhelpful to beginners. Change from "Validation Set" to "Test Set"?
Correct links
n/a
Parameters defined
n/a
Returns defined
n/a
Raises listed and defined
n/a
Usage example
n/a
Request visuals, if applicable
n/a
Submit a pull request?
No. I'm a beginner, so I don't want to do anything, lest I create more confusion.

Hi, @omatai let me help you clarify.
We usually perform training on data using model.fit(x_train, y_train). Since we know this data we cannot perform testing on this data. Hence we create two sets.

One is called the validation set and other called the test set. We validate our learning on the validation and performing testing on the test set. Here for simplicity, we did not divide our data into three parts.

I think rather than skipping it maybe raise a PR which shows training, validation and testing all three done clearly.

Would that be fine @omatai ?

As a beginner quickstart example, it might help to say:

Model.fit() has fit the model to the data using the training set, but we have not provided a validation set to model.fit() so that we can control the quality of fit. Lets bypass any concerns about over-fitting, and proceed to use model.evaluate() on a test set.

That is in line with my understanding of what the code is doing. I would keep the code the same, and adjust the documentation text.
From: Aditya Oke<EMAIL_ADDRESS>Sent: Wednesday, 22 April 2020 9:16 am
To: tensorflow/tensorflow<EMAIL_ADDRESS>Cc: Paul Qualtrough<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [tensorflow/tensorflow] Confusing use of "Validation Set" in beginner example (#38735)
Hi, @omataihttps://github.com/omatai let me help you clarify.
We usually perform training on data using model.fit(x_train, y_train). Since we know this data we cannot perform testing on this data. Hence we create two sets.

One is called the validation set and other called the test set. We validate our learning on the validation and performing testing on the test set. Here for simplicity, we did not divide our data into three parts.
I think rather than skipping it maybe raise a PR which shows training, validation and testing all three done clearly.

Would that be fine @omataihttps://github.com/omatai ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/tensorflow/tensorflow/issues/38735#issuecomment-617418014, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIQOMW5FCPOQXAJAFN4N5ALRNYEHJANCNFSM4MNAGXIA.

Ok Let me fix as you say. I will make it clearer in documentation.

@omatai I have fixed as you said. Please have a look at the pull request.

@omatai This PR link is added. I hope it solves the issue.

It is fixed. PR is merged.

I don’t think it is fixed in  a way that will prevent future confusion – the links to both Validation Set and Test Set describe different things: validation set is related to training phase; test set is related to testing phase. The comment relates to testing phase, so if validation set is going to be mentioned at all, it needs further explanation as to why… which could be that the standard naming convention used on Tensorflow is not strictly adhered to and not universal.
I don’t think I’m confused any more, but I expect others will be confused at some point. But hey: I’m reasonably rusty on all this… and have much higher priorities. Thanks for your time 😊
From: Aditya Oke<EMAIL_ADDRESS>Sent: Thursday, 23 April 2020 2:27 am
To: tensorflow/tensorflow<EMAIL_ADDRESS>Cc: Paul Qualtrough<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [tensorflow/tensorflow] Confusing use of "Validation Set" in beginner example (#38735)
@omataihttps://github.com/omatai This PR linkhttps://github.com/tensorflow/docs/pull/1549 is added. I hope it solves the issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/tensorflow/tensorflow/issues/38735#issuecomment-617812384, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIQOMW6SGCQ6FRMEU4IHG3DRN35DLANCNFSM4MNAGXIA.

@omatai I think the fix solves the problem as the definition for validation and test set has been interlinked. Please go ahead and close this issue as I think there is no confusion here. Thanks!


Outlook Calendar Tool - Recurring meetings get listed twice.

Electron build -f8cb3bd9
Steps to reproduce the problem:

Chat with tildy
Add Outlook Calendar Tool
Ask to list all emails from today (In my case I had 1 recurring meeting)

Notice that recurring meeting gets listed twice.

Also when I try to list details of the recurring meeting -  List details of Standup,  details of the meeting gets listed and which  I also get this message:
Unfortunately, I encountered an error while fetching details for the other Standup meeting from the Engineering calendar. If you need more information or assistance with anything else, let me know! 😊

Tool call is made with same event id twice:
Input
"List details of Standup"

Messages
<tool call> getEventDetails -> {"event_id": "AAMkADU5ZWEzNWE1LThlMTktNDAyYy05ZjdiLTExNDE3YWEzYzcxNAFRAAgI3M09r3LAAEYAAAAAvX4Q10ohJkmJ0yPnfZMK7gcA9tFJHAVLWUG0jSZ8iSqBOQAAAAABDQAA9tFJHAVLWUG0jSZ8iSqBOQAA10h1jQAAEA=="} <tool call> getEventDetails -> {"event_id": "AAMkADhmYzUxMmE5LTlkZmItNDJhOS04M2IyLTZlMzMwYTQ0ODY3MAFRAAgI3M09r3LAAEYAAAAAcdLA5eW0CUmdy4Qd1GmASQcAvfiIDcgqiEuNWjnnx5Z6TAAAAAABDQAAvfiIDcgqiEuNWjnnx5Z6TAABdtx5JAAAEA=="}

Also when I try to list details of the recurring meeting -  List details of Standup , after listing the details of the meeting I also get this message:
Unfortunately, I encountered an error while fetching details for the other Standup meeting from the Engineering calendar. If you need more information or assistance with anything else, let me know! 😊

This is not an issue.
We are listing the calendar entries for each of the calendar view we have configured.


Although not critical, I'd be prefer that the SPI controller's pins (MOSI, CLK) are configured back to the GPIO input mode when the controller is not active as done with the I2C controller. I think this should also minimize power consumption.

Added PM_DEVICE_RUNTIME base on i2c and other drivers


Use ghcr for supervisor and builder

@pvizeli any chance on getting this PR merged?

You can modify /usr/bin/supervisor_run and /etc/supervisor_scripts/common in your local devcontainer with the contents of this PR until it's merged and released.

Thanks - I didn't realise you could do this. Nice to get this working!


MINOR: Add unit test for SerDe auto-configuration

Follow up of (including) #6461 (first commit only -- original commits squashed)
If we are happy with this, we can merge #6461 and than I can rebase this and drop the first commit.

Call for review @tedyu @guozhangwang @bbejeck @vvcephei @ableegoldman @cadonna

Knows flaky test failed on Java11. Java8 passed.
Retest this please.

Rebased this, as #6461 was merged.

Java8 passed. Java11 failed (tracked via Jira).
Retest this please.

Call for second review. @tedyu @bbejeck @vvcephei @ableegoldman @cadonna (Ted had a look, but did not give a +1 yet).

+1


Fix: prevent editing/deleting already-deleted assistants

Description
Deleted assistants can still be accessed in former conversations. Their details can still be viewed.
At this time, when seeing the details of a deleted assistant:

there is no visual indication that it was deleted;
users can still edit them, saving actually revives the assistant;
users can "delete" them (no-op, confusing).

This PR adresses the 3 as a first effort by preventing edition of deleted assistants and visually indicating deletion.
We might want a deeper reflexion on what to do on deleted assistants (maybe a menu with "revive", etc.). This PR can be seen as a hotfix in the meantime.
Before:

After:

Risk
No real risk here
Deploy Plan
Deploy front

As per IRL, removing strikethrough and using a ContentMessage

LGTM, but I feel like we still display the instructions, right?

We do; I think it's better, and I gather @Duncid said it was ok

@philipperolet sitrep ?

Wow forgot to merge. Will take care of it


A11y Bug #1513640

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


Open Source Helps!

Thanks for your work to help the people in need! Your site has been added! I currently maintain the OpenSourceWuhan page, which collects all open source projects related to COVID-19, including maps, data, news, api, analysis, medical and supply information, etc. Please share to anyone who might need the information in the list, or will possibly contribute to some of those projects. You are also welcome to recommend more projects.
https://weileizeng.github.io/OpenSourceWuhan/world
Cheers!

Thank you for adding.
I will share on my Twitter account.


Ignore auto-generated jars.txt from Experiment module

Rationale
Earlier this year, we switched to auto-generate this file. Thus, we don't need it committed to Git.
Changes

Ignore the file

A customer asked about this on his dev machine this morning. I told him to ignore it in the list of changed/added files, and then noticed I had one sitting around too.


typo: Double word "Recovery"

@nschonni : Thanks for your contribution! The author, @rajani-janaki-ram, has been notified to review your proposed change.

#sign-off


Change plugins class names

Just wanted to get a bug out there.  All plugins class names should change to simply class Plugin(object): in order to support dynamic loading during custom incidents

Fixed in development.  Closing.


fix(tabs): fix inverted boolean storybook

Description
Fix inverted prop on storybook controller to work as a boolean

I will check out all the components on vue2 to confirm boolean value are correct.

I fixed all boolean controls I found that need to be fixed in our components. Will require review, thanks

Removed no-visual-test label to trigger percy check.


Cannot locate L4 plugin when committing ingress yaml.

I was validating the fix for #781.  The panic does not occur anymore, however, the ibmcloud plugin does not work (example simulator does seem to work).  I get the following looping in the logs and the routes never get added:
INFO[12-01|09:53:05] Connected to Docker                      module=controller/ingress/swarm client="&{scheme:http host:unix:///var/run/docker.sock proto:unix addr:/var/run/docker.sock basePath: client:0xc4206085d0 version:1.24 customHTTPHeaders:map[User-Agent:Docker-Client/1.24 (darwin)] manualOverride:false}" fn=github.com/docker/infrakit/pkg/controller/ingress/swarm.(*handler).Routes
INFO[12-01|09:53:05] Listing services from swarm              module=controller/ingress/swarm fn=github.com/docker/infrakit/pkg/controller/ingress/swarm.(*Routes).List
WARN[12-01|09:53:05] cannot locate L4 plugin                  module=controller/ingress/types vhost= spec="{Vhost: L4Plugin:ibmcloud/csy-infrakit-lb2 RouteSources:map[swarm:{\"CertificateLabel\":\"lb-cert-label\",\"Host\":\"unix:///var/run/docker.sock\"}] Routes:[{Port:8080 Protocol:HTTP LoadBalancerPort:80 LoadBalancerProtocol:HTTPS Certificate:0xc420269cb0}] Backends:{Groups:[group/workers group/managers] Instances:[]} HealthChecks:[]}" err="plugin not found:ibmcloud/csy-infrakit-lb2" fn=github.com/docker/infrakit/pkg/controller/ingress/types.Properties.L4Func.func1
WARN[12-01|09:53:05] cannot locate L4 plugin                  module=controller/ingress/types vhost=system spec="{Vhost:system L4Plugin:ibmcloud/csy-infrakit-lb1 RouteSources:map[] Routes:[{Port:443 Protocol:TCP LoadBalancerPort:443 LoadBalancerProtocol:TCP Certificate:<nil>} {Port:12391 Protocol:TCP LoadBalancerPort:12391 LoadBalancerProtocol:TCP Certificate:<nil>} {Port:12392 Protocol:TCP LoadBalancerPort:12392 LoadBalancerProtocol:TCP Certificate:<nil>}] Backends:{Groups:[group/managers] Instances:[]} HealthChecks:[]}" err="plugin not found:ibmcloud/csy-infrakit-lb1" fn=github.com/docker/infrakit/pkg/controller/ingress/types.Properties.L4Func.func1
WARN[12-01|09:53:05] cannot locate L4 plugin                  module=controller/ingress/types vhost= spec="{Vhost: L4Plugin:ibmcloud/csy-infrakit-lb2 RouteSources:map[swarm:{\"CertificateLabel\":\"lb-cert-label\",\"Host\":\"unix:///var/run/docker.sock\"}] Routes:[{Port:8080 Protocol:HTTP LoadBalancerPort:80 LoadBalancerProtocol:HTTPS Certificate:0xc420269cb0}] Backends:{Groups:[group/workers group/managers] Instances:[]} HealthChecks:[]}" err="plugin not found:ibmcloud/csy-infrakit-lb2" fn=github.com/docker/infrakit/pkg/controller/ingress/types.Properties.L4Func.func1
WARN[12-01|09:53:05] cannot locate L4 plugin                  module=controller/ingress/types vhost=system spec="{Vhost:system L4Plugin:ibmcloud/csy-infrakit-lb1 RouteSources:map[] Routes:[{Port:443 Protocol:TCP LoadBalancerPort:443 LoadBalancerProtocol:TCP Certificate:<nil>} {Port:12391 Protocol:TCP LoadBalancerPort:12391 LoadBalancerProtocol:TCP Certificate:<nil>} {Port:12392 Protocol:TCP LoadBalancerPort:12392 LoadBalancerProtocol:TCP Certificate:<nil>}] Backends:{Groups:[group/managers] Instances:[]} HealthChecks:[]}" err="plugin not found:ibmcloud/csy-infrakit-lb1" fn=github.com/docker/infrakit/pkg/controller/ingress/types.Properties.L4Func.func1
WARN[12-01|09:53:05] cannot locate L4 plugin                  module=controller/ingress/types vhost= spec="{Vhost: L4Plugin:ibmcloud/csy-infrakit-lb2 RouteSources:map[swarm:{\"CertificateLabel\":\"lb-cert-label\",\"Host\":\"unix:///var/run/docker.sock\"}] Routes:[{Port:8080 Protocol:HTTP LoadBalancerPort:80 LoadBalancerProtocol:HTTPS Certificate:0xc420269cb0}] Backends:{Groups:[group/workers group/managers] Instances:[]} HealthChecks:[]}" err="plugin not found:ibmcloud/csy-infrakit-lb2" fn=github.com/docker/infrakit/pkg/controller/ingress/types.Properties.L4Func.func1
WARN[12-01|09:53:05] cannot locate L4 plugin                  module=controller/ingress/types vhost=system spec="{Vhost:system L4Plugin:ibmcloud/csy-infrakit-lb1 RouteSources:map[] Routes:[{Port:443 Protocol:TCP LoadBalancerPort:443 LoadBalancerProtocol:TCP Certificate:<nil>} {Port:12391 Protocol:TCP LoadBalancerPort:12391 LoadBalancerProtocol:TCP Certificate:<nil>} {Port:12392 Protocol:TCP LoadBalancerPort:12392 LoadBalancerProtocol:TCP Certificate:<nil>}] Backends:{Groups:[group/managers] Instances:[]} HealthChecks:[]}" err="plugin not found:ibmcloud/csy-infrakit-lb1" fn=github.com/docker/infrakit/pkg/controller/ingress/types.Properties.L4Func.func1

The workaround to move the group before ingress does work fine.  Just opening this issue to note there is still a problem.

Can you include some details:

How did you start up your plugins?   The ordering of what worked and what didn't will help me.
When do you infrakit plugin ls what does it show?
A copy of your ingress.yml?

Ah, so now that you mentioned that, I realized what I did wrong.  I copied/pasted directly from the issue and forgot to start the ibmcloud plugin.  Oops.  My mistake, it works fine.  Closing.


Refactoring needed in database/extensions.dart

The code in packages/serverpod/lib/src/database/extensions.dart needs refactoring. Bath the big if-statement and the switch statement below it, selecting based on type, should be replaced with inheritance (it's a general observation that switching based on type is an anti-pattern).
/// SQL code generation methods for [FilterConstraint].
extension FilterConstraintGenerator on FilterConstraint {
  /// Generates a SQL WHERE clause from the filter constraint.
  String toQuery(ColumnDefinition columnDefinition) {
    assert(columnDefinition.name == column);

    var columnType = columnDefinition.columnType;
    String formattedValue;
    if (columnType == ColumnType.integer ||
        columnType == ColumnType.bigint ||
        columnType == ColumnType.doublePrecision ||
        columnType == ColumnType.boolean) {
      formattedValue = value;
    } else if (columnType == ColumnType.text) {
      formattedValue = DatabasePoolManager.encoder.convert(value);
    } else if (columnType == ColumnType.timestampWithoutTimeZone) {
      if (type == FilterConstraintType.inThePast) {
        formattedValue = _microsecondsToInterval(int.parse(value));
      } else {
        var dateTime = DateTime.tryParse(value);
        if (dateTime != null) {
          formattedValue = DatabasePoolManager.encoder.convert(dateTime);
        } else {
          formattedValue = 'NULL';
        }
      }
    } else {
      throw Exception(
        'Unsupported column type ${columnDefinition.columnType} for column '
        '"${columnDefinition.name}".',
      );
    }

    switch (type) {
      case FilterConstraintType.equals:
        return '"$column" = $formattedValue';
      case FilterConstraintType.notEquals:
        return '"$column" != $formattedValue';
      case FilterConstraintType.greaterThan:
        return '"$column" > $formattedValue';
      case FilterConstraintType.greaterThanOrEquals:
        return '"$column" >= $formattedValue';
      case FilterConstraintType.lessThan:
        return '"$column" < $formattedValue';
      case FilterConstraintType.lessThanOrEquals:
        return '"$column" <= $formattedValue';
      case FilterConstraintType.like:
        return '"$column" LIKE $formattedValue';
      case FilterConstraintType.notLike:
        return '"$column" NOT LIKE $formattedValue';
      case FilterConstraintType.iLike:
        return '"$column" ILIKE $formattedValue';
      case FilterConstraintType.notILike:
        return '"$column" NOT ILIKE $formattedValue';
      case FilterConstraintType.between:
        return '"$column" BETWEEN $formattedValue AND $value2';
      case FilterConstraintType.inThePast:
        return '"$column" > (NOW() - $formattedValue)';
      case FilterConstraintType.isNull:
        return '"$column" IS NULL';
      case FilterConstraintType.isNotNull:
        return '"$column" IS NOT NULL';
    }
  }
}

Both the if statements and the switch are made over enums. Why do you think this is an anti-pattern, could you expand on that?

There's some explanation of the thinking here, going back to Martin Fowler's thoughts on it:
https://wiki.c2.com/?SwitchStatementsSmell
However, the main reason I pointed this out is that you already have a Column class with many subclasses, so the ColumnType enum seems to duplicate that class taxonomy, and many of the FilterConstraintType types are specific to only one Column subclass (and others could be specific to a shared superclass that is a subclass of Column, for example all the scalar or ordinal types, including DateTime, could share a common ancestor class that would define the filtering behavior of the inequality operators).
Basically, in both cases, you're duplicating what you have already made object oriented in the Column hierarchy.
Case statements over enum values are not always indicative of code that should be rewritten into object oriented form, but as the link above explains, they do go against OO design when you need to do something different for all values (e.g. in a switch statement, or in this case in a class hierarchy) in more than one place in the program.

Alright got it, I'm generally a fan of Martin Fowler he usually has good takes. I think it depends on the use-case and the problem, I also think there are many other problems with OO and inheritance in general. But I digress it is irrelevant to this discussion.
Truth be told I have not looked at this piece of code before. I believe what is going on here is that we have some technical debt and duplicated code that has been left here after we refactored the query code. As far as I can tell this code is only used for the "insights" app and not when executing normal queries. When executing normal queries we build them with the internal query builder and we use the classes in the database/columns.dart file to construct the where queries.
The code in there very much follows your suggestion here.
Example:
class _BetweenExpression<T> extends _MinMaxColumnExpression<T> {
  _BetweenExpression(super.column, super.min, super.max);

  @override
  String get operator => 'BETWEEN';
}

class _NotBetweenExpression<T> extends _MinMaxColumnExpression<T> {
  _NotBetweenExpression(super.column, super.min, super.max);

  @override
  String get operator => 'NOT BETWEEN';
}

Perhaps I should add that you can find all our unit tests on the query builders in packages/serverpod/test/database/** and then our integration tests for database queries in tests/serverpod_test_server/test_integration/column_operations and tests/serverpod_test_server/test_integration/database_operations. These are pretty good at showing how the system work.

Ah, OK.
Thanks for the pointers -- I spent a lot of time getting to know the code back when I was trying to get columns to serialize to strings, but now a lot of things changed, so at this point I can't invest a lot of time into the internals anymore, I have to push forward with Serverpod as it is today.
You can close this issue if this code is not important. Thanks.


[SOLVED] Problem finding binary using PyCharm on Ubuntu

When I was testing the new implementation for the new cosmic ray rejection tool I kept getting errors where the command interpreter wasn't able to find the ecr binary or could not read properly the ecr.par file. All this when running the program from PyCharm,
The simplest solution was to, instead of starting PyCharm from Ubuntu Launcher do it from a terminal.
There is a second posible solution but so far didn't work for me and it is to add the path to pycharm.

I would in general avoid running programs from specialized plataforms
like PyCharm. Let's just make sure the pipeline can be run from the
terminal/console, in the good-ol fashioned way.  We can get fancy later,
maybe for version V1.
On 6/23/17 10:25 AM, Simon Torres wrote:

When I was testing the new implementation for the new cosmic ray
rejection tool http://users.camk.edu.pl/pych/DCR/ I kept getting
errors where the command interpreter wasn't able to find the |ecr|
binary or could not read properly the |ecr.par| file. All this when
running the program from PyCharm,
The simplest solution was to, instead of starting PyCharm from Ubuntu
Launcher do it from a terminal.
There is a second posible solution but so far didn't work for me and
it is to add the path to pycharm.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/soar-telescope/goodman/issues/86, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AWURSviQrOuD54Jexe5tnWy1s8PdTzFFks5sG8rWgaJpZM4ODomm.

--

César Briceño                               *

AURA/CTIO                                   *
Observatorio Interamericano de Cerro Tololo *
Casilla 603                                 *
La Serena, Chile                            *
Email<EMAIL_ADDRESS>             *

<EMAIL_ADDRESS>             *

Absolutely. But we need to be able to run it directly from pycharm for development purposes, otherwise the enormous benefits from using it will be canceled out by this.
This is not for the users but this issue is useful for documentation purposes, even for other people searching in google.

Ok. Agreed.
On 6/23/17 12:41 PM, Simon Torres wrote:

Absolutely. But we need to be able to run it directly from pycharm for
development purposes, otherwise the enormous benefits from using it
will be canceled out by this.
This is not for the users but this issue is useful for documentation
purposes, even for other people searching in google.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/soar-telescope/goodman/issues/86#issuecomment-310715049,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWURSjyyNHVUxzjXhz_fzY5Ft9zVUIIFks5sG-rQgaJpZM4ODomm.

--

César Briceño                               *

AURA/CTIO                                   *
Observatorio Interamericano de Cerro Tololo *
Casilla 603                                 *
La Serena, Chile                            *
Email<EMAIL_ADDRESS>             *

<EMAIL_ADDRESS>             *

This issue is actually solved.


react-toolbox-themr fails in beta-10

Setup

Upgraded from beta-8 to beta-10
Running inside a create-react-app 1.0.6

Error running themr
→ yarn run react-toolbox-themr
yarn run v0.24.5
$ react-toolbox-themr
Error: Expected pseudo-class or pseudo-element
    at new error (~/MyProject/node_modules/postcss-selector-parser/dist/processor.js:29:23)
    at Parser.error (~/MyProject/node_modules/postcss-selector-parser/dist/parser.js:234:15)
    at Parser.pseudo (~/MyProject/node_modules/postcss-selector-parser/dist/parser.js:336:25)
    at Parser.parse (~/MyProject/node_modules/postcss-selector-parser/dist/parser.js:551:22)
    at Parser.loop (~/MyProject/node_modules/postcss-selector-parser/dist/parser.js:519:18)
    at new Parser (~/MyProject/node_modules/postcss-selector-parser/dist/parser.js:104:21)
    at Processor.process (~/MyProject/node_modules/postcss-selector-parser/dist/processor.js:26:21)
    at ~/MyProject/node_modules/postcss-attribute-case-insensitive/dist/index.js:107:68
    at ~/MyProject/node_modules/postcss/lib/container.js:241:28
    at ~/MyProject/node_modules/postcss/lib/container.js:148:26

Error building app
Additionally I also get an error building the project.
→ yarn build
[..]
./~/react-toolbox/lib/autocomplete/theme.css
Module build failed: Error: Unexpected "space" found.

Everything works with beta-8. I think it's caused by the postcss dependency upgrade.

Yes, I'm fixing it and releasing a new version, sorry!

Done, there was an small error, now it should work :)

Beta-11 works like a charm! Thanks for fixing this!

Glad to know!! Thank you! :)


Fix the claim mentors route

Context
Previously the claims mentors edit route was not a collection so when editing the mentor list on the claim the url was /schools/:school_id/claims/:claims_id/mentors/:mentor_id/edit
Now is /schools/:school_id/claims/:claims_id/mentors/edit as we are editing a collection of mentors not a particular one
Changes proposed in this pull request
Changed mentors route
Guidance to review
Create a claim and edit the mentors on the check page.
The URL should not contain any mentor id
Do the same as support user
Screenshots
https://github.com/DFE-Digital/itt-mentor-services/assets/11318084/f4e7a42b-9c9e-460c-bfe9-464909461787

I love this change though ❤


Upgrade autosuggest to new widget

Consistently use the new autosuggest widget for:
Report Editing

authorization groups ( templates: all, recent)
tasks ( templates: all, recent, assigned to my org, assigned to org of report)
locations ( templates: all, recent)

Org editing

parent org (templates: all + something we still needs to refine)
approver position (templates: all advisor positions, colleagues)

Position editing

org (templates: all + something we still needs to refine)
location (templates: all)

The new widget should also be used for:

authorization group editing => positions field
task editing => responsible organization and parent task fields
merge people => in order to select the duplicate person
assign position overlay =>select position
assign person overlay => select person
advanced search => organization filter
advanced search for reports => author, attendee, pending approval of, author position, attendee position, location, tag, task
advanced search for people => location
advanced search for positions => location


Nuxt $fetch throws exception when accessing API with subdomain

Environment

Operating System: Linux
Node Version:     v18.20.3
Nuxt Version:     3.12.4
CLI Version:      3.12.0
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      compatibilityDate, devtools
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-n6demm
Describe the bug
In attempting to access an API with a subdomain from Ecowitt, $fetch throws an exception while axios handles the API appropriately.
Have reproduced the issue with a stackblitz project. However in the project I don't use the full API with specific keys in order to safeguard them. Using only a part of the API in order to force it to reply with a response. Ultimately it's the same result, $fetch throws an exception while axios is fine.
Axios returns the following message: {"code":40010,"msg":"Invalid application Key","time":"1723600494","data":[]}
$fetch throws the following error: [GET] "https://api.ecowitt.net/api/v3/device/real_time":  fetch failed
at async $fetchRaw2 (node_modules/ofetch/dist/shared/ofetch.37386b05.mjs:248:14)
at async $fetch2 (node_modules/ofetch/dist/shared/ofetch.37386b05.mjs:285:15)
at async Object.eval (.nuxt/dev/index.mjs:1066:12)
at async eval (node_modules/h3/dist/index.mjs:2100:19)
at async Object.callAsync (node_modules/unctx/dist/index.mjs:91:16)
at async Server.toNodeHandle (node_modules/h3/dist/index.mjs:2391:7)
Additional context
No response
Logs
No response

This seems to be a problem with the API when being called via Node.js (not in-browser). Same happens with plain fetch in node.
 const result = await fetch('https://api.ecowitt.net/api/v3/device/real_time')
    if(result.ok){
      return await result.json()
    }
    console.log(result)
    return 'error, resp not ok'

@manniL understand your comment but any thoughts why Axios succeeds?

Debugged it a bit but unsure. Maybe bc it relies on XMLHttpRequest and not fetch 🤷🏻♂


Testing pull request

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Testing pull


i18n(es): Updated i18n.mdx

What kind of changes does this PR include?

Changes or translations of Starlight docs site content

Description

What does this PR change? Give us a brief description.
Updated i18n.mdx

fdc18b5
af2e43c

Done!, thanks for the review!


Smart Editor - Geocoding Addresses

Widget
Smart Editor
Version of widget
2.5
Enhancement
New feature created, populate the address field if they exists.

[ ] Use selection or intersect as first option to get address. If no address info, then this option.
[ ] Geocoder mappings defined in the config
[ ] Geocode the address of the digitized point (only) and add address information to the attribute fields

Repo Steps or Enhancement details

We should try to pull the list of valid geocoders from the search widget.  If the widget is not configured, we should only show the portals default.  If the user changes the search dijit and the SE uses one of those items, the smart editor should use the updated settings.  If the item does not exist, we should raise an error on loading the smart editor that the search was modified and the following settings need to be changed in the smart editor config
I would rather not have to recreate the search settings in the smart editor.  Of if we do, we need to be able to import from teh search widget to pull in its all its settings


chore: Remove logging samples that have been migrated to google-cloud-ruby

Migrated samples to google-cloud-ruby: https://github.com/googleapis/google-cloud-ruby/pull/13137
Pointed cloud documentation links to new location: cl/391295180

For some reason, the samples pages haven't been updated even though the guides pages have.

Okay, according to the Drift team, the sample browser will update automatically, so we should be ready to delete these.


Code not fully executed

This code isn't fully executed with latest editor build (v3.2.3). Debugger has no errors.
import RecoyxLogo from 'res://res/img/company-logo/recoyx.png';

export default class PresentsState extends godot.Node2D {
    constructor() {
        super();
    }

    async _ready() {
        await godot.yield(this.get_tree(), 'idle_frame');
        // logs "[StreamTexture...]"
        console.log(RecoyxLogo);
        let recoyxLogo = new RecoyxLogo;
        recoyxLogo.position = new godot.Vector2(0, 0);
        this.add_child(recoyxLogo);
        // does not log "Here"
        console.log('Here');
        console.log(recoyxLogo);
    }

    _process(delta) {
    }
}

try not new RecoyxLogo, use RecoyxLogo.instance() instead?
I still wonder if there is a chance to instance an image.

you should get an instance of the image when you import it

I tried:
await godot.yield(this.get_tree(), 'idle_frame');
console.log('a');
let recoyxLogo = RecoyxLogo;
console.log('b');
recoyxLogo.position = new godot.Vector2(0, 0);
console.log('c');
this.add_child(recoyxLogo);
console.log('d');

Did not log 'd' and did not show the image on the scene.
I also tried RecoyxLogo.instance() as @Devilsparta said.

Finally I also tried let recoyxLogo = await import('res://img/company-logo/recoyx.png');. It doesn't log 'c' nor show the image.

Please re-open the issue, code is still not properly executed.


Login error: User <phonenumber> does not exist

Enter the username not the phone number. if a phone number is to be used, please make sure that you are trying to login from the country where that number is from. (exmple: do not try to login into an indian phone number without the country code entered while in Sri Lanka). If you are to do this, enter the country code but I strongly recommend using the username.


RTD/Sphinx: first argument of class is missing

Code (notice the ifolder):
https://github.com/ivadomed/ivadomed/blob/master/ivadomed/scripts/extract_small_dataset.py#L44
Rendered doc (ifolder missing):

This might be due to the automatic removal of the 1st argument for classes (which typically have the 'self' argument)

Using .. autofunction:: in scripts.rst solves the problem but it's not as pretty

thanks for looking @lrouhier . Not as pretty indeed. Wondering if there is a cleaner alternative... 🤔

Using add_module_name=False in conf.y results in this:

It is cleaner (maybe less explicit)

@lrouhier great, thanks.
a few things:

if possible, we should not display "prepare_dataset_vertebral_labeling", but only show the function called by __main__
we should add those scripts in the setup.py as scripts entry points

Do you mean something like this?

@lrouhier : I like the last design a lot, like a lot!


Generate and use dep-file for --preview-dart-2

tests?

@Hixie wrote:

tests?

Something like #15176 ?

Sure.


Create 3LM Class

Create new Class for using LLama3.1 as AtomicFact Evaluation
Class should inherit from
https://github.com/lflage/OpenFActScore/blob/c6e150ffc0f14162c7f603744f19d63c0e5b5469/factscore/lm.py#L5

Solved in 5f2431f


Update geological-sites.ttl

Update to Geological Sites based on feedback from mapping for data migration.
New term Regolith
New term Saprolith
Altered term Soil -> Pedolith
New term Subcrop
Term split Tailings -> Tailings and Spoil
SME @BlakePaul
Vocabulary Owner @geoderekh
Tech Check @DavidCrosswellGSQ or @LukeHauck
FYI for UAT @lisa-woody

OK.  Lets see if I can use this.  I agree with the 3 new terms and the splitting of Tailings.
However, the changing of Soil to Pedolith, does that mean that Soil will no longer be available?  I think we should retain a general "Soil" of Geological Site.

OK. Lets see if I can use this. I agree with the 3 new terms and the splitting of Tailings.
However, the changing of Soil to Pedolith, does that mean that Soil will no longer be available? I think we should retain a general "Soil" of Geological Site.

@BlakePaul Soil will be an alias of pedolith. So if you search for soil you'll still get the results. If we find in the future that we start getting lots of sites that need a greater level of distinction we can always split pedolith into its components.


Accuracy is hard to reach

Hello！
Thank you for your excellent work and analysis
Can you reproduce the PIDNet results with the same accuracy as the original version in the cityscapes test set 78.6? Why is my val 78.4 test 77.0?
Looking forward to your answer, thank you！

It is quite hard to get. I have not been able to reproduce the IOU that the paper stated.

这很难得到。我无法复制论文所述的 IOU。
Thanks for your reply


Issue no #3331

@see after @example

For #3331

@AmayAgrawal just that one comment.  Otherwise, this looks great!  Thanks again.

@AmayAgrawal just that one comment.  Otherwise, this looks great!  Thanks again.

Sir I  have made the change.Have a look at it !

Thanks @AmayAgrawal, this looks good!
Are you using the GitHub UI for editing?  If so, in the future, it would be better to clone the repo, and make one commit for an entire change like this instead of one commit per file.


[VL] row_constructor in gluten will make top-level struct to null if its child fields have null value

Describe the bug
I have a RowVector, its top-level struct value is non-null, but some fields in child vectors are null. However, if the row data is evaluated in row_construct function in gluten, its top-level struct will become null if any of its child fields have null value.  This behavior happens in https://github.com/oap-project/gluten/blob/2685c1f95aa4bd19ad37892d6d87940f77c77815/cpp/velox/compute/RowConstructor.h#L43-L47
I can't understand why when the child filed is null, the index corresponding to the top-level row also needs to be set to null?
@rui-mo I see that you participated in the review of #404, do you know the reason here? I need to make changes here, and I have to go to the reason for this code here.

@liujiayi771 Actually we do that intentionally. Velox has a implementation for row_constuctor, and we implement another in Gluten to override that one. As you noticed, the main difference is some special logic is added for the null handling. Currently, row_constuctor is only used in Aggregation to combine partial aggregation output columns into struct. For example, the partial outputs of avg is sum and count, and they are combined into ROW(DOUBLE(), BIGINT()) for Velox's final aggregation.
The problem we met was, at AverageAggregate.h#L187, because Velox's  row_constuctor did not set null for the struct, decodedPartial_.isNullAt(i) was always false and invalid value was accessed during aggregation. With this overriden function, decodedPartial_.isNullAt(i) can be true when sum or count is null.

I find this behavior but not know the reason, but it seems that based on this behavior, my implementation is fine. When I check whether the sum is null, I do it when the top-level struct is null.
https://github.com/oap-project/velox/blob/c5663fe99c64db7acfa8fbe9ca5d6a238fe0de75/velox/functions/sparksql/aggregates/DecimalSumAggregate.h#L345-L353

I find this behavior but not know the reason, but it seems that based on this behavior, my implementation is fine. When I check whether the sum is null, I do it when the top-level struct is null.
https://github.com/oap-project/velox/blob/c5663fe99c64db7acfa8fbe9ca5d6a238fe0de75/velox/functions/sparksql/aggregates/DecimalSumAggregate.h#L345-L353

Yes, that's another solution because isEmptyVector and sumVector is the children of the struct, not the struct itself.
We discussed this issue with Velox upstream, they suggest us supporting struct type in Shuffle, then the row_constructor process is not needed. But for now, we override the row_constructor to make least change to Velox code.

@rui-mo @liujiayi771 Do you guys know whether we still need this customized function? I found an issue in min_by/max_by that this logic doesn't fit.

@rui-mo @liujiayi771 Do you guys know whether we still need this customized function? I found an issue in min_by/max_by that this logic doesn't fit.

I found there is a case match for row_constructor/row_constructor_with_null for different agg functions, let me try this first.

@yma11 Could you help check which aggregate functions are still using row_constructor_with_null? If the corresponding logics on Velox side are customized well for Spark, we shall remove this hack.

Seems we still need the hack because tpch/tpcds failed after removing it. But the existing hack doesn't cover case for min_by/max_by, so I update it in PR 5544.

Seems we still need the hack because tpch/tpcds failed after removing it.

@yma11 Thanks for checking. Have you noticed which function causes the failure?

I didn't check the functions because I think the root cause of these issues are caused by the additional projects we added. From the functions perspective, it will do necessary value null checks at addRawInput or addSingleGroupRawInput and only check the row level null in XXIntermediateResults. This is reasonable.


Adds the missing return statement.

If the user is null, then the method is void. Adds the missing return line.

It's not missing, this return is useless... =X

It is not 'int|void' it is 'int|null' seing nothing after if ... return made me double check the implementation. Maybe works the same but this way it seems clearer  to me.

Well its just code's good manners in the end 😄

If we started adding this everywhere we would have to add it a lot of places. Calling this function already produces null if it doesn't return something.


Features/mass detection

Fix the exploit where you can set the inverse mass of the protein to be zero so you can freeze the simulation.
This PR was heavily supported by BC | tensorized, who figured out what the exploiting miners were doing. The breakdown is something like this:
How it works
The CUDA code of the integrators use the array velm, which contains the velocity (x,y,z) and the inverse masses (w) of each particle.
The value of the inverse masses is important when the velocity is updated. If it is zero or very small, no update happens and the particles stay frozen. For the Langevin Integrator, this is for example visible in the first step here: https://github.com/openmm/openmm/blob/53770948682c40bd460b39830d4e0f0fd3a4b868/platforms/common/src/kernels/langevinMiddle.cc#L12
The velocities AND the masses can be set and acquired by the openmm api. If you set the velocities of all the atoms to 0, then you have essentially frozen the system, making it more or less impossible to move from the checkpoint's conformation.
When a checkpoint is loaded, the whole array velm is copied to the GPU memory for CUDA; see https://github.com/openmm/openmm/blob/53770948682c40bd460b39830d4e0f0fd3a4b868/platforms/common/src/CommonKernels.cpp#L438
The (inverse) masses are the fourth coordinate (w) of velm. They are not checked after loading and do never change anymore. Herein lies the exploit: simply change the masses of the checkpoint since they have never been checked.
Solution

We save a copy of the initial checkpoint (new simulation) as a decoded txt file, which allows us to look at all the states of the memory addresses. These velocities will be zero.
From here, we apply a velocity on the first and last atom, and dump this checkpoint to a txt file
We can compare these two files to see what rows changed, and therefore get start and end indices for the velm array
When loading a miner checkpoint in process_md_output, we use check_masses to see if the masses in the miner checkpoint are identically equal to the ones in the base checkpoint. If not, the miner is exploiting.

Is there really no way to do this through the Python API? The miner said that they used a method in the openmm cookbook on Discord.

Is there really no way to do this through the Python API? The miner said that they used a method in the openmm cookbook on Discord.

yeah you're right - -I was able to do this with the api


shareWithOptions is not sending  url image

iOS:

Don't attach a valid image sharing to Whatsapp

Android:

Don't attach valid image sharing to any app.
Important note: FILE FORMAT NOT SUPPORTED.Image passed as string url.
Ionic 3.2.0

Did you find a solutions for this problem? I am facing the same issue :(

It's now working on version 5.4.1


Angular material formfield label bug with density -2 or -3 (and appearance="fill")

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
When changing the density to -2 or -3 with appearance="fill" results in labels not showing in mat-form-field

Reproduction
StackBlitz link: https://stackblitz.com/edit/tptcsx?file=src%2Ftheme.scss
Steps to reproduce:

Go to theme.scss and add density: -1 --> labels are showing
Go to theme.scss and add density: -2 --> labels are not showing
Go to theme.scss and add density: -3 --> labels are not showing
Go to theme.scss and add density: -2 with mat-form-field appearance="outline" added in select-overview-example.html --> labels are showing
Go to theme.scss and add density: -3 with mat-form-field appearance="outline" added in select-overview-example.html --> labels are showing

Expected Behavior
Labels showing with appearance="fill" with density to -2 or -3
Actual Behavior
Labels are not showing with appearance="fill" with density to -2 or -3
Environment

Angular: 17.0.5
CDK/Material: 17.0.5
Browser(s): Chrome (119.0.6045.200)
Operating System (e.g. Windows, macOS, Ubuntu): Windows

Looks like a duplicate of #26579

Closed because of duplication #26579


Restructure repo

This PR adds some minimal structure to the repo.

You probably did not want to add the folder __pycache__. If it fits to your workflow I would add some .gitignore to exclude folders with that name.
Overall the change in folder structure looks reasonable. I did not check if the new code installs and runs correctly, though.

Oh good catch. Yeah let me fix that


Discrepancy between farming balance and wallet balance that consists only of farming rewards

"Total Chia Farmed" balance @ Farm page is different from the balance @ Wallet page. Wallet's balance consists of farming rewards and gets updated correctly. However Farm Overview summary is not updated. Reloading via View/Reload(Ctrl-R) or by selecting the key from Keys page does not fix it.

(venv) ubuntu@pi401:~/chia-blockchain$ chia wallet show
Balances, fingerprint: <FINGERPRINT>
Wallet ID 1 type STANDARD_WALLET
   -Confirmed:<PHONE_NUMBER>00000 mojo (176.0 TXCH)
   -Unconfirmed:<PHONE_NUMBER>00000 mojo (176.0 TXCH)
   -Spendable:<PHONE_NUMBER>00000 mojo (176.0 TXCH)
   -Pending change: 0 mojo (0.0 TXCH)
(venv) ubuntu@pi401:~/chia-blockchain$

It is related to https://github.com/Chia-Network/chia-blockchain-gui/issues/69. I am closing it. Please follow https://github.com/Chia-Network/chia-blockchain-gui/issues/69


Add blade command services

Add new blade command services.
execute 'blade services' will output all of service extension points
execute 'blade services -s ' will output bundle name and version that contain this service

Just started reviewing :) :octocat: Sent from GH.

Lets add this code to our IDE directly, I don't think this will be useful to command line users outside of the use-case of the IDE wizards.

Hey Greg,
If put this code in our IDE, That's mean if user want to use command create service project or service wrapper ,we don't know this service belong to which bundle. So we can't be able to configure dependencies. Compiling will be failed.
this function "public String[] parseSymbolicName(String info)" will provide to search service bundle and version , it can be used by blade create command to perfect the dependency of build.gradle file.
So what do you think ?

So the same code used to connect to portal can be called from IDE just as
it could from blade.  So its exactly the same.  I just don't think this
function is useful outside of IDE so I want it to go in IDE instead.
On Tue, Dec 29, 2015 at 8:54 PM, Lovett Li<EMAIL_ADDRESS>wrote:

Hey Greg,
If put this code in our IDE, That's mean if user want to use command
create service project or service wrapper ,we don't know this service
belong to which bundle. So we can't be able to configure dependencies.
Compiling will be failed.
this function "public String[] parseSymbolicName(String info)" will
provide to search service bundle and version , it can be used by blade
create command to perfect the dependency of build.gradle file.
So what do you think ?
—
Reply to this email directly or view it on GitHub
https://github.com/gamerson/liferay-blade-tools/pull/49#issuecomment-167922728
.

--
Greg Amerson
Liferay Developer Tools
Liferay, Inc. www.liferay.com

OK. you mean we don't worry about the dependency of build.gradle file is right or wrong when user create a service project via blade. Right? we will fix dependency problem in our IDE side Right?

Right.
On Tue, Dec 29, 2015 at 9:37 PM, Lovett Li<EMAIL_ADDRESS>wrote:

OK. you mean we don't worry about the dependency of build.gradle file is
right or wrong when user create a service project via blade. Right? we will
fix dependency problem in our IDE side Right?
—
Reply to this email directly or view it on GitHub
https://github.com/gamerson/liferay-blade-tools/pull/49#issuecomment-167931430
.

--
Greg Amerson
Liferay Developer Tools
Liferay, Inc. www.liferay.com

OK.got it.Thanks


Game broken

When attempting to access this at https://githubschool.github.io/github-games-krutin86, I am getting a 404. This could be caused by a couple things:

GitHub pages needs to be enabled on master. You can fix this in the repository settings.
the index.html file is incorrectly named inde.html. We will fix this together in class.

Can you please fix the first bullet @krutin86?

First bullet fixed 💪 😎


Add ZLIB dependency for compression in rocksdb

zlib is under the zlib license, and it is compatible to be included in an Apache 2.0 licensed project: https://www.apache.org/legal/resolved.html

LGTM, sorry for didn't look the previous commits closely.

Never mind, this error will only appear when executed in a specific environment, not easy to find.

Thanks all. Merging...

@PragmaTwice Why we should support ZLIB, IIRC, ZLIB has poor performance, we  already have there compression algorithms, snappy is general algorithm from leveldb period, not bad in everything, also is default compression in kvrocks. lz4 has powerful performance, and zstd has good compression ratio and not bad performance. So i think current compression algorithms are enough.

@PragmaTwice Why we should support ZLIB, IIRC, ZLIB has poor performance, we already have there compression algorithms, snappy is general algorithm from leveldb period, not bad in everything, also is default compression in kvrocks. lz4 has powerful performance, and zstd has good compression ratio and not bad performance. So i think current compression algorithms are enough.

I think there is no cost to support zlib since rocksdb already supports zlib. And if we really need high performance, we just need to replace zlib with zlib-ng, which is several to ten times faster than zlib.

@PragmaTwice That said, is there a real-world use case for this functionality? I ever saw a number of open-source projects failed by implementing whatever they can and ruined into a complexity monster. It may not be the case for every single commit but for their combination - If you aren't gonna need it, don't implement it.


Update webhook endpoint URLs

The documentation for the endpoint URL was incorrect, as Geth no longer runs on hack.p6c.org. The updated URL no longer needs an explicit port defined, but one does need to update the protocol from http to https. As far as I know, this is already the case for all webhooks currently making use of Geth.

Why is this a pull request.


feat-cli-use-conf-file-bundle-config

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[x] No

The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[ ] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
For now I just added support for the targets array. I should throw in the rest of the configuration with this PR.

Whoops I included another PR's commit. This is what happens when I code tired I guess :shrug:.

There's so many merge conflicts I'm just gonna trash this and open a new PR.

Looks like this got fixed already :)


Move test libraries into testlib components, mildly sanitize

This does something I've wanted to do for ages, and pulls the "test library" code that we have into testlib components, and organizes them a little better. At the moment I just moved things, but I'm going to do another pass after this and try and make things saner and more consistent too.
This makes it a lot easier to reuse the test library stuff. For plutus-core we were just exporting it in the main library (ew), but for e.g. plutus-tx we weren't. Which was annoying, because I wanted to use some of the code there for writing some tests in plutus-benchmark.
Not including the test stuff in plutus-core is also a win for our dependency footprint: now we no longer force a transitive dependency on tasty on all our downstream users!

It may be good to split up the main library for UPLC/PLC/PIR......probably should be in a separate PR though.

The main thing that's preventing me from doing this is that then we'd actually have to disentangle them! There's a lot of common code in plutus-core that's used by the others, for example. So it would require quite a bit of work.

One comment/question though.  When I saw Test.Tasty.Extras appearing in  a number of files I initially assumed it was from some external library, not something that we were providing ourselves (I even unsuccessfully consulted Google to see what was in it).  Is it common practice to name things like this?

I don't think there is any convention agreed among the community. The whole naming of haskell module paths is controversial to begin with...

re: Test.Tasty.Extras, I was actually following our own convention! We have a lot of modules called Foo.Extras which contain our own little extra bit for the library Foo (e.g. Data.Aeson.Extras). Not sure whether it's a great convention, but I think it's okay? Should write it down, though.

re: Test.Tasty.Extras, I was actually following our own convention!

OK, I seem to have managed not to notice that!  There are tons of libraries called *-extras on Hackage, and I think that's why I was confused.  I  suppose we could have called these things something like OurExtras or PlcExtras, but it would be a bit pointless to change that at this stage.

Okay, let's do this.


privacy manifest for third part sdk

Report
Could you please provide Privacy Manifest support for appsflyer SDK? Some relevant links as follows:
https://developer.apple.com/support/third-party-SDK-requirements/
https://developer.apple.com/documentation/bundleresources/privacy_manifest_files
https://stackoverflow.com/questions/78163859/itms-91053-missing-api-declaration-privacy
Thank you!
Plugin Version
ℹ Please replace these two lines with the plugin version.
6.13.0
On what Platform are you having the issue?
React Native
iOS
What did you do?
N/A
What did you expect to happen?
N/A
What happened instead?
N/A
Please provide any other relevant information.
Linked above

Hi! I would like to know when you have an idea of when it will be released!

@amit-kremer93 any news regarding the timeline for this release?

I see that 6.14.0 was released on the iOS SDK with the privacy manifest file. Any timeline for when this will be updated to use that SDK?

Do we have any updates about this issue? I believe we are getting too close to the deadline.
Trying to breakdown what we need to do, given we are 13 days away from May 1st, but 4 days are weekends. We have 9 working days to:

increment iOS SDK version
release the appsflyer-react-native-plugin
upgrade the plugin on our apps
test and release our apps with the latest plugin version

@amit-kremer93 would love an update on this as well. This is a critical issue.

Opened PR https://github.com/AppsFlyerSDK/appsflyer-react-native-plugin/pull/543

I found the documentation about manifests and have created the manifest file myself.
However, I don't understand what should be in the tracking domain.
If anyone can help me, please let me know.
https://support.appsflyer.com/hc/en-us/articles/21677433322641-Privacy-Manifest

This got fixed by the #546


archive/1.1.5.tar.gz does not contain latest bug patch

Hey anthcourtney,
It looks like this commit:
https://github.com/anthcourtney/ansible-role-cis-amazon-linux/commit/7461a802e63f5a93296a0d4f38f8f50d64805e50
did not make it into:
https://github.com/anthcourtney/ansible-role-cis-amazon-linux/archive/1.1.5.tar.gz
As referenced by:
https://github.com/awslabs/ami-builder-packer
The AWS Labs project fails with:
AWS AMI Builder - CIS: ·[0;31mERROR! 'always_run' is not a valid attribute for a Task·[0m
AWS AMI Builder - CIS: ·[0;31m·[0m
AWS AMI Builder - CIS: ·[0;31mThe error appears to have been in '/tmp/packer-provisioner-ansible-local/5beccddf-7900-0274-6943-28bedd74a785/roles/anthcourtney.cis-amazon-linux/tasks/level-1/1.5.2.yml': line 6, column 3, but may·[0m

Hi @seangreathouse ,
1.1.5 was released in January.
Please use current pre-release which is 1.1.6 from
https://github.com/anthcourtney/ansible-role-cis-amazon-linux/archive/1.1.6.tar.gz
and let me know if this still causes any issue.


Unique Key Error

This patch fixes an error caused by non-unique keys.

Closing this since https://github.com/opencast/opencast-editor/pull/895 is available.


⚠ BCR Bot has degraded performance

In 2002ad2, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 1277 ms

Resolved: BCR Bot performance has improved in 94241fa.


Add support for deploying the unbound resolver

This patch adds support for deploying the unbound resolver. Note that it does not use kolla for managing and merging configs or launching the service. This is mainly because unbound doesn't require access to many openstack infrastructure services like the database, rabbitmq. etc.
Todo: add support for configuring forwarders.

/test all

/test designate-operator-build-deploy-kuttl

I'm marking as a WIP temporarily - none of the ready counts are showing up so I'm going to do a bit of experimentation.

/test all

/test all

/test designate-operator-build-deploy-kuttl

/test designate-operator-build-deploy-kuttl

/test all

/test all

/test all

/test designate-operator-build-deploy-kuttl

/test designate-operator-build-deploy-kuttl

/test designate-operator-build-deploy-kuttl

/test designate-operator-build-deploy-kuttl
db-sync job is failing atm - hard to say how it's related to this patch

/test designate-operator-build-deploy-kuttl

/test designate-operator-build-deploy-kuttl

This might be failing because of this error:
2023-11-08 20:55:08 1 [ERROR] Slave SQL: Error 'Can't create table designate.shared_zones (errno: 150 "Foreign key constraint is incorrectly formed")' on query. Default database: 'designate'. Query: 'CREATE TABLE shared_zones (
id CHAR(32) NOT NULL,
created_at DATETIME,
updated_at DATETIME,
zone_id CHAR(32) NOT NULL,
project_id VARCHAR(36) NOT NULL,
target_project_id VARCHAR(36) NOT NULL,
PRIMARY KEY (id),
CONSTRAINT unique_shared_zone UNIQUE (zone_id, project_id, target_project_id),
FOREIGN KEY(zone_id) REFERENCES zones (id) ON DELETE CASCADE
)', Internal MariaDB error code: 1005

Problems in kuttl are not related to this patch.

Problems in kuttl are not related to this patch.

@beagles If I do make designate_kuttl with the existing latest Designate operator image, the KUTTL test passes.  If I build this patch, however, and do DESIGNATE_IMG=quay.io/andrewbays/designate-operator-index:v0.0.1 make designate_kuttl, then it fails as the CI job is reporting here.  So I think something in this PR is actually contributing to the failure.  I need to investigate to figure out what, exactly.

Problems in kuttl are not related to this patch.

@beagles If I do make designate_kuttl with the existing latest Designate operator image, the KUTTL test passes. If I build this patch, however, and do DESIGNATE_IMG=quay.io/andrewbays/designate-operator-index:v0.0.1 make designate_kuttl, then it fails as the CI job is reporting here. So I think something in this PR is actually contributing to the failure. I need to investigate to figure out what, exactly.

I see that the DesignateAPI pod's init container fails to start properly and crashes like so:
$ oc logs designate-api-5d475ddb9b-r99n7 -c init -n designate-kuttl-tests
+ export PASSWORD=12345678
+ PASSWORD=12345678
/usr/local/bin/container-scripts/init.sh: line 23: DatabaseHost: Please specify a DatabaseHost variable

/test designate-operator-build-deploy-kuttl

So the alm-examples was being populated by those individual sample files. Removing that commit should resolve that issue


More boilerplate needed when creating a Scala baker instance than for Java

To stick with the documentation when creating my first recipe I have been using the JBaker object.
However today I refactored the code to be full Scala.
Using JBaker
val baker = new JBaker(
      recipe,
      ImmutableList.<Interaction>of(
                ...
        );
    )

while using Baker directly requires more boilerplate
  implicit val timeout: FiniteDuration = new FiniteDuration(10, java.util.concurrent.TimeUnit.SECONDS)
    val baker = new Baker(
      recipe,
      implementations = Map(
        classOf[InteractionExample] -> (() => new InteractionExample()),
        ....
      ),
      actorSystem = ActorSystem.apply("BakerActorSystem", JBaker.defaultConfig)
    )

It would be nice to have a SBaker object, just like for Java, that avoids having to consider things like actorSystems and timeout.
Especially since those items are currently not part of the available documentation 👼

This is already improved with the recent additions to the scaladsl.