updates README

closes https://github.com/Kong/httpsnippet/issues/249
So, since quite a lot has happened, it was time to overhaul the README.  Hopefully it reads much simpler now, and if not hopefully the newly added quickstart sections span that gap.
Preview it here: https://github.com/dimitropoulos/httpsnippet/blob/redo-readme/README.md

@erunion please take a gander if you could


Set isAggregationTypeImplicit to true for columns in duplicate-key table

duplicate of https://github.com/StarRocks/starrocks/pull/2889


Update for request usage monitoring

Updated database schema to support monitoring
Updated proxy service to support monitoring

#84


Can no longer browse Nuget Feed to add dependancies

Type:

Bug : Can no longer browse NuGet Feeds to add dependancies

OS version: (Windows 10 Pro)
Installed from: Microsoft Store
In case of a BUG:

What is the current result? Browse online NuGet feed
What is the expected result? View all NuGet Packages

Can you please provide repro steps?
I just tried this and it works for me.

I click there, it shows me the package chooser, I double-click a package, then it enters it. Then hitting the + adds it.

Hi
Thanks for responding. I had this issue for about 2 days and no matter the nuget source it didn’t seem to work. I have however tried again and it seems to be working. Not sure what might have gone wrong.
But thanks for following up and great work on the app, I use it often.
From: Oren Novotny<EMAIL_ADDRESS>Sent: Saturday, November 9, 2019 6:29 PM
To: NuGetPackageExplorer/NuGetPackageExplorer<EMAIL_ADDRESS>Cc: CodeSeraph<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [NuGetPackageExplorer/NuGetPackageExplorer] Can no longer browse Nuget Feed to add dependancies (#850)
Can you please provide repro steps?
I just tried this and it works for me.
https://user-images.githubusercontent.com/1427284/68531770-03562e00-02e4-11ea-8bd0-11813a1d02cd.png
I click there, it shows me the package chooser, I double-click a package, then it enters it. Then hitting the + adds it.
https://user-images.githubusercontent.com/1427284/68531794-284aa100-02e4-11ea-95b3-a6b212623c18.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/NuGetPackageExplorer/NuGetPackageExplorer/issues/850?email_source=notifications&email_token=ALXWGV3WZABGR2QANQ3Y3DTQS3QONA5CNFSM4JHXMUFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDUJY2I#issuecomment-552115305 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ALXWGVZWF4O6K554AAW3BX3QS3QONANCNFSM4JHXMUFA .

This email has been checked for viruses by AVG.
https://www.avg.com

I get the same error:
Application: NuGetPackageExplorer.exe
CoreCLR Version: <IP_ADDRESS>904
.NET Version: 5.0.0
Description: The process was terminated due to an unhandled exception.
Exception Info: NuGet.Configuration.NuGetConfigurationException: NuGet.Config is not valid XML. Path: 'C:\Users\ivan.mireles\AppData\Roaming\NuGet\NuGet.Config'.
---> System.Xml.XmlException: '.', hexadecimal value 0x00, is an invalid character. Line 1, position 1.
at System.Xml.XmlTextReaderImpl.Throw(Exception e)
at System.Xml.XmlTextReaderImpl.Throw(String res, String[] args)
at System.Xml.XmlTextReaderImpl.ParseRootLevelWhitespace()
at System.Xml.XmlTextReaderImpl.ParseDocumentContent()
at System.Xml.XmlTextReaderImpl.Read()
at System.Xml.Linq.XDocument.Load(XmlReader reader, LoadOptions options)
at NuGet.Configuration.XmlUtility.LoadSafe(Stream input, LoadOptions options)
at NuGet.Configuration.XmlUtility.GetDocument(String fullPath)
at NuGet.Configuration.XmlUtility.GetOrCreateDocument(XDocument content, String fullPath)
at NuGet.Configuration.SettingsFile.<>c__DisplayClass23_0.<.ctor>b__0()
at NuGet.Configuration.SettingsFile.<>c__DisplayClass32_0.b__0()
--- End of inner exception stack trace ---
at NuGet.Configuration.SettingsFile.<>c__DisplayClass32_0.b__0()
at NuGet.Common.ConcurrencyUtilities.ExecuteWithFileLocked(String filePath, Action action)
at NuGet.Configuration.SettingsFile.ExecuteSynchronized(Action ioOperation)
at NuGet.Configuration.SettingsFile..ctor(String directoryPath, String fileName, Boolean isMachineWide, Boolean isReadOnly)
at NuGet.Configuration.Settings.ReadSettings(String settingsRoot, String settingsPath, Boolean isMachineWideSettings, Boolean isAdditionalUserWideConfig, SettingsLoadingContext settingsLoadingContext)
at NuGet.Configuration.Settings.LoadUserSpecificSettings(String root, String configFileName, Boolean useTestingGlobalPath, SettingsLoadingContext settingsLoadingContext)+MoveNext()
at System.Collections.Generic.List1.InsertRange(Int32 index, IEnumerable1 collection)
at System.Collections.Generic.List1.AddRange(IEnumerable1 collection)
at NuGet.Configuration.Settings.LoadSettingsForSpecificConfigs(String root, String configFileName, List`1 validSettingFiles, IMachineWideSettings machineWideSettings, Boolean loadUserWideSettings, Boolean useTestingGlobalPath, SettingsLoadingContext settingsLoadingContext)
at NuGet.Configuration.Settings.LoadSettings(String root, String configFileName, IMachineWideSettings machineWideSettings, Boolean loadUserWideSettings, Boolean useTestingGlobalPath, SettingsLoadingContext settingsLoadingContext)
at NuGet.Configuration.Settings.LoadDefaultSettings(String root)
at PackageExplorer.SettingsManager.GetPackageSources() in D:\a\1\s\PackageExplorer\MefServices\SettingsManager.cs:line 159
at PackageExplorerViewModel.PackageSourceSettings.GetSources() in //PackageViewModel/PackageChooser/PackageSourceSettings.cs:line 28
at PackageExplorerViewModel.MruPackageSourceManager.LoadDataFromSettings() in //PackageViewModel/MruPackageSourceManager.cs:line 32
at PackageExplorerViewModel.MruPackageSourceManager..ctor(ISourceSettings sourceSettings) in //PackageViewModel/MruPackageSourceManager.cs:line 17
at PackageExplorerViewModel.PackageViewModelFactory.CreatePackageChooserViewModel(String fixedPackageSource) in //PackageViewModel/PackageViewModelFactory.cs:line 87
at PackageExplorer.PackageChooserService.SelectPackage(String searchTerm) in D:\a\1\s\PackageExplorer\MefServices\PackageChooserService.cs:line 44
at PackageExplorer.MainWindow.OpenPackageFromRepository(String searchTerm) in D:\a\1\s\PackageExplorer\MainWindow.xaml.cs:line 357
at PackageExplorer.MainWindow.OpenFeedItem_Click(Object sender, ExecutedRoutedEventArgs e) in D:\a\1\s\PackageExplorer\MainWindow.xaml.cs:line 331
at System.Threading.Tasks.Task.<>c.b__140_0(Object state)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
at System.Windows.Threading.DispatcherOperation.InvokeImpl()
at System.Windows.Threading.DispatcherOperation.InvokeInSecurityContext(Object state)
at MS.Internal.CulturePreservingExecutionContext.CallbackWrapper(Object obj)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location ---
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
at MS.Internal.CulturePreservingExecutionContext.Run(CulturePreservingExecutionContext executionContext, ContextCallback callback, Object state)
at System.Windows.Threading.DispatcherOperation.Invoke()
at System.Windows.Threading.Dispatcher.ProcessQueue()
at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
at System.Windows.Threading.Dispatcher.LegacyInvokeImpl(DispatcherPriority priority, TimeSpan timeout, Delegate method, Object args, Int32 numArgs)
at MS.Win32.HwndSubclass.SubclassWndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam)
at MS.Win32.UnsafeNativeMethods.DispatchMessage(MSG& msg)
at System.Windows.Threading.Dispatcher.PushFrameImpl(DispatcherFrame frame)
at System.Windows.Threading.Dispatcher.PushFrame(DispatcherFrame frame)
at System.Windows.Threading.Dispatcher.Run()
at System.Windows.Application.RunDispatcher(Object ignore)
at System.Windows.Application.RunInternal(Window window)
at System.Windows.Application.Run()
at PackageExplorer.App.Main()

@ivanm07 Your C:\Users\ivan.mireles\AppData\Roaming\NuGet\NuGet.Config seems to be corrupt. Can you please rename or delete it? Then it should work again. You can also edit it in a code editor to see what exactly is not XML conform.

@ivanm07 Your C:\Users\ivan.mireles\AppData\Roaming\NuGet\NuGet.Config seems to be corrupt. Can you please rename or delete it? Then it should work again. You can also edit it in a code editor to see what exactly is not XML conform.

Yes that works for me, thanks!

Closing due to age. We've seen corruption in that file before but it's not clear why.

Closing due to age. We've seen corruption in that file before but it's not clear why.


Update application-gateway-crs-rulegroups-rules.md

Updated note regarding CRS 2.2.9

@vhorne
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@vhorne,
Can you review this old PR and determine whether it needs to be closed or merged?


can’t save profile pic

I’m a new user. I want my friends to see a picture of me instead of the default grey square. I touch the picture box in the UI and select a picture from my phone library. Then I touch Save. I get an error that I don’t understand. See attached screenshot.

iPhone 11 Pro
iOS 13.6
Safari

It has something to do with s3 tokens expiring. Looking into it.

I think / hope this is resolved. There were a fair number of changes here in Sept.

Verified this is resolved via iOS app.


server ignore OPS when spigot is used

The problem is, on initial boot up of the container, the specified OP Player is not being OP if the server type is Spigot. If the type is not specified, hence server is a typical minecraft
The OP Player get OP permission on the server.
I have checked ops.json and ops.txt.converted, both contains information related to my Minecraft user account.
I have made sure that I deleted any volume and container when i switch between Spigot and Minecraft Server
Below is my docker compose
version: "3.5"

services:
  mc:
    image: itzg/minecraft-server
    restart: unless-stopped
    ports:
      - 25565:25565
    environment:
      - OPS=<USER>
      - ONLINE_MODE=TRUE
      - EULA=TRUE
      - MODE=survival
      - LEVEL_TYPE=LARGEBIOMES
      - SPAWN_NPCS=TRUE
      - SPAWN_ANIMALS=TRUE
      - SPAWN_MONSTERS=TRUE
      - GENERATE_STRUCTURES=TRUE
      - FORCE_GAMEMODE=TRUE
      - ENABLE_COMMAND_BLOCK=TURE
      - ANNOUNCE_PLAYER_ACHIEVEMENTS=TRUE
      - ALLOW_NETHER=true
      - SERVER_PORT=25565
      - VERSION=1.14.4
      - DIFFICULTY=normal
      - FORCE_REDOWNLOAD=TRUE
      - TYPE=SPIGOT

If ops.json looks correct, then it must be something about Spigot. I vaguely remember Spigot has its own permission system. Maybe that mechanism isn’t honoring the ops.json.

It only worked for me when online_mode false. When online I had to use rcon and give initially op manually

It looks like Spigot and/or PaperMC are the root cause of this issue, such as this one which got a PR merge a few days ago
https://github.com/PaperMC/Paper/issues/4081
I am seeing the same behavior where ops.txt.converted is processing by Spigot's startup, ops.json contains my player at first, but then I do an op some-other-player it blows away my user in ops.json.
Unless someone has a suggestion, I don't see how the container startup scripts can work around this.


[GLUTEN-1907][CH] Fix config_file error.

What changes were proposed in this pull request?
Fix excetpion after config spark.gluten.sql.columnar.backend.ch.runtime_config.config_file=/etc/native_engine/config.xml
(Fixes: #1907)
How was this patch tested?
start thrift after config config spark.gluten.sql.columnar.backend.ch.runtime_config.config_file

Pls rebase on main

Run Gluten Clickhouse CI


feat(jqLite): add withDataAndEvents to clone method

While writing a custom directive I needed to clone an element and retain the events. The jQuery version of clone takes a second parameter that allows copying the events and data, so I thought I'd implement that.
Found jqCache references that were not deallocated!
The only thing left is cloning the data.
The problem I'm facing is that karma is telling my that I've got a leak.
Any suggestions why?

@leon what's the status of this?

This is a low frequency feature that hasn't seen activity in a while, so I am going to close this. Remember that changes to jqLite need to serve the core first, and the user space later.


refactor: lazy load css per component

TODO: measure how this affects

number of modules loaded during development
number of modules loaded in production
number of style recalculations and repaints
duration of style recalculations and repaints


some very basic 'best way' generic patterns for very simple actions would help newbies

Category

[ x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

solutions are great but a few definitive, 1 only of each, 'best practice' patterns for
a few generic operations that were labelled as such  in github would remove confusion between
solutions where different authors have their own pet approach to the same operation or
the practice changes over time but old solutions still exist.
Could  the guru's agree on a current 'best practice' (evolving)
for some few basic common atomic sharepoint snippets for content output.
This would
really really help newbies onboarding to the framework and make it easier to
see when the 'best' approach changes and to stay aligned with the leading edge.
eg.

async fetch list items into an object array

 async read taxonomy group for key and return members as object array

 fetch current user profile into an object from graph

 fetch current users role and group membership

 restrict action based on users role

 fetch current users recent documents from graph

 fetch a tenant property

 issue a search request and get results back into a renderable object

 fetch any persons info-card using email address as a key

 fetch a documents dynamic preview image

make it competitive and give a "SNIP of the month" , "Best of Breed" in the SIG, for better refinements on the best way to do simple atomic building block operations. focus tightly in on a very few actions and apply the best thinking to them. that would educate us all.
The biggest problem as a community consumer is working out which way is the best way to do very simple actions when there are so many possible approaches. The solutions give a great wide-angle view of putting it all together but we need the tight focus zoom-in close-up as well for very common ops to raise the game level of all of us.

@kath561 Thanks for the feedback! Can you please request this on SharePoint UserVoice, thanks!


[Bug]: VectorStoreIndex as chat engine doesn't return sources

Bug Description
Expectation
When using index.as_chat_engine().chat my expectation is that response.sources would include sources.
If I use index.as_query_engine().query I can see sources, no problem.
Actual result
No sources are present.
Reproduction is below.
Version
0.8.50
Steps to Reproduce
import llama_index
llama_index.__version__

from llama_index import Document
from llama_index import VectorStoreIndex, ServiceContext, set_global_service_context
from llama_index.llms import OpenAI
from llama_index.embeddings import OpenAIEmbedding
from llama_index import (
    SimpleDirectoryReader,
    load_index_from_storage,
    VectorStoreIndex,
    StorageContext,
)

embed_model = OpenAIEmbedding()
llm = OpenAI(model="gpt-4")

service_context = ServiceContext.from_defaults(llm=llm, embed_model=embed_model)
set_global_service_context(service_context)

corpus_of_documents = [
    "Take a leisurely walk in the park and enjoy the fresh air.",
    "Visit a local museum and discover something new.",
    "Attend a live music concert and feel the rhythm.",
    "Go for a hike and admire the natural scenery.",
    "Have a picnic with friends and share some laughs.",
    "Explore a new cuisine by dining at an ethnic restaurant.",
    "Take a yoga class and stretch your body and mind.",
    "Join a local sports league and enjoy some friendly competition.",
    "Attend a workshop or lecture on a topic you're interested in.",
    "Visit an amusement park and ride the roller coasters."
]
documents = [Document(text=t) for t in corpus_of_documents]

from llama_index import VectorStoreIndex
index = VectorStoreIndex.from_documents(
    documents, show_progress=True
)

response = index.as_chat_engine().chat("Should I hike?")
print(response.response) # doesn't seem to refer to the documents
print(response.sources) # == [], it's empty, but it shouldn't. Not clear to me whether or not it's actually querying the documents.

Relevant Logs/Tracbacks
No response

I think in this case, it's indeed not querying the documents.
By default, the chat engine is an agent that decides whether to consult the knowledge base. You can try the context mode to always retrieve relevant context on each question.

Running the above with:
response = index.as_chat_engine(mode='context').chat("Should I hike?")

mode='context' doesn't seem to force a query either.

I think I figured this out. I was using mode instead of chat_mode.
response = index.as_chat_engine(chat_mode='context').chat("Should I hike?")


watchQuery should respect addTypename

as explained here #870, the options.reducer function of graphql does currently not automatically include the __typename.

I'll make some changes in another branch and then merge this. Thanks!


Can't extract encrypted 7z LZMA2 files

I have an encrypted 7zip archive (linked below) encrypted with the password "dontgoout", but whenever I try to extract it, it throws an error at MoveToNextEntry.
Here's a screenshot of the stack trace:
https://gyazo.com/6fb0b0c0731d6e23d04a26e301cc2dbe
I created the archive using the 7zip client for Windows using these settings:
https://gyazo.com/ca4e67e6a259d79cdf0675dc42fd8268
Original name was 0.tmp, but Github only allows zip files.
0.tmp.encrypted.zip
Extracting the same archive without any password works perfectly fine and I can extract the encrypted 7z archive using the 7zip desktop client, so this has to be a bug in the library (Or I'm just too dumb for life and I missed something I should have done)
Here's the code I used to extract that archive:
SharpCompress.Readers.ReaderOptions test = new SharpCompress.Readers.ReaderOptions();
test.LookForHeader = true;
test.LeaveStreamOpen = false;
test.Password = "dontgoout";

var archive = SevenZipArchive.Open(filename, test);

if (!archive.IsComplete)
{
    MessageBox.Show("One of the downloaded resources is corrupt, please try again.");
    return;
}

long totalbytes = archive.TotalUncompressSize;
long processedbytes = 0;

var reader = archive.ExtractAllEntries();

while (reader.MoveToNextEntry()) // <-- Here it crashes
{
    if (!reader.Entry.IsDirectory)
    {

I was about to say "sharpcompress doesn't support encrypted 7zip files" but honestly I can't remember.
However, the error you got is fixable because it's using a stream that doesn't support seeking though 7zip requires seeking.

But if the library requires seekable streams (which totally makes sense the way 7z works), why did it work with unencrypted files? I mean they'd have to be seekable too, right?

Woops

It must be that the decryption code just needs to know the length for some reason and it's using a different stream class.
The class with the error is one I wrote for the purpose of ensuring no seekable stream was used in other parts of the code.
I haven't done much with the 7zip code personally.  I really hate the 7zip archive format.

Can you point me in the right direction please? I tried using your API examples, but they only seem to work with zip archives ..
(Aka replacing
var archive = ...;
with
var archive = ArchiveFactory.Open(...);
doesn't work either)

It's not going to work without a code change.   Or are you saying that my encrypted 7zip file is broken?

I have a feeling encrypted 7zip just isn't supported.  I don't recall putting that in the supported formats matrix

Ye, I just assumed that encrypted 7zips wouldn't make a difference to the 7z.dll ...
Anyway, I "fixed" it now by encrypting the final 7z with AES. For my purposes that's acceptable

This code doesn't used the 7z dll.  That's why he behavior is different than other libraries as others do use the native dll.


num_samples should be a positive integer value, but got num_samples=0

python train.py --epochs 12 --batch_size 2 --learning_rate .001 --hidden_size 64 --n_layers 1 --dropout_p .1
number of trained word vectors of data/glove.6B.100d.txt: 400000
Traceback (most recent call last):
File "train.py", line 200, in
train_loader = DataLoader(dataset=qa_train, batch_size=config.batch_size, shuffle=True, num_workers=4)
File "/usr/local/lib/python3.6/dist-packages/torch/utils/data/dataloader.py", line 176, in init
sampler = RandomSampler(dataset)
File "/usr/local/lib/python3.6/dist-packages/torch/utils/data/sampler.py", line 66, in init
"value, but got num_samples={}".format(self.num_samples))
ValueError: num_samples should be a positive integer value, but got num_samples=0
Please Let Know What's the exact issue

HI, @staticgsm
Could you check if the code below works well?
There seems to be no data qa_train variable.
qa_train = QuestionAnswerDataset(train, tokenizer, negative_sampling=True)


"Path to native podman CLI" disappears after saving

I type the path do my podman CLI, and when I hit Save, it disappears. Is this expected?
https://user-images.githubusercontent.com/233589/167876711-b45368d7-e73a-4af1-a0b2-eeaa7ed90055.mov
Note: I had typed both "Path to native podman CLI" and "Connection string" and saved since they weren't auto-detected.
I should note that I don't know that to do after this. I can't see any other screens on this app other than this initial settings screen, so the app is unable for me at the moment.

Awesome report - no, it is not ok that it does not save, it is a bug! At least is not crashing and I might be able to finally know why it does it for so many users.
Can you try to upload the logs ?
You can also try to click connect after you type the path and see if it does connect

does not connect

Thank you, those errors are expected. I've just release the 4.1.0-rc.10 above -it should be able to deal with both saving and connecting.
If still having issues, you can set the logs to Debug , restart and see more info - my podman on mac is in /usr/local/bin/podman as I think I did a brew link podman at a point

@iongion in the screenshots version 4.1.0-rc.10 with Debug mode

Have you already performed podman machine init ?

yes

my podman version

Huge thanks  - I was able to reproduce also

Done, I think I've found it with your awesome help - THANK YOU! rc.11 is building

While I fix it, you could try to install lima just to have the cli available brew install lima and see if the app starts, this was it. That exception was preventing the entire startup.

installing lima did not help

Uploading dmg files right now https://github.com/iongion/podman-desktop-companion/releases/edit/4.1.0-rc.11

Damn, thank you the same, truly helpful, imagine it now solves for all users.

Unfortunately, saving still doesn't for for me using 4.1.0-rc.11
Now it reverts back to /usr/local/bin/podman which is the default instead of just disappearing.

I see the podman machines list is empty, it could be a chicken or the egg problem - to get the list, one needs to set the program, to save, one needs both :)
I am going to replicate the same homebrew setup as yours and see what is going on

I solved it here I think, in next unreleased v12 https://github.com/iongion/podman-desktop-companion/issues/64

Solved in https://github.com/iongion/podman-desktop-companion/releases/edit/4.1.0-rc.21 - re-open if not the case


Filter by family friendly

Filters by familFriendly variable in search files.

Upon working on adding a second filter, I think I have a better way to filter and will be making changes to this branch. Please hold on reviewing.

Upon working on adding a second filter, I think I have a better way to filter and will be making changes to this branch. Please hold on reviewing.

Not needed anymore


Specifying socket options

Hello,
I am using Cassandra 3 cluster (5 machines) and 3.1.0 version of your driver.
It seems that we cannot specify other  values for socket options, because in SessionHolder I see only next:
builder.withSocketOptions(new SocketOptions().setKeepAlive(true));
Can we somehow get new version with possibility to specify:

connectTimeoutMillis
readTimeoutMillis
tcpNoDelay

Thanks in advance.

changes.zip


[Subtask] add topic pre event for Gravitino server

Describe the subtask

add corresponding TopicPreEvent
generate pre-event in TopicEventDispatcher
add test in TestTopicEvent
add document in gravitino-server-config.md

Please refer to the implemetation of table pre event in #5539
Parent issue
#5317

Hi @FANNG1 ,
I want to follow up on this issue.
Could it be assigned to me?😄
Best Regards,
Edward


As a lazy social media user, I can log out from all my social media account with one command.

so that I can remove potential unauthorised access effectively.


Create message_checker.py

just a family friendly version of yoni's attempt
Summary
Just another example explaining how you can use message and message.content to look for some words inside of the message sent

Checklist

[ ] If code changes were made then they have been tested.

[ ] I have updated the documentation to reflect the changes.

[ ] This PR fixes an issue.
[x] This PR adds something new (e.g. new method or parameters).
[x] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[x] This PR is not a code change (e.g. documentation, README, ...)

What do you mean? You're just altering a dictionary I suppose.

What do you mean? You're just altering a dictionary I suppose.

i used the if any method you gave me

im gonna add another example inside of that file relating to on_message

Why did you add another example I suppose.
Where's the command I requested I suppose.

is this was you were looking for?

import voltage #import the voltage module
from voltage.ext import commands # importing commands for our client

client = commands.CommandsClient("!") # initialize our client

ungodly_words = ["sus", "baka", "suppose", "real", "amogus"] # create our list to iterate through and add items later

@client.listen('message') # specify what we're going to listen to
async def on_message(message): # the name for this can be anything you want it to be
  if any([word in message.content.lower() for word in ungodly words]): # run the if statement to trigger if the message has words in the array
    await message.reply("*GASP!* You can't say that word!", delete_after=5) # reply to the message sent and delete OUR message after 5 seconds
    await message.delete() # delete the USERS message
  await bot.handle_commands(message)
  # Handle afterwards so other commands will work after the on_message,

@commands.command(name="add_word") # Create our command
async def add_word(ctx, word): # Define our command
  ungodly_words.append(word.lower()) # Append to our list
  await ctx.send(f"Added `{word.lower()}` to the list of `{len(ungodly_words)}` words!") # Tell user that the word was added to the list.

client.run("TOKEN") # Run the bot

you should note that the added words wont be permanent, and will be discarded on every restart

this doesnt make sense because then id have to impliment like a database or a json file and it will get way too overcomplicated and completely overlap the point of the example

you should note that the added words wont be permanent, and will be discarded on every restart

this doesnt make sense because then id have to impliment like a database or a json file and it will get way too overcomplicated and completely overlap the point of the example

no, just say it wont be persistent, no need to do that


mgmt: smp: Fix compilation problem

Fix problem with compilation due to incompatible type assignments.
Fixes #28031
Signed-off-by: Dominik Ermel<EMAIL_ADDRESS>

@galak Can you check this?


Add installation to support find_package

This now can be installed with cmake and can be used, like:
find_package(sugar)

For some reason I can't leave the comments in review. Writing them here:
CMakeLists.txt (line 2)
I don't think it makes any difference in current case but it's a good habit to put project right after cmake_minimum_required:

http://cgold.readthedocs.io/en/latest/tutorials/project.html#when-not-declared

CMakeLists.txt (line 6)
Why do we need enable_testing?
sugar-config.cmake
There is no check of components. If user write find_package(sugar CONFIG REQUIRED missing-component) we should report an error. This can be done by check_required_components:

https://github.com/forexample/package-example/blob/cf2ea1d6a209fb9eca2ab83fdd0ac15fe4d3e807/Foo/cmake/Config.cmake.in#L4

@pfultz2 This idea is great, thanks. I was thinking about unification of CMake modules and regular C++ packages (https://docs.hunter.sh/en/latest/overview/anything.html) before, this seems to be a nice solution.

Why do we need enable_testing?

It there just in case you want to add testing. I can remove it for now.

I also have updated this based on your feedback.

Looks good, merged. Thank you!


Error raised when using llama3.2 vision

What is the issue?
ollama run x/llama3.2-vision:latest "describe this image: /home/papillon/Downloads/objectdetection.jpg"
Added image '/home/papillon/Downloads/objectdetection.jpg'
Error: POST predict: Post "http://<IP_ADDRESS>:32927/completion": EOF
ollama run x/llama3.2-vision:latest "hello"
Error: POST predict: Post "http://<IP_ADDRESS>:39683/completion": EOF
ollama run x/llama3.2-vision:latest

Hello
Error: POST predict: Post "http://<IP_ADDRESS>:39645/completion": EOF

OS
Linux
GPU
Nvidia
CPU
AMD
Ollama version
0.4.0-rc5

Can you please post the server logs?

Sure, here it is
[
ollama_logs.txt
](url)

Thanks for the logs. This looks mostly the same as #7440 although the log messages are slightly different. We are currently working on a fix for that and it will be available in rc7.
I think most likely it will solve your problem but would it be possible for you to test it when it comes out and report back to confirm?

It works now with rc8

Thanks for the logs. This looks mostly the same as #7440 although the log messages are slightly different. We are currently working on a fix for that and it will be available in rc7.
I think most likely it will solve your problem but would it be possible for you to test it when it comes out and report back to confirm?

I am getting the same issue, is my only resort to build the rcx image manually?

0.4.0 has been officially released and contains the fix, so you can just install it from the binaries on the download page.

0.4.0 has been officially released and contains the fix, so you can just install it from the binaries on the download page.
Still not working for me on 0.0.4...

0.4.0 has been officially released and contains the fix, so you can just install it from the binaries on the download page.

It works on Windows but not on Linux

@theodufort Can you please post the full server logs?
There's a good chance it is the same as #7546, which was just fixed in main.

@theodufort Can you please post the full server logs?
There's a good chance it is the same as #7546, which was just fixed in main.
Here (I checked and doesn't seem to have any specific errors):
ollama_logs.txt

Here:

Thanks for the logs - this looks like it is the same as #7558 so we can track it there.


Enable native Services menu on OS X

Looks like Atom disables the native Services menu on OS X. Any chance of enabling it? I have loads of useful services that I don't want to have to reimplement/replace as packages.

I know this issue has been marked fixed, and I can indeed see a Services menu, but it is empty. It should be filled with numerous text services.

Confirmed on 0.182.0 as well, still no services.

This is indeed not fixed; we need to implement support for this. Maybe I'll find some time over the weekend, but if anybody else wants to give it a go, here's a starting point:
https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/SysServices/Articles/using.html#//apple_ref/doc/uid/20000854-CEGDDHJJ

The Services menu shows up under Atom > Services and those services work, but nothing in the context menu when I right-click within the document (meaning those text services don't show up).

I see the Services menu but I'm only seeing a small subset of the enabled services that should be appearing. Seems weird that I should see some but not others?

To add to this, I'm seeing the correct services, and I can click on one and it will activate from the menu. The issue I'm having is that the keyboard shortcut isn't working for them.
Ah, though while writing this I figured it would be a good idea to check the keybindings, and it looks like Linter is taking over the shortcut I had set the service up to use. Not sure what the solution could be for that... should services over-ride?

I'm now on 0.192.0 and still no services.

Version .204 and while I see the Services menu, there are no services available.

ditto to @stevencombs

Any idea when (or if) this will be actioned?

The Services menu does show up (although not the equivalent contextual menu), however they don't seem to have access to Atom. You can't use the system level text manipulation, or any such services you create.

Use of the services within Atom would be great. Where do we vote on new features?

I agree, it would be great to have services

Does this issue being closed imply that this feature is coming out whenever 1.3.0 is out of beta?  Because I'm on 1.2.4, and still no services...

Does this issue being closed imply that this feature is coming out whenever 1.3.0 is out of beta?

Yes.

I use 1.3.1 (on OS X 10.11.1), but the services function doesn't seem to be working.  What I noticed is the following.  If I select some text in the editor, no services appears under Atom > Services.  However, for instance, if I select some selectable text in Atom > Preferences... > Settings (the line starting with "These are Atom's..." is selectable), then services do appear under Atom > Services.

I agree with naoboo. Same here.

Based on @naoboo's observation, I would guess that Atom's editor view simply doesn't use NSText, thereby not feeding selected text to Services*, but the config panels do.

Same issue. I can see my non-text services but not my text services. This would confirm @kfix hypothesis. Non-text services work as expected.

@kfix @magp That sounds right – the text editor portion of Atom is a custom component, where as the views in settings-view are just standard DOM components that allow for native text Services. Would you mind opening a new issue for enhancing Atom's Services support to include text services?

any update on this?

It's 2017 and the services are still not supported. Surprised

@jjiangweilan " its " does not advance a feature request. see #10213, which is the core problem for Text services, and its not trivial.

Atom 1.17.2, Services menu still broken.

Similarly frustrated by this, here's my work-around. It's not for everyone, as it involves commercial OS X software. And it requires more keystrokes than a regular service would. But it might inspire other solutions.
This workflow uses Launchbar. One could use Alfred or another keyboard shortcut tool for this.

Select text in Atom
Long press ⌘-space to send that text to Launchbar (or your custom shortcut)
Type part of the name of a text service and select it. (The text is processed in the buffer.)
Hit tab to invoke a new command.
Select Paste in Frontmost Application to replace the originally-selected text in Atom with the processed text.


Update appstudio-utils image ref

The change was wrongly introduced in the infra-deployments deploy.yaml files, making the updates from pipeline-service to override the change.
ref: https://github.com/redhat-appstudio/infra-deployments/pull/3848

/test test-pipeline-service-upgrade-ocp-414

/test test-pipeline-service-upgrade-ocp-414

/test test-pipeline-service-upgrade-ocp-414


How can i create a 13B version?

33b version is way too slow for me. I'm currently using 6.7b, but I wonder if I can use a model between 6.7b to 13b.
Is there any way to downscale 33B model to a model with custom hyper-parameters?
Thanks

We don't pre-train 13b. You can try to use quantized 33b model.


add retry parameter?

[Failed to initialize GitHub Application connection using provided id and private key]
This happens sometimes. I don't know if this is due to some sort of github api latency issue.
I rerun my workflow and it disappeared.
Is it possible to add a retry parameter?

I have updated the v1 and released v1.0.3 that will print the actual error object to the actions logs as an error that would provide context into the underlying failure you are encountering.
If this happens again, could you possibly share the error that is recorded as it would allow me to focus on where this is failing for you.

I have updated the v1 and released v1.0.3 that will print the actual error object to the actions logs as an error that would provide context into the underlying failure you are encountering.
If this happens again, could you possibly share the error that is recorded as it would allow me to focus on where this is failing for you.

sure. thank you


Added an example of using jsQuestPlus as an ES6 module

This example was inspired on post #7  on the jsQuestPlus discussion board

Thank you very much for your usefull code!


Value corruption (occasional)

I'm displaying the battery state of charge on my instrument panel in a basic numeric value widget. The value comes from VEdirect-serial-USB and is a float / ratio. Usually the value is shown correctly, but sometimes KIP adds digits, as if it made some internal calculation with the value and ends up with a rounding problem:
Value in SK: 0.944, value shown in KIP: 94.39999999999999999%
Settings of the widget:
Minimum integer places: 1
Minimum of decimals: 0
Display format: % - Percentage value (value multiplied by 100)

In your exemple, is 8.944 received the capacity percentage - the state of charge is at 8.944% ?

The state of charge was 94.4%, represented in SK as a ratio value of 0.944.
KIP is then multiplying by 100 and doing this in Javascript gives 94.39999999999. Yeah!

One could use .toFixed(n) to produce a nice output whenever a ratio (float) is multiplied by 100 for showing a percentage. The amount of decimal places (n) could be determined by looking at the number of decimals of the original value (.944 for 94.4% => one decimal, or number of decimals - 2).

It should account for configured Int and Decimal display format from the widget configuration properties. Is it not doing so? If not please confirm and report a bug. Thanks

The decimal setting label reads "minimum decimals", so actually it's ok when it's showing 12 decimals (= more than "minimum"). Or is this supposed to set a fixed number of decimals?

It's supposed to be decimal places to display.

please help..
How can I modify something and build? I have run "ng build" or "ng build --configuration=production" but I am getting the same error. Kip is not showing anything, just WHITE PAGE./ I click to "inspect" and show this error:
main.js:1 Uncaught TypeError: Cannot read properties of undefined (reading 'production')
at 5974 (main.js:1:64836)
at n (runtime.js:1:128)
at main.js:1:64914
at f (runtime.js:1:698)
at main.js:1:49
Thanks!!! : )

Is it ok or is there a bug?

I'm trying to reproduce this problem but can't. I'm using self.tanks.fuel.0.currentLevel - also a ratio value but it works fine! Could it be that you get null values from the SK? You might see something in the in the console log that could help.

Looks like the problem is fixed in v2.2.2. For testing I manually set the path to the value of 0.944 in data fiddler and the widget shows 0.9 when decimals is set to 1, 0.94 when decimals is set to 2. Just like it should be. Great!


add length, pressure, temperature to ipc

Can we try to have the temperature unit in Celsius?

Now as µ˚C


Mouse interaction

Clicking a label or the center of a face should turn that side 90º
Dragging a sticker should also turn a face

closed in 72c027901a10a20af24aae2b1dc628e5922cb782


enable str objects to be converted to tensorflow

🚀 🚀 Pull Request
Checklist:

[x]  My code follows the style guidelines of this project and the Contributing document
[x]  I have commented my code, particularly in hard-to-understand areas
[ ]  I have kept the coverage-rate up
[x]  I have performed a self-review of my own code and resolved any problems
[x]  I have checked to ensure there aren't any other open Pull Requests for the same change
[ ]  I have described and made corresponding changes to the relevant documentation
[x]  New and existing unit tests pass locally with my changes

Changes

fixes issue #1308 , cannot convert text tensors using .tensorflow()

Thanks for the contribution!


[Feature Request] - Expose a property to retrieve configured image

It would be helpful (at least for me) to be able to get back configured/original image. I see imageView is private var and cannot get image without modifying your code.

Hi @sabiland, does the API .cropAndGetImage() do the job for you, or the delegate method didCropImage(image: UIImage, fromView: InteractiveImageView)?

I would like to retrieve exactly (without any modification, no cropping, nothing) the same image which was configured. That would be very practical (again, at least for me :)).
One extra question. I need to update image a lot. How should be done best way? Just calling configure again or?

Now I understand, I will try to push the improvement to retrieve the original image today. About your questions:

I can introduce a new method to change only the image of the image view, and not change other attributes.
You can use the first option to set the image to nil.

These requests seem useful also to me, do these changes resolve your challenges?

Yes, great, everything you said above resolve my challenges :).

Sorry if I ask too many questions, they are just popping to my mind now that I am testing your library.

IIVContentMode mode to be added to be able to show whole loaded picture

One more thing, I see you used ImageScrollView as reference? I have already long time ago locally modified ImageScrollView's method for device orientations, because otherwise there were issues when zooming/panning/etc.

isValidInterfaceOrientation check (this solved problems)
And I ignored that method on iPhones (but this is probably just for my case, I allow only Portrait mode in my App), maybe this could be configurable also?

@objc func changeOrientationNotification() {

        // SABI TWEAK
        // SABI TWEAK
        // SABI TWEAK
        if Helper.checkIPhone()
        {
            // Ignore ON IPHONE !!!
            // Ignore ON IPHONE !!!

                // A weird bug that frames are not update right after orientation changed. Need delay a little bit with async.
                DispatchQueue.main.async {
                    self.configureImageForSize(self.imageSize)
                    self.imageScrollViewDelegate?.imageScrollViewDidChangeOrientation(imageScrollView: self)
                }
            }
        }
    }

Point 3 requires more work and I will try to implement it next week. For the "changeOrientationNotification", I will have to investigate more.

Thx for your effort in advance.

New methods added in version 1.0.23:

interactiveImageView.getOriginalImage()
interactiveImageView.setImage(UIImage)

@IBOutlet weak var interactiveImageView: InteractiveImageView!

I have tested it.

Now whole image (toggle) presented works OK. Did you change anything about this?
Also getting/setting image works OK
But yes, there is an issue when tilting device and zooming/panning at the same time. Twitchy re-zooming-positionings happen. I think that solution above (valid orientations) should/could solve the problem.


Update lucia auth guidance

Description (required)
Update lucia to reflect it's change from a library to an auth resource.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
guides/authentication.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Sure, that sounds perfect


[12.x] chore: remove support for Carbon v2

Hello!
The addReal* methods no longer exist in Carbon v3, so this uses the add* equivalents.
Thanks!

addReal* was deprecated in Carbon 3.2 but should still work: https://github.com/briannesbitt/Carbon/commit/a6e64c765cefc7f42922db9cfa11b209cb67677c
It might be best to target master branch for v12 and remove support for Carbon 2

Ah, PHPStan was complaining about the methods not existing and I missed the dynamic addRealUnit method

Thanks @kylekatarnls


Fire submit action

RCTTextEventTypeSubmit was not being fired, so couldn't use onSubmitEditing.

This is a fairly straightforward change, would it be possible to get it merged in? Thanks.

Thank you, rebased and merged


Syndactyly, Type I (MONDO:0008512) incorrect exactmatch OMIM ID and ORPHA ID

Possible error in mapping?
Syndactyly, Type I (MONDO:0008512) currently has
exactmatch http://identifiers.org/omim/185900 and http://purl.obolibrary.org/obo/Orphanet_93402
The Orphanet term (ORPHA:93402) represents the parent term, Syndactyly type 1 or Zygodactyly – whereas the OMIM term (OMIM:185900) represents a daughter term, zygodactyly type 3 (ORPHA:295191).
Suggest OMIM:185900 should map to zygodactyly type 3 (MONDO:0017544) and not Syndactyly, Type I (MONDO:0008512).

pinging @lcdaugherty

@nicolevasilevsky Sorry, for delay responding to this, it looks like my notification settings were not set up right- I had changed them (muted) when I moved to my old job but now my new job requires me to look at ontologies again :) I have now updated my settings, so I should be a bit more responsive in future!
Looking at this again, I thought OMIM:185900 should map to MONDO:0017544 and not MONDO:0008512.
the reason I was querying it is that MeSH C566096 https://www.ncbi.nlm.nih.gov/mesh/?term=C566096 used to be mapped MONDO:0008512, but has since been removed so I was trying to figure out why.

Ok, is it okay if I close this ticket? Or is there action needed?
Thanks!


terminology: 1.3.0 -> 1.3.2

Motivation for this change
Update to version 1.3.2.
News:

https://www.enlightenment.org/news/2018-12-16-terminology-1.3.1
https://www.enlightenment.org/news/2018-12-18-terminology-1.3.2

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build enlightenment.terminology


Avoid draining the ArrayPool with undisposed JsonDocuments

My understanding is this code is likely going to be overhauled to not use JsonDocument at all, but in the meantime we can make a small tweak to significantly improve throughput. JsonDocument.Parse creates a JsonDocument backed by one or more ArrayPool arrays; those arrays are returned when the instance is disposed.  If it's never disposed, the arrays are never returned to the pool. That means that creating but not disposing lots of JsonDocument instances ends up incurring the cost (and contention) of searching the ArrayPool but ends up falling back to allocating anyway.
The problem is that JsonClaimSet isn't disposable and so the underlying JsonDocument is never disposed. As a patch until the code is overhauled, we can use JsonElement.Clone(), which creates a new JsonDocument not backed by ArrayPool arrays... we can then dispose of the original in order to return the arrays to the pool.
This still isn't ideal, as we're doing more allocating and copying than we'd like, but it ends up being much better than without.

#2259 2259


Task and launch config does not get updated for multi-root workspace

Repro steps:

File -> Add folder to workspace
Add an empty folder
Create a new project in that folder

Expected:
.vscode/tasks.json and .vscode/launch.json exist and enable user to F5/deploy
Actual:
Those files don't exist

I tried to repro this bug and couldn't. It worked each time for me. What is the status of this bug, specifically in relation to #1236?

Well I implemented a workaround in #1236, which is why you can't repro, but my workaround shouldn't be necessary. I believe there's a bug in VS Code, but at this point I don't think it's worth the time filing


SyscallResultTrait import missing in account

Hello,
I was looking at the Account implementation and I couldn't build the _execute_single_call function. Importing use starknet::SyscallResultTrait; seems to have fixed it for me. It might be because we are not using the same cairo version but I prefer to open an issue if this can help someone.

they @Th0rgal, could you share in what context the Account is not building? tests? deployment? are you building this entire repo or a subset of the modules?
sharing more detail of the setup might help us look into this. thanks!


Grid Outage UI Improvements

Move grid outage and related inputs to a separate page for ease of use.
UI text in Beta build is clear enough for now, but moving to separate page would be better.

Duplicate of #891


feat: add service management functionality to snap.py

Motivation

Add daemon service management functionality within snapctl alongside a Snap instance, keeping it contained with the popular Snap instance for charming

Main Changes

feat: add start, stop, logs, restart methods to Snap object (matching snapctl)

Methods can take a list,str or None services, evaluating to snap start lxd.activate lxd.user-daemon, snap start lxd.user-daemon or snap start lxd
These are called by Snap._snap_daemons() which (attempts) to ensure that a given snap instance can only launch services containing it's own _name

feat: add Snap.services, Snap.apps properties to get metadata from the API for installed snaps

Snap.apps looks like {"snap":"charmcraft","name":"charmcraft"},{"snap":"juju","name":"fetch-oci","daemon":"oneshot","daemon-scope":"system","enabled":true}, which could be useful for a user to know what commands are facilitated by the installed snap
Snap.services looks like below. I felt it gave a useful format for accessing snap daemons by key, with the snap field removed due to it being redundant for a named Snap instance, and filling information missing from the API (i.e active would not exist in the API response if not active){
  "fetch-oci": {
    "daemon": "oneshot",
    "daemon-scope": "system",
    "enabled": False,
    "active": False,
    "activators": [],
  },
}

Minor Changes

style: set API request return type to Any, because JSON is JSON
style: minor formatting changes
fix: update some old str.format usages that I don't think worked before

Merged and published the the store, if you run charmcraft fetch-lib you'll get the changes! :) Thanks!


Update to .Net 4.8

Update to .Net 4.8
Changed Ionic.Zip and Newtonsoft.Json to NuGet refrences
Updated .gitignore to VS version

I'm closing this PR because I've updated the project to .NET Core 3.1 in f058f9bdd14c7f2b21a2d1ba15047ee02500ab93


Integrate dokka

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #432
Type
Documentation
Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

Pull Request Test Coverage Report for Build #220

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

Totals

Change from base Build #218:
0.0%

Covered Lines:
1256

Relevant Lines:
1456

💛  - Coveralls

The current output is pointing to the app/build/dokka directory but we can redirect it to the desired location

The current output is pointing to the app/build/dokka directory but we can redirect it to the desired location

cool, can you change it to /docs/dokka for now please?
we might adjust again later on once we have a plan for publishing this maybe w/gh pages, idk tbd


AttributeError: type object 'Substance' has no attribute 'from_formula'

While trying to replicate the sample application at http://hera.physchem.kth.se/~chempy/branches/master/examples/aqueous_radiolysis.html I installed a fresh copy of Anaconda (linux, x64), installed chempy via
conda install -c https://conda.anaconda.org/bjodah chempy

and ran a slightly modified version of the radiolysis example code.
This failed with AttributeError: type object 'Substance' has no attribute 'from_formula' so I reduced the sample code to a minimal viable example (attached)
c2.zip
which is:
from chempy.chemistry import Substance

_species = ['H', 'H+', 'H2', 'e-(aq)', 'HO2-', 'HO2', 'H2O2', 'HO3', 'O-', 'O2', 'O2-', 'O3', 'O3-', 'OH', 'OH-', 'H2O']

species = [Substance.from_formula(s) for s in _species]

Strangely, the later code which builds the reactions list of Reaction() objects from the _reactions dict seems to work, indicating that there is not a generic problem resolving method names in the chempy.chemistry namespace. The problem seems isolated to chempy.chemistry.Substance
Also, I was able to get this part of the sample code to work successfully using my distribution's stock python installation, though the sample code would fail later for other reasons which is why I installed Anaconda.

So this is a false alarm regarding the code; the problem is in the installation or documentation:
$ ls -l /opt/apthorpe/anaconda2/lib/python2.7/site-packages/chempy/chemistry.py /usr/local/lib/python2.7/dist-packages/chempy/chemistry.py    [15:03:56]
-rw-rw-r-- 2 apthorpe apthorpe 19806 Jan 26 09:05 /opt/apthorpe/anaconda2/lib/python2.7/site-packages/chempy/chemistry.py
-rw-r--r-- 1 root     staff    21214 Feb 13 16:22 /usr/local/lib/python2.7/dist-packages/chempy/chemistry.py

The package installed by conda is obsolete (see https://anaconda.org/bjodah/chempy ).
What is the proper method for installing chempy?
I'll defer trying to explain the problems I've had installing from source (involves incompatibilities with recent versions of numpy and pyodeint) until after trying the recommended installation method.

@apthorpe thank you for reporting this. I simply hadn't pushed the build of the latest release yet, it should be there now. The build of the release is essentially automatic but I havn't automated the push of the build files yet.
I'm about to update the installation docs here:
https://github.com/bjodah/chempy/pull/17/files
(never mind the python -m pytest --pyargs chempy part, pytest from conda with Py2.7 seem to have a hard time finding installed packages for some reason).
Let me know if it works correctly for you now (I tried on a fresh miniconda2 install on a 64-bit Ubuntu 14.04 box).
I'm also interested in feedback in general and curious about your application in mind.

I just installed from the most recent source to anaconda and it worked with no problems. I'm still trying to verify the results look reasonable but there are no obvious problems.
chempy is a godsend for the problem I'm currently fighting with. I'm working on replicating results from several papers which look at radiolysis with variations in pH and radiation exposure so I was extremely interested in getting the sample aqueous radiolysis case working. I have been using the CHEMEQ2 solver http://www.dtic.mil/cgi-bin/GetTRDoc?AD=ADA392490 to estimate steady-state concentration of hydrogen gas due to radiolytic generation. As you can imagine, it's a stiff problem so while setting up the problem is straightforward, getting reasonable results can be difficult, requiring a lot of code tuning and experimentation.
If there's anything I can do to help with testing, documentation, etc, please let me know. I really appreciate that you've made this code available.

Also, I'm very curious where you got the reaction set for the radiolysis example; we're using the reactions listed in https://www.researchgate.net/publication/243349685_Model_for_radiolysis_of_water_and_aqueous_solutions_of_H-2_H2O2_and_O-2 which have some overlap with https://www.researchgate.net/publication/257432707_A_combined_experimental_and_model_analysis_on_the_effect_of_pH_and_O2aq_on_g-radiolytically_produced_H2_and_H2O2 On reviewing a few papers on the subject, there seems to be no rationale for which reactions are considered significant for modeling radiolysis. So far we've had inconsistent success replicating some of the published results and it's not clear if that's due to deficiencies in our software or in our choice of reaction set.
This isn't directly related to the initial bug report but these papers provide benchmark data for testing and validating chempy and may be of use to you.

No worries about writing in this issue - it's a place as good as any (but it is open to the whole of internet so in case you want to contact me directly you are welcome to do so on my gmail address: bjodah).
I am glad to hear that you find ChemPy useful, especially since (or because of) the fact
that I've developed it as an underlying tool for studying chemical problems in radiation chemistry
(I felt the need for an abstraction layer that was easy to manipulate). Feel free to suggest improvements, I'll happily review suggested code inclusions too if you modify it in any way you think is useful for others.
The project is still quite young so things may change quite fast, but I try hard to deprecate things properly with at least on intermediate version before introducing any breaking changes.
I was actually not aware of CHEMEQ2, so thank you for the link. Is the program freely available? (it would be interesting to compare results). There seem to be quite a few codes around for modelling the deterministic "chemical stage" of radiolysis: FACSIMILE, Maksima chemist, supposedly a code named "Bora" (which I haven't been able to hunt down) and probably a few more. And then we have not even touched upon the early stages which involve Monte Carlo simulations of spur reactions.
Currently I am pursuing a PhD in chemistry at KTH, and my radiolysis modelling efforts are done under Prof. Mats Jonsson's supervision. The reaction set in that notebook was actually generated from another Python library that I am writing which aims to collect models of (aqueous) radiolysis, and in this case the data is from AECL report 153-127160-450 evaluated at 20 °C (with some minor changes of mine to make the radiolytic yields respect mass conservation). I intend to make the library collecting the models open source under the same permissive BSD-license within a couple of months in association with an article describing that and some modelling efforts of mine (including contributions from diffusion in systems with inhomogeneous radiation fields and/or surface reactions), and put it up here on github for easy of reuse, collaboration and collective validation.
I think that there is consensus around the core set of reactions in aqueous radiolysis, but yields and rate constants (especially at other than neutral pH values and temperatures far from room temperature) are subject of continuous review, many researchers in the field frequent the
Miller Conference on Radiation Chemistry series and the Gordon conferences on Radiation Chemistry (which last time I heard, is to be replaced by another conference at BNL later this year).
Just out of curiousity: in what setting are you working on this project, academia or industry?


setupAutomaticSilentRefresh in passwordFlow

Hi guys,
i have angular project and using OauthService and i have question.
It is possible use setupAutomaticSilentRefresh() in passwordflow,if it possible please show me expmple.
i tried someting like this, but is not working
    authConfig.issuer = this.uaaUrl +"/oauth/token";
    this.oauthService.configure(authConfig);

  }

  login() {
    this.configure();
    let httpHeaders = new HttpHeaders({
      'Content-Type': "application/x-www-form-urlencoded"
    })
    this.oauthService.loadDiscoveryDocument().then(()=>{
      this.oauthService
      .fetchTokenUsingPasswordFlow(
        this.username,
        this.password,
        httpHeaders
      )
      .then(() => {
        console.debug('successfully logged in');
        this.loginFailed = false;
        this.router.navigate(["/dashboard"]);
        this.oauthService.setupAutomaticSilentRefresh();
      })
      .catch(err => {
        console.error('error logging in', err);
        this.loginFailed = true;
      });
    })
  }

Thank you.

Quick question: are you configuring the library to ask for offline_access (refresh tokens)? As far as I know that is the only way a "silent" refresh would work with Password Flow.
Another thing you could try is calling setupAutomaticSilentRefresh before requesting a token, if I look at the source code it seems it listens for token_received to trigger refresh intervals. For other flows I also set it up beforehand.
Note that Password flow has been deprecated and is no longer recommended for quite some time already, there's #898 advocating removal of the feature from this library.

Thank you for your response, i will try other way (implicit or code flow).


Apps & Marketplace // Chart Repositories // Get "https://helm.gravitee.io/index.yaml": x509: certificate signed by unknown authority

Hello
I have this error in :
Rancher UI : Apps & Marketplace -> Chart Repositories -> Create New chart
Get "https://xxxxxxxxxx/index.yaml": x509: certificate signed by unknown authority

I tried to add the ca certificate in all the containers with no result!
Thanks for your help.

I fixed the problem with modifying the yaml chat [kind: ClusterRepo]
The rancher documentation doesn't talk about this! I had to read the source code!

In the source code we find the keys to the spec. look at this page : https://github.com/rancher/rancher/blob/master/pkg/apis/catalog.cattle.io/v1/types.go
// CABundle is a PEM encoded CA bundle which will be used to validate the repo's certificate.
// If unspecified, system trust roots will be used.
CABundle []byte json:"caBundle,omitempty"
// InsecureSkipTLSverify will use insecure HTTPS to download the repo's index.
InsecureSkipTLSverify bool json:"insecureSkipTLSVerify,omitempty"

I hope this will help you
cordially


Failed to compile

Hi, im getting this error after install vuesax via npm.
Failed to compile.
./node_modules/vuesax/dist/fonts/MaterialIcons-Regular.ttf Module parse failed: Unexpected character '' (1:0) You may need an appropriate loader to handle this file type. (Source code omitted for this binary file)  @ ./node_modules/css-loader!./node_modules/vuesax/dist/vuesax.css 7:13584-13628  @ ./node_modules/vuesax/dist/vuesax.css  @ ./src/main.js  @ multi (webpack)-dev-server/client?http://localhost:8080 webpack/hot/dev-server ./src/main.js

Thanks for your support.
OS: Mac os High Sierra
Node version: 8.11.1
vuesax version: 3.0.5
Browser: Google Chrome version 66.0.3359.117

You can provide a reproduction repository. That eliminates the riddling work and saves us time.


Implement API v2 endpoints to complete the support for filtering

The new API v2 has only implemented the (new) resource_types endpoint, but we lack the followin:

categories
keywords
regions
owners

The new home page is still using the legacy API but we want to stop relying on it, both for the new frontend and the new QGIS plugin.
@ricardogsilva and @allyoucanmap I invite you to add comments if something more is missing and if there are more specific requirements.

@marthamareal you can see the calls when you open the filters panel in the new home page

@giohappy  i have assigned points to this.

@marthamareal your estimates look pretty high... The implementation should be quite automatic. I'll move it to Ready but I hope it will take less time.

@giohappy  @afabiani  Should we create these apis under resources api eg /resources/owners, resources/categories such that the results returned are ones where we have some resources? or we create them independent of the resources eg api/v2/categories , api/v2/owners(should this have only users who have ever created a resource)

Good point @marthamareal.
Given that these concepts are strictly connected to resources I would keep them under /resources.
What do you think @afabiani

@giohappy should we have pagination on these endpoints? also, are the following fields enough in the response? Or is should we change their formats to have a list of values? Should i also include thesaurus_keywords ?
1 resources/owners
[
    {
        "pk": 1008,
        "username": "kolly",
        "first_name": "",
        "last_name": "",
        "avatar": "https://www.gravatar.com/avatar/419998901a83c565ae7ae4b2ea8a6bc1/?s=240",
        "perms": [
            "add_resource"
        ]
    }
]

/resources/categories

[
   {
       "identifier": "biota"
   }
]

resources/keywords

[
   {
       "name": "AC04078710_georef_final",
       "slug": "wcs"
   }
 ]

resources/regions

[
    {
        "code": "GLO",
        "name": "Global"
    }
 ]

First thing @marthamareal as agreed offline we will use root endpoints instead of subpaths of the /resources endpoints.
I invited you to give a look to the current (legacy API) endpoints to see the responses they provide.
One is particularly important: count. This property returns the number of resources connected to the specific category, owner, region, etc. and it is used by the faceted search / filters.
owners should return the same response as API v2 users/. AFAICS /users currently returns only the logged in users for normal users. However /owners should return the full list ownes also for non-admins.
We need pagination
Let's keep them read-only for the moment (only GET), since their management must be discussed a bit further, in particular for keywords which is quite a convoluted concept.


580 linking entry complexity note

https://github.com/uwlib-cams/MARC2RDA/blob/main/Working Documents/5XX.csv

@CECSpecialistI @pennylenger
It looks like subfield $5 was added in June 2024. Should we add it to the mapping?

I don't think it is necessary.


Stat transfer changes

Stat transfers under 10 are free, however there is a 24 hour cool down
AK cost dynamically changes if below 10.
Fixed bug where default skill would be ninjutsu selected if above rank 2.
Default rank 1 skill is ninjutsu.

.

will have to add the column "last_free_stat_change" after "last_ai" in the users table


Add Sample Application.yml to the Getting Started Wiki Page

Currently in the getting started it says to download the .war and the application.yml but nowhere that i could see was the application.yml linked. I just feel that it should be added for easy startup

download the .war and application.yml
Download the latest war from the releases page... Suggestion: use /data/streama for the yml and the war and /data/streama/files for the uploaded files.

Where i found it after some looking:
https://github.com/dularion/streama/blob/master/docs/sample_application.yml

Agreed. Thanks for pointing that out. I've updated the section.


Should we include data from journal supplementary files?

@sckott mentioned this in https://github.com/datacite/freya/issues/2
Pros:

A lot of data is stored this way.  It greatly expands the range of data that would be made available to users and the degree to which the package could improve data linkage

Cons:

We may want to discourage using supplementary data in journals to store data. OTOH we really don't have much influence through this tool.
It would be a lot harder to do this client-side than it would be for data repositories. Repositories are limited in number, so client-side mapping of DOI to resource would only require so much custom coding.  For most of the we can identify the repository, and thus the mapping, from the DOI.  There are many more journals, and journals themselves aren't the relevant unit - we need to understand how DOI --> file URL maps for each publisher's platform.

Hello folks!
I've got a comment about the weirdness of publishers who use "Figshare for publishers" like PLOS ONE.
Take this article for instance: https://doi.org/10.1371/journal.pone.0198684

Query the Figshare API for the collection ID: (4126502)

https://api.figshare.com/v2/collections?doi=10.1371%2Fjournal.pone.0198684

Return all assets from the collection:

https://api.figshare.com/v2/collections/4126502
These assets include the actual paper itself, and all figures and tables included in the paper. This is tremendously useful!
BUT
This does not return the "supporting information" file https://doi.org/10.1371/journal.pone.0198684.s001
Summary
As a user of the doidata package, I would appreciate a method for accessing ALL of these assets from a paper when the publisher uses Figshare behind the scenes.

@martinjhnhadley Do you know the package suppdata?
The suggestion by @sckott (https://github.com/ropenscilabs/doidata/issues/2#issuecomment-355088644) is realised in that package, i.e. the DOI-based download from the fullext package is it's own package now: https://github.com/ropensci/suppdata
We're planning to have a hackathon as part of the Mozilla Global Sprint (https://github.com/ropensci/suppdata/issues/35) around the suppdata package. Maybe that is a good occasion to revive doidata ?

Thanks @nuest! I wasn't aware of the suppdata package, it looks like this will definitely solve some of my requirements.
I'm not sure if my skills are developed enough to usefully participate in the devel of doidata, but I'll register and give it a go.


Add trained model weights

I think it will be really helpful if you guys added trained models for some commonly used agents and environments so that people can use it for further development.

Thanks for the feedback! We plan to release the trained models as well as the benchmark results https://github.com/chainer/chainerrl/pull/414


[tab] loading event executes before previously active tabs have been hidden

When switching tabs, previously active tabs stay visible until the content of the selected tab has finished loading.
Tabs: A, B y C.

User clicks on tab A
Content of tab A is dynamically loaded (ajax)
User clicks on tab B
Content of tab B starts loading (ajax)
Content of tab A stays visible until step 4 is finished

Expected:
5. Tab A should be hidden right after step 3

I'll take a look

Thanks for the nice bug report. This will be fixed in next.


Changed pom.xml

@rultor please merge

@rultor please merge

@avatar46 OK, I'll try to merge now. You can check the progress of the merge here

@rultor please merge

@avatar46 Done! FYI, the full log is here (took me 1min)


Compile all on checkjdk11

Flag things like https://github.com/square/okhttp/pull/7600 earlier.

@swankjesse request post review


Switch to deep_preloader

The whole preload_dependencies / reduce_association stack is still a bit hairy. We talked a bit about simplifying it. Did you get anywhere with that?

Do we have a style guide for optional brackets? I see lots of
DeepPreloader::PolymorphicSpec.new

and lots of
DeepPreloader::PolymorphicSpec.new()

Minor comments only, LGTM.

This should fix #1.


fix: correct typo in README.md

Fixes typo in the README.md file.

[x] I have followed Sample Format Guide
[ ] pom.xml parent set to latest shared-configuration
[x] Appropriate changes to README are included in PR
[ ] API's need to be enabled to test (tell us)
[ ] Environment Variables need to be set (ask us to set them)
[ ] Tests pass:   mvn clean verify required
[ ] Lint  passes: mvn -P lint checkstyle:check required
[ ] Static Analysis:  mvn -P lint clean compile pmd:cpd-check spotbugs:check advisory only
[x] Please merge this PR for me once it is approved.

@googlebot I signed it!


Hook fails with PermissionError for not accessible files

Hi,
when you create a directory in your repository, with permissions e.g. chmod 000, the extension fails with the following error:
Traceback (most recent call last):
  File "/home/pavel/.cache/pre-commit/repoxamyetpl/py_env-python3.10/bin/detect-missing-init", line 8, in <module>
    sys.exit(main())
  File "/home/pavel/.cache/pre-commit/repoxamyetpl/py_env-python3.10/lib/python3.10/site-packages/hook/detect_missing_init.py", line 136, in main
    missing_init_files = find_missing_init_files(
  File "/home/pavel/.cache/pre-commit/repoxamyetpl/py_env-python3.10/lib/python3.10/site-packages/hook/detect_missing_init.py", line 73, in find_missing_init_files
    contains_python_file(folder),
  File "/home/pavel/.cache/pre-commit/repoxamyetpl/py_env-python3.10/lib/python3.10/site-packages/hook/detect_missing_init.py", line 58, in contains_python_file
    if file.is_dir() and contains_python_file(file):
  File "/home/pavel/.cache/pre-commit/repoxamyetpl/py_env-python3.10/lib/python3.10/site-packages/hook/detect_missing_init.py", line 55, in contains_python_file
    for file in folder.iterdir():
  File "/home/pavel/.pyenv/versions/3.10.2/lib/python3.10/pathlib.py", line 1015, in iterdir
    for name in self._accessor.listdir(self):
PermissionError: [Errno 13] Permission denied: '.tmp/docker'

The problem could be resolved on the following lines (by skipping those directories/files which are inaccessible)
https://github.com/lk16/detect-missing-init/blob/master/hook/detect_missing_init.py#L55-L59
But I'm not sure what is the best solution.
BTW this error occurs even though we have specified --python-folders which should exclude this inaccessible directory. So it really doesn't make sense it is failing.

Hi,
The hook should only be searching through folders that are whitelisted with the --python-folders flag when it's used. It currently checks all folders, which is an implementation flaw, leading to this problem. I will fix it.

Release https://github.com/lk16/detect-missing-init/releases/tag/v0.1.6 fixes this issue.
Please reopen this issue if you still have problems after updating.


Adding template and configuration item for JIRA Datacenter

Setting up a node for a JIRA datacenter requires this additional file to be created and specification of a shared location.

@ronaldvanrij this could use some documentation, perhaps some tests, too!

Added tests and updated README.md

@ronaldvanrij this will need a rebase now

@igalic rebase completed

thank you @ronaldvanrij!


updated cryptol (2.4.0)

Changes to a cask
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Merged as e76585c1a5b489992fe80007c31aacb24bf58298.


fix bugs about ModelBO's member variable calling

Closes #

What has been done to verify that this works as intended?
The main purpose of this pull request is to show my contribution in the last 4 days in detail.
I've tried to run the newly added "playground_convert.py" to test the profiler module. Between line 10  and the line 18 is the first part of the code, where I convert pytorch model into torchscript model which the frame can actually serve with. Before running this part, I've changed the weight address in "./example/resnet50.yml" to the address of the post-converting model so that I can use the register_model_from_yaml API to register the post-converting model.
After that, I've tried to run the second part of the code (from line 19). Through this process I've found that in the manager.py as well as *client.py, the programers try to call the member variable 'name' of ModelBO, which doesn't exist. I've fixed these bugs, and finally I get stuck at grpc errors.
Why is this the best possible solution? Were any other approaches considered?
...
How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?
It has no affects to users.
Does this change require updates to documentation?
No.
Before submitting this PR, please make sure you have:

[] run python -m pytest tests/ and confirmed all checks still pass.
[] verified that any code or assets from external sources are properly credited.

@YuanmingLeee Is this ok?


filter out cluster namespaces for getUserAccessCredentials()

Signed-off-by: Yu Cao<EMAIL_ADDRESS>https://github.com/open-cluster-management/backlog/issues/12992

add userNamespaces and clusterNamespaces to req.user and make it available in kubeconnector
only check user permission in userNamespaces instead of all namespaces

clusterNamespaces are reserved for GRC to create replicated policies and user should not create root policies. All other namespaces are user namespaces where user can create root policies.


Add idle timeout, fix read timeout, and improve error handling

Should fix #79, fix #55

Test for this is not easy...
We need to change MySQL / MariaDB configuration file before the test.
Nobody want the test to run for several hours. 😂

These test failures are caused by (possible) Deno bug (https://github.com/denoland/deno/issues/7844).

awesome!

@manyuanrong Thanks!
I am wondering why this PR is merged as a single commit. These commit messages were lost. :see_no_evil:


Cypress: navigating through app pages without error

Description

Enhancement: Cypress Test Suite for Navigating Through App Pages.

Related Issue
Issue #674

@aakrem All done


Email Settings Blank in BotFramework Portal

Bot Info

SDK Platform:  Node.js
Active Channels: Skype
Deployment Environment: Azure Bot Service - portal.azure.com

Issue Description
Cant change the settings endpoint, error received:
{"readyState":4,"responseText":"{"error":{"code":"InvalidBotData","message":"OwnerIds: OwnerIds must contain at least one email address. "}}","responseJSON":{"error":{"code":"InvalidBotData","message":"OwnerIds: OwnerIds must contain at least one email address. "}},"status":400,"statusText":"Bad Request"}
Reproduction Steps

Login to - https://portal.azure.com
Go to the resource - bot
Settings tab
Change Configuration Messaging endpoint
Save

Expected Behavior
Messaging endpoint save successfully.
Actual Results
Nothing happened.
Bot handle is "AstridBot"
appId is c77c3dd6-7278-4f4b-92fc-f3642cb7f29c

[Posted in response to @mintxelas.]
@mintxelas, please confirm you are the bot creator and that other owners were not removed via GDPR requests.

Yes, I am the creator. What are GDPR requests?

"General Data Protection Regulation" request to remove PII such as email. This doesn't sound applicable; correct?

No, not at all. I've tried adding new owner with another account but it
still gives the same error. :(
El jue., 29 nov. 2018 22:55, Cory Healey<EMAIL_ADDRESS>escribió:

"General Data Protection Regulation" request to remove PII such as email.
This doesn't sound applicable; correct?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/5190#issuecomment-443007891,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAzk-_kelpGbStuDhkShFYwn9CwnOJOvks5u0FfagaJpZM4Y6jKA
.

OK, thanks. We'll correct it and get back to you shortly.

Thanks a lot!
El jue., 29 nov. 2018 23:18, Cory Healey<EMAIL_ADDRESS>escribió:

OK, thanks. We'll correct it and get back to you shortly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/5190#issuecomment-443014876,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAzk-__cU4A_MD4EKGHhWcttNQ5GjQsxks5u0F1CgaJpZM4Y6jKA
.


updated package with apiVersion fix

Required items, please complete
Change(s):

updated package mainTemplate.json with apiVersion fix.  was fixed in prior PR but did not update package zip file so failed marketplace automations.  updated package version from 2.0.0 to 2.0.1

Reason for Change(s):

arm-ttk fail during marketplace validations for certification

Version Updated:

2.0.0 updated to 2.0.1

Testing Completed:

re-ran software validation and passed

Checked that the validations are passing and have addressed any issues that are present:

yes

Hi @daspiker, please include this file Solutions\Island\Data\Solution_Island.json in this PR and in 2.0.1 zip there is a folder _MACOSX, please remove this folder

hey @v-rbajaj not sure what the workflow errors are.  looking at the file changes i don't see anything that would cause these.  let me know if you need me to do anything on my side to kick off the workflow again.

Hi @daspiker, sorry for delay in response, I see you have removed 2.0.0 zip, please discard that change as we do need the older version of the package, thanks!

@v-rbajaj 2.0.0.zip package added back in per your request

@v-rbajaj i re-ran the packaging with createSolutionV2.ps1, fixed the apiVersion issue and successfully tested with Test-AzTemplate.  I also loaded the new Solution_Island.json file i'm using for packaging to the Island/Data/Data folder.


Don't merge! Regression PR perf testing without metadata validation

Docs Build status updates of commit d6ca436:
:clock10: Pending: waiting for processors (17 builds ahead of you)
:warning: Docs Build is busy, currently there are 17 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit d6ca436:
:clock10: Full build: the average full build time is 5 min(s) 57 sec(s), this is based on the last 664 build(s) for this repository.

Docs Build status updates of commit d6ca436:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

docs-ref-conceptual/TestingPullRequest.md
:warning:Warning
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)
Details

docs-ref-conceptual/TestingPullRequest.md

[Warning] Metadata with following name(s) are not allowed to be set in Yaml header, or as file level metadata in docfx.json, or as global metadata in docfx.json: content_type. They are generated by Docs platform, so the values set in these 3 places will be ignored. Please remove them from all 3 places to resolve the warning.
[Warning] Metadata with following name(s) are not allowed to be set in Yaml header, or as file level metadata in docfx.json, or as global metadata in docfx.json: content_type. They are generated by Docs platform, so the values set in these 3 places will be ignored. Please remove them from all 3 places to resolve the warning.
[Warning] Metadata with following name(s) are not allowed to be set in Yaml header, or as file level metadata in docfx.json, or as global metadata in docfx.json: content_type. They are generated by Docs platform, so the values set in these 3 places will be ignored. Please remove them from all 3 places to resolve the warning.
[Warning] the value invalid_type is undefined for metadata content_type
[Warning] the value invalid_type is undefined for metadata content_type
[Warning] the value invalid_type is undefined for metadata content_type
Line 7: [Warning] Invalid file link:(~/nonexisted1.md).
Line 7: [Warning] Invalid file link:(~/nonexisted1.md).
Line 7: [Warning] Invalid file link:(~/nonexisted1.md).
Line 8: [Warning] Invalid file link:(~/nonexisted2.md).
Line 8: [Warning] Invalid file link:(~/nonexisted2.md).
Line 8: [Warning] Invalid file link:(~/nonexisted2.md).
Line 9: [Warning] Invalid file link:(~/nonexisted3.md).
Line 9: [Warning] Invalid file link:(~/nonexisted3.md).
Line 9: [Warning] Invalid file link:(~/nonexisted3.md).

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


Add test for fill() with every built-in type

Add a test for the fill() algorithm with every built-in type (e.g. int_, float_, uint4_). This will provide good code coverage for the fill() algorithm and for the built-in type wrappers. Use the BOOST_AUTO_TEST_CASE_TEMPLATE macro to repeat the same test code with every type.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/3622096-add-test-for-fill-with-every-built-in-type?utm_campaign=plugin&utm_content=tracker%2F402515&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F402515&utm_medium=issues&utm_source=github).

It can be closed, see https://github.com/boostorg/compute/issues/461.

Thanks!


Issue "InvalidStateError" in Firefox Browser

Hi!
I'm using Heatmap.js and it's Leaflet plugin. I really love your work! Unfortunately I am getting an error when I zoom out: "InvalidStateError: An attempt was made to use an object that is not, or is no longer, usable".
This error happens only in Firefox web browser. In chrome it's working flawlessly. I am using your sample app.
Here is the code of this app:
var map;
var heatmapLayer;

$(document).ready(function() {
	initHeatMap();
});

var cfg = {
	"radius": 0.003,
	"maxOpacity": .8,
	"scaleRadius": true,
	"useLocalExtrema": false,
	latField: 'lat',
        lngField: 'lon',
	valueField: 'count'
};

function initHeatMap() {
	heatmapLayer = new HeatmapOverlay(cfg);

	var OpenStreetMap_DE = L.tileLayer('http://{s}.tile.openstreetmap.de/tiles/osmde/{z}/{x}/{y}.png', {
	        maxZoom: 18,
		attribution: '© OpenStreetMap'
	});

	map = L.map('map', {
		center: new L.LatLng(25.6586, -80.3568),
		zoom: 4,
		layers: [OpenStreetMap_DE, heatmapLayer]
	});
	map.setView([47.714535, 10.314098], 12);

	getHeatData();
}

function getHeatData() {
        var testData = {
              max: 8,
              data: [{lat: 24.6408, lon:46.7728, count: 3},{lat: 50.75, lon:-1.55, count: 1}]
        };
	heatmapLayer.setData(testData);
}

The error points to this line (425) in Heatmap.js file:
shadowCtx.drawImage(tpl, rectX, rectY);

I hope this helps!
Cheers,
Robert

I have this same issue although it only happens when I have more than 10000 data points (which shouldn't be a problem).
I also get it in chrome though so our issues might be unrelated.

I've been looking into this for about half an hour now and I've been able to narrow down why an IllegalStateError is being thrown.
        var tpl;
        if (!this._templates[radius]) {
          this._templates[radius] = tpl = _getPointTemplate(radius, blur);
        } else {
          tpl = this._templates[radius];
        }
        // value from minimum / value range
        // => [0, 1]
        shadowCtx.globalAlpha = (value-min)/(max-min);

        shadowCtx.drawImage(tpl, rectX, rectY);

In the above code, the exception is supposed to be thrown if "tpl" is ever null or undefined.  In my project, a null "tpl" is used when rectX is -333.093344 and rectY is -193.093344.  In all of my other projects, I have never had negative rectXs or rectYs.  It might be useful to note that there is a possibility of negative radius values being parsed as array indices since no checking is done on them.

Hey @bk5115545,
did you manage to get a workaround to avoid this error? In my case Firefox crashes if will reach a datapoint where tpl = "". That probably means canvas == null.  This happens in my case when rectX = 505.808, rectY = 276.808. Chrome does not crash at this point, it contiues with canvas == null until the last point.

I've tracked the issue to be running out of memory actually.  The piece of code I linked caches the blur map for a specific radius, which for my code is rather inconvenient since I have almost 15k unique radii that need to have very correct decimal precision.
If you can decrease the memory usage, then "tpl" should always be valid (provided something else isn't the issue).  I use a development system with 32GB of memory and an Nvidia GTX 970.  The _getPointTemplate function returns invalid "tpl" when my graphics memory usage is higher than about 90%.  My code works slowly with about 1000 unique radii but it crashes with 10k.
If the code above is your entire application, then you might be using a graphics card with very little memory (Integrated/Embedded/Virtual).  Otherwise, just try to reduce the number of unique radii.
I have moved to using https://github.com/pyalot/webgl-heatmap which is considerably faster (60FPS with 40k points) even though I had to write my own leaflet integration and some smaller features.

Thanks for your quick response. I think we have different problems. I just found out what my problem is. Because of my cfg some of my radius-values gets extremly small (0.193 for example). The _getPointTemplate function creates a tmpCanvas and sets its height and width to 2*radius. So in my case its sets the width and height of the canves to 0.386 and since the html canvas can only take int values its 0.0. I managed to get rid of the error doing the following: In case of 0 width or height i set it to 1:
var _getPointTemplate = function(radius, blurFactor) {
    var tplCanvas = document.createElement('canvas');
    var tplCtx = tplCanvas.getContext('2d');
    var x = radius;
    var y = radius;
    var tmpRadius = radius * 2;
    if(tmpRadius < 1.0 ) {
      tmpRadius = 1.0;
    }
    tplCanvas.width = tplCanvas.height = tmpRadius;

    if (blurFactor == 1) {
      tplCtx.beginPath();
      tplCtx.arc(x, y, radius, 0, 2 * Math.PI, false);
      tplCtx.fillStyle = 'rgba(0,0,0,1)';
      tplCtx.fill();
    } else {
      var gradient = tplCtx.createRadialGradient(x, y, radius*blurFactor, x, y, radius);
      gradient.addColorStop(0, 'rgba(0,0,0,1)');
      gradient.addColorStop(1, 'rgba(0,0,0,0)');
      tplCtx.fillStyle = gradient;
      tplCtx.fillRect(0, 0, 2*radius, 2*radius);
    }
    return tplCanvas;
  };

Now it works for me.
Rob


Rework windows build

According to upstream https://github.com/lballabio/QuantLib/issues/1552 build_shared_libs is not supported on Windows, so build a static QuantLib library but use the dynamic runtime so the library can be used with python.
Furthermore use an out-of-source build directory as done in the upstream documentation https://www.quantlib.org/install/cmake.shtml this results in a correctly generated quantlib.hpp file so there is no need to replace it in the quantlib-python recipe.
Furthermore I enabled building the test suite on Windows and running the tests in the test step similar to on Unix.
I also used ninja as the build tool on Windows because it was significantly faster on my local machine and allowed faster testing.

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

@conda-forge-admin, please restart ci

@timkpaine do you have time to look at this? If this is merged I would like to make sure my changes for building quantlib-python on windows also work on the conda-forge infrastructure.

@adament it will take a few hours for the build to propagate to prod but then you can pickup the PR on the other repo

Thank you!


Update the two qubit test

What
Cleanup the integration test for the two qubit device.
Why
It was a mess.
How
Moved setup code to proper pytest fixtures, including parametric testing for all search algorithms (currently limited to verified ones)
Remarks
This should be a simpler, more reliable test that includes fewer moving parts. The algorithm library should be cleaned and included in the parametric testing.
Checklist
Please include and complete the following checklist. Your Pull Request is (in most cases) not ready for review until the following have been completed. You can create a draft PR while you are still completing the checklist. Check the Contribution Guidelines for more details. You can mark an item as complete with the - [x] prefix

[x] Tests - Added unit tests for new code, regression tests for bugs and updated the integration tests if required
[x] Formatting & Linting - black and flake8 have been used to ensure styling guidelines are met
[x] Type Annotations - All new code has been type annotated in the function signatures using type hints
[ ] Docstrings - Docstrings have been provided for functions in the numpydoc style
[x] Documentation - The tutorial style documentation has been updated to explain changes & new features
[x] Notebooks - Example notebooks have been updated to incorporate changes and new features
[ ] Changelog - A short note on this PR has been added to the Upcoming Release section

This should be a simpler, more reliable test

What do you have in mind when you say more reliable? Do you mean this test fails when something isn't wrong or that the pass/fail of the test is not reproducible? If that is the case, then we definitely should look deeper into it. If you mean it is difficult to debug this test, because it seems like this is the only test that fails while other tests pass, that in most cases is indicative of the fact that we do not have the necessary unit tests covering the buggy section of the code and not of any inherent issue with this integration test.

that includes fewer moving parts

I am not entirely sure if we want this test to have fewer moving parts. As an integration test, this is expected to interface as many moving parts as possible to ensure the whole big machinery works as expected in a typical workflow. The example notebooks also provide a similar level of integration testing but it is usually not part of the development workflow.

a21ed101af4aeb668ea5c2435b26d5f7f2f8e84a - I am not sure if this is the right approach. The decorator is a perfectly valid way to create the library of available algorithms. Checks and balances to ensure that people don't use algorithms that are not tested could be put in place through Warnings and Errors.
I would rather use the pytest fixture to decide which algorithms need/are ready to be tested, and make it explicit in the tests by defining the algorithm dictionary inside the fixture, rather than excluding algorithms from being added to the library with the decorator. This is because the algorithms library contains algorithms that are not all meant for Optimal Control workflows (eg sweep, adaptive_scan etc). This is the reason behind the failing test_sensitivity and test/test_optim_init.py::test_main_sens.